Repository navigation
Guard workflow syntax and exact pushed refs before research publication - #4548
Clear1nsight wants to merge 6 commits into
Conversation
…ext-20261019 Fix P112 workflow parse failure caused by runner context in job env
Pin actionlint 1.7.12 with release checksums, lint all workflows on push/PR, and test P112 expression regression, reusable/local action metadata, non-checked-out push refs, Git replacement refs and linked worktrees. Document reviewed tooling overlays without changing scientific pins. Prepared with OpenAI assistance.
|
Independent review (James Reed, with Claude Code). Head Verdict: the workflow-validation tooling is sound and works against today's main. The branch now has a merge conflict and one small installer robustness issue. Checks
Issues
The opt-in hook is documented and not auto-installed, which matches AGENTS.md's no-surprise-tooling stance. Merge or close decisions are for maintainers. |
|
Readiness audit: this PR remains a draft at head The current installer still uses the fixed Independent finding: #4548 (comment) |
Malformed workflows could reach research branches before GitHub parsed them. This adds checksum-pinned actionlint 1.7.12, an exact-commit validator, a standalone workflow-lint job and an opt-in pre-push hook that checks every pushed non-deletion ref, including non-checked-out branches and annotated tags. The hook is documented and is not installed automatically.
The upstream planning run also exposed four selected-result navigation failures. This revision synchronizes the tooling branch with upstream main
f626db5c3a3d781d8f7695038776c621651f75e1, updates the CI guide to the selected PR #4359 replay, and places the status history banner before the first historical checkpoint. The selected result remains conditional κ = 0.0023.Source-preserving CI integration
Documentation changes select four registered packages. This revision reuses the authenticated historical replay adapter, source-role repairs and six regression tests from PR #4542, exact predecessor head
af86fb28f4fe039d4bb8419850af99a6fc474587. It also corrects the adapter's fork-only publication reference to canonical upstream introductiond11655a9fd1c7eedc52e0e9720158bd17d29192a, preserving the identical manifest blob. A seventh adapter test requires the publication pin to be reachable from HEAD, so a locally cached fork object cannot hide missing upstream history.All original invalidation paths remain declared. No frozen verifier, mathematical source, receipt, source hash or selected-result record is rewritten. No global planner bypass, assertion weakening or path-filter suppression is added.
Validation
Final repair head:
ae19b6868d5981fbacdf6950a226d969e603b78d.The final-head workflow-lint run and registered-package run report hosted validation. Read outcomes at this exact head. Workflow validation and scoped package replay do not establish a new κ or a complete multiplication theorem. This remains a draft tooling contribution.
Prepared for Clear1nsight with substantial OpenAI Codex assistance. Apache-2.0.