Skip to content

POC tool discovery proposal (dont merge)#716

Draft
maleck13 wants to merge 30 commits intoKuadrant:mainfrom
maleck13:worktree-tool-discovery-proposal
Draft

POC tool discovery proposal (dont merge)#716
maleck13 wants to merge 30 commits intoKuadrant:mainfrom
maleck13:worktree-tool-discovery-proposal

Conversation

@maleck13
Copy link
Copy Markdown
Contributor

@maleck13 maleck13 commented Apr 2, 2026

Related to #499

This is a purely a POC DO NOT MERGE

It shows what we may be able to do with categories hints and some gateway specific tools

You can clone this branch and follow the guide here to see it in action

https://github.com/maleck13/mcp-gateway/blob/c29bfe548fe6b0e8246f1f25e1755189767815ba/docs/guides/tool-discovery.md

maleck13 added 11 commits April 1, 2026 12:45
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
…roker config

Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 2, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e8d2a59c-5a81-4f3b-8531-01e0cb674c69

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

maleck13 added 15 commits April 13, 2026 08:54
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
…roker config

Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
- restaurant-server and messaging-server mock MCP servers for demo
- discovery tool threshold flag (--discovery-tool-threshold, default 10)
- fix GetConfig() missing Category/Hint fields in upstream/mcp.go
- default-hidden tools when above threshold, all visible when below
- walkthrough guide for progressive tool discovery flow

Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
@maleck13 maleck13 force-pushed the worktree-tool-discovery-proposal branch from 54ceaa9 to e0b77f1 Compare April 13, 2026 07:56
Signed-off-by: craig <cbrookes@redhat.com>
Signed-off-by: craig <cbrookes@redhat.com>
…v2 proposal

- discover_tools now respects x-authorized-tools JWT header
- select_tools validates tools against auth-filtered set
- virtual-mcp-server-v2.md: category-based routing, path routing, discovery integration
- tool-discovery.md: tightened language, auth filtering noted as TODO

Signed-off-by: craig <cbrookes@redhat.com>
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.

1 participant