Skip to content

fix(queue): reconcile verified package source PRs - #80

Merged
IanHollow merged 6 commits into
mainfrom
codex/ci-source-pr-validation-ac837fe3
Oct 12, 2026
Merged

IanHollow merged 6 commits into
mainfrom
codex/ci-source-pr-validation-ac837fe3

Conversation

@IanHollow

Copy link
Copy Markdown
Member

Summary

Expose verified package source-PR validation results to GitHub’s protected merge gate. Explicitly dispatched bot jobs exist in the Actions API but are absent from the PR rollup, preventing unattended source updates from entering the normal queue.

The shared reconciler gains an opt-in workflow-to-required-job map. It proves exact bot/repository/branch identity, actual parent ancestry, selected package/catalog ownership, and current protected workflow blobs before reporting real job outcomes. Genuine failures link their actual failed jobs; missing, running, skipped, stale, or untrusted evidence never becomes success. Existing synthetic queue reporting is unchanged.

Evidence

The live Claude source PR has successful native validation and an actual DCO failure omitted from its PR rollup. The new adapter rejects its stale workflow contract and identifies the exact failed DCO job without mutating the repository.

All 133 repository regressions and offline workflow/security validators pass, including 73 queue/source cases. Fixtures cover immutable identity and scope proofs, approval runs with zero jobs, real failure URLs, producer reruns during collection/publication, API errors, idempotence, and request budgets. Six idle source PRs require 103 reads and no writes; admitted-head scans require 12 reads; exact-head events inspect only the relevant source. Native Nix and representative caller checks are required before merge.

Merge danger

This changes privileged admission/reporting only for callers that explicitly enable package source checks. The representative caller must provide all protected job names and requested/in-progress notifications; it then regenerates bot branches from reviewed protected main. Normal expected-head admission and full merge-group validation still apply. No additional token permissions or caller-code execution are introduced. Rollback restores the previous shared commit and disables the source opt-in.

Closes #79.

Signed-off-by: IanHollow <72767437+IanHollow@users.noreply.github.com>
Signed-off-by: IanHollow <72767437+IanHollow@users.noreply.github.com>
Signed-off-by: IanHollow <72767437+IanHollow@users.noreply.github.com>
Signed-off-by: IanHollow <72767437+IanHollow@users.noreply.github.com>
Signed-off-by: IanHollow <72767437+IanHollow@users.noreply.github.com>
Signed-off-by: IanHollow <72767437+IanHollow@users.noreply.github.com>
@IanHollow
IanHollow added this pull request to the merge queue Oct 12, 2026
Merged via the queue into main with commit 1845f37 Oct 12, 2026
8 checks passed
@IanHollow
IanHollow deleted the codex/ci-source-pr-validation-ac837fe3 branch October 12, 2026 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support verified package source PR results and unattended queue admission

1 participant