Skip to content

Fixed: preserve Responses computer tool passthrough#420

Merged
luispater merged 1 commit intorouter-for-me:mainfrom
Skadli:codex/responses-computer-tool
Mar 9, 2026
Merged

Fixed: preserve Responses computer tool passthrough#420
luispater merged 1 commit intorouter-for-me:mainfrom
Skadli:codex/responses-computer-tool

Conversation

@Skadli
Copy link

@Skadli Skadli commented Mar 8, 2026

Keep the OpenAI Responses computer tool intact when normalizing requests for the GitHub Copilot executor.

This change preserves built-in computer tool definitions instead of dropping them as non-function tools, keeps explicit computer tool_choice selections unchanged, and classifies computer_call / computer_call_output items as assistant and tool turns when deriving the initiator header.

Together these adjustments allow Responses requests that use the computer tool to reach the upstream executor without losing tool metadata or switching turn ownership unexpectedly.

Keep the OpenAI Responses computer tool intact when normalizing requests for the GitHub Copilot executor.

This change preserves built-in computer tool definitions instead of dropping them as non-function tools, keeps explicit computer tool_choice selections unchanged, and classifies computer_call / computer_call_output items as assistant and tool turns when deriving the initiator header.

Together these adjustments allow Responses requests that use the computer tool to reach the upstream executor without losing tool metadata or switching turn ownership unexpectedly.
@gemini-code-assist
Copy link

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@luispater luispater merged commit 99d1ce2 into router-for-me:main Mar 9, 2026
2 checks passed
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