Repository navigation
[Draft] Add Lannis plugin (remote MCP) - #932
Draft
cursor[bot] wants to merge 1 commit into
Draft
cursor[bot] wants to merge 1 commit into
cursor[bot] wants to merge 1 commit into
Conversation
Co-authored-by: Danny Jiang <djiang-jq@users.noreply.github.com>
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: Missing Grok Bot server placement
- Added placement "server" to the lannis hosted HTTP server in mcp.json and the matching README example so Grok Bot dials the hosted path and can show the connect card.
Or push these changes by commenting:
@cursor push 3144ff35e6
Preview (3144ff35e6)
diff --git a/third_party/lannis/README.md b/third_party/lannis/README.md
--- a/third_party/lannis/README.md
+++ b/third_party/lannis/README.md
@@ -19,7 +19,8 @@
"mcpServers": {
"lannis": {
"type": "http",
- "url": "https://mcp.lannis.app/mcp"
+ "url": "https://mcp.lannis.app/mcp",
+ "placement": "server"
}
}
}
diff --git a/third_party/lannis/mcp.json b/third_party/lannis/mcp.json
--- a/third_party/lannis/mcp.json
+++ b/third_party/lannis/mcp.json
@@ -2,7 +2,8 @@
"mcpServers": {
"lannis": {
"type": "http",
- "url": "https://mcp.lannis.app/mcp"
+ "url": "https://mcp.lannis.app/mcp",
+ "placement": "server"
}
}
}You can send follow-ups to the cloud agent here.
Reviewed by Cursor Bugbot for commit 85ce47a. Configure here.
| "lannis": { | ||
| "type": "http", | ||
| "url": "https://mcp.lannis.app/mcp" | ||
| } |
Author
There was a problem hiding this comment.
Missing Grok Bot server placement
Medium Severity · Logic Bug
The lannis server omits placement server. Grok Bot is a listed install target, and without that field the connector stays UNDECIDED and never shows a connect or sign-in card.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 85ce47a. 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.



Adds
third_party/lannis/for Lannis — "Facturá y seguí tus impuestos" (issue invoices and track your taxes in Argentina).Contact
plugin_url
mcp.json)Changes
third_party/lannis/mcp.jsonwith thelannishttp server;.cursor-plugin/plugin.jsonreferences./mcp.json(same shape as Add PitchBook third-party MCP plugin #796 PitchBook)..cursor-plugin/marketplace.jsonand one row in the root README integrations table..claude-plugin/marketplace.jsonis unchanged.node scripts/validate-plugins.mjspasses.Logo — needs follow-up
No image file is committed.
https://lannis.app/icon-512.png(submitted logo_url), the favicon, www/mcp/docs subdomains all failed the TLS handshake from the build VM, and the web fetcher blocks lannis.app. There is no verified GitHub org:Lannis-appis an empty, unlinked org, so its avatar was not used.plugin.jsonlogois set to the absolute submitted URLhttps://lannis.app/icon-512.png; replace it withassets/logo.pngonce the file can be downloaded.Testing instructions (as submitted; test-account credentials redacted because this repo is public — see the original submission)
POSITIVE TESTS
NEGATIVE TESTS
Note
Medium Risk
Unreviewed third-party remote MCP with OAuth and tools that can issue invoices; marketplace exposure depends on server trust and user approval flows.
Overview
Adds a new Lannis third-party integration so agents can connect to Lannis’s hosted MCP at
https://mcp.lannis.app/mcpfor Argentine invoicing and tax workflows (list/preview/issue invoices, tax obligations, VEP payment slips), with OAuth sign-in and no local API keys.The PR introduces
third_party/lannis/(manifest,mcp.json, README, CHANGELOG, MIT LICENSE) and registerslannisin.cursor-plugin/marketplace.jsonand the root README integrations table. The listing author is Lannis (not Cursor).plugin.jsonuses a remote logo URL because the image could not be fetched at add time; there is no committedassets/logo.png.The PR description marks this as an unreviewed form submission—worth verifying the remote server, publisher, and invoice-issuance behavior before merge.
Reviewed by Cursor Bugbot for commit 85ce47a. Bugbot is set up for automated code reviews on this repo. Configure here.