fix(cli): classify broken pipes as user aborts - #559
Conversation
Co-authored-by: Blaine Kasten <blainekasten@gmail.com>
Broly Security ScanWarning
Warning Summary 1 actionable finding(s) in this PR
All actionable items are in the table below.
Dismiss false positivesImportant Secrets cannot be dismissed with a checkbox. Broly decides whether a credential is real or a test fixture using AI, so overriding that verdict means adding evidence it did not have — reply with
Note Re-scan this PR anytime with
|
Summary
The daily CLI failure export contained two current-version
endpoints eventsfailures that completed all API requests and then reported only status1. Rich 14.2 convertsBrokenPipeErrortoSystemExit(1), which made expected early pipe closure look like a CLI defect.Verification
TOGETHER_TELEMETRY_DISABLED=1 uv run pytest tests/cli/test_command_telemetry.py tests/cli/test_console_theme.py -n 0(24 passed)./scripts/lint