The stale `src/cleveragents/cli/output.py` file (391 lines) was added by
this PR alongside the existing `output/` package. Python silently uses the
package, leaving output.py permanently unreachable and at 0% coverage,
which dragged the project total below the 96.5% threshold.
Additionally, the json/yaml/plain format branches in
`_cli_output_manager.py` had no BDD test coverage. The existing table
scenarios also set the output format after calling display_table(), so
the non-rich branches were never exercised.
Fixes:
- Delete src/cleveragents/cli/output.py (unreachable stale module)
- Add 18 BDD scenarios covering json/yaml/plain format branches for all
display functions: success, warning, info, error panel, success panel,
table, handle_exception
- Add step defs for CLIOutputManager instance method calls
- Fix scenario ordering: output format set before display calls
ISSUES CLOSED: #10655
Resolves ImportError in cli_error_handling_steps.py by implementing the CLIOutputManager class and display_* helper functions that were referenced in BDD tests but missing from the output package.
Also fixes duplicate step definitions that conflicted with existing cli_output_formats_steps.py by renaming the ambiguous steps.
- Create centralized CLIOutputManager class for consistent error handling
- Implement unified error display with debug flag support
- Add display helpers for success, warning, info, panels, and tables
- Support all output formats (rich, color, table, plain, json, yaml)
- Stack traces only shown when --debug flag is enabled
- Add comprehensive BDD tests for error handling scenarios
- Ensure consistent styling and iconography across all CLI commands
HAL9000