Skip to content

Add SARIF 2.1.0 findings export#264

Merged
JE-Chen merged 1 commit into
devfrom
feat/sarif-batch
Jun 20, 2026
Merged

Add SARIF 2.1.0 findings export#264
JE-Chen merged 1 commit into
devfrom
feat/sarif-batch

Conversation

@JE-Chen

@JE-Chen JE-Chen commented Jun 20, 2026

Copy link
Copy Markdown
Member

Interop batch (from multi-agent web research; verified absent). Full layers + tests + EN/Zh v56 docs + README.

Feature (utils/sarif, pure-stdlib json+hashlib)

  • to_sarif / write_sarif: build a SARIF 2.1.0 document from normalized findings ({rule_id, level, message, file?, line?}) — auto rule catalog + stable partialFingerprints for cross-run dedupe — so action-lint / secrets-scan / WCAG-audit / guardrail results land in GitHub/Azure DevOps code scanning as line-anchored alerts. make_finding builder; from_lint_issues (index/severity/code/message) and from_audit_findings (sc/criterion/kind/severity) adapt existing producers. Severity → SARIF error/warning/note.
  • Executor AC_export_sarif (findings list or JSON string, optional path/tool_name); MCP ac_export_sarif; Builder under Report.

Verification

  • 11 tests pass (document shape, no-location, fingerprint stable/distinct, explicit rules, from_lint + negative-index, from_audit, write_sarif, executor round-trip, wiring, facade); ruff clean; radon no CC≥C; bandit clean; PySide6-free.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 48 complexity · 0 duplication

Metric Results
Complexity 48
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@JE-Chen JE-Chen merged commit 7ec1a09 into dev Jun 20, 2026
16 checks passed
@JE-Chen JE-Chen deleted the feat/sarif-batch branch June 20, 2026 15:29
@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant