Skip to content

fix: wrap .mcp.json config in mcpServers key#1246

Merged
OrKoN merged 1 commit intoChromeDevTools:mainfrom
ujjalcal:fix/mcp-json-schema
Mar 30, 2026
Merged

fix: wrap .mcp.json config in mcpServers key#1246
OrKoN merged 1 commit intoChromeDevTools:mainfrom
ujjalcal:fix/mcp-json-schema

Conversation

@ujjalcal
Copy link
Copy Markdown
Contributor

Summary

  • The .mcp.json file was missing the required mcpServers wrapper key, causing MCP clients (e.g., Claude Code) to fail parsing with a schema validation error.
  • Wraps the existing config inside mcpServers to match the expected MCP configuration schema.

Test plan

  • Run claude /doctor — config parse error is resolved

🤖 Generated with Claude Code

@google-cla
Copy link
Copy Markdown

google-cla Bot commented Mar 26, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@ujjalcal ujjalcal force-pushed the fix/mcp-json-schema branch from 8e60096 to 614afba Compare March 26, 2026 15:10
@Lightning00Blade Lightning00Blade requested a review from OrKoN March 30, 2026 10:07
The .mcp.json file was missing the required `mcpServers` wrapper,
causing MCP clients to fail parsing with a schema validation error.
@OrKoN OrKoN force-pushed the fix/mcp-json-schema branch from 614afba to 5ae0026 Compare March 30, 2026 10:38
@OrKoN OrKoN enabled auto-merge March 30, 2026 10:39
@OrKoN OrKoN added this pull request to the merge queue Mar 30, 2026
Merged via the queue into ChromeDevTools:main with commit c7948cf Mar 30, 2026
17 checks passed
OrKoN pushed a commit to mvanhorn/chrome-devtools-mcp that referenced this pull request Apr 1, 2026
## Summary
- The `.mcp.json` file was missing the required `mcpServers` wrapper
key, causing MCP clients (e.g., Claude Code) to fail parsing with a
schema validation error.
- Wraps the existing config inside `mcpServers` to match the expected
MCP configuration schema.

## Test plan
- [x] Run `claude /doctor` — config parse error is resolved

🤖 Generated with [Claude Code](https://claude.com/claude-code)
github-merge-queue Bot pushed a commit that referenced this pull request Apr 1, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.21.0](chrome-devtools-mcp-v0.20.3...chrome-devtools-mcp-v0.21.0)
(2026-04-01)


### 🎉 Features

* add a skill for detecting memory leaks using take_memory_snapshot tool
([#1162](#1162))
([d19a235](d19a235))


### 🛠️ Fixes

* **cli:** avoid defaulting to isolated when userDataDir is provided
([#1258](#1258))
([73e1e24](73e1e24))
* list_pages should work after selected page is closed
([#1145](#1145))
([2664455](2664455))
* mark lighthouse and memory as non-read-only
([#1769](#1769))
([bec9dae](bec9dae))
* **telemetry:** record client name
([9a47b65](9a47b65))
* versioning for Claude Code plugin
([#1233](#1233))
([966b86f](966b86f))
* wrap .mcp.json config in mcpServers key
([#1246](#1246))
([c7948cf](c7948cf))


### 📄 Documentation

* Command Code CLI instructions for MCP server
([0a7c0a7](0a7c0a7))
* provide disclaimer about supported browsers
([#1237](#1237))
([8676b72](8676b72))


### 🏗️ Refactor

* set process titles for easier debugging
([#1770](#1770))
([0fe3896](0fe3896))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: Alex Rudenko <alexrudenko@chromium.org>
wolfib pushed a commit that referenced this pull request Apr 1, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.21.0](chrome-devtools-mcp-v0.20.3...chrome-devtools-mcp-v0.21.0)
(2026-04-01)


### 🎉 Features

* add a skill for detecting memory leaks using take_memory_snapshot tool
([#1162](#1162))
([d19a235](d19a235))


### 🛠️ Fixes

* **cli:** avoid defaulting to isolated when userDataDir is provided
([#1258](#1258))
([73e1e24](73e1e24))
* list_pages should work after selected page is closed
([#1145](#1145))
([2664455](2664455))
* mark lighthouse and memory as non-read-only
([#1769](#1769))
([bec9dae](bec9dae))
* **telemetry:** record client name
([9a47b65](9a47b65))
* versioning for Claude Code plugin
([#1233](#1233))
([966b86f](966b86f))
* wrap .mcp.json config in mcpServers key
([#1246](#1246))
([c7948cf](c7948cf))


### 📄 Documentation

* Command Code CLI instructions for MCP server
([0a7c0a7](0a7c0a7))
* provide disclaimer about supported browsers
([#1237](#1237))
([8676b72](8676b72))


### 🏗️ Refactor

* set process titles for easier debugging
([#1770](#1770))
([0fe3896](0fe3896))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: Alex Rudenko <alexrudenko@chromium.org>
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.

2 participants