fix: SearXNG search provider improvements and bug fixes#930
fix: SearXNG search provider improvements and bug fixes#930norci wants to merge 9 commits intoRightNow-AI:mainfrom
Conversation
…dd runtime integration
- Add list_searxng_categories tool to discover SearXNG categories dynamically - Merge searxng/SKILL.md into web-search/SKILL.md - Remove hardcoded category table, use tool instead - Delete searxng/SKILL.md (no longer needed)
The page parameter was always hardcoded to 1 and never passed from the upper layer. Remove it to simplify the method signature.
Add list_searxng_categories to all agents that have web_search in their tools list, so they can discover available SearXNG categories.
After removing searxng skill, update all references to the bundled skills count in documentation and test assertions.
Moved the SearXNG Search section after Search Techniques so it appears earlier in the file. Condensed the file to 1996 chars.
|
PR #920 was created by the agent without my permission and has some flaws. This is the complete implementation. I have run it on my local machine and the agent can now correctly use searxng for searches. |
CI Status UpdateFixed
Pre-existing failures (not related to this PR)The following CI checks are failing on the main branch as well and are unrelated to the SearXNG changes in this PR:
Recent main branch merges (#753, #877, #881) all have the same CI failures, confirming these are pre-existing issues that should be addressed in a separate PR (e.g., upgrading |
jaberjaber23
left a comment
There was a problem hiding this comment.
LGTM. Good simplification — removes per-request category validation (users can use ! prefix syntax), adds list_searxng_categories tool, consolidates skill docs. Please rebase on main.
|
This PR has merge conflicts. Please rebase onto the latest main branch and resolve conflicts so we can merge. |
Related
Follow-up to #920
Changes
Bug Fixes
../bundled/searxng/SKILL.mdpageparameter fromsearch_searxngFeatures
list_searxng_categoriestool to agent tool listsDocumentation
Refactoring
Configuration
The SearXNG configuration is simple — only
urlis required:Note: Unlike other providers (Brave, Tavily), SearXNG does not have a
max_resultsfield. The docs inconfiguration.mdare already correct.Files Changed
39 files, +1245/-346 lines