Skip to content

Conversation

@yt3trees
Copy link
Contributor

@yt3trees yt3trees commented Mar 8, 2025

Context

This PR is a modification to support the use of the o3-mini model in Azure OpenAI.

Previously, an error occurred when using the "OpenAI Compatible" API Provider with the "o3-mini" model selected and stream enabled. This fix resolves the issue and ensures it functions correctly.

Related issue


Important

Adds support for o3-mini model in OpenAiHandler, including streaming and non-streaming message handling.

  • Behavior:
    • Adds support for o3-mini model in OpenAiHandler in openai.ts.
    • Introduces handleO3FamilyMessage() to process o3-mini messages, supporting both streaming and non-streaming modes.
    • Fixes issue with streaming enabled for o3-mini model.
  • Functions:
    • Adds handleO3FamilyMessage() and handleStreamResponse() in openai.ts to manage o3-mini model responses.
    • Modifies createMessage() to route o3-mini model requests to handleO3FamilyMessage().
  • Misc:
    • Updates logic to check for o3-mini model in createMessage() function.

This description was created by Ellipsis for 9246cf8. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Mar 8, 2025

🦋 Changeset detected

Latest commit: a186537

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
roo-cline Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Mar 8, 2025
@dosubot dosubot bot added the bug Something isn't working label Mar 8, 2025
Copy link
Collaborator

@mrubens mrubens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 8, 2025
@mrubens mrubens merged commit 73f4350 into RooCodeInc:main Mar 8, 2025
13 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Mar 8, 2025
@yt3trees yt3trees deleted the roocode-openaicompat-o3-mini branch March 9, 2025 02:06
ipattis pushed a commit to ipattis/roo-code that referenced this pull request Mar 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants