migrate .vscode/mcp.json to .mcp.json #336751

Merged
aeschli opened 4:37am on September 18, 2026 wanted to merge 1 commit into microsoft/vscode main from
aeschli/acute-blackbird-147

Pull Request Overview

  • Opened on September 18, 2026
  • Status Merged
  • Commit count 1 with first commit September 18, 2026

Total Delta

0 Total Diff Delta

Open Days

Open 0 weekdays

Test Delta

0 Diff Delta in Test Files
Breakdown by Phase

How long has this pull request spent in each phase of its lifecycle?

Fraction of total time Business days Phase
 
0.0 days Authoring 1 commit before pull request opened for review
 
0.0 days Awaiting first review
 
0.7 days Merge to deploy (still awaiting deployment)

Total time for pull request 0.0 business days from first commit authored to merge
Still awaiting deployment

Author avatar

migrate .vscode/mcp.json to .mcp.json

Comments Threads Pending Resolution

ljharb reviewed on September 18, 2026
ljharb left a comment

## Copilot review overview

### πŸ”΅ Needs a closer look

The MCP README still directs developers to the removed `.vscode/mcp.json` file.

**Review effort:** Balanced
**Findings:** None

What changed in this PR

Migrates development MCP configuration to the workspace-root `.mcp.json` format.

**Changes:**
- Uses the `mcpServers` schema and replaces `cwd` with `npm --prefix`.
- Explicitly includes Node.js typings for the MCP test project.

| File | Description |
| ---- | ----------- |
| `.mcp.json` | Migrates MCP server definitions to the root format. |
| `test/​mcp/​tsconfig.json` | Adds explicit Node.js type declarations. |

---

πŸ’‘ Add a `code-review` agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Resolved Comment Threads

No resolved comments have been left on this PR.