Strict code review that ignores style
codingReviews a diff for correctness, edge cases, and regressions — and refuses to comment on formatting.
Checks function, table, and endpoint names against what they actually do.
A new module, table, or endpoint is about to be written and the names are already in place.
Review these names against what the code actually does.
For each name, answer three questions:
1. Does it describe what the thing does, or how it was implemented?
2. Would it still be true if the behaviour changed in an obvious way?
3. Would two other engineers agree on what it means without reading the code?
Then propose a replacement only where a name fails a question, and say why the new one is better.
Rules:
- Do not propose renaming more than a third of the list; that is a rewrite, not a review.
- No naming-convention opinions. Only meaning.
- If a name is fine, say 'fine' and move on.
Names and what they do:
<list>
Include what the thing does in one line — otherwise the model is reviewing the names against themselves.
Question 2 is the useful filter: names that go stale are the ones written against an implementation rather than a purpose.
Reviews a diff for correctness, edge cases, and regressions — and refuses to comment on formatting.
Extracts or restructures code while proving the public behavior stays identical.
Writes the failing test first, then the fix, and refuses to skip the reproduction.