Structured CLI summary output for analysis results#899
Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit intomainfrom Apr 30, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the datadog-static-analyzer CLI summary to report the total number of input files analyzed (rather than only files that produced findings), aligning the output with actual scan scope.
Changes:
- Static analysis summary now reports
files_to_analyze.len()as the file count. - Secrets summary now reports
secrets_files.len()as the file count. - Removes the prior “total files” computation derived from results (which excluded empty/no-finding files).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
🎯 Code Coverage (details) 🔗 Commit SHA: 1f99a38 | Docs | Datadog PR Page | Give us feedback! |
Replace the single-line "Found X violation(s) in Y file(s)..." format with a structured multi-line summary that separately reports all relevant metrics: files scanned, files with findings, total findings, rules evaluated, rules with matches, and sub-second duration. This removes the ambiguity where "in Y file(s)" could mean either total files scanned or files with at least one finding, and provides both values explicitly.
MikaYuoadas
force-pushed
the
akim.sadaoui/fix-files-count-output
branch
from
f6683e5 to
1f99a38
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
MikaYuoadas
changed the title
jasonforal
approved these changes
Apr 28, 2026
gh-worker-dd-devflow-36fce6
Bot
added
mergequeue-status: queued
mergequeue-status: in_progress
and removed
mergequeue-status: queued
labels
gh-worker-dd-mergequeue-cf854d
Bot
merged commit 8876cf0
into
main
92 of 94 checks passed
gh-worker-dd-mergequeue-cf854d
Bot
deleted the
akim.sadaoui/fix-files-count-output
branch
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Found X violation(s) in Y file(s)...) were ambiguous: the file count only reflected files with findings, not total files scannedOld output format
New output format
Test plan