Code review comments
Skill Amey-Thakur/AI-SKILLS/skills/code-quality/code-review-comments
Plug-and-play skills and prompts for every AI coding agent
npx -y skills add Amey-Thakur/AI-SKILLS --skill code-review-commentsAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
2 things to look at
- 19 days oldThe repository was created 19 days ago. New is not bad, but a brand new repository carrying a familiar-sounding name is the shape a typosquat arrives in, and there has been no time for anyone else to find a problem with it.
- 4 stars4 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its author says it does
Copied from the file, not written here
Write review comments that land by labeling severity and separating genuine questions from directives. Use when leaving comments on a pull request or coaching someone on how theirs read.
SKILL.md
2.5 KB, as published. Nobody here has run it
Code review comments
A review comment interrupts a person mid-flow. It lands when the author can tell in one read how much it matters and what to do next. Comments that bury a blocker among nitpicks, or dress a firm objection as a coy question, waste both people's time and stall the merge.
Method
- Label severity up front. Prefix each comment:
blocking:,nit:,question:,praise:, following a convention like Conventional Comments. The author triages a 30-comment review in seconds when severity is explicit, and guesses wrong when it is not. - Separate directives from questions honestly. If a change must happen, say so: "blocking: this leaks the DB connection on the error path." If you are genuinely unsure, ask a real question and mean it. A rhetorical "did you consider...?" that actually demands a change reads as passive.
- Say why, and show how where it is cheap. "This N+1 fires one query
per row; batch it with a single
WHERE id IN (...)" beats "inefficient." The reason lets the author generalize; a GitHub suggestion block lets them accept the fix in one click. - Comment on the code, never the coder. "This function does three things," not "you always over-scope." Keep it to the diff in front of you; opinions about someone's habits belong in a private conversation.
- Cap the nits, or automate them. More than a few
nit:on style means a linter rule is missing. Push spacing, import order, and naming casing intoprettier,eslint, orruffso review spends on logic. - Reach a verdict. Do not leave a review in limbo with ten comments and no decision. If nothing blocks, approve and trust the author with the nits. Reserve "request changes" for real blockers.
Litmus tests
- Can the author sort your comments into "must fix now" and "later" without asking you?
- Does each blocking comment state a concrete failure, not a preference?
- Would you say the comment out loud, in these words, to the author's face?
Boundaries
Team norms on approval gates and required reviewers override personal style:
follow the repo's CODEOWNERS and review policy. A design disagreement too
large for a line comment belongs in a call or a design doc, not a buried
thread.