Skip to content

Add launchOptions and performanceLogger module injection - #59257

Closed
christophpurrer wants to merge 4 commits into
react:mainfrom
christophpurrer:export-D126567157
Closed

christophpurrer wants to merge 4 commits into
react:mainfrom
christophpurrer:export-D126567157

Conversation

@christophpurrer

Copy link
Copy Markdown
Contributor

Summary:
Extend RCTBridgeModule with optional launchOptions and performanceLogger properties, injected through RCTBridgeModuleDecorator from RCTInstance. This lets TurboModules read launch configuration and perf events without touching the deprecated bridge property.

Changelog:
[iOS][Added] - Add injectable launchOptions and performanceLogger module APIs

Differential Revision: D126567157

Summary:
RCTDevMenu, RCTLogBox, RCTRedBox, and RCTBlobManager synthesized the
deprecated bridge property but never read it. They use moduleRegistry,
bundleManager, and callableJSModules instead.

Changelog:
[Internal]

Differential Revision: D126545276
Summary:
Seven implementation files imported the bridge header without
referencing any symbol it provides. Nine public headers either
imported it redundantly or narrowed it to the module protocol
header they actually need.

Changelog:
[Internal]

Differential Revision: D126549484
Summary: RCTEventDispatcher routes via _dispatchToJSThread; RCTImageLoader and RCTNetworking require their registry-backed providers; RCTTestModule uses the injected viewRegistry_DEPRECATED; RCTViewManager drops a no-op registry write. Legacy RCTBridge is a stub, so these reads only reached RCTBridgeProxy shims.

Differential Revision: D126558187
Summary:
Extend RCTBridgeModule with optional launchOptions and performanceLogger properties, injected through RCTBridgeModuleDecorator from RCTInstance. This lets TurboModules read launch configuration and perf events without touching the deprecated bridge property.

Changelog:
[iOS][Added] - Add injectable launchOptions and performanceLogger module APIs

Differential Revision: D126567157
@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 21, 2026
@meta-codesync

meta-codesync Bot commented Oct 21, 2026

Copy link
Copy Markdown

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

christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:

Extend RCTBridgeModule with optional launchOptions and performanceLogger properties, injected through RCTBridgeModuleDecorator from RCTInstance. This lets TurboModules read launch configuration and perf events without touching the deprecated bridge property.

Changelog:
[iOS][Added] - Add injectable launchOptions and performanceLogger module APIs

Differential Revision: D126567157
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:

Extend RCTBridgeModule with optional launchOptions and performanceLogger properties, injected through RCTBridgeModuleDecorator from RCTInstance. This lets TurboModules read launch configuration and perf events without touching the deprecated bridge property.

Changelog:
[iOS][Added] - Add injectable launchOptions and performanceLogger module APIs

Differential Revision: D126567157
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:

Extend RCTBridgeModule with optional launchOptions and performanceLogger properties, injected through RCTBridgeModuleDecorator from RCTInstance. This lets TurboModules read launch configuration and perf events without touching the deprecated bridge property.

Changelog:
[iOS][Added] - Add injectable launchOptions and performanceLogger module APIs

Differential Revision: D126567157
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:

Extend RCTBridgeModule with optional launchOptions and performanceLogger properties, injected through RCTBridgeModuleDecorator from RCTInstance. This lets TurboModules read launch configuration and perf events without touching the deprecated bridge property.

Changelog:
[iOS][Added] - Add injectable launchOptions and performanceLogger module APIs

Differential Revision: D126567157
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:

Extend RCTBridgeModule with optional launchOptions and performanceLogger properties, injected through RCTBridgeModuleDecorator from RCTInstance. This lets TurboModules read launch configuration and perf events without touching the deprecated bridge property.

Changelog:
[iOS][Added] - Add injectable launchOptions and performanceLogger module APIs

Differential Revision: D126567157
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 21, 2026
Summary:

Extend RCTBridgeModule with optional launchOptions and performanceLogger properties, injected through RCTBridgeModuleDecorator from RCTInstance. This lets TurboModules read launch configuration and perf events without touching the deprecated bridge property.

Changelog:
[iOS][Added] - Add injectable launchOptions and performanceLogger module APIs

Reviewed By: shwanton

Differential Revision: D126567157
@meta-codesync meta-codesync Bot closed this in 9d7447a Oct 22, 2026
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @christophpurrer in 9d7447a

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 22, 2026
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