Skip to content

feat: add Webhound MCP server - #1401

Open
mfkhalil wants to merge 2 commits into
stacklok:mainfrom
mfkhalil:codex/add-webhound-server
Open

feat: add Webhound MCP server#1401
mfkhalil wants to merge 2 commits into
stacklok:mainfrom
mfkhalil:codex/add-webhound-server

Conversation

@mfkhalil

Copy link
Copy Markdown

Summary

  • add Webhound as a remote Streamable HTTP MCP server
  • configure per-user OAuth with dynamic client registration and PKCE; no API key, bearer token, client secret, or shared publisher credential is included
  • list the complete 30-tool surface and document the dollar-budget research contract, terminal done=true rule, reports, datasets, and evidence packs

Testing

  • official catalog validator: all 111 entries valid
  • official catalog build passed
  • catalog Go test suite passed
  • current ToolHive OAuth / DCR / PKCE source tests passed
  • the same production endpoint completed a live DCR / PKCE account flow in Goose, discovered all 30 tools, and called webhound_account without starting research or spending credits

I did not claim a native ToolHive runtime smoke: Docker was initially stopped, and starting it automatically resumed nine unrelated pre-existing containers. I created no ToolHive workload, touched no container, and returned Docker Desktop to its prior stopped state rather than risking user workloads.

@rdimitrov

Copy link
Copy Markdown
Member

Hey @mfkhalil, nice work on the OAuth/DCR/PKCE setup, and thanks for the straight note about the runtime smoke test. Three things to fix and then I think this is good to go: the metadata block is CI-populated so it needs to come out, the entry needs a repo-hosted icon.svg in the server directory instead of the external PNG, and version says 0.4.3 while npm is on 0.5.3. One note: I couldn't verify the 30-tool list because the endpoint is OAuth-gated, so I'm taking that on trust unless you can paste a tools/list response.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@e130547). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1401   +/-   ##
=======================================
  Coverage        ?   53.72%           
=======================================
  Files           ?       18           
  Lines           ?     1424           
  Branches        ?        0           
=======================================
  Hits            ?      765           
  Misses          ?      572           
  Partials        ?       87           
Flag Coverage Δ
unittests 53.72% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants