fix(compound-engineering): reduce /ce:review context pressure#228
Open
Neo1228 wants to merge 2 commits intoEveryInc:mainfrom
Open
fix(compound-engineering): reduce /ce:review context pressure#228Neo1228 wants to merge 2 commits intoEveryInc:mainfrom
Neo1228 wants to merge 2 commits intoEveryInc:mainfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #166.
Summary
/ce:reviewwas fanning broad PR context out to every spawned reviewer. On larger or high-effort reviews, that makes compaction/context-limit failures much more likely before the agents get to the risky code.This PR reduces that pressure while keeping the normal review flow intact:
code-simplicity-reviewertwicesetupfrom writing always-on agents back intoreview_agentsWhy this is a safe fix
compound-engineering.local.mdguidance now matches actual/ce:reviewruntime expectationsValidation
bun testplugins/compound-engineeringplugin into a temp Codex home and verifies the generatedce-reviewprompt/skill plussetupskill preserve the new guardrailsbun run src/index.ts convert plugins/compound-engineering --to codex --codex-home <tmp>bun run src/index.ts convert plugins/compound-engineering --to opencode --output <tmp>review_agentsconfig confirmed the revised prompt reads local config and branch diff metadata before reviewer planningFiles changed
plugins/compound-engineering/commands/ce/review.md: compact packet step, deduped base agent set, staged large-PR guidance, tighter conditional-agent inputsplugins/compound-engineering/skills/setup/SKILL.md: updated review-depth copy and removed always-on agent duplicationtests/compound-review-command.test.ts: source-content regression checks for packet + dedupe behaviortests/cli.test.ts: Codex conversion integration check for the actual compound-engineering plugin outputplugins/compound-engineering/CHANGELOG.md: release note for issue Context limit reached in review agents #166Remaining caveat
This change improves the prompt contract and verifies that the contract survives real conversion output, but a fully deterministic end-to-end multi-agent
/ce:reviewrun is still not something the automated suite can assert. The new coverage is aimed at the highest-risk regression points for this issue: packet creation guidance, agent dedupe, large-PR staging instructions, and setup/runtime alignment.