Repository navigation
Guard workflow syntax and exact pushed refs before research publication - #13
Draft
Clear1nsight wants to merge 1 commit into
Draft
Clear1nsight wants to merge 1 commit into
Clear1nsight wants to merge 1 commit into
Conversation
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
Tooling-only safeguard against invalid GitHub Actions syntax/expression contexts, including the P112
runner.tempjob-level environment regression. Based on fork main155f57f1ff4d431f6ef96104fdb47d4017ac7382, which already contains the reviewed correction from #8 / CrocSwap#3885.Existing workflows, permissions, branch protection, research packages, selected-source pins, upstream archives, intake history and contributor credits are unchanged. No hook configuration is installed automatically. The new lint workflow has read-only contents permission and disables checkout credential persistence.
Verification
runner.temp, invalid YAML/expressions, reusable/local action metadata, exact pushed refs, dirty checkouts, replacement refs, linked worktrees, and symlink rejection20ecf2891d80b5acf18d980b2164fb44ea7faa90: all 139 workflows inspected; the expected P112 line 19 expression-context error is correctly rejectedgit diff --checkpassesCommands:
All 136 workflows also passed the exact-commit check at
14e598894a2c37fdf5fab7fc2f6572ce8d29bae1before the branch was published. Remote CI passed: exact-head push lint, PR lint, and registered-package planning. The planning job’s 253-test onboarding/helper suite completed successfully (242 passed, 11 skipped) and selected no packages; its scientific replay job was correctly skipped. Workflow lint is not scientific replay or theorem approval; no historical, Lean, Docker, or mathematical replay is claimed by this change. Optional external ShellCheck/Pyflakes integrations are deliberately outside this workflow-syntax gate.Credit and follow-up
Retains the reviewed P112 correction from #8 / CrocSwap#3885. Prepared with OpenAI assistance. The remaining deployment step is maintainer review/merge and explicit local hook opt-in; adoption onto existing research branches must respect their scope and owners.