Restore exact repository inputs in protected ABI checks - #1257
Closed
brandonpayton wants to merge 2 commits into
Closed
Restore exact repository inputs in protected ABI checks#1257brandonpayton wants to merge 2 commits into
brandonpayton wants to merge 2 commits into
Conversation
Protected ABI jobs inspect candidate product manifests without executing\ncandidate code. The SDK product now hashes musl license data, but both\ninert candidate checkouts omitted submodule contents and failed before\nrequirements could be derived.\n\nMaterialize only the candidate\x27s exact musl gitlink through the fixed,\nprotected repository transport with credentials removed. Guard both\ncollector paths with mutation-tested workflow contracts and refresh the\nversion-10 implementation projection.
The ready-to-ship evidence gate independently derives requirements from\nan inert candidate checkout. Materialize the same exact musl gitlink from\nthe protected transport there, so the normal merge path does not retain\nthe missing-license-data failure fixed in the collector and request feed.\n\nAdd omission, ordering, and candidate-transport mutations, then refresh\nthe version-10 implementation projection.
Member
Author
|
Superseded by #1254. That PR now covers all three protected consumers (PR projection, request derivation, and merge-gate revalidation) through one credential-stripped, adversarially tested helper and request-policy version 11. Closing this duplicate so the two policy projections cannot race. |
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.
Why
Protected ABI checks inspect a pull request without executing its code.
The developer SDK product now includes musl license data, but those jobs
check out the candidate without submodules. They therefore fail before
deriving requirements because
libc/musl/COPYRIGHTis absent.This blocks ABI validation and the protected ready-to-ship gate for PR
#1253 and any later change that selects the developer SDK product, even
when the candidate itself is valid.
What changed
the protected checkout.
ready-to-ship merge-gate derivations.
candidate-controlled transport.
Validation
musl commit
0784374d561435f7c787a555aeab8ede699ed298,6,204-byte
COPYRIGHT, and a clean parent checkout afterward.git diff --check.No runtime, ABI layout, package artifact, or browser behavior changes.