Skip to content

feat(ci): add Codex Security release qualification - #3087

Merged
alangou merged 3 commits into
mainfrom
2989-codex-security-release-qualification/ag
Sep 1, 2026
Merged

feat(ci): add Codex Security release qualification#3087
alangou merged 3 commits into
mainfrom
2989-codex-security-release-qualification/ag

Conversation

@alangou

@alangou alangou commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Add release-scoped Codex Security qualification for pre-release candidates. Each run analyzes the cumulative diff from the previous stable release through NVIDIA-hosted GPT-5.6 Sol inference and publishes train-scoped SARIF to Code Scanning.

Related Issue

Part of #2989.

This PR implements cumulative release-train scans and SARIF publication. Periodic full scans and deduplicated private draft GitHub Security Advisories remain follow-up work.

Changes

  • Add a reusable, manually dispatchable workflow triggered by vX.Y.Z-pre.N tags
  • Resolve and validate the previous-stable-to-candidate range, including explicit first-release bootstrap
  • Run Codex Security through the NVIDIA Responses endpoint while keeping raw output ephemeral
  • Publish SARIF under a stable category per release train and prevent pre-release tags from entering the stable release workflow
  • Add release-range tests and synchronize CI and architecture documentation

Operational prerequisite: the repository secret CODEX_SECURITY_API_KEY must contain a key authorized for openai/openai/gpt-5.6-sol on the NVIDIA inference endpoint.

Testing

  • mise run pre-commit passes
  • mise run test passes
  • mise run test:codex-security-release-range passes
  • Actionlint passes for .github/workflows/codex-security.yml
  • Zizmor 1.29 reports no high-severity findings for .github/workflows/codex-security.yml
  • mise run markdown:lint passes
  • Codex Security dry-run accepts the NVIDIA provider, endpoint, model, effort, and release range
  • E2E tests added/updated (not applicable to this CI-only change)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated

Scan cumulative release-train diffs through NVIDIA inference and publish findings to Code Scanning.

Signed-off-by: alangou <alangou@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Sep 1, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Comment thread .github/workflows/codex-security.yml Fixed
Prevent cache poisoning in the tag-triggered Codex Security workflow.

Signed-off-by: alangou <alangou@nvidia.com>
Remove custom inference cost accounting so the workflow remains focused on scanning and SARIF publication.

Signed-off-by: alangou <alangou@nvidia.com>
@alangou

alangou commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

/ok to test 325efd2

@alangou
alangou added this pull request to the merge queue Sep 1, 2026
Merged via the queue into main with commit f7180c0 Sep 1, 2026
56 checks passed
@alangou
alangou deleted the 2989-codex-security-release-qualification/ag branch September 1, 2026 14:06
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.

3 participants