Fix: missing Tavily config in loop.go, invalid web_search param#660
Fix: missing Tavily config in loop.go, invalid web_search param#660lxowalle merged 1 commit intosipeed:mainfrom
loop.go, invalid web_search param#660Conversation
|
Adds the missing config and reverts the invalid param I had set during testing. Tested and added screens in Evidence I missed adding all the commit from my test branch when raising the last PR. |
nikolasdehor
left a comment
There was a problem hiding this comment.
Two straightforward fixes:
-
Tavily config fields were missing from the tool registration in
loop.go, meaning the Tavily API key and settings were never wired through. Good catch. -
"include_raw_content": "false"was sending a JSON string"false"instead of the booleanfalse. The Tavily API likely rejected or misinterpreted it.
LGTM.
|
Thank you for the timely fix.@ulolol |
|
Apologies once again, and thanks for merging! |
|
@ulolol Thanks for plugging the missing Tavily config in loop.go and reverting the invalid web_search param. Without that config entry users would hit confusing errors trying to use Tavily search. We have the PicoClaw Dev Group on Discord for contributors to connect and collaborate. If you're interested, send an email to |
📝 Description
This PR adds the missing config and reverts the invalid web_search param changed during testing
🗣️ Type of Change
🤖 AI Code Generation
🔗 Related Issue
Fixes #654
📚 Technical Context (Skip for Docs)
🧪 Test Environment
📸 Evidence (Optional)
Click to view Logs/Screenshots
☑️ Checklist