Skip to content

fix(sdk): lazily load optional gRPC transport - #647

Draft
simonyang08 wants to merge 1 commit into
openagents-org:developfrom
simonyang08:codex/openagents-563
Draft

fix(sdk): lazily load optional gRPC transport#647
simonyang08 wants to merge 1 commit into
openagents-org:developfrom
simonyang08:codex/openagents-563

Conversation

@simonyang08

@simonyang08 simonyang08 commented Aug 28, 2026

Copy link
Copy Markdown

Summary

  • Keep non-gRPC SDK transports importable without the optional grpcio dependency.
  • Load gRPC exports only when callers explicitly request them.
  • Preserve the existing installation guidance when gRPC is explicitly selected.

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

  • Current develop baseline reproduces an import failure without grpcio.
  • Focused optional-gRPC regression passes after the change.
  • Explicit gRPC import still returns the existing pip install openagents[sdk] guidance.
  • Ruff passes for the changed transport module and regression test.
  • compileall passes for all changed files.

AI disclosure

This change was prepared with OpenAI Codex assistance and reviewed and validated locally by the contributor.

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

@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.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants