Skip to content

feat: improve SARIF memory usage (signed) - #393

Open
cx-diogo-rocha wants to merge 9 commits into
masterfrom
AST-140881-dhl-improve-memory-usage-signed
Open

feat: improve SARIF memory usage (signed)#393
cx-diogo-rocha wants to merge 9 commits into
masterfrom
AST-140881-dhl-improve-memory-usage-signed

Conversation

@cx-diogo-rocha

@cx-diogo-rocha cx-diogo-rocha commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Closes #

Proposed Changes

  1. Reduced the amount of time the report was held in memory for Sarif report. Now only in memory once (this can be reduced later with a deeper refactor).
  2. Added a flag to disable console report, since this was another point in the code where the full report was duplicated in memory. We should later stream to console as well even with the flag absent.

Replaces #386, having all commits signed

Checklist

  • I covered my changes with tests.
  • I Updated the documentation that is affected by my changes:
    • Change in the CLI arguments
    • Change in the configuration file

I submit this contribution under the Apache-2.0 license.

@cx-diogo-rocha
cx-diogo-rocha requested a review from a team as a code owner August 5, 2026 14:22
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