Skip to content

ci: add trial CodeQL workflow - #10344

Open
drousselbot wants to merge 1 commit into
NousResearch:mainfrom
drousselbot:ci/codeql-trial-clean
Open

ci: add trial CodeQL workflow#10344
drousselbot wants to merge 1 commit into
NousResearch:mainfrom
drousselbot:ci/codeql-trial-clean

Conversation

@drousselbot

Copy link
Copy Markdown

Summary

  • add a trial CodeQL workflow for Python and JavaScript/TypeScript
  • pin CodeQL and checkout actions to exact SHAs
  • validate CodeQL on a PR before deciding whether to require it or proceed with Renovate

Validation

  • YAML parsed locally
  • action refs verified as SHA-pinned

Notes

  • This is intentionally a trial rollout per our current workflow discussion.
  • Renovate should wait until CodeQL is confirmed working cleanly.

@alt-glitch alt-glitch added type/security Security vulnerability or hardening P3 Low — cosmetic, nice to have labels Apr 26, 2026

@teknium1 teknium1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for adding a narrowly scoped, SHA-pinned CodeQL trial. The current main workflow tree has no CodeQL workflow; this addition is therefore not superseded.

Problems

  • .github/workflows/codeql.yml:31 pins actions/checkout to v4, while current workflows use the v6.0.2 pin, including .github/workflows/ci.yml:48. Please align this new workflow with the repository-wide pin.

Suggested changes

  • Use actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 at .github/workflows/codeql.yml:31.

Automated hermes-sweeper review.

matrix:
language:
- python
- javascript-typescript

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please align this with the repository-wide checkout pin: current workflows, including .github/workflows/ci.yml:48, use de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 rather than the older v4 reference.

@teknium1 teknium1 added sweeper:risk-automation Sweeper risk: may affect CI, automerge, label sync, or maintainer automation sweeper:risk-security-boundary Sweeper risk: may affect sandboxing, auth, credentials, or sensitive data sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform labels Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform sweeper:risk-automation Sweeper risk: may affect CI, automerge, label sync, or maintainer automation sweeper:risk-security-boundary Sweeper risk: may affect sandboxing, auth, credentials, or sensitive data type/security Security vulnerability or hardening

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants