Revert "Enable all supported extensions by default in config (#770)"

This reverts commit c8343a7.
aws · Feb 22, 2024 · b83fa48 · b83fa48
1 parent dedf1f5
commit b83fa48
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 11 deletions.
diff --git a/src/cli.ts b/src/cli.ts
@@ -24,6 +24,7 @@
 // eslint-disable-next-line @typescript-eslint/ban-ts-comment
 // @ts-nocheck
 import path from "path";
+import { DEFAULT_EXTENSIONS } from "@babel/core";
 import Runner from "jscodeshift/dist/Runner";
 
 import {
@@ -40,6 +41,16 @@ if (args[2] === "--help" || args[2] === "-h") {
   process.stdout.write(getHelpParagraph(transforms));
 }
 
+// Refs: https://github.com/facebook/jscodeshift/issues/582
+if (!args.some((arg) => arg.startsWith("--extensions"))) {
+  // Explicitly add all extensions as default to avoid bug in jscodeshift.
+  // Refs: https://github.com/facebook/jscodeshift/blob/51da1a5c4ba3707adb84416663634d4fc3141cbb/src/Worker.js#L80
+  const babelExtensions = DEFAULT_EXTENSIONS.map((ext) =>
+    ext.startsWith(".") ? ext.substring(1) : ext
+  );
+  args.push(`--extensions=${[...babelExtensions, "ts", "tsx"].join(",")}`);
+}
+
 const disclaimerLines = [
   `╔════════════════════════════════════════════════════════╗`,
   `║ Please review the code change thoroughly for required  ║`,

diff --git a/src/utils/getJsCodeshiftParser.ts b/src/utils/getJsCodeshiftParser.ts
@@ -6,7 +6,6 @@
 
 import { readFileSync } from "fs";
 import { dirname, join } from "path";
-import { DEFAULT_EXTENSIONS } from "@babel/core";
 import argsParser from "jscodeshift/dist/argsParser";
 
 // eslint-disable-next-line @typescript-eslint/ban-ts-comment
@@ -77,16 +76,7 @@ export const getJsCodeshiftParser = () =>
     },
     extensions: {
       display_index: 3,
-      // Explicitly add all extensions as default to avoid bug in jscodeshift.
-      // Refs: https://github.com/facebook/jscodeshift/issues/582
-      // Source code: https://github.com/facebook/jscodeshift/blob/51da1a5c4ba3707adb84416663634d4fc3141cbb/src/Worker.js#L80
-      default: [
-        ...DEFAULT_EXTENSIONS.map((ext) => (ext.startsWith(".") ? ext.substring(1) : ext)),
-        "ts",
-        "tsx",
-      ]
-        .sort()
-        .join(","),
+      default: "js",
       help: "transform files with these file extensions (comma separated list)",
       metavar: "EXT",
     },