feat(context): add CPG-07 explicit sidecar composition gates - #25
Conversation
|
✅ Deterministic PR hygiene checks passed. |
8f32b98 to
eab5a3d
Compare
ac7ee9b to
771d8c2
Compare
4054492 to
465d0e6
Compare
771d8c2 to
6b856fd
Compare
4fe3405 to
22d5800
Compare
c93d127 to
6fb741f
Compare
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
b06d215 to
5e1f0d8
Compare
6fb741f to
98c2fee
Compare
5e1f0d8 to
6de79db
Compare
98c2fee to
d0d8e81
Compare
6de79db to
7d2dba0
Compare
d0d8e81 to
57f5ca9
Compare
Codex advertises hosted web_search on ordinary coding turns. That created a sidecar plan and disabled live recovery. Keep advertised web_search intercept-on-call, and suspend only vision/image/video rewrite helpers. Combined on-mode also collapses duplicates first.
7d2dba0 to
8a44406
Compare
Summary
Scope decision after deep review
The original CP-07 roadmap called this stage "stateless sidecar composition". This PR intentionally implements the safe composition boundary, not a merged recovery+sidecar execution loop. Full recovery composition with web/media/vision remains deferred until CPG-08 has live-provider task-quality, cost-per-success and latency evidence for the simpler stateless recovery path.
This narrowing is deliberate. Until that gate passes, an actual active sidecar remains authoritative and large recovery fails open for that turn.
Replacement note
This replaces PR #22. During stack repair, the CPG-06 base ref was temporarily pointed at the CPG-07 candidate so the repository's existing CPG-06 hardening workflow could be reused. GitHub therefore auto-marked #22 as merged when its head briefly became reachable from its base. The CPG-06 ref was immediately restored and contains no CPG-07 commits. GitHub does not allow reopening a merged PR, so this replacement PR is required for the corrected CPG-07 implementation commit.
TDD
The first commit defines the actual-plan composition-boundary semantics. The implementation commit preserves the earlier wire-safety and hidden-usage fixes while moving recovery preparation behind resolved sidecar planning.
Stack
agent/context-projection-cp06/ PR feat(context): add CPG-06 stateless recovery-priority loop #21