Skip to content

Restore Fantom DevTools launching with FANTOM_DEBUG_JS (#59410) - #59410

Closed
rubennorte wants to merge 1 commit into
react:mainfrom
rubennorte:export-D127171074
Closed

rubennorte wants to merge 1 commit into
react:mainfrom
rubennorte:export-D127171074

Conversation

@rubennorte

@rubennorte rubennorte commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Summary:

Restore DevTools launching when FANTOM_DEBUG_JS is set by allowing the existing native tester to launch through the shared browser launcher. Explicitly allow app launching for interactive test debugging, use a Chrome app window instead of the standalone shell, and run debugging sessions in band.

Changelog: [Internal]

Reviewed By: lenaic, andrewdacenko

Differential Revision: D127171074

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 23, 2026
@meta-codesync

meta-codesync Bot commented Oct 23, 2026

Copy link
Copy Markdown

@rubennorte has exported this pull request. If you are a Meta employee, you can view the originating Diff in D127171074.

@meta-codesync meta-codesync Bot changed the title Restore Fantom DevTools launching with FANTOM_DEBUG_JS Restore Fantom DevTools launching with FANTOM_DEBUG_JS (#59410) Oct 23, 2026
rubennorte added a commit to rubennorte/react-native that referenced this pull request Oct 23, 2026
Summary:

Restore DevTools launching when `FANTOM_DEBUG_JS` is set. Use a Chrome app window instead of the standalone shell, which can fail to start in restricted launch environments. Request DevTools from the runner after the inspector target registers, deduplicate launch requests for each tester connection, and run debugging sessions in band. Report failed launcher commands in the test output.

Changelog: [Internal]

Differential Revision: D127171074
@rubennorte
rubennorte force-pushed the export-D127171074 branch 2 times, most recently from 8f11048 to 933431a Compare October 26, 2026 11:35
rubennorte added a commit to rubennorte/react-native that referenced this pull request Oct 26, 2026
Summary:

Restore DevTools launching when `FANTOM_DEBUG_JS` is set by reusing the existing browser launcher with an explicit opt-in for interactive test debugging. Use a Chrome app window, request DevTools after the inspector target registers, deduplicate launch requests for each tester connection, and run debugging sessions in band. Preserve launcher failures in the test output.

Changelog: [Internal]

Reviewed By: lenaic, andrewdacenko

Differential Revision: D127171074
Summary:

Restore DevTools launching when `FANTOM_DEBUG_JS` is set by allowing the existing native tester to launch through the shared browser launcher. Explicitly allow app launching for interactive test debugging, use a Chrome app window instead of the standalone shell, and run debugging sessions in band.

Changelog: [Internal]

Reviewed By: lenaic, andrewdacenko

Differential Revision: D127171074
rubennorte added a commit to rubennorte/react-native that referenced this pull request Oct 26, 2026
Summary:

Restore DevTools launching when `FANTOM_DEBUG_JS` is set by allowing the existing native tester to launch through the shared browser launcher. Explicitly allow app launching for interactive test debugging, use a Chrome app window instead of the standalone shell, and run debugging sessions in band.

Changelog: [Internal]

Reviewed By: lenaic, andrewdacenko

Differential Revision: D127171074
@rubennorte
rubennorte force-pushed the export-D127171074 branch 2 times, most recently from cc72bee to 872e920 Compare October 26, 2026 11:51
@meta-codesync meta-codesync Bot closed this in bd2074c Oct 26, 2026
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @rubennorte in bd2074c

When will my fix make it into a release? | How to file a pick request?

@react-native-bot react-native-bot added the Merged This PR has been merged. label Oct 26, 2026
@meta-codesync

meta-codesync Bot commented Oct 26, 2026

Copy link
Copy Markdown

This pull request has been merged in bd2074c.

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants