Skip to content

chore: Scaffold mongodb-mcp-remote package MCP-529#1255

Merged
manupedrozo merged 2 commits into
mainfrom
MCP-529-scaffold-wrapper-package
Jun 17, 2026
Merged

chore: Scaffold mongodb-mcp-remote package MCP-529#1255
manupedrozo merged 2 commits into
mainfrom
MCP-529-scaffold-wrapper-package

Conversation

@manupedrozo

@manupedrozo manupedrozo commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Description

Scaffolding the mongodb-mcp-remote package.

"private": true in package.json -> the package is skipped during publish. We are still deciding on how the publishing process will work for this package, leaving as private until then.

Ticket: MCP-529

Checklist

@manupedrozo manupedrozo marked this pull request as ready for review June 17, 2026 13:00
@manupedrozo manupedrozo requested a review from a team as a code owner June 17, 2026 13:00
@manupedrozo manupedrozo requested review from cveticm, fmenezes and gagik and removed request for a team and cveticm June 17, 2026 13:00
@manupedrozo

Copy link
Copy Markdown
Collaborator Author

Test failure is unrelated.

@@ -0,0 +1,7 @@
import { describe, expect, it } from "vitest";

@gagik gagik Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: can we move this into src as cli.test.ts instead. so right in packages/src/cli.test.ts

#1122 will switch the whole repo to this structure for unit tests (and will match how Remote MCP does it), you can see the tsconfig for this there too.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sounds good, moved: 1b4d289

@gagik gagik left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, might be good to have tests next to src like it is in remote MCP and soon with v2

@manupedrozo manupedrozo enabled auto-merge (squash) June 17, 2026 16:39
@manupedrozo manupedrozo merged commit 4ad6ce3 into main Jun 17, 2026
25 of 31 checks passed
@manupedrozo manupedrozo deleted the MCP-529-scaffold-wrapper-package branch June 17, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants