esupgrade 2025.18.0 → 2025.19.0

package/bin/esupgrade.js

@@ -58,6 +58,7 @@ class FileProcessor {
    * @param {string} options.baseline - Baseline level for transformations.
    * @param {boolean} options.check - Whether to only check for changes.
    * @param {boolean} options.write - Whether to write changes to file.
+   * @param {boolean} options.verbose - The verbosity level for logging.
    * @param {boolean} options.jQuery - Whether to include jQuery transformers.
    * @returns {Promise<{modified: boolean, error: boolean}>} Result of processing.
    */
@@ -82,7 +83,10 @@ class FileProcessor {
       )
 
       if (!workerResult.success) {
-        console.error(`\x1b[31m✗\x1b[0m Error: ${filePath}: ${workerResult.error}`)
+        if (options.verbose) console.error(workerResult.error)
+        console.error(
+          `\x1b[31m✗\x1b[0m Error: ${filePath}: ${workerResult.error.message}`,
+        )
         return { modified: false, error: true }
       }
 
@@ -266,6 +270,7 @@ program
       .choices(["widely-available", "newly-available"])
       .default("widely-available"),
   )
+  .option("--verbose, -v", "Show more detailed output.", false)
   .option(
     "--check",
     "Report which files need upgrading and exit with code 1 if any do",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "esupgrade",
-  "version": "2025.18.0",
+  "version": "2025.19.0",
   "description": "Auto-upgrade your JavaScript syntax",
   "type": "module",
   "main": "src/index.js",

package/src/types.js

@@ -293,6 +293,10 @@ export class NodeTest {
    * @returns {boolean} True if predicate returned true for any node
    */
   #traverseForPredicate(astNode, predicate) {
+    if (!astNode) {
+      return false
+    }
+
     if (predicate(astNode)) {
       return true
     }

package/src/worker.js

@@ -23,6 +23,6 @@ try {
   parentPort.postMessage({
     success: false,
     filePath,
-    error: error.message,
+    error,
   })
 }

package/tests/cli.test.js

@@ -542,4 +542,30 @@ describe("CLI", () => {
     )
     assert.equal(resultWithJQuery.status, 0, "exits successfully with --jQuery")
   })
+
+  test("show error message without stack trace at default verbosity", () => {
+    const testFile = path.join(tempDir, "test.js")
+    fs.writeFileSync(testFile, `var x = {{{;`)
+
+    const result = spawnSync(process.execPath, [CLI_PATH, testFile], {
+      encoding: "utf8",
+    })
+
+    assert.match(result.stderr, /Error:/, "displays error message")
+    assert.doesNotMatch(result.stderr, /\n\s*at /, "omits stack trace")
+  })
+
+  test("show full error with stack trace with --verbose", () => {
+    const testFile = path.join(tempDir, "test.js")
+    fs.writeFileSync(testFile, `var x = {{{;`)
+
+    const result = spawnSync(
+      process.execPath,
+      [CLI_PATH, testFile, "--verbose", "--verbose"],
+      { encoding: "utf8" },
+    )
+
+    assert.match(result.stderr, /Error:/, "displays error message")
+    assert.match(result.stderr, /\n\s*at /, "includes stack trace at verbosity level 2")
+  })
 })

package/tests/widelyAvailable/named-arrow-function-to-named-function.test.js

@@ -116,6 +116,15 @@ suite("widely-available", () => {
       assert.match(result.code, /console\.info\('test'\)/)
     })
 
+    test("arrow function with array destructuring holes in body", () => {
+      const result = transform(
+        `const f = (arr) => { const [a, , b] = arr; return a + b; }`,
+      )
+
+      assert(result.modified, "transform arrow function with array holes in body")
+      assert.match(result.code, /function f\(arr\)/)
+    })
+
     test("skip arrow function using this", () => {
       const result = transform(`const method = () => { return this.value; }`)