fix(iOS): hide react symbols from swift

.github/workflows/ios-build-test-fabric.yml

@@ -74,3 +74,9 @@ jobs:
       - name: Build app
         working-directory: ${{ env.WORKING_DIRECTORY }}
         run: npx react-native run-ios --no-packager --simulator="iPhone 17"
+
+        # Verify that the build above did not leak any React (React-Core / Fabric)
+        # symbols into the RNScreens Clang module consumed by Swift. See the script
+        # header for the rationale and detection details.
+      - name: Check React symbols are not leaked to Swift
+        run: node ./scripts/ios/check-react-symbols-not-leaked-to-swift.js

ios/RNSBarButtonItem.h

@@ -1,16 +1,20 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTImageLoader.h>
+#endif //defined(__cplusplus)
 #import <UIKit/UIKit.h>
 
 typedef void (^RNSBarButtonItemAction)(NSString *buttonId);
 typedef void (^RNSBarButtonMenuItemAction)(NSString *menuId);
 
 @interface RNSBarButtonItem : UIBarButtonItem
 
+#if defined(__cplusplus)
 - (instancetype)initWithConfig:(NSDictionary<NSString *, id> *)dict
                         action:(RNSBarButtonItemAction)action
                     menuAction:(RNSBarButtonMenuItemAction)menuAction
                    imageLoader:(RCTImageLoader *)imageLoader;
+#endif //defined(__cplusplus)
 
 @end

ios/RNSFullWindowOverlay.h

@@ -1,13 +1,19 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTViewManager.h>
+#endif // defined(__cplusplus)
 #import "RNSReactBaseView.h"
 
 #if defined(__cplusplus)
 namespace react = facebook::react;
 #endif // __cplusplus
 
+#if defined(__cplusplus)
 @interface RNSFullWindowOverlayManager : RCTViewManager
+#else
+@interface RNSFullWindowOverlayManager : NSObject
+#endif // __cplusplus
 
 @end
 

ios/RNSScreen.h

@@ -1,7 +1,9 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTComponent.h>
 #import <React/RCTViewManager.h>
+#endif // __cplusplus
 
 #import "RNSEnums.h"
 #import "RNSSafeAreaProviding.h"
@@ -22,6 +24,7 @@ namespace react = facebook::react;
 
 NS_ASSUME_NONNULL_BEGIN
 
+#if defined(__cplusplus)
 @interface RCTConvert (RNSScreen)
 
 #if !TARGET_OS_TV
@@ -31,6 +34,7 @@ NS_ASSUME_NONNULL_BEGIN
 #endif
 
 @end
+#endif
 
 @class RNSScreenView;
 
@@ -172,7 +176,11 @@ NS_ASSUME_NONNULL_BEGIN
 - (UIViewController *)parentViewController;
 @end
 
+#if defined(__cplusplus)
 @interface RNSScreenManager : RCTViewManager
+#else
+@interface RNSScreenManager : NSObject
+#endif // __cplusplus
 
 @end
 

ios/RNSScreenContainer.h

@@ -1,6 +1,8 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTViewManager.h>
+#endif // __cplusplus
 #import "RNSReactBaseView.h"
 
 NS_ASSUME_NONNULL_BEGIN
@@ -22,7 +24,11 @@ NS_ASSUME_NONNULL_BEGIN
 
 @end
 
+#if defined(__cplusplus)
 @interface RNSScreenContainerManager : RCTViewManager
+#else
+@interface RNSScreenContainerManager : NSObject
+#endif // __cplusplus
 
 @end
 

ios/RNSScreenContentWrapper.h

@@ -1,6 +1,8 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTViewManager.h>
+#endif // __cplusplus
 #import <UIKit/UIKit.h>
 #import "RNSDefines.h"
 #import "RNSReactBaseView.h"
@@ -43,7 +45,11 @@ typedef struct {
 
 @end
 
+#if defined(__cplusplus)
 @interface RNSScreenContentWrapperManager : RCTViewManager
+#else
+@interface RNSScreenContentWrapperManager : NSObject
+#endif // __cplusplus
 
 @end
 

ios/RNSScreenFooter.h

@@ -1,6 +1,8 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTViewManager.h>
+#endif // __cplusplus
 #import <UIKit/UIKit.h>
 #import "RNSReactBaseView.h"
 
@@ -18,7 +20,11 @@ typedef void (^OnLayoutCallback)(CGRect frame);
 
 @end
 
+#if defined(__cplusplus)
 @interface RNSScreenFooterManager : RCTViewManager
+#else
+@interface RNSScreenFooterManager : NSObject
+#endif // __cplusplus
 
 @end
 

ios/RNSScreenNavigationContainer.h

@@ -1,6 +1,8 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTViewManager.h>
+#endif // __cplusplus
 
 #import "RNSScreenContainer.h"
 #import "RNSScreenStack.h"

ios/RNSScreenStack.h

@@ -47,7 +47,11 @@ NS_ASSUME_NONNULL_BEGIN
 
 @end
 
+#if defined(__cplusplus)
 @interface RNSScreenStackManager : RCTViewManager
+#else
+@interface RNSScreenStackManager : NSObject
+#endif // __cplusplus
 
 @end
 

ios/RNSScreenStackHeaderConfig.h

@@ -1,6 +1,8 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTConvert.h>
+#endif // __cplusplus
 #import "RNSReactBaseView.h"
 #import "RNSScreen.h"
 #import "RNSScreenStackHeaderSubview.h"
@@ -107,6 +109,10 @@ NS_ASSUME_NONNULL_END
 
 #pragma mark - View Manager
 
+#if defined(__cplusplus)
 @interface RNSScreenStackHeaderConfigManager : RCTViewManager
+#else
+@interface RNSScreenStackHeaderConfigManager : NSObject
+#endif // __cplusplus
 
 @end

ios/RNSScreenStackHeaderSubview.h

@@ -1,7 +1,9 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTConvert.h>
 #import <React/RCTViewManager.h>
+#endif // __cplusplus
 #import "RNSEnums.h"
 #import "RNSReactBaseView.h"
 
@@ -46,7 +48,11 @@ NS_ASSUME_NONNULL_BEGIN
 
 @end
 
+#if defined(__cplusplus)
 @interface RNSScreenStackHeaderSubviewManager : RCTViewManager
+#else
+@interface RNSScreenStackHeaderSubviewManager : NSObject
+#endif // __cplusplus
 
 @property (nonatomic) RNSScreenStackHeaderSubviewType type;
 

ios/RNSSearchBar.h

@@ -5,10 +5,10 @@
 #if defined(__cplusplus)
 #import <React/RCTViewComponentView.h>
 #import <react/renderer/components/rnscreens/RCTComponentViewHelpers.h>
-#endif
-
 #import <React/RCTComponent.h>
 #import <React/RCTViewManager.h>
+#endif
+
 #import "RNSDefines.h"
 #import "RNSEnums.h"
 
@@ -32,6 +32,10 @@
 
 @end
 
+#if defined(__cplusplus)
 @interface RNSSearchBarManager : RCTViewManager
+#else
+@interface RNSSearchBarManager : NSObject
+#endif
 
 @end

ios/events/RNSHeaderHeightChangeEvent.h

@@ -1,8 +1,13 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTEventDispatcherProtocol.h>
+#endif // defined(__cplusplus)
 
-@interface RNSHeaderHeightChangeEvent : NSObject <RCTEvent>
+@interface RNSHeaderHeightChangeEvent : NSObject
+#if defined(__cplusplus)
+ <RCTEvent>
+#endif // defined(__cplusplus)
 
 - (instancetype)initWithEventName:(NSString *)eventName reactTag:(NSNumber *)reactTag headerHeight:(double)headerHeight;
 

ios/events/RNSScreenViewEvent.h

@@ -1,8 +1,13 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTEventDispatcherProtocol.h>
+#endif // defined(__cplusplus)
 
-@interface RNSScreenViewEvent : NSObject <RCTEvent>
+@interface RNSScreenViewEvent : NSObject
+#if defined(__cplusplus)
+ <RCTEvent>
+#endif // defined(__cplusplus)
 
 - (instancetype)initWithEventName:(NSString *)eventName
                          reactTag:(NSNumber *)reactTag

ios/gamma/split/RNSSplitHostComponentViewManager.h

@@ -1,10 +1,16 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTViewManager.h>
+#endif // __cplusplus
 
 NS_ASSUME_NONNULL_BEGIN
 
+#if defined(__cplusplus)
 @interface RNSSplitHostComponentViewManager : RCTViewManager
+#else
+@interface RNSSplitHostComponentViewManager : NSObject
+#endif // __cplusplus
 
 @end
 

ios/gamma/split/RNSSplitScreenComponentViewManager.h

@@ -1,10 +1,16 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTViewManager.h>
+#endif // __cplusplus
 
 NS_ASSUME_NONNULL_BEGIN
 
+#if defined(__cplusplus)
 @interface RNSSplitScreenComponentViewManager : RCTViewManager
+#else
+@interface RNSSplitScreenComponentViewManager : NSObject
+#endif
 
 @end
 

ios/gamma/stack/host/RNSStackHostComponentViewManager.h

@@ -1,10 +1,16 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTViewManager.h>
+#endif // __cplusplus
 
 NS_ASSUME_NONNULL_BEGIN
 
+#if defined(__cplusplus)
 @interface RNSStackHostComponentViewManager : RCTViewManager
+#else
+@interface RNSStackHostComponentViewManager : NSObject
+#endif
 
 @end
 

ios/gamma/stack/screen/RNSStackScreenComponentViewManager.h

@@ -1,10 +1,16 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTViewManager.h>
+#endif // __cplusplus
 
 NS_ASSUME_NONNULL_BEGIN
 
+#if defined(__cplusplus)
 @interface RNSStackScreenComponentViewManager : RCTViewManager
+#else
+@interface RNSStackScreenComponentViewManager : NSObject
+#endif
 
 @end
 

ios/helpers/image/RNSImageLoadingHelper.h

@@ -1,5 +1,7 @@
 #pragma once
 
+#if defined(__cplusplus)
+
 #import <React/RCTImageLoader.h>
 #import <React/RCTImageSource.h>
 
@@ -26,3 +28,5 @@
             completionBlock:(void (^_Nonnull)(UIImage *_Nullable image))imageLoadingCompletionBlock;
 
 @end
+
+#endif //defined(__cplusplus)

ios/tabs/RCTConvert+RNSTabs.h

@@ -1,5 +1,6 @@
 #pragma once
 
+#if defined(__cplusplus)
 #import <React/RCTConvert.h>
 #import <UIKit/UIKit.h>
 
@@ -12,3 +13,5 @@ NS_ASSUME_NONNULL_BEGIN
 @end
 
 NS_ASSUME_NONNULL_END
+
+#endif // defined(__cplusplus)

ios/tabs/RNSTabBarAppearanceCoordinator.h

@@ -7,7 +7,9 @@
 
 NS_ASSUME_NONNULL_BEGIN
 
+#if defined (__cplusplus)
 @class RCTImageLoader;
+#endif // defined(__cplusplus)
 
 /**
  * Responsible for creating & applying appearance to the tab bar.
@@ -17,6 +19,7 @@ NS_ASSUME_NONNULL_BEGIN
  */
 @interface RNSTabBarAppearanceCoordinator : NSObject
 
+#if defined (__cplusplus)
 /**
  * Applies the tab bar appearance props to the tab bar and respective tab bar items, basing on information contained in
  * provided params.
@@ -28,6 +31,7 @@ NS_ASSUME_NONNULL_BEGIN
            withHostComponentView:(nullable RNSTabsHostComponentView *)hostComponentView
             tabScreenControllers:(nullable NSArray<RNSTabsScreenViewController *> *)tabScreenCtrls
                      imageLoader:(nullable RCTImageLoader *)imageLoader;
+#endif // defined(__cplusplus)
 
 /**
  * Configures UITabBarAppearance object using appearance props provided in the param.