Skip to content

fix(ci): allow security check reporting - #26

Merged
m-szymanska merged 1 commit into
mainfrom
agent/fix-security-audit-permissions
Aug 23, 2026
Merged

fix(ci): allow security check reporting#26
m-szymanska merged 1 commit into
mainfrom
agent/fix-security-audit-permissions

Conversation

@m-szymanska

Copy link
Copy Markdown
Member

Summary

  • grant the Security Audit job read access to repository contents
  • grant rustsec/audit-check permission to create its Check Run
  • keep all other workflow jobs and token scopes unchanged

Root cause

The post-merge main run completed cargo audit with zero vulnerabilities, then failed while creating a Check Run because the repository defaults GITHUB_TOKEN to read-only.

Validation

  • actionlint .github/workflows/ci.yml
  • cargo fmt --check
  • cargo +1.94 clippy --all-targets --all-features -- -D warnings
  • cargo test (hermetic Semgrep test double)

Copilot AI lite review requested due to automatic review settings August 23, 2026 21:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the CI “Security Audit” job failing to publish results by granting only that job the minimum additional GITHUB_TOKEN permissions needed to create a GitHub Check Run, while keeping the rest of the workflow unchanged.

Changes:

  • Add job-scoped permissions to the security job in .github/workflows/ci.yml.
  • Grant contents: read (for checkout) and checks: write (to publish the audit check run) specifically for the Security Audit job.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@m-szymanska
m-szymanska merged commit 598bd93 into main Aug 23, 2026
15 checks passed
@m-szymanska
m-szymanska deleted the agent/fix-security-audit-permissions branch August 23, 2026 22:12
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.

2 participants