hermes-transform 0.36.0 → 0.37.0

package/dist/src/generated/special-case-node-types/DeclareExportDeclaration.js

@@ -9,19 +9,9 @@ exports.DeclareExportDefaultDeclaration = DeclareExportDefaultDeclaration;
 
 var _detachedNode = require("../../detachedNode");
 
-/**
- * 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 DeclareExportDefaultDeclaration(props) {
   const node = (0, _detachedNode.detachedProps)(props.parent, {
     type: 'DeclareExportDeclaration',
-    // $FlowFixMe[incompatible-type]
     declaration: (0, _detachedNode.asDetachedNode)(props.declaration),
     specifiers: [],
     source: null,
@@ -34,7 +24,6 @@ function DeclareExportDefaultDeclaration(props) {
 function DeclareExportDeclarationNamedWithDeclaration(props) {
   const node = (0, _detachedNode.detachedProps)(props.parent, {
     type: 'DeclareExportDeclaration',
-    // $FlowFixMe[incompatible-type]
     declaration: (0, _detachedNode.asDetachedNode)(props.declaration),
     specifiers: [],
     source: null,
@@ -49,7 +38,6 @@ function DeclareExportDeclarationNamedWithSpecifiers(props) {
     type: 'DeclareExportDeclaration',
     declaration: null,
     specifiers: props.specifiers.map(n => (0, _detachedNode.asDetachedNode)(n)),
-    // $FlowFixMe[incompatible-type]
     source: (0, _detachedNode.asDetachedNode)(props.source),
     default: false
   });

package/dist/src/generated/special-case-node-types/DeclareHook.js

@@ -7,15 +7,6 @@ exports.DeclareHook = DeclareHook;
 
 var _detachedNode = require("../../detachedNode");
 
-/**
- * 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 DeclareHook(props) {
   const node = (0, _detachedNode.detachedProps)(props.parent, {
     type: 'DeclareHook',

package/dist/src/generated/special-case-node-types/ExportNamedDeclaration.js

@@ -8,19 +8,9 @@ exports.ExportNamedDeclarationWithSpecifiers = ExportNamedDeclarationWithSpecifi
 
 var _detachedNode = require("../../detachedNode");
 
-/**
- * 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 ExportNamedDeclarationWithDeclaration(props) {
   const node = (0, _detachedNode.detachedProps)(props.parent, {
     type: 'ExportNamedDeclaration',
-    // $FlowFixMe[incompatible-type]
     declaration: (0, _detachedNode.asDetachedNode)(props.declaration),
     specifiers: [],
     source: null,
@@ -35,7 +25,6 @@ function ExportNamedDeclarationWithSpecifiers(props) {
     type: 'ExportNamedDeclaration',
     declaration: null,
     specifiers: props.specifiers.map(n => (0, _detachedNode.asDetachedNode)(n)),
-    // $FlowFixMe[incompatible-type]
     source: (0, _detachedNode.asDetachedNode)(props.source),
     exportKind: props.exportKind
   });

package/dist/src/generated/special-case-node-types/Literal.js

@@ -12,15 +12,6 @@ exports.StringLiteral = StringLiteral;
 
 var _detachedNode = require("../../detachedNode");
 
-/**
- * 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 BigIntLiteral(props) {
   var _props$raw;
 
@@ -57,9 +48,7 @@ function NullLiteral(props = { ...null
     value: null,
     raw: 'null'
   });
-} // pattern/flags are on a subobject in the estree spec, but are flat on the hermes types
-// also the value is supposed to be a RegExp instance
-
+}
 
 function RegExpLiteral(props) {
   const value = new RegExp(props.pattern, props.flags);

package/dist/src/generated/special-case-node-types/ObjectTypeProperty.js

@@ -9,15 +9,6 @@ exports.ObjectTypePropertySignature = ObjectTypePropertySignature;
 
 var _detachedNode = require("../../detachedNode");
 
-/**
- * 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 ObjectTypeMethodSignature(props) {
   var _props$proto, _props$static;
 
@@ -48,7 +39,6 @@ function ObjectTypePropertySignature(props) {
     proto: (_props$proto2 = props.proto) != null ? _props$proto2 : false,
     static: (_props$static2 = props.static) != null ? _props$static2 : false,
     value: (0, _detachedNode.asDetachedNode)(props.value),
-    // $FlowFixMe[incompatible-type]
     variance: (0, _detachedNode.asDetachedNode)(props.variance)
   });
   (0, _detachedNode.setParentPointersInDirectChildren)(node);

package/dist/src/generated/special-case-node-types/Property.js

@@ -14,15 +14,6 @@ exports.ObjectPropertyWithShorthandStaticName = ObjectPropertyWithShorthandStati
 
 var _detachedNode = require("../../detachedNode");
 
-/**
- * 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 DestructuringObjectProperty(props) {
   const node = (0, _detachedNode.detachedProps)(props.parent, {
     type: 'Property',

package/dist/src/generated/special-case-node-types/misc.js

@@ -14,28 +14,15 @@ exports.TemplateElement = TemplateElement;
 
 var _detachedNode = require("../../detachedNode");
 
-/**
- * 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 ArrowFunctionExpression(props) {
   const node = (0, _detachedNode.detachedProps)(props.parent, {
     type: 'ArrowFunctionExpression',
     id: null,
-    // $FlowExpectedError[incompatible-use]
     expression: props.body.type !== 'BlockStatement',
     params: props.params.map(n => (0, _detachedNode.asDetachedNode)(n)),
     body: (0, _detachedNode.asDetachedNode)(props.body),
-    // $FlowFixMe[incompatible-type]
     typeParameters: (0, _detachedNode.asDetachedNode)(props.typeParameters),
-    // $FlowFixMe[incompatible-type]
     returnType: (0, _detachedNode.asDetachedNode)(props.returnType),
-    // $FlowFixMe[incompatible-type]
     predicate: (0, _detachedNode.asDetachedNode)(props.predicate),
     async: props.async
   });
@@ -48,13 +35,9 @@ function ClassDeclaration(props) {
 
   const node = (0, _detachedNode.detachedProps)(props.parent, {
     type: 'ClassDeclaration',
-    // $FlowFixMe[incompatible-type]
     id: (0, _detachedNode.asDetachedNode)(props.id),
-    // $FlowFixMe[incompatible-type]
     typeParameters: (0, _detachedNode.asDetachedNode)(props.typeParameters),
-    // $FlowFixMe[incompatible-type]
     superClass: (0, _detachedNode.asDetachedNode)(props.superClass),
-    // $FlowFixMe[incompatible-type]
     superTypeArguments: (0, _detachedNode.asDetachedNode)(props.superTypeArguments),
     decorators: ((_props$decorators = props.decorators) != null ? _props$decorators : []).map(n => (0, _detachedNode.asDetachedNode)(n)),
     implements: ((_props$implements = props.implements) != null ? _props$implements : []).map(n => (0, _detachedNode.asDetachedNode)(n)),
@@ -62,8 +45,7 @@ function ClassDeclaration(props) {
   });
   (0, _detachedNode.setParentPointersInDirectChildren)(node);
   return node;
-} // raw/cooked are on a subobject in the estree spec, but are flat on the hermes types
-
+}
 
 function TemplateElement(props) {
   return (0, _detachedNode.detachedProps)(props.parent, {
@@ -74,9 +56,7 @@ function TemplateElement(props) {
       raw: props.raw
     }
   });
-} // Identifier has a bunch of stuff that usually you don't want to provide - so we have
-// this manual def to allow us to default some values
-
+}
 
 function Identifier(props) {
   var _props$optional;
@@ -85,14 +65,11 @@ function Identifier(props) {
     type: 'Identifier',
     name: props.name,
     optional: (_props$optional = props.optional) != null ? _props$optional : false,
-    // $FlowFixMe[incompatible-type]
     typeAnnotation: (0, _detachedNode.asDetachedNode)(props.typeAnnotation)
   });
   (0, _detachedNode.setParentPointersInDirectChildren)(node);
   return node;
-} // Program has a bunch of stuff that usually you don't want to provide - so we have
-// this manual def to allow us to default some values
-
+}
 
 function Program(props) {
   var _props$sourceType, _props$tokens, _props$comments;
@@ -103,17 +80,13 @@ function Program(props) {
     body: props.body.map(n => (0, _detachedNode.asDetachedNode)(n)),
     tokens: (_props$tokens = props.tokens) != null ? _props$tokens : [],
     comments: (_props$comments = props.comments) != null ? _props$comments : [],
-    interpreter: props.interpreter != null ? // $FlowFixMe[incompatible-type]
-    (0, _detachedNode.asDetachedNode)({
+    interpreter: props.interpreter != null ? (0, _detachedNode.asDetachedNode)({
       type: 'InterpreterDirective',
       value: props.interpreter
     }) : null,
     docblock: props.docblock
   });
-} // the type annotation is stored on the Identifier's typeAnnotation
-// which is super awkward to work with and type - so we flatten the input
-// and put it in the right spot after
-
+}
 
 function DeclareFunction(props) {
   const node = (0, _detachedNode.detachedProps)(props.parent, {
@@ -126,7 +99,6 @@ function DeclareFunction(props) {
         typeAnnotation: (0, _detachedNode.asDetachedNode)(props.functionType)
       })
     }),
-    // $FlowFixMe[incompatible-type]
     predicate: (0, _detachedNode.asDetachedNode)(props.predicate)
   });
   (0, _detachedNode.setParentPointersInDirectChildren)(node);
@@ -145,8 +117,7 @@ function MemberExpression(props) {
   });
   (0, _detachedNode.setParentPointersInDirectChildren)(node);
   return node;
-} // Ignore the hermes-specific `implicit` property.
-
+}
 
 function BlockStatement(props) {
   const node = (0, _detachedNode.detachedProps)(props.parent, {

package/dist/src/index.js

@@ -1,12 +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';
 
 Object.defineProperty(exports, "__esModule", {

package/dist/src/transform/Errors.js

@@ -5,15 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.UnexpectedTransformationState = exports.NodeIsMutatedError = exports.NodeIsDeletedError = exports.InvalidStatementError = exports.InvalidReplacementError = exports.InvalidRemovalError = exports.InvalidInsertionError = void 0;
 
-/**
- * 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
- */
 class NodeIsDeletedError extends Error {}
 
 exports.NodeIsDeletedError = NodeIsDeletedError;

package/dist/src/transform/MutationContext.js

@@ -9,17 +9,7 @@ var _Errors = require("./Errors");
 
 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
- */
 class MutationContext {
-  // TODO - do we care about this? Arrays are pretty safe to concurrently mutate
   constructor(code) {
     this._deletedNodes = new Set();
     this._mutatedKeys = new Map();
@@ -27,42 +17,25 @@ class MutationContext {
     this.code = void 0;
     this.code = code;
   }
-  /**
-   * Marks a node and its entire subtree as deleted.
-   */
-
 
   markDeletion(node) {
     this._deletedNodes.add(node);
   }
-  /**
-   * Marks the key of the node as having been mutated.
-   */
-
 
   markMutation(node, key) {
     var _map$get$add, _map$get;
 
     this.assertNotDeleted(node, `Attempted to mutate a \`${node.type}.${key}\` on a deleted node.`);
     this.assertNotMutated(node, key, `Attempted to mutate a \`${node.type}.${key}\` when it has already been mutated.`);
-    const map = Array.isArray( // $FlowExpectedError[prop-missing]
-    node[key]) ? this._mutatedArrays : this._mutatedKeys;
+    const map = Array.isArray(node[key]) ? this._mutatedArrays : this._mutatedKeys;
     map.set(node, (_map$get$add = (_map$get = map.get(node)) == null ? void 0 : _map$get.add(key)) != null ? _map$get$add : new Set([key]));
   }
-  /**
-   * Throws if the node has been deleted
-   */
-
 
   assertNotDeleted(node, message) {
     if (this._deletedNodes.has(node)) {
       throw new _Errors.NodeIsDeletedError(message);
     }
   }
-  /**
-   * Throws if the key of the node has been mutated
-   */
-
 
   assertNotMutated(node, key, message) {
     var _this$_mutatedKeys$ge;

package/dist/src/transform/TransformContext.js

@@ -29,19 +29,7 @@ var _ReplaceStatementWithMany = require("./mutations/ReplaceStatementWithMany");
 
 var _ModifyNodeInPlace = require("./mutations/ModifyNodeInPlace");
 
-/**
- * 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 getTransformContext() {
-  /**
-   * The mutations in order of collection.
-   */
   const mutations = [];
 
   function pushMutation(mutation) {
@@ -51,7 +39,6 @@ function getTransformContext() {
   }
 
   const cloneAPIs = {
-    // $FlowFixMe[incompatible-exact]
     shallowCloneNode: node => {
       if (node == null) {
         return null;
@@ -66,7 +53,6 @@ function getTransformContext() {
 
       return (0, _detachedNode.shallowCloneNode)(node, newProps);
     },
-    // $FlowFixMe[incompatible-exact]
     shallowCloneArray: nodes => {
       if (nodes == null) {
         return null;
@@ -76,14 +62,12 @@ function getTransformContext() {
         const node = node_;
 
         if (node == null) {
-          // $FlowExpectedError[incompatible-type]
           return node;
         }
 
         return (0, _detachedNode.shallowCloneNode)(node, {});
       });
     },
-    // $FlowFixMe[incompatible-exact]
     deepCloneNode: node => {
       if (node == null) {
         return null;
@@ -176,7 +160,6 @@ function getTransformContext() {
 
       for (const [key, value] of Object.entries(newProps)) {
         if ((0, _hermesParser.isNode)(value)) {
-          // $FlowFixMe[incompatible-type]
           const node = value;
           detachedProps[key] = (0, _detachedNode.asDetachedNode)(node);
         } else {
@@ -190,7 +173,6 @@ function getTransformContext() {
   return {
     mutations,
 
-    // $FlowExpectedError[unsafe-getters-setters]
     get astWasMutated() {
       return mutations.length > 0;
     },

package/dist/src/transform/comments/comments.js

@@ -36,18 +36,6 @@ var _os = require("os");
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-/**
- * 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
- */
-// $FlowExpectedError[untyped-import]
-// $FlowExpectedError[untyped-import]
-// $FlowExpectedError[untyped-import]
 const CommentPlacement = require("flow-enums-runtime").Mirrored(["LEADING_OWN_LINE", "LEADING_INLINE", "TRAILING_OWN_LINE", "TRAILING_INLINE"]);
 
 exports.CommentPlacement = CommentPlacement;
@@ -61,30 +49,18 @@ function attachComments(comments, ast, text) {
 }
 
 function mutateESTreeASTCommentsForPrettier(program, text) {
-  let code = text; // we need to delete the comments prop or else prettier will do
-  // its own attachment pass after the mutation and duplicate the
-  // comments on each node, borking the output
-  // $FlowExpectedError[cannot-write]
-
-  delete program.comments; // The docblock comment is never attached to any AST nodes, since its technically
-  // attached to the program. However this is specific to our AST and in order for
-  // prettier to correctly print it we need to attach it to the first node in the
-  // program body.
+  let code = text;
+  delete program.comments;
 
   if (program.docblock != null && program.docblock.comment != null) {
     const docblockComment = program.docblock.comment;
     const isDocblockCommentNew = !isAttachedComment(docblockComment);
 
     if (isDocblockCommentNew) {
-      // $FlowExpectedError[prop-missing]
-      docblockComment.printed = false; // $FlowExpectedError[prop-missing]
-
-      docblockComment.leading = true; // $FlowExpectedError[prop-missing]
-
+      docblockComment.printed = false;
+      docblockComment.leading = true;
       docblockComment.trailing = false;
-    } // If we have a first node in the program body, attach the comment to that
-    // otherwise set it on the program.
-
+    }
 
     if (program.body.length > 0) {
       const firstStatement = program.body[0];
@@ -94,17 +70,13 @@ function mutateESTreeASTCommentsForPrettier(program, text) {
         setCommentsOnNode(firstStatement, [docblockComment, ...getCommentsForNode(firstStatement)]);
 
         if (isDocblockCommentNew) {
-          // Add markers to the code block to ensure docblock comment is printed on its
-          // own line.
           code = makeCommentOwnLine(code, docblockComment);
         }
       }
     } else {
       setCommentsOnNode(program, [docblockComment]);
     }
-  } // Should not be needed anymore
-  // $FlowExpectedError[cannot-write]
-
+  }
 
   delete program.docblock;
   return code;
@@ -121,40 +93,30 @@ function cloneCommentsToNewNode(oldNode, newNode) {
 
 function cloneJSDocCommentsToNewNode(oldNode, newNode) {
   const comments = getCommentsForNode(oldNode).filter(comment => {
-    return (0, _hermesEstree.isBlockComment)(comment) && // JSDoc comments always start with an extra asterisk
-    comment.value.startsWith('*');
+    return (0, _hermesEstree.isBlockComment)(comment) && comment.value.startsWith('*');
   });
   setCommentsOnNode(newNode, [...getCommentsForNode(newNode), ...comments.map(cloneCommentWithMarkers)]);
 }
 
 function setCommentsOnNode(node, comments) {
-  // $FlowExpectedError[prop-missing] - this property is secretly added by prettier.
-  // $FlowExpectedError[incompatible-use] - this property is secretly added by prettier.
-  // $FlowExpectedError[cannot-write] - this property is secretly added by prettier.
   node.comments = comments;
 }
 
 function getCommentsForNode(node) {
   var _node$comments;
 
-  // $FlowExpectedError[prop-missing] - this property is secretly added by prettier.
-  // $FlowExpectedError[incompatible-use] - this property is secretly added by prettier.
   return (_node$comments = node.comments) != null ? _node$comments : [];
 }
 
 function isAttachedComment(comment) {
-  // Prettier adds this property to comments that have been attached.
-  // $FlowExpectedError[prop-missing] - this property is secretly added by prettier.
   return comment.printed === false;
 }
 
 function isLeadingComment(comment) {
-  // $FlowExpectedError[prop-missing] - this property is secretly added by prettier.
   return comment.leading === true;
 }
 
 function isTrailingComment(comment) {
-  // $FlowExpectedError[prop-missing] - this property is secretly added by prettier.
   return comment.trailing === true;
 }
 
@@ -185,7 +147,6 @@ function addComment(node, comment, placement) {
 }
 
 function cloneComment(comment) {
-  // $FlowExpectedError[incompatible-type]
   return {
     type: comment.type,
     value: comment.value,
@@ -195,7 +156,6 @@ function cloneComment(comment) {
 }
 
 function cloneCommentWithMarkers(comment) {
-  // $FlowExpectedError[incompatible-type]
   return {
     type: comment.type,
     value: comment.value,
@@ -215,19 +175,13 @@ function getFirstNonWhitespaceIndex(code) {
 }
 
 function makeCommentOwnLine(code, comment) {
-  let newCode = code; // Since we always want a line break we need to ensure a newline is found when
-  // searching out from either side of the comment range.
-
+  let newCode = code;
   let firstNewline = getFirstNewlineIndex(code);
 
   if (firstNewline === -1) {
-    // No newline in file, lets add one.
     newCode += _os.EOL;
     firstNewline = newCode.length;
-  } // Prettier only uses these ranges for detecting whitespace, so this nonsensical
-  // range is safe.
-  // $FlowExpectedError[cannot-write]
-
+  }
 
   comment.range = [firstNewline + 1, firstNewline];
   return newCode;
@@ -239,11 +193,6 @@ function appendCommentToSource(code, comment, placement) {
   switch (comment.type) {
     case 'Block':
       {
-        // Prettier decides if a newline is necessary between the comment and its node by looking
-        // to see if a newline seperates them in the source text. We can trick prettier into
-        // formatting how we want for new comments by placing the range such that a newline
-        // will (OWN_LINE) or will not (INLINE) be found when searching from the specified range
-        // position.
         switch (placement) {
           case CommentPlacement.LEADING_OWN_LINE:
           case CommentPlacement.TRAILING_OWN_LINE:
@@ -255,18 +204,13 @@ function appendCommentToSource(code, comment, placement) {
           case CommentPlacement.LEADING_INLINE:
           case CommentPlacement.TRAILING_INLINE:
             {
-              // Since we don't want a line break we need to ensure a non whitespace char is
-              // always found before a newline when searching out from either side of the
-              // comment range.
               let firstNonWhitespace = getFirstNonWhitespaceIndex(code);
 
               if (firstNonWhitespace === -1) {
-                // No non whitespace chars in file, lets add an identifiable statement for prettier to find.
                 newCode += '$FORCE_INLINE_ON_EMPTY_FILE_TOKEN$;';
                 firstNonWhitespace = newCode.length;
                 break;
-              } // $FlowExpectedError[cannot-write]
-
+              }
 
               comment.range = [firstNonWhitespace + 1, firstNonWhitespace];
               break;
@@ -278,30 +222,20 @@ function appendCommentToSource(code, comment, placement) {
 
     case 'Line':
       {
-        // For `Line` comments prettier slices comments directly from the source code when printing
-        // https://github.com/prettier/prettier/blob/5f0ee39fa03532c85bd1c35291450fe7ac3667b3/src/language-js/print/comment.js#L15-L20
-        // this means that we need to have any appended comments directly in the
-        // source code or else prettier will slice nothing and bork up the transform
         const commentText = `//${comment.value}`;
         const lastChar = newCode[newCode.length - 1];
 
         if (lastChar !== '\n' && lastChar !== '\r') {
           newCode += _os.EOL;
-        } // Line comments cannot be inline before a node so we only place trailing Line comments inline.
-
+        }
 
         if (placement === CommentPlacement.TRAILING_INLINE) {
-          // Prettier determines an "end of line" comment by walking backwards from
-          // the comment start range through the source code to see if it finds a non
-          // newline token. In order to trick prettier for new comments we need to
-          // insert fake source code for it to find.
           newCode += '$FORCE_END_OF_LINE_COMMENT_TOKEN$;';
         }
 
         const start = newCode.length;
         newCode += commentText;
-        const end = newCode.length; // $FlowExpectedError[cannot-write]
-
+        const end = newCode.length;
         comment.range = [start, end];
         break;
       }