Skip to content

fix(scan report): render readable text - #1601

Open
John-David Dalton (jdalton) wants to merge 1 commit into
v1.xfrom
codex/ask302-v1x
Open

John-David Dalton (jdalton) wants to merge 1 commit into
v1.xfrom
codex/ask302-v1x

Conversation

@jdalton

@jdalton John-David Dalton (jdalton) commented Oct 11, 2026 •

Copy link
Copy Markdown
Collaborator

LLM Description written by Codex:GPT-6

Summary

Render the default socket scan report output as readable plain text for terminal and CI logs. Keep JSON, Markdown, short output, error handling, and exit status behavior intact. Share alert flattening between plain text and Markdown renderers.

Validation

  • Before the large-report regression was added, the focused unit file passed all 9 cases when run as SOCKET_SHIM_ACTIVE_PNPM=1 pnpm run test:unit src/commands/scan/output-scan-report-text.test.mts. This prefix bypassed the managed wrapper's token and SFW launcher. The normal wrapper rejected its configured token, so I have not rerun locally. CI is pending for the 200,000-alert regression.
  • CI lint identified no-control-regex in the original ANSI check. The check now uses a string/codepoint test; CI must verify the fix.
  • git diff --check passed.
  • pnpm run check:tsc:src reported an unrelated existing error at src/commands/optimize/update-manifest-by-agent.mts:140: EditablePackageJson.fromJSON is missing.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant