Skip to content

Draft: Add registry format converters and support for the Official MCP Registry #2469

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Draft: Add registry format converters and support for the Official MCP Registry #2469

wants to merge 10 commits into from

Conversation

@rdimitrov
Copy link
Member

@rdimitrov rdimitrov commented Nov 5, 2025

The following PR:

  • Adds converter functions for going back and forth between the 2 formats (toolhive's current format and the upstream one). This is temporary as those should live under toolhive-registry and not toolhive.
  • Implements the registry provider interface by adding support for using upstream API compliant registries. This allows users to discover and run servers they already published upstream (or in another compliant registry)
  • Fixes a few issues discovered while implementing this, i.e. we were not reseting the default registry when the config was changed.

github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 5, 2025
View reviewed changes
pkg/config/registry.go
openapiURL += "/"
}
openapiURL += "openapi.yaml"
_, err = registryClient.Get(openapiURL)

Check failure

Code scanning / CodeQL

Uncontrolled data used in network request Critical

The
URL
Loading
of this request depends on a
user-provided value
Loading
.
@rdimitrov
Initial PoC of supporting the official MCP registry
349b2fd 
Signed-off-by: Radoslav Dimitrov <[email protected]>
@rdimitrov
Temporarily copy the converters from toolhive-registry
613ccd0 
Signed-off-by: Radoslav Dimitrov <[email protected]>
@rdimitrov
Complete the initial implementation
b4abda4 
Signed-off-by: Radoslav Dimitrov <[email protected]>
@rdimitrov
Support setting the registry API via REST
8cb1639 
Signed-off-by: Radoslav Dimitrov <[email protected]>
@rdimitrov
Move the whole converters package from toolhive-registry
210a92f 
Signed-off-by: Radoslav Dimitrov <[email protected]>
@rdimitrov
Fix the imports in the tests
f14597d 
Signed-off-by: Radoslav Dimitrov <[email protected]>
@rdimitrov
Config changes now take effect immediately without a restart
63c2a70 
Signed-off-by: Radoslav Dimitrov <[email protected]>
@rdimitrov
Fix a small issue with converting the names
7aee598 
Signed-off-by: Radoslav Dimitrov <[email protected]>
@rdimitrov
Normalise the full name instead of simplifying it
1c52d7a 
Signed-off-by: Radoslav Dimitrov <[email protected]>
@rdimitrov
Fix imports after rebase - update to use pkg/registry/types
02c7c3c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rdimitrov