fix(cli): show scan scope before verified review progress - #464
fix(cli): show scan scope before verified review progress#464mldangelo-oai wants to merge 3 commits into
Conversation
|
@codex review |
|
Codex Review: Didn't find any major issues. Chef's kiss. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
Security review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
|
@codex review Please review the current head, |
|
Codex Review: Didn't find any major issues. Chef's kiss. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
Security review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
|
@codex review |
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
Security review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
Summary
Show the number of files in scope until verified review progress becomes available. This addresses the inventory-before-progress portion of #70.
Changes
N in scopewhen inventory is known but no reviewed files have been verified.completed / total reviewedonce a positive completed count is available.Testing
d3dad7ec: 38 passed, 196 assertions.pnpm run types,pnpm run format, andgit diff --check: passed.check:package, and separate installed-package smoke: passed. The archive contains 222 entries and 106 bundled plugin files.e7d07822each passed 1,293 tests with 11 skips and zero failures. Those results are historical; a new full suite was not run for this unchanged main integration.Risk and rollout
Presentation-only dashboard change. No new telemetry, estimated percentages, scan-state tracking, or phase/activity behavior. Broader progress concerns in #70 remain out of scope. Bundled-plugin and npm versions are unchanged. Preserve the current draft state and recheck current main before merge.
Public disclosure review
Newly authored material was reviewed for public disclosure. Existing automated review comments contain access-restricted report references; maintainer cleanup remains outstanding.