Skip to content

[codex-security] Polish completed scan summary - #113

Merged
ianw-oai merged 1 commit into
mainfrom
dev/ianw/codex-security-polished-scan-summary
Jul 30, 2026
Merged

[codex-security] Polish completed scan summary#113
ianw-oai merged 1 commit into
mainfrom
dev/ianw/codex-security-polished-scan-summary

Conversation

@ianw-oai

Copy link
Copy Markdown
Collaborator

Completed security scans currently bury the report beneath repeated codex-security: prefixes and unstructured status lines.

Lead with the report path, then show aligned findings, coverage, elapsed time, token usage, cost, and results. Interactive terminals use their existing color palette for severity and underline the report path; redirected output, NO_COLOR, and TERM=dumb stay plain. Structured scan output is unchanged.

Validated with focused completion-output tests, both SDK test suites, formatting checks, and TypeScript checks.

@ianw-oai
ianw-oai marked this pull request as ready for review July 30, 2026 03:49
@ianw-oai
ianw-oai requested a review from mldangelo-oai July 30, 2026 03:49
@ianw-oai
ianw-oai merged commit 9668fc1 into main Jul 30, 2026
8 checks passed
@ianw-oai
ianw-oai deleted the dev/ianw/codex-security-polished-scan-summary branch July 30, 2026 05:34
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.

3 participants