Skip to content

fix(cli): emit recursive JSON reports to stdout - #467

Open
rioyu123 wants to merge 2 commits into
NVIDIA:mainfrom
rioyu123:codex/fix-recursive-json-stdout
Open

fix(cli): emit recursive JSON reports to stdout#467
rioyu123 wants to merge 2 commits into
NVIDIA:mainfrom
rioyu123:codex/fix-recursive-json-stdout

Conversation

@rioyu123

@rioyu123 rioyu123 commented Sep 1, 2026

Copy link
Copy Markdown

Summary

  • emit the existing recursive combined JSON document to stdout when --format json is used without --output
  • route recursive human-readable status, progress, scores, summaries, verbose output, and scan advisories to stderr so stdout remains machine-readable
  • preserve the existing JSON file schema, output bounds, exit codes, and terminal/file behavior
  • retain a parseable combined JSON document when an individual child scan fails

Fixes #449.

Verification

  • uv run --extra dev pytest tests/unit/test_cli.py -q (109 passed, 2 skipped)
  • uv run --extra dev ruff check src/ tests/
  • uv run --extra dev ruff format --check src/ tests/
  • regression coverage parses JSON stdout for verbose recursive scans, wrapper advisories, and child-scan failures
  • full Windows unit selection before the final CLI-only follow-up: 3944 passed, 26 skipped, 4 xfailed; 23 pre-existing platform failures remain in CRLF, symlink/mkfifo, secure-open, runtime-isolation, and release-script tests

Signed-off-by: Rio Yu <52408936+rioyu123@users.noreply.github.com>
Signed-off-by: Rio Yu <52408936+rioyu123@users.noreply.github.com>
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.

[Feature Request] Skillspector recursive to support JSON output

1 participant