Skip to content

chore: update config docs #170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Apr 30, 2025
Merged

chore: update config docs #170

merged 10 commits into from
Apr 30, 2025

Conversation

blva
Copy link
Collaborator

@blva blva commented Apr 29, 2025

Closes #171

@blva blva marked this pull request as ready for review April 29, 2025 16:56
@Copilot Copilot AI review requested due to automatic review settings April 29, 2025 16:56
@blva blva requested a review from a team as a code owner April 29, 2025 16:56
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the configuration documentation in the README to include detailed setup instructions and examples for both WindSurf and existing tools. Key changes include:

  • Introducing a new WindSurf section in the navigation and content.
  • Providing detailed JSON configuration examples for environment variables and command-line arguments.
  • Updating the VSCode and Claude Desktop sections to use proper setup instructions instead of outdated references.
Comments suppressed due to low confidence (1)

README.md:282

  • [nitpick] The section 'MCP configuration file examples' appears twice with similar content. Consider merging these sections into one with distinct subsections to reduce redundancy and improve clarity.
#### MCP configuration file examples

README.md Outdated
"servers": {
"MongoDB": {
"command": "npx",
"args": ["-y", "mongodb-mcp-server"]

Choose a reason for hiding this comment

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

@blva - as i understood, if you don't have the atlas API token OR a connection string here, the server will basically not start, correct?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yep, updated

@GaurabAryal
Copy link

GaurabAryal commented Apr 29, 2025

@blva – could we also remove the caution banner in the readme as part of this update? Windsurf template is going live tomorrow EOD so we should get this merged before that.

@fmenezes
Copy link
Collaborator

we have Windsurf, VSCode and Claude Desktop, how about adding Cursor?

@blva blva requested review from GaurabAryal and Copilot April 30, 2025 11:00
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the configuration documentation for the MongoDB MCP Server to improve clarity and provide additional examples for various configuration methods.

  • Updated introduction and description for connecting to MongoDB Databases and MongoDB Atlas
  • Added detailed examples for setting up the server via environment variables and command-line arguments
  • Revised the table of contents to better organize configuration options

@@ -250,6 +250,41 @@ export MDB_MCP_LOG_PATH="/path/to/logs"

```

#### MCP configuration file examples
Copy link
Preview

Copilot AI Apr 30, 2025

Choose a reason for hiding this comment

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

The section title 'MCP configuration file examples' is used again at a later section (line 296). Consider consolidating these examples or using distinct subheadings to clearly differentiate between configuration methods for environment variables and command-line arguments.

Suggested change
#### MCP configuration file examples
#### MCP configuration file examples (Environment Variables)

Copilot uses AI. Check for mistakes.

README.md Outdated
- Input command `npx -y mongodb-mcp-server`.
- Choose between user / workspace
- Add arguments to the file
- **Windsurf**: On MacOS, save the file in `~/.codeium/windsurf/mcp_config.json` or check the[latest instructions](https://docs.windsurf.com/windsurf/mcp) for your environment
Copy link
Collaborator

Choose a reason for hiding this comment

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

[q] Windsurf is the only one that has macos instructions, is there a reason?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated

@blva blva enabled auto-merge (squash) April 30, 2025 11:58
@blva blva dismissed GaurabAryal’s stale review April 30, 2025 12:06

Gaurab is flying, so I'll dismiss

@blva blva merged commit 6ee5634 into main Apr 30, 2025
15 checks passed
@blva blva deleted the docs-2 branch April 30, 2025 12:06
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.

Update docs with more examples on how to setup MCP
3 participants