Skip to content

chore: remove remote::passthrough inference provider - #6368

Merged
mattf merged 1 commit into
ogx-ai:mainfrom
mattf:remove-passthrough-inference-provider
Jul 30, 2026
Merged

chore: remove remote::passthrough inference provider#6368
mattf merged 1 commit into
ogx-ai:mainfrom
mattf:remove-passthrough-inference-provider

Conversation

@mattf

@mattf mattf commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Remove the remote::passthrough provider, its registry entry, and all supporting code: the passthrough module (passthrough.py, config.py, init.py), the forward_headers utility module, the unit test suite, and documentation files.

Update the codex CLI test to use a generic api_key instead of passthrough_api_key since X-OGX-Provider-Data forwarding is a generic mechanism not tied to any one provider.

Remove the remote::passthrough provider, its registry entry, and all
supporting code: the passthrough module (passthrough.py, config.py,
__init__.py), the forward_headers utility module, the unit test suite,
and documentation files.

Update the codex CLI test to use a generic api_key instead of
passthrough_api_key since X-OGX-Provider-Data forwarding is a generic
mechanism not tied to any one provider.

Signed-off-by: Matthew Farrellee <matt@cs.wisc.edu>
@mattf
mattf force-pushed the remove-passthrough-inference-provider branch from 3ed758c to e6b4ef1 Compare July 29, 2026 15:32
@mattf

mattf commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator Author

@NickGagan @franciscojavierarceo do you have users for this?

@mattf
mattf marked this pull request as ready for review July 30, 2026 12:00
@mattf
mattf added this pull request to the merge queue Jul 30, 2026
@mattf

mattf commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator Author

we can revert if this becomes an issue

Merged via the queue into ogx-ai:main with commit a7de599 Jul 30, 2026
56 checks passed
@mattf
mattf deleted the remove-passthrough-inference-provider branch July 30, 2026 15:06
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