Skip to content

Add single host inspector state assertion over CDP#54407

Closed
huntie wants to merge 1 commit into
react:mainfrom
huntie:export-D86201689
Closed

Add single host inspector state assertion over CDP#54407
huntie wants to merge 1 commit into
react:mainfrom
huntie:export-D86201689

Conversation

@huntie

@huntie huntie commented Nov 4, 2025

Copy link
Copy Markdown
Member

Summary:
Introduces a new InspectorSystemState class and ReactNativeApplication.systemStateChanged CDP event, used to assert whether more than one React Native Host has been registered for the current app lifetime.

This will be used to disable the Performance and Network features in React Native DevTools when the debugger backend is in this currently unsupported state. We intend to implement host lifecycle correctness across all features soon.

Changelog: [Internal]

Differential Revision: D86201689

@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 Nov 4, 2025
@meta-codesync

meta-codesync Bot commented Nov 4, 2025

Copy link
Copy Markdown

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

Summary:
Pull Request resolved: #54407

Introduces a new `InspectorSystemState` object and `ReactNativeApplication.systemStateChanged` CDP event, used to assert whether more than one React Native Host has been registered for the current app lifetime.

This will be used to disable the Performance and Network features in React Native DevTools when the debugger backend is in this currently unsupported state. We intend to implement host lifecycle correctness across all features soon.

Changelog: [Internal]

Differential Revision: D86201689
@meta-codesync meta-codesync Bot closed this in d1944f1 Nov 10, 2025
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Nov 10, 2025
@meta-codesync

meta-codesync Bot commented Nov 10, 2025

Copy link
Copy Markdown

This pull request has been merged in d1944f1.

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @huntie in d1944f1

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

@huntie
huntie deleted the export-D86201689 branch November 10, 2025 14:45
huntie added a commit that referenced this pull request Nov 11, 2025
Summary:
Pull Request resolved: #54407

Introduces a new `InspectorSystemState` object and `ReactNativeApplication.systemStateChanged` CDP event, used to assert whether more than one React Native Host has been registered for the current app lifetime.

This will be used to disable the Performance and Network features in React Native DevTools when the debugger backend is in this currently unsupported state. We intend to implement host lifecycle correctness across all features soon.

Changelog: [Internal]

Reviewed By: cipolleschi

Differential Revision: D86201689

fbshipit-source-id: b8525359fc01eb7ae666879f2c0fd4c94f0af81a
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @huntie in 74d687c

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

douglowder pushed a commit to react-native-tvos/react-native-tvos that referenced this pull request Nov 15, 2025
Summary:
Pull Request resolved: react/react-native#54407

Introduces a new `InspectorSystemState` object and `ReactNativeApplication.systemStateChanged` CDP event, used to assert whether more than one React Native Host has been registered for the current app lifetime.

This will be used to disable the Performance and Network features in React Native DevTools when the debugger backend is in this currently unsupported state. We intend to implement host lifecycle correctness across all features soon.

Changelog: [Internal]

Reviewed By: cipolleschi

Differential Revision: D86201689

fbshipit-source-id: b8525359fc01eb7ae666879f2c0fd4c94f0af81a
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. fb-exported 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.

3 participants