Skip to content

[codex] Deprecate all-checks and tighten operator wording#61

Merged
TacoRocket merged 2 commits intomainfrom
review/all-checks-deprecation-tone
Apr 6, 2026
Merged

[codex] Deprecate all-checks and tighten operator wording#61
TacoRocket merged 2 commits intomainfrom
review/all-checks-deprecation-tone

Conversation

@TacoRocket
Copy link
Copy Markdown
Owner

What changed

  • marked all-checks and broad all-checks --section ... sweeps as deprecated across help, README, and runtime messaging
  • tightened operator-facing wording so partial visibility is described in terms of what current credentials do and do not show
  • updated CLI/help/UX tests to lock the new wording and deprecation surfaces in place

Why

The current product direction is to move grouped operator workflows toward chains while keeping flat commands available for narrower runs. This slice makes that direction explicit now and cleans up older wording that drifted into collector-oriented or tool-blaming language.

Impact

Operators now see clearer deprecation guidance around all-checks, and the visible text surfaces use more direct attacker-side language about current credential visibility.

Validation

  • python3 -m pytest tests/test_help.py tests/test_cli_smoke.py tests/test_terminal_ux.py -q
  • python3 -m ruff check src/azurefox/help.py tests/test_help.py tests/test_terminal_ux.py
  • ACCR: no implementation-vs-plan drift found

@TacoRocket TacoRocket marked this pull request as ready for review April 6, 2026 06:46
@TacoRocket TacoRocket merged commit ba4c68b into main Apr 6, 2026
4 checks passed
@TacoRocket TacoRocket deleted the review/all-checks-deprecation-tone branch April 6, 2026 06:46
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.

1 participant