Skip to content

ci/quality: Enable guideline recategorization in CodeQL - #894

Merged
castler merged 3 commits into
mainfrom
js_guidelines_recategorization
Aug 12, 2026
Merged

ci/quality: Enable guideline recategorization in CodeQL#894
castler merged 3 commits into
mainfrom
js_guidelines_recategorization

Conversation

@castler

@castler castler commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Apply guideline recategorization to CodeQL SARIF before compliance reporting.
  • Add the Bazel-wrapped recategorizer, schemas, configuration, dependencies, and integration test.
  • Pin jsonpath-ng to 1.7.0 for compatibility with the recategorizer path conversion.

Validation

  • bazel test //quality/static_analysis:codeql_lint_test
  • Full CodeQL run for //score/message_passing and //score/mw/com completed successfully.

Comment thread quality/static_analysis/codeql_lint_test.py Fixed
castler and others added 3 commits August 11, 2026 11:42
Adds the upstream MISRA guideline recategorization helper as a Bazel-wrapped
value and calls it from `quality/static_analysis/codeql_lint.py` before
producing the final SARIF and compliance reports.

The change:

- Adds a dedicated `guideline_recategorize` target in the vendored
  `codeql_coding_standards` BUILD file.
- Wires `codeql_lint` to invoke that tool with the repository
  `coding-standards.yaml` before running `analysis_report`.
- Keeps the dashboard/docs aligned with the generated recategorization
  report.
- Adds a focused unit test for the SARIF recategorization helper.
- Updates the pip lock for the helper's runtime dependencies.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Pin jsonpath-ng to 1.7.0 because the recategorizer is incompatible with the path formatting introduced by 1.8.0. Regenerate the lockfile, including the jsonpath-ng runtime dependency ply.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The recategorization integration is validated by the full CodeQL run; remove the standalone test and its Bazel target.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@castler
castler added this pull request to the merge queue Aug 12, 2026
Merged via the queue into main with commit e8bd94e Aug 12, 2026
13 checks passed
@castler
castler deleted the js_guidelines_recategorization branch August 12, 2026 09:56
@github-project-automation github-project-automation Bot moved this from Backlog to Done in COM - Communication FT Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants