Repository navigation
feat(quickbooks-online): declare Intuit's HTTP MCP server (reland of #642) - #3
Draft
djiang-jq wants to merge 1 commit into
Conversation
Relands cursor#642, which was reverted by cursor#643. The listing is gating-only again after the revert: it carries no server definition, so an install produces no mcp_server row and native MCP has nothing to point at. Declare the hosted HTTP endpoint the way Square does, so the marketplace index exposes a type === "HTTP" server for the connector. Restores the 1.1.0 tree exactly as it was published, so the version keeps meaning the same content. Co-authored-by: Danny Jiang <[email protected]>
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.
Relands cursor/plugins#642, which was reverted 19 minutes after merge by cursor/plugins#643.
Heads up before merging
The revert gave no stated reason. cursor#643 is titled only "Reverts cursor#642", with no comment explaining what went wrong. This PR restores the reverted change byte-for-byte, so if something about cursor#642 caused the rollback, this reintroduces it. The most likely candidate is the endpoint — see the review note below.
What
Rebuilt on top of current upstream
main(f713f14, which includes both the merge and the revert). The listing is gating-only again after the revert: it carries nomcp.json, so an install produces nomcp_serverrow and native MCP has nothing to point at. This declares the HTTP server in the same shape asthird_party/square/mcp.json.third_party/quickbooks-online/mcp.jsonthird_party/quickbooks-online/.cursor-plugin/plugin.json"mcpServers": "./mcp.json", version1.0.0→1.1.0third_party/quickbooks-online/README.mdthird_party/quickbooks-online/CHANGELOG.md1.1.0entry{ "mcpServers": { "quickbooks-online": { "type": "http", "url": "https://mcp-e2e.quickbooks.intuit.com/mcp" } } }The resulting tree is byte-identical to what was merged as 1.1.0 in cursor#642 (verified with
git diffagainstf506741). Produced by reverting the revert commited4397c, not by rebasing the old branch — the original commits are already in upstream history via the cursor#642 merge, so a rebase would have silently dropped them.On the version number
This keeps
1.1.0rather than bumping to1.1.1, so the version→content mapping stays consistent:1.1.0has only ever meant this exact tree. Clients on the reverted1.0.0see1.1.0as an upgrade and pick upmcp.json; clients that fetched1.1.0during the 19-minute window already have this content.Worth knowing: Bugbot flagged on cursor#643 that the revert's downgrade to
1.0.0may not reach installs that already took1.1.0, and its autofix proposed1.1.1. That fix was never landed, so upstreammainis at1.0.0today. If you would rather force a re-index than rely on the version comparison, say so and I'll bump this to1.1.1with a matching changelog entry.Why
Once this ships and the marketplace re-indexes, installs get an
mcp_serverrow withtype === "HTTP". That row is whatquickbooksOnlineNativeMcpCatalogOverride(everysphere #1132191) reads whenmerge_connector_settings.connectors["quickbooks-online"].nativeMcpis true.Review note
The endpoint is
mcp-e2e.quickbooks.intuit.com. The-e2elabel reads like a test/staging host for a publicly shipped listing. Both that host andmcp.quickbooks.intuit.comresolve in DNS, and TLS is blocked from the agent VM's egress, so it could not be verified from here. Bugbot raised the same point on cursor#642 ("the configured URL uses an-e2ehost that may be non-production"). If this is what triggered the revert, that is the line to change.This PR's base
The real target is
cursor/pluginsmain; this agent has no write access there (Permission to cursor/plugins.git denied to cursor[bot]), so the PR lives on thedjiang-jq/pluginsfork. Its base iscursor/upstream-main-base-reland-fbc4, pinned atcursor/plugins@f713f14, so the diff shows the 4 files that changed rather than the fork'smaindrift.To open it upstream — the same path cursor#642 took:
https://github.com/cursor/plugins/compare/main...djiang-jq:plugins:cursor/quickbooks-online-native-mcp-reland-fbc4?expand=1
Validation
node scripts/validate-plugins.mjspasses against current upstreammainwith this change applied.