-
Notifications
You must be signed in to change notification settings - Fork 14
refactor(mistral): enhance example with dynamic(pattern1) multi-tool support #28
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
base: main
Are you sure you want to change the base?
refactor(mistral): enhance example with dynamic(pattern1) multi-tool support #28
Conversation
WalkthroughThe script for an agent using dynamic tool discovery was updated to clarify prompt instructions, switch the Mistral chat model from "large" to "medium", and add a new user task involving starring a GitHub repository after performing a web search. Changes
Possibly related PRs
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🔇 Additional comments (3)
✨ Finishing Touches
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
✨ No issues found! Your code is sparkling clean! ✨ Need help? Join our Discord for support! |
| "ACI_SEARCH_FUNCTIONS" | ||
| "You can use ACI_SEARCH_FUNCTIONS to find relevant functions across all apps." | ||
| "Once you have identified the functions you need to use, you can append them to the tools list and use them in future tool calls." | ||
| "Taking complete user input as parameter value, you can use ACI_SEARCH_FUNCTIONS to find relevant functions across all apps." |
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.
| "Taking complete user input as parameter value, you can use ACI_SEARCH_FUNCTIONS to find relevant functions across all apps." | |
| "Based on user intent, you can use ACI_SEARCH_FUNCTIONS to find relevant functions across all apps." |
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.
Hey, Could you please check if the changes work as expected? I’ve updated the prompt to better handle complex queries involving multiple tools.
Previously, If just "based on the user query", the search tool didn’t always return all the necessary tools then the model will output "can't ……" as the ouput.
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.
I think this was because the model couldn’t generate suitable values for the search tool, so I updated the prompt to take the full user input as the parameter value.
| "You can use ACI_SEARCH_FUNCTIONS to find relevant functions across all apps." | ||
| "Once you have identified the functions you need to use, you can append them to the tools list and use them in future tool calls." | ||
| "Taking complete user input as parameter value, you can use ACI_SEARCH_FUNCTIONS to find relevant functions across all apps." | ||
| "Once you have identified the functions you need to use, you can refresh the tools list and use them in future tool calls." |
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.
"Once you have identified the functions you need to use, you can refresh the tools list and use them in future tool calls." this should not be part of the prompt, as it is orchestrated by you (dev) not the agent/LLM directly.
Key improvements:
Note: Upgrading to paid premium models can deliver more stable performance, and the current free open model is sensitive to the prompt.
Comparison of runtime results:
OR
...
Summary by CodeRabbit