Skip to content

fix(python-sdk): log silent sidecar cleanup failures#1279

Open
realfishsam wants to merge 1 commit into
mainfrom
fix/empty-catch-logging-1256-1260
Open

fix(python-sdk): log silent sidecar cleanup failures#1279
realfishsam wants to merge 1 commit into
mainfrom
fix/empty-catch-logging-1256-1260

Conversation

@realfishsam

@realfishsam realfishsam commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replace silent pass handlers in the Python SDK sidecar manager with debug logging.
  • Add debug logging when API error body parsing falls back to the original exception string.

Fixes #1256
Fixes #1257
Fixes #1258
Fixes #1259
Fixes #1260

Test Plan

  • python3 -m py_compile sdks/python/pmxt/server_manager.py sdks/python/pmxt/client.py
  • git diff --check

Notes

  • This is a focused Python SDK observability fix; no generated client or API surface changes are intended.

@realfishsam

realfishsam commented Jun 22, 2026

Copy link
Copy Markdown
Contributor Author

Focused validation passed locally:

  • python3 -m py_compile sdks/python/pmxt/server_manager.py sdks/python/pmxt/client.py
  • git diff --check

The current red generated-sync checks are unrelated repo-wide drift: regenerators want broad hosted-client/API reference/COMPLIANCE changes (for example removing hosted-mode dispatch sections from generated clients and rewriting Hunch/API reference docs). I am intentionally keeping this PR scoped to the Python SDK empty-catch diagnostics fix rather than folding that unrelated generated churn into it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment