-
Notifications
You must be signed in to change notification settings - Fork 841
Fix HostedCodeInterpreterTool with Responses #6817
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix HostedCodeInterpreterTool with Responses #6817
Conversation
When no file IDs are present, we're outputting the wrong JSON.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR fixes a JSON formatting issue in the HostedCodeInterpreterTool when no file IDs are present, ensuring proper OpenAI API compliance.
- Fixed incorrect JSON structure for code interpreter tool when container type is "auto"
- Added integration test to validate code interpreter functionality
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| OpenAIResponsesChatClient.cs | Fixed JSON structure for code interpreter tool container configuration |
| OpenAIResponseClientIntegrationTests.cs | Added test to verify code interpreter tool produces expected results |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
When no file IDs are present, we're outputting the wrong JSON.
Microsoft Reviewers: Open in CodeFlow