Skip to content

ref(api): establish transport boundary for API client - #641

Merged
Abhijeet Prasad (AbhiPrasad) merged 1 commit into
mainfrom
abhi-ref-639-prepare-api-client-transport
Aug 4, 2026
Merged

ref(api): establish transport boundary for API client#641
Abhijeet Prasad (AbhiPrasad) merged 1 commit into
mainfrom
abhi-ref-639-prepare-api-client-transport

Conversation

@AbhiPrasad

Copy link
Copy Markdown
Member

Braintrust HTTP mechanics currently live inside logger.py, coupling transport to logging and leaving no isolated foundation for a resource-oriented API client. Establish this boundary first so retries, structured errors, and endpoint services can be centralized in follow-up work without mixing those behavior changes into the initial refactor.

Braintrust HTTP mechanics currently live inside logger.py, coupling transport to logging and leaving no isolated foundation for a resource-oriented API client. Establish this boundary first so retries, structured errors, and endpoint services can be centralized in follow-up work without mixing those behavior changes into the initial refactor.
@AbhiPrasad
Abhijeet Prasad (AbhiPrasad) merged commit 625a5e7 into main Aug 4, 2026
82 checks passed
@AbhiPrasad
Abhijeet Prasad (AbhiPrasad) deleted the abhi-ref-639-prepare-api-client-transport branch August 4, 2026 16:39
Abhijeet Prasad (AbhiPrasad) added a commit that referenced this pull request Aug 4, 2026
builds on top of
#641

Add operation-level retry policies, bounded status and transport
retries, and structured API errors while leaving legacy HTTPConnection
call sites unchanged. Include real-socket coverage for retry safety,
Retry-After handling, deadlines, custom transports, and session
lifecycle.

Next, add BraintrustClient bootstrap, endpoint routing, and the resource
service facade on top of this transport.
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