[codex] compile packets from repository evidence - #2107
Merged
Conversation
TheGreenCedar
marked this pull request as ready for review
September 2, 2026 10:36
This was referenced Sep 2, 2026
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.
Closes #2106
Refs #2098
Context
PR #2103 repaired the Horizon A substrate and removed benchmark-shaped packet policy. This separate Horizon B slice wires the repository-derived compiler promised by the frozen 0.18 evidence-compiler plan. It does not establish product usefulness or authorize a release.
Base:
7794d6e2696450a683163b3926842b44f7aa4b38Head:
e51b6389f43eb1196ae711b702defa110720464bTree:
14416957638a8f749a3b669872bd882b1292dd6bWhat changed
RetrievalSeedPlanV1 -> PacketCandidateDescriptorV1 -> PacketAdmissionReceiptV1 -> PacketCompilationInputV1 -> PacketEvidenceProductV3boundary.answer_sufficiency: not_asserted.How to review
Start at:
crates/codestory-contracts/src/compilation.rscrates/codestory-agent/src/packet_plan.rscrates/codestory-agent/src/evidence_compiler.rscrates/codestory-runtime/src/agent/orchestrator.rscrates/codestory-runtime/src/agent/packet_compiler.rscrates/codestory-store/src/storage_impl/mod.rscrates/codestory-agent/tests/repository_evidence_metamorphic.rsThe critical boundary is that
PacketCompilationInputV1cannot carry the question, prompt terms, task class, obligations, roles, carriers, answer stages, or sufficiency policy. With frozen admitted evidence, paraphrase, bijective rename, opaque edge-ID changes, and input permutation cannot change selection shape.Focused verification
cargo test --locked -p codestory-runtime --lib: 832 passed, 4 ignored.git diff --check: passed.Earlier unchanged component checks on this branch also passed: contracts 124, store 284, retrieval 406 with 3 ignored, CLI golden path 18, both proof-feature transport profiles, evidence-only conformance, and strict library clippy for retrieval/indexer.
Risk
This deliberately changes packet evidence ordering and removes legacy presentation artifacts as selection authority. The main semantic risk is losing useful evidence that old fitted policy happened to preserve. The next gate is the builder-visible five-arm ablation; this PR itself makes no accuracy claim.
Non-claims