Skip to content

Commit fed38ba

Browse files
committed
fix(bugfix): always recommend /review before /pr in controller
In testing, the agent skipped /review and recommended /pr directly after /test, rationalizing that the fix was simple. Add targeted guidance: always recommend /review before /pr — the agent implemented the fix and is not in a position to objectively evaluate its quality. Only the user can decide to skip review. Made-with: Cursor
1 parent 2e962af commit fed38ba

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

  • workflows/bugfix/.claude/skills/controller

workflows/bugfix/.claude/skills/controller/SKILL.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,12 @@ After presenting results, consider what just happened, then offer options that m
9191
- A trivial fix might go straight from `/fix``/test``/review``/pr`
9292
- If the user already has their own PR process, they may stop after `/review`
9393

94+
**Always recommend `/review` before `/pr`.** Do not recommend skipping review, even for
95+
fixes that seem simple or mechanical. You implemented the fix and wrote the
96+
tests — you are not in a position to objectively evaluate their quality.
97+
Review exists precisely to catch what the fixer misses. Only the user can
98+
decide to skip it.
99+
94100
### How to Present Options
95101

96102
Lead with your top recommendation, then list alternatives briefly:

0 commit comments

Comments
 (0)