Skip to content

ci: add Actions security/quality audit (actionlint + zizmor) (#143) - #214

Merged
Chris-Wolfgang merged 2 commits into
vNextfrom
ci/actions-audit
Jul 21, 2026
Merged

ci: add Actions security/quality audit (actionlint + zizmor) (#143)#214
Chris-Wolfgang merged 2 commits into
vNextfrom
ci/actions-audit

Conversation

@Chris-Wolfgang

Copy link
Copy Markdown
Owner

Stacked on #213. Adds actions-audit.yaml:

  • actionlint — static checker for workflow YAML + embedded shell (via shellcheck). Gates the PR on findings.
  • zizmor — Actions security auditor (injection, over-broad permissions:, unpinned actions). Uploads SARIF to Code Scanning; report-only for now (|| true), promote to a gate once the baseline is clean.

Runs on every PR (not path-filtered, so it can be a required check without hanging unrelated PRs). Ported from D20-Dice.

Notes / risk

  • First run will reveal any pre-existing actionlint findings in the repo's other workflows (this PR is the first to enable it). My newly-added workflows (sourcelink-stepinto, aot-smoke) use quoted shell vars, so shellcheck should pass; if actionlint flags anything pre-existing I'll fix it in this PR.
  • All actions SHA-pinned (canonical checkout@9c091bb v7), so zizmor's unpinned-uses won't fire.
  • AC follow-up (not blocking): promote zizmor to gating + add a documented .zizmor.yml suppression baseline once the first run is clean.

Closes #143 when the vNext cycle merges to main.

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings July 21, 2026 18:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Comment thread .github/workflows/actions-audit.yaml Fixed
Comment thread .github/workflows/actions-audit.yaml Fixed
Comment thread .github/workflows/actions-audit.yaml Fixed
Chris-Wolfgang and others added 2 commits July 21, 2026 16:30
actionlint gates workflow YAML + embedded shell (via shellcheck); zizmor
audits for injection / over-broad permissions / unpinned actions and uploads
SARIF to Code Scanning (report-only, promote to gate once baseline clean).
Runs on every PR. Ported from D20-Dice.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
#143)

Aligns with the fleet SHA-pin migration (#219): checkout v7.0.0 -> v7,
setup-dotnet v5 -> v6, upload-artifact v4 -> v7, upload-sarif v3/v4 -> v4.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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