metro 0.83.3 → 0.84.0

package/src/node-haste/DependencyGraph/ModuleResolution.js

@@ -13,28 +13,32 @@ var _createDefaultContext = _interopRequireDefault(
 );
 var _path = _interopRequireDefault(require("path"));
 var _util = _interopRequireDefault(require("util"));
-function _getRequireWildcardCache(e) {
-  if ("function" != typeof WeakMap) return null;
-  var r = new WeakMap(),
-    t = new WeakMap();
-  return (_getRequireWildcardCache = function (e) {
-    return e ? t : r;
-  })(e);
-}
-function _interopRequireWildcard(e, r) {
-  if (!r && e && e.__esModule) return e;
-  if (null === e || ("object" != typeof e && "function" != typeof e))
-    return { default: e };
-  var t = _getRequireWildcardCache(r);
-  if (t && t.has(e)) return t.get(e);
-  var n = { __proto__: null },
-    a = Object.defineProperty && Object.getOwnPropertyDescriptor;
-  for (var u in e)
-    if ("default" !== u && {}.hasOwnProperty.call(e, u)) {
-      var i = a ? Object.getOwnPropertyDescriptor(e, u) : null;
-      i && (i.get || i.set) ? Object.defineProperty(n, u, i) : (n[u] = e[u]);
+function _interopRequireWildcard(e, t) {
+  if ("function" == typeof WeakMap)
+    var r = new WeakMap(),
+      n = new WeakMap();
+  return (_interopRequireWildcard = function (e, t) {
+    if (!t && e && e.__esModule) return e;
+    var o,
+      i,
+      f = { __proto__: null, default: e };
+    if (null === e || ("object" != typeof e && "function" != typeof e))
+      return f;
+    if ((o = t ? n : r)) {
+      if (o.has(e)) return o.get(e);
+      o.set(e, f);
     }
-  return ((n.default = e), t && t.set(e, n), n);
+    for (const t in e)
+      "default" !== t &&
+        {}.hasOwnProperty.call(e, t) &&
+        ((i =
+          (o = Object.defineProperty) &&
+          Object.getOwnPropertyDescriptor(e, t)) &&
+        (i.get || i.set)
+          ? o(f, t, i)
+          : (f[t] = e[t]));
+    return f;
+  })(e, t);
 }
 function _interopRequireDefault(e) {
   return e && e.__esModule ? e : { default: e };
@@ -51,13 +55,13 @@ class ModuleResolver {
       emptyModule = this.resolveDependency(
         this._projectRootFakeModulePath,
         {
-          name: this._options.emptyModulePath,
           data: {
-            key: this._options.emptyModulePath,
             asyncType: null,
             isESMImport: false,
+            key: this._options.emptyModulePath,
             locs: [],
           },
+          name: this._options.emptyModulePath,
         },
         false,
         null,
@@ -91,6 +95,7 @@ class ModuleResolver {
       unstable_conditionNames,
       unstable_conditionsByPlatform,
       unstable_enablePackageExports,
+      unstable_incrementalResolution,
     } = this._options;
     try {
       const result = Resolver.resolve(
@@ -98,31 +103,32 @@ class ModuleResolver {
           {
             allowHaste,
             assetExts,
+            customResolverOptions: resolverOptions.customResolverOptions ?? {},
             dev: resolverOptions.dev,
             disableHierarchicalLookup,
             doesFileExist,
             extraNodeModules,
             fileSystemLookup,
+            getPackage: this._getPackage,
+            getPackageForModule: (absoluteModulePath) =>
+              this._getPackageForModule(absoluteModulePath),
             isESMImport: dependency.data.isESMImport,
             mainFields,
             nodeModulesPaths,
+            originModulePath,
             preferNativePlatform,
             resolveAsset,
+            resolveHasteModule: (name) =>
+              this._options.getHasteModulePath(name, platform),
+            resolveHastePackage: (name) =>
+              this._options.getHastePackagePath(name, platform),
             resolveRequest,
             sourceExts,
             unstable_conditionNames,
             unstable_conditionsByPlatform,
             unstable_enablePackageExports,
+            unstable_incrementalResolution,
             unstable_logWarning: this._logWarning,
-            customResolverOptions: resolverOptions.customResolverOptions ?? {},
-            originModulePath,
-            resolveHasteModule: (name) =>
-              this._options.getHasteModulePath(name, platform),
-            resolveHastePackage: (name) =>
-              this._options.getHastePackagePath(name, platform),
-            getPackage: this._getPackage,
-            getPackageForModule: (absoluteModulePath) =>
-              this._getPackageForModule(absoluteModulePath),
           },
           dependency,
         ),
@@ -198,9 +204,9 @@ class ModuleResolver {
     } catch (e) {}
     return result != null
       ? {
-          rootPath: _path.default.dirname(result.pkg.path),
           packageJson: result.pkg.read(),
           packageRelativePath: result.packageRelativePath,
+          rootPath: _path.default.dirname(result.pkg.path),
         }
       : null;
   };
@@ -212,8 +218,8 @@ class ModuleResolver {
         const arbitrary = getArrayLowestItem(resolution.filePaths);
         (0, _invariant.default)(arbitrary != null, "invalid asset resolution");
         return {
-          type: "sourceFile",
           filePath: arbitrary,
+          type: "sourceFile",
         };
       case "empty":
         return this._getEmptyModule();
@@ -224,8 +230,8 @@ class ModuleResolver {
   }
   _logWarning = (message) => {
     this._options.reporter.update({
-      type: "resolver_warning",
       message,
+      type: "resolver_warning",
     });
   };
   _removeRoot(candidates) {
@@ -313,8 +319,8 @@ function refineDependencyLocation(loc, fileContents, targetSpecifier) {
       if (isQuote(maybeQuoteBefore) && maybeQuoteBefore === maybeQuoteAfter) {
         return {
           start: {
-            line: line + 1,
             column: minColumn + offset + 1,
+            line: line + 1,
           },
         };
       }
@@ -322,20 +328,20 @@ function refineDependencyLocation(loc, fileContents, targetSpecifier) {
   }
   if (loc.start.line === loc.end.line) {
     return {
-      start: {
-        line: loc.start.line,
-        column: loc.start.column + 1,
-      },
       end: {
-        line: loc.end.line,
         column: loc.end.column + 1,
+        line: loc.end.line,
+      },
+      start: {
+        column: loc.start.column + 1,
+        line: loc.start.line,
       },
     };
   }
   return {
     start: {
-      line: loc.start.line,
       column: loc.start.column + 1,
+      line: loc.start.line,
     },
   };
 }

package/src/node-haste/DependencyGraph/ModuleResolution.js.flow

@@ -56,8 +56,8 @@ export type PackageishCache<TPackage> = interface {
   },
 };
 
-type Options<TPackage> = $ReadOnly<{
-  assetExts: $ReadOnlySet<string>,
+type Options<TPackage> = Readonly<{
+  assetExts: ReadonlySet<string>,
   dirExists: DirExistsFn,
   disableHierarchicalLookup: boolean,
   doesFileExist: DoesFileExist,
@@ -66,20 +66,21 @@ type Options<TPackage> = $ReadOnly<{
   fileSystemLookup: FileSystemLookup,
   getHasteModulePath: (name: string, platform: ?string) => ?string,
   getHastePackagePath: (name: string, platform: ?string) => ?string,
-  mainFields: $ReadOnlyArray<string>,
+  mainFields: ReadonlyArray<string>,
   packageCache: PackageishCache<TPackage>,
-  nodeModulesPaths: $ReadOnlyArray<string>,
+  nodeModulesPaths: ReadonlyArray<string>,
   preferNativePlatform: boolean,
   projectRoot: string,
   reporter: Reporter,
   resolveAsset: ResolveAsset,
   resolveRequest: ?CustomResolver,
-  sourceExts: $ReadOnlyArray<string>,
-  unstable_conditionNames: $ReadOnlyArray<string>,
-  unstable_conditionsByPlatform: $ReadOnly<{
-    [platform: string]: $ReadOnlyArray<string>,
+  sourceExts: ReadonlyArray<string>,
+  unstable_conditionNames: ReadonlyArray<string>,
+  unstable_conditionsByPlatform: Readonly<{
+    [platform: string]: ReadonlyArray<string>,
   }>,
   unstable_enablePackageExports: boolean,
+  unstable_incrementalResolution: boolean,
 }>;
 
 export class ModuleResolver<TPackage: Packageish> {
@@ -101,13 +102,13 @@ export class ModuleResolver<TPackage: Packageish> {
       emptyModule = this.resolveDependency(
         this._projectRootFakeModulePath,
         {
-          name: this._options.emptyModulePath,
           data: {
-            key: this._options.emptyModulePath,
             asyncType: null,
             isESMImport: false,
+            key: this._options.emptyModulePath,
             locs: [],
           },
+          name: this._options.emptyModulePath,
         },
         false,
         null,
@@ -140,6 +141,7 @@ export class ModuleResolver<TPackage: Packageish> {
       unstable_conditionNames,
       unstable_conditionsByPlatform,
       unstable_enablePackageExports,
+      unstable_incrementalResolution,
     } = this._options;
 
     try {
@@ -148,31 +150,32 @@ export class ModuleResolver<TPackage: Packageish> {
           {
             allowHaste,
             assetExts,
+            customResolverOptions: resolverOptions.customResolverOptions ?? {},
             dev: resolverOptions.dev,
             disableHierarchicalLookup,
             doesFileExist,
             extraNodeModules,
             fileSystemLookup,
+            getPackage: this._getPackage,
+            getPackageForModule: (absoluteModulePath: string) =>
+              this._getPackageForModule(absoluteModulePath),
             isESMImport: dependency.data.isESMImport,
             mainFields,
             nodeModulesPaths,
+            originModulePath,
             preferNativePlatform,
             resolveAsset,
+            resolveHasteModule: (name: string) =>
+              this._options.getHasteModulePath(name, platform),
+            resolveHastePackage: (name: string) =>
+              this._options.getHastePackagePath(name, platform),
             resolveRequest,
             sourceExts,
             unstable_conditionNames,
             unstable_conditionsByPlatform,
             unstable_enablePackageExports,
+            unstable_incrementalResolution,
             unstable_logWarning: this._logWarning,
-            customResolverOptions: resolverOptions.customResolverOptions ?? {},
-            originModulePath,
-            resolveHasteModule: (name: string) =>
-              this._options.getHasteModulePath(name, platform),
-            resolveHastePackage: (name: string) =>
-              this._options.getHastePackagePath(name, platform),
-            getPackage: this._getPackage,
-            getPackageForModule: (absoluteModulePath: string) =>
-              this._getPackageForModule(absoluteModulePath),
           },
           dependency,
         ),
@@ -260,9 +263,9 @@ export class ModuleResolver<TPackage: Packageish> {
 
     return result != null
       ? {
-          rootPath: path.dirname(result.pkg.path),
           packageJson: result.pkg.read(),
           packageRelativePath: result.packageRelativePath,
+          rootPath: path.dirname(result.pkg.path),
         }
       : null;
   };
@@ -279,19 +282,19 @@ export class ModuleResolver<TPackage: Packageish> {
         // not just an arbitrary item!
         const arbitrary = getArrayLowestItem(resolution.filePaths);
         invariant(arbitrary != null, 'invalid asset resolution');
-        return {type: 'sourceFile', filePath: arbitrary};
+        return {filePath: arbitrary, type: 'sourceFile'};
       case 'empty':
         return this._getEmptyModule();
       default:
-        (resolution.type: empty);
+        resolution.type as empty;
         throw new Error('invalid type');
     }
   }
 
   _logWarning = (message: string): void => {
     this._options.reporter.update({
-      type: 'resolver_warning',
       message,
+      type: 'resolver_warning',
     });
   };
 
@@ -306,7 +309,7 @@ export class ModuleResolver<TPackage: Packageish> {
   }
 }
 
-function getArrayLowestItem(a: $ReadOnlyArray<string>): string | void {
+function getArrayLowestItem(a: ReadonlyArray<string>): string | void {
   if (a.length === 0) {
     return undefined;
   }
@@ -343,7 +346,7 @@ export class UnableToResolveError extends Error {
     originModulePath: string,
     targetModuleName: string,
     message: string,
-    options?: $ReadOnly<{
+    options?: Readonly<{
       dependency?: ?TransformResultDependency,
       cause?: Error,
     }>,
@@ -429,8 +432,8 @@ function refineDependencyLocation(
       if (isQuote(maybeQuoteBefore) && maybeQuoteBefore === maybeQuoteAfter) {
         return {
           start: {
-            line: line + 1,
             column: minColumn + offset + 1,
+            line: line + 1,
           },
         };
       }
@@ -439,13 +442,13 @@ function refineDependencyLocation(
   // Otherwise, if this is a single-line loc, return it exactly, as a range.
   if (loc.start.line === loc.end.line) {
     return {
-      start: {
-        line: loc.start.line,
-        column: loc.start.column + 1,
-      },
       end: {
-        line: loc.end.line,
         column: loc.end.column + 1,
+        line: loc.end.line,
+      },
+      start: {
+        column: loc.start.column + 1,
+        line: loc.start.line,
       },
     };
   }
@@ -453,8 +456,8 @@ function refineDependencyLocation(
   // much unnecessary context.
   return {
     start: {
-      line: loc.start.line,
       column: loc.start.column + 1,
+      line: loc.start.line,
     },
   };
 }

package/src/node-haste/DependencyGraph/createFileMap.js

@@ -6,28 +6,32 @@ Object.defineProperty(exports, "__esModule", {
 exports.default = createFileMap;
 var _ciInfo = _interopRequireDefault(require("ci-info"));
 var _metroFileMap = _interopRequireWildcard(require("metro-file-map"));
-function _getRequireWildcardCache(e) {
-  if ("function" != typeof WeakMap) return null;
-  var r = new WeakMap(),
-    t = new WeakMap();
-  return (_getRequireWildcardCache = function (e) {
-    return e ? t : r;
-  })(e);
-}
-function _interopRequireWildcard(e, r) {
-  if (!r && e && e.__esModule) return e;
-  if (null === e || ("object" != typeof e && "function" != typeof e))
-    return { default: e };
-  var t = _getRequireWildcardCache(r);
-  if (t && t.has(e)) return t.get(e);
-  var n = { __proto__: null },
-    a = Object.defineProperty && Object.getOwnPropertyDescriptor;
-  for (var u in e)
-    if ("default" !== u && {}.hasOwnProperty.call(e, u)) {
-      var i = a ? Object.getOwnPropertyDescriptor(e, u) : null;
-      i && (i.get || i.set) ? Object.defineProperty(n, u, i) : (n[u] = e[u]);
+function _interopRequireWildcard(e, t) {
+  if ("function" == typeof WeakMap)
+    var r = new WeakMap(),
+      n = new WeakMap();
+  return (_interopRequireWildcard = function (e, t) {
+    if (!t && e && e.__esModule) return e;
+    var o,
+      i,
+      f = { __proto__: null, default: e };
+    if (null === e || ("object" != typeof e && "function" != typeof e))
+      return f;
+    if ((o = t ? n : r)) {
+      if (o.has(e)) return o.get(e);
+      o.set(e, f);
     }
-  return ((n.default = e), t && t.set(e, n), n);
+    for (const t in e)
+      "default" !== t &&
+        {}.hasOwnProperty.call(e, t) &&
+        ((i =
+          (o = Object.defineProperty) &&
+          Object.getOwnPropertyDescriptor(e, t)) &&
+        (i.get || i.set)
+          ? o(f, t, i)
+          : (f[t] = e[t]));
+    return f;
+  })(e, t);
 }
 function _interopRequireDefault(e) {
   return e && e.__esModule ? e : { default: e };
@@ -63,16 +67,35 @@ function getIgnorePattern(config) {
   return ignorePattern;
 }
 function createFileMap(config, options) {
-  const dependencyExtractor =
-    options?.extractDependencies === false
-      ? null
-      : config.resolver.dependencyExtractor;
-  const computeDependencies = dependencyExtractor != null;
   const watch = options?.watch == null ? !_ciInfo.default.isCI : options.watch;
   const { enabled: autoSaveEnabled, ...autoSaveOpts } =
     config.watcher.unstable_autoSaveCache ?? {};
   const autoSave = watch && autoSaveEnabled ? autoSaveOpts : false;
-  return _metroFileMap.default.create({
+  const plugins = [];
+  let dependencyPlugin = null;
+  if (
+    config.resolver.dependencyExtractor != null &&
+    options?.extractDependencies !== false
+  ) {
+    dependencyPlugin = new _metroFileMap.DependencyPlugin({
+      dependencyExtractor: config.resolver.dependencyExtractor,
+      computeDependencies: true,
+      rootDir: config.projectRoot,
+    });
+    plugins.push(dependencyPlugin);
+  }
+  const hasteMap = new _metroFileMap.HastePlugin({
+    platforms: new Set([
+      ...config.resolver.platforms,
+      _metroFileMap.default.H.NATIVE_PLATFORM,
+    ]),
+    hasteImplModulePath: config.resolver.hasteImplModulePath,
+    enableHastePackages: config.resolver.enableGlobalPackages,
+    rootDir: config.projectRoot,
+    failValidationOnConflicts: options?.throwOnModuleCollision ?? true,
+  });
+  plugins.push(hasteMap);
+  const fileMap = new _metroFileMap.default({
     cacheManagerFactory:
       config?.unstable_fileMapCacheManagerFactory ??
       ((factoryParams) =>
@@ -83,12 +106,8 @@ function createFileMap(config, options) {
           autoSave,
         })),
     perfLoggerFactory: config.unstable_perfLoggerFactory,
-    computeDependencies,
     computeSha1: !config.watcher.unstable_lazySha1,
-    dependencyExtractor: config.resolver.dependencyExtractor,
-    enableHastePackages: config?.resolver.enableGlobalPackages,
     enableSymlinks: true,
-    enableWorkerThreads: config.watcher.unstable_workerThreads,
     extensions: Array.from(
       new Set([
         ...config.resolver.sourceExts,
@@ -97,22 +116,21 @@ function createFileMap(config, options) {
       ]),
     ),
     forceNodeFilesystemAPI: !config.resolver.useWatchman,
-    hasteImplModulePath: config.resolver.hasteImplModulePath,
     healthCheck: config.watcher.healthCheck,
     ignorePattern: getIgnorePattern(config),
     maxWorkers: config.maxWorkers,
-    mocksPattern: "",
-    platforms: [
-      ...config.resolver.platforms,
-      _metroFileMap.default.H.NATIVE_PLATFORM,
-    ],
+    plugins,
     retainAllFiles: true,
     resetCache: config.resetCache,
     rootDir: config.projectRoot,
     roots: config.watchFolders,
-    throwOnModuleCollision: options?.throwOnModuleCollision ?? true,
     useWatchman: config.resolver.useWatchman,
     watch,
     watchmanDeferStates: config.watcher.watchman.deferStates,
   });
+  return {
+    fileMap,
+    hasteMap,
+    dependencyPlugin,
+  };
 }

package/src/node-haste/DependencyGraph/createFileMap.js.flow

@@ -10,9 +10,14 @@
  */
 
 import type {ConfigT} from 'metro-config';
+import type {HasteMap} from 'metro-file-map';
 
 import ci from 'ci-info';
-import MetroFileMap, {DiskCacheManager} from 'metro-file-map';
+import MetroFileMap, {
+  DependencyPlugin,
+  DiskCacheManager,
+  HastePlugin,
+} from 'metro-file-map';
 
 function getIgnorePattern(config: ConfigT): RegExp {
   // For now we support both options
@@ -54,25 +59,52 @@ function getIgnorePattern(config: ConfigT): RegExp {
 
 export default function createFileMap(
   config: ConfigT,
-  options?: $ReadOnly<{
+  options?: Readonly<{
     extractDependencies?: boolean,
     watch?: boolean,
     throwOnModuleCollision?: boolean,
     cacheFilePrefix?: string,
   }>,
-): MetroFileMap {
-  const dependencyExtractor =
-    options?.extractDependencies === false
-      ? null
-      : config.resolver.dependencyExtractor;
-  const computeDependencies = dependencyExtractor != null;
-
+): {
+  fileMap: MetroFileMap,
+  hasteMap: HasteMap,
+  dependencyPlugin: ?DependencyPlugin,
+} {
   const watch = options?.watch == null ? !ci.isCI : options.watch;
   const {enabled: autoSaveEnabled, ...autoSaveOpts} =
     config.watcher.unstable_autoSaveCache ?? {};
   const autoSave = watch && autoSaveEnabled ? autoSaveOpts : false;
 
-  return MetroFileMap.create({
+  const plugins: Array<DependencyPlugin | HastePlugin> = [];
+
+  let dependencyPlugin = null;
+  // Add DependencyPlugin if dependencies should be extracted
+  if (
+    config.resolver.dependencyExtractor != null &&
+    options?.extractDependencies !== false
+  ) {
+    dependencyPlugin = new DependencyPlugin({
+      dependencyExtractor: config.resolver.dependencyExtractor,
+      computeDependencies: true,
+      rootDir: config.projectRoot,
+    });
+    plugins.push(dependencyPlugin);
+  }
+
+  const hasteMap = new HastePlugin({
+    platforms: new Set([
+      ...config.resolver.platforms,
+      MetroFileMap.H.NATIVE_PLATFORM,
+    ]),
+    hasteImplModulePath: config.resolver.hasteImplModulePath,
+    enableHastePackages: config.resolver.enableGlobalPackages,
+    rootDir: config.projectRoot,
+    failValidationOnConflicts: options?.throwOnModuleCollision ?? true,
+  });
+
+  plugins.push(hasteMap);
+
+  const fileMap = new MetroFileMap({
     cacheManagerFactory:
       config?.unstable_fileMapCacheManagerFactory ??
       (factoryParams =>
@@ -83,12 +115,8 @@ export default function createFileMap(
           autoSave,
         })),
     perfLoggerFactory: config.unstable_perfLoggerFactory,
-    computeDependencies,
     computeSha1: !config.watcher.unstable_lazySha1,
-    dependencyExtractor: config.resolver.dependencyExtractor,
-    enableHastePackages: config?.resolver.enableGlobalPackages,
     enableSymlinks: true,
-    enableWorkerThreads: config.watcher.unstable_workerThreads,
     extensions: Array.from(
       new Set([
         ...config.resolver.sourceExts,
@@ -97,19 +125,17 @@ export default function createFileMap(
       ]),
     ),
     forceNodeFilesystemAPI: !config.resolver.useWatchman,
-    hasteImplModulePath: config.resolver.hasteImplModulePath,
     healthCheck: config.watcher.healthCheck,
     ignorePattern: getIgnorePattern(config),
     maxWorkers: config.maxWorkers,
-    mocksPattern: '',
-    platforms: [...config.resolver.platforms, MetroFileMap.H.NATIVE_PLATFORM],
+    plugins,
     retainAllFiles: true,
     resetCache: config.resetCache,
     rootDir: config.projectRoot,
     roots: config.watchFolders,
-    throwOnModuleCollision: options?.throwOnModuleCollision ?? true,
     useWatchman: config.resolver.useWatchman,
     watch,
     watchmanDeferStates: config.watcher.watchman.deferStates,
   });
+  return {fileMap, hasteMap, dependencyPlugin};
 }