vrr/qwen-code
2
0
Fork 0
mirror of https://github.com/QwenLM/qwen-code.git synced 2026-05-01 21:20:44 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

fix(review): enforce one-line summary body for Approve/Request changes

Browse source 
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) <noreply@anthropic.com>
This commit is contained in:
wenshao 2026-04-07 17:27:52 +08:00
parent 820191d99d
commit 297d81ee5f
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
packages/core/src/skills/bundled/review/SKILL.md
View file

@ -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.