From 297d81ee5febaeb7e13a9a00804279b1d64b766a Mon Sep 17 00:00:00 2001 From: wenshao Date: Tue, 7 Apr 2026 17:27:52 +0800 Subject: [PATCH] fix(review): enforce one-line summary body for Approve/Request changes LLM was writing detailed analysis in the review summary body despite "minimal body" instruction. Strengthened to "one-line body only, do NOT include analysis/findings/explanations" with concrete examples. Co-Authored-By: Claude Opus 4.6 (1M context) --- packages/core/src/skills/bundled/review/SKILL.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/core/src/skills/bundled/review/SKILL.md b/packages/core/src/skills/bundled/review/SKILL.md index 4eca57e3e..200451254 100644 --- a/packages/core/src/skills/bundled/review/SKILL.md +++ b/packages/core/src/skills/bundled/review/SKILL.md @@ -455,7 +455,7 @@ If posting an inline comment fails (e.g., line not part of the diff, auth error) Only call `gh pr review` in these cases: -- **Approve** or **Request changes** → submit with minimal body (e.g., "Request changes — see inline comments.\n\n*Reviewed by {{model}} via Qwen Code /review*"). These verdicts carry approval/blocking status. +- **Approve** or **Request changes** → submit with a **one-line** body only. Do NOT include analysis, findings summary, or explanations — those are already in the inline comments. Example: "Approve — LGTM.\n\n*Reviewed by {{model}} via Qwen Code /review*" or "Request changes — see inline comments.\n\n*Reviewed by {{model}} via Qwen Code /review*". These verdicts carry approval/blocking status. - **Comment** but **some** inline comments failed → submit with the failed findings in the body. - **No** inline comments posted (all failed or terminal-only) → submit with full findings summary.