Skip to content

fix(web): discover provider plugins before web dispatch - #28202

Closed
timothykersten wants to merge 1 commit into
NousResearch:mainfrom
timothykersten:fix/web-provider-plugin-discovery
Closed

fix(web): discover provider plugins before web dispatch#28202
timothykersten wants to merge 1 commit into
NousResearch:mainfrom
timothykersten:fix/web-provider-plugin-discovery

Conversation

@timothykersten

Copy link
Copy Markdown

Summary

  • Ensure web provider plugins are discovered before search/extract/crawl registry dispatch.
  • Prevent configured plugin backends such as brave-free and firecrawl from appearing unavailable when web tools are called outside normal CLI plugin initialization.
  • Add regression coverage for direct web search dispatch.

Test plan

  • python -m pytest tests/tools/test_web_tools_config.py::TestWebSearchSchema::test_web_search_discovers_provider_plugins_before_registry_lookup -q -o 'addopts='

Operational context

This is the second local carried hotfix from Tim/Petra's production Hermes instance. Kept separate from #27680 so the SMTPS fix PR remains narrowly scoped.

@alt-glitch alt-glitch added type/bug Something isn't working comp/tools Tool registry, model_tools, toolsets tool/web Web search and extraction P2 Medium — degraded but workaround exists labels May 18, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #27700 and #27584 — all three PRs add _ensure_plugins_discovered() to web_tools.py dispatch sites. Fixes #27683/#27580.

@timothykersten

Copy link
Copy Markdown
Author

Thanks — confirmed. This PR is a duplicate of #27700 and #27584 and fixes the same underlying issues (#27683/#27580). Closing this one rather than adding noise. I’ll track the existing upstream PRs/issues instead.

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

Labels

comp/tools Tool registry, model_tools, toolsets P2 Medium — degraded but workaround exists 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.

2 participants