metro-source-map 0.63.0 → 0.64.0

package/package.json

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

package/src/Consumer/AbstractConsumer.js

@@ -13,9 +13,8 @@ const invariant = require("invariant");
 
 const _require = require("./constants"),
   GENERATED_ORDER = _require.GENERATED_ORDER,
-  iterationOrderToString = _require.iterationOrderToString;
+  iterationOrderToString = _require.iterationOrderToString; // Implementation details shared between MappingsConsumer and SectionsConsumer
 
-// Implementation details shared between MappingsConsumer and SectionsConsumer
 class AbstractConsumer {
   constructor(sourceMap) {
     this._sourceMap = sourceMap;

package/src/Consumer/DelegatingConsumer.js

@@ -20,6 +20,7 @@ function _defineProperty(obj, key, value) {
   } else {
     obj[key] = value;
   }
+
   return obj;
 }
 
@@ -30,12 +31,12 @@ const _require = require("./constants"),
   ORIGINAL_ORDER = _require.ORIGINAL_ORDER,
   GREATEST_LOWER_BOUND = _require.GREATEST_LOWER_BOUND,
   LEAST_UPPER_BOUND = _require.LEAST_UPPER_BOUND;
-
 /**
  * A source map consumer that supports both "basic" and "indexed" source maps.
  * Uses `MappingsConsumer` and `SectionsConsumer` under the hood (via
  * `createConsumer`).
  */
+
 class DelegatingConsumer {
   constructor(sourceMap) {
     this._rootConsumer = createConsumer(sourceMap);

package/src/Consumer/MappingsConsumer.js

@@ -60,7 +60,9 @@ function _unsupportedIterableToArray(o, minLen) {
 
 function _arrayLikeToArray(arr, len) {
   if (len == null || len > arr.length) len = arr.length;
+
   for (var i = 0, arr2 = new Array(len); i < len; i++) arr2[i] = arr[i];
+
   return arr2;
 }
 
@@ -71,6 +73,7 @@ function _iterableToArrayLimit(arr, i) {
   var _n = true;
   var _d = false;
   var _e = undefined;
+
   try {
     for (
       var _i = arr[Symbol.iterator](), _s;
@@ -78,6 +81,7 @@ function _iterableToArrayLimit(arr, i) {
       _n = true
     ) {
       _arr.push(_s.value);
+
       if (i && _arr.length === i) break;
     }
   } catch (err) {
@@ -90,6 +94,7 @@ function _iterableToArrayLimit(arr, i) {
       if (_d) throw _e;
     }
   }
+
   return _arr;
 }
 
@@ -99,6 +104,7 @@ function _arrayWithHoles(arr) {
 
 function ownKeys(object, enumerableOnly) {
   var keys = Object.keys(object);
+
   if (Object.getOwnPropertySymbols) {
     var symbols = Object.getOwnPropertySymbols(object);
     if (enumerableOnly)
@@ -107,12 +113,14 @@ function ownKeys(object, enumerableOnly) {
       });
     keys.push.apply(keys, symbols);
   }
+
   return keys;
 }
 
 function _objectSpread(target) {
   for (var i = 1; i < arguments.length; i++) {
     var source = arguments[i] != null ? arguments[i] : {};
+
     if (i % 2) {
       ownKeys(Object(source), true).forEach(function(key) {
         _defineProperty(target, key, source[key]);
@@ -129,6 +137,7 @@ function _objectSpread(target) {
       });
     }
   }
+
   return target;
 }
 
@@ -143,6 +152,7 @@ function _defineProperty(obj, key, value) {
   } else {
     obj[key] = value;
   }
+
   return obj;
 }
 
@@ -171,11 +181,11 @@ const _require3 = require("ob1"),
 
 const _require4 = require("vlq"),
   decodeVlq = _require4.decode;
-
 /**
  * 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);

package/src/Consumer/SectionsConsumer.js

@@ -11,6 +11,7 @@
 
 function ownKeys(object, enumerableOnly) {
   var keys = Object.keys(object);
+
   if (Object.getOwnPropertySymbols) {
     var symbols = Object.getOwnPropertySymbols(object);
     if (enumerableOnly)
@@ -19,12 +20,14 @@ function ownKeys(object, enumerableOnly) {
       });
     keys.push.apply(keys, symbols);
   }
+
   return keys;
 }
 
 function _objectSpread(target) {
   for (var i = 1; i < arguments.length; i++) {
     var source = arguments[i] != null ? arguments[i] : {};
+
     if (i % 2) {
       ownKeys(Object(source), true).forEach(function(key) {
         _defineProperty(target, key, source[key]);
@@ -41,6 +44,7 @@ function _objectSpread(target) {
       });
     }
   }
+
   return target;
 }
 
@@ -55,6 +59,7 @@ function _defineProperty(obj, key, value) {
   } else {
     obj[key] = value;
   }
+
   return obj;
 }
 
@@ -85,7 +90,9 @@ function _unsupportedIterableToArray(o, minLen) {
 
 function _arrayLikeToArray(arr, len) {
   if (len == null || len > arr.length) len = arr.length;
+
   for (var i = 0, arr2 = new Array(len); i < len; i++) arr2[i] = arr[i];
+
   return arr2;
 }
 
@@ -96,6 +103,7 @@ function _iterableToArrayLimit(arr, i) {
   var _n = true;
   var _d = false;
   var _e = undefined;
+
   try {
     for (
       var _i = arr[Symbol.iterator](), _s;
@@ -103,6 +111,7 @@ function _iterableToArrayLimit(arr, i) {
       _n = true
     ) {
       _arr.push(_s.value);
+
       if (i && _arr.length === i) break;
     }
   } catch (err) {
@@ -115,6 +124,7 @@ function _iterableToArrayLimit(arr, i) {
       if (_d) throw _e;
     }
   }
+
   return _arr;
 }
 
@@ -144,11 +154,11 @@ const _require4 = require("ob1"),
   add0 = _require4.add0,
   sub1 = _require4.sub1,
   sub = _require4.sub;
-
 /**
  * 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/positionMath.js

@@ -11,6 +11,7 @@
 
 function ownKeys(object, enumerableOnly) {
   var keys = Object.keys(object);
+
   if (Object.getOwnPropertySymbols) {
     var symbols = Object.getOwnPropertySymbols(object);
     if (enumerableOnly)
@@ -19,12 +20,14 @@ function ownKeys(object, enumerableOnly) {
       });
     keys.push.apply(keys, symbols);
   }
+
   return keys;
 }
 
 function _objectSpread(target) {
   for (var i = 1; i < arguments.length; i++) {
     var source = arguments[i] != null ? arguments[i] : {};
+
     if (i % 2) {
       ownKeys(Object(source), true).forEach(function(key) {
         _defineProperty(target, key, source[key]);
@@ -41,6 +44,7 @@ function _objectSpread(target) {
       });
     }
   }
+
   return target;
 }
 
@@ -55,6 +59,7 @@ function _defineProperty(obj, key, value) {
   } else {
     obj[key] = value;
   }
+
   return obj;
 }
 

package/src/Consumer/types.flow.js

@@ -7,7 +7,4 @@
  *
  * @format
  */
-// NOTE: The linter and formatter are fighting over the following lint rule.
-
-/* eslint-disable flowtype/object-type-delimiter */
 "use strict";

package/src/Consumer/types.flow.js.flow

@@ -8,9 +8,6 @@
  * @format
  */
 
-// NOTE: The linter and formatter are fighting over the following lint rule.
-/* eslint-disable flowtype/object-type-delimiter */
-
 'use strict';
 
 import type {IterationOrder, LookupBias} from './constants';

package/src/Generator.js

@@ -11,6 +11,7 @@
 
 function ownKeys(object, enumerableOnly) {
   var keys = Object.keys(object);
+
   if (Object.getOwnPropertySymbols) {
     var symbols = Object.getOwnPropertySymbols(object);
     if (enumerableOnly)
@@ -19,12 +20,14 @@ function ownKeys(object, enumerableOnly) {
       });
     keys.push.apply(keys, symbols);
   }
+
   return keys;
 }
 
 function _objectSpread(target) {
   for (var i = 1; i < arguments.length; i++) {
     var source = arguments[i] != null ? arguments[i] : {};
+
     if (i % 2) {
       ownKeys(Object(source), true).forEach(function(key) {
         _defineProperty(target, key, source[key]);
@@ -41,6 +44,7 @@ function _objectSpread(target) {
       });
     }
   }
+
   return target;
 }
 
@@ -55,11 +59,11 @@ function _defineProperty(obj, key, value) {
   } else {
     obj[key] = value;
   }
+
   return obj;
 }
 
 const B64Builder = require("./B64Builder");
-
 /**
  * Generates a source map from raw mappings.
  *
@@ -71,6 +75,7 @@ const B64Builder = require("./B64Builder");
  *
  * Mappings have to be passed in the order appearance in the generated source.
  */
+
 class Generator {
   constructor() {
     this.builder = new B64Builder();

package/src/composeSourceMaps.js

@@ -14,9 +14,8 @@ const Consumer = require("./Consumer"); // TODO(t67648443): Bypass the `sort-req
 Consumer;
 
 const _require = require("source-map"),
-  SourceMapGenerator = _require.SourceMapGenerator;
+  SourceMapGenerator = _require.SourceMapGenerator; // Originally based on https://github.com/jakobwesthoff/source-map-merger
 
-// 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");

package/src/generateFunctionMap.js

@@ -11,8 +11,14 @@
 
 var _traverse = _interopRequireDefault(require("@babel/traverse"));
 
+var _types = require("@babel/types");
+
 function _interopRequireDefault(obj) {
-  return obj && obj.__esModule ? obj : { default: obj };
+  return obj && obj.__esModule
+    ? obj
+    : {
+        default: obj
+      };
 }
 
 function _slicedToArray(arr, i) {
@@ -37,6 +43,7 @@ function _iterableToArrayLimit(arr, i) {
   var _n = true;
   var _d = false;
   var _e = undefined;
+
   try {
     for (
       var _i = arr[Symbol.iterator](), _s;
@@ -44,6 +51,7 @@ function _iterableToArrayLimit(arr, i) {
       _n = true
     ) {
       _arr.push(_s.value);
+
       if (i && _arr.length === i) break;
     }
   } catch (err) {
@@ -56,6 +64,7 @@ function _iterableToArrayLimit(arr, i) {
       if (_d) throw _e;
     }
   }
+
   return _arr;
 }
 
@@ -99,7 +108,9 @@ function _arrayWithoutHoles(arr) {
 
 function _arrayLikeToArray(arr, len) {
   if (len == null || len > arr.length) len = arr.length;
+
   for (var i = 0, arr2 = new Array(len); i < len; i++) arr2[i] = arr[i];
+
   return arr2;
 }
 
@@ -107,8 +118,9 @@ const B64Builder = require("./B64Builder");
 
 const fsPath = require("path");
 
-const t = require("@babel/types");
+const nullthrows = require("nullthrows");
 
+const t = require("@babel/types");
 /**
  * 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
@@ -117,6 +129,7 @@ const t = require("@babel/types");
  * 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));
@@ -193,22 +206,25 @@ function forEachMapping(ast, context, pushMapping) {
   const basename = context.filename
     ? fsPath.basename(context.filename).replace(/\..+$/, "")
     : null;
-  (0, _traverse.default)(ast, {
-    "Function|Program|Class": {
-      enter(path) {
-        let name = getNameForPath(path);
+  const visitor = {
+    enter(path) {
+      let name = getNameForPath(path);
 
-        if (basename) {
-          name = removeNamePrefix(name, basename);
-        }
+      if (basename) {
+        name = removeNamePrefix(name, basename);
+      }
 
-        pushFrame(name, path.node.loc);
-      },
+      pushFrame(name, nullthrows(path.node.loc));
+    },
 
-      exit(path) {
-        popFrame();
-      }
+    exit(path) {
+      popFrame();
     }
+  };
+  (0, _traverse.default)(ast, {
+    Function: visitor,
+    Program: visitor,
+    Class: visitor
   });
 }
 
@@ -224,20 +240,22 @@ function getNameForPath(path) {
     parent = path.parent,
     parentPath = path.parentPath;
 
-  if (t.isProgram(node)) {
+  if ((0, _types.isProgram)(node)) {
     return "<global>";
   }
 
-  let id = path.id; // has an `id` so we don't need to infer one
+  let _ref = path,
+    id = _ref.id; // 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 = "";
 
-  if (t.isObjectMethod(node) || t.isClassMethod(node)) {
+  if ((0, _types.isObjectMethod)(node) || (0, _types.isClassMethod)(node)) {
     id = node.key;
 
     if (node.kind !== "method" && node.kind !== "constructor") {
@@ -245,30 +263,55 @@ function getNameForPath(path) {
     }
 
     propertyPath = path;
-  } else if (t.isObjectProperty(parent) || t.isClassProperty(parent)) {
+  } else if (
+    (0, _types.isObjectProperty)(parent) ||
+    (0, _types.isClassProperty)(parent)
+  ) {
     // { foo() {} };
     id = parent.key;
     propertyPath = parentPath;
-  } else if (t.isVariableDeclarator(parent)) {
+  } else if ((0, _types.isVariableDeclarator)(parent)) {
     // let foo = function () {};
     id = parent.id;
-  } else if (t.isAssignmentExpression(parent)) {
+  } else if ((0, _types.isAssignmentExpression)(parent)) {
     // foo = function () {};
     id = parent.left;
-  } else if (t.isJSXExpressionContainer(parent)) {
-    if (t.isJSXElement(parentPath.parentPath.node)) {
+  } else if ((0, _types.isJSXExpressionContainer)(parent)) {
+    var _parentPath$parentPat;
+
+    const grandParentNode =
+      parentPath === null || parentPath === void 0
+        ? void 0
+        : (_parentPath$parentPat = parentPath.parentPath) === null ||
+          _parentPath$parentPat === void 0
+        ? void 0
+        : _parentPath$parentPat.node;
+
+    if ((0, _types.isJSXElement)(grandParentNode)) {
       // <foo>{function () {}}</foo>
-      const openingElement = parentPath.parentPath.node.openingElement;
-      id = t.JSXMemberExpression(
-        t.JSXMemberExpression(openingElement.name, t.JSXIdentifier("props")),
-        t.JSXIdentifier("children")
+      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 (t.isJSXAttribute(parentPath.parentPath.node)) {
-      // <foo bar={function () {}} />
-      const openingElement = parentPath.parentPath.parentPath.node;
-      const prop = parentPath.parentPath.node;
-      id = t.JSXMemberExpression(
-        t.JSXMemberExpression(openingElement.name, t.JSXIdentifier("props")),
+    } else if ((0, _types.isJSXAttribute)(grandParentNode)) {
+      var _parentPath$parentPat2, _parentPath$parentPat3; // <foo bar={function () {}} />
+
+      const openingElement =
+        parentPath === null || parentPath === void 0
+          ? void 0
+          : (_parentPath$parentPat2 = parentPath.parentPath) === null ||
+            _parentPath$parentPat2 === void 0
+          ? void 0
+          : (_parentPath$parentPat3 = _parentPath$parentPat2.parentPath) ===
+              null || _parentPath$parentPat3 === void 0
+          ? void 0
+          : _parentPath$parentPat3.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
       );
     }
@@ -277,7 +320,10 @@ function getNameForPath(path) {
   let name = getNameFromId(id);
 
   if (name == null) {
-    if (t.isCallExpression(parent) || t.isNewExpression(parent)) {
+    if (
+      (0, _types.isCallExpression)(parent) ||
+      (0, _types.isNewExpression)(parent)
+    ) {
       // foo(function () {})
       const argIndex = parent.arguments.indexOf(node);
 
@@ -285,7 +331,7 @@ function getNameForPath(path) {
         const calleeName = getNameFromId(parent.callee); // var f = Object.freeze(function () {})
 
         if (CALLEES_TO_SKIP.indexOf(calleeName) !== -1) {
-          return getNameForPath(parentPath);
+          return getNameForPath(nullthrows(parentPath));
         }
 
         if (calleeName) {
@@ -294,6 +340,13 @@ function getNameForPath(path) {
       }
     }
 
+    if (
+      (0, _types.isTypeCastExpression)(parent) &&
+      parent.expression === node
+    ) {
+      return getNameForPath(nullthrows(parentPath));
+    }
+
     return ANONYMOUS_NAME;
   }
 
@@ -302,15 +355,17 @@ function getNameForPath(path) {
   }
 
   if (propertyPath) {
-    if (t.isClassBody(propertyPath.parent)) {
+    if ((0, _types.isClassBody)(propertyPath.parent)) {
+      // $FlowFixMe Disvoered when typing babel-traverse
       const className = getNameForPath(propertyPath.parentPath.parentPath);
 
       if (className !== ANONYMOUS_NAME) {
+        // $FlowFixMe Flow error uncovered by typing Babel more strictly
         const separator = propertyPath.node.static ? "." : "#";
         name = className + separator + name;
       }
-    } else if (t.isObjectExpression(propertyPath.parent)) {
-      const objectName = getNameForPath(propertyPath.parentPath);
+    } else if ((0, _types.isObjectExpression)(propertyPath.parent)) {
+      const objectName = getNameForPath(nullthrows(propertyPath.parentPath));
 
       if (objectName !== ANONYMOUS_NAME) {
         name = objectName + "." + name;
@@ -322,11 +377,13 @@ function getNameForPath(path) {
 }
 
 function isAnyMemberExpression(node) {
-  return t.isMemberExpression(node) || t.isJSXMemberExpression(node);
+  return (
+    node.type === "MemberExpression" || node.type === "JSXMemberExpression"
+  );
 }
 
 function isAnyIdentifier(node) {
-  return t.isIdentifier(node) || t.isJSXIdentifier(node);
+  return (0, _types.isIdentifier)(node) || (0, _types.isJSXIdentifier)(node);
 }
 
 function getNameFromId(id) {
@@ -356,11 +413,11 @@ function getNamePartsFromId(id) {
     return [];
   }
 
-  if (t.isCallExpression(id) || t.isNewExpression(id)) {
+  if ((0, _types.isCallExpression)(id) || (0, _types.isNewExpression)(id)) {
     return getNamePartsFromId(id.callee);
   }
 
-  if (t.isTypeCastExpression(id)) {
+  if ((0, _types.isTypeCastExpression)(id)) {
     return getNamePartsFromId(id.expression);
   }
 
@@ -368,21 +425,25 @@ function getNamePartsFromId(id) {
 
   if (isAnyIdentifier(id)) {
     name = id.name;
-  } else if (t.isNullLiteral(id)) {
+  } else if ((0, _types.isNullLiteral)(id)) {
     name = "null";
-  } else if (t.isRegExpLiteral(id)) {
-    name = `_${id.pattern}_${id.flags}`;
-  } else if (t.isTemplateLiteral(id)) {
+  } else if ((0, _types.isRegExpLiteral)(id)) {
+    var _id$flags;
+
+    name = `_${id.pattern}_${
+      (_id$flags = id.flags) !== null && _id$flags !== void 0 ? _id$flags : ""
+    }`;
+  } else if ((0, _types.isTemplateLiteral)(id)) {
     name = id.quasis.map(quasi => quasi.value.raw).join("");
-  } else if (t.isLiteral(id) && id.value != null) {
-    name = id.value + "";
+  } else if ((0, _types.isLiteral)(id) && id.value != null) {
+    name = String(id.value);
   }
 
   if (name != null) {
     return [t.toBindingIdentifierName(name)];
   }
 
-  if (t.isImport(id)) {
+  if ((0, _types.isImport)(id)) {
     name = "import";
   }
 
@@ -394,7 +455,7 @@ function getNamePartsFromId(id) {
     if (
       isAnyIdentifier(id.object) &&
       id.object.name === "Symbol" &&
-      (isAnyIdentifier(id.property) || t.isLiteral(id.property))
+      (isAnyIdentifier(id.property) || (0, _types.isLiteral)(id.property))
     ) {
       const propertyName = getNameFromId(id.property);
 
@@ -436,9 +497,9 @@ function removeNamePrefix(name, namePrefix) {
 
   const shortenedName = name.substr(namePrefix.length);
 
-  const _ref = shortenedName.match(DELIMITER_START_RE) || [],
-    _ref2 = _slicedToArray(_ref, 1),
-    delimiterMatch = _ref2[0];
+  const _ref2 = shortenedName.match(DELIMITER_START_RE) || [],
+    _ref3 = _slicedToArray(_ref2, 1),
+    delimiterMatch = _ref3[0];
 
   if (delimiterMatch) {
     return shortenedName.substr(delimiterMatch.length) || name;
@@ -488,9 +549,9 @@ class MappingEncoder {
     };
   }
 
-  push(_ref3) {
-    let name = _ref3.name,
-      start = _ref3.start;
+  push(_ref4) {
+    let name = _ref4.name,
+      start = _ref4.start;
 
     let nameIndex = this._namesMap.get(name);
 

package/src/generateFunctionMap.js.flow

@@ -13,12 +13,38 @@
 const B64Builder = require('./B64Builder');
 
 const fsPath = require('path');
+const nullthrows = require('nullthrows');
 const t = require('@babel/types');
 
 import type {FBSourceFunctionMap} from './source-map';
-import type {Ast} from '@babel/core';
 import traverse from '@babel/traverse';
-import type {Path} from '@babel/traverse';
+import type {NodePath} from '@babel/traverse';
+import {
+  isProgram,
+  isIdentifier,
+  isJSXIdentifier,
+  isCallExpression,
+  isNewExpression,
+  isTypeCastExpression,
+  isRegExpLiteral,
+  isTemplateLiteral,
+  isLiteral,
+  isObjectMethod,
+  isClassMethod,
+  isObjectProperty,
+  isClassProperty,
+  isVariableDeclarator,
+  isAssignmentExpression,
+  isJSXExpressionContainer,
+  isJSXElement,
+  isJSXAttribute,
+  isNullLiteral,
+  isImport,
+  isClassBody,
+  isObjectExpression,
+} from '@babel/types';
+import type {Node} from '@babel/types';
+
 type Position = {
   line: number,
   column: number,
@@ -39,7 +65,10 @@ type Context = {filename?: string, ...};
  * The output is encoded for use in a source map. For details about the format,
  * see MappingEncoder below.
  */
-function generateFunctionMap(ast: Ast, context?: Context): FBSourceFunctionMap {
+function generateFunctionMap(
+  ast: BabelNode,
+  context?: Context,
+): FBSourceFunctionMap {
   const encoder = new MappingEncoder();
   forEachMapping(ast, context, mapping => encoder.push(mapping));
   return encoder.getResult();
@@ -52,7 +81,7 @@ function generateFunctionMap(ast: Ast, context?: Context): FBSourceFunctionMap {
  * Lines are 1-based and columns are 0-based.
  */
 function generateFunctionMappingsArray(
-  ast: Ast,
+  ast: BabelNode,
   context?: Context,
 ): $ReadOnlyArray<RangeMapping> {
   const mappings = [];
@@ -67,7 +96,7 @@ function generateFunctionMappingsArray(
  * mappings, one at a time.
  */
 function forEachMapping(
-  ast: Ast,
+  ast: BabelNode,
   context: ?Context,
   pushMapping: RangeMapping => void,
 ) {
@@ -89,7 +118,7 @@ function forEachMapping(
     tailPos = pos;
   }
 
-  function pushFrame(name, loc) {
+  function pushFrame(name: string, loc: BabelNodeSourceLocation) {
     advanceToPos(loc.start);
     nameStack.unshift({name, loc});
   }
@@ -111,19 +140,25 @@ function forEachMapping(
     ? fsPath.basename(context.filename).replace(/\..+$/, '')
     : null;
 
-  traverse(ast, {
-    'Function|Program|Class': {
-      enter(path) {
-        let name = getNameForPath(path);
-        if (basename) {
-          name = removeNamePrefix(name, basename);
-        }
-        pushFrame(name, path.node.loc);
-      },
-      exit(path) {
-        popFrame();
-      },
+  const visitor = {
+    enter(path) {
+      let name = getNameForPath(path);
+      if (basename) {
+        name = removeNamePrefix(name, basename);
+      }
+
+      pushFrame(name, nullthrows(path.node.loc));
     },
+
+    exit(path): void {
+      popFrame();
+    },
+  };
+
+  traverse(ast, {
+    Function: visitor,
+    Program: visitor,
+    Class: visitor,
   });
 }
 
@@ -134,48 +169,54 @@ const CALLEES_TO_SKIP = ['Object.freeze'];
  * Derive a contextual name for the given AST node (Function, Program, Class or
  * ObjectExpression).
  */
-function getNameForPath(path: Path): string {
+function getNameForPath(path: NodePath<>): string {
   const {node, parent, parentPath} = path;
-  if (t.isProgram(node)) {
+  if (isProgram(node)) {
     return '<global>';
   }
-  let {id} = path;
+
+  let {id} = (path: any);
   // 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 = '';
-  if (t.isObjectMethod(node) || t.isClassMethod(node)) {
+  if (isObjectMethod(node) || isClassMethod(node)) {
     id = node.key;
     if (node.kind !== 'method' && node.kind !== 'constructor') {
       kind = node.kind;
     }
     propertyPath = path;
-  } else if (t.isObjectProperty(parent) || t.isClassProperty(parent)) {
+  } else if (isObjectProperty(parent) || isClassProperty(parent)) {
     // { foo() {} };
     id = parent.key;
     propertyPath = parentPath;
-  } else if (t.isVariableDeclarator(parent)) {
+  } else if (isVariableDeclarator(parent)) {
     // let foo = function () {};
     id = parent.id;
-  } else if (t.isAssignmentExpression(parent)) {
+  } else if (isAssignmentExpression(parent)) {
     // foo = function () {};
     id = parent.left;
-  } else if (t.isJSXExpressionContainer(parent)) {
-    if (t.isJSXElement(parentPath.parentPath.node)) {
+  } else if (isJSXExpressionContainer(parent)) {
+    const grandParentNode = parentPath?.parentPath?.node;
+    if (isJSXElement(grandParentNode)) {
       // <foo>{function () {}}</foo>
-      const openingElement = parentPath.parentPath.node.openingElement;
-      id = t.JSXMemberExpression(
-        t.JSXMemberExpression(openingElement.name, t.JSXIdentifier('props')),
-        t.JSXIdentifier('children'),
+      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 (t.isJSXAttribute(parentPath.parentPath.node)) {
+    } else if (isJSXAttribute(grandParentNode)) {
       // <foo bar={function () {}} />
-      const openingElement = parentPath.parentPath.parentPath.node;
-      const prop = parentPath.parentPath.node;
-      id = t.JSXMemberExpression(
-        t.JSXMemberExpression(openingElement.name, t.JSXIdentifier('props')),
+      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,
       );
     }
@@ -184,20 +225,23 @@ function getNameForPath(path: Path): string {
   let name = getNameFromId(id);
 
   if (name == null) {
-    if (t.isCallExpression(parent) || t.isNewExpression(parent)) {
+    if (isCallExpression(parent) || isNewExpression(parent)) {
       // foo(function () {})
       const argIndex = parent.arguments.indexOf(node);
       if (argIndex !== -1) {
         const calleeName = getNameFromId(parent.callee);
         // var f = Object.freeze(function () {})
         if (CALLEES_TO_SKIP.indexOf(calleeName) !== -1) {
-          return getNameForPath(parentPath);
+          return getNameForPath(nullthrows(parentPath));
         }
         if (calleeName) {
           return `${calleeName}$argument_${argIndex}`;
         }
       }
     }
+    if (isTypeCastExpression(parent) && parent.expression === node) {
+      return getNameForPath(nullthrows(parentPath));
+    }
     return ANONYMOUS_NAME;
   }
 
@@ -206,14 +250,16 @@ function getNameForPath(path: Path): string {
   }
 
   if (propertyPath) {
-    if (t.isClassBody(propertyPath.parent)) {
+    if (isClassBody(propertyPath.parent)) {
+      // $FlowFixMe Disvoered when typing babel-traverse
       const className = getNameForPath(propertyPath.parentPath.parentPath);
       if (className !== ANONYMOUS_NAME) {
+        // $FlowFixMe Flow error uncovered by typing Babel more strictly
         const separator = propertyPath.node.static ? '.' : '#';
         name = className + separator + name;
       }
-    } else if (t.isObjectExpression(propertyPath.parent)) {
-      const objectName = getNameForPath(propertyPath.parentPath);
+    } else if (isObjectExpression(propertyPath.parent)) {
+      const objectName = getNameForPath(nullthrows(propertyPath.parentPath));
       if (objectName !== ANONYMOUS_NAME) {
         name = objectName + '.' + name;
       }
@@ -223,15 +269,17 @@ function getNameForPath(path: Path): string {
   return name;
 }
 
-function isAnyMemberExpression(node: Ast): boolean {
-  return t.isMemberExpression(node) || t.isJSXMemberExpression(node);
+function isAnyMemberExpression(node: Node): boolean %checks {
+  return (
+    node.type === 'MemberExpression' || node.type === 'JSXMemberExpression'
+  );
 }
 
-function isAnyIdentifier(node: Ast): boolean {
-  return t.isIdentifier(node) || t.isJSXIdentifier(node);
+function isAnyIdentifier(node: Node): boolean %checks {
+  return isIdentifier(node) || isJSXIdentifier(node);
 }
 
-function getNameFromId(id: Ast): ?string {
+function getNameFromId(id: Node): ?string {
   const parts = getNamePartsFromId(id);
 
   if (!parts.length) {
@@ -251,16 +299,16 @@ function getNameFromId(id: Ast): ?string {
   return parts.join('.');
 }
 
-function getNamePartsFromId(id: Ast): $ReadOnlyArray<string> {
+function getNamePartsFromId(id: Node): $ReadOnlyArray<string> {
   if (!id) {
     return [];
   }
 
-  if (t.isCallExpression(id) || t.isNewExpression(id)) {
+  if (isCallExpression(id) || isNewExpression(id)) {
     return getNamePartsFromId(id.callee);
   }
 
-  if (t.isTypeCastExpression(id)) {
+  if (isTypeCastExpression(id)) {
     return getNamePartsFromId(id.expression);
   }
 
@@ -268,21 +316,21 @@ function getNamePartsFromId(id: Ast): $ReadOnlyArray<string> {
 
   if (isAnyIdentifier(id)) {
     name = id.name;
-  } else if (t.isNullLiteral(id)) {
+  } else if (isNullLiteral(id)) {
     name = 'null';
-  } else if (t.isRegExpLiteral(id)) {
-    name = `_${id.pattern}_${id.flags}`;
-  } else if (t.isTemplateLiteral(id)) {
+  } else if (isRegExpLiteral(id)) {
+    name = `_${id.pattern}_${id.flags ?? ''}`;
+  } else if (isTemplateLiteral(id)) {
     name = id.quasis.map(quasi => quasi.value.raw).join('');
-  } else if (t.isLiteral(id) && id.value != null) {
-    name = id.value + '';
+  } else if (isLiteral(id) && id.value != null) {
+    name = String(id.value);
   }
 
   if (name != null) {
     return [t.toBindingIdentifierName(name)];
   }
 
-  if (t.isImport(id)) {
+  if (isImport(id)) {
     name = 'import';
   }
 
@@ -294,7 +342,7 @@ function getNamePartsFromId(id: Ast): $ReadOnlyArray<string> {
     if (
       isAnyIdentifier(id.object) &&
       id.object.name === 'Symbol' &&
-      (isAnyIdentifier(id.property) || t.isLiteral(id.property))
+      (isAnyIdentifier(id.property) || isLiteral(id.property))
     ) {
       const propertyName = getNameFromId(id.property);
       if (propertyName) {

package/src/source-map.js

@@ -17,6 +17,7 @@ function asyncGeneratorStep(gen, resolve, reject, _next, _throw, key, arg) {
     reject(error);
     return;
   }
+
   if (info.done) {
     resolve(value);
   } else {
@@ -30,12 +31,15 @@ function _asyncToGenerator(fn) {
       args = arguments;
     return new Promise(function(resolve, reject) {
       var gen = fn.apply(self, args);
+
       function _next(value) {
         asyncGeneratorStep(gen, resolve, reject, _next, _throw, "next", value);
       }
+
       function _throw(err) {
         asyncGeneratorStep(gen, resolve, reject, _next, _throw, "throw", err);
       }
+
       _next(undefined);
     });
   };
@@ -45,9 +49,8 @@ const Consumer = require("./Consumer");
 
 const Generator = require("./Generator");
 
-const SourceMap = require("source-map");
+const SourceMap = require("source-map"); // We need to export this for `metro-symbolicate`
 
-// We need to export this for `metro-symbolicate`
 const normalizeSourcePath = require("./Consumer/normalizeSourcePath");
 
 const composeSourceMaps = require("./composeSourceMaps");