Merge branch 'main' into augustin/dep0194

Author: AugustinMauroy

.github/workflows/ci.yml

@@ -68,7 +68,7 @@ jobs:
           persist-credentials: false
           show-progress: false
       - name: Set up Node.js
-        uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5.0.0
+        uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6.0.0
         with:
           cache: "npm"
           check-latest: true
@@ -120,7 +120,7 @@ jobs:
           persist-credentials: false
           show-progress: false
       - name: Set up Node.js
-        uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5.0.0
+        uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6.0.0
         with:
           cache: "npm"
           check-latest: true
@@ -156,7 +156,7 @@ jobs:
           persist-credentials: false
           show-progress: false
       - name: Set up Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5.0.0
+        uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6.0.0
         with:
           cache: "npm"
           check-latest: true

.github/workflows/codemod_publish.yml

@@ -86,7 +86,7 @@ jobs:
           ls -lah "$CODEMOD_PATH"
 
       - name: Setup Node.js environment
-        uses: actions/setup-node@a0853c24544627f65ddf259abe73b1d18a591444 # v5.0.0
+        uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6.0.0
         with:
           node-version-file: ".nvmrc"
           cache: npm

.vscode/settings.json

@@ -8,5 +8,8 @@
     "[markdown]": {
         "editor.wordWrap": "off"
     },
+    "[typescript][javascript]": {
+        "editor.defaultFormatter": "biomejs.biome"
+    },
     "typescript.experimental.useTsgo": true
 }

package-lock.json

@@ -27,9 +27,9 @@
   },
   "homepage": "https://nodejs.org/learn/userland-migrations",
   "devDependencies": {
-    "@biomejs/biome": "2.2.6",
-    "@types/node": "^24.7.2",
-    "@typescript/native-preview": "^7.0.0-dev.20251013.1"
+    "@biomejs/biome": "2.3.3",
+    "@types/node": "^24.10.0",
+    "@typescript/native-preview": "^7.0.0-dev.20251104.1"
   },
   "workspaces": [
     "./recipes/*",

package.json

@@ -1,4 +1,4 @@
-import type { Edit, Kinds, Range, SgNode, SgRoot } from '@codemod.com/jssg-types/main';
+import type { Edit, Range, SgNode, SgRoot } from '@codemod.com/jssg-types/main';
 import type Js from '@codemod.com/jssg-types/langs/javascript';
 import { resolveBindingPath } from '@nodejs/codemod-utils/ast-grep/resolve-binding-path';
 import { getNodeRequireCalls } from '@nodejs/codemod-utils/ast-grep/require-call';
@@ -8,30 +8,37 @@ import { removeBinding } from '@nodejs/codemod-utils/ast-grep/remove-binding';
 
 export default function transform(root: SgRoot<Js>): string | null {
 	const rootNode = root.root();
-	const bindingStatementFnTuples: [string, SgNode<Js>, (arg: string) => string][] = [];
+	const bindingStatementFnTuples: [
+		string,
+		SgNode<Js>,
+		(arg: string) => string,
+	][] = [];
 	const edits: Edit[] = [];
 	const linesToRemove: Range[] = [];
 
 	const updates = [
 		{
-			oldBind: "$.atob",
-			replaceFn: (arg: string) => `Buffer.from(${arg}, 'base64').toString('binary')`
+			oldBind: '$.atob',
+			replaceFn: (arg: string) =>
+				`Buffer.from(${arg}, 'base64').toString('binary')`,
 		},
 		{
-			oldBind: "$.btoa",
-			replaceFn: (arg: string) => `Buffer.from(${arg}, 'binary').toString('base64')`
-		}
+			oldBind: '$.btoa',
+			replaceFn: (arg: string) =>
+				`Buffer.from(${arg}, 'binary').toString('base64')`,
+		},
 	];
 
 	const statements = [
 		...getNodeRequireCalls(root, 'buffer'),
-		...getNodeImportStatements(root, 'buffer')
+		...getNodeImportStatements(root, 'buffer'),
 	];
 
 	for (const statement of statements) {
 		for (const update of updates) {
 			const binding = resolveBindingPath(statement, update.oldBind);
-			if (binding) bindingStatementFnTuples.push([binding, statement, update.replaceFn]);
+			if (binding)
+				bindingStatementFnTuples.push([binding, statement, update.replaceFn]);
 		}
 	}
 
@@ -43,32 +50,32 @@ export default function transform(root: SgRoot<Js>): string | null {
 		// Check for calls to the specified binding
 		const calls = rootNode.findAll({
 			rule: {
-				pattern: `${binding}($ARG)`
-			}
+				pattern: `${binding}($ARG)`,
+			},
 		});
 		// Check for any other calls, so as to not remove a statement that is still being used
 		const otherCalls = rootNode.findAll({
 			rule: {
 				all: [
 					{
-						pattern: 'buffer.$FN'
+						pattern: 'buffer.$FN',
 					},
 					{
 						not: {
-							pattern: '$.btoa($ARG)'
-						}
+							pattern: '$.btoa($ARG)',
+						},
 					},
 					{
 						not: {
-							pattern: '$.atob($ARG)'
-						}
-					}
-				]
-			}
+							pattern: '$.atob($ARG)',
+						},
+					},
+				],
+			},
 		});
 
 		for (const call of calls) {
-			const argMatch = call.getMatch("ARG");
+			const argMatch = call.getMatch('ARG');
 			if (argMatch) edits.push(call.replace(fn(argMatch.text())));
 		}
 

recipes/buffer-atob-btoa/src/workflow.ts

@@ -36,6 +36,6 @@
   },
   "devDependencies": {
     "@nodejs/codemod-utils": "*",
-    "@types/node": "^24.7.2"
+    "@types/node": "^24.10.0"
   }
 }

recipes/correct-ts-specifiers/package.json

@@ -9,15 +9,15 @@ import type JS from "@codemod.com/jssg-types/langs/javascript";
  *
  * Handles:
  * 1. Updates import/require statements that import `createRequireFromPath`:
- *    - `const { createRequireFromPath } = require('module')` -> `const { createRequire } = require('module')`
- *    - `const { createRequireFromPath } = require('node:module')` -> `const { createRequire } = require('node:module')`
- *    - `import { createRequireFromPath } from 'module'` -> `import { createRequire } from 'module'`
- *    - `import { createRequireFromPath } from 'node:module'` -> `import { createRequire } from 'node:module'`
+ *    - `const { createRequireFromPath } = require('module')` → `const { createRequire } = require('module')`
+ *    - `const { createRequireFromPath } = require('node:module')` → `const { createRequire } = require('node:module')`
+ *    - `import { createRequireFromPath } from 'module'` → `import { createRequire } from 'module'`
+ *    - `import { createRequireFromPath } from 'node:module'` → `import { createRequire } from 'node:module'`
  *
  * 2. Updates variable declarations that use `createRequireFromPath`:
- *    - `const myRequire = createRequireFromPath(arg)` -> `const myRequire = createRequire(arg)`
- *    - `let myRequire = createRequireFromPath(arg)` -> `let myRequire = createRequire(arg)`
- *    - `var myRequire = createRequireFromPath(arg)` -> `var myRequire = createRequire(arg)`
+ *    - `const myRequire = createRequireFromPath(arg)` → `const myRequire = createRequire(arg)`
+ *    - `let myRequire = createRequireFromPath(arg)` → `let myRequire = createRequire(arg)`
+ *    - `var myRequire = createRequireFromPath(arg)` → `var myRequire = createRequire(arg)`
  *
  * 3. Preserves original variable names and declaration types.
  */

recipes/create-require-from-path/src/workflow.ts

@@ -0,0 +1,45 @@
+# `crypto.fips` DEP0093
+
+This recipe transforms the usage from the deprecated `crypto.fips` to `crypto.getFips()` and `crypto.setFips()`.
+
+See [DEP0093](https://nodejs.org/api/deprecations.html#DEP0093).
+
+## Examples
+
+**Before:**
+
+```js
+import crypto from "node:crypto";
+import { fips } from "node:crypto";
+
+// Using crypto.fips
+crypto.fips;
+fips;
+
+// Using crypto.fips = true
+crypto.fips = true;
+fips = true;
+
+// Using crypto.fips = false
+crypto.fips = false;
+fips = false;
+```
+
+**After:**
+
+```js
+import crypto from "node:crypto";
+import { getFips, setFips } from "node:crypto";
+
+// Using crypto.getFips()
+crypto.getFips();
+getFips();
+
+// Using crypto.setFips(true)
+crypto.setFips(true);
+setFips(true);
+
+// Using crypto.setFips(false)
+crypto.setFips(false);
+setFips(false);
+```

recipes/crypto-fips-to-getFips/README.md

@@ -0,0 +1,21 @@
+schema_version: "1.0"
+name: "@nodejs/crypto-fips-to-getFips"
+version: 1.0.0
+description: Handle DEP0093 via transforming `crypto.fips` to `crypto.getFips()` and `crypto.setFips()`
+author: Usman S.
+license: MIT
+workflow: workflow.yaml
+category: migration
+
+targets:
+  languages:
+    - javascript
+    - typescript
+
+keywords:
+  - transformation
+  - migration
+
+registry:
+  access: public
+  visibility: public