Skip to content

Pin the native reader to the policy that registers the no-retries CI bytes - #375

Merged
rmcdaniel merged 1 commit into
mainfrom
general/295b-native-policy-pin
Nov 13, 2026
Merged

rmcdaniel merged 1 commit into
mainfrom
general/295b-native-policy-pin

Conversation

@rmcdaniel

Copy link
Copy Markdown
Member

Part of #295. Third step of the staged change: #374 merged, then this PR, then #373. It's the same one-line pattern as #365 and #366.

native-policy.yml checks out the reviewer at a pinned ref and reads the reviewed-bytes registry from it. The pin is still 4ad153a3, the merge of #364, which predates #374. So Validate native policy subject keeps refusing #373 (run 43368725978): Protected pilot requires closed literal commands or exact reviewed workflow bytes.

Change: one line. The reviewer checkout ref moves from 4ad153a3… to 114ad7dc84e43661a1a667dcf21c6b3f60117b6c, the merge of #374.

What else moves with the ref (4ad153a3..114ad7dc, files under scripts/ and qualification/):

Checked locally against 114ad7d's protected_policy_context.validate_context:

This PR's own run of the check uses the new ref, because pull_request runs the workflow from the PR. After it merges, re-run #373's check.

🤖 Generated with Claude Code

…bytes

114ad7d is the merge of #374, which adds
the exact ci.yml/sonar.yml identities from #373 (Acquire::Retries "0"
before setup-php, CVE-2026-109518). The reader stays at 4ad153a until
this moves, so #373 is still refused. Between the two refs the only
validator change is that registry entry; native_policy_check.py,
qualification_policy.py and requirements.lock are unchanged.

Refs #295

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@rmcdaniel rmcdaniel added the agent:general Pull request opened by General agent label Nov 13, 2026
@sonarqubecloud

Copy link
Copy Markdown

@rmcdaniel

Copy link
Copy Markdown
Member Author

Security PASS at bb9ef6b (CI 9 green). One line: the native-policy.yml checkout ref moves from 4ad153a to 114ad7d, the #374 merge. In scripts/ and qualification/, the range brings in only already-reviewed changes: #374's registry entry, #369's php_linux_upstream_pins.py (unused by the native check) and #371's removal of php-linux-bootstrap.json plus a README edit. native_policy_check.py, qualification_policy.py and the requirements lock are unchanged. OK to merge, then re-run #373.

@rmcdaniel
rmcdaniel merged commit 586a45f into main Nov 13, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent:general Pull request opened by General agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant