Skip to content

fix(openai): Improve reasoning parameter handling#1001

Merged
Ayush8923 merged 4 commits into
mainfrom
chore/openai-mapper
Jul 3, 2026
Merged

fix(openai): Improve reasoning parameter handling#1001
Ayush8923 merged 4 commits into
mainfrom
chore/openai-mapper

Conversation

@vprashrex

@vprashrex vprashrex commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Issue: #1002

Summary

  • Before: The handling of the reasoning parameter in OpenAI mapping was inconsistent.
  • Now: The reasoning parameter is updated and handled more effectively in the OpenAI mapping.
  • Required updates to the parameter handling logic.
  • Adjustments to maintain compatibility with existing mapping functionality.

Checklist

Before submitting a pull request, please ensure that you mark these tasks.

  • Ran fastapi run --reload app/main.py or docker compose up in the repository root and test.
  • If you've fixed a bug or added code that is tested and has test cases.

@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • ready-for-review

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: b6c0a04a-55a8-4768-b7ea-8440e549371a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/openai-mapper

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot changed the title fix(mappers): Update reasoning parameter handling in OpenAI mapping fix(openai): Improve reasoning parameter handling Jul 3, 2026
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

OpenAPI changes   ⚪ No API surface changes

Note

This PR does not modify the API contract.

main6053fa4e · generated by oasdiff

@Ayush8923 Ayush8923 self-requested a review July 3, 2026 11:43
@vprashrex vprashrex linked an issue Jul 3, 2026 that may be closed by this pull request
@codecov

codecov Bot commented Jul 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Ayush8923 Ayush8923 merged commit a147a4a into main Jul 3, 2026
4 checks passed
@Ayush8923 Ayush8923 deleted the chore/openai-mapper branch July 3, 2026 12:35
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 1.3.0-main.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OpenAI: Fix effort level handling

2 participants