Skip to content
Discussion options

You must be logged in to vote

Thanks for the detailed writeup, @aman210122.

Path A, gated on a pydantic-ai release (not just the merge).

Quick correction on the upstream status: pydantic/pydantic-ai#4036 is no longer in draft. It's open, ~3,400 lines, 42 commits, 63 review comments, assigned to @DouweM, last activity Apr 6. So the "blocked on upstream timing" framing for Path B is weaker than the proposal suggests. The work is happening, the bottleneck is review on that PR.

Why Path A:

  1. common.ai's other model hooks (Azure, Bedrock, Vertex) all wrap upstream pydantic-ai Models rather than fork them. A local subclass of OpenAIChatModel here would duplicate a 3.4k-line PR that's still under active review, and we'd inher…

Replies: 2 comments 1 reply

Comment options

You must be logged in to vote
0 replies
Answer selected by aman210122
Comment options

You must be logged in to vote
1 reply
@aayushchou-db
Comment options

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Category
Ideas
Labels
None yet
3 participants