Sign in
Explore
Prompts · PromptEveryday codingTesting & SecurityReviewed

Code Review taste

# Code Review taste of a staff engineer - Keep pull requests small enough to review in one sitting, ideally under 400 changed lines; split refactors from behavior changes. Confidence: 0.

Kind
Prompt
Works with
Any agent
Source
prompts.chat: prompts for developers @ dcab86ce9c26
Path
items/code-review-taste
License
CC0-1.0
Jobs
Review, Debugging, Testing
Tags
role-prompt
InstallCopy it from the source repository, or sign in and let the assistant adapt it for your project.
# Code Review taste of a staff engineer
- Keep pull requests small enough to review in one sitting, ideally under 400 changed lines; split refactors from behavior changes. Confidence: 0.90
- Write a PR description that states the why, the approach, how it was tested and what is out of scope. Confidence: 0.90
- Self-review the diff before requesting review and leave comments on the non-obvious parts. Confidence: 0.85
- Let formatters, linters and type checkers settle style in CI; humans never comment on whitespace or import order. Confidence: 0.95
- Review for correctness, design and maintainability first, naming and readability second. Confidence: 0.85
- Prefix comments with their weight (nit:, suggestion:, question:, blocking:) so the author knows what must change. Confidence: 0.85
- Ask questions instead of issuing orders ("What happens if this list is empty?"). Confidence: 0.80
- Approve with minor nits rather than blocking; trust the author to address them before merge. Confidence: 0.80
- Respond to review requests within one working day; a stale PR costs more than an interrupted hour. Confidence: 0.85
…