Skip to content

Modernize SARIF contextual training for CodeQL v4 - #255

Open
felickz wants to merge 3 commits into
SecureCodeWarrior:v1from
forks-felickz:modernize-sarif-enrichment
Open

felickz wants to merge 3 commits into
SecureCodeWarrior:v1from
forks-felickz:modernize-sarif-enrichment

Conversation

@felickz

@felickz felickz commented Sep 3, 2026

Copy link
Copy Markdown

Summary

Modernize SARIF contextual training for current CodeQL and code scanning behavior.

  • preserve the CodeQL tool name by default, with renameCodeQLTool: true for the legacy compatibility workaround
  • add IntegrationId=github while retaining the existing github-sarif-action:<owner> partner tracking
  • add optional Secure Code Warrior languageKey support
  • cache, serialize, throttle, timeout, and retry Direct Linking API requests
  • make enrichment idempotent and recover missing entries after partial API failures
  • suppress phrase matches that duplicate CWE training destinations
  • resolve SARIF rules by component identity, ID, index, name, and GUID
  • update the runtime to Node 24 and document the CodeQL v4 post-processing workflow
  • expand unit coverage and modernize CI actions

Motivation

The existing action still works with CodeQL v4, but currently renames the tool to GitHub CodeQL. Live ingestion testing confirmed current code scanning retains enriched help.markdown while the tool remains CodeQL, allowing CodeQL-specific integrations to remain intact.

Validation

The published action was tested unchanged before implementing these updates, confirming the existing workflow and providing a comparison for tool-name and duplicate-training behavior.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: bb61d8a3-7594-46a2-960a-683f06d1992a
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: bb61d8a3-7594-46a2-960a-683f06d1992a
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: bb61d8a3-7594-46a2-960a-683f06d1992a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant