Skip to content

Delete orphan RCTBridgeProxy shims (#59444) - #59444

Open
christophpurrer wants to merge 1 commit into
react:mainfrom
christophpurrer:export-D127294703
Open

christophpurrer wants to merge 1 commit into
react:mainfrom
christophpurrer:export-D127294703

Conversation

@christophpurrer

@christophpurrer christophpurrer commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary:

Remove orphan RCTBridgeProxy shims whose declarations are gone (parentBridge, moduleProvider, moduleSetupComplete, updateModuleWithInstance:, profiling, callNativeModule, logMessage, _immediatelyCallTimer:, object). Keep batchedBridge, bridgeDescription, runtime (Expo/Reanimated call them) and jsCallInvoker (Skia fallback read; macOS fork keeps its reader).

Changelog: [Internal]

Reviewed By: andrewdacenko

Differential Revision: D127294703

@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 24, 2026
@meta-codesync

meta-codesync Bot commented Oct 24, 2026

Copy link
Copy Markdown

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

@meta-codesync meta-codesync Bot changed the title Delete orphan RCTBridgeProxy shims Delete orphan RCTBridgeProxy shims (#59444) Oct 24, 2026
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 24, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `bridgeDescription`, `moduleProvider`, `batchedBridge`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer`, `moduleSetupComplete`, `updateModuleWithInstance:`) and never-declared ones (`runtime`, `object`, write-only `jsCallInvoker`). Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Differential Revision: D127294703
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 24, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `bridgeDescription`, `moduleProvider`, `batchedBridge`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer`, `moduleSetupComplete`, `updateModuleWithInstance:`) and never-declared ones (`runtime`, `object`, write-only `jsCallInvoker`). Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Differential Revision: D127294703
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 24, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `bridgeDescription`, `moduleProvider`, `batchedBridge`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer`, `moduleSetupComplete`, `updateModuleWithInstance:`) and never-declared ones (`runtime`, `object`, write-only `jsCallInvoker`). Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Differential Revision: D127294703
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 27, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `bridgeDescription`, `moduleProvider`, `batchedBridge`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer`, `moduleSetupComplete`, `updateModuleWithInstance:`) and never-declared ones (`runtime`, `object`, write-only `jsCallInvoker`). Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Differential Revision: D127294703
@christophpurrer
christophpurrer force-pushed the export-D127294703 branch 2 times, most recently from 105b2f6 to 64987ae Compare October 27, 2026 14:18
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 27, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `bridgeDescription`, `moduleProvider`, `batchedBridge`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer`, `moduleSetupComplete`, `updateModuleWithInstance:`) and never-declared ones (`runtime`, `object`, write-only `jsCallInvoker`). Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Differential Revision: D127294703
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 27, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`) and never-declared ones (`object`, write-only `jsCallInvoker`). Keep `batchedBridge`, `bridgeDescription`, and `runtime`: Expo and Reanimated still call them. Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Differential Revision: D127294703
@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Visual regression

1 visual difference(s) at 6de8ab87af40. Visual differences need maintainer approval. Review the report, then apply visual-change-approved.

ios touchable/highlight/default: Changed, 42 px (0.039%)

diff

Expected and Captured
Expected Captured
expected captured

christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 27, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`) and never-declared ones (`object`, write-only `jsCallInvoker`). Keep `batchedBridge`, `bridgeDescription`, and `runtime`: Expo and Reanimated still call them. Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Reviewed By: andrewdacenko

Differential Revision: D127294703
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 28, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`) and never-declared ones (`object`, write-only `jsCallInvoker`). Keep `batchedBridge`, `bridgeDescription`, and `runtime`: Expo and Reanimated still call them. Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Reviewed By: andrewdacenko

Differential Revision: D127294703
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 28, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`) and never-declared ones (`object`, write-only `jsCallInvoker`). Keep `batchedBridge`, `bridgeDescription`, and `runtime`: Expo and Reanimated still call them. Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Reviewed By: andrewdacenko

Differential Revision: D127294703
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 28, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`) and never-declared ones (`object`, write-only `jsCallInvoker`). Keep `batchedBridge`, `bridgeDescription`, and `runtime`: Expo and Reanimated still call them. Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Reviewed By: andrewdacenko

Differential Revision: D127294703
@meta-codesync meta-codesync Bot changed the title Delete orphan RCTBridgeProxy shims (#59444) Delete orphan RCTBridgeProxy shims Oct 29, 2026
@meta-codesync meta-codesync Bot changed the title Delete orphan RCTBridgeProxy shims Delete orphan RCTBridgeProxy shims (#59444) Oct 30, 2026
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Oct 30, 2026
Summary:
Pull Request resolved: react#59444

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`) and never-declared ones (`object`, write-only `jsCallInvoker`). Keep `batchedBridge`, `bridgeDescription`, and `runtime`: Expo and Reanimated still call them. Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Reviewed By: andrewdacenko

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

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`) and never-declared ones (`object`, write-only `jsCallInvoker`). Keep `batchedBridge`, `bridgeDescription`, and `runtime`: Expo and Reanimated still call them. Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Reviewed By: andrewdacenko

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

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`) and never-declared ones (`object`, write-only `jsCallInvoker`). Keep `batchedBridge`, `bridgeDescription`, and `runtime`: Expo and Reanimated still call them. Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Reviewed By: andrewdacenko

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

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`) and never-declared ones (`object`, write-only `jsCallInvoker`). Keep `batchedBridge`, `bridgeDescription`, and `runtime`: Expo and Reanimated still call them. Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Reviewed By: andrewdacenko

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

Remove proxy shims whose declarations were deleted from private headers (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`) and never-declared ones (`object`, write-only `jsCallInvoker`). Keep `batchedBridge`, `bridgeDescription`, and `runtime`: Expo and Reanimated still call them. Shims backing deleted public API stay as runtime nets.

Changelog: [Internal]

Reviewed By: andrewdacenko

Differential Revision: D127294703
Summary:

Remove orphan `RCTBridgeProxy` shims whose declarations are gone (`parentBridge`, `moduleProvider`, `moduleSetupComplete`, `updateModuleWithInstance:`, profiling, `callNativeModule`, `logMessage`, `_immediatelyCallTimer:`, `object`). Keep `batchedBridge`, `bridgeDescription`, `runtime` (Expo/Reanimated call them) and `jsCallInvoker` (Skia fallback read; macOS fork keeps its reader).

Changelog: [Internal]

Reviewed By: andrewdacenko

Differential Revision: D127294703

This branch has not been deployed

No deployments
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. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants