Remove bridgelessEnabled from RCTRootViewFactoryConfiguration (#58416) - #58416
Open
christophpurrer wants to merge 2 commits into
Open
Remove bridgelessEnabled from RCTRootViewFactoryConfiguration (#58416)#58416christophpurrer wants to merge 2 commits into
bridgelessEnabled from RCTRootViewFactoryConfiguration (#58416)#58416christophpurrer wants to merge 2 commits into
Conversation
meta-cla
Bot
added
the
CLA Signed
|
@christophpurrer has exported this pull request. If you are a Meta employee, you can view the originating Diff in D116318829. |
…eact#58415) Summary: Bridgeless is the only supported mode in the New Architecture, so the `bridgelessEnabled` flag on `DefaultNewArchitectureEntryPoint` was dead configuration: every `load(...)` path passed `true`, and `isConfigurationValid` raised an error when it was `false`. The entry point advertised a toggle that could only ever hold the one value that is already mandatory. This removes that surface from `ReactAndroid`: - Removed the public `bridgelessEnabled` getter and its backing field. - Removed the deprecated three-argument `load(turboModulesEnabled, fabricEnabled, bridgelessEnabled)` overload. Its implementation body moved into the two-argument overload, since dropping the parameter alone would have collided with the existing `load(Boolean, Boolean)` signature. - Removed the `bridgelessEnabled` parameter from `isConfigurationValid`, reducing the guard to `!turboModulesEnabled || !fabricEnabled` and shortening the resulting error message. - `loadWithFeatureFlags` no longer reads `enableBridgelessArchitecture()`. Behavior note: `loadWithFeatureFlags` previously raised an error when a feature flags provider returned `enableBridgelessArchitecture() == false`. That check is gone. It was unreachable in practice because bridgeless is not optional, but it is a removed validation rather than a pure no-op cleanup. Bridgeless remains unconditionally enabled. Callers using the no-argument `load()` are unaffected. The regenerated `ReactAndroid.api` drops exactly `getBridgelessEnabled ()Z`, `load (ZZZ)V`, and `load$default (ZZZILjava/lang/Object;)V`. The equivalent iOS cleanup is intentionally left to a follow-up change. Changelog: [Android][Breaking] - Remove `DefaultNewArchitectureEntryPoint.bridgelessEnabled` and the deprecated three-argument `load(turboModulesEnabled, fabricEnabled, bridgelessEnabled)` overload; bridgeless is always enabled in the New Architecture Reviewed By: rubennorte Differential Revision: D116317210
…act#58416) Summary: Follow-up to the Android change in the previous diff, applying the same cleanup to iOS. Bridgeless is the only supported mode, so the `bridgelessEnabled` surface on `RCTRootViewFactoryConfiguration` was already deprecated and hardcoded: the property was assigned `YES` in every initializer, the two deprecated initializers ignored the argument entirely, and `RCTDefaultReactNativeFactoryDelegate` returned `YES` unconditionally. Nothing read the value. Changes: - Removed the `bridgelessEnabled` property from `RCTRootViewFactoryConfiguration`. - Removed the two deprecated initializers `initWithBundleURLBlock:newArchEnabled:turboModuleEnabled:bridgelessEnabled:` and `initWithBundleURL:newArchEnabled:turboModuleEnabled:bridgelessEnabled:`. Both were marked `__deprecated` and discarded all arguments except the bundle URL, delegating to the designated initializer. - Removed the `bridgelessEnabled` method from `RCTDefaultReactNativeFactoryDelegate`. It was not declared in any header or protocol. - Updated the one caller in `RCTReactNativeFactory` to use `initWithBundleURLBlock:newArchEnabled:`. Behavior is unchanged: bridgeless remains unconditionally enabled. Callers of the designated `initWithBundleURLBlock:newArchEnabled:` and `initWithBundleURL:newArchEnabled:` initializers are unaffected. Changelog: [iOS][Breaking] - Remove the deprecated `bridgelessEnabled` property and the deprecated `initWithBundleURLBlock:newArchEnabled:turboModuleEnabled:bridgelessEnabled:` / `initWithBundleURL:newArchEnabled:turboModuleEnabled:bridgelessEnabled:` initializers from `RCTRootViewFactoryConfiguration`; bridgeless is always enabled Reviewed By: rubennorte Differential Revision: D116318829
meta-codesync
Bot
changed the title
bridgelessEnabled from RCTRootViewFactoryConfigurationbridgelessEnabled from RCTRootViewFactoryConfiguration (#58416)
christophpurrer
force-pushed
the
export-D116318829
branch
from
September 10, 2026 01:36
e6ff757 to
069b690
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary:
Follow-up to the Android change in the previous diff, applying the same cleanup
to iOS.
Bridgeless is the only supported mode, so the
bridgelessEnabledsurface onRCTRootViewFactoryConfigurationwas already deprecated and hardcoded: theproperty was assigned
YESin every initializer, the two deprecatedinitializers ignored the argument entirely, and
RCTDefaultReactNativeFactoryDelegatereturnedYESunconditionally. Nothingread the value.
Changes:
bridgelessEnabledproperty fromRCTRootViewFactoryConfiguration.initWithBundleURLBlock:newArchEnabled:turboModuleEnabled:bridgelessEnabled:and
initWithBundleURL:newArchEnabled:turboModuleEnabled:bridgelessEnabled:.Both were marked
__deprecatedand discarded all arguments except the bundleURL, delegating to the designated initializer.
bridgelessEnabledmethod fromRCTDefaultReactNativeFactoryDelegate. It was not declared in any header orprotocol.
RCTReactNativeFactoryto useinitWithBundleURLBlock:newArchEnabled:.Behavior is unchanged: bridgeless remains unconditionally enabled. Callers of
the designated
initWithBundleURLBlock:newArchEnabled:andinitWithBundleURL:newArchEnabled:initializers are unaffected.Changelog:
[iOS][Breaking] - Remove the deprecated
bridgelessEnabledproperty and the deprecatedinitWithBundleURLBlock:newArchEnabled:turboModuleEnabled:bridgelessEnabled:/initWithBundleURL:newArchEnabled:turboModuleEnabled:bridgelessEnabled:initializers fromRCTRootViewFactoryConfiguration; bridgeless is always enabledReviewed By: rubennorte
Differential Revision: D116318829