Implement Phase 2 items and Phase 3 task 14 from plan.md#49
Conversation
Signed-off-by: rafsanneloy <rafsanneloy@gmail.com>
|
Thanks, Rafsan! This looks close: pre-commit and unit tests are passing, but the capture test is failing because the expected CLI capture was not updated for an intentional JSONL output change. The failing command is:
It now outputs JSONL like this: but Could you update |
Yes, I'll update it. I've marked it as a draft PR for the remaining works. After that it will be ready for review asap. |
Contributor checklist
pre-commit run --all-filesand/orpytestas appropriate (see Tests in the README). A clean Ruff/pre-commit run is necessary but not sufficient—behavior still needs tests and review.capture_tests/expected_captures.txt--json, and--jsonloutput modes (or noted why not applicable)print()calls for user-visible outputType of change
Description
Implemented improved API request handling by adding a default HTTP timeout, strengthening connection and timeout error handling with clearer user feedback, and extending JSON output support across multiple commands.
Related issue