Repository navigation
Conversation
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
P00-R8-B01/active-anchor-run-id-contradiction: the closed immutable active anchor excludesrunId, while the prior admission clause incorrectly required that later per-invocation value inside itrun-binding.jsoncreated only after controller-owned immutable-anchor selection, with exact CSPRNG identity derivation, exclusive root creation, controller ownership, JCS schema, descriptor validation, collision/crash/re-arm behavior, terminal binding, and final canonical reopenFinding disposition
P00-R8-B01 / RELEASE_BLOCKING /
active-anchor-run-id-contradictionP00-R9 now requires:
runId; onlyarm-phaseinstalls the anchor, andverify-phasenever rewrites, augments, repairs, replaces, or re-arms it;run-binding.jsonhas exactlyschemaVersion,kind,runId,anchorSha256,candidateCommitSha,phase,role, andcreatedAt; every repeated value must match the held anchor, and the binding is never selection authority;1/0/0.The binding references the already-authoritative anchor digest. It is not a second candidate, checkout, plan, toolchain, role, writable-root, or evidence-root selector.
Identity and scope
63c550808a6623c6c4fba9c538514243989b8d1eP00-R8-B01main/9dcb4b2d39607273a8528a24657cdb4f5bfc3412ce5e6ae533b65cde6d7996ec572c6d35315b5bde02ee6ddec78d6e4ea9e2de3c0303ffd6bc9f45bfdocs/implementation/phase-execution-packet.mddesign.mdremains byte-identical, SHA-25631819873972e1bc02650a1d6bfab1c7f8c687647a19731aa3881e0d71f7fe64aphase/P00-execution-packetMechanical and structural proof
git diff --check: PASS9b7918fd7f37c56c2c265ac45a22cbae62d654a1090e59d847f3009920357f24failure.json,success.json,success.json; aggregate vector1/0/0; qualification UUID schema unchangedmarkdownlint-cli2@0.19.0reports the exact inherited baseline, 343 findings before and after (MD013 231/231; MD060 112/112), so this revision adds zero Markdown findingsAuthoring-check disclosure
No product test or controller binary was run: this is a one-document execution-contract correction. Checker iterations are disclosed rather than hidden:
Review requirement
Fresh
P00/review-9is mandatory against exact SHA02ee6ddec78d6e4ea9e2de3c0303ffd6bc9f45bf. Review must independently verify the one-path diff, byte-identicaldesign.md, immutable anchor exclusion, closed non-selecting binding schema, creation/ownership/descriptor lifecycle, collision/crash/re-arm/final-reopen behavior, P12 three-run separation, every frozen inventory, exact[7,0]negative-control contract, PR identity/checks, and absence of unresolved actionable feedback.Do not merge this draft PR until the controller records the required exact-SHA review disposition.