hermes-transform 0.10.1 → 0.11.1

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

@@ -7,6 +7,7 @@ exports.ArrowFunctionExpression = ArrowFunctionExpression;
 exports.ClassDeclaration = ClassDeclaration;
 exports.DeclareFunction = DeclareFunction;
 exports.Identifier = Identifier;
+exports.MemberExpression = MemberExpression;
 exports.Program = Program;
 exports.TemplateElement = TemplateElement;
 
@@ -116,4 +117,18 @@ function DeclareFunction(props) {
   });
   (0, _detachedNode.setParentPointersInDirectChildren)(node);
   return node;
+}
+
+function MemberExpression(props) {
+  var _props$optional2;
+
+  const node = (0, _detachedNode.detachedProps)(props.parent, {
+    type: 'MemberExpression',
+    object: (0, _detachedNode.asDetachedNode)(props.object),
+    property: (0, _detachedNode.asDetachedNode)(props.property),
+    computed: props.computed,
+    optional: (_props$optional2 = props.optional) != null ? _props$optional2 : false
+  });
+  (0, _detachedNode.setParentPointersInDirectChildren)(node);
+  return node;
 }

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

@@ -21,6 +21,7 @@ import type {
   TemplateElement as TemplateElementType,
   Program as ProgramType,
   DocblockMetadata as DocblockMetadataType,
+  MemberExpression as MemberExpressionType,
 } from 'hermes-estree';
 import type {DetachedNode, MaybeDetachedNode} from '../../detachedNode';
 
@@ -203,3 +204,25 @@ export function DeclareFunction(props: {
   setParentPointersInDirectChildren(node);
   return node;
 }
+
+export type MemberExpressionProps = {
+  +object: MaybeDetachedNode<MemberExpressionType['object']>,
+  +property: MaybeDetachedNode<MemberExpressionType['property']>,
+  +computed: MemberExpressionType['computed'],
+  +optional?: MemberExpressionType['optional'],
+};
+
+export function MemberExpression(props: {
+  ...$ReadOnly<MemberExpressionProps>,
+  +parent?: ESNode,
+}): DetachedNode<MemberExpressionType> {
+  const node = detachedProps<MemberExpressionType>(props.parent, {
+    type: 'MemberExpression',
+    object: asDetachedNode(props.object),
+    property: asDetachedNode(props.property),
+    computed: props.computed,
+    optional: props.optional ?? false,
+  });
+  setParentPointersInDirectChildren(node);
+  return node;
+}

package/dist/index.js

@@ -12,7 +12,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.traverseWithContext = exports.traverse = exports.transform = exports.t = exports.print = exports.parse = exports.cloneJSDocCommentsToNewNode = exports.asDetachedNode = exports.SimpleTraverser = void 0;
+exports.traverseWithContext = exports.traverse = exports.transform = exports.t = exports.print = exports.parse = exports.makeCommentOwnLine = exports.cloneJSDocCommentsToNewNode = exports.asDetachedNode = exports.SimpleTraverser = void 0;
 
 var _hermesParser = require("hermes-parser");
 
@@ -46,6 +46,7 @@ exports.asDetachedNode = _detachedNode.asDetachedNode;
 var _comments = require("./transform/comments/comments");
 
 exports.cloneJSDocCommentsToNewNode = _comments.cloneJSDocCommentsToNewNode;
+exports.makeCommentOwnLine = _comments.makeCommentOwnLine;
 
 function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function (nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
 

package/dist/index.js.flow

@@ -22,4 +22,7 @@ export {parse} from './transform/parse';
 export {print} from './transform/print';
 export * as t from './generated/node-types';
 export {asDetachedNode} from './detachedNode';
-export {cloneJSDocCommentsToNewNode} from './transform/comments/comments';
+export {
+  cloneJSDocCommentsToNewNode,
+  makeCommentOwnLine,
+} from './transform/comments/comments';

package/dist/transform/comments/comments.js

@@ -16,6 +16,7 @@ exports.getLeadingCommentsForNode = getLeadingCommentsForNode;
 exports.getTrailingCommentsForNode = getTrailingCommentsForNode;
 exports.isLeadingComment = isLeadingComment;
 exports.isTrailingComment = isTrailingComment;
+exports.makeCommentOwnLine = makeCommentOwnLine;
 exports.moveCommentsToNewNode = moveCommentsToNewNode;
 exports.setCommentsOnNode = setCommentsOnNode;
 
@@ -67,7 +68,7 @@ function cloneJSDocCommentsToNewNode(oldNode, newNode) {
     return (0, _hermesEstree.isBlockComment)(comment) && // JSDoc comments always start with an extra asterisk
     comment.value.startsWith('*');
   });
-  setCommentsOnNode(newNode, comments.map(cloneCommentWithMarkers));
+  setCommentsOnNode(newNode, [...getCommentsForNode(newNode), ...comments.map(cloneCommentWithMarkers)]);
 }
 
 function setCommentsOnNode(node, comments) {
@@ -165,6 +166,25 @@ function getFirstNonWhitespaceIndex(code) {
   return code.search(/\S/);
 }
 
+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 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;
+}
+
 function appendCommentToSource(code, comment, placement) {
   let newCode = code;
 
@@ -180,20 +200,7 @@ function appendCommentToSource(code, comment, placement) {
           case CommentPlacement.LEADING_OWN_LINE:
           case CommentPlacement.TRAILING_OWN_LINE:
             {
-              // 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 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];
+              newCode = makeCommentOwnLine(code, comment);
               break;
             }
 

package/dist/transform/comments/comments.js.flow

@@ -66,7 +66,10 @@ export function cloneJSDocCommentsToNewNode(
       comment.value.startsWith('*')
     );
   });
-  setCommentsOnNode(newNode, comments.map(cloneCommentWithMarkers));
+  setCommentsOnNode(newNode, [
+    ...getCommentsForNode(newNode),
+    ...comments.map(cloneCommentWithMarkers),
+  ]);
 }
 
 export function setCommentsOnNode(
@@ -172,6 +175,25 @@ function getFirstNonWhitespaceIndex(code: string): number {
   return code.search(/\S/);
 }
 
+export function makeCommentOwnLine(code: string, comment: Comment): string {
+  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 firstNewline = getFirstNewlineIndex(code);
+  if (firstNewline === -1) {
+    // No newline in file, lets add one.
+    newCode += 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;
+}
+
 export function appendCommentToSource(
   code: string,
   comment: Comment,
@@ -188,18 +210,7 @@ export function appendCommentToSource(
       switch (placement) {
         case CommentPlacement.LEADING_OWN_LINE:
         case CommentPlacement.TRAILING_OWN_LINE: {
-          // 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 firstNewline = getFirstNewlineIndex(code);
-          if (firstNewline === -1) {
-            // No newline in file, lets add one.
-            newCode += 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];
+          newCode = makeCommentOwnLine(code, comment);
           break;
         }
         case CommentPlacement.LEADING_INLINE:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hermes-transform",
-  "version": "0.10.1",
+  "version": "0.11.1",
   "description": "Tools built on top of Hermes-ESTree to enable codebase transformation",
   "main": "dist/index.js",
   "license": "MIT",
@@ -12,9 +12,9 @@
     "@babel/code-frame": "^7.16.0",
     "esquery": "^1.4.0",
     "flow-enums-runtime": "^0.0.6",
-    "hermes-eslint": "0.10.1",
-    "hermes-estree": "0.10.1",
-    "hermes-parser": "0.10.1"
+    "hermes-eslint": "0.11.1",
+    "hermes-estree": "0.11.1",
+    "hermes-parser": "0.11.1"
   },
   "peerDependencies": {
     "prettier": "^2.7.1"