Repository navigation
Conversation
The listing was gating-only: it carried no server definition, so an install produced no mcp_server row and native MCP had 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. Bumps the plugin to 1.1.0 and updates the README and CHANGELOG, which both documented the absent mcp.json as deliberate. Co-authored-by: Danny Jiang <[email protected]>
Co-authored-by: Danny Jiang <[email protected]>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.
Reviewed by Cursor Bugbot for commit f506741. Configure here.
| "mcpServers": { | ||
| "quickbooks-online": { | ||
| "type": "http", | ||
| "url": "https://mcp-e2e.quickbooks.intuit.com/mcp" |
There was a problem hiding this comment.
Staging MCP URL used in production
High Severity · Logic Bug
mcp.json points at mcp-e2e.quickbooks.intuit.com, Intuit's end-to-end test host, not the documented production MCP at mcp.quickbooks.intuit.com. Marketplace installs would hit a non-production environment instead of live QuickBooks Online.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit f506741. Configure here.
This was referenced Oct 16, 2026
Draft
djiang-jq
added a commit
that referenced
this pull request
Oct 16, 2026
Relands #642, which was reverted by #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 was referenced Oct 16, 2026
Draft
djiang-jq
added a commit
that referenced
this pull request
Oct 17, 2026
Relands #650, which was reverted by #651, which itself relanded #642, reverted by #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 unchanged, including the mcp-e2e host that Bugbot flagged as non-production on #650. That host is carried over deliberately rather than silently corrected, since it is the value the paired everysphere override expects. 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
2 participants
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.
Note
Medium Risk
Changes integration wiring and OAuth client behavior for a finance connector, requiring reauthorization for existing users; the configured URL uses an
-e2ehost that may be non-production.Overview
QuickBooks Online moves from a Cursor-only MCP server to Intuit’s hosted HTTP endpoint, matching how other integration plugins ship
mcp.json.The plugin is bumped to 1.1.0 and wires
mcpServersto a newmcp.jsonthat points athttps://mcp-e2e.quickbooks.intuit.com/mcp. README and CHANGELOG now describe Intuit as the server host instead of a server with no local definition.OAuth is still Cursor-managed and per company file, but the client is now the native OAuth client, so existing installs must reauthorize after upgrading.
Reviewed by Cursor Bugbot for commit f506741. Bugbot is set up for automated code reviews on this repo. Configure here.