Skip to content

ci(security): add Muninn and remediate scan findings#713

Open
sg0nzalez wants to merge 2 commits into
huggingface:mainfrom
sg0nzalez:feat/add-muninn
Open

ci(security): add Muninn and remediate scan findings#713
sg0nzalez wants to merge 2 commits into
huggingface:mainfrom
sg0nzalez:feat/add-muninn

Conversation

@sg0nzalez

@sg0nzalez sg0nzalez commented Jul 21, 2026

Copy link
Copy Markdown

Summary

  • Add Muninn v0.3.6 as a CI security gate (SARIF + PR comments), with muninn.yml loaded from the trusted PR base SHA so suppressions cannot be widened from a fork PR
  • Fix findings Muninn reported on this repo:
    • Add least-privilege permissions: contents: read on documentation build workflows
    • Guard the Hugging Face doc-builder workflow_run upload so it only runs after a successful PR docs build (and keep the required trigger with an explicit zizmor ignore)
    • Upgrade quiz lockfile packages flagged by OSV/Trivy (tornado, aiohttp, requests, urllib3, filelock, pygments, jupyter-core, idna, pyarrow)
  • Local verification: muninn --fail-on info exits 0 (only remaining finding is the expected poutine unverified-creator note for skaldlab/muninn, fingerprint-suppressed)

Test plan

  • Confirm the new Security / Muninn workflow runs on this PR and posts a comment / SARIF upload
  • Confirm documentation build workflows still trigger for PRs and main
  • Spot-check quiz/uv.lock still resolves (uv sync in quiz/) if maintainers use that environment

sg0nzalez and others added 2 commits July 21, 2026 14:34
Wire Muninn v0.3.5 into CI, tighten documentation workflow permissions,
guard the doc-builder workflow_run upload, and bump quiz lockfile packages
flagged by OSV/Trivy (tornado, aiohttp, requests, urllib3, and others).

Co-authored-by: Cursor <cursoragent@cursor.com>
Update the commit-SHA pin to Muninn v0.3.6 for current scanner images.

Co-authored-by: Cursor <cursoragent@cursor.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.

1 participant