Skip to content

feat(context): add CPG-07 explicit sidecar composition gates - #25

Open
Wibias wants to merge 3 commits into
agent/context-projection-cp06from
agent/context-projection-cp07
Open

feat(context): add CPG-07 explicit sidecar composition gates#25
Wibias wants to merge 3 commits into
agent/context-projection-cp06from
agent/context-projection-cp07

Conversation

@Wibias

@Wibias Wibias commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • Stack CPG-07 on CPG-06.
  • Make live recovery eligibility depend on actual sidecar plans, not coarse parser/config markers.
  • Preserve the conservative rollout: active web-search, image, video, or vision sidecars suspend large-result recovery for that turn.
  • Preserve the CPG-05/06 review hardening: provider-wire recovery-tool verification and fail-open hidden-usage accounting remain intact.

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

@github-actions

Copy link
Copy Markdown

Deterministic PR hygiene checks passed.

@Wibias
Wibias force-pushed the agent/context-projection-cp06 branch from 8f32b98 to eab5a3d Compare August 16, 2026 03:07
@Wibias
Wibias force-pushed the agent/context-projection-cp07 branch from ac7ee9b to 771d8c2 Compare August 16, 2026 03:08
@Wibias
Wibias marked this pull request as ready for review August 16, 2026 03:13
@Wibias
Wibias force-pushed the agent/context-projection-cp06 branch from 4054492 to 465d0e6 Compare August 16, 2026 03:14
@Wibias
Wibias force-pushed the agent/context-projection-cp07 branch from 771d8c2 to 6b856fd Compare August 16, 2026 03:14
@github-actions
github-actions Bot force-pushed the agent/context-projection-cp07 branch from 4fe3405 to 22d5800 Compare August 16, 2026 04:48
@github-actions
github-actions Bot force-pushed the agent/context-projection-cp06 branch from c93d127 to 6fb741f Compare August 16, 2026 05:03
@coderabbitai

coderabbitai Bot commented Aug 16, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (2)
  • ^dev$
  • ^preview$

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 237eec75-887b-4ed6-9486-7400f67a8121

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions
github-actions Bot force-pushed the agent/context-projection-cp07 branch from b06d215 to 5e1f0d8 Compare August 16, 2026 05:04
@Wibias
Wibias force-pushed the agent/context-projection-cp06 branch from 6fb741f to 98c2fee Compare August 16, 2026 20:42
@Wibias
Wibias force-pushed the agent/context-projection-cp07 branch from 5e1f0d8 to 6de79db Compare August 16, 2026 20:42
@Wibias
Wibias force-pushed the agent/context-projection-cp06 branch from 98c2fee to d0d8e81 Compare August 16, 2026 20:51
@Wibias
Wibias force-pushed the agent/context-projection-cp07 branch from 6de79db to 7d2dba0 Compare August 16, 2026 20:51
@Wibias
Wibias force-pushed the agent/context-projection-cp06 branch from d0d8e81 to 57f5ca9 Compare August 16, 2026 23:21
Wibias added 3 commits August 17, 2026 01:21
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.
@Wibias
Wibias force-pushed the agent/context-projection-cp07 branch from 7d2dba0 to 8a44406 Compare August 16, 2026 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant