metro-source-map 0.80.5 → 0.80.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "metro-source-map",
-  "version": "0.80.5",
+  "version": "0.80.6",
   "description": "🚇 Source map generator for Metro.",
   "main": "src/source-map.js",
   "repository": {
@@ -15,9 +15,9 @@
     "@babel/traverse": "^7.20.0",
     "@babel/types": "^7.20.0",
     "invariant": "^2.2.4",
-    "metro-symbolicate": "0.80.5",
+    "metro-symbolicate": "0.80.6",
     "nullthrows": "^1.1.1",
-    "ob1": "0.80.5",
+    "ob1": "0.80.6",
     "source-map": "^0.5.6",
     "vlq": "^1.0.0"
   },

package/src/B64Builder.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 encode = require("./encode");
@@ -16,29 +5,12 @@ const MAX_SEGMENT_LENGTH = 7;
 const ONE_MEG = 1024 * 1024;
 const COMMA = 0x2c;
 const SEMICOLON = 0x3b;
-
-/**
- * Efficient builder for base64 VLQ mappings strings.
- *
- * This class uses a buffer that is preallocated with one megabyte and is
- * reallocated dynamically as needed, doubling its size.
- *
- * Encoding never creates any complex value types (strings, objects), and only
- * writes character values to the buffer.
- *
- * For details about source map terminology and specification, check
- * https://docs.google.com/document/d/1U1RGAehQwRypUTovF1KRlpiOFze0b-_2gc6fAH0KY0k/edit
- */
 class B64Builder {
   constructor() {
     this.buffer = Buffer.alloc(ONE_MEG);
     this.pos = 0;
     this.hasSegment = false;
   }
-
-  /**
-   * Adds `n` markers for generated lines to the mappings.
-   */
   markLines(n) {
     if (n < 1) {
       return this;
@@ -52,10 +24,6 @@ class B64Builder {
     }
     return this;
   }
-
-  /**
-   * Starts a segment at the specified column offset in the current line.
-   */
   startSegment(column) {
     if (this.hasSegment) {
       this._writeByte(COMMA);
@@ -65,10 +33,6 @@ class B64Builder {
     this.append(column);
     return this;
   }
-
-  /**
-   * Appends a single number to the mappings.
-   */
   append(value) {
     if (this.pos + MAX_SEGMENT_LENGTH >= this.buffer.length) {
       this._realloc();
@@ -76,10 +40,6 @@ class B64Builder {
     this.pos = encode(value, this.buffer, this.pos);
     return this;
   }
-
-  /**
-   * Returns the string representation of the mappings.
-   */
   toString() {
     return this.buffer.toString("ascii", 0, this.pos);
   }

package/src/BundleBuilder.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 EMPTY_MAP = {
@@ -17,21 +6,6 @@ const EMPTY_MAP = {
   names: [],
   mappings: "A",
 };
-
-/**
- * Builds a source-mapped bundle by concatenating strings and their
- * corresponding source maps (if any).
- *
- * Usage:
- *
- * const builder = new BundleBuilder('bundle.js');
- * builder
- *   .append('foo\n', fooMap)
- *   .append('bar\n')
- *   // ...
- * const code = builder.getCode();
- * const map = builder.getMap();
- */
 class BundleBuilder {
   constructor(file) {
     this._file = file;

package/src/Consumer/AbstractConsumer.js

@@ -1,20 +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";
 
 const { GENERATED_ORDER, iterationOrderToString } = require("./constants");
 const invariant = require("invariant");
-
-// Implementation details shared between MappingsConsumer and SectionsConsumer
 class AbstractConsumer {
   constructor(sourceMap) {
     this._sourceMap = sourceMap;
@@ -34,8 +21,6 @@ class AbstractConsumer {
       callback.call(context, mapping);
     }
   }
-
-  // flowlint-next-line unsafe-getters-setters:off
   get file() {
     return this._sourceMap.file;
   }

package/src/Consumer/DelegatingConsumer.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 {
@@ -18,18 +7,11 @@ const {
   ORIGINAL_ORDER,
 } = require("./constants");
 const createConsumer = require("./createConsumer");
-
-/**
- * A source map consumer that supports both "basic" and "indexed" source maps.
- * Uses `MappingsConsumer` and `SectionsConsumer` under the hood (via
- * `createConsumer`).
- */
 class DelegatingConsumer {
   static GENERATED_ORDER = GENERATED_ORDER;
   static ORIGINAL_ORDER = ORIGINAL_ORDER;
   static GREATEST_LOWER_BOUND = GREATEST_LOWER_BOUND;
   static LEAST_UPPER_BOUND = LEAST_UPPER_BOUND;
-  // $FlowFixMe[incompatible-return]
   constructor(sourceMap) {
     this._rootConsumer = createConsumer(sourceMap);
     return this._rootConsumer;
@@ -43,8 +25,6 @@ class DelegatingConsumer {
   eachMapping(callback, context, order) {
     return this._rootConsumer.eachMapping(callback, context, order);
   }
-
-  // flowlint-next-line unsafe-getters-setters:off
   get file() {
     return this._rootConsumer.file;
   }

package/src/Consumer/MappingsConsumer.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 AbstractConsumer = require("./AbstractConsumer");
@@ -24,11 +13,6 @@ const { greatestLowerBound } = require("./search");
 const invariant = require("invariant");
 const { add, add0, get0, inc, sub } = require("ob1");
 const { decode: decodeVlq } = require("vlq");
-
-/**
- * A source map consumer that supports "basic" source maps (that have a
- * `mappings` field and no sections).
- */
 class MappingsConsumer extends AbstractConsumer {
   constructor(sourceMap) {
     super(sourceMap);
@@ -47,7 +31,6 @@ class MappingsConsumer extends AbstractConsumer {
       invariant(
         generatedPosition.bias === GREATEST_LOWER_BOUND,
         `Unimplemented lookup bias: ${lookupBiasToString(
-          // $FlowFixMe[incompatible-call]
           generatedPosition.bias
         )}`
       );
@@ -98,7 +81,6 @@ class MappingsConsumer extends AbstractConsumer {
         case ";":
           generatedLine = inc(generatedLine);
           generatedColumn = FIRST_COLUMN;
-        /* falls through */
         case ",":
           next = i + 1;
           continue;
@@ -106,7 +88,6 @@ class MappingsConsumer extends AbstractConsumer {
       findNext: for (next = i + 1; next < mappingsRaw.length; ++next) {
         switch (mappingsRaw[next]) {
           case ";":
-          /* falls through */
           case ",":
             break findNext;
         }

package/src/Consumer/SectionsConsumer.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 AbstractConsumer = require("./AbstractConsumer");
@@ -17,11 +6,6 @@ const createConsumer = require("./createConsumer");
 const { subtractOffsetFromPosition } = require("./positionMath");
 const { greatestLowerBound } = require("./search");
 const { add, add0, get0, get1, sub, sub1 } = require("ob1");
-
-/**
- * A source map consumer that supports "indexed" source maps (that have a
- * `sections` field and no top-level mappings).
- */
 class SectionsConsumer extends AbstractConsumer {
   constructor(sourceMap) {
     super(sourceMap);

package/src/Consumer/constants.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 { add0, add1 } = require("ob1");

package/src/Consumer/createConsumer.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 invariant = require("invariant");
@@ -19,8 +8,6 @@ function createConsumer(sourceMap) {
   );
   const MappingsConsumer = require("./MappingsConsumer");
   const SectionsConsumer = require("./SectionsConsumer");
-
-  // eslint-disable-next-line lint/strictly-null
   if (sourceMap.mappings === undefined) {
     return new SectionsConsumer(sourceMap);
   }

package/src/Consumer/index.js

@@ -1,16 +1,4 @@
-/**
- * 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";
 
-// Implements an API-compatible subset of source-map's `SourceMapConsumer`.
 const DelegatingConsumer = require("./DelegatingConsumer");
 module.exports = DelegatingConsumer;

package/src/Consumer/normalizeSourcePath.js

@@ -1,32 +1,11 @@
-/**
- * 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";
 
-// flowlint-next-line untyped-import:off
 const util = require("source-map/lib/util");
-
-// Extracted from source-map@0.5.6's SourceMapConsumer
 function normalizeSourcePath(sourceInput, map) {
   const { sourceRoot } = map;
   let source = sourceInput;
   source = String(source);
-  // Some source maps produce relative source paths like "./foo.js" instead of
-  // "foo.js".  Normalize these first so that future comparisons will succeed.
-  // See bugzil.la/1090768.
   source = util.normalize(source);
-  // Always ensure that absolute sources are internally stored relative to
-  // the source root, if the source root is absolute. Not doing this would
-  // be particularly problematic when the source root is a prefix of the
-  // source (valid, but why??). See github issue #199 and bugzil.la/1188982.
   source =
     sourceRoot != null && util.isAbsolute(sourceRoot) && util.isAbsolute(source)
       ? util.relative(sourceRoot, source)

package/src/Consumer/positionMath.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 { add, add0, add1, neg } = require("ob1");

package/src/Consumer/search.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";
 
 function greatestLowerBound(elements, target, comparator) {

package/src/Consumer/types.flow.js

@@ -1,12 +1 @@
-/**
- * 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";

package/src/Generator.js

@@ -1,37 +1,12 @@
-/**
- * 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 B64Builder = require("./B64Builder");
-/**
- * Generates a source map from raw mappings.
- *
- * Raw mappings are a set of 2, 4, or five elements:
- *
- * - line and column number in the generated source
- * - line and column number in the original source
- * - symbol name in the original source
- *
- * Mappings have to be passed in the order appearance in the generated source.
- */
 class Generator {
-  // https://developer.chrome.com/blog/devtools-better-angular-debugging/#the-x_google_ignorelist-source-map-extension
-
   constructor() {
     this.builder = new B64Builder();
     this.last = {
       generatedColumn: 0,
       generatedLine: 1,
-      // lines are passed in 1-indexed
       name: 0,
       source: 0,
       sourceColumn: 0,
@@ -44,10 +19,6 @@ class Generator {
     this.x_facebook_sources = [];
     this.x_google_ignoreList = [];
   }
-
-  /**
-   * Mark the beginning of a new source file.
-   */
   startFile(file, code, functionMap, flags) {
     const { addToIgnoreList = false } = flags ?? {};
     const sourceIndex = this.sources.push(file) - 1;
@@ -58,17 +29,9 @@ class Generator {
       this.x_google_ignoreList.push(sourceIndex);
     }
   }
-
-  /**
-   * Mark the end of the current source file
-   */
   endFile() {
     this.source = -1;
   }
-
-  /**
-   * Adds a mapping for generated code without a corresponding source location.
-   */
   addSimpleMapping(generatedLine, generatedColumn) {
     const last = this.last;
     if (
@@ -91,10 +54,6 @@ class Generator {
     this.builder.startSegment(generatedColumn - last.generatedColumn);
     last.generatedColumn = generatedColumn;
   }
-
-  /**
-   * Adds a mapping for generated code with a corresponding source location.
-   */
   addSourceMapping(generatedLine, generatedColumn, sourceLine, sourceColumn) {
     this.addSimpleMapping(generatedLine, generatedColumn);
     const last = this.last;
@@ -106,10 +65,6 @@ class Generator {
     last.sourceColumn = sourceColumn;
     last.sourceLine = sourceLine;
   }
-
-  /**
-   * Adds a mapping for code with a corresponding source location + symbol name.
-   */
   addNamedSourceMapping(
     generatedLine,
     generatedColumn,
@@ -128,10 +83,6 @@ class Generator {
     this.builder.append(nameIndex - last.name);
     last.name = nameIndex;
   }
-
-  /**
-   * Return the source map as object.
-   */
   toMap(file, options) {
     const content =
       options && options.excludeSource === true
@@ -162,12 +113,6 @@ class Generator {
       mappings: this.builder.toString(),
     };
   }
-
-  /**
-   * Return the source map as string.
-   *
-   * This is ~2.5x faster than calling `JSON.stringify(generator.toMap())`
-   */
   toString(file, options) {
     let content;
     if (options && options.excludeSource === true) {
@@ -204,11 +149,6 @@ class Generator {
       "}"
     );
   }
-
-  /**
-   * Determine whether we need to write the `x_facebook_sources` field.
-   * If the metadata is all `null`s, we can omit the field entirely.
-   */
   hasSourcesMetadata() {
     return this.x_facebook_sources.some(
       (metadata) => metadata != null && metadata.some((value) => value != null)

package/src/composeSourceMaps.js

@@ -1,25 +1,9 @@
-/**
- * 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 Consumer = require("./Consumer");
 const { SourceMapGenerator } = require("source-map");
-
-// TODO(t67648443): Bypass the `sort-requires` rule for this file because of a dependency cycle.
 Consumer;
-
-// Originally based on https://github.com/jakobwesthoff/source-map-merger
 function composeSourceMaps(maps) {
-  // NOTE: require() here to break dependency cycle
   const SourceMetadataMapConsumer = require("metro-symbolicate/src/SourceMetadataMapConsumer");
   const GoogleIgnoreListConsumer = require("metro-symbolicate/src/GoogleIgnoreListConsumer");
   if (maps.length < 1) {
@@ -111,7 +95,6 @@ function findOriginalPosition(consumers, generatedLine, generatedColumn) {
       };
     }
   }
-  // $FlowFixMe[incompatible-return] `Number0`, `Number1` is incompatible with number
   return original;
 }
 module.exports = composeSourceMaps;

package/src/encode.js

@@ -1,57 +1,5 @@
-/**
- * Portions 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
- */
-
-/**
- * Copyright 2011 Mozilla Foundation and contributors
- * Licensed under the New BSD license. See LICENSE or:
- * http://opensource.org/licenses/BSD-3-Clause
- *
- * Based on the Base 64 VLQ implementation in Closure Compiler:
- * https://git.io/vymuA
- *
- * Copyright 2011 The Closure Compiler Authors. All rights reserved.
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met:
- *
- *  * Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- *  * Redistributions in binary form must reproduce the above
- *    copyright notice, this list of conditions and the following
- *    disclaimer in the documentation and/or other materials provided
- *    with the distribution.
- *  * Neither the name of Google Inc. nor the names of its
- *    contributors may be used to endorse or promote products derived
- *    from this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- *
- * @copyright
- */
-
-/* eslint-disable no-bitwise */
-
 "use strict";
 
-// A map of values to characters for the b64 encoding
 const CHAR_MAP = [
   0x41, 0x42, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48, 0x49, 0x4a, 0x4b, 0x4c, 0x4d,
   0x4e, 0x4f, 0x50, 0x51, 0x52, 0x53, 0x54, 0x55, 0x56, 0x57, 0x58, 0x59, 0x5a,
@@ -59,49 +7,13 @@ const CHAR_MAP = [
   0x6e, 0x6f, 0x70, 0x71, 0x72, 0x73, 0x74, 0x75, 0x76, 0x77, 0x78, 0x79, 0x7a,
   0x30, 0x31, 0x32, 0x33, 0x34, 0x35, 0x36, 0x37, 0x38, 0x39, 0x2b, 0x2f,
 ];
-
-// A single base 64 digit can contain 6 bits of data. For the base 64 variable
-// length quantities we use in the source map spec, the first bit is the sign,
-// the next four bits are the actual value, and the 6th bit is the
-// continuation bit. The continuation bit tells us whether there are more
-// digits in this value following this digit.
-//
-//   Continuation
-//   |    Sign
-//   |    |
-//   V    V
-//   101011
-
 const VLQ_BASE_SHIFT = 5;
-
-// binary: 100000
 const VLQ_BASE = 1 << VLQ_BASE_SHIFT;
-
-// binary: 011111
 const VLQ_BASE_MASK = VLQ_BASE - 1;
-
-// binary: 100000
 const VLQ_CONTINUATION_BIT = VLQ_BASE;
-
-/**
- * Converts from a two-complement value to a value where the sign bit is
- * placed in the least significant bit.  For example, as decimals:
- *   1 becomes 2 (10 binary), -1 becomes 3 (11 binary)
- *   2 becomes 4 (100 binary), -2 becomes 5 (101 binary)
- */
 function toVLQSigned(value) {
   return value < 0 ? (-value << 1) + 1 : (value << 1) + 0;
 }
-
-/**
- * Encodes a number to base64 VLQ format and appends it to the passed-in buffer
- *
- * DON'T USE COMPOUND OPERATORS (eg `>>>=`) ON `let`-DECLARED VARIABLES!
- * V8 WILL DEOPTIMIZE THIS FUNCTION AND MAP CREATION WILL BE 25% SLOWER!
- *
- * DON'T ADD MORE COMMENTS TO THIS FUNCTION TO KEEP ITS LENGTH SHORT ENOUGH FOR
- * V8 OPTIMIZATION!
- */
 function encode(value, buffer, position) {
   let vlq = toVLQSigned(value);
   let digit;
@@ -109,8 +21,6 @@ function encode(value, buffer, position) {
     digit = vlq & VLQ_BASE_MASK;
     vlq = vlq >>> VLQ_BASE_SHIFT;
     if (vlq > 0) {
-      // There are still more digits in this value, so we must make sure the
-      // continuation bit is marked.
       digit = digit | VLQ_CONTINUATION_BIT;
     }
     buffer[position++] = CHAR_MAP[digit];

package/src/generateFunctionMap.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";
 
 var _traverse = _interopRequireDefault(require("@babel/traverse"));
@@ -21,26 +10,11 @@ const t = require("@babel/types");
 const invariant = require("invariant");
 const nullthrows = require("nullthrows");
 const fsPath = require("path");
-/**
- * Generate a map of source positions to function names. The names are meant to
- * describe the stack frame in an error trace and may contain more contextual
- * information than just the actual name of the function.
- *
- * The output is encoded for use in a source map. For details about the format,
- * see MappingEncoder below.
- */
 function generateFunctionMap(ast, context) {
   const encoder = new MappingEncoder();
   forEachMapping(ast, context, (mapping) => encoder.push(mapping));
   return encoder.getResult();
 }
-
-/**
- * Same as generateFunctionMap, but returns the raw array of mappings instead
- * of encoding it for use in a source map.
- *
- * Lines are 1-based and columns are 0-based.
- */
 function generateFunctionMappingsArray(ast, context) {
   const mappings = [];
   forEachMapping(ast, context, (mapping) => {
@@ -50,9 +24,6 @@ function generateFunctionMappingsArray(ast, context) {
 }
 function functionMapBabelPlugin() {
   return {
-    // Eagerly traverse the tree on `pre`, before any visitors have run, so
-    // that regardless of plugin order we're dealing with the AST before any
-    // mutations.
     visitor: {},
     pre: ({ path, metadata, opts }) => {
       const { filename } = nullthrows(opts);
@@ -67,8 +38,6 @@ function functionMapBabelPlugin() {
         path && t.isProgram(path.node),
         "path missing or not a program node"
       );
-      // $FlowFixMe[prop-missing] checked above
-      // $FlowFixMe[incompatible-type-arg] checked above
       const programPath = path;
       visitor.enter(programPath);
       programPath.traverse({
@@ -76,12 +45,8 @@ function functionMapBabelPlugin() {
         Class: visitor,
       });
       visitor.exit(programPath);
-
-      // $FlowFixMe[prop-missing] Babel `File` is not generically typed
       const metroMetadata = metadata;
       const functionMap = encoder.getResult();
-
-      // Set the result on a metadata property
       if (!metroMetadata.metro) {
         metroMetadata.metro = {
           functionMap,
@@ -101,7 +66,7 @@ function getFunctionMapVisitor(context, pushMapping) {
   let tailName = null;
   function advanceToPos(pos) {
     if (tailPos && positionGreater(pos, tailPos)) {
-      const name = nameStack[0].name; // We always have at least Program
+      const name = nameStack[0].name;
       if (name !== tailName) {
         pushMapping({
           name,
@@ -149,22 +114,11 @@ function getFunctionMapVisitor(context, pushMapping) {
     },
   };
 }
-
-/**
- * Traverses a Babel AST and calls the supplied callback with function name
- * mappings, one at a time.
- */
 function forEachMapping(ast, context, pushMapping) {
   const visitor = getFunctionMapVisitor(context, pushMapping);
-
-  // Traversing populates/pollutes the path cache (`traverse.cache.path`) with
-  // values missing the `hub` property needed by Babel transformation, so we
-  // save, clear, and restore the cache around our traversal.
-  // See: https://github.com/facebook/metro/pull/854#issuecomment-1336499395
   const previousCache = _traverse.default.cache.path;
   _traverse.default.cache.clearPath();
   (0, _traverse.default)(ast, {
-    // Our visitor doesn't care about scope
     noScope: true,
     Function: visitor,
     Program: visitor,
@@ -173,88 +127,59 @@ function forEachMapping(ast, context, pushMapping) {
   _traverse.default.cache.path = previousCache;
 }
 const ANONYMOUS_NAME = "<anonymous>";
-
-/**
- * Derive a contextual name for the given AST node (Function, Program, Class or
- * ObjectExpression).
- */
 function getNameForPath(path) {
   const { node, parent, parentPath } = path;
   if ((0, _types.isProgram)(node)) {
     return "<global>";
   }
   let { id } = path;
-  // has an `id` so we don't need to infer one
   if (node.id) {
-    // $FlowFixMe Flow error uncovered by typing Babel more strictly
     return node.id.name;
   }
   let propertyPath;
   let kind;
-
-  // Find or construct an AST node that names the current node.
   if ((0, _types.isObjectMethod)(node) || (0, _types.isClassMethod)(node)) {
-    // ({ foo() {} });
     id = node.key;
     if (node.kind !== "method" && node.kind !== "constructor") {
-      // Store the method's kind so we can add it to the final name.
       kind = node.kind;
     }
-    // Also store the path to the property so we can find its context
-    // (object/class) later and add _its_ name to the result.
     propertyPath = path;
   } else if (
     (0, _types.isObjectProperty)(parent) ||
     (0, _types.isClassProperty)(parent)
   ) {
-    // ({ foo: function() {} });
     id = parent.key;
-    // Also store the path to the property so we can find its context
-    // (object/class) later and add _its_ name to the result.
     propertyPath = parentPath;
   } else if ((0, _types.isVariableDeclarator)(parent)) {
-    // let foo = function () {};
     id = parent.id;
   } else if ((0, _types.isAssignmentExpression)(parent)) {
-    // foo = function () {};
     id = parent.left;
   } else if ((0, _types.isJSXExpressionContainer)(parent)) {
     const grandParentNode = parentPath?.parentPath?.node;
     if ((0, _types.isJSXElement)(grandParentNode)) {
-      // <foo>{function () {}}</foo>
       const openingElement = grandParentNode.openingElement;
       id = t.jsxMemberExpression(
-        // $FlowFixMe Flow error uncovered by typing Babel more strictly
         t.jsxMemberExpression(openingElement.name, t.jsxIdentifier("props")),
         t.jsxIdentifier("children")
       );
     } else if ((0, _types.isJSXAttribute)(grandParentNode)) {
-      // <foo bar={function () {}} />
       const openingElement = parentPath?.parentPath?.parentPath?.node;
       const prop = grandParentNode;
       id = t.jsxMemberExpression(
-        // $FlowFixMe Flow error uncovered by typing Babel more strictly
         t.jsxMemberExpression(openingElement.name, t.jsxIdentifier("props")),
-        // $FlowFixMe Flow error uncovered by typing Babel more strictly
         prop.name
       );
     }
   }
-
-  // Collapse the name AST, if any, into a string.
   let name = getNameFromId(id);
   if (name == null) {
-    // We couldn't find a name directly. Try the parent in certain cases.
     if (isAnyCallExpression(parent)) {
-      // foo(function () {})
       const argIndex = parent.arguments.indexOf(node);
       if (argIndex !== -1) {
         const calleeName = getNameFromId(parent.callee);
-        // var f = Object.freeze(function () {})
         if (argIndex === 0 && calleeName === "Object.freeze") {
           return getNameForPath(nullthrows(parentPath));
         }
-        // var f = useCallback(function () {})
         if (
           argIndex === 0 &&
           (calleeName === "useCallback" || calleeName === "React.useCallback")
@@ -275,19 +200,13 @@ function getNameForPath(path) {
     if ((0, _types.isExportDefaultDeclaration)(parent)) {
       return "default";
     }
-    // We couldn't infer a name at all.
     return ANONYMOUS_NAME;
   }
-
-  // Annotate getters and setters.
   if (kind != null) {
     name = kind + "__" + name;
   }
-
-  // Annotate members with the name of their containing object/class.
   if (propertyPath) {
     if ((0, _types.isClassBody)(propertyPath.parent)) {
-      // $FlowFixMe Discovered when typing babel-traverse
       const className = getNameForPath(propertyPath.parentPath.parentPath);
       if (className !== ANONYMOUS_NAME) {
         const separator = propertyPath.node.static ? "." : "#";
@@ -302,8 +221,6 @@ function getNameForPath(path) {
   }
   return name;
 }
-
-// $FlowFixMe[deprecated-type]
 function isAnyCallExpression(node) {
   return (
     node.type === "CallExpression" ||
@@ -311,8 +228,6 @@ function isAnyCallExpression(node) {
     node.type === "OptionalCallExpression"
   );
 }
-
-// $FlowFixMe[deprecated-type]
 function isAnyMemberExpression(node) {
   return (
     node.type === "MemberExpression" ||
@@ -320,8 +235,6 @@ function isAnyMemberExpression(node) {
     node.type === "OptionalMemberExpression"
   );
 }
-
-// $FlowFixMe[deprecated-type]
 function isAnyIdentifier(node) {
   return (0, _types.isIdentifier)(node) || (0, _types.isJSXIdentifier)(node);
 }
@@ -399,12 +312,6 @@ function getNamePartsFromId(id) {
   return name ? [name] : [];
 }
 const DELIMITER_START_RE = /^[^A-Za-z0-9_$@]+/;
-
-/**
- * Strip the given prefix from `name`, if it occurs there, plus any delimiter
- * characters that follow (of which at least one is required). If an empty
- * string would be returned, return the original name instead.
- */
 function removeNamePrefix(name, namePrefix) {
   if (!namePrefix.length || !name.startsWith(namePrefix)) {
     return name;
@@ -416,31 +323,6 @@ function removeNamePrefix(name, namePrefix) {
   }
   return name;
 }
-
-/**
- * Encodes function name mappings as deltas in a Base64 VLQ format inspired by
- * the standard source map format.
- *
- * Mappings on different lines are separated with a single `;` (even if there
- * are multiple intervening lines).
- * Mappings on the same line are separated with `,`.
- *
- * The first mapping of a line has the fields:
- *  [column delta, name delta, line delta]
- *
- * where the column delta is relative to the beginning of the line, the name
- * delta is relative to the previously occurring name, and the line delta is
- * relative to the previously occurring line.
- *
- * The 2...nth other mappings of a line have the fields:
- *  [column delta, name delta]
- *
- * where both fields are relative to their previous running values. The line
- * delta is omitted since it is always 0 by definition.
- *
- * Lines and columns are both 0-based in the serialised format. In memory,
- * lines are 1-based while columns are 0-based.
- */
 class MappingEncoder {
   constructor() {
     this._namesMap = new Map();
@@ -466,7 +348,6 @@ class MappingEncoder {
     const lineDelta = this._line.next(start.line);
     const firstOfLine = this._mappings.pos === 0 || lineDelta > 0;
     if (lineDelta > 0) {
-      // The next entry will have the line offset, so emit just one semicolon.
       this._mappings.markLines(1);
       this._column.reset(0);
     }

package/src/generateFunctionMap.js.flow

@@ -249,7 +249,7 @@ function getNameForPath(path: NodePath<>): string {
     return '<global>';
   }
 
-  let {id} = (path: any);
+  let {id}: any = path;
   // has an `id` so we don't need to infer one
   if (node.id) {
     // $FlowFixMe Flow error uncovered by typing Babel more strictly

package/src/source-map.js

@@ -1,27 +1,14 @@
-/**
- * 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 { BundleBuilder, createIndexMap } = require("./BundleBuilder");
 const composeSourceMaps = require("./composeSourceMaps");
 const Consumer = require("./Consumer");
-// We need to export this for `metro-symbolicate`
 const normalizeSourcePath = require("./Consumer/normalizeSourcePath");
 const {
   functionMapBabelPlugin,
   generateFunctionMap,
 } = require("./generateFunctionMap");
 const Generator = require("./Generator");
-// $FlowFixMe[untyped-import] - source-map
 const SourceMap = require("source-map");
 function fromRawMappingsImpl(isBlocking, onDone, modules, offsetLines) {
   const modulesToProcess = modules.slice();
@@ -52,14 +39,9 @@ function fromRawMappingsImpl(isBlocking, onDone, modules, offsetLines) {
         break;
       }
       if (!isBlocking) {
-        // Keep the loop running but try to avoid blocking
-        // for too long because this is not in a worker yet.
         const diff = process.hrtime(time);
         const NS_IN_MS = 1000000;
         if (diff[1] > 50 * NS_IN_MS) {
-          // We've blocked for more than 50ms.
-          // This code currently runs on the main thread,
-          // so let's give Metro an opportunity to handle requests.
           setImmediate(workLoop);
           break;
         }
@@ -68,14 +50,6 @@ function fromRawMappingsImpl(isBlocking, onDone, modules, offsetLines) {
   }
   workLoop();
 }
-
-/**
- * Creates a source map from modules with "raw mappings", i.e. an array of
- * tuples with either 2, 4, or 5 elements:
- * generated line, generated column, source line, source line, symbol name.
- * Accepts an `offsetLines` argument in case modules' code is to be offset in
- * the resulting bundle, e.g. by some prefix code.
- */
 function fromRawMappings(modules, offsetLines = 0) {
   let generator;
   fromRawMappingsImpl(
@@ -96,11 +70,6 @@ async function fromRawMappingsNonBlocking(modules, offsetLines = 0) {
     fromRawMappingsImpl(false, resolve, modules, offsetLines);
   });
 }
-
-/**
- * Transforms a standard source map object into a Raw Mappings object, to be
- * used across the bundler.
- */
 function toBabelSegments(sourceMap) {
   const rawMappings = [];
   new SourceMap.SourceMapConsumer(sourceMap).eachMapping((map) => {
@@ -152,7 +121,6 @@ function addMappingsForFile(generator, mappings, module, carryOver) {
 }
 function addMapping(generator, mapping, carryOver) {
   const line = mapping[0] + carryOver;
-  // lines start at 1, columns start at 0
   const column = mapping[1];
   switch (mapping.length) {
     case 2: