Fix variance in react-native-github/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/ (#56992)

marcoww6 · meta-codesync[bot] · commit 707e366585c5 · 2026-05-28T20:34:54.000-07:00
Summary: Pull Request resolved: #56992 tsia Changelog: [Internal] landed-with-radar-review Reviewed By: SamChou19815 Differential Revision: D106707074 fbshipit-source-id: 0db19484d36a988b4382b37e43c5a43a49f028ec
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeArrayTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeArrayTurboModule.js
@@ -16,9 +16,12 @@ export type ArrayType = string;
 type AnotherArray = Array<ArrayType>;
 
 export interface Spec extends TurboModule {
-  +getArray: (a: Array<any>) => Array<string>;
-  +getReadOnlyArray: (a: Array<any>) => ReadonlyArray<string>;
-  +getArrayWithAlias: (a: AnotherArray, b: Array<ArrayType>) => AnotherArray;
+  readonly getArray: (a: Array<any>) => Array<string>;
+  readonly getReadOnlyArray: (a: Array<any>) => ReadonlyArray<string>;
+  readonly getArrayWithAlias: (
+    a: AnotherArray,
+    b: Array<ArrayType>,
+  ) => AnotherArray;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeBooleanTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeBooleanTurboModule.js
@@ -16,8 +16,8 @@ export type Boolean = boolean;
 type AnotherBoolean = Boolean;
 
 export interface Spec extends TurboModule {
-  +getBoolean: (arg: boolean) => boolean;
-  +getBooleanWithAlias: (arg: Boolean) => AnotherBoolean;
+  readonly getBoolean: (arg: boolean) => boolean;
+  readonly getBooleanWithAlias: (arg: Boolean) => AnotherBoolean;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeCallbackTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeCallbackTurboModule.js
@@ -16,8 +16,8 @@ export type String = string;
 type CB = (value: String) => void;
 
 export interface Spec extends TurboModule {
-  +getValueWithCallback: (callback: (value: string) => void) => void;
-  +getValueWithCallbackWithAlias: (c: CB) => void;
+  readonly getValueWithCallback: (callback: (value: string) => void) => void;
+  readonly getValueWithCallbackWithAlias: (c: CB) => void;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeEnumTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeEnumTurboModule.js
@@ -49,17 +49,17 @@ export type StateTypeWithEnums = {
 };
 
 export interface Spec extends TurboModule {
-  +getStatusRegular: (statusProp: StateType) => StatusRegularEnum;
-  +getStatusStr: (statusProp: StateType) => StatusStrEnum;
-  +getStatusNum: (statusProp: StateType) => StatusNumEnum;
-  +getStatusLowerCase: (statusProp: StateType) => StatusLowerCaseEnum;
-  +getStateType: (
+  readonly getStatusRegular: (statusProp: StateType) => StatusRegularEnum;
+  readonly getStatusStr: (statusProp: StateType) => StatusStrEnum;
+  readonly getStatusNum: (statusProp: StateType) => StatusNumEnum;
+  readonly getStatusLowerCase: (statusProp: StateType) => StatusLowerCaseEnum;
+  readonly getStateType: (
     a: StatusRegularEnum,
     b: StatusStrEnum,
     c: StatusNumEnum,
     d: StatusLowerCaseEnum,
   ) => StateType;
-  +getStateTypeWithEnums: (
+  readonly getStateTypeWithEnums: (
     paramOfTypeWithEnums: StateTypeWithEnums,
   ) => StateTypeWithEnums;
 }
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeNullableTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeNullableTurboModule.js
@@ -13,12 +13,12 @@ import type {TurboModule} from 'react-native/Libraries/TurboModule/RCTExport';
 import * as TurboModuleRegistry from 'react-native/Libraries/TurboModule/TurboModuleRegistry';
 
 export interface Spec extends TurboModule {
-  +getBool: (a: ?boolean) => ?boolean;
-  +getNumber: (a: ?number) => ?number;
-  +getString: (a: ?number) => ?string;
-  +getArray: (a: ?Array<any>) => ?Array<any>;
-  +getObject: (a: ?Object) => ?Object;
-  +getValueWithPromise: () => ?Promise<string>;
+  readonly getBool: (a: ?boolean) => ?boolean;
+  readonly getNumber: (a: ?number) => ?number;
+  readonly getString: (a: ?number) => ?string;
+  readonly getArray: (a: ?Array<any>) => ?Array<any>;
+  readonly getObject: (a: ?Object) => ?Object;
+  readonly getValueWithPromise: () => ?Promise<string>;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeNumberTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeNumberTurboModule.js
@@ -16,8 +16,8 @@ export type Number = number;
 type AnotherNumber = Number;
 
 export interface Spec extends TurboModule {
-  +getNumber: (arg: number) => number;
-  +getNumberWithAlias: (arg: Number) => AnotherNumber;
+  readonly getNumber: (arg: number) => number;
+  readonly getNumberWithAlias: (arg: Number) => AnotherNumber;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeObjectTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeObjectTurboModule.js
@@ -16,10 +16,12 @@ export type GenericObject = Object;
 type AnotherGenericObject = GenericObject;
 
 export interface Spec extends TurboModule {
-  +getGenericObject: (arg: Object) => Object;
-  +getGenericObjectReadOnly: (arg: Object) => Readonly<{a: string}>;
-  +getGenericObjectWithAlias: (arg: GenericObject) => AnotherGenericObject;
-  +difficultObject: (A: {
+  readonly getGenericObject: (arg: Object) => Object;
+  readonly getGenericObjectReadOnly: (arg: Object) => Readonly<{a: string}>;
+  readonly getGenericObjectWithAlias: (
+    arg: GenericObject,
+  ) => AnotherGenericObject;
+  readonly difficultObject: (A: {
     D: boolean,
     E: {
       D: boolean,
@@ -40,7 +42,7 @@ export interface Spec extends TurboModule {
     },
     F: string,
   };
-  +getConstants: () => {
+  readonly getConstants: () => {
     D: boolean,
     E: {
       D: boolean,
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeOptionalObjectTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeOptionalObjectTurboModule.js
@@ -13,7 +13,7 @@ import type {TurboModule} from 'react-native/Libraries/TurboModule/RCTExport';
 import * as TurboModuleRegistry from 'react-native/Libraries/TurboModule/TurboModuleRegistry';
 
 export interface Spec extends TurboModule {
-  +getConstants: () => {
+  readonly getConstants: () => {
     D?: ?boolean,
     A?: Array<any>,
     E?: ?{
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativePartialAnnotationTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativePartialAnnotationTurboModule.js
@@ -22,10 +22,10 @@ export type SomeObj = {
 export type PartialSomeObj = Partial<SomeObj>;
 
 export interface Spec extends TurboModule {
-  +getSomeObj: () => SomeObj;
-  +getPartialSomeObj: () => Partial<SomeObj>;
-  +getSomeObjFromPartialSomeObj: (value: Partial<SomeObj>) => SomeObj;
-  +getPartialPartial: (
+  readonly getSomeObj: () => SomeObj;
+  readonly getPartialSomeObj: () => Partial<SomeObj>;
+  readonly getSomeObjFromPartialSomeObj: (value: Partial<SomeObj>) => SomeObj;
+  readonly getPartialPartial: (
     value1: Partial<SomeObj>,
     value2: PartialSomeObj,
   ) => SomeObj;
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativePromiseTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativePromiseTurboModule.js
@@ -16,8 +16,8 @@ export type String = string;
 type AnotherPromise = Promise<String>;
 
 export interface Spec extends TurboModule {
-  +getValueWithPromise: (error: boolean) => Promise<string>;
-  +getValueWithPromiseWithAlias: (arg: String) => AnotherPromise;
+  readonly getValueWithPromise: (error: boolean) => Promise<string>;
+  readonly getValueWithPromiseWithAlias: (arg: String) => AnotherPromise;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeSampleTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeSampleTurboModule.js
@@ -21,23 +21,23 @@ type Animal = {
 
 export interface Spec extends TurboModule {
   // Exported methods.
-  +getConstants: () => {
+  readonly getConstants: () => {
     const1: boolean,
     const2: number,
     const3: string,
   };
-  +voidFunc: () => void;
-  +getBool: (arg: boolean) => boolean;
-  +getNumber: (arg: number) => number;
-  +getString: (arg: string) => string;
-  +getArray: (arg: Array<any>) => Array<any>;
-  +getObject: (arg: Object) => Object;
-  +getObjectShape: (arg: {prop: number}) => {prop: number};
-  +getAlias: (arg: Animal) => Animal;
-  +getRootTag: (arg: RootTag) => RootTag;
-  +getValue: (x: number, y: string, z: Object) => Object;
-  +getValueWithCallback: (callback: (value: string) => void) => void;
-  +getValueWithPromise: (error: boolean) => Promise<string>;
+  readonly voidFunc: () => void;
+  readonly getBool: (arg: boolean) => boolean;
+  readonly getNumber: (arg: number) => number;
+  readonly getString: (arg: string) => string;
+  readonly getArray: (arg: Array<any>) => Array<any>;
+  readonly getObject: (arg: Object) => Object;
+  readonly getObjectShape: (arg: {prop: number}) => {prop: number};
+  readonly getAlias: (arg: Animal) => Animal;
+  readonly getRootTag: (arg: RootTag) => RootTag;
+  readonly getValue: (x: number, y: string, z: Object) => Object;
+  readonly getValueWithCallback: (callback: (value: string) => void) => void;
+  readonly getValueWithPromise: (error: boolean) => Promise<string>;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeSampleTurboModuleArrays.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeSampleTurboModuleArrays.js
@@ -21,28 +21,34 @@ type Animal = {
 
 export interface Spec extends TurboModule {
   // Exported methods.
-  +getConstants: () => {
+  readonly getConstants: () => {
     const1: Array<boolean>,
     const2: Array<number>,
     const3: Array<string>,
     id?: Array<?{prop: number}>,
   };
-  +voidFunc: () => void;
-  +getBool: (id: Array<boolean>) => Array<boolean>;
-  +getNumber: (arg: Array<number>) => Array<number>;
-  +getString: (arg: Array<string>) => Array<string>;
-  +getArray: (arg: Array<Array<any>>) => Array<Array<any>>;
-  +getObject: (arg: Array<Object>) => Array<Object>;
-  +getObjectShape: (arg: Array<{prop: number}>) => Array<{prop: number}>;
-  +getAlias: (arg: Array<Animal>) => Array<Animal>;
-  +getRootTag: (arg: Array<RootTag>) => Array<RootTag>;
-  +getValue: (
+  readonly voidFunc: () => void;
+  readonly getBool: (id: Array<boolean>) => Array<boolean>;
+  readonly getNumber: (arg: Array<number>) => Array<number>;
+  readonly getString: (arg: Array<string>) => Array<string>;
+  readonly getArray: (arg: Array<Array<any>>) => Array<Array<any>>;
+  readonly getObject: (arg: Array<Object>) => Array<Object>;
+  readonly getObjectShape: (
+    arg: Array<{prop: number}>,
+  ) => Array<{prop: number}>;
+  readonly getAlias: (arg: Array<Animal>) => Array<Animal>;
+  readonly getRootTag: (arg: Array<RootTag>) => Array<RootTag>;
+  readonly getValue: (
     x: Array<number>,
     y: Array<string>,
     z: Array<Object>,
   ) => Array<Object>;
-  +getValueWithCallback: (callback: (value: Array<string>) => void) => void;
-  +getValueWithPromise: (error: Array<boolean>) => Promise<Array<string>>;
+  readonly getValueWithCallback: (
+    callback: (value: Array<string>) => void,
+  ) => void;
+  readonly getValueWithPromise: (
+    error: Array<boolean>,
+  ) => Promise<Array<string>>;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeSampleTurboModuleNullable.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeSampleTurboModuleNullable.js
@@ -21,23 +21,23 @@ type Animal = ?{
 
 export interface Spec extends TurboModule {
   // Exported methods.
-  +getConstants: () => {
+  readonly getConstants: () => {
     const1: ?boolean,
     const2: ?number,
     const3: ?string,
   };
-  +voidFunc: () => void;
-  +getBool: (arg: ?boolean) => ?boolean;
-  +getNumber: (arg: ?number) => ?number;
-  +getString: (arg: ?string) => ?string;
-  +getArray: (arg: ?Array<any>) => ?Array<any>;
-  +getObject: (arg: ?Object) => ?Object;
-  +getObjectShape: (arg: ?{prop: ?number}) => ?{prop: ?number};
-  +getAlias: (arg: ?Animal) => ?Animal;
-  +getRootTag: (arg: ?RootTag) => ?RootTag;
-  +getValue: (x: ?number, y: ?string, z: ?Object) => ?Object;
-  +getValueWithCallback: (callback: (value: ?string) => void) => void;
-  +getValueWithPromise: (error: ?boolean) => ?Promise<string>;
+  readonly voidFunc: () => void;
+  readonly getBool: (arg: ?boolean) => ?boolean;
+  readonly getNumber: (arg: ?number) => ?number;
+  readonly getString: (arg: ?string) => ?string;
+  readonly getArray: (arg: ?Array<any>) => ?Array<any>;
+  readonly getObject: (arg: ?Object) => ?Object;
+  readonly getObjectShape: (arg: ?{prop: ?number}) => ?{prop: ?number};
+  readonly getAlias: (arg: ?Animal) => ?Animal;
+  readonly getRootTag: (arg: ?RootTag) => ?RootTag;
+  readonly getValue: (x: ?number, y: ?string, z: ?Object) => ?Object;
+  readonly getValueWithCallback: (callback: (value: ?string) => void) => void;
+  readonly getValueWithPromise: (error: ?boolean) => ?Promise<string>;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeSampleTurboModuleNullableAndOptional.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeSampleTurboModuleNullableAndOptional.js
@@ -21,23 +21,25 @@ type Animal = ?{
 
 export interface Spec extends TurboModule {
   // Exported methods.
-  +getConstants?: () => {
+  readonly getConstants?: () => {
     const1?: ?boolean,
     const2?: ?number,
     const3?: ?string,
   };
-  +voidFunc?: () => void;
-  +getBool?: (arg?: ?boolean) => ?boolean;
-  +getNumber?: (arg?: ?number) => ?number;
-  +getString?: (arg?: ?string) => ?string;
-  +getArray?: (arg?: ?Array<any>) => ?Array<any>;
-  +getObject?: (arg?: ?Object) => ?Object;
-  +getObjectShape?: (arg?: {prop?: ?number}) => {prop?: ?number};
-  +getAlias?: (arg?: ?Animal) => ?Animal;
-  +getRootTag?: (arg?: ?RootTag) => ?RootTag;
-  +getValue?: (x?: ?number, y?: ?string, z?: ?Object) => ?Object;
-  +getValueWithCallback?: (callback?: ?(value?: ?string) => void) => void;
-  +getValueWithPromise?: (error?: ?boolean) => ?Promise<string>;
+  readonly voidFunc?: () => void;
+  readonly getBool?: (arg?: ?boolean) => ?boolean;
+  readonly getNumber?: (arg?: ?number) => ?number;
+  readonly getString?: (arg?: ?string) => ?string;
+  readonly getArray?: (arg?: ?Array<any>) => ?Array<any>;
+  readonly getObject?: (arg?: ?Object) => ?Object;
+  readonly getObjectShape?: (arg?: {prop?: ?number}) => {prop?: ?number};
+  readonly getAlias?: (arg?: ?Animal) => ?Animal;
+  readonly getRootTag?: (arg?: ?RootTag) => ?RootTag;
+  readonly getValue?: (x?: ?number, y?: ?string, z?: ?Object) => ?Object;
+  readonly getValueWithCallback?: (
+    callback?: ?(value?: ?string) => void,
+  ) => void;
+  readonly getValueWithPromise?: (error?: ?boolean) => ?Promise<string>;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeSampleTurboModuleOptional.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeSampleTurboModuleOptional.js
@@ -21,23 +21,23 @@ type Animal = {
 
 export interface Spec extends TurboModule {
   // Exported methods.
-  +getConstants?: () => {
+  readonly getConstants?: () => {
     const1?: boolean,
     const2?: number,
     const3?: string,
   };
-  +voidFunc?: () => void;
-  +getBool?: (arg?: boolean) => boolean;
-  +getNumber?: (arg?: number) => number;
-  +getString?: (arg?: string) => string;
-  +getArray?: (arg?: Array<any>) => Array<any>;
-  +getObject?: (arg?: Object) => Object;
-  +getObjectShape?: (arg?: {prop?: number}) => {prop?: number};
-  +getAlias?: (arg?: Animal) => Animal;
-  +getRootTag?: (arg?: RootTag) => RootTag;
-  +getValue?: (x?: number, y?: string, z?: Object) => Object;
-  +getValueWithCallback?: (callback?: (value?: string) => void) => void;
-  +getValueWithPromise?: (error?: boolean) => Promise<string>;
+  readonly voidFunc?: () => void;
+  readonly getBool?: (arg?: boolean) => boolean;
+  readonly getNumber?: (arg?: number) => number;
+  readonly getString?: (arg?: string) => string;
+  readonly getArray?: (arg?: Array<any>) => Array<any>;
+  readonly getObject?: (arg?: Object) => Object;
+  readonly getObjectShape?: (arg?: {prop?: number}) => {prop?: number};
+  readonly getAlias?: (arg?: Animal) => Animal;
+  readonly getRootTag?: (arg?: RootTag) => RootTag;
+  readonly getValue?: (x?: number, y?: string, z?: Object) => Object;
+  readonly getValueWithCallback?: (callback?: (value?: string) => void) => void;
+  readonly getValueWithPromise?: (error?: boolean) => Promise<string>;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(
diff --git a/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeStringTurboModule.js b/packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeStringTurboModule.js
@@ -16,8 +16,8 @@ export type String = string;
 type AnotherString = String;
 
 export interface Spec extends TurboModule {
-  +getString: (arg: string) => string;
-  +getStringWithAlias: (arg: String) => AnotherString;
+  readonly getString: (arg: string) => string;
+  readonly getStringWithAlias: (arg: String) => AnotherString;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(

Original file line number	Diff line number	Diff line change
`@@ -16,8 +16,8 @@ export type Boolean = boolean;`
`16`	`16`	`type AnotherBoolean = Boolean;`
`17`	`17`
`18`	`18`	`export interface Spec extends TurboModule {`
`19`		`- +getBoolean: (arg: boolean) => boolean;`
`20`		`- +getBooleanWithAlias: (arg: Boolean) => AnotherBoolean;`
	`19`	`+ readonly getBoolean: (arg: boolean) => boolean;`
	`20`	`+ readonly getBooleanWithAlias: (arg: Boolean) => AnotherBoolean;`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`export default TurboModuleRegistry.getEnforcing<Spec>(`
Original file line number	Diff line number	Diff line change
`@@ -16,8 +16,8 @@ export type String = string;`
`16`	`16`	`type CB = (value: String) => void;`
`17`	`17`
`18`	`18`	`export interface Spec extends TurboModule {`
`19`		`- +getValueWithCallback: (callback: (value: string) => void) => void;`
`20`		`- +getValueWithCallbackWithAlias: (c: CB) => void;`
	`19`	`+ readonly getValueWithCallback: (callback: (value: string) => void) => void;`
	`20`	`+ readonly getValueWithCallbackWithAlias: (c: CB) => void;`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`export default TurboModuleRegistry.getEnforcing<Spec>(`
Original file line number	Diff line number	Diff line change
`@@ -16,8 +16,8 @@ export type Number = number;`
`16`	`16`	`type AnotherNumber = Number;`
`17`	`17`
`18`	`18`	`export interface Spec extends TurboModule {`
`19`		`- +getNumber: (arg: number) => number;`
`20`		`- +getNumberWithAlias: (arg: Number) => AnotherNumber;`
	`19`	`+ readonly getNumber: (arg: number) => number;`
	`20`	`+ readonly getNumberWithAlias: (arg: Number) => AnotherNumber;`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`export default TurboModuleRegistry.getEnforcing<Spec>(`