Skip to content

ci: add zizmor security scan - #53

Closed
tanmayeepemmaraju wants to merge 1 commit into
mainfrom
zizmor/add-security-scan
Closed

tanmayeepemmaraju wants to merge 1 commit into
mainfrom
zizmor/add-security-scan

Conversation

@tanmayeepemmaraju

Copy link
Copy Markdown

Background

Add Zizmor scanning for GitHub Actions changes, matching the configuration merged in mparticle-web-sdk (mParticle/mparticle-web-sdk#1444) and mparticle-android-sdk (mParticle/mparticle-android-sdk#823).

What Has Changed

  • Add a Zizmor workflow for changes under .github/workflows/** and .github/actions/**.
  • Run it for main, with manual execution available.
  • Pin checkout and Zizmor actions to reviewed commit SHAs.
  • Deny permissions by default and grant only contents: read and actions: read to the scan job.
  • Preserve the Rokt SDK rollout behavior with continue-on-error: true, so findings appear as annotations without blocking unrelated work.

Validation

  • Workflow parsed as YAML and audited with zizmor 1.23.1: the new file reports no findings of its own.
  • Formatting matches Prettier defaults, so it passes Trunk in repos that run it.

Screenshots/Video

  • N/A

Checklist

  • Self-review completed
  • Tests added or updated
  • Tested locally

🤖 Generated with Claude Code

Adds GitHub Actions security scanning in advisory mode, matching the
configuration merged in mparticle-web-sdk#1444.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@tanmayeepemmaraju
tanmayeepemmaraju requested a review from a team as a code owner September 18, 2026 18:17
@cursor

cursor Bot commented Sep 18, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Adds a read-only, non-blocking CI security scanning workflow with minimal permissions.

Overview
Adds a new GitHub Actions workflow using zizmor to statically scan workflow and action files for security issues on pushes, pull requests, and manual triggers.

The scan runs in advisory mode with pinned action SHAs and minimal read-only permissions, surfacing findings as PR annotations without blocking builds.

Reviewed by Cursor Bugbot for commit 5c0ceb0. Bugbot is set up for automated code reviews on this repo. Configure here.

@tanmayeepemmaraju

Copy link
Copy Markdown
Author

Superseded by #54 — reopened on a ci/ branch to satisfy the semantic branch name check.

@tanmayeepemmaraju
tanmayeepemmaraju deleted the zizmor/add-security-scan branch September 18, 2026 18:24
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.

1 participant