Add pre-call ASM selection to the receipt composition test - #3
Open
YE-YI7 wants to merge 2 commits into
Open
Conversation
YE-YI7
marked this pull request as ready for review
August 17, 2026 01:26
Author
|
One factual follow-up: The two review questions remain intentionally narrow: whether |
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.
What this adds
This extends the existing ScopeBlind + APS composition test with a separate pre-call ASM Selection Receipt:
The three artifacts do not merge authority. ASM explains the fixture choice and pins the two consulted manifests. ScopeBlind and APS remain authoritative for their own signed policy/execution events.
Concrete changes
extensions.asmin the two ScopeBlind-signed receiptsverify-all.shWhile exercising the current fixture, I found that the ScopeBlind generator minted a different
action_reffrom the checked-in APS composition receipts. This patch reuses the existing APS anchor byte-for-byte and makesverify-all.shfail if they diverge again.Verification performed
Both manifests also pass the public
asm-protocol==0.5.2release (asm-lint) withselection_readiness=ready.Review questions
extensions.asmreference acceptable here, or would ScopeBlind prefer a reverse-DNS key or a ScopeBlind-owned reference field?action-ref.txtthe intended canonical anchor for this composition fixture? This PR assumes yes and stops generating a competing one.The repository-native composition verifier above is self-contained and does not depend on the package release. The separately tested lint command is now public as
asm-protocol==0.5.2; this PR intentionally does not add that package to ScopeBlind CI.This PR does not claim ScopeBlind adoption, partnership, endorsement, or shared governance. If the extra pre-call artifact makes this example less clear, closing the PR is a valid outcome.