Disallow usage of "NativeProps" symbol in internal components (#51889)

Summary:
Pull Request resolved: #51889

Changelog: [Internal]

Reviewed By: huntie

Differential Revision: D76244543

fbshipit-source-id: 0ccbc29f99e3fac992b8f0040e16a73f72350969

Author: j-piasecki

packages/react-native/Libraries/Components/TextInput/AndroidTextInputNativeComponent.js

@@ -63,7 +63,7 @@ export type ReturnKeyType =
 
 export type SubmitBehavior = 'submit' | 'blurAndSubmit' | 'newline';
 
-export type NativeProps = $ReadOnly<{
+export type AndroidTextInputNativeProps = $ReadOnly<{
   // This allows us to inherit everything from ViewProps except for style (see below)
   // This must be commented for Fabric codegen to work.
   ...Omit<ViewProps, 'style'>,
@@ -606,7 +606,7 @@ export type NativeProps = $ReadOnly<{
   text?: ?string,
 }>;
 
-type NativeType = HostComponent<NativeProps>;
+type NativeType = HostComponent<AndroidTextInputNativeProps>;
 
 type NativeCommands = TextInputNativeCommands<NativeType>;
 
@@ -725,10 +725,11 @@ export const __INTERNAL_VIEW_CONFIG: PartialViewConfig = {
   },
 };
 
-let AndroidTextInputNativeComponent = NativeComponentRegistry.get<NativeProps>(
-  'AndroidTextInput',
-  () => __INTERNAL_VIEW_CONFIG,
-);
+let AndroidTextInputNativeComponent =
+  NativeComponentRegistry.get<AndroidTextInputNativeProps>(
+    'AndroidTextInput',
+    () => __INTERNAL_VIEW_CONFIG,
+  );
 
 // flowlint-next-line unclear-type:off
-export default ((AndroidTextInputNativeComponent: any): HostComponent<NativeProps>);
+export default ((AndroidTextInputNativeComponent: any): HostComponent<AndroidTextInputNativeProps>);

packages/react-native/Libraries/Image/TextInlineImageNativeComponent.js

@@ -18,7 +18,7 @@ import type {ImageResizeMode} from './ImageResizeMode';
 
 import * as NativeComponentRegistry from '../NativeComponent/NativeComponentRegistry';
 
-type NativeProps = $ReadOnly<{
+type RCTTextInlineImageNativeProps = $ReadOnly<{
   ...ViewProps,
   resizeMode?: ?ImageResizeMode,
   src?: ?$ReadOnlyArray<?$ReadOnly<{uri?: ?string, ...}>>,
@@ -40,8 +40,8 @@ export const __INTERNAL_VIEW_CONFIG: PartialViewConfig = {
   },
 };
 
-const TextInlineImage: HostComponent<NativeProps> =
-  NativeComponentRegistry.get<NativeProps>(
+const TextInlineImage: HostComponent<RCTTextInlineImageNativeProps> =
+  NativeComponentRegistry.get<RCTTextInlineImageNativeProps>(
     'RCTTextInlineImage',
     () => __INTERNAL_VIEW_CONFIG,
   );

packages/react-native/Libraries/__tests__/__snapshots__/public-api-test.js.snap

@@ -2436,7 +2436,7 @@ export type ReturnKeyType =
   | \\"route\\"
   | \\"yahoo\\";
 export type SubmitBehavior = \\"submit\\" | \\"blurAndSubmit\\" | \\"newline\\";
-export type NativeProps = $ReadOnly<{
+export type AndroidTextInputNativeProps = $ReadOnly<{
   ...Omit<ViewProps, \\"style\\">,
   autoComplete?: WithDefault<
     | \\"birthdate-day\\"
@@ -2594,11 +2594,11 @@ export type NativeProps = $ReadOnly<{
   mostRecentEventCount: Int32,
   text?: ?string,
 }>;
-type NativeType = HostComponent<NativeProps>;
+type NativeType = HostComponent<AndroidTextInputNativeProps>;
 type NativeCommands = TextInputNativeCommands<NativeType>;
 declare export const Commands: NativeCommands;
 declare export const __INTERNAL_VIEW_CONFIG: PartialViewConfig;
-declare export default HostComponent<NativeProps>;
+declare export default HostComponent<AndroidTextInputNativeProps>;
 "
 `;
 
@@ -4616,15 +4616,15 @@ declare module.exports: typeof RelativeImageStub;
 `;
 
 exports[`public API should not change unintentionally Libraries/Image/TextInlineImageNativeComponent.js 1`] = `
-"type NativeProps = $ReadOnly<{
+"type RCTTextInlineImageNativeProps = $ReadOnly<{
   ...ViewProps,
   resizeMode?: ?ImageResizeMode,
   src?: ?$ReadOnlyArray<?$ReadOnly<{ uri?: ?string, ... }>>,
   tintColor?: ?ColorValue,
   headers?: ?{ [string]: string },
 }>;
 declare export const __INTERNAL_VIEW_CONFIG: PartialViewConfig;
-declare const TextInlineImage: HostComponent<NativeProps>;
+declare const TextInlineImage: HostComponent<RCTTextInlineImageNativeProps>;
 declare export default typeof TextInlineImage;
 "
 `;

packages/react-native/src/private/specs_DEPRECATED/components/ActivityIndicatorViewNativeComponent.js

@@ -15,7 +15,7 @@ import type {HostComponent} from '../../types/HostComponent';
 
 import codegenNativeComponent from '../../../../Libraries/Utilities/codegenNativeComponent';
 
-type NativeProps = $ReadOnly<{
+type RCTActivityIndicatorViewNativeProps = $ReadOnly<{
   ...ViewProps,
 
   /**
@@ -48,6 +48,9 @@ type NativeProps = $ReadOnly<{
   size?: WithDefault<'small' | 'large', 'small'>,
 }>;
 
-export default (codegenNativeComponent<NativeProps>('ActivityIndicatorView', {
-  paperComponentName: 'RCTActivityIndicatorView',
-}): HostComponent<NativeProps>);
+export default (codegenNativeComponent<RCTActivityIndicatorViewNativeProps>(
+  'ActivityIndicatorView',
+  {
+    paperComponentName: 'RCTActivityIndicatorView',
+  },
+): HostComponent<RCTActivityIndicatorViewNativeProps>);

packages/react-native/src/private/specs_DEPRECATED/components/AndroidDrawerLayoutNativeComponent.js

@@ -30,7 +30,7 @@ type DrawerSlideEvent = $ReadOnly<{
   offset: Float,
 }>;
 
-type NativeProps = $ReadOnly<{
+type AndroidDrawerLayoutNativeProps = $ReadOnly<{
   ...ViewProps,
   /**
    * Determines whether the keyboard gets dismissed in response to a drag.
@@ -108,7 +108,7 @@ type NativeProps = $ReadOnly<{
   statusBarBackgroundColor?: ?ColorValue,
 }>;
 
-type NativeType = HostComponent<NativeProps>;
+type NativeType = HostComponent<AndroidDrawerLayoutNativeProps>;
 
 interface NativeCommands {
   +openDrawer: (viewRef: React.ElementRef<NativeType>) => void;
@@ -119,6 +119,6 @@ export const Commands: NativeCommands = codegenNativeCommands<NativeCommands>({
   supportedCommands: ['openDrawer', 'closeDrawer'],
 });
 
-export default (codegenNativeComponent<NativeProps>(
+export default (codegenNativeComponent<AndroidDrawerLayoutNativeProps>(
   'AndroidDrawerLayout',
 ): NativeType);

packages/react-native/src/private/specs_DEPRECATED/components/AndroidHorizontalScrollContentViewNativeComponent.js

@@ -13,15 +13,15 @@ import type {HostComponent} from '../../types/HostComponent';
 
 import codegenNativeComponent from '../../../../Libraries/Utilities/codegenNativeComponent';
 
-type NativeProps = $ReadOnly<{
+type AndroidHorizontalScrollContentViewNativeProps = $ReadOnly<{
   ...ViewProps,
 
   removeClippedSubviews?: ?boolean,
 }>;
 
-type NativeType = HostComponent<NativeProps>;
+type NativeType = HostComponent<AndroidHorizontalScrollContentViewNativeProps>;
 
-export default (codegenNativeComponent<NativeProps>(
+export default (codegenNativeComponent<AndroidHorizontalScrollContentViewNativeProps>(
   'AndroidHorizontalScrollContentView',
   {interfaceOnly: true},
 ): NativeType);

packages/react-native/src/private/specs_DEPRECATED/components/AndroidSwipeRefreshLayoutNativeComponent.js

@@ -21,7 +21,7 @@ import codegenNativeCommands from '../../../../Libraries/Utilities/codegenNative
 import codegenNativeComponent from '../../../../Libraries/Utilities/codegenNativeComponent';
 import * as React from 'react';
 
-type NativeProps = $ReadOnly<{
+type AndroidSwipeRefreshLayoutNativeProps = $ReadOnly<{
   ...ViewProps,
 
   /**
@@ -56,7 +56,7 @@ type NativeProps = $ReadOnly<{
   refreshing: boolean,
 }>;
 
-type NativeType = HostComponent<NativeProps>;
+type NativeType = HostComponent<AndroidSwipeRefreshLayoutNativeProps>;
 
 interface NativeCommands {
   +setNativeRefreshing: (
@@ -69,6 +69,6 @@ export const Commands: NativeCommands = codegenNativeCommands<NativeCommands>({
   supportedCommands: ['setNativeRefreshing'],
 });
 
-export default (codegenNativeComponent<NativeProps>(
+export default (codegenNativeComponent<AndroidSwipeRefreshLayoutNativeProps>(
   'AndroidSwipeRefreshLayout',
 ): NativeType);

packages/react-native/src/private/specs_DEPRECATED/components/AndroidSwitchNativeComponent.js

@@ -21,12 +21,12 @@ import codegenNativeCommands from '../../../../Libraries/Utilities/codegenNative
 import codegenNativeComponent from '../../../../Libraries/Utilities/codegenNativeComponent';
 import * as React from 'react';
 
-type SwitchChangeEvent = $ReadOnly<{
+type AndroidSwitchChangeEvent = $ReadOnly<{
   value: boolean,
   target: Int32,
 }>;
 
-type NativeProps = $ReadOnly<{
+type AndroidSwitchNativeProps = $ReadOnly<{
   ...ViewProps,
 
   // Props
@@ -41,10 +41,10 @@ type NativeProps = $ReadOnly<{
   trackTintColor?: ?ColorValue,
 
   // Events
-  onChange?: BubblingEventHandler<SwitchChangeEvent>,
+  onChange?: BubblingEventHandler<AndroidSwitchChangeEvent>,
 }>;
 
-type NativeType = HostComponent<NativeProps>;
+type NativeType = HostComponent<AndroidSwitchNativeProps>;
 
 interface NativeCommands {
   +setNativeValue: (
@@ -57,6 +57,9 @@ export const Commands: NativeCommands = codegenNativeCommands<NativeCommands>({
   supportedCommands: ['setNativeValue'],
 });
 
-export default (codegenNativeComponent<NativeProps>('AndroidSwitch', {
-  interfaceOnly: true,
-}): NativeType);
+export default (codegenNativeComponent<AndroidSwitchNativeProps>(
+  'AndroidSwitch',
+  {
+    interfaceOnly: true,
+  },
+): NativeType);

packages/react-native/src/private/specs_DEPRECATED/components/DebuggingOverlayNativeComponent.js

@@ -16,10 +16,11 @@ import codegenNativeCommands from '../../../../Libraries/Utilities/codegenNative
 import codegenNativeComponent from '../../../../Libraries/Utilities/codegenNativeComponent';
 import * as React from 'react';
 
-type NativeProps = $ReadOnly<{
+type DebuggingOverlayNativeProps = $ReadOnly<{
   ...ViewProps,
 }>;
-export type DebuggingOverlayNativeComponentType = HostComponent<NativeProps>;
+export type DebuggingOverlayNativeComponentType =
+  HostComponent<DebuggingOverlayNativeProps>;
 
 export type TraceUpdate = {
   id: number,
@@ -56,6 +57,6 @@ export const Commands: NativeCommands = codegenNativeCommands<NativeCommands>({
   ],
 });
 
-export default (codegenNativeComponent<NativeProps>(
+export default (codegenNativeComponent<DebuggingOverlayNativeProps>(
   'DebuggingOverlay',
-): HostComponent<NativeProps>);
+): HostComponent<DebuggingOverlayNativeProps>);

packages/react-native/src/private/specs_DEPRECATED/components/ProgressBarAndroidNativeComponent.js

@@ -18,7 +18,7 @@ import type {HostComponent} from '../../types/HostComponent';
 
 import codegenNativeComponent from '../../../../Libraries/Utilities/codegenNativeComponent';
 
-type NativeProps = $ReadOnly<{
+type AndroidProgressBarNativeProps = $ReadOnly<{
   ...ViewProps,
 
   //Props
@@ -31,6 +31,9 @@ type NativeProps = $ReadOnly<{
   testID?: WithDefault<string, ''>,
 }>;
 
-export default (codegenNativeComponent<NativeProps>('AndroidProgressBar', {
-  interfaceOnly: true,
-}): HostComponent<NativeProps>);
+export default (codegenNativeComponent<AndroidProgressBarNativeProps>(
+  'AndroidProgressBar',
+  {
+    interfaceOnly: true,
+  },
+): HostComponent<AndroidProgressBarNativeProps>);