docs: add VS Code Copilot MCP config#622
Conversation
|
@VIVAAN-DHAWAN is attempting to deploy a commit to the rohitg00's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughUpdates README to document GitHub Copilot / VS Code MCP configuration: adds a table row noting VS Code uses the ChangesCopilot / VS Code MCP Setup Documentation
🎯 1 (Trivial) | ⏱️ ~3 minutes
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
README.md (1)
550-563: ⚡ Quick winInclude optional
envkeys in the VS Code example for parity and remote/protected setups.This snippet works locally, but adding
AGENTMEMORY_URL/AGENTMEMORY_SECRETavoids confusion and aligns with the rest of the MCP docs.Suggested doc tweak
{ "servers": { "agentmemory": { "type": "stdio", "command": "npx", - "args": ["-y", "`@agentmemory/mcp`"] + "args": ["-y", "`@agentmemory/mcp`"], + "env": { + "AGENTMEMORY_URL": "http://localhost:3111", + "AGENTMEMORY_SECRET": "${AGENTMEMORY_SECRET}" + } } } }🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@README.md` around lines 550 - 563, Update the VS Code example JSON for the "servers.agentmemory" entry to include an optional "env" object with keys AGENTMEMORY_URL and AGENTMEMORY_SECRET so remote/protected setups are represented; locate the "servers.agentmemory" block (type, command, args) and add an "env" property documenting that AGENTMEMORY_URL points to the persistent server URL and AGENTMEMORY_SECRET holds the secret token, making clear these are optional and used when proxying to a remote MCP instead of the local fallback.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@README.md`:
- Around line 550-563: Update the VS Code example JSON for the
"servers.agentmemory" entry to include an optional "env" object with keys
AGENTMEMORY_URL and AGENTMEMORY_SECRET so remote/protected setups are
represented; locate the "servers.agentmemory" block (type, command, args) and
add an "env" property documenting that AGENTMEMORY_URL points to the persistent
server URL and AGENTMEMORY_SECRET holds the secret token, making clear these are
optional and used when proxying to a remote MCP instead of the local fallback.
c3eff54 to
d2acca2
Compare
|
Addressed the CodeRabbit nitpick in d2acca2 by adding optional AGENTMEMORY_URL / AGENTMEMORY_SECRET env keys to the VS Code example and clarifying when to keep or omit the env block. Re-ran rg checks and git diff --check. |
Summary
serversshape instead ofmcpServers.Closes #589
Verification
rg -n "GitHub Copilot|\"servers\"|\.vscode/mcp.json" README.mdgit diff --checkSummary by CodeRabbit