Remove Google MCP entries from small-business plugin#240
Open
localden wants to merge 1 commit into
Open
Conversation
The Google-hosted MCP endpoints (gmail, calendar, drive) require an OAuth client_id at connection time and do not support dynamic client registration. The .mcp.json schema currently has no way to provide a client_id for these endpoints, so the connection cannot succeed from this plugin. This brings small-business in line with the other plugins in this repo, which had the same entries removed in #184. Users can connect Gmail, Calendar, and Drive through the Connectors Directory in the app, which handles the authentication separately.
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
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.
What
Removes the
gmail,google calendar, andgoogle driveMCP server entries fromsmall-business/.mcp.json.Why
The Google-hosted MCP endpoints at
*.googleapis.comuse Google's OAuth server, which does not support dynamic client registration. Connecting requires a pre-registered OAuth client_id, and there is currently no client_id available for this plugin to use, so connections from this plugin to these endpoints cannot complete.This brings
small-businessin line with the other plugins in this repo, which had the same entries removed in #184 for the same reason. Thesmall-businessplugin was added after #184 and reintroduced them.Users can still connect Gmail, Calendar, and Drive through the Connectors Directory in the app, which is a separate connection path with its own authentication handling and is unaffected by this change.
Effect
On merge, the marketplace sync re-publishes the plugin and Desktop picks it up on next launch. Removed entries are pruned from the user's connector list automatically. No client release needed.
Follow-up
Restoring these entries would require a registered OAuth client suitable for local-loopback use plus a way to specify it in
.mcp.json. That is tracked separately.