Skip to content

Honor JSON byte budgets for audit SARIF - #5011

Merged
Widthdom merged 5 commits into
mainfrom
fix-issue4903
Jul 31, 2026
Merged

Honor JSON byte budgets for audit SARIF#5011
Widthdom merged 5 commits into
mainfrom
fix-issue4903

Conversation

@Widthdom

@Widthdom Widthdom commented Jul 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • preflight complete recipe/audit SARIF against the exact UTF-8 stdout budget
  • omit only whole results with safe recovery metadata and no partial SARIF
  • document and test exact boundaries, Unicode, oversized/empty/multi-query runs, help, and completion

Root cause

Audit/recipe SARIF streamed directly to stdout and the generic byte-limit validation rejected SARIF, so no complete-document preflight or safe truncation contract existed.

Validation

Documentation and changelog

  • updated README.md, DEVELOPER_GUIDE.md, and TESTING_GUIDE.md in English and Japanese
  • added changelog.d/unreleased/4903.fixed.md

Related: #4982
Related: #5012

Fixes #4903

@Widthdom
Widthdom marked this pull request as ready for review July 31, 2026 13:58
@Widthdom
Widthdom marked this pull request as draft July 31, 2026 14:14
@Widthdom
Widthdom marked this pull request as ready for review July 31, 2026 14:16
@Widthdom
Widthdom merged commit cd1afe9 into main Jul 31, 2026
12 checks passed
@Widthdom
Widthdom deleted the fix-issue4903 branch July 31, 2026 14:17
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.

Add a byte budget to audit SARIF output

1 participant