mcp: metastore-scope discovery, short service names, and bulk --all - #674
Open
nachill-databricks wants to merge 1 commit into
Open
nachill-databricks wants to merge 1 commit into
nachill-databricks wants to merge 1 commit into
Conversation
Three MCP-configuration improvements: - Discovery: list_all_mcp_services now uses the mcp-services metastore scope (parent omitted) -- one paginated call returning every service the caller can access -- instead of walking catalogs -> schemas -> services. The walk couldn't enumerate a large metastore (thousands of catalogs) within its budget, so the picker surfaced nothing but the system.ai list. Removes the now-unused walk helpers. - Naming: UC MCP services register under their bare service id (github) rather than the full dashed path (system-ai-github), so the agent tool prefix is mcp__github__. The full UC name stays in the entry URL, so loading is unaffected. Leaf names are collision-safe: services sharing an id across schemas, or clashing with a non-service entry, keep the full dashed path, and the picker matches either form for back-compat with existing configs. - Bulk: `configure mcp --all` (and `mcp add --all`) registers every accessible MCP service with no picker; the onboarding "Configure MCP servers..." prompt now uses it and points to `configure mcp` for manual selection. --all targets the agents configured with ucode (not every installed MCP CLI) and reports the registered count, surfacing the listing-failure reason when empty. Co-authored-by: Isaac <no-reply@databricks.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Three related MCP fixes for
ucode:timed out on large workspaces and surfaced only
system.ai.*) with a singlemetastore-scoped
/api/2.1/unity-catalog/mcp-servicescall (emptyparent).Returns every service the caller can access, permission-filtered server-side.
(
github, notsystem-ai-github), with collision fallback to the full dashedpath. Full UC name is preserved in the URL.
--all— install every MCP the user has permission to in one step;onboarding offers this,
ug configure mcpstays the granular picker.This pull request and its description were written by Isaac.