docs: Update VS Code installation instructions to use mcp.json for issue #2320 #2328
Conversation
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
- Replace outdated 'Preferences: Open Settings (JSON)' instructions - Add proper guidance for 'MCP: Open User Configuration' command - Update JSON examples to remove mcp wrapper key - Clarify user vs workspace configuration methods - Fixes issue with outdated documentation across all servers Affects: sequentialthinking, filesystem, memory, everything, git
|
@kunalbabre Thanks for opening this PR! I have two quick questions for the maintainers to weigh in on:
These are just my personal thoughts—happy to defer to whatever the maintainers consider best. |
- Add link to official VS Code MCP documentation in all server READMEs - Enhances existing VS Code installation instructions with authoritative reference - Provides users with comprehensive documentation for advanced configuration - Complements the existing two-method approach with additional resources Affects: everything, filesystem, git, memory, sequentialthinking
- Remove unnecessary explanatory note about JSON format - JSON examples already demonstrate the correct format clearly - Streamlines documentation to focus on essential information - Maintains reference to official VS Code MCP documentation The removed note was redundant since both configuration methods use identical JSON structure shown in the examples.
Contributor
Author
|
@luke396 - great points
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Description
Affects: sequentialthinking, filesystem, memory, everything, git
Server Details
-- None
Motivation and Context
To fix VS Code installation instructions to use mcp.json as raised in #2320
How Has This Been Tested?
-- N/A: Verified manually
Breaking Changes
-- N/A
Types of changes
Checklist
Additional context