Skip to content

feat: print "Next round" command on review exit + restructure agent prompts#421

Merged
tomasz-tomczyk merged 1 commit intomainfrom
improve-claude-md
May 2, 2026
Merged

feat: print "Next round" command on review exit + restructure agent prompts#421
tomasz-tomczyk merged 1 commit intomainfrom
improve-claude-md

Conversation

@tomasz-tomczyk
Copy link
Copy Markdown
Owner

Summary

  • When a review round ends without approval, crit now prints Next round: crit <args> to stdout. Agents can run that command verbatim instead of remembering the args from earlier in the conversation.
  • Restructured the crit and crit-cli skills (and their equivalents) across all 8 integrations: claude-code, cursor, codex, github-copilot, opencode, windsurf, cline, aider. Same described behavior, leaner prose, deduped reply guidance, broader auto-load triggers for crit-cli.
  • Promoted the anti-resolve rule to a prominent position in every replying section and added it to the single-file integrations (cline, aider, windsurf) where it was missing — addresses Anti-resolve guidance missing from most integration prompts #414.
  • Bumped claude-code plugin to 1.5.0 so users get the updated prompts.

Review

  • Code review: passed (1 quoting bug found and fixed pre-merge — POSIX single-quote instead of strconv.Quote)
  • Parity audit: N/A (no review-page surface touched)

Test plan

  • go test -race ./... — all passes including new TestReviewCycle_NextCommand (table-driven across no-args, single file, multi-file, space-quoted, single-quote-escaped, non-ASCII, flag-style)
  • golangci-lint run ./... — clean
  • go generate ./...integration_hashes_gen.go regenerated; TestCheckInstalledIntegrations / TestDetectInstalledIntegrations pass

Closes #414.

🤖 Generated with Claude Code

…on prompts

Binary:
- review-cycle response now includes next_command; client prints
  "Next round: crit <args>" to stdout when more rounds are expected
- skill prompts can collapse "match args from earlier" prose into
  "run whatever crit printed after Next round:"

Integrations (claude-code, cursor, codex, github-copilot, opencode,
windsurf, cline, aider):
- restructured crit + crit-cli skills for clarity and brevity
- claude-code uses <important if> wrappers for conditional context;
  other integrations get plain markdown sections with same content
- broadened crit-cli description to fire on intent ("multi-agent
  workflows commenting on code/plans/docs/proposals") with explicit
  /crit disambiguation
- promoted anti-resolve guidance to the top of replying sections
  (issue #414) and added it where it was missing in single-file
  integrations (cline, aider, windsurf)
- removed weak/obvious rules and deduped reply prose

Bumped claude-code plugin to 1.5.0; regenerated integration hashes.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@tomasz-tomczyk tomasz-tomczyk merged commit 02c3a0a into main May 2, 2026
6 checks passed
@codecov
Copy link
Copy Markdown

codecov Bot commented May 2, 2026

Codecov Report

❌ Patch coverage is 75.00000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.33%. Comparing base (fd74c94) to head (3f04ac4).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
main.go 20.00% 4 Missing ⚠️
server.go 89.47% 1 Missing and 1 partial ⚠️

❌ Your patch status has failed because the patch coverage (75.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #421      +/-   ##
==========================================
+ Coverage   67.27%   67.33%   +0.06%     
==========================================
  Files          26       26              
  Lines        9668     9687      +19     
==========================================
+ Hits         6504     6523      +19     
- Misses       2653     2654       +1     
+ Partials      511      510       -1     
Flag Coverage Δ
e2e 34.11% <4.16%> (-0.06%) ⬇️
unit 63.53% <70.83%> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tomasz-tomczyk tomasz-tomczyk deleted the improve-claude-md branch May 2, 2026 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Anti-resolve guidance missing from most integration prompts

1 participant