hermes-transform 0.23.0 → 0.24.0

package/dist/transform/mutations/utils/isValidModuleDeclarationParent.js

@@ -12,10 +12,16 @@ exports.isValidModuleDeclarationParent = isValidModuleDeclarationParent;
  * LICENSE file in the root directory of this source tree.
  *
  * 
- * @format
+ * @noformat
  */
-// $FlowFixMe[deprecated-type]
-function isModuleDeclaration(node) {
+function isModuleDeclaration(node)
+/*: node is (
+| ImportDeclaration
+| ExportNamedDeclaration
+| ExportDefaultDeclaration
+| ExportAllDeclaration
+) */
+{
   return node.type === 'ImportDeclaration' || node.type === 'ExportNamedDeclaration' || node.type === 'ExportDefaultDeclaration' || node.type === 'ExportAllDeclaration';
 }
 

package/dist/transform/mutations/utils/isValidModuleDeclarationParent.js.flow

@@ -5,14 +5,34 @@
  * LICENSE file in the root directory of this source tree.
  *
  * @flow strict-local
- * @format
+ * @noformat
  */
 
-import type {ESNode, ModuleDeclaration, Statement} from 'hermes-estree';
+import type {
+  ESNode,
+  // Used in flow comments
+  // eslint-disable-next-line no-unused-vars
+  ExportAllDeclaration,
+  // Used in flow comments
+  // eslint-disable-next-line no-unused-vars
+  ExportDefaultDeclaration,
+  // Used in flow comments
+  // eslint-disable-next-line no-unused-vars
+  ExportNamedDeclaration,
+  // Used in flow comments
+  // eslint-disable-next-line no-unused-vars
+  ImportDeclaration,
+  ModuleDeclaration,
+  Statement,
+} from 'hermes-estree';
 import type {DetachedNode} from '../../../detachedNode';
 
-// $FlowFixMe[deprecated-type]
-function isModuleDeclaration(node: ESNode): boolean %checks {
+function isModuleDeclaration(node: ESNode) /*: node is (
+  | ImportDeclaration
+  | ExportNamedDeclaration
+  | ExportDefaultDeclaration
+  | ExportAllDeclaration
+) */ {
   return (
     node.type === 'ImportDeclaration' ||
     node.type === 'ExportNamedDeclaration' ||

package/dist/transform/print.js

@@ -29,9 +29,24 @@ const cacheBase = Math.random();
 
 async function print(ast, originalCode, prettierOptions = {}, visitorKeys) {
   // $FlowExpectedError[incompatible-type] This is now safe to access.
-  const program = ast; // Fix up the AST to match what prettier expects.
+  const program = ast; // If the AST body is empty, we can skip the cost of prettier by returning a static string of the contents.
+
+  if (program.body.length === 0) {
+    var _program$docblock;
+
+    // If the program had a docblock comment, we need to create the string manually.
+    const docblockComment = (_program$docblock = program.docblock) == null ? void 0 : _program$docblock.comment;
+
+    if (docblockComment != null) {
+      return '/*' + docblockComment.value + '*/\n';
+    }
+
+    return '';
+  } // Cleanup the comments from the AST and generate the "orginal" code needed for prettier.
+
+
+  const codeForPrinting = (0, _comments.mutateESTreeASTCommentsForPrettier)(program, originalCode); // Fix up the AST to match what prettier expects.
 
-  const codeForPrinting = (0, _comments.mutateESTreeASTCommentsForPrettier)(program, originalCode);
   (0, _hermesParser.mutateESTreeASTForPrettier)(program, visitorKeys);
 
   switch (getPrettierMajorVersion()) {

package/dist/transform/print.js.flow

@@ -30,11 +30,24 @@ export async function print(
   // $FlowExpectedError[incompatible-type] This is now safe to access.
   const program: Program = ast;
 
-  // Fix up the AST to match what prettier expects.
+  // If the AST body is empty, we can skip the cost of prettier by returning a static string of the contents.
+  if (program.body.length === 0) {
+    // If the program had a docblock comment, we need to create the string manually.
+    const docblockComment = program.docblock?.comment;
+    if (docblockComment != null) {
+      return '/*' + docblockComment.value + '*/\n';
+    }
+
+    return '';
+  }
+
+  // Cleanup the comments from the AST and generate the "orginal" code needed for prettier.
   const codeForPrinting = mutateESTreeASTCommentsForPrettier(
     program,
     originalCode,
   );
+
+  // Fix up the AST to match what prettier expects.
   mutateESTreeASTForPrettier(program, visitorKeys);
 
   switch (getPrettierMajorVersion()) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hermes-transform",
-  "version": "0.23.0",
+  "version": "0.24.0",
   "description": "Tools built on top of Hermes-ESTree to enable codebase transformation",
   "main": "dist/index.js",
   "license": "MIT",
@@ -12,14 +12,14 @@
     "@babel/code-frame": "^7.16.0",
     "esquery": "^1.4.0",
     "flow-enums-runtime": "^0.0.6",
-    "hermes-eslint": "0.23.0",
-    "hermes-estree": "0.23.0",
-    "hermes-parser": "0.23.0",
+    "hermes-eslint": "0.24.0",
+    "hermes-estree": "0.24.0",
+    "hermes-parser": "0.24.0",
     "string-width": "4.2.3"
   },
   "peerDependencies": {
     "prettier": "^3.0.0 || ^2.7.1",
-    "prettier-plugin-hermes-parser": "0.23.0"
+    "prettier-plugin-hermes-parser": "0.24.0"
   },
   "files": [
     "dist",