GH#17503: restore MERGE_SUMMARY compliance — executable template, contract V3, continuation prompt#17519
Conversation
… V3, continuation prompt (GH#17503) Two-factor root cause for 65% MERGE_SUMMARY miss rate: 1. Simplification (738e4ca) compressed the executable gh pr comment template into a single prose line — workers stopped following it 2. Neither the headless contract pre-exit check nor the continuation prompt mentioned MERGE_SUMMARY — workers that stopped after PR creation were never reminded Three fixes: - full-loop.md: restore the copy-paste gh pr comment command with template and add a verification command - headless contract V2→V3: add MERGE_SUMMARY to the pre-exit self-check checklist with the exact verification command - continuation prompt: explicitly remind about MERGE_SUMMARY after PR creation
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 8 minutes and 41 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🔍 Code Quality Report�[0;35m[MONITOR]�[0m Code Review Monitoring Report SonarCloud: 0 bugs, 0 vulnerabilities, 1 code smells Mon Apr 6 01:21:49 UTC 2026: Code review monitoring started 📈 Current Quality Metrics
Generated on: Mon Apr 6 01:21:52 UTC 2026 Generated by AI DevOps Framework Code Review Monitoring |
|
|
Completed via PR #17519, merged to main. Merged by deterministic merge pass (pulse-wrapper.sh). No worker summary was available — the worker either crashed before writing one or this PR predates the merge summary convention.aidevops.sh v3.6.113 spent 5m on this as a headless bash routine. |
Up to standards ✅🟢 Issues
|
1 participant
Summary
gh pr commenttemplate in full-loop.md step 4.2.1 (removed by simplification commit738e4ca49)Root Cause
Two-factor regression causing 65% MERGE_SUMMARY miss rate (13 of 20 recent PRs):
Simplification (
738e4ca49) compressed a full executable code block with template into a single prose line. Workers went from having a copy-paste command to a compressed instruction competing with 15+ other mandatory steps.Continuation gap: Neither the headless contract pre-exit check nor the continuation prompt mentioned MERGE_SUMMARY. Workers that stop after PR creation (natural model stopping point) are never reminded about this step.
Evidence: PRs before simplification (17030, 17017, 17324) had 100% MERGE_SUMMARY rate. Post-simplification rate dropped to ~35%.
Changes
.agents/scripts/commands/full-loop.md— restoregh pr commentcode block with template + add verification command.agents/scripts/headless-runtime-helper.sh— contract V2→V3 (add MERGE_SUMMARY to pre-exit checklist), continuation prompt (add MERGE_SUMMARY reminder)Defense in Depth
This PR complements #17517 (already merged) which added PR body text as a fallback source for
_extract_merge_summary. Together:Closes #17503