Revert recent Flow syntax codemods in react-native-github (facebook#56998)

Summary:
Pull Request resolved: facebook#56998

Reverts the recent Flow syntax codemods applied across `packages/`, `private/`, and `scripts/` in react-native-github. Specifically restores the previous form for:

- `+T` / `+Instance` style variance annotations on generic type parameters that had been converted to the newer `out T` / `in T` keyword form.
- `+field:` covariant object/interface properties that had been converted to the `readonly field:` modifier form.
- `+[K in keyof T]:` mapped-type covariance that had been converted to `readonly [K in keyof T]:`.

These are purely Flow type-annotation changes with no runtime behavior impact, restoring the form that the rest of the toolchain (in particular Fantom) already supports.

Changelog: [Internal]

Reviewed By: christophpurrer

Differential Revision: D106813102

Author: rubennorte

packages/assets-registry/registry.js

@@ -14,16 +14,16 @@
 export type AssetDestPathResolver = 'android' | 'generic';
 
 export type PackagerAsset = {
-  readonly __packager_asset: boolean,
-  readonly fileSystemLocation: string,
-  readonly httpServerLocation: string,
-  readonly width: ?number,
-  readonly height: ?number,
-  readonly scales: Array<number>,
-  readonly hash: string,
-  readonly name: string,
-  readonly type: string,
-  readonly resolver?: AssetDestPathResolver,
+  +__packager_asset: boolean,
+  +fileSystemLocation: string,
+  +httpServerLocation: string,
+  +width: ?number,
+  +height: ?number,
+  +scales: Array<number>,
+  +hash: string,
+  +name: string,
+  +type: string,
+  +resolver?: AssetDestPathResolver,
   ...
 };
 */

packages/dev-middleware/src/__tests__/InspectorDebuggerUtils.js

@@ -84,7 +84,7 @@ export class DebuggerAgent {
 
 export class DebuggerMock extends DebuggerAgent {
   // Empty handlers
-  readonly handle: JestMockFn<[message: JSONSerializable], void> = jest.fn();
+  +handle: JestMockFn<[message: JSONSerializable], void> = jest.fn();
 
   __handle(message: JSONSerializable): void {
     this.handle(message);

packages/dev-middleware/src/__tests__/InspectorDeviceUtils.js

@@ -98,18 +98,16 @@ export class DeviceAgent {
 
 export class DeviceMock extends DeviceAgent {
   // Empty handlers
-  readonly connect: JestMockFn<[message: ConnectRequest], void> = jest.fn();
-  readonly disconnect: JestMockFn<[message: DisconnectRequest], void> =
-    jest.fn();
-  readonly getPages: JestMockFn<
+  +connect: JestMockFn<[message: ConnectRequest], void> = jest.fn();
+  +disconnect: JestMockFn<[message: DisconnectRequest], void> = jest.fn();
+  +getPages: JestMockFn<
     [message: GetPagesRequest],
     | GetPagesResponse['payload']
     | Promise<GetPagesResponse['payload'] | void>
     | void,
   > = jest.fn();
-  readonly wrappedEvent: JestMockFn<[message: WrappedEventToDevice], void> =
-    jest.fn();
-  readonly wrappedEventParsed: JestMockFn<
+  +wrappedEvent: JestMockFn<[message: WrappedEventToDevice], void> = jest.fn();
+  +wrappedEventParsed: JestMockFn<
     [
       payload: {
         ...WrappedEventToDevice['payload'],

packages/dev-middleware/src/__tests__/StandaloneFuseboxShell-test.js

@@ -25,7 +25,7 @@ const PAGES_POLLING_DELAY = 2100;
 jest.useFakeTimers();
 
 async function setupDevice(
-  serverRef: {readonly serverBaseWsUrl: string, ...},
+  serverRef: {+serverBaseWsUrl: string, ...},
   signal: AbortSignal,
 ) {
   const device = await createDeviceMock(

packages/dev-middleware/src/inspector-proxy/Device.js

@@ -138,7 +138,7 @@ export default class Device {
   // Logging reporting batches of cdp messages
   #cdpDebugLogging: CdpDebugLogging;
 
-  readonly #experiments: Experiments;
+  +#experiments: Experiments;
 
   constructor(deviceOptions: DeviceOptions) {
     this.#experiments = deviceOptions.experiments;

packages/dev-middleware/src/inspector-proxy/InspectorProxy.js

@@ -93,7 +93,7 @@ export default class InspectorProxy implements InspectorProxyQueries {
 
   #eventReporter: ?EventReporter;
 
-  readonly #experiments: Experiments;
+  +#experiments: Experiments;
 
   // custom message handler factory allowing implementers to handle unsupported CDP messages.
   #customMessageHandler: ?CreateCustomMessageHandlerFn;

packages/dev-middleware/src/inspector-proxy/types.js

@@ -156,11 +156,11 @@ export type JSONSerializable =
   | string
   | null
   | ReadonlyArray<JSONSerializable>
-  | {readonly [string]: JSONSerializable};
+  | {+[string]: JSONSerializable};
 
 export type DeepReadOnly<T> =
   T extends ReadonlyArray<infer V>
     ? ReadonlyArray<DeepReadOnly<V>>
     : T extends {...}
-      ? {readonly [K in keyof T]: DeepReadOnly<T[K]>}
+      ? {+[K in keyof T]: DeepReadOnly<T[K]>}
       : T;

packages/dev-middleware/src/types/DevToolLauncher.js

@@ -27,7 +27,7 @@ export interface DevToolLauncher {
    * the host of dev-middleware. Implementations are responsible for rewriting
    * this as necessary where the server is remote.
    */
-  readonly launchDebuggerAppWindow: (url: string) => Promise<void>;
+  +launchDebuggerAppWindow: (url: string) => Promise<void>;
 
   /**
    * Attempt to open a debugger frontend URL in a standalone shell window
@@ -47,10 +47,7 @@ export interface DevToolLauncher {
    * the host of dev-middleware. Implementations are responsible for rewriting
    * this as necessary where the server is remote.
    */
-  readonly launchDebuggerShell?: (
-    url: string,
-    windowKey: string,
-  ) => Promise<void>;
+  +launchDebuggerShell?: (url: string, windowKey: string) => Promise<void>;
 
   /**
    * Attempt to prepare the debugger shell for use and returns a coded result
@@ -63,5 +60,5 @@ export interface DevToolLauncher {
    * SHOULD NOT return a rejecting promise in any case, and instead SHOULD report
    * errors via the returned result object.
    */
-  readonly prepareDebuggerShell?: () => Promise<DebuggerShellPreparationResult>;
+  +prepareDebuggerShell?: () => Promise<DebuggerShellPreparationResult>;
 }

packages/dev-middleware/src/types/ReadonlyURL.js

@@ -15,7 +15,7 @@ export interface ReadonlyURLSearchParams {
   get(name: string): string | null;
   getAll(name: string): Array<string>;
   has(name: string, value?: string): boolean;
-  readonly size: number;
+  +size: number;
   entries(): Iterator<[string, string]>;
   keys(): Iterator<string>;
   values(): Iterator<string>;
@@ -37,18 +37,18 @@ export interface ReadonlyURLSearchParams {
  * Used for URLs passed between module boundaries.
  */
 export interface ReadonlyURL {
-  readonly hash: string;
-  readonly host: string;
-  readonly hostname: string;
-  readonly href: string;
-  readonly origin: string;
-  readonly password: string;
-  readonly pathname: string;
-  readonly port: string;
-  readonly protocol: string;
-  readonly search: string;
-  readonly searchParams: ReadonlyURLSearchParams;
-  readonly username: string;
+  +hash: string;
+  +host: string;
+  +hostname: string;
+  +href: string;
+  +origin: string;
+  +password: string;
+  +pathname: string;
+  +port: string;
+  +protocol: string;
+  +search: string;
+  +searchParams: ReadonlyURLSearchParams;
+  +username: string;
   toString(): string;
   toJSON(): string;
 }

packages/react-native-codegen/e2e/deep_imports/__test_fixtures__/modules/NativeArrayTurboModule.js

@@ -16,12 +16,9 @@ export type ArrayType = string;
 type AnotherArray = Array<ArrayType>;
 
 export interface Spec extends TurboModule {
-  readonly getArray: (a: Array<any>) => Array<string>;
-  readonly getReadOnlyArray: (a: Array<any>) => ReadonlyArray<string>;
-  readonly getArrayWithAlias: (
-    a: AnotherArray,
-    b: Array<ArrayType>,
-  ) => AnotherArray;
+  +getArray: (a: Array<any>) => Array<string>;
+  +getReadOnlyArray: (a: Array<any>) => ReadonlyArray<string>;
+  +getArrayWithAlias: (a: AnotherArray, b: Array<ArrayType>) => AnotherArray;
 }
 
 export default TurboModuleRegistry.getEnforcing<Spec>(