esupgrade 2025.1.0 → 2025.2.1

package/AGENTS.md

@@ -0,0 +1,3 @@
+When writing code, you MUST ALWAYS follow the [naming-things](https://raw.githubusercontent.com/codingjoe/naming-things/refs/heads/main/README.md) guidlines.
+
+All code must be fully tested with a 100% coverage. Unreachable code must be removed.

package/README.md

@@ -93,6 +93,21 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 +const message = `You have ${count} items`;
 ```
 
+#### Traditional `for` loops → [`for...of` loops][mdn-for-of]
+
+```diff
+-for (let i = 0; i < items.length; i++) {
+-  const item = items[i];
+-  console.log(item);
+-}
++for (const item of items) {
++  console.log(item);
++}
+```
+
+> [!NOTE]
+> Transformations are limited to loops that start at 0, increment by 1, and where the index variable is not used in the loop body.
+
 #### `Array.from().forEach()` → [`for...of` loops][mdn-for-of]
 
 ```diff
@@ -104,6 +119,29 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 +}
 ```
 
+#### DOM `forEach()` → [`for...of` loops][mdn-for-of]
+
+```diff
+-document.querySelectorAll('.item').forEach(item => {
+-  item.classList.add('active');
+-});
++for (const item of document.querySelectorAll('.item')) {
++  item.classList.add('active');
++}
+```
+
+Supports:
+
+- `document.querySelectorAll()`
+- `document.getElementsByTagName()`
+- `document.getElementsByClassName()`
+- `document.getElementsByName()`
+- `window.frames`
+
+> [!NOTE]
+> Transformations limited to inline arrow or function expressions with block statement bodies.
+> Callbacks with index parameters or expression bodies are not transformed.
+
 #### `Array.from()` → [Array spread [...]][mdn-spread]
 
 ```diff
@@ -136,6 +174,15 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 +const withItem = [...array, item];
 ```
 
+#### `Math.pow()` → [Exponentiation operator \*\*][mdn-exponentiation]
+
+```diff
+-const result = Math.pow(2, 3);
+-const area = Math.PI * Math.pow(radius, 2);
++const result = 2 ** 3;
++const area = Math.PI * radius ** 2;
+```
+
 #### Function expressions → [Arrow functions][mdn-arrow-functions]
 
 ```diff
@@ -174,6 +221,7 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 [baseline]: https://web.dev/baseline/
 [mdn-arrow-functions]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions
 [mdn-const]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/const
+[mdn-exponentiation]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Exponentiation
 [mdn-for-of]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/for...of
 [mdn-let]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/let
 [mdn-promise-try]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/try

package/bin/esupgrade.js

@@ -98,50 +98,52 @@ function processFile(filePath, options) {
 
     if (result.modified) {
       if (options.check) {
-        console.log(`✗ ${filePath}`)
+        // Group changes by type for summary
+        const changesByType = {}
         if (result.changes && result.changes.length > 0) {
-          // Group changes by type
-          const changesByType = {}
-
           for (const change of result.changes) {
             if (!changesByType[change.type]) {
-              changesByType[change.type] = []
+              changesByType[change.type] = 0
             }
-            changesByType[change.type].push(change.line)
+            changesByType[change.type]++
           }
+        }
 
-          for (const [type, lines] of Object.entries(changesByType)) {
-            const uniqueLines = [...new Set(lines)].sort((a, b) => a - b)
+        const transformations = Object.keys(changesByType)
+          .map((type) => {
             const displayName = type
               .replace(/([A-Z])/g, " $1")
               .trim()
               .toLowerCase()
-            console.log(
-              `  - ${displayName} (line${uniqueLines.length > 1 ? "s" : ""}: ${uniqueLines.join(", ")})`,
-            )
-          }
+            return displayName
+          })
+          .join(", ")
+
+        console.log(`✗ ${filePath}`)
+        if (transformations) {
+          console.log(`  ${transformations}`)
         }
       }
 
       if (options.write) {
         fs.writeFileSync(filePath, result.code, "utf8")
         if (!options.check) {
-          console.log(`✓ Upgraded: ${filePath}`)
+          console.log(`✓ ${filePath}`)
         } else {
-          console.log(`  ✓ Changes written to file`)
+          console.log(`  ✓ written`)
         }
       }
 
-      return true
+      return { modified: true, changes: result.changes }
     } else {
       if (!options.check) {
-        console.log(`  No changes: ${filePath}`)
+        console.log(`  ${filePath}`)
       }
-      return false
+      return { modified: false, changes: [] }
     }
   } catch (error) {
-    console.error(`Error processing ${filePath}: ${error.message}`)
-    return false
+    console.error(`✗ Error: ${filePath}: ${error.message}`)
+    return { modified: false, changes: [] }
   }
 }
 
@@ -153,27 +155,42 @@ function processFiles(patterns, options) {
     process.exit(0)
   }
 
-  console.log(`Processing ${files.length} file(s) with baseline: ${options.baseline}\n`)
-
   let modifiedCount = 0
+  const allChanges = []
+
   for (const file of files) {
-    if (processFile(file, options)) {
+    const result = processFile(file, options)
+    if (result.modified) {
       modifiedCount++
+      allChanges.push(...result.changes)
     }
   }
 
   // Summary
-  console.log("")
-  if (modifiedCount > 0) {
-    if (options.write && options.check) {
-      console.log(`Summary: ${modifiedCount} file(s) upgraded`)
-    } else if (options.write) {
-      console.log(`Summary: ${modifiedCount} file(s) upgraded`)
+  if (options.check) {
+    console.log("")
+    if (modifiedCount > 0) {
+      // Count unique transformation types
+      const transformTypes = new Set(allChanges.map((c) => c.type))
+      const typeCount = transformTypes.size
+      const totalChanges = allChanges.length
+
+      console.log(
+        `${modifiedCount} file(s) need upgrading (${totalChanges} change${totalChanges !== 1 ? "s" : ""}, ${typeCount} type${typeCount !== 1 ? "s" : ""})`,
+      )
+      if (options.write) {
+        console.log("Changes have been written")
+      }
     } else {
-      console.log(`Summary: ${modifiedCount} file(s) need upgrading`)
+      console.log("All files are up to date")
     }
   } else {
-    console.log("Summary: All files are already modern")
+    console.log("")
+    if (modifiedCount > 0) {
+      console.log(`✓ ${modifiedCount} file(s) upgraded`)
+    } else {
+      console.log("All files are up to date")
+    }
   }
 
   // Exit with code 1 if --check specified and there were changes

package/package.json

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