esupgrade 2025.0.2 → 2025.2.0

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

@@ -73,7 +73,7 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 
 ### Widely available
 
-#### `var` → `let`/`const`
+#### `var` → [const][mdn-const] & [let][mdn-let]
 
 ```diff
 -var x = 1;
@@ -84,7 +84,7 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 +y = 3;
 ```
 
-#### String concatenation → Template literals
+#### String concatenation → [Template literals][mdn-template-literals]
 
 ```diff
 -const greeting = 'Hello ' + name + '!';
@@ -93,7 +93,22 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 +const message = `You have ${count} items`;
 ```
 
-#### `Array.from().forEach()` → `for...of` loops
+#### 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
 -Array.from(items).forEach(item => {
@@ -104,7 +119,44 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 +}
 ```
 
-#### `Object.assign({}, ...)` → Object spread
+#### 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
+-const doubled = Array.from(numbers).map(n => n * 2);
+-const filtered = Array.from(items).filter(x => x > 5);
+-const arr = Array.from(iterable);
++const doubled = [...numbers].map(n => n * 2);
++const filtered = [...items].filter(x => x > 5);
++const arr = [...iterable];
+```
+
+> [!NOTE]
+> `Array.from()` with a mapping function or thisArg is not converted.
+
+#### `Object.assign({}, ...)` → [Object spread {...}][mdn-spread]
 
 ```diff
 -const obj = Object.assign({}, obj1, obj2);
@@ -113,7 +165,7 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 +const copy = { ...original };
 ```
 
-#### `.concat()` → Array spread
+#### `.concat()` → [Array spread [...]][mdn-spread]
 
 ```diff
 -const combined = arr1.concat(arr2, arr3);
@@ -122,7 +174,16 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 +const withItem = [...array, item];
 ```
 
-#### Function expressions → Arrow functions
+#### `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
 -const fn = function(x) { return x * 2; };
@@ -145,7 +206,7 @@ For more information about Baseline browser support, visit [web.dev/baseline][ba
 > [!CAUTION]
 > These transformations are mainly to harden code for future releases and should be used with caution.
 
-#### `new Promise((resolve) => { ... })` → `Promise.try(() => { ... })`
+#### `new Promise((resolve) => { ... })` → [Promise.try][mdn-promise-try]
 
 ```diff
 -new Promise((resolve) => {
@@ -158,4 +219,12 @@ 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
+[mdn-spread]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Spread_syntax
+[mdn-template-literals]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Template_literals
 [pre-commit]: https://pre-commit.com/

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/images/logo-dark.svg

@@ -1,10 +1,10 @@
 <svg xmlns="http://www.w3.org/2000/svg" width="640" height="320" font-family="Segoe UI, system-ui, sans-serif" viewBox="0 0 1280 640">
   <circle cx="260" cy="320" r="200" fill="#f7df1e"/>
   <path fill="none" stroke="#191919" stroke-width="48" d="m167 409 95-95 95 95m-190 -95 95-95 95 95m415 50"/>
-  <text x="241" y="148" fill="#c9d1d9" font-size="92" font-weight="bold" transform="matrix(1.6 0 0 1.6 120 120)">
+  <text x="241" y="148" fill="#c9d1d9" font-size="90" font-weight="bold" transform="matrix(1.6 0 0 1.6 120 120)">
     esupgrade
   </text>
-  <text x="241" y="192" fill="#c9d1d9" font-size="28" font-style="italic" transform="matrix(1.6 0 0 1.6 120 120)">
+  <text x="241" y="192" fill="#c9d1d9" font-size="27" font-style="italic" transform="matrix(1.6 0 0 1.6 120 120)">
     Auto-upgrade your JavaScript syntax
   </text>
 </svg>

package/images/logo-light.svg

@@ -1,10 +1,10 @@
 <svg xmlns="http://www.w3.org/2000/svg" width="640" height="320" font-family="Segoe UI, system-ui, sans-serif" viewBox="0 0 1280 640">
   <circle cx="260" cy="320" r="200" fill="#f7df1e"/>
   <path fill="none" stroke="#191919" stroke-width="48" d="m167 409 95-95 95 95m-190 -95 95-95 95 95m415 50"/>
-  <text x="241" y="148" font-size="92" font-weight="bold" transform="matrix(1.6 0 0 1.6 120 120)">
+  <text x="241" y="148" font-size="90" font-weight="bold" transform="matrix(1.6 0 0 1.6 120 120)">
     esupgrade
   </text>
-  <text x="241" y="192" font-size="28" font-style="italic" transform="matrix(1.6 0 0 1.6 120 120)">
+  <text x="241" y="192" font-size="27" font-style="italic" transform="matrix(1.6 0 0 1.6 120 120)">
     Auto-upgrade your JavaScript syntax
   </text>
 </svg>

package/package.json

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

package/src/newlyAvailable.js

@@ -1,5 +1,6 @@
 /**
  * Transform new Promise((resolve, reject) => { resolve(fn()) }) to Promise.try(fn)
+ * @see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/try
  */
 export function promiseTry(j, root) {
   let modified = false