bugfix/timeout-related-issues #8

tksadik92 · 2026-05-02T15:49:44Z

tksadik92 commented

2026-05-02 15:49:44 +00:00

No description provided.

tksadik92 added 3 commits 2026-05-02 15:49:44 +00:00

fix: set httpx read timeout for long-running commands and handle 4a7db8e204

non-JSON error responses
- Set per-request httpx timeout (command timeout + 10s buffer) in
  Commands.run() and AsyncCommands.run() for foreground exec calls,
  preventing HTTP read timeouts on long-running commands
- Raise WrennInternalError instead of raw httpx.HTTPStatusError when
  handle_response() encounters a non-JSON error body (e.g. 502 from
  a reverse proxy)

Fix error handling, resource leaks, and logic bugs across the SDK 04e5dc652f

Bugs fixed:
- files.py: use typed error checking (_raise_for_status) instead of raw
  raise_for_status(), ensuring WrennNotFoundError etc. are raised
  correctly
- exceptions.py: check both "capsule_ids" and "sandbox_ids" response
  keys
  for backwards compatibility
- code_interpreter: retry _ensure_kernel on 5xx errors (only fail on
  4xx),
  remove redundant TimeoutError in bare except, clean up non-standard
  top-level msg_id/msg_type from Jupyter messages

Resource leaks fixed:
- capsule.py: close WrennClient if capsule creation or init fails
- code_interpreter: add close()/__del__ for _proxy_client cleanup when
  not using context manager

Logic fixes:
- pty.py: yield exit events to callers instead of silently discarding
  them
- capsule.py: auto-resume paused capsules in wait_ready instead of
  failing
- capsule.py: log warnings on destroy failure in __exit__ instead of
  silently swallowing errors

Merge issues fixed

ci/woodpecker/pr/check Pipeline was successful

Details

06b4a8cbcb

tksadik92 closed this pull request

2026-05-02 15:54:44 +00:00

ci/woodpecker/pr/check Pipeline was successful

Details

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: wrenn/python-sdk#8