Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(ce-plan): route confidence-gate pass to document-review#462

Merged
tmchow merged 2 commits into
mainfrom
feat/fix-ce-plan-review-skip
Mar 31, 2026
Merged

fix(ce-plan): route confidence-gate pass to document-review#462
tmchow merged 2 commits into
mainfrom
feat/fix-ce-plan-review-skip

Conversation

@tmchow
Copy link
Copy Markdown
Collaborator

@tmchow tmchow commented Mar 31, 2026

Phase 5.3.2 in ce:plan told the model to "proceed to Phase 5.4" when the confidence gate passed, which skipped document-review (5.3.8) entirely. Plans that didn't need deepening never got reviewed. The fix routes the gate-pass case to 5.3.8 so document-review always runs — it catches different issues than the confidence check and should never be skipped.

Also adds a "Debugging Plugin Bugs" section to the plugin AGENTS.md. Developers of this plugin use it via their marketplace install, which can be older than the repo. When investigating a reported usage bug, agents should now diff the installed version against the repo before diving in — avoiding wasted effort on already-fixed issues or misdiagnosis when the two versions diverge.


Compound Engineering v2.59.0
🤖 Generated with Claude Opus 4.6 (1M context, extended thinking) via Claude Code

tmchow added 2 commits March 30, 2026 23:16
…-gen options

Phase 5.3.2 said "proceed to Phase 5.4" when the confidence gate passed,
which skipped document-review (5.3.8) entirely. Now routes to 5.3.8 so
document-review always runs regardless of whether deepening was needed.
…n versions

When developers report bugs from their own usage, the installed
marketplace version may be stale. This teaches agents to diff the
installed copy against the repo before diving into a fix.
@tmchow tmchow merged commit 1962f54 into main Mar 31, 2026
2 checks passed
@github-actions github-actions Bot mentioned this pull request Mar 31, 2026
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.

1 participant