Skip to content

Reject mutable GitHub Action references - #26

Closed
biru-codeastromer wants to merge 2 commits into
mainfrom
codex/immutable-action-audit
Closed

Reject mutable GitHub Action references#26
biru-codeastromer wants to merge 2 commits into
mainfrom
codex/immutable-action-audit

Conversation

@biru-codeastromer

@biru-codeastromer biru-codeastromer commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • scan GitHub workflows and composite action definitions during the repository audit
  • accept local actions and external actions pinned to a full lowercase 40-character commit SHA
  • reject mutable tags, branches, short SHAs, dynamic expressions, and container tags with file-and-line diagnostics
  • document the enforcement in contribution and security policy

Evidence

  • make release-gate: 102 passed, 1 optional skip, 91.83% coverage
  • eight focused valid/adversarial action-reference cases pass
  • the current workflow set passes the new scanner
  • repository audit, text policy, clean wheel install, dependency audit, example workflow, and Gitleaks passed

This PR is independent of #22-#25 and is ready for maintainer review.

@biru-codeastromer

biru-codeastromer commented Aug 23, 2026

Copy link
Copy Markdown
Contributor Author

The YAML-format and nested-action blockers are fixed at the new head. The audit now uses a safe YAML syntax tree, recognizes spaced and quoted uses keys plus flow mappings, rejects malformed and non-scalar references, and discovers action.yml or action.yaml in any tracked directory. Exact local gate: 112 passed, 1 optional skip, 91.83% coverage, repository/dependency/text audits, clean installs, example, and full-history Gitleaks all green.

@biru-codeastromer

Copy link
Copy Markdown
Contributor Author

Superseded by merged #41, which preserves this change in the protected v0.4.0 release stack.

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.

3 participants