Repository navigation
Conversation
…s-owner-p52-configuration-admission
…s-owner-p52-configuration-admission
…s-owner-p52-configuration-admission
…s-owner-p52-configuration-admission
…s-owner-p52-configuration-admission
|
|
Heads-up: the required checks on The
This branch still runs the old workflow, so those checks never report here and the PR cannot merge as it is. What to do when this PR is ready to land: wait until PR #3185 is merged, then bring this branch up to date with |
|
Superseded by the replanning of #3146 under epic #2992 on branch The user authorized this restart through room-37 (P50 option 1, 2026-10-27). The original changes target an obsolete file structure and will not be ported mechanically. The old branch |



Reject repeated decoded JSON member paths before consuming AgentsCommander configuration, returning the physical file and cause while preserving its bytes and native syntax/IO policy. Add checked configuration providers and read-only startup session selection; consumer migration continues in subsequent phases of #2992.
Validated locally on Windows at bf18098: all 11 admission tests and 9 layering checks passed, along with all-targets, formatting, the existing Clippy/cognitive gate, module arcs and complete SCC comparison. Rust, architecture and UI reviewers approved code and proof for this exact commit, including the full change since the previous review. Temporary function-level dead-code annotations retain prepared helpers until their planned consumer migrations. The PR changes ten owned paths; eleven historical upstream imports match current main.
Approved specification: master v29 / P52 v5, Full R23. Proof SHA256: 2B71FCBEB469A0CD42802CEF4D80C765F4D3A687C8B253EE2247EE147EB226CF. Three-review receipt SHA256: C00F6A6F7A016409EA36AF3E9512ABA1585C89853AA5345325F9ED7FB0345749. Main was fetched and remains f183d6e, already included in this commit. Fresh full CI is required before merge.
Closes #3146