hermes-transform 0.36.1 → 0.37.0

package/dist/transform/comments/prettier/language-js/utils.js

@@ -1,45 +1,16 @@
-/**
- * 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';
-/**
- * @param {Comment} comment
- * @returns {boolean}
- */
 
 function isBlockComment(comment) {
-  return comment.type === 'Block' || comment.type === 'CommentBlock' || // `meriyah`
-  comment.type === 'MultiLine';
+  return comment.type === 'Block' || comment.type === 'CommentBlock' || comment.type === 'MultiLine';
 }
-/**
- * @param {Comment} comment
- * @returns {boolean}
- */
-
 
 function isLineComment(comment) {
-  return comment.type === 'Line' || comment.type === 'CommentLine' || // `meriyah` has `SingleLine`, `HashbangComment`, `HTMLOpen`, and `HTMLClose`
-  comment.type === 'SingleLine' || comment.type === 'HashbangComment' || comment.type === 'HTMLOpen' || comment.type === 'HTMLClose';
+  return comment.type === 'Line' || comment.type === 'CommentLine' || comment.type === 'SingleLine' || comment.type === 'HashbangComment' || comment.type === 'HTMLOpen' || comment.type === 'HTMLClose';
 }
-/**
- * @param {Node} node
- * @returns {boolean}
- */
-
 
 function isCallExpression(node) {
   return node && (node.type === 'CallExpression' || node.type === 'OptionalCallExpression');
 }
-/**
- * @param {Node} node
- * @returns {boolean}
- */
-
 
 function isMemberExpression(node) {
   return node && (node.type === 'MemberExpression' || node.type === 'OptionalMemberExpression');
@@ -56,8 +27,7 @@ function getFunctionParameters(node) {
 
   if (node.this) {
     parameters.push(node.this);
-  } // `params` vs `parameters` - see https://github.com/babel/babel/issues/9231
-
+  }
 
   if (Array.isArray(node.parameters)) {
     parameters.push(...node.parameters);

package/dist/transform/comments/prettier/main/comments.js

@@ -1,11 +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
- */
 'use strict';
 
 const assert = require('assert');
@@ -32,9 +24,6 @@ function getSortedChildNodes(node, options, resultArray) {
 
   if (resultArray) {
     if (printer.canAttachComment && printer.canAttachComment(node)) {
-      // This reverse insertion sort almost always takes constant
-      // time because we almost always (maybe always?) append the
-      // nodes in order anyway.
       let i;
 
       for (i = resultArray.length - 1; i >= 0; --i) {
@@ -66,10 +55,7 @@ function getSortedChildNodes(node, options, resultArray) {
   }
 
   return resultArray;
-} // As efficiently as possible, decorate the comment object with
-// .precedingNode, .enclosingNode, and/or .followingNode properties, at
-// least one of which is guaranteed to be defined.
-
+}
 
 function decorateComment(node, comment, options, enclosingNode) {
   const {
@@ -80,8 +66,7 @@ function decorateComment(node, comment, options, enclosingNode) {
   const commentEnd = locEnd(comment);
   const childNodes = getSortedChildNodes(node, options);
   let precedingNode;
-  let followingNode; // Time to dust off the old binary search robes and wizard hat.
-
+  let followingNode;
   let left = 0;
   let right = childNodes.length;
 
@@ -89,39 +74,26 @@ function decorateComment(node, comment, options, enclosingNode) {
     const middle = left + right >> 1;
     const child = childNodes[middle];
     const start = locStart(child);
-    const end = locEnd(child); // The comment is completely contained by this child node.
+    const end = locEnd(child);
 
     if (start <= commentStart && commentEnd <= end) {
-      // Abandon the binary search at this level.
       return decorateComment(child, comment, options, child);
     }
 
     if (end <= commentStart) {
-      // This child node falls completely before the comment.
-      // Because we will never consider this node or any nodes
-      // before it again, this node must be the closest preceding
-      // node we have encountered so far.
       precedingNode = child;
       left = middle + 1;
       continue;
     }
 
     if (commentEnd <= start) {
-      // This child node falls completely after the comment.
-      // Because we will never consider this node or any nodes after
-      // it again, this node must be the closest following node we
-      // have encountered so far.
       followingNode = child;
       right = middle;
       continue;
     }
-    /* istanbul ignore next */
-
 
     throw new Error('Comment location overlaps with node location');
-  } // We don't want comments inside of different expressions inside of the same
-  // template literal to move to another expression.
-
+  }
 
   if (enclosingNode && enclosingNode.type === 'TemplateLiteral') {
     const {
@@ -159,8 +131,7 @@ function attach(comments, ast, text, options) {
     printer: {
       handleComments = {}
     }
-  } = options; // TODO: Make this as default behavior
-
+  } = options;
   const {
     avoidAstMutation,
     ownLine: handleOwnLineComment = returnFalse,
@@ -211,51 +182,33 @@ function attach(comments, ast, text, options) {
     }
 
     if (isOwnLineComment(text, options, decoratedComments, index)) {
-      comment.placement = 'ownLine'; // If a comment exists on its own line, prefer a leading comment.
-      // We also need to check if it's the first line of the file.
+      comment.placement = 'ownLine';
 
-      if (handleOwnLineComment(...args)) {// We're good
-      } else if (followingNode) {
-        // Always a leading comment.
+      if (handleOwnLineComment(...args)) {} else if (followingNode) {
         addLeadingComment(followingNode, comment);
       } else if (precedingNode) {
         addTrailingComment(precedingNode, comment);
       } else if (enclosingNode) {
         addDanglingComment(enclosingNode, comment);
       } else {
-        // There are no nodes, let's attach it to the root of the ast
-
-        /* istanbul ignore next */
         addDanglingComment(ast, comment);
       }
     } else if (isEndOfLineComment(text, options, decoratedComments, index)) {
       comment.placement = 'endOfLine';
 
-      if (handleEndOfLineComment(...args)) {// We're good
-      } else if (precedingNode) {
-        // There is content before this comment on the same line, but
-        // none after it, so prefer a trailing comment of the previous node.
+      if (handleEndOfLineComment(...args)) {} else if (precedingNode) {
         addTrailingComment(precedingNode, comment);
       } else if (followingNode) {
         addLeadingComment(followingNode, comment);
       } else if (enclosingNode) {
         addDanglingComment(enclosingNode, comment);
       } else {
-        // There are no nodes, let's attach it to the root of the ast
-
-        /* istanbul ignore next */
         addDanglingComment(ast, comment);
       }
     } else {
       comment.placement = 'remaining';
 
-      if (handleRemainingComment(...args)) {// We're good
-      } else if (precedingNode && followingNode) {
-        // Otherwise, text exists both before and after the comment on
-        // the same line. If there is both a preceding and following
-        // node, use a tie-breaking algorithm to determine if it should
-        // be attached to the next or previous node. In the last case,
-        // simply attach the right node;
+      if (handleRemainingComment(...args)) {} else if (precedingNode && followingNode) {
         const tieCount = tiesToBreak.length;
 
         if (tieCount > 0) {
@@ -274,9 +227,6 @@ function attach(comments, ast, text, options) {
       } else if (enclosingNode) {
         addDanglingComment(enclosingNode, comment);
       } else {
-        // There are no nodes, let's attach it to the root of the ast
-
-        /* istanbul ignore next */
         addDanglingComment(ast, comment);
       }
     }
@@ -286,9 +236,6 @@ function attach(comments, ast, text, options) {
 
   if (!avoidAstMutation) {
     for (const comment of comments) {
-      // These node references were useful for breaking ties, but we
-      // don't need them anymore, and they create cycles in the AST that
-      // may lead to infinite recursion if we don't delete them here.
       delete comment.precedingNode;
       delete comment.enclosingNode;
       delete comment.followingNode;
@@ -310,7 +257,6 @@ function isOwnLineComment(text, options, decoratedComments, commentIndex) {
   let start = locStart(comment);
 
   if (precedingNode) {
-    // Find first comment on the same line
     for (let index = commentIndex - 1; index >= 0; index--) {
       const {
         comment,
@@ -342,7 +288,6 @@ function isEndOfLineComment(text, options, decoratedComments, commentIndex) {
   let end = locEnd(comment);
 
   if (followingNode) {
-    // Find last comment on the same line
     for (let index = commentIndex + 1; index < decoratedComments.length; index++) {
       const {
         comment,
@@ -373,12 +318,7 @@ function breakTies(tiesToBreak, text, options) {
     enclosingNode
   } = tiesToBreak[0];
   const gapRegExp = options.printer.getGapRegex && options.printer.getGapRegex(enclosingNode) || /^[\s(]*$/;
-  let gapEndPos = options.locStart(followingNode); // Iterate backwards through tiesToBreak, examining the gaps
-  // between the tied comments. In order to qualify as leading, a
-  // comment must be separated from followingNode by an unbroken series of
-  // gaps (or other comments). Gaps should only contain whitespace or open
-  // parentheses.
-
+  let gapEndPos = options.locStart(followingNode);
   let indexOfFirstLeadingComment;
 
   for (indexOfFirstLeadingComment = tieCount; indexOfFirstLeadingComment > 0; --indexOfFirstLeadingComment) {
@@ -394,8 +334,6 @@ function breakTies(tiesToBreak, text, options) {
     if (gapRegExp.test(gap)) {
       gapEndPos = options.locStart(comment);
     } else {
-      // The gap string contained something other than whitespace or open
-      // parentheses.
       break;
     }
   }
@@ -426,11 +364,7 @@ function findExpressionIndexForComment(quasis, comment, options) {
     if (startPos < options.locStart(quasis[i])) {
       return i - 1;
     }
-  } // We haven't found it, it probably means that some of the locations are off.
-  // Let's just return the first one.
-
-  /* istanbul ignore next */
-
+  }
 
   return 0;
 }

package/dist/transform/comments/prettier/utils/get-last.js

@@ -1,11 +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
- */
 'use strict';
 
 const getLast = arr => arr[arr.length - 1];

package/dist/transform/mutations/AddComments.js

@@ -8,15 +8,6 @@ exports.performAddCommentsMutation = performAddCommentsMutation;
 
 var _comments = require("../comments/comments");
 
-/**
- * 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
- */
 function createAddCommentsMutation(node, comments) {
   if (comments.length === 0) {
     return null;

package/dist/transform/mutations/AddComments.js.flow

@@ -15,9 +15,9 @@ import type {CommentPlacement} from '../comments/comments';
 
 import {addComment, cloneComment} from '../comments/comments';
 
-export type AddCommentsMutation = $ReadOnly<{
+export type AddCommentsMutation = Readonly<{
   type: 'addComments',
-  comments: $ReadOnlyArray<{comment: Comment, placement: CommentPlacement}>,
+  comments: ReadonlyArray<{comment: Comment, placement: CommentPlacement}>,
   node: ESNode | DetachedNode<ESNode>,
 }>;
 

package/dist/transform/mutations/CloneCommentsTo.js

@@ -8,15 +8,6 @@ exports.performCloneCommentsToMutation = performCloneCommentsToMutation;
 
 var _comments = require("../comments/comments");
 
-/**
- * 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
- */
 function createCloneCommentsToMutation(target, destination) {
   return {
     type: 'cloneCommentsTo',

package/dist/transform/mutations/CloneCommentsTo.js.flow

@@ -14,7 +14,7 @@ import type {MutationContext} from '../MutationContext';
 
 import {cloneCommentsToNewNode} from '../comments/comments';
 
-export type CloneCommentsToMutation = $ReadOnly<{
+export type CloneCommentsToMutation = Readonly<{
   type: 'cloneCommentsTo',
   target: ESNode | DetachedNode<ESNode>,
   destination: ESNode | DetachedNode<ESNode>,

package/dist/transform/mutations/InsertStatement.js

@@ -20,15 +20,6 @@ function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "functio
 
 function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
 
-/**
- * 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
- */
 function createInsertStatementMutation(side, target, nodesToInsert) {
   if (nodesToInsert.length === 0) {
     return null;
@@ -44,7 +35,7 @@ function createInsertStatementMutation(side, target, nodesToInsert) {
 
 function performInsertStatementMutation(mutationContext, mutation) {
   mutationContext.assertNotDeleted(mutation.target, `Attempted to insert ${mutation.side} a deleted ${mutation.target.type} node. This likely means that you attempted to mutate around the target after it was deleted/replaced.`);
-  const insertionParent = (0, _getStatementParent.getStatementParent)(mutation.target); // enforce that if we are inserting module declarations - they are being inserted in a valid location
+  const insertionParent = (0, _getStatementParent.getStatementParent)(mutation.target);
 
   if (!(0, _isValidModuleDeclarationParent.isValidModuleDeclarationParent)(insertionParent.parent, mutation.nodesToInsert)) {
     throw new _Errors.InvalidInsertionError(`import/export cannot be inserted into a ${insertionParent.parent.type}.`);
@@ -72,15 +63,12 @@ function performInsertStatementMutation(mutationContext, mutation) {
     return insertionParent.parent;
   }
 
-  const statementsToInsert = // $FlowExpectedError[incompatible-type] -- this is enforced by isValidModuleDeclarationParent above
-  mutation.nodesToInsert;
+  const statementsToInsert = mutation.nodesToInsert;
   const {
     parent,
     key
-  } = insertionParent; // $FlowExpectedError[prop-missing]
-
-  const statementToWrap = parent[key]; // we need to wrap this key in a BlockStatement so we can insert the new statement
-
+  } = insertionParent;
+  const statementToWrap = parent[key];
   const blockStatement = t.BlockStatement({
     body: mutation.side === 'before' ? [...statementsToInsert, statementToWrap] : [statementToWrap, ...statementsToInsert],
     parent: insertionParent.parent

package/dist/transform/mutations/InsertStatement.js.flow

@@ -18,11 +18,11 @@ import {isValidModuleDeclarationParent} from './utils/isValidModuleDeclarationPa
 import {InvalidInsertionError} from '../Errors';
 import * as t from '../../generated/node-types';
 
-export type InsertStatementMutation = $ReadOnly<{
+export type InsertStatementMutation = Readonly<{
   type: 'insertStatement',
   side: 'before' | 'after',
   target: ModuleDeclaration | Statement,
-  nodesToInsert: $ReadOnlyArray<DetachedNode<Statement | ModuleDeclaration>>,
+  nodesToInsert: ReadonlyArray<DetachedNode<Statement | ModuleDeclaration>>,
 }>;
 
 export function createInsertStatementMutation(
@@ -69,7 +69,7 @@ export function performInsertStatementMutation(
 
   if (insertionParent.type === 'array') {
     const parent: interface {
-      [string]: $ReadOnlyArray<DetachedNode<Statement | ModuleDeclaration>>,
+      [string]: ReadonlyArray<DetachedNode<Statement | ModuleDeclaration>>,
     } = insertionParent.parent;
     switch (mutation.side) {
       case 'before': {
@@ -96,7 +96,7 @@ export function performInsertStatementMutation(
 
   const statementsToInsert =
     // $FlowExpectedError[incompatible-type] -- this is enforced by isValidModuleDeclarationParent above
-    (mutation.nodesToInsert: $ReadOnlyArray<DetachedNode<Statement>>);
+    mutation.nodesToInsert as ReadonlyArray<DetachedNode<Statement>>;
 
   const {parent, key} = insertionParent;
 
@@ -111,8 +111,9 @@ export function performInsertStatementMutation(
     parent: insertionParent.parent,
   });
 
-  (insertionParent.parent: interface {[string]: mixed})[insertionParent.key] =
-    blockStatement;
+  (insertionParent.parent as interface {[string]: unknown})[
+    insertionParent.key
+  ] = blockStatement;
   statementToWrap.parent = blockStatement;
 
   return insertionParent.parent;

package/dist/transform/mutations/ModifyNodeInPlace.js

@@ -8,15 +8,6 @@ exports.performModifyNodeInPlaceMutation = performModifyNodeInPlaceMutation;
 
 var _hermesParser = require("hermes-parser");
 
-/**
- * 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
- */
 function createModifyNodeInPlaceMutation(target, newProps) {
   return {
     type: 'modifyNodeInPlace',
@@ -32,13 +23,11 @@ function performModifyNodeInPlaceMutation(mutationContext, mutation) {
   } = mutation;
 
   for (const [key, newPropValue] of Object.entries(newProps)) {
-    const prevPropValue = target[key]; // If the value did not change, skip.
-    // $FlowFixMe[invalid-compare]
+    const prevPropValue = target[key];
 
     if (prevPropValue === newPropValue) {
       continue;
-    } // Mark removed nodes as deleted
-
+    }
 
     if ((0, _hermesParser.isNode)(prevPropValue)) {
       mutationContext.markDeletion(prevPropValue);
@@ -48,11 +37,9 @@ function performModifyNodeInPlaceMutation(mutationContext, mutation) {
           mutationContext.markDeletion(prevPropValueItem);
         }
       }
-    } // Mark node property as mutated.
-
-
-    mutationContext.markMutation(target, key); // Assign new property.
+    }
 
+    mutationContext.markMutation(target, key);
     target[key] = newPropValue;
   }
 

package/dist/transform/mutations/ModifyNodeInPlace.js.flow

@@ -13,10 +13,10 @@ import type {MutationContext} from '../MutationContext';
 
 import {isNode} from 'hermes-parser';
 
-export type ModifyNodeInPlaceMutation = $ReadOnly<{
+export type ModifyNodeInPlaceMutation = Readonly<{
   type: 'modifyNodeInPlace',
   target: ESNode,
-  newProps: $ReadOnly<{...}>,
+  newProps: Readonly<{...}>,
 }>;
 
 export function createModifyNodeInPlaceMutation(

package/dist/transform/mutations/RemoveComment.js

@@ -10,15 +10,6 @@ var _hermesParser = require("hermes-parser");
 
 var _comments = require("../comments/comments");
 
-/**
- * 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
- */
 function createRemoveCommentMutation(comment) {
   return {
     type: 'removeComment',
@@ -55,7 +46,6 @@ function performRemoveCommentMutations(ast, mutations) {
       (0, _comments.setCommentsOnNode)(node, Array.from(nodeCommentsSet));
 
       if (commentsToRemove.size === 0) {
-        // no more comments to process - so we can exit traversal
         throw _hermesParser.SimpleTraverserBreak;
       }
     },

package/dist/transform/mutations/RemoveComment.js.flow

@@ -13,7 +13,7 @@ import type {Comment, Program} from 'hermes-estree';
 import {SimpleTraverser, SimpleTraverserBreak} from 'hermes-parser';
 import {getCommentsForNode, setCommentsOnNode} from '../comments/comments';
 
-export type RemoveCommentMutation = $ReadOnly<{
+export type RemoveCommentMutation = Readonly<{
   type: 'removeComment',
   comment: Comment,
 }>;
@@ -29,7 +29,7 @@ export function createRemoveCommentMutation(
 
 export function performRemoveCommentMutations(
   ast: Program,
-  mutations: $ReadOnlyArray<RemoveCommentMutation>,
+  mutations: ReadonlyArray<RemoveCommentMutation>,
 ): void {
   if (mutations.length === 0) {
     return;
@@ -65,8 +65,8 @@ export function performRemoveCommentMutations(
 }
 
 function intersectSets<T>(
-  first: $ReadOnlySet<T>,
-  other: $ReadOnlySet<T>,
+  first: ReadonlySet<T>,
+  other: ReadonlySet<T>,
 ): Set<T> {
   const ret = new Set<T>();
   for (const value of first) {

package/dist/transform/mutations/RemoveNode.js

@@ -10,15 +10,6 @@ var _hermesParser = require("hermes-parser");
 
 var _Errors = require("../Errors");
 
-/**
- * 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
- */
 function createRemoveNodeMutation(node) {
   return {
     type: 'removeNode',
@@ -46,7 +37,6 @@ function getRemovalParent(node) {
     }
 
     switch (node.type) {
-      // ClassMember
       case 'PropertyDefinition':
       case 'MethodDefinition':
         assertParent('ClassBody');
@@ -58,7 +48,6 @@ function getRemovalParent(node) {
       case 'EnumStringMember':
         assertParent(VALID_ENUM_MEMBER_PARENTS);
         return 'members';
-      // Components
 
       case 'ComponentParameter':
         assertParent('ComponentDeclaration');
@@ -67,7 +56,6 @@ function getRemovalParent(node) {
       case 'ComponentTypeParameter':
         assertParent(VALID_COMPONENT_TYPE_PARAMETER_PARENTS);
         return 'params';
-      // FunctionParameter
 
       case 'AssignmentPattern':
       case 'ArrayPattern':
@@ -107,7 +95,6 @@ function getRemovalParent(node) {
       case 'Property':
         assertParent(VALID_PROPERTY_PARENTS);
         return 'properties';
-      // Identifier can be the child of a number of usecases
 
       case 'Identifier':
         switch (node.parent.type) {
@@ -124,8 +111,6 @@ function getRemovalParent(node) {
             throw new _Errors.InvalidRemovalError(getErrorMessage(['ArrowFunctionExpression', 'FunctionDeclaration', 'FunctionExpression', 'ArrayExpression', 'ArrayPattern']));
         }
 
-      // RestElement can be the child of a number of usecases
-
       case 'RestElement':
         switch (node.parent.type) {
           case 'ArrowFunctionExpression':
@@ -139,8 +124,6 @@ function getRemovalParent(node) {
 
           case 'ObjectPattern':
             return 'properties';
-          // $FlowFixMe[incompatible-type]
-          // $FlowFixMe[invalid-compare]
 
           case 'OptionalCallExpression':
           case 'CallExpression':
@@ -151,8 +134,6 @@ function getRemovalParent(node) {
             throw new _Errors.InvalidRemovalError(getErrorMessage(['ArrowFunctionExpression', 'FunctionDeclaration', 'FunctionExpression', 'ComponentDeclaration', 'ArrayPattern', 'ObjectPattern', 'CallExpression', 'OptionalCallExpression', 'NewExpression']));
         }
 
-      // SpreadElement can be the child of a number of usecases
-
       case 'SpreadElement':
         switch (node.parent.type) {
           case 'ArrayExpression':
@@ -160,8 +141,6 @@ function getRemovalParent(node) {
 
           case 'ObjectExpression':
             return 'properties';
-          // $FlowFixMe[incompatible-type]
-          // $FlowFixMe[invalid-compare]
 
           case 'OptionalCallExpression':
           case 'CallExpression':
@@ -178,9 +157,8 @@ function getRemovalParent(node) {
   })();
 
   const targetIndex = (() => {
-    // $FlowExpectedError[prop-missing]
     const arr = node.parent[key];
-    const idx = arr.indexOf(node); // $FlowFixMe[invalid-compare]
+    const idx = arr.indexOf(node);
 
     if (idx === -1) {
       throw new _Errors.InvalidRemovalError(`Could not find target in array of \`${node.parent.type}.${key}\`.`);