Implement Phase 2 items and Phase 3 task 14 & task 17 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. |
…sion handling Signed-off-by: rafsanneloy <rafsanneloy@gmail.com>
Signed-off-by: rafsanneloy <rafsanneloy@gmail.com>
plan.mdplan.md
plan.mdplan.md
|
Looks good! |
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. also implemented confirmation prompts for destructive commands and version handling.
Related issue