From fe83b82dab343d35dbc6af164e64c7f96ce21b01 Mon Sep 17 00:00:00 2001 From: wenshao Date: Tue, 7 Apr 2026 00:38:15 +0800 Subject: [PATCH] feat(review): add "fix these issues" follow-up tip MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit After review with unfixed findings (autofix declined/partial/N/A), suggest "type fix these issues" so the LLM can interactively fix each finding using the edit tool without re-running the review. Follow-up tips now cover the complete post-review flow: - Unfixed findings → "fix these issues" - PR with findings → "post comments" - Local all clear → "commit" Co-Authored-By: Claude Opus 4.6 (1M context) --- packages/core/src/skills/bundled/review/SKILL.md | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/packages/core/src/skills/bundled/review/SKILL.md b/packages/core/src/skills/bundled/review/SKILL.md index 31438d486..c94e8baae 100644 --- a/packages/core/src/skills/bundled/review/SKILL.md +++ b/packages/core/src/skills/bundled/review/SKILL.md @@ -290,10 +290,13 @@ One of: - **Request changes** — Has critical issues that need fixing - **Comment** — Has suggestions but no blockers -Append a follow-up tip after the verdict based on the review target: +Append a follow-up tip after the verdict (and after Step 3.5 Autofix if applicable). Choose based on remaining state: -- **PR review with confirmed findings**: "Tip: type `post comments` to publish these findings as PR inline comments." -- **Local review with Approve verdict**: "Tip: type `commit` to commit your changes." +- **Unfixed findings remain** (autofix declined, partial, or not applicable): "Tip: type `fix these issues` to apply fixes interactively." +- **PR review with findings** (whether fixed or not): also append "Type `post comments` to publish findings as PR inline comments." +- **Local review, all clear** (Approve or all issues fixed): "Tip: type `commit` to commit your changes." + +If the user responds with "fix these issues" (or similar intent), use the `edit` tool to fix each remaining finding interactively based on the suggested fixes from the review — do NOT re-run Steps 1-3.5. If the user responds with "post comments" (or similar intent like "yes post them", "publish comments"), proceed directly to Step 4 using the findings already collected — do NOT re-run Steps 1-3.5.