esupgrade 2025.22.0 → 2025.22.2

package/.github/workflows/ci.yml

@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     timeout-minutes: 1
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v7
       - id: read_readme
         run: |
           echo "::set-output name=content::$(cat README.md)"
@@ -21,9 +21,9 @@ jobs:
     name: Test
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v7
       - name: Set up Node.js
-        uses: actions/setup-node@v6
+        uses: actions/setup-node@v7
         with:
           node-version-file: package.json
       - run: npm ci

package/.github/workflows/release.yml

@@ -10,8 +10,8 @@ jobs:
     name: npmjs.org
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v6
-      - uses: actions/setup-node@v6
+      - uses: actions/checkout@v7
+      - uses: actions/setup-node@v7
         with:
           node-version-file: package.json
           registry-url: 'https://registry.npmjs.org'

package/.pre-commit-config.yaml

@@ -32,7 +32,7 @@ repos:
       - id: write-good
         args: [--no-passive]
   - repo: https://github.com/pre-commit/mirrors-eslint
-    rev: v10.4.1
+    rev: v10.7.0
     hooks:
       - id: eslint
         args: ["--fix"]

package/README.md

@@ -69,10 +69,10 @@ echo "git diff --cached --name-only --diff-filter=ACMR -z -- '*.js' '*.jsx' '*.t
 
 ### Agent Skill
 
-esupgrade is available as a skill in [Claude Code]. To use it:
+esupgrade is available as a skill in [Claude Code]. The plugin is distributed via the [codingjoe/claude-plugins](https://github.com/codingjoe/claude-plugins) marketplace. To use it:
 
-1. Run `/plugin marketplace add codingjoe/esupgrade`
-1. Run `/plugin install esupgrade@esupgrade`
+1. Run `/plugin marketplace add codingjoe/claude-plugins`
+1. Run `/plugin install esupgrade@codingjoe`
 
 The skill will analyze your selected code and suggest transformations based on the Baseline browser support policy.
 

package/package.json

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

package/{SKILL.md → skills/esugprade/SKILL.md}

@@ -1,11 +1,7 @@
 ---
 name: esupgrade
-description: Auto-update JavaScript and TypeScript syntax to new ECMAScript features based on browser support.
-license: BSD-2-Clause
-metadata:
-  author: codingjoe <security@codingjoe.dev>
-allowed-tools: Bash(npx -y esupgrade *) READ
-compatibility: Requires Node.js 24 or later and npx.
+description: Auto-update JavaScript and TypeScript syntax to new ECMAScript features based on browser support. Use when the user asks to modernize, upgrade, or clean up JavaScript or TypeScript syntax.
+allowed-tools: Bash(npx -y esupgrade *) Read
 ---
 
 Use the CLI with `npx` on files, directories, or standard input:

package/src/widelyAvailable/anonymousFunctionToArrow.js

@@ -68,6 +68,16 @@ export function anonymousFunctionToArrow(root) {
         arrowFunction.async = true
       }
 
+      // Preserve TypeScript generic type parameters
+      if (node.typeParameters) {
+        arrowFunction.typeParameters = node.typeParameters
+      }
+
+      // Preserve TypeScript return type annotation
+      if (node.returnType) {
+        arrowFunction.returnType = node.returnType
+      }
+
       j(path).replaceWith(arrowFunction)
 
       modified = true

package/src/widelyAvailable/varToLetOrConst.js

@@ -1,6 +1,36 @@
 import { default as j } from "jscodeshift"
 import { processMultipleDeclarators, processSingleDeclarator } from "../types.js"
 
+/**
+ * Detect whether a path is nested in a declared TypeScript module.
+ *
+ * @param {import("ast-types").NodePath} path - The path to check.
+ * @returns {boolean} True when the path is inside a declared TypeScript module.
+ */
+function isInDeclaredTypeScriptModule({ parentPath: currentPath }) {
+  while (currentPath) {
+    const currentNode = currentPath.node
+
+    if (j.TSModuleDeclaration.check(currentNode) && currentNode.declare === true) {
+      return true
+    }
+
+    currentPath = currentPath.parentPath
+  }
+
+  return false
+}
+
+/**
+ * Detect whether a variable declaration is ambient in TypeScript.
+ *
+ * @param {import("ast-types").NodePath} path - The variable declaration path.
+ * @returns {boolean} True when TypeScript ambient syntax requires keeping `var`.
+ */
+function isAmbientTypeScriptVar(path) {
+  return path.node.declare === true || isInDeclaredTypeScriptModule(path)
+}
+
 /**
  * Transform var to const or let.
  *
@@ -13,6 +43,10 @@ export function varToLetOrConst(root) {
   let modified = false
 
   root.find(j.VariableDeclaration, { kind: "var" }).forEach((path) => {
+    if (isAmbientTypeScriptVar(path)) {
+      return
+    }
+
     const isSingleDeclarator = path.node.declarations.length === 1
 
     const result = isSingleDeclarator

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

@@ -220,5 +220,25 @@ suite("widely-available", () => {
 
       assert(!result.modified, "skip named function expression")
     })
+
+    test("generic type parameters are preserved", () => {
+      const result = transform(`
+  globalThis.Promise.withResolvers = function<T>() {
+    return { promise: new Promise<T>(), resolve: null };
+  };
+`)
+
+      assert(result.modified, "transform generic function expression")
+      assert.match(result.code, /<T>\(\) =>/)
+    })
+
+    test("generic type parameters with return type are preserved", () => {
+      const result = transform(`
+  arr.map(function<T>(x: T): T { return x; });
+`)
+
+      assert(result.modified, "transform generic callback function")
+      assert.match(result.code, /<T>\(x: T\): T =>/)
+    })
   })
 })

package/tests/widelyAvailable/var-to-let-or-const.test.js

@@ -505,5 +505,29 @@ suite("widely-available", () => {
       assert.doesNotMatch(result.code, /var prop/)
       assert.doesNotMatch(result.code, /let prop/)
     })
+
+    test("preserve var in declare global", () => {
+      const result = transform(`
+  declare global {
+    var csrftoken: string;
+  }
+`)
+
+      assert(!result.modified, "skip ambient declare global var")
+      assert.match(result.code, /var csrftoken: string;/)
+      assert.doesNotMatch(result.code, /const csrftoken: string;/)
+      assert.doesNotMatch(result.code, /let csrftoken: string;/)
+    })
+
+    test("preserve top-level declare var", () => {
+      const result = transform(`
+  declare var csrftoken: string;
+`)
+
+      assert(!result.modified, "skip top-level ambient declare var")
+      assert.match(result.code, /declare var csrftoken: string;/)
+      assert.doesNotMatch(result.code, /declare const csrftoken: string;/)
+      assert.doesNotMatch(result.code, /declare let csrftoken: string;/)
+    })
   })
 })

package/.claude-plugin/marketplace.json

@@ -1,32 +0,0 @@
-{
-  "name": "esupgrade",
-  "description": "Auto-upgrade JavaScript and TypeScript syntax to new ECMAScript features based on browser support.",
-  "owner": {
-    "name": "codingjoe",
-    "email": "security@codingjoe.dev"
-  },
-  "plugins": [
-    {
-      "name": "esupgrade",
-      "source": "./",
-      "description": "Auto-upgrade JavaScript and TypeScript syntax to new ECMAScript features based on browser support.",
-      "displayName": "esupgrade",
-      "author": {
-        "name": "Johannes Maron",
-        "email": "security@codingjoe.dev"
-      },
-      "homepage": "https://github.com/codingjoe/esupgrade",
-      "repository": "https://github.com/codingjoe/esupgrade",
-      "license": "BSD-2-Clause",
-      "keywords": [
-        "javascript",
-        "ecmascript",
-        "modernize",
-        "upgrade",
-        "codemod",
-        "ast",
-        "transform"
-      ]
-    }
-  ]
-}