From 2c06985d0e627f6ed903b40d9767e52a34fdd0e6 Mon Sep 17 00:00:00 2001 From: Christoph Purrer Date: Sat, 4 Oct 2025 07:26:37 -0700 Subject: [PATCH] Use correct base class for NativeReactNativeFeatureFlags (#54046) Summary: Changelog: [Internal] Per documentation the base class to extend from is the non `JSI` one: https://reactnative.dev/docs/the-new-architecture/pure-cxx-modules#3-write-the-native-code Reviewed By: rubennorte Differential Revision: D83815509 --- .../featureflags/NativeReactNativeFeatureFlags.cpp | 4 ++-- .../nativemodule/featureflags/NativeReactNativeFeatureFlags.h | 4 ++-- .../js/NativeReactNativeFeatureFlags.cpp-template.js | 2 +- .../templates/js/NativeReactNativeFeatureFlags.h-template.js | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.cpp b/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.cpp index fa79d14c6459..c7b856aa8eeb 100644 --- a/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.cpp +++ b/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.cpp @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<> + * @generated SignedSource<<4d5d0795ed4cea8c1e185f6fe3fa0fad>> */ /** @@ -35,7 +35,7 @@ namespace facebook::react { NativeReactNativeFeatureFlags::NativeReactNativeFeatureFlags( std::shared_ptr jsInvoker) - : NativeReactNativeFeatureFlagsCxxSpecJSI(std::move(jsInvoker)) {} + : NativeReactNativeFeatureFlagsCxxSpec(std::move(jsInvoker)) {} bool NativeReactNativeFeatureFlags::commonTestFlag( jsi::Runtime& /*runtime*/) { diff --git a/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.h b/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.h index 00ca1a309364..cf2875d1563d 100644 --- a/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.h +++ b/packages/react-native/ReactCommon/react/nativemodule/featureflags/NativeReactNativeFeatureFlags.h @@ -4,7 +4,7 @@ * This source code is licensed under the MIT license found in the * LICENSE file in the root directory of this source tree. * - * @generated SignedSource<<990b6518e7b1d196deb0965e9ada6605>> + * @generated SignedSource<> */ /** @@ -28,7 +28,7 @@ namespace facebook::react { class NativeReactNativeFeatureFlags - : public NativeReactNativeFeatureFlagsCxxSpecJSI { + : public NativeReactNativeFeatureFlagsCxxSpec { public: NativeReactNativeFeatureFlags(std::shared_ptr jsInvoker); diff --git a/packages/react-native/scripts/featureflags/templates/js/NativeReactNativeFeatureFlags.cpp-template.js b/packages/react-native/scripts/featureflags/templates/js/NativeReactNativeFeatureFlags.cpp-template.js index 184e260f8d7b..b925934ecd6d 100644 --- a/packages/react-native/scripts/featureflags/templates/js/NativeReactNativeFeatureFlags.cpp-template.js +++ b/packages/react-native/scripts/featureflags/templates/js/NativeReactNativeFeatureFlags.cpp-template.js @@ -47,7 +47,7 @@ namespace facebook::react { NativeReactNativeFeatureFlags::NativeReactNativeFeatureFlags( std::shared_ptr jsInvoker) - : NativeReactNativeFeatureFlagsCxxSpecJSI(std::move(jsInvoker)) {} + : NativeReactNativeFeatureFlagsCxxSpec(std::move(jsInvoker)) {} ${Object.entries(definitions.common) .map(([flagName, flagConfig]) => diff --git a/packages/react-native/scripts/featureflags/templates/js/NativeReactNativeFeatureFlags.h-template.js b/packages/react-native/scripts/featureflags/templates/js/NativeReactNativeFeatureFlags.h-template.js index 64f0f403b427..96fd93bf9e3f 100644 --- a/packages/react-native/scripts/featureflags/templates/js/NativeReactNativeFeatureFlags.h-template.js +++ b/packages/react-native/scripts/featureflags/templates/js/NativeReactNativeFeatureFlags.h-template.js @@ -36,7 +36,7 @@ ${DO_NOT_MODIFY_COMMENT} namespace facebook::react { class NativeReactNativeFeatureFlags - : public NativeReactNativeFeatureFlagsCxxSpecJSI { + : public NativeReactNativeFeatureFlagsCxxSpec { public: NativeReactNativeFeatureFlags(std::shared_ptr jsInvoker);