Skip to content

Extract react/bridging into its own React-bridging pod (#57354)#57354

Closed
javache wants to merge 1 commit into
react:mainfrom
javache:export-D109868509
Closed

Extract react/bridging into its own React-bridging pod (#57354)#57354
javache wants to merge 1 commit into
react:mainfrom
javache:export-D109868509

Conversation

@javache

@javache javache commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Summary:

Split the react/bridging headers out of the ReactCommon pod's turbomodule/bridging subspec into a standalone React-bridging podspec, mirroring the existing React-bridging SwiftPM target. This makes bridging a dependency-graph leaf so other modules can depend on it without pulling in ReactCommon / React-cxxreact.

Consumers that previously reached react/bridging headers transitively through ReactCommon now resolve them through the standalone pod. The CocoaPods header-search-path injection in update_search_paths lists React-bridging alongside the other core frameworks so the headers resolve everywhere, and pods that link the LongLivedObject / CallbackWrapper symbols (React-Fabric, the nativemodule feature pods, React-NativeModulesApple, React-RCTFBReactNativeSpec) gain an explicit React-bridging dependency. All ReactCommon/turbomodule/bridging references in podspecs, the autolinker, the codegen template, and their test snapshots are repointed to React-bridging.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D109868509

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

meta-codesync Bot commented Jun 26, 2026

Copy link
Copy Markdown

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

@meta-codesync meta-codesync Bot changed the title Extract react/bridging into its own React-bridging pod Extract react/bridging into its own React-bridging pod (#57354) Jun 29, 2026
javache added a commit to javache/react-native that referenced this pull request Jun 29, 2026
Summary:

Split the `react/bridging` headers out of the `ReactCommon` pod's `turbomodule/bridging` subspec into a standalone `React-bridging` podspec, mirroring the existing `React-bridging` SwiftPM target. This makes bridging a dependency-graph leaf so other modules can depend on it without pulling in `ReactCommon` / `React-cxxreact`.

Consumers that previously reached `react/bridging` headers transitively through `ReactCommon` now resolve them through the standalone pod. The CocoaPods header-search-path injection in `update_search_paths` lists `React-bridging` alongside the other core frameworks so the headers resolve everywhere, and pods that link the `LongLivedObject` / `CallbackWrapper` symbols (`React-Fabric`, the nativemodule feature pods, `React-NativeModulesApple`, `React-RCTFBReactNativeSpec`) gain an explicit `React-bridging` dependency. All `ReactCommon/turbomodule/bridging` references in podspecs, the autolinker, the codegen template, and their test snapshots are repointed to `React-bridging`.

Changelog:
[Internal]

Differential Revision: D109868509
@javache javache force-pushed the export-D109868509 branch from a8d1b71 to 7eb996b Compare June 29, 2026 10:04

@cortinico cortinico left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

javache added a commit to javache/react-native that referenced this pull request Jul 3, 2026
Summary:

Split the `react/bridging` headers out of the `ReactCommon` pod's `turbomodule/bridging` subspec into a standalone `React-bridging` podspec, mirroring the existing `React-bridging` SwiftPM target. This makes bridging a dependency-graph leaf so other modules can depend on it without pulling in `ReactCommon` / `React-cxxreact`.

Consumers that previously reached `react/bridging` headers transitively through `ReactCommon` now resolve them through the standalone pod. The CocoaPods header-search-path injection in `update_search_paths` lists `React-bridging` alongside the other core frameworks so the headers resolve everywhere, and pods that link the `LongLivedObject` / `CallbackWrapper` symbols (`React-Fabric`, the nativemodule feature pods, `React-NativeModulesApple`, `React-RCTFBReactNativeSpec`) gain an explicit `React-bridging` dependency. All `ReactCommon/turbomodule/bridging` references in podspecs, the autolinker, the codegen template, and their test snapshots are repointed to `React-bridging`.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D109868509
@javache javache force-pushed the export-D109868509 branch from 7eb996b to 18511cc Compare July 3, 2026 15:29
Summary:

Split the `react/bridging` headers out of the `ReactCommon` pod's `turbomodule/bridging` subspec into a standalone `React-bridging` podspec, mirroring the existing `React-bridging` SwiftPM target. This makes bridging a dependency-graph leaf so other modules can depend on it without pulling in `ReactCommon` / `React-cxxreact`.

Consumers that previously reached `react/bridging` headers transitively through `ReactCommon` now resolve them through the standalone pod. The CocoaPods header-search-path injection in `update_search_paths` lists `React-bridging` alongside the other core frameworks so the headers resolve everywhere, and pods that link the `LongLivedObject` / `CallbackWrapper` symbols (`React-Fabric`, the nativemodule feature pods, `React-NativeModulesApple`, `React-RCTFBReactNativeSpec`) gain an explicit `React-bridging` dependency. All `ReactCommon/turbomodule/bridging` references in podspecs, the autolinker, the codegen template, and their test snapshots are repointed to `React-bridging`.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D109868509
@javache javache force-pushed the export-D109868509 branch from 18511cc to d558527 Compare July 3, 2026 15:35
@meta-codesync meta-codesync Bot closed this in 4cd2782 Jul 3, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Jul 3, 2026
@meta-codesync

meta-codesync Bot commented Jul 3, 2026

Copy link
Copy Markdown

This pull request has been merged in 4cd2782.

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