codemod objects to interfaces where they appear as supertypes of classes

Summary:
Flow is changing the behavior of object types to no longer be valid supertypes of classes. This replaces object types when they appear as supertypes of classes to be interfaces to avoid errors when this change rolls out.

Changelog: [Internal]

Reviewed By: pieterv

Differential Revision: D27193522

fbshipit-source-id: c3e3fca8a4cacd90770a95b773ff2c659774b9a6

packages/metro-cache/src/types.flow.js

@@ -10,9 +10,8 @@
 
 'use strict';
 
-export type CacheStore<T> = {
+export type CacheStore<T> = interface {
   get(key: Buffer): ?T | Promise<?T>,
   set(key: Buffer, value: T): void | Promise<void>,
   clear(): void | Promise<void>,
-  ...
 };

packages/metro/src/lib/TerminalReporter.js

@@ -45,11 +45,11 @@ export type TerminalReportableEvent =
 
 type BuildPhase = 'in_progress' | 'done' | 'failed';
 
-type SnippetError = ErrnoError & {
-  filename?: string,
-  snippet?: string,
-  ...
-};
+type SnippetError = ErrnoError &
+  interface {
+    filename?: string,
+    snippet?: string,
+  };
 
 const GLOBAL_CACHE_DISABLED_MESSAGE_FORMAT =
   'The global cache is now disabled because %s';

packages/metro/src/lib/attachWebsocketServer.js

@@ -13,7 +13,7 @@
 import type {Server as HttpServer} from 'http';
 import type {Server as HttpsServer} from 'https';
 
-type WebsocketServiceInterface<T> = {
+type WebsocketServiceInterface<T> = interface {
   +onClientConnect: (
     url: string,
     sendFn: (data: string) => void,
@@ -25,7 +25,6 @@ type WebsocketServiceInterface<T> = {
     message: string,
     sendFn: (data: string) => void,
   ) => mixed,
-  ...
 };
 
 type HMROptions<TClient> = {

packages/metro/src/lib/formatBundlingError.js

@@ -26,18 +26,18 @@ const {AmbiguousModuleResolutionError} = require('metro-core');
 
 import type {FormattedError} from 'metro-runtime/src/modules/types.flow';
 
-export type CustomError = Error & {
-  type?: string,
-  filename?: string,
-  lineNumber?: number,
-  errors?: Array<{
-    description: string,
-    filename: string,
-    lineNumber: number,
-    ...
-  }>,
-  ...
-};
+export type CustomError = Error &
+  interface {
+    type?: string,
+    filename?: string,
+    lineNumber?: number,
+    errors?: Array<{
+      description: string,
+      filename: string,
+      lineNumber: number,
+      ...
+    }>,
+  };
 
 function formatBundlingError(error: CustomError): FormattedError {
   if (error instanceof AmbiguousModuleResolutionError) {

packages/metro/src/lib/reporting.js

@@ -138,7 +138,7 @@ export type ReportableEvent =
  * TerminalReporter, that should be the only place in the application should
  * access the `terminal` module (nor the `console`).
  */
-export type Reporter = {update(event: ReportableEvent): void, ...};
+export type Reporter = interface {update(event: ReportableEvent): void};
 
 /**
  * A standard way to log a warning to the terminal. This should not be called

packages/metro/src/node-haste/DependencyGraph/ModuleResolution.js

@@ -30,20 +30,18 @@ import type {
 
 export type DirExistsFn = (filePath: string) => boolean;
 
-export type Packageish = {
+export type Packageish = interface {
   path: string,
   redirectRequire(
     toModuleName: string,
     mainFields: $ReadOnlyArray<string>,
   ): string | false,
   getMain(mainFields: $ReadOnlyArray<string>): string,
-  ...
 };
 
-export type Moduleish = {
+export type Moduleish = interface {
   +path: string,
   getPackage(): ?Packageish,
-  ...
 };
 
 /**
@@ -63,14 +61,13 @@ export type ModuleMap = {
   ...
 };
 
-export type ModuleishCache<TModule, TPackage> = {
+export type ModuleishCache<TModule, TPackage> = interface {
   getPackage(
     name: string,
     platform?: string,
     supportsNativePlatform?: boolean,
   ): TPackage,
   getModule(path: string): TModule,
-  ...
 };
 
 type Options<TModule, TPackage> = {|