fix(sdk): lazily load optional gRPC transport - #647
simonyang08 wants to merge 1 commit into
Conversation
|
@ppt5928-dot is attempting to deploy a commit to the Raphael's projects Team on Vercel. A member of the Team first needs to authorize it. |
dae12af to
61077ca
Compare
|
Closing this draft as part of clearing the queue; it has been sitting in draft state without updates since Sep 2. To be clear, the underlying problem is real and still present on develop: grpcio is only an optional extra in pyproject, yet sdk/openagents/sdk/transports/init.py imports the gRPC transport eagerly, so the non-gRPC transports are unimportable without it. If you'd like to carry this forward, please open a fresh non-draft PR rebased on current develop with the lazy-export approach from this branch (the getattr shim plus the direct .transports.grpc imports in topology.py) and we'll review it promptly. Thanks for the thorough validation notes here. |
Summary
grpciodependency.This PR addresses the current SDK optional-dependency contract on
develop. Issue #563 described an obsolete Docker image path and is not claimed as the active bug basis.Validation
developbaseline reproduces an import failure withoutgrpcio.pip install openagents[sdk]guidance.compileallpasses for all changed files.AI disclosure
This change was prepared with OpenAI Codex assistance and reviewed and validated locally by the contributor.