-
Notifications
You must be signed in to change notification settings - Fork 423
fix: Prefer OpenAI schema for ReAct and Tool Calling Agents #923
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: Prefer OpenAI schema for ReAct and Tool Calling Agents #923
Conversation
WalkthroughUpdated example README docs to replace HTTP payload examples from {"input_message": "..."} to a structured {"messages": [{"role": "user", "content": "..."}]} format in both non-streaming and streaming sections for React and tool-calling agents. No code, logic, or API declarations were changed. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (1)
🧰 Additional context used📓 Path-based instructions (3)**/README.@(md|ipynb)📄 CodeRabbit inference engine (.cursor/rules/general.mdc)
Files:
**/*⚙️ CodeRabbit configuration file
Files:
examples/**/*⚙️ CodeRabbit configuration file
Files:
🔇 Additional comments (1)
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. Comment |
Signed-off-by: Will Killian <[email protected]>
cf971e1 to
31e3684
Compare
|
/merge |
Description
Ensure that ReAct and Tool Calling examples use the OpenAI specification for input.
Closes nvbugs-5560815
By Submitting this PR I confirm:
Summary by CodeRabbit