[codex] Implement PR11.10d container proof closure#198
Merged
berkeleynerd merged 2 commits intoberkeleynerd:mainfrom Apr 3, 2026
Merged
[codex] Implement PR11.10d container proof closure#198berkeleynerd merged 2 commits intoberkeleynerd:mainfrom
berkeleynerd merged 2 commits intoberkeleynerd:mainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Closes the PR11.10 container proof milestone by introducing an explicit PR11.10d “parent checkpoint” that is defined as the union of the already-shipped PR11.10a/10b/10c proof manifests, while keeping the single documented runtime-only exclusion visible in both tooling output and documentation.
Changes:
- Add a PR11.10d checkpoint manifest (as the union of PR11.10a–c) to the proof inventory and validate it in the proof runner.
- Extend
run_proofs.pysummary output to report PR11.10d checkpoint results as a rolled-up union of PR11.10a–c. - Update roadmap and emitted verification matrix docs to explicitly state the closed container checkpoint surface and its one named exclusion.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| scripts/run_proofs.py | Validates the new PR11.10d manifest and prints a PR11.10d checkpoint summary (rolled up from PR11.10a–c results). |
| scripts/_lib/proof_inventory.py | Defines PR11_10D_CHECKPOINT_FIXTURES as PR11.10a–c union and reassigns the empty-list pop_last proof exclusion milestone to PR11.10d. |
| docs/PR11.x-series-proposed.md | Adjusts PR11.10d roadmap text to reflect closure as a union and allow named documented runtime-only exclusions. |
| docs/emitted_output_verification_matrix.md | Adds a matrix row and an explicit PR11.10 container checkpoint corpus section documenting PR11.10d and the one exclusion. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Summary
PR11.10dproof checkpoint as the explicit union of the shippedPR11.10a,PR11.10b, andPR11.10cmanifestspop_lastwitness as a named runtime-only exclusion underPR11.10dWhy
PR11.10a,PR11.10b, andPR11.10calready shipped the optional/list/map surface with proof-backed fixtures. This follow-up closes the parentPR11.10milestone as a checkpoint/ledger milestone rather than another feature milestone, and makes the live proof runner and docs match that state.Validation
cd compiler_impl && alr buildpython3 scripts/run_tests.py->717 passed, 1 skipped, 0 failedpython3 scripts/run_samples.py->19 passed, 0 failedpython3 scripts/run_proofs.py->176 proved, 0 failedpython3 scripts/run_proofs.pysummary includesPR11.10d checkpoint: 12 proved, 0 failed