Skip to content

feat(mandoline-mcp): add Mandoline MCP installer module - #17

Merged
zyannes merged 7 commits into
mainfrom
t3code/add-mandoline-mcp
Oct 15, 2026
Merged

zyannes merged 7 commits into
mainfrom
t3code/add-mandoline-mcp

Conversation

@zyannes

@zyannes zyannes commented Oct 15, 2026

Copy link
Copy Markdown
Member

Summary

  • Add a mandoline-mcp Coder module that downloads the Linux x86_64 Mandoline release from GitHub onto the agent PATH
  • Expose mandoline / ["mcp"] as the client registration command and args (no wrapper launcher)
  • Include Terraform tests and a path-filtered CI workflow matching other MCP modules

Test plan

  • cd modules/mandoline-mcp && terraform init -backend=false && terraform test
  • terraform fmt -check in modules/mandoline-mcp
  • Workspace start downloads Mandoline when the GitHub release asset is available
  • Client MCP config with command: mandoline, args: ["mcp"] discovers tools

Made with Cursor

Download the Linux x86_64 Mandoline release onto the agent PATH so clients can run `mandoline mcp` as a local stdio MCP server.

Co-authored-by: Cursor <cursoragent@cursor.com>
@zyannes
zyannes requested a review from tpetr October 15, 2026 15:18
Use a mandoline state directory and install only the mandoline binary on PATH.

Co-authored-by: Cursor <cursoragent@cursor.com>
@zyannes
zyannes requested a review from wesleyjellis October 15, 2026 15:24
@wesleyjellis

Copy link
Copy Markdown
Contributor

Is this going to work with how we mount the ~/ as a volume? You might need to put this in /opt

@zyannes

zyannes commented Oct 15, 2026

Copy link
Copy Markdown
Member Author

Is this going to work with how we mount the ~/ as a volume? You might need to put this in /opt

Good call! Updating. Also changing the state_dir to /tmp

zyannes and others added 2 commits October 15, 2026 11:39
Keep the binary off the mounted HOME volume; use /tmp only for the download extract.

Co-authored-by: Cursor <cursoragent@cursor.com>
Stage the archive via mktemp -d, install the binary to /opt/bin/mandoline, and prepend /opt/bin to PATH.

Co-authored-by: Cursor <cursoragent@cursor.com>

@wesleyjellis wesleyjellis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

One other question, I'm not sure if we will have the auth to download the binary off github when this runs?

@zyannes
zyannes marked this pull request as draft October 15, 2026 18:21
Use gh release list/download for semgrep/mandoline instead of a public HTTPS asset URL.

Co-authored-by: Cursor <cursoragent@cursor.com>
@zyannes
zyannes marked this pull request as ready for review October 15, 2026 18:34
@zyannes

zyannes commented Oct 15, 2026

Copy link
Copy Markdown
Member Author

@wesleyjellis you are correct. The wget version failed. Updated to use gh release in the latest commit (and verified manually on my instance).

zyannes and others added 2 commits October 15, 2026 14:36
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@zyannes
zyannes merged commit 5173078 into main Oct 15, 2026
4 checks passed
@zyannes
zyannes deleted the t3code/add-mandoline-mcp branch October 15, 2026 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants