Skip to content

Commit eda9f45

Browse files
j-piaseckimeta-codesync[bot]
authored andcommitted
Cover jsiexecutor with Stable API guards (#58133)
Summary: Pull Request resolved: #58133 Classifies `jsiexecutor:jsiexecutor` as a private target under the C++ stable API three-tier visibility model. Adds `#include <react/cxxstableapi/PrivateGuard.h>` to both exported headers (`JSIExecutor.h` and `JSINativeModules.h`), and wires the guard dependency into BUCK, CMake and the podspec. The podspec needs no `USE_FRAMEWORKS` header search path edit: it already sets `HEADER_SEARCH_PATHS` to `"$(PODS_TARGET_SRCROOT)/.."` unconditionally, and that resolves to `ReactCommon`. The guards are inert unless a consumer defines `RN_STRICT_API`, so there is no behavior change. Changelog: [Internal] Reviewed By: cipolleschi Differential Revision: D117330790 fbshipit-source-id: 698a30cf536c4c4e6c6f61aed9afbc1267153c13
1 parent ba398a5 commit eda9f45

4 files changed

Lines changed: 6 additions & 0 deletions

File tree

packages/react-native/ReactCommon/jsiexecutor/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ target_include_directories(jsireact PUBLIC .)
1717

1818
target_link_libraries(jsireact
1919
react_cxxreact
20+
react_cxxstableapi
2021
reactperflogger
2122
folly_runtime
2223
glog

packages/react-native/ReactCommon/jsiexecutor/React-jsiexecutor.podspec

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ Pod::Spec.new do |s|
3535
s.dependency "React-jsi"
3636
s.dependency "React-jsitooling"
3737
s.dependency "React-perflogger"
38+
s.dependency "React-cxxstableapi"
3839
add_dependency(s, "React-debug")
3940
add_dependency(s, "React-runtimeexecutor", :additional_framework_paths => ["platform/ios"])
4041
add_dependency(s, "React-jsinspector", :framework_name => 'jsinspector_modern')

packages/react-native/ReactCommon/jsiexecutor/jsireact/JSIExecutor.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@
77

88
#pragma once
99

10+
#include <react/cxxstableapi/PrivateGuard.h>
11+
1012
#include <cxxreact/JSBigString.h>
1113
#include <cxxreact/JSExecutor.h>
1214
#include <cxxreact/RAMBundleRegistry.h>

packages/react-native/ReactCommon/jsiexecutor/jsireact/JSINativeModules.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@
77

88
#pragma once
99

10+
#include <react/cxxstableapi/PrivateGuard.h>
11+
1012
#ifndef RCT_REMOVE_LEGACY_ARCH
1113

1214
#include <memory>

0 commit comments

Comments
 (0)