metro 0.71.0 → 0.71.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "metro",
-  "version": "0.71.0",
+  "version": "0.71.3",
   "description": "🚇 The JavaScript bundler for React Native.",
   "main": "src/index.js",
   "bin": "src/cli.js",
@@ -31,27 +31,27 @@
     "error-stack-parser": "^2.0.6",
     "fs-extra": "^1.0.0",
     "graceful-fs": "^4.2.4",
-    "hermes-parser": "0.6.0",
+    "hermes-parser": "0.8.0",
     "image-size": "^0.6.0",
     "invariant": "^2.2.4",
     "jest-worker": "^27.2.0",
     "lodash.throttle": "^4.1.1",
-    "metro-babel-transformer": "0.71.0",
-    "metro-cache": "0.71.0",
-    "metro-cache-key": "0.71.0",
-    "metro-config": "0.71.0",
-    "metro-core": "0.71.0",
-    "metro-file-map": "0.71.0",
-    "metro-hermes-compiler": "0.71.0",
-    "metro-inspector-proxy": "0.71.0",
-    "metro-minify-uglify": "0.71.0",
-    "metro-react-native-babel-preset": "0.71.0",
-    "metro-resolver": "0.71.0",
-    "metro-runtime": "0.71.0",
-    "metro-source-map": "0.71.0",
-    "metro-symbolicate": "0.71.0",
-    "metro-transform-plugins": "0.71.0",
-    "metro-transform-worker": "0.71.0",
+    "metro-babel-transformer": "0.71.3",
+    "metro-cache": "0.71.3",
+    "metro-cache-key": "0.71.3",
+    "metro-config": "0.71.3",
+    "metro-core": "0.71.3",
+    "metro-file-map": "0.71.3",
+    "metro-hermes-compiler": "0.71.3",
+    "metro-inspector-proxy": "0.71.3",
+    "metro-minify-uglify": "0.71.3",
+    "metro-react-native-babel-preset": "0.71.3",
+    "metro-resolver": "0.71.3",
+    "metro-runtime": "0.71.3",
+    "metro-source-map": "0.71.3",
+    "metro-symbolicate": "0.71.3",
+    "metro-transform-plugins": "0.71.3",
+    "metro-transform-worker": "0.71.3",
     "mime-types": "^2.1.27",
     "node-fetch": "^2.2.0",
     "nullthrows": "^1.1.1",
@@ -66,13 +66,14 @@
   },
   "devDependencies": {
     "@babel/plugin-transform-flow-strip-types": "^7.0.0",
-    "acorn": "^5.1.2",
+    "acorn": "^8.7.1",
     "babel-jest": "^26.6.3",
     "dedent": "^0.7.0",
     "jest-snapshot": "^26.5.2",
-    "metro-memory-fs": "0.71.0",
-    "metro-react-native-babel-preset": "0.71.0",
-    "metro-react-native-babel-transformer": "0.71.0",
+    "metro-babel-register": "0.71.3",
+    "metro-memory-fs": "0.71.3",
+    "metro-react-native-babel-preset": "0.71.3",
+    "metro-react-native-babel-transformer": "0.71.3",
     "stack-trace": "^0.0.10"
   },
   "license": "MIT"

package/src/Bundler.js

@@ -13,8 +13,6 @@ const Transformer = require("./DeltaBundler/Transformer");
 
 const DependencyGraph = require("./node-haste/DependencyGraph");
 
-const { EventEmitter } = require("events");
-
 class Bundler {
   constructor(config, options) {
     this._depGraph = new DependencyGraph(config, options);

package/src/Bundler.js.flow

@@ -13,10 +13,10 @@
 import type {TransformResultWithSource} from './DeltaBundler';
 import type {TransformOptions} from './DeltaBundler/Worker';
 import type {ConfigT} from 'metro-config/src/configTypes.flow';
+import type EventEmitter from 'events';
 
 const Transformer = require('./DeltaBundler/Transformer');
 const DependencyGraph = require('./node-haste/DependencyGraph');
-const {EventEmitter} = require('events');
 
 export type BundlerOptions = $ReadOnly<{
   hasReducedPerformance?: boolean,

package/src/DeltaBundler/DeltaCalculator.js

@@ -130,6 +130,8 @@ class DeltaCalculator extends EventEmitter {
   getGraph() {
     return this._graph;
   }
+  /* $FlowFixMe[missing-local-annot] The type annotation(s) required by Flow's
+   * LTI update could not be added via codemod */
 
   _handleMultipleFileChanges = ({ eventsQueue }) => {
     eventsQueue.forEach(this._handleFileChange);

package/src/DeltaBundler/DeltaCalculator.js.flow

@@ -158,6 +158,8 @@ class DeltaCalculator<T> extends EventEmitter {
     return this._graph;
   }
 
+  /* $FlowFixMe[missing-local-annot] The type annotation(s) required by Flow's
+   * LTI update could not be added via codemod */
   _handleMultipleFileChanges = ({eventsQueue}) => {
     eventsQueue.forEach(this._handleFileChange);
   };

package/src/DeltaBundler/Worker.flow.js

@@ -0,0 +1,78 @@
+/**
+ * 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.
+ *
+ *
+ * @format
+ */
+"use strict";
+
+const traverse = require("@babel/traverse").default;
+
+const crypto = require("crypto");
+
+const fs = require("fs");
+
+const path = require("path");
+
+async function transform(
+  filename,
+  transformOptions,
+  projectRoot,
+  transformerConfig
+) {
+  // eslint-disable-next-line no-useless-call
+  const Transformer = require.call(null, transformerConfig.transformerPath);
+
+  const transformFileStartLogEntry = {
+    action_name: "Transforming file",
+    action_phase: "start",
+    file_name: filename,
+    log_entry_label: "Transforming file",
+    start_timestamp: process.hrtime(),
+  };
+  const data = fs.readFileSync(path.resolve(projectRoot, filename));
+  const sha1 = crypto.createHash("sha1").update(data).digest("hex");
+  const result = await Transformer.transform(
+    transformerConfig.transformerConfig,
+    projectRoot,
+    filename,
+    data,
+    transformOptions
+  ); // The babel cache caches scopes and pathes for already traversed AST nodes.
+  // Clearing the cache here since the nodes of the transformed file are no longer referenced.
+  // This isn't stritcly necessary since the cache uses a WeakMap. However, WeakMap only permit
+  // that unreferenced keys are collected but the values still hold references to the Scope and NodePaths.
+  // Manually clearing the cache allows the GC to collect the Scope and NodePaths without checking if there
+  // exist any other references to the keys.
+
+  traverse.cache.clear();
+  const transformFileEndLogEntry = getEndLogEntry(
+    transformFileStartLogEntry,
+    filename
+  );
+  return {
+    result,
+    sha1,
+    transformFileStartLogEntry,
+    transformFileEndLogEntry,
+  };
+}
+
+function getEndLogEntry(startLogEntry, filename) {
+  const timeDelta = process.hrtime(startLogEntry.start_timestamp);
+  const duration_ms = Math.round((timeDelta[0] * 1e9 + timeDelta[1]) / 1e6);
+  return {
+    action_name: "Transforming file",
+    action_phase: "end",
+    file_name: filename,
+    duration_ms,
+    log_entry_label: "Transforming file",
+  };
+}
+
+module.exports = {
+  transform,
+};

package/src/DeltaBundler/Worker.flow.js.flow

@@ -0,0 +1,121 @@
+/**
+ * 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
+ * @format
+ */
+
+'use strict';
+
+import type {TransformResult} from './types.flow';
+import type {LogEntry} from 'metro-core/src/Logger';
+import type {
+  JsTransformerConfig,
+  JsTransformOptions,
+} from 'metro-transform-worker';
+
+const traverse = require('@babel/traverse').default;
+const crypto = require('crypto');
+const fs = require('fs');
+const path = require('path');
+
+export type {JsTransformOptions as TransformOptions} from 'metro-transform-worker';
+
+export type Worker = {
+  +transform: typeof transform,
+};
+
+type TransformerInterface = {
+  transform(
+    JsTransformerConfig,
+    string,
+    string,
+    Buffer,
+    JsTransformOptions,
+  ): Promise<TransformResult<>>,
+};
+
+export type TransformerConfig = {
+  transformerPath: string,
+  transformerConfig: JsTransformerConfig,
+  ...
+};
+
+type Data = $ReadOnly<{
+  result: TransformResult<>,
+  sha1: string,
+  transformFileStartLogEntry: LogEntry,
+  transformFileEndLogEntry: LogEntry,
+}>;
+
+async function transform(
+  filename: string,
+  transformOptions: JsTransformOptions,
+  projectRoot: string,
+  transformerConfig: TransformerConfig,
+): Promise<Data> {
+  // eslint-disable-next-line no-useless-call
+  const Transformer = (require.call(
+    null,
+    transformerConfig.transformerPath,
+  ): TransformerInterface);
+
+  const transformFileStartLogEntry = {
+    action_name: 'Transforming file',
+    action_phase: 'start',
+    file_name: filename,
+    log_entry_label: 'Transforming file',
+    start_timestamp: process.hrtime(),
+  };
+
+  const data = fs.readFileSync(path.resolve(projectRoot, filename));
+  const sha1 = crypto.createHash('sha1').update(data).digest('hex');
+
+  const result = await Transformer.transform(
+    transformerConfig.transformerConfig,
+    projectRoot,
+    filename,
+    data,
+    transformOptions,
+  );
+
+  // The babel cache caches scopes and pathes for already traversed AST nodes.
+  // Clearing the cache here since the nodes of the transformed file are no longer referenced.
+  // This isn't stritcly necessary since the cache uses a WeakMap. However, WeakMap only permit
+  // that unreferenced keys are collected but the values still hold references to the Scope and NodePaths.
+  // Manually clearing the cache allows the GC to collect the Scope and NodePaths without checking if there
+  // exist any other references to the keys.
+  traverse.cache.clear();
+
+  const transformFileEndLogEntry = getEndLogEntry(
+    transformFileStartLogEntry,
+    filename,
+  );
+
+  return {
+    result,
+    sha1,
+    transformFileStartLogEntry,
+    transformFileEndLogEntry,
+  };
+}
+
+function getEndLogEntry(startLogEntry: LogEntry, filename: string): LogEntry {
+  const timeDelta = process.hrtime(startLogEntry.start_timestamp);
+  const duration_ms = Math.round((timeDelta[0] * 1e9 + timeDelta[1]) / 1e6);
+
+  return {
+    action_name: 'Transforming file',
+    action_phase: 'end',
+    file_name: filename,
+    duration_ms,
+    log_entry_label: 'Transforming file',
+  };
+}
+
+module.exports = ({
+  transform,
+}: Worker);

package/src/DeltaBundler/Worker.js

@@ -8,71 +8,13 @@
  * @format
  */
 "use strict";
+/*::
+export type * from './Worker.flow';
+*/
 
-const traverse = require("@babel/traverse").default;
+try {
+  // $FlowFixMe[untyped-import]
+  require("metro-babel-register").unstable_registerForMetroMonorepo();
+} catch {}
 
-const crypto = require("crypto");
-
-const fs = require("fs");
-
-const path = require("path");
-
-async function transform(
-  filename,
-  transformOptions,
-  projectRoot,
-  transformerConfig
-) {
-  // eslint-disable-next-line no-useless-call
-  const Transformer = require.call(null, transformerConfig.transformerPath);
-
-  const transformFileStartLogEntry = {
-    action_name: "Transforming file",
-    action_phase: "start",
-    file_name: filename,
-    log_entry_label: "Transforming file",
-    start_timestamp: process.hrtime(),
-  };
-  const data = fs.readFileSync(path.resolve(projectRoot, filename));
-  const sha1 = crypto.createHash("sha1").update(data).digest("hex");
-  const result = await Transformer.transform(
-    transformerConfig.transformerConfig,
-    projectRoot,
-    filename,
-    data,
-    transformOptions
-  ); // The babel cache caches scopes and pathes for already traversed AST nodes.
-  // Clearing the cache here since the nodes of the transformed file are no longer referenced.
-  // This isn't stritcly necessary since the cache uses a WeakMap. However, WeakMap only permit
-  // that unreferenced keys are collected but the values still hold references to the Scope and NodePaths.
-  // Manually clearing the cache allows the GC to collect the Scope and NodePaths without checking if there
-  // exist any other references to the keys.
-
-  traverse.cache.clear();
-  const transformFileEndLogEntry = getEndLogEntry(
-    transformFileStartLogEntry,
-    filename
-  );
-  return {
-    result,
-    sha1,
-    transformFileStartLogEntry,
-    transformFileEndLogEntry,
-  };
-}
-
-function getEndLogEntry(startLogEntry, filename) {
-  const timeDelta = process.hrtime(startLogEntry.start_timestamp);
-  const duration_ms = Math.round((timeDelta[0] * 1e9 + timeDelta[1]) / 1e6);
-  return {
-    action_name: "Transforming file",
-    action_phase: "end",
-    file_name: filename,
-    duration_ms,
-    log_entry_label: "Transforming file",
-  };
-}
-
-module.exports = {
-  transform,
-};
+module.exports = require("./Worker.flow");

package/src/DeltaBundler/Worker.js.flow

@@ -10,112 +10,13 @@
 
 'use strict';
 
-import type {TransformResult} from './types.flow';
-import type {LogEntry} from 'metro-core/src/Logger';
-import type {
-  JsTransformerConfig,
-  JsTransformOptions,
-} from 'metro-transform-worker';
+/*::
+export type * from './Worker.flow';
+*/
 
-const traverse = require('@babel/traverse').default;
-const crypto = require('crypto');
-const fs = require('fs');
-const path = require('path');
+try {
+  // $FlowFixMe[untyped-import]
+  require('metro-babel-register').unstable_registerForMetroMonorepo();
+} catch {}
 
-export type {JsTransformOptions as TransformOptions} from 'metro-transform-worker';
-
-export type Worker = {
-  +transform: typeof transform,
-};
-
-type TransformerInterface = {
-  transform(
-    JsTransformerConfig,
-    string,
-    string,
-    Buffer,
-    JsTransformOptions,
-  ): Promise<TransformResult<>>,
-};
-
-export type TransformerConfig = {
-  transformerPath: string,
-  transformerConfig: JsTransformerConfig,
-  ...
-};
-
-type Data = $ReadOnly<{
-  result: TransformResult<>,
-  sha1: string,
-  transformFileStartLogEntry: LogEntry,
-  transformFileEndLogEntry: LogEntry,
-}>;
-
-async function transform(
-  filename: string,
-  transformOptions: JsTransformOptions,
-  projectRoot: string,
-  transformerConfig: TransformerConfig,
-): Promise<Data> {
-  // eslint-disable-next-line no-useless-call
-  const Transformer = (require.call(
-    null,
-    transformerConfig.transformerPath,
-  ): TransformerInterface);
-
-  const transformFileStartLogEntry = {
-    action_name: 'Transforming file',
-    action_phase: 'start',
-    file_name: filename,
-    log_entry_label: 'Transforming file',
-    start_timestamp: process.hrtime(),
-  };
-
-  const data = fs.readFileSync(path.resolve(projectRoot, filename));
-  const sha1 = crypto.createHash('sha1').update(data).digest('hex');
-
-  const result = await Transformer.transform(
-    transformerConfig.transformerConfig,
-    projectRoot,
-    filename,
-    data,
-    transformOptions,
-  );
-
-  // The babel cache caches scopes and pathes for already traversed AST nodes.
-  // Clearing the cache here since the nodes of the transformed file are no longer referenced.
-  // This isn't stritcly necessary since the cache uses a WeakMap. However, WeakMap only permit
-  // that unreferenced keys are collected but the values still hold references to the Scope and NodePaths.
-  // Manually clearing the cache allows the GC to collect the Scope and NodePaths without checking if there
-  // exist any other references to the keys.
-  traverse.cache.clear();
-
-  const transformFileEndLogEntry = getEndLogEntry(
-    transformFileStartLogEntry,
-    filename,
-  );
-
-  return {
-    result,
-    sha1,
-    transformFileStartLogEntry,
-    transformFileEndLogEntry,
-  };
-}
-
-function getEndLogEntry(startLogEntry: LogEntry, filename: string): LogEntry {
-  const timeDelta = process.hrtime(startLogEntry.start_timestamp);
-  const duration_ms = Math.round((timeDelta[0] * 1e9 + timeDelta[1]) / 1e6);
-
-  return {
-    action_name: 'Transforming file',
-    action_phase: 'end',
-    file_name: filename,
-    duration_ms,
-    log_entry_label: 'Transforming file',
-  };
-}
-
-module.exports = ({
-  transform,
-}: Worker);
+module.exports = require('./Worker.flow');

package/src/DeltaBundler/WorkerFarm.js.flow

@@ -136,7 +136,7 @@ class WorkerFarm {
     return null;
   }
 
-  _formatGenericError(err, filename: string): TransformError {
+  _formatGenericError(err: any, filename: string): TransformError {
     const error = new TransformError(`${filename}: ${err.message}`);
 
     return Object.assign(error, {
@@ -145,7 +145,7 @@ class WorkerFarm {
     });
   }
 
-  _formatBabelError(err, filename: string): TransformError {
+  _formatBabelError(err: any, filename: string): TransformError {
     const error = new TransformError(
       `${err.type || 'Error'}${
         err.message.includes(filename) ? '' : ' in ' + filename

package/src/DeltaBundler/__fixtures__/hasteImpl.js

@@ -26,4 +26,8 @@ module.exports = {
 
     return matches[1];
   },
+
+  getCacheKey() {
+    return "hasteImplFixture";
+  },
 };