From 1e216dd170a46d99ef400ca6bf98f97b23c1db9b Mon Sep 17 00:00:00 2001 From: Craig Jennings Date: Fri, 22 May 2026 18:17:45 -0500 Subject: docs(skills): keep review-code re-review approvals bare MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Extended the posted-summary voice guidance: a re-review confirming requested changes is just "Approving" plus at most a bare positive. An approve summary must not carry a clause describing what the change does or why it works — that's the banned pattern, since the author already knows the rationale and restating it reads as padding. --- review-code/SKILL.md | 2 ++ 1 file changed, 2 insertions(+) (limited to 'review-code') diff --git a/review-code/SKILL.md b/review-code/SKILL.md index 95ac053..3e63b4a 100644 --- a/review-code/SKILL.md +++ b/review-code/SKILL.md @@ -370,6 +370,8 @@ The structured report above stays local. When the verdict is posted as a GitHub Name the good thing and stop: do not explain *why* it's good. The author made the change and already knows the rationale, so justifying the praise reads as sycophantic. "Clean migration off the window globals, tests cover the right edges" lands; appending "...because there are no stragglers and the provider, mocks, and Normal/Boundary/Error cases are all covered" turns a compliment into padding. Elaboration is for findings (something is wrong, here's the failure mode and the fix), never for compliments. +This holds for re-review approvals too. A re-review confirming requested changes is just "Approving." Mechanical rule: an approve summary is the verdict plus at most a bare positive ("Clean.", "Solid fix."). It must contain no clause that says what the change does or why it works. "The hoist to App fixes the crash, and the new test locks it in" is the banned pattern — it describes and justifies the change on an approve. If a clause references the code's behavior, cut it. + Good: - "Nice, clean, good coverage. One small naming nit inline. Approving." - "Clean shape, tests cover the right edges. Approving." -- cgit v1.2.3