metro-transform-worker 0.80.4 → 0.80.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "metro-transform-worker",
-  "version": "0.80.4",
+  "version": "0.80.6",
   "description": "🚇 Transform worker for Metro.",
   "main": "src/index.js",
   "repository": {
@@ -17,17 +17,17 @@
     "@babel/generator": "^7.20.0",
     "@babel/parser": "^7.20.0",
     "@babel/types": "^7.20.0",
-    "metro": "0.80.4",
-    "metro-babel-transformer": "0.80.4",
-    "metro-cache": "0.80.4",
-    "metro-cache-key": "0.80.4",
-    "metro-source-map": "0.80.4",
-    "metro-transform-plugins": "0.80.4",
+    "metro": "0.80.6",
+    "metro-babel-transformer": "0.80.6",
+    "metro-cache": "0.80.6",
+    "metro-cache-key": "0.80.6",
+    "metro-minify-terser": "0.80.6",
+    "metro-source-map": "0.80.6",
+    "metro-transform-plugins": "0.80.6",
     "nullthrows": "^1.1.1"
   },
   "devDependencies": {
-    "metro-memory-fs": "0.80.4",
-    "metro-minify-terser": "0.80.4",
+    "metro-memory-fs": "0.80.6",
     "@react-native/metro-babel-transformer": "0.73.11"
   },
   "engines": {

package/src/index.js

@@ -1,14 +1,3 @@
-/**
- * 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
- * @oncall react_native
- */
-
 "use strict";
 
 const getMinifier = require("./utils/getMinifier");
@@ -61,10 +50,8 @@ const minifyCode = async (
       code,
       source,
       map,
-      // functionMap is overridden by the serializer
       functionMap: null,
       path: filename,
-      // isIgnored is overriden by the serializer
       isIgnored: false,
     },
   ]).toMap(undefined, {});
@@ -106,17 +93,12 @@ class InvalidRequireCallError extends Error {
   }
 }
 async function transformJS(file, { config, options, projectRoot }) {
-  // Transformers can output null ASTs (if they ignore the file). In that case
-  // we need to parse the module source code to get their AST.
   let ast =
     file.ast ??
     babylon.parse(file.code, {
       sourceType: "unambiguous",
     });
   const { importDefault, importAll } = generateImportNames(ast);
-
-  // Add "use strict" if the file was parsed as a module, and the directive did
-  // not exist yet.
   const { directives } = ast.program;
   if (
     ast.program.sourceType === "module" &&
@@ -125,9 +107,6 @@ async function transformJS(file, { config, options, projectRoot }) {
   ) {
     directives.push(types.directive(types.directiveLiteral("use strict")));
   }
-
-  // Perform the import-export transform (in case it's still needed), then
-  // fold requires and perform constant folding (if in dev).
   const plugins = [];
   const babelPluginOpts = {
     ...options,
@@ -158,18 +137,10 @@ async function transformJS(file, { config, options, projectRoot }) {
       filename: file.filename,
       plugins,
       sourceMaps: false,
-      // Not-Cloning the input AST here should be safe because other code paths above this call
-      // are mutating the AST as well and no code is depending on the original AST.
-      // However, switching the flag to false caused issues with ES Modules if `experimentalImportSupport` isn't used https://github.com/facebook/metro/issues/641
-      // either because one of the plugins is doing something funky or Babel messes up some caches.
-      // Make sure to test the above mentioned case before flipping the flag back to false.
       cloneInputAst: true,
     }).ast
   );
   if (!options.dev) {
-    // Run the constant folding plugin in its own pass, avoiding race conditions
-    // with other plugins that have exit() visitors on Program (e.g. the ESM
-    // transform).
     ast = nullthrows(
       transformFromAstSync(ast, "", {
         ast: true,
@@ -189,11 +160,6 @@ async function transformJS(file, { config, options, projectRoot }) {
   let dependencyMapName = "";
   let dependencies;
   let wrappedAst;
-
-  // If the module to transform is a script (meaning that is not part of the
-  // dependency graph and it code will just be prepended to the bundle modules),
-  // we need to wrap it differently than a commonJS module (also, scripts do
-  // not have dependencies).
   if (file.type === "js/script") {
     dependencies = [];
     wrappedAst = JsFileWrapping.wrapPolyfill(ast);
@@ -297,10 +263,6 @@ async function transformJS(file, { config, options, projectRoot }) {
     output,
   };
 }
-
-/**
- * Transforms an asset file
- */
 async function transformAsset(file, context) {
   const assetTransformer = require("./utils/assetTransformer");
   const { assetRegistryPath, assetPlugins } = context.config;
@@ -317,17 +279,10 @@ async function transformAsset(file, context) {
   };
   return transformJS(jsFile, context);
 }
-
-/**
- * Transforms a JavaScript file with Babel before processing the file with
- * the generic JavaScript transformation.
- */
 async function transformJSWithBabel(file, context) {
   const { babelTransformerPath } = context.config;
-  // $FlowFixMe[unsupported-syntax] dynamic require
   const transformer = require(babelTransformerPath);
   const transformResult = await transformer.transform(
-    // functionMapBabelPlugin populates metadata.metro.functionMap
     getBabelTransformArgs(file, context, [functionMapBabelPlugin])
   );
   const jsFile = {
@@ -335,7 +290,6 @@ async function transformJSWithBabel(file, context) {
     ast: transformResult.ast,
     functionMap:
       transformResult.metadata?.metro?.functionMap ??
-      // Fallback to deprecated explicitly-generated `functionMap`
       transformResult.functionMap ??
       null,
   };
@@ -347,8 +301,6 @@ async function transformJSON(file, { options, config, projectRoot }) {
       ? JsFileWrapping.jsonToCommonJS(file.code)
       : JsFileWrapping.wrapJson(file.code, config.globalPrefix);
   let map = [];
-
-  // TODO: When we can reuse transformJS for JSON, we should not derive `minify` separately.
   const minify =
     options.minify &&
     options.unstable_transformProfile !== "hermes-canary" &&
@@ -466,8 +418,6 @@ module.exports = {
       require.resolve("metro/src/ModuleGraph/worker/JsFileWrapping"),
       ...metroTransformPlugins.getTransformPluginCacheKeyFiles(),
     ]);
-
-    // $FlowFixMe[unsupported-syntax]
     const babelTransformer = require(babelTransformerPath);
     return [
       filesKey,

package/src/utils/assetTransformer.js

@@ -1,14 +1,3 @@
-/**
- * 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
- * @oncall react_native
- */
-
 "use strict";
 
 const { getAssetData } = require("metro/src/Assets");

package/src/utils/getMinifier.js

@@ -1,22 +1,7 @@
-/**
- * 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
- * @oncall react_native
- */
-
 "use strict";
 
 function getMinifier(minifierPath) {
-  // Note: minifierPath should be an absolute path OR a module name here!
-  // The options allow relative paths but they HAVE to be normalized at
-  // any entry point that accepts them...
   try {
-    // $FlowFixMe TODO t0 cannot do require with literal
     return require(minifierPath);
   } catch (e) {
     throw new Error(