Skip to content

feat(runner-policy): register ci-workflows v0.49.0 intake contract #802

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
kyle-sexton merged 2 commits into from
Oct 18, 2026
Merged

feat(runner-policy): register ci-workflows v0.49.0 intake contract #802

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d186036
feat(runner-policy): register ci-workflows v0.49.0 intake contract
kyle-sexton Oct 18, 2026
42cfb09
docs(runner-policy): record the v0.49.0 intake sign-off
kyle-sexton Oct 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions components/runner-policy/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1263,6 +1263,18 @@ or COLLABORATOR of that issue's own repository; issue titles and bodies are not
filtered by author. The lockstep
declines it on `credentialReferences`; the other two are byte-identical. No
input, secret, workflow or job permission, or `runs-on` routing changes.
`intake-triage` is registered at the v0.49.0 tag
`ff8cd36321423de961381e608e4a31563e74adb3` (ci-workflows#773, #774), signed off
by hand on 2026-10-18. It copies its v0.45.0 entry and adds `issue-number`,
`allowed-author-ids` and `allowed-bot-sender-ids`. The OWNER or MEMBER gate
becomes a numeric-id allowlist: an `opened` issue runs only when its author
opened it and the author's id is in `allowed-author-ids` (a bot's sender id in
`allowed-bot-sender-ids`). A `workflow_dispatch` run triages `issue-number`,
validated against `^[1-9][0-9]{0,9}$` and read from the API, only for an actor
whose id is in one of the two lists. A new `skipped` job takes
`issues: write`, inside the caller grant, to apply the new `skipped-label`
input, which stays unregistered because no caller passes it. No secret or
`runs-on` routing changes.
The Zizmor contract at `de50a08b6093d231519ee7a4c9371db76c0a7e1e`
uses its reviewed `runner` input and checksum-verified native Linux binary, so
enrolled consumers may route that advisory lane onto the managed fleet
Expand Down
23 changes: 23 additions & 0 deletions components/runner-policy/policy.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,29 @@
"issues": "write"
}
},
"melodic-software/ci-workflows/.github/workflows/intake-triage.yml@ff8cd36321423de961381e608e4a31563e74adb3": {
"routing": "runner-input",
"runnerInput": "runner",
"allowedInputs": [
"runner",
"allowed-labels",
"escalation-label",
"floor-label",
"instructions",
"cli-version",
"claude-args",
"issue-number",
"allowed-author-ids",
"allowed-bot-sender-ids"
],
"allowedSecrets": {
"CLAUDE_CODE_OAUTH_TOKEN": "${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}"
},
"allowedCallerPermissions": {
"contents": "read",
"issues": "write"
}
},
"melodic-software/ci-workflows/.github/workflows/pr-review.yml@ab83b01273026ab5c23c6f3b40e946d97a863fa2": {
"routing": "runner-input",
"runnerInput": "runner",
Expand Down
23 changes: 23 additions & 0 deletions components/runner-policy/runner-policy.test.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ const REPINE_LANE_SHA_V0_44_0 = "58a60eac76cf36ed8ef4f59460acb4a8d34d3ff7";
const REPINE_LANE_SHA_V0_44_1 = "bc02e0504d83df70a2c95fcf3651d0f1eac3e252";
const REPINE_LANE_SHA_V0_45_0 = "3c8a4b2aeff5871cf72d35a686692bc6f0d44815";
const REPINE_LANE_SHA_V0_48_0 = "96b50d451c4483c476f2f963674e0a3ba9d50fe3";
const REPINE_LANE_SHA_V0_49_0 = "ff8cd36321423de961381e608e4a31563e74adb3";
const STANDARDS_SYNC_SHA = "35f2684ac953794b854bac1959df00e74eeca1d9";
const REUSABLE_PATH = "melodic-software/ci-workflows/.github/workflows/osv-scanner.yml";
const REUSABLE_REFERENCE = `${REUSABLE_PATH}@${SHA}`;
Expand Down Expand Up @@ -9356,6 +9357,28 @@ test("every reusable contract at the v0.48.0 tag copies its v0.45.0 entry forwar
);
});

// v0.49.0 gates intake by author id and admits a workflow_dispatch for a
// listed actor; intake-triage is its v0.45.0 entry plus exactly those inputs.
test("intake-triage at the v0.49.0 tag copies its v0.45.0 entry and adds only the author-gate inputs", () => {
const contracts = BASE_POLICY.approvedReusableWorkflowContracts;
const workflowPath = "melodic-software/ci-workflows/.github/workflows/intake-triage.yml";
const previous = contracts[`${workflowPath}@${REPINE_LANE_SHA_V0_45_0}`];
assert.ok(previous, "expected a v0.45.0 contract for intake-triage");
assert.deepEqual(contracts[`${workflowPath}@${REPINE_LANE_SHA_V0_49_0}`], {
...previous,
allowedInputs: [
...previous.allowedInputs,
"issue-number",
"allowed-author-ids",
"allowed-bot-sender-ids",
],
});
assert.equal(
Object.keys(contracts).filter((key) => key.endsWith(`@${REPINE_LANE_SHA_V0_49_0}`)).length,
1,
);
});

// Convergence as a property, never as a hardcoded SHA. The daily
// claude-lanes-repin lane rewrites these components on every ci-workflows
// release, and a test naming one revision would go red by construction on
Expand Down
Loading