Skip to content

[DevTools] Upgrade chrome-devtools-mcp to 1.8.0 in cdt-mcp e2e - #37497

Merged
hoxyq merged 1 commit into
mainfrom
devtools/cdt-mcp-2-upgrade-mcp
Sep 4, 2026
Merged

hoxyq merged 1 commit into
mainfrom
devtools/cdt-mcp-2-upgrade-mcp

Conversation

@hoxyq

@hoxyq hoxyq commented Sep 2, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Upgrade the cdt-mcp e2e dependency from chrome-devtools-mcp 1.3.0 to 1.8.0.
  • Pass pageId to page-scoped CLI tools, which 1.8.0 requires.
  • Use a hex sessionId (crypto.randomUUID()). 1.8.0 rejects ids that are not /[a-fA-F0-9-]+/.

Stacked on #37496

Test plan

  • yarn --cwd packages/react-devtools-cdt-mcp test:e2e

@meta-cla meta-cla Bot added the CLA Signed label Sep 2, 2026
@github-actions github-actions Bot added the React Core Team Opened by a member of the React Core Team label Sep 2, 2026
@hoxyq
hoxyq force-pushed the devtools/cdt-mcp-2-upgrade-mcp branch from e6038c3 to 8c1d695 Compare September 2, 2026 19:35
@hoxyq
hoxyq force-pushed the devtools/cdt-mcp-2-upgrade-mcp branch from 8c1d695 to 5c52084 Compare September 2, 2026 19:45
@hoxyq
hoxyq requested a review from eps1lon September 2, 2026 19:46
@hoxyq
hoxyq marked this pull request as ready for review September 2, 2026 19:46
@hoxyq
hoxyq disabled the stack merge September 3, 2026 13:14
Base automatically changed from devtools/cdt-mcp-1-dom-hooks to main September 3, 2026 13:14
Pass pageId to page-scoped CLI tools and use a hex session id, which 1.8.0 requires.
@hoxyq
hoxyq force-pushed the devtools/cdt-mcp-2-upgrade-mcp branch from 5c52084 to 7397960 Compare September 3, 2026 13:14
@github-actions

github-actions Bot commented Sep 3, 2026 •

Copy link
Copy Markdown

The build for this commit did not complete, so there is no size report. See the workflow run for details.

Generated by sizebot against 7397960

@hoxyq
hoxyq requested a review from eps1lon September 3, 2026 13:15
@hoxyq
hoxyq merged commit 8425b69 into main Sep 4, 2026
238 of 239 checks passed
@hoxyq
hoxyq deleted the devtools/cdt-mcp-2-upgrade-mcp branch September 4, 2026 14:28
harshil585581 added a commit to harshil585581/react that referenced this pull request Sep 4, 2026
[DevTools] Upgrade chrome-devtools-mcp to 1.8.0 in cdt-mcp e2e (react#37497)
hoxyq added a commit that referenced this pull request Sep 7, 2026
## Summary
- Rewrite the package README so npm consumers can tell this is a
page-side library, not an MCP server.
- Document the chrome-devtools-mcp 1.3.0+ requirement, the experimental
third-party flag, and the corrected DOM-tool output.

Stacked on #37497

## Test plan
- [ ] Read the README as a first-time installer and confirm install,
import order, and MCP config are clear.
- [ ] Confirm `react_get_component_by_dom_element` no longer documents
`hooks`.
github-actions Bot pushed a commit to srpatcha/react that referenced this pull request Sep 8, 2026
…7503)

## Summary
- Rewrite the package README so npm consumers can tell this is a
page-side library, not an MCP server.
- Document the chrome-devtools-mcp 1.3.0+ requirement, the experimental
third-party flag, and the corrected DOM-tool output.

Stacked on react#37497

## Test plan
- [ ] Read the README as a first-time installer and confirm install,
import order, and MCP config are clear.
- [ ] Confirm `react_get_component_by_dom_element` no longer documents
`hooks`.

DiffTrain build for [9b7a0d4](react@9b7a0d4)
github-actions Bot pushed a commit to srpatcha/react that referenced this pull request Sep 8, 2026
…7503)

## Summary
- Rewrite the package README so npm consumers can tell this is a
page-side library, not an MCP server.
- Document the chrome-devtools-mcp 1.3.0+ requirement, the experimental
third-party flag, and the corrected DOM-tool output.

Stacked on react#37497

## Test plan
- [ ] Read the README as a first-time installer and confirm install,
import order, and MCP config are clear.
- [ ] Confirm `react_get_component_by_dom_element` no longer documents
`hooks`.

DiffTrain build for [9b7a0d4](react@9b7a0d4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants