@office-iss/react-native-win32 0.0.0-canary.298 → 0.0.0-canary.299

package/Libraries/ReactNative/ReactFabricPublicInstance/ReactNativeAttributePayload.js

@@ -61,7 +61,7 @@ function restoreDeletedValuesInNestedArray(
   } else if (node && removedKeyCount > 0) {
     const obj = node;
     for (const propKey in removedKeys) {
-      // $FlowFixMe[incompatible-use] found when upgrading Flow
+      // $FlowFixMe[incompatible-use] removedKeys is always non-null
       if (!removedKeys[propKey]) {
         continue;
       }
@@ -133,12 +133,12 @@ function diffNestedArrayProperty(
     );
   }
   for (; i < nextArray.length; i++) {
-    // Add all remaining properties.
-    updatePayload = addNestedProperty(
-      updatePayload,
-      nextArray[i],
-      validAttributes,
-    );
+    // Add all remaining properties
+    const nextProp = nextArray[i];
+    if (!nextProp) {
+      continue;
+    }
+    updatePayload = addNestedProperty(updatePayload, nextProp, validAttributes);
   }
   return updatePayload;
 }
@@ -183,9 +183,7 @@ function diffNestedProperty(
   if (Array.isArray(prevProp)) {
     return diffProperties(
       updatePayload,
-      // $FlowFixMe - We know that this is always an object when the input is.
       flattenStyle(prevProp),
-      // $FlowFixMe - We know that this isn't an array because of above flow.
       nextProp,
       validAttributes,
     );
@@ -194,43 +192,11 @@ function diffNestedProperty(
   return diffProperties(
     updatePayload,
     prevProp,
-    // $FlowFixMe - We know that this is always an object when the input is.
     flattenStyle(nextProp),
     validAttributes,
   );
 }
 
-/**
- * addNestedProperty takes a single set of props and valid attribute
- * attribute configurations. It processes each prop and adds it to the
- * updatePayload.
- */
-function addNestedProperty(
-  updatePayload: null | Object,
-  nextProp: NestedNode,
-  validAttributes: AttributeConfiguration,
-): $FlowFixMe {
-  if (!nextProp) {
-    return updatePayload;
-  }
-
-  if (!Array.isArray(nextProp)) {
-    // Add each property of the leaf.
-    return addProperties(updatePayload, nextProp, validAttributes);
-  }
-
-  for (let i = 0; i < nextProp.length; i++) {
-    // Add all the properties of the array.
-    updatePayload = addNestedProperty(
-      updatePayload,
-      nextProp[i],
-      validAttributes,
-    );
-  }
-
-  return updatePayload;
-}
-
 /**
  * clearNestedProperty takes a single set of props and valid attributes. It
  * adds a null sentinel to the updatePayload, for each prop key.
@@ -285,14 +251,19 @@ function diffProperties(
     prevProp = prevProps[propKey];
     nextProp = nextProps[propKey];
 
-    // functions are converted to booleans as markers that the associated
-    // events should be sent from native.
     if (typeof nextProp === 'function') {
-      nextProp = (true: any);
-      // If nextProp is not a function, then don't bother changing prevProp
-      // since nextProp will win and go into the updatePayload regardless.
-      if (typeof prevProp === 'function') {
-        prevProp = (true: any);
+      const attributeConfigHasProcess =
+        typeof attributeConfig === 'object' &&
+        typeof attributeConfig.process === 'function';
+      if (!attributeConfigHasProcess) {
+        // functions are converted to booleans as markers that the associated
+        // events should be sent from native.
+        nextProp = (true: any);
+        // If nextProp is not a function, then don't bother changing prevProp
+        // since nextProp will win and go into the updatePayload regardless.
+        if (typeof prevProp === 'function') {
+          prevProp = (true: any);
+        }
       }
     }
 
@@ -442,16 +413,69 @@ function diffProperties(
   return updatePayload;
 }
 
-/**
- * addProperties adds all the valid props to the payload after being processed.
- */
-function addProperties(
-  updatePayload: null | Object,
+function addNestedProperty(
+  payload: null | Object,
   props: Object,
   validAttributes: AttributeConfiguration,
 ): null | Object {
-  // TODO: Fast path
-  return diffProperties(updatePayload, emptyObject, props, validAttributes);
+  // Flatten nested style props.
+  if (Array.isArray(props)) {
+    for (let i = 0; i < props.length; i++) {
+      payload = addNestedProperty(payload, props[i], validAttributes);
+    }
+    return payload;
+  }
+
+  for (const propKey in props) {
+    const prop = props[propKey];
+
+    const attributeConfig = ((validAttributes[
+      propKey
+    ]: any): AttributeConfiguration);
+
+    if (attributeConfig == null) {
+      continue;
+    }
+
+    let newValue;
+
+    if (prop === undefined) {
+      // Discard the prop if it was previously defined.
+      if (payload && payload[propKey] !== undefined) {
+        newValue = null;
+      } else {
+        continue;
+      }
+    } else if (typeof attributeConfig === 'object') {
+      if (typeof attributeConfig.process === 'function') {
+        // An atomic prop with custom processing.
+        newValue = attributeConfig.process(prop);
+      } else if (typeof attributeConfig.diff === 'function') {
+        // An atomic prop with custom diffing. We don't need to do diffing when adding props.
+        newValue = prop;
+      }
+    } else {
+      if (typeof prop === 'function') {
+        // A function prop. It represents an event handler. Pass it to native as 'true'.
+        newValue = true;
+      } else {
+        // An atomic prop. Doesn't need to be flattened.
+        newValue = prop;
+      }
+    }
+
+    if (newValue !== undefined) {
+      if (!payload) {
+        payload = ({}: {[string]: $FlowFixMe});
+      }
+      payload[propKey] = newValue;
+      continue;
+    }
+
+    payload = addNestedProperty(payload, prop, attributeConfig);
+  }
+
+  return payload;
 }
 
 /**
@@ -463,7 +487,6 @@ function clearProperties(
   prevProps: Object,
   validAttributes: AttributeConfiguration,
 ): null | Object {
-  // TODO: Fast path
   return diffProperties(updatePayload, prevProps, emptyObject, validAttributes);
 }
 
@@ -471,11 +494,7 @@ export function create(
   props: Object,
   validAttributes: AttributeConfiguration,
 ): null | Object {
-  return addProperties(
-    null, // updatePayload
-    props,
-    validAttributes,
-  );
+  return addNestedProperty(null, props, validAttributes);
 }
 
 export function diff(

package/Libraries/ReactPrivate/ReactNativePrivateInterface.js

@@ -51,7 +51,7 @@ export type {PublicRootInstance} from '../ReactNative/ReactFabricPublicInstance/
 export type PublicTextInstance = ReturnType<createPublicTextInstance>;
 
 // flowlint unsafe-getters-setters:off
-// eslint-disable-next-line lint/no-commonjs-exports
+// eslint-disable-next-line @react-native/monorepo/no-commonjs-exports
 module.exports = {
   get BatchedBridge(): BatchedBridge {
     return require('../BatchedBridge/BatchedBridge').default;

package/Libraries/ReactPrivate/ReactNativePrivateInterface.js.flow

@@ -8,7 +8,7 @@
  * @format
  */
 
-import typeof {createPublicTextInstance} from '../ReactNative/ReactFabricPublicInstance/ReactFabricPublicInstance';
+import typeof {createPublicTextInstance as createPublicTextInstanceT} from '../ReactNative/ReactFabricPublicInstance/ReactFabricPublicInstance';
 
 export type {
   HostInstance as PublicInstance,
@@ -21,7 +21,7 @@ export type {
 } from '../../src/private/types/HostInstance';
 
 export type {PublicRootInstance} from '../ReactNative/ReactFabricPublicInstance/ReactFabricPublicInstance';
-export type PublicTextInstance = ReturnType<createPublicTextInstance>;
+export type PublicTextInstance = ReturnType<createPublicTextInstanceT>;
 
 export {default as BatchedBridge} from '../BatchedBridge/BatchedBridge';
 export {default as ExceptionsManager} from '../Core/ExceptionsManager';

package/Libraries/Renderer/implementations/ReactFabric-dev.js

@@ -7,7 +7,7 @@
  * @noflow
  * @nolint
  * @preventMunge
- * @generated SignedSource<<d944291b21bfca6afd58b1fd5a118042>>
+ * @generated SignedSource<<6b3694149dd8856f0c741d3d94e05646>>
  *
  * This file was sync'd from the facebook/react repository.
  */
@@ -10561,6 +10561,8 @@ __DEV__ &&
             (offscreenSubtreeWasHidden ||
               null === current ||
               safelyDetachRef(current, current.return));
+          null !== finishedWork.alternate &&
+            (finishedWork.alternate.stateNode = finishedWork.stateNode);
           break;
         case 6:
           recursivelyTraverseMutationEffects(root, finishedWork);

package/Libraries/Renderer/implementations/ReactFabric-prod.js

@@ -7,7 +7,7 @@
  * @noflow
  * @nolint
  * @preventMunge
- * @generated SignedSource<<8c6274bec0d166bb311325a3c6c468b5>>
+ * @generated SignedSource<<7430b56dac3186eca669ae53bbf1f23d>>
  *
  * This file was sync'd from the facebook/react repository.
  */
@@ -8075,6 +8075,8 @@ function commitMutationEffectsOnFiber(finishedWork, root) {
         (offscreenSubtreeWasHidden ||
           null === current ||
           safelyDetachRef(current, current.return));
+      null !== finishedWork.alternate &&
+        (finishedWork.alternate.stateNode = finishedWork.stateNode);
       break;
     case 6:
       recursivelyTraverseMutationEffects(root, finishedWork);

package/Libraries/Renderer/implementations/ReactFabric-profiling.js

@@ -7,7 +7,7 @@
  * @noflow
  * @nolint
  * @preventMunge
- * @generated SignedSource<<49b14f3fcc953f4ddb02c1ef3f6e8c4e>>
+ * @generated SignedSource<<cf19390911979c415bb5e95c5270fba6>>
  *
  * This file was sync'd from the facebook/react repository.
  */
@@ -8530,6 +8530,8 @@ function commitMutationEffectsOnFiber(finishedWork, root) {
         (offscreenSubtreeWasHidden ||
           null === current ||
           safelyDetachRef(current, current.return));
+      null !== finishedWork.alternate &&
+        (finishedWork.alternate.stateNode = finishedWork.stateNode);
       break;
     case 6:
       recursivelyTraverseMutationEffects(root, finishedWork);

package/Libraries/Renderer/shims/ReactNativeTypes.js

@@ -7,7 +7,7 @@
  * @noformat
  * @nolint
  * @flow strict
- * @generated SignedSource<<4216c04f5a8c40a833b0146588fab5fa>>
+ * @generated SignedSource<<deb7924d11c790f99448a1c2f0edddb9>>
  */
 
 import type {
@@ -35,16 +35,7 @@ export type AttributeType<T, V> =
 export type AnyAttributeType = AttributeType<$FlowFixMe, $FlowFixMe>;
 
 export type AttributeConfiguration = $ReadOnly<{
-  [propName: string]: AnyAttributeType,
-  style: $ReadOnly<{
-    [propName: string]: AnyAttributeType,
-    ...
-  }>,
-  ...
-}>;
-
-export type PartialAttributeConfiguration = $ReadOnly<{
-  [propName: string]: AnyAttributeType,
+  [propName: string]: AnyAttributeType | void,
   style?: $ReadOnly<{
     [propName: string]: AnyAttributeType,
     ...
@@ -84,7 +75,7 @@ export type PartialViewConfig = $ReadOnly<{
   directEventTypes?: ViewConfig['directEventTypes'],
   supportsRawText?: boolean,
   uiViewClassName: string,
-  validAttributes?: PartialAttributeConfiguration,
+  validAttributes?: AttributeConfiguration,
 }>;
 
 type InspectorDataProps = $ReadOnly<{

package/Libraries/Renderer/shims/ReactNativeTypes.win32.js

@@ -7,7 +7,7 @@
  * @noformat
  * @nolint
  * @flow strict
- * @generated SignedSource<<4216c04f5a8c40a833b0146588fab5fa>>
+ * @generated SignedSource<<deb7924d11c790f99448a1c2f0edddb9>>
  */
 
 import type {
@@ -39,16 +39,7 @@ export type AttributeType<T, V> =
 export type AnyAttributeType = AttributeType<$FlowFixMe, $FlowFixMe>;
 
 export type AttributeConfiguration = $ReadOnly<{
-  [propName: string]: AnyAttributeType,
-  style: $ReadOnly<{
-    [propName: string]: AnyAttributeType,
-    ...
-  }>,
-  ...
-}>;
-
-export type PartialAttributeConfiguration = $ReadOnly<{
-  [propName: string]: AnyAttributeType,
+  [propName: string]: AnyAttributeType | void,
   style?: $ReadOnly<{
     [propName: string]: AnyAttributeType,
     ...
@@ -88,7 +79,7 @@ export type PartialViewConfig = $ReadOnly<{
   directEventTypes?: ViewConfig['directEventTypes'],
   supportsRawText?: boolean,
   uiViewClassName: string,
-  validAttributes?: PartialAttributeConfiguration,
+  validAttributes?: AttributeConfiguration,
 }>;
 
 type InspectorDataProps = $ReadOnly<{

package/Libraries/Settings/NativeSettingsManager.js

@@ -10,4 +10,5 @@
 
 export * from '../../src/private/specs_DEPRECATED/modules/NativeSettingsManager';
 import NativeSettingsManager from '../../src/private/specs_DEPRECATED/modules/NativeSettingsManager';
+
 export default NativeSettingsManager;

package/Libraries/Share/NativeShareModule.js

@@ -10,4 +10,5 @@
 
 export * from '../../src/private/specs_DEPRECATED/modules/NativeShareModule';
 import NativeShareModule from '../../src/private/specs_DEPRECATED/modules/NativeShareModule';
+
 export default NativeShareModule;

package/Libraries/StyleSheet/StyleSheet.js

@@ -25,6 +25,7 @@ import type {
   ____ViewStyleProp_Internal,
   NativeColorValue,
 } from './StyleSheetTypes';
+
 const StyleSheet: StyleSheetExports = (
   require('./StyleSheetExports') as $FlowFixMe
 ).default;

package/Libraries/StyleSheet/processAspectRatio.js

@@ -19,6 +19,8 @@ function processAspectRatio(aspectRatio?: number | string): ?number {
   if (typeof aspectRatio !== 'string') {
     if (__DEV__) {
       invariant(
+        /* $FlowFixMe[constant-condition] Error discovered during Constant
+         * Condition roll out. See https://fburl.com/workplace/1v97vimq. */
         !aspectRatio,
         'aspectRatio must either be a number, a ratio string or `auto`. You passed: %s',
         aspectRatio,

package/Libraries/Text/Text.js

@@ -19,7 +19,7 @@ import usePressability from '../Pressability/usePressability';
 import flattenStyle from '../StyleSheet/flattenStyle';
 import processColor from '../StyleSheet/processColor';
 import Platform from '../Utilities/Platform';
-import TextAncestor from './TextAncestor';
+import TextAncestorContext from './TextAncestorContext';
 import {NativeText, NativeVirtualText} from './TextNativeComponent';
 import * as React from 'react';
 import {useContext, useMemo, useState} from 'react';
@@ -169,7 +169,7 @@ const TextImpl: component(
 
   const _nativeID = id ?? nativeID;
 
-  const hasTextAncestor = useContext(TextAncestor);
+  const hasTextAncestor = useContext(TextAncestorContext);
   if (hasTextAncestor) {
     if (isPressable) {
       return (
@@ -302,7 +302,7 @@ const TextImpl: component(
   }
 
   // If the children do not contain a JSX element it would not be possible to have a
-  // nested `Text` component so we can skip adding the `TextAncestor` context wrapper
+  // nested `Text` component so we can skip adding the `TextAncestorContext` context wrapper
   // which has a performance overhead. Since we do this for performance reasons we need
   // to keep the check simple to avoid regressing overall perf. For this reason the
   // `children.length` constant is set to `3`, this should be a reasonable tradeoff
@@ -322,9 +322,7 @@ const TextImpl: component(
     return nativeText;
   }
 
-  return (
-    <TextAncestor.Provider value={true}>{nativeText}</TextAncestor.Provider>
-  );
+  return <TextAncestorContext value={true}>{nativeText}</TextAncestorContext>;
 };
 
 TextImpl.displayName = 'Text';

package/Libraries/Text/TextAncestor.js

@@ -8,16 +8,9 @@
  * @format
  */
 
-'use strict';
+// Compatibility module for ReactStrictDOMTextAncestorContext.native.js.flow (react-strict-dom)
+// TODO(huntie): Delete after we've fixed this cross-repo reference
 
-import * as React from 'react';
-import {createContext} from 'react';
+import TextAncestorContext from './TextAncestorContext';
 
-/**
- * Whether the current element is the descendant of a <Text> element.
- */
-const TextAncestorContext: React.Context<boolean> = createContext(false);
-if (__DEV__) {
-  TextAncestorContext.displayName = 'TextAncestorContext';
-}
 export default TextAncestorContext;

package/Libraries/Text/TextAncestorContext.js

@@ -0,0 +1,23 @@
+/**
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ * @flow strict
+ * @format
+ */
+
+'use strict';
+
+import * as React from 'react';
+import {createContext} from 'react';
+
+/**
+ * Whether the current element is the descendant of a <Text> element.
+ */
+const TextAncestorContext: React.Context<boolean> = createContext(false);
+if (__DEV__) {
+  TextAncestorContext.displayName = 'TextAncestorContext';
+}
+export default TextAncestorContext;

package/Libraries/Text/TextProps.js

@@ -32,7 +32,7 @@ export type PressRetentionOffset = $ReadOnly<{
   right: number,
 }>;
 
-type PointerEventProps = $ReadOnly<{
+type TextPointerEventProps = $ReadOnly<{
   onPointerEnter?: (event: PointerEvent) => void,
   onPointerLeave?: (event: PointerEvent) => void,
   onPointerMove?: (event: PointerEvent) => void,
@@ -269,7 +269,7 @@ type TextBaseProps = $ReadOnly<{
  * @see https://reactnative.dev/docs/text#reference
  */
 export type TextProps = $ReadOnly<{
-  ...PointerEventProps,
+  ...TextPointerEventProps,
   ...TextPropsIOS,
   ...TextPropsAndroid,
   ...TextBaseProps,

package/Libraries/Text/TextProps.win32.js

@@ -32,7 +32,7 @@ export type PressRetentionOffset = $ReadOnly<{
   right: number,
 }>;
 
-type PointerEventProps = $ReadOnly<{
+type TextPointerEventProps = $ReadOnly<{
   onPointerEnter?: (event: PointerEvent) => void,
   onPointerLeave?: (event: PointerEvent) => void,
   onPointerMove?: (event: PointerEvent) => void,
@@ -283,7 +283,7 @@ type TextBaseProps = $ReadOnly<{
  * @see https://reactnative.dev/docs/text#reference
  */
 export type TextProps = $ReadOnly<{
-  ...PointerEventProps,
+  ...TextPointerEventProps,
   ...TextPropsIOS,
   ...TextPropsAndroid,
   ...TextBaseProps,

package/Libraries/TurboModule/samples/NativeSampleTurboModule.js

@@ -10,4 +10,5 @@
 
 export * from '../../../src/private/specs_DEPRECATED/modules/NativeSampleTurboModule';
 import NativeSampleTurboModule from '../../../src/private/specs_DEPRECATED/modules/NativeSampleTurboModule';
+
 export default NativeSampleTurboModule;

package/Libraries/Types/CodegenTypesNamespace.js

@@ -11,4 +11,5 @@
 // This module exists to fit CodegenTypes into a namespace under Flow
 
 import * as CodegenTypes from './CodegenTypes';
+
 export type {CodegenTypes};

package/Libraries/Utilities/HMRClient.js

@@ -23,6 +23,7 @@ const prettyFormat = require('pretty-format');
 const pendingEntryPoints = [];
 let hmrClient = null;
 let hmrUnavailableReason: string | null = null;
+let hmrOrigin: string | null = null;
 let currentCompileErrorMessage: string | null = null;
 let didConnect: boolean = false;
 let pendingLogs: Array<[LogLevel, $ReadOnlyArray<mixed>]> = [];
@@ -100,7 +101,14 @@ const HMRClient: HMRClientNativeInterface = {
   },
 
   registerBundle(requestUrl: string) {
-    invariant(hmrClient, 'Expected HMRClient.setup() call at startup.');
+    invariant(
+      hmrOrigin != null && hmrClient != null,
+      'Expected HMRClient.setup() call at startup.',
+    );
+    // only process registerBundle calls from the same origin
+    if (!requestUrl.startsWith(hmrOrigin)) {
+      return;
+    }
     pendingEntryPoints.push(requestUrl);
     registerBundleEntryPoints(hmrClient);
   },
@@ -162,8 +170,10 @@ const HMRClient: HMRClientNativeInterface = {
 
     const serverScheme = scheme;
 
-    const client = new MetroHMRClient(`${serverScheme}://${serverHost}/hot`);
+    const origin = `${serverScheme}://${serverHost}`;
+    const client = new MetroHMRClient(`${origin}/hot`);
 
+    hmrOrigin = origin;
     hmrClient = client;
 
     const {fullBundleUrl} = getDevServer();

package/Libraries/Utilities/NativeAppearance.js

@@ -10,4 +10,5 @@
 
 export type * from '../../src/private/specs_DEPRECATED/modules/NativeAppearance';
 import NativeAppearance from '../../src/private/specs_DEPRECATED/modules/NativeAppearance';
+
 export default NativeAppearance;

package/Libraries/Utilities/NativeDevLoadingView.js

@@ -10,4 +10,5 @@
 
 export * from '../../src/private/specs_DEPRECATED/modules/NativeDevLoadingView';
 import NativeDevLoadingView from '../../src/private/specs_DEPRECATED/modules/NativeDevLoadingView';
+
 export default NativeDevLoadingView;

package/Libraries/Utilities/NativeDeviceInfo.js

@@ -9,5 +9,4 @@
  */
 
 export * from '../../src/private/specs_DEPRECATED/modules/NativeDeviceInfo';
-import NativeDeviceInfo from '../../src/private/specs_DEPRECATED/modules/NativeDeviceInfo';
-export default NativeDeviceInfo;
+export {default} from '../../src/private/specs_DEPRECATED/modules/NativeDeviceInfo';

package/Libraries/Utilities/NativePlatformConstantsAndroid.js

@@ -10,4 +10,5 @@
 
 export * from '../../src/private/specs_DEPRECATED/modules/NativePlatformConstantsAndroid';
 import NativePlatformConstantsAndroid from '../../src/private/specs_DEPRECATED/modules/NativePlatformConstantsAndroid';
+
 export default NativePlatformConstantsAndroid;

package/Libraries/Utilities/NativePlatformConstantsIOS.js

@@ -10,4 +10,5 @@
 
 export * from '../../src/private/specs_DEPRECATED/modules/NativePlatformConstantsIOS';
 import NativePlatformConstantsIOS from '../../src/private/specs_DEPRECATED/modules/NativePlatformConstantsIOS';
+
 export default NativePlatformConstantsIOS;

package/Libraries/Utilities/NativePlatformConstantsWin.js

@@ -8,4 +8,5 @@
 
 export * from '../../src/private/specs_DEPRECATED/modules/NativePlatformConstantsWin';
 import NativePlatformConstantsWin from '../../src/private/specs_DEPRECATED/modules/NativePlatformConstantsWin';
+
 export default NativePlatformConstantsWin;

package/Libraries/Utilities/Platform.android.js

@@ -8,10 +8,7 @@
  * @format
  */
 
-import type {
-  Platform as PlatformType,
-  PlatformSelectSpec,
-} from './PlatformTypes';
+import type {PlatformSelectSpec, PlatformType} from './PlatformTypes';
 
 import NativePlatformConstantsAndroid from './NativePlatformConstantsAndroid';
 

package/Libraries/Utilities/Platform.ios.js

@@ -8,10 +8,7 @@
  * @format
  */
 
-import type {
-  Platform as PlatformType,
-  PlatformSelectSpec,
-} from './PlatformTypes';
+import type {PlatformSelectSpec, PlatformType} from './PlatformTypes';
 
 import NativePlatformConstantsIOS from './NativePlatformConstantsIOS';
 

package/Libraries/Utilities/Platform.js.flow

@@ -8,6 +8,6 @@
  * @format
  */
 
-import type {Platform} from './PlatformTypes';
+import type {PlatformType} from './PlatformTypes';
 
-declare export default Platform;
+declare export default PlatformType;

package/Libraries/Utilities/Platform.win32.js

@@ -6,10 +6,7 @@
  * @format
  */
 
-import type {
-  Platform as PlatformType,
-  PlatformSelectSpec,
-} from './PlatformTypes';
+import type {PlatformSelectSpec, PlatformType} from './PlatformTypes';
 
 import NativePlatformConstantsWin from './NativePlatformConstantsWin';