Skip to content

ci: migrate CodeQL to advanced setup#190

Merged
zeevdr merged 1 commit into
mainfrom
ci/codeql-advanced
Apr 27, 2026
Merged

ci: migrate CodeQL to advanced setup#190
zeevdr merged 1 commit into
mainfrom
ci/codeql-advanced

Conversation

@zeevdr
Copy link
Copy Markdown
Member

@zeevdr zeevdr commented Apr 27, 2026

Summary

  • Add .github/workflows/codeql.yml with explicit Analyze jobs for go / actions / python
  • paths-ignore for generated code: api/centralconfig/v1/**, internal/storage/dbstore/*.gen.go
  • actions/setup-go with module cache before CodeQL init for Go runs
  • Matches default-setup's default query suite + remote threat model for findings parity

Projected ~1-2 min faster CI per PR vs default-setup (CodeQL Analyze (go) was the long pole at ~5m 40s).

Activation steps (post-merge, ordering: b)

  1. Merge this PR — workflow uploads fail while default-setup is still enabled
  2. Disable default-setup: gh api -X DELETE repos/opendecree/decree/code-scanning/default-setup (or Settings → Code scanning → CodeQL → Switch to advanced)
  3. Re-trigger workflow on main; verify uploads succeed
  4. Compare findings against the last default-setup run for parity

Branch protection currently requires only CI check — no required-status-check rename needed.

Test plan

  • PR's CodeQL run kicks off (uploads will fail this round — expected, default-setup still active)
  • After merge + default-setup disable, next run uploads cleanly
  • Findings on parity commit equivalent to last default-setup run
  • Wall-clock time reduced vs ~5m 40s baseline

Closes #183 (decree only — follow-up issues to be filed for decree-python, decree-typescript, decree-ui)

🤖 Generated with Claude Code

Replaces default-setup. Advanced unlocks paths-ignore for generated
code (api/centralconfig/v1, internal/storage/dbstore/*.gen.go) and Go
module caching across runs — projected ~1-2 min faster vs default-setup.

Matches default-setup's `default` query suite and `remote` threat model
for findings parity. Languages: actions, go, python.

Default-setup must be disabled in repo settings post-merge for this
workflow's uploads to take effect.

Refs #183

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@zeevdr zeevdr added the ci CI/Infrastructure label Apr 27, 2026
@zeevdr zeevdr added this to the Security Review milestone Apr 27, 2026
@zeevdr zeevdr added enhancement New feature or request priority: P0 Blocks alpha or release size: M Moderate — a day or two, clear scope labels Apr 27, 2026
@zeevdr zeevdr merged commit a3ab620 into main Apr 27, 2026
18 of 21 checks passed
@zeevdr zeevdr deleted the ci/codeql-advanced branch April 27, 2026 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI/Infrastructure enhancement New feature or request priority: P0 Blocks alpha or release size: M Moderate — a day or two, clear scope

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate CodeQL to advanced setup for faster CI

1 participant