From 8b8753d32170435f27fa83908ec69cfa4c5e0fad Mon Sep 17 00:00:00 2001 From: AGrushikhin Date: Sun, 26 Jul 2026 09:40:32 +0000 Subject: [PATCH] fix(review-pr): always leave a final verdict when no issues found MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The review posts via claude-code-action with track_progress + sticky comment. The 'no issues -> do not post any comments' instruction left the sticky comment stuck on the interim 'I'll analyze this and get back to you' placeholder, so clean PRs showed no visible result. Instruct the reviewer to finish with a short verdict (e.g. 'Reviewed — no issues found') in the sticky comment, clarified as the tracking comment (not a separate review report). --- plugins/review/skills/review-pr/SKILL.md | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/plugins/review/skills/review-pr/SKILL.md b/plugins/review/skills/review-pr/SKILL.md index dc25966..34e802b 100644 --- a/plugins/review/skills/review-pr/SKILL.md +++ b/plugins/review/skills/review-pr/SKILL.md @@ -276,7 +276,15 @@ When using `gh api repos/{owner}/{repo}/pulls/{pr_number}/comments`, post just o #### If you found no issues -Do not post any comments. Simply report to the user that no issues were found. +Do not post inline review comments. But you MUST still end with a clear final +verdict in the sticky progress comment — never leave it on the interim +"I'll analyze this and get back to you" placeholder. Finish with a short line +such as: + +> ✅ Reviewed — no issues found. + +This is the tracking/sticky comment, not a separate overall review report, so it +does not violate the "inline comments only" rule above. ## Remember