-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
There was a problem hiding this 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"] |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, updated
@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. |
we have Windsurf, VSCode and Claude Desktop, how about adding Cursor? |
There was a problem hiding this 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 |
There was a problem hiding this comment.
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.
#### 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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
Closes #171