Repository navigation
Add Skydropx plugin (third-party submission, unreviewed) - #1026
Open
cursor[bot] wants to merge 1 commit into
Open
cursor[bot] wants to merge 1 commit into
cursor[bot] wants to merge 1 commit into
Conversation
Co-authored-by: Danny Jiang <[email protected]>
Author
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Grok Bot missing server placement
- Added "placement": "server" to the Skydropx HTTP MCP server so Grok Bot can show a connect card instead of leaving the row UNDECIDED.
Or push these changes by commenting:
@cursor push 2631acab7f
Preview (2631acab7f)
diff --git a/third_party/skydropx/mcp.json b/third_party/skydropx/mcp.json
--- a/third_party/skydropx/mcp.json
+++ b/third_party/skydropx/mcp.json
@@ -2,7 +2,8 @@
"mcpServers": {
"skydropx": {
"type": "http",
- "url": "https://mcp.skydropx.com"
+ "url": "https://mcp.skydropx.com",
+ "placement": "server"
}
}
}You can send follow-ups to the cloud agent here.
Reviewed by Cursor Bugbot for commit 1fd2505. Configure here.
| "skydropx": { | ||
| "type": "http", | ||
| "url": "https://mcp.skydropx.com" | ||
| } |
Author
There was a problem hiding this comment.
Grok Bot missing server placement
High Severity · Logic Bug
The Skydropx server object has no "placement": "server", but this listing targets Grok Bot. Without that field Grok Bot leaves the row UNDECIDED and never shows a connect or sign-in card, so the integration is unusable there.
Reviewed by Cursor Bugbot for commit 1fd2505. Configure here.
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.



This is an unreviewed third-party plugin submission. Please convert this PR to a draft; the automation that opened it has no draft option. Do not merge until someone has reviewed it.
Summary
Adds
third_party/skydropx/, a remote MCP plugin for Skydropx: quote, buy, and track shipments across Mexico's main carriers, print labels, schedule pickups, and manage store orders.mcp.jsonwith theskydropxserver (type: http,url: https://mcp.skydropx.com), referenced from.cursor-plugin/plugin.jsonas"mcpServers": "./mcp.json"(same layout as PitchBook in Add PitchBook third-party MCP plugin #796).cursor-plugin/plugin.json,README.md,CHANGELOG.md,LICENSE,assets/logo.png.cursor-plugin/marketplace.jsonand one row in the root README integrations table..claude-plugin/marketplace.jsonis unchanged.Submission
skydropxURL to check:
plugin_urlishttps://mcp.skydropx.com(no path), andmcp.jsonuses it exactly as submitted. The testing instructions say to add the connector athttps://mcp.skydropx.com/mcp. A reviewer should confirm which endpoint is right and updatemcp.jsonand the README if it should be/mcp.Testing instructions (as submitted)
Logo
The submitted
logo_url(https://ps.w.org/skydropx/assets/icon-128x128.png) could not be fetched from the build sandbox (TLS connection blocked), and neither could skydropx.com or its favicon. The committedassets/logo.pngis the avatar of the officialSkydropxGitHub organization (https://avatars.githubusercontent.com/skydropx, 420×420 PNG; the org lists https://skydropx.com as its site). A reviewer may want to swap in the submitted WordPress icon.Verification
node scripts/validate-plugins.mjspasses.tools/list.Note
Medium Risk
New third-party remote MCP with OAuth and write actions (label purchases charged to the user’s Skydropx balance); repo changes are manifests/docs only, but the listed endpoint wasn’t probed in CI.
Overview
Adds a new Skydropx third-party integration under
third_party/skydropx/, wiring agents to Skydropx’s hosted HTTP MCP athttps://mcp.skydropx.comviamcp.jsonandplugin.json(OAuth with dynamic client registration; no local API keys).The change also registers the plugin in the marketplace (
.cursor-plugin/marketplace.json) and documents it in the root README integrations table, with install/README copy covering rates, labels, tracking, pickups, and store orders.This is an unreviewed third-party submission; reviewers should confirm the correct MCP base URL (submission tests mention
…/mcpwhile the committed config uses the host root) and exercise OAuth plus label purchase flows before merge.Reviewed by Cursor Bugbot for commit 1fd2505. Bugbot is set up for automated code reviews on this repo. Configure here.