Skip to content

feat: allow web tools to discover plugin-registered backends - #33902

Closed
zhanglingfei112 wants to merge 1 commit into
NousResearch:mainfrom
zhanglingfei112:feat/web-plugin-backend-detection
Closed

feat: allow web tools to discover plugin-registered backends#33902
zhanglingfei112 wants to merge 1 commit into
NousResearch:mainfrom
zhanglingfei112:feat/web-plugin-backend-detection

Conversation

@zhanglingfei112

Copy link
Copy Markdown

_is_backend_available() only checks hardcoded backends. Plugin-registered backends via web_search_registry were never detected. Add a fallback check so custom web plugins work.

@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/tools Tool registry, model_tools, toolsets comp/plugins Plugin system and bundled plugins tool/web Web search and extraction duplicate This issue or pull request already exists labels May 28, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #31829 — same fix adding registry fallback in _is_backend_available() for plugin-registered web search backends. Also competes with #33516, #31887, #28652. Fixes #31873 / #28651.

@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the focused plugin-registry fallback contribution. This is now implemented on main through a broader, tested fix, so this PR is superseded.

Automated hermes-sweeper review.

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

Labels

comp/plugins Plugin system and bundled plugins comp/tools Tool registry, model_tools, toolsets duplicate This issue or pull request already exists P3 Low — cosmetic, nice to have sweeper:implemented-on-main Sweeper: behavior already present on current main tool/web Web search and extraction type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants