ci: harden Dependabot dependency refreshes#111
Conversation
Verify bot-owned, GitHub-signed Dependabot commits before skipping human metadata gates. Keep generated workflows and immutable dependency pins synchronized. Signed-off-by: John McChesney TenEyck Jr <59268465+jmcte@users.noreply.github.com>
|
Exact-head evidence for
Review history: accepted and fixed PR-owner authentication, GitHub-verified commit provenance, and CodeQL init/analyze version consistency findings. The final review found no actionable issue. |
|
@athena-omt @pheidon please review exact head All code and provenance checks pass; the sole failing governance rule is the required independent approval. Exact-head OpenAI autoreview is clean at 0.90 confidence, and auto-merge is armed. This PR closes #108 and, once merged, safely replaces #102-#106. |
athena-omt
left a comment
There was a problem hiding this comment.
Approved for d6f4797. Reviewed the full diff, generated workflow call paths, Dependabot verifier pagination/fail-closed logic, and adjacent tests. Local validation passed: 24 test files / 204 tests, TypeScript build, action-pin validation (42 pins), workflow YAML parsing (16 files), and diff whitespace checks. The current Governance/CI Gate failures are the expected missing independent-review evidence; no code blocker found.
Summary
Governing Issue
Closes #108.
Validation
TMPDIR=/tmp npm run check(24 test files, 204 tests passed)npm run buildbash scripts/ci/check-action-pins.sh(42 immutable pins passed)git diff --check origin/main...HEADorigin/maincompleted cleanly with no accepted/actionable findings (confidence 0.90).CI Gate.Autoreview command and result:
/Users/johnteneyckjr./.codex/skills/autoreview/scripts/autoreview --mode branch --base origin/main --stream-engine-output— clean, no findings.Bootstrap Governance
src/archetypes.tscarry the same policy and pins.Material change: yes
ADR: docs/decisions/ADR-0005-public-repository-security-baseline.md
Merge Automation
Notes
This PR supersedes #102, #103, #104, #105, and #106. Those branches mix valid dependency bumps with generated-file drift or insufficient Dependabot provenance checks; they should close only after this replacement merges.