Skip to content

ci: scan the workflows with CodeQL and default every workflow to read-only - #61

Merged
revtex merged 1 commit into
mainfrom
dev
Sep 29, 2026
Merged

revtex merged 1 commit into
mainfrom
dev

Conversation

@revtex

@revtex revtex commented Sep 29, 2026

Copy link
Copy Markdown
Owner

Pull Request

Summary

CI only. Closes the three actions/missing-workflow-permissions code-scanning alerts on ci.yml.

The alerts came from GitHub's default CodeQL setup in April. ci.yml was given permissions: contents: read the same day (3f8cb6f), but the repo then moved to codeql.yml, which never analysed the actions language, so nothing re-scanned the workflows and the alerts stayed open.

  • codeql.yml now analyses actions as well as Go and TypeScript. On dev it ran with 0 results.
  • codeql.yml and docker.yml default to contents: read at the top, like the other workflows, so any job added later starts read-only. Every existing job already declared its own permissions.

Checklist

  • Tests pass locally (go test ./..., pnpm test), and CI is green on dev
  • docs/ updated if user-facing behavior changed (not user-facing)
  • CHANGELOG.md: skipped with the skip-changelog label; this is a CI-only change
  • No new secrets, tokens, or keys in logs or error messages

🤖 Generated with Claude Code

…-only

Three actions/missing-workflow-permissions alerts on ci.yml have been open
since April. ci.yml was fixed the same day (3f8cb6f), but they came from
GitHub's default setup, which was later replaced by codeql.yml, and that
never analysed the actions language, so nothing re-scanned the workflows
and the alerts could not close. codeql.yml now includes actions.

codeql.yml and docker.yml set permissions only per job; both now default
to contents: read at the top, like ci.yml, release.yml and
ghcr-cleanup.yml, so a job added later starts read-only.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@revtex revtex added the skip-changelog PR has no user-visible impact; skip CHANGELOG check label Sep 29, 2026
@revtex
revtex merged commit d201fab into main Sep 29, 2026
14 checks passed
@revtex
revtex deleted the dev branch September 29, 2026 04:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog PR has no user-visible impact; skip CHANGELOG check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant