Skip to content

[react-devtools-cdt-mcp] add chrome-devtools E2E coverage#36823

Open
hoxyq wants to merge 1 commit into
rtcm-1-cdt-mcp-dom-component-toolfrom
rtcm-1-cdt-mcp-e2e
Open

[react-devtools-cdt-mcp] add chrome-devtools E2E coverage#36823
hoxyq wants to merge 1 commit into
rtcm-1-cdt-mcp-dom-component-toolfrom
rtcm-1-cdt-mcp-e2e

Conversation

@hoxyq

@hoxyq hoxyq commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

A node runner that bootstraps fixtures/app, and then uses chrome-devtools CLI to test third-party tools.

$ node e2e/run.js
Starting fixture at http://127.0.0.1:60033/
Starting chrome-devtools daemon...
Checking third-party tool discovery...
Checking tree, details, search, and DOM lookup...
Checking source, owners, and error payloads...
Checking profiling through a real CLI click...
react-devtools-cdt-mcp E2E passed.
✨  Done in 11.66s.

@meta-cla meta-cla Bot added the CLA Signed label Jun 19, 2026
@hoxyq hoxyq requested review from eps1lon and jackpope June 19, 2026 13:55
@hoxyq hoxyq force-pushed the rtcm-1-cdt-mcp-e2e branch from a65e637 to f6d1472 Compare June 19, 2026 14:07
@hoxyq hoxyq force-pushed the rtcm-1-cdt-mcp-e2e branch from f6d1472 to 192c850 Compare June 19, 2026 14:17
@hoxyq hoxyq marked this pull request as ready for review June 19, 2026 14:18

@daltino daltino left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DECISION: APPROVE

This PR introduces E2E test coverage for the chrome-devtools integration within react-devtools-cdt-mcp. It adds a robust testing script (run.flow.js), updates the package configuration to support E2E tests, and includes relevant dependency updates. The changes align with the contribution guidelines and improve test automation, making it a valuable addition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants