feat(net-setsimultaneousaccepts-migration): create recipe

package.json

@@ -19,8 +19,7 @@
     "automation",
     "codemod",
     "migrations",
-    "node.js"
-  ],
+    "node.js"  ],
   "license": "MIT",
   "bugs": {
     "url": "https://github.com/nodejs/userland-migrations/issues"

recipes/net-setSimultaneousAccepts-migration/README.md

@@ -0,0 +1,17 @@
+# `net._setSimultaneousAccepts()` DEP0121
+
+This recipe provides a guide for removing `net._setSimultaneousAccepts()`.
+
+See [DEP0121](https://nodejs.org/api/deprecations.html#DEP0121).
+
+## Examples
+
+**Before:**
+
+```js
+```
+
+**After:**
+
+```js
+```

recipes/net-setSimultaneousAccepts-migration/codemod.yaml

@@ -0,0 +1,21 @@
+schema_version: "1.0"
+name: "@nodejs/net-setSimultaneousAccepts-migration"
+version: 1.0.0
+description: "Handle DEDEP0121: Remove net._setSimultaneousAccepts()"
+author: Alejandro Espa
+license: MIT
+workflow: workflow.yaml
+category: migration
+
+targets:
+  languages:
+    - javascript
+    - typescript
+
+keywords:
+  - transformation
+  - migration
+
+registry:
+  access: public
+  visibility: public

recipes/net-setSimultaneousAccepts-migration/package.json

@@ -0,0 +1,24 @@
+{
+  "name": "@nodejs/net-setsimultaneousaccepts-migration",
+  "version": "1.0.0",
+  "description": "Handle DEP0121: Remove net._setSimultaneousAccepts().",
+  "type": "module",
+  "scripts": {
+    "test": "npx codemod jssg test -l typescript ./src/workflow.ts ./"
+  },
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/nodejs/userland-migrations.git",
+    "directory": "recipes/net-setsimultaneousaccepts-migration",
+    "bugs": "https://github.com/nodejs/userland-migrations/issues"
+  },
+  "author": "Alejandro Espa",
+  "license": "MIT",
+  "homepage": "https://github.com/nodejs/userland-migrations/blob/main/recipes/net-setsimultaneousaccepts-migration/README.md",
+  "devDependencies": {
+    "@codemod.com/jssg-types": "^1.0.9"
+  },
+  "dependencies": {
+    "@nodejs/codemod-utils": "*"
+  }
+}

recipes/net-setSimultaneousAccepts-migration/src/workflow.ts

@@ -0,0 +1,82 @@
+import {
+	getNodeImportStatements,
+	getNodeImportCalls,
+} from '@nodejs/codemod-utils/ast-grep/import-statement';
+import { getNodeRequireCalls } from '@nodejs/codemod-utils/ast-grep/require-call';
+import { resolveBindingPath } from '@nodejs/codemod-utils/ast-grep/resolve-binding-path';
+import { removeLines } from '@nodejs/codemod-utils/ast-grep/remove-lines';
+import type { SgRoot, Edit, Range } from '@codemod.com/jssg-types/main';
+import type Js from '@codemod.com/jssg-types/langs/javascript';
+
+/**
+ * Transform function that removes deprecated net._setSimultaneousAccepts() calls
+ *
+ * Handles:
+ * 1. net._setSimultaneousAccepts(true) → removed
+ * 2. net._setSimultaneousAccepts(false) → removed
+ * 3. Works with both CommonJS (require) and ESM (import) syntax
+ * 4. Handles aliased imports/requires
+ *
+ * Note: This was an internal API (DEP0121) that reached End-of-Life in Node.js v24.0.0
+ */
+export default function transform(root: SgRoot<Js>): string | null {
+	const rootNode = root.root();
+	const edits: Edit[] = [];
+	const linesToRemove: Range[] = [];
+
+	// Collect all 'node:net' module imports/requires
+	const allStatements = [
+		...getNodeImportStatements(root, 'node:net'),
+		...getNodeImportCalls(root, 'node:net'),
+		...getNodeRequireCalls(root, 'node:net'),
+	];
+
+	// If no net module statements found, skip transformation
+	if (allStatements.length === 0) return null;
+
+	for (const statementNode of allStatements) {
+		// Resolve the binding path for the net module
+		const bindingPath = resolveBindingPath(statementNode, '$');
+
+		if (!bindingPath) continue;
+
+		// Find all calls to _setSimultaneousAccepts on the resolved binding
+		const callExpressions = rootNode.findAll({
+			rule: {
+				pattern: `${bindingPath}._setSimultaneousAccepts($$$ARGS)`,
+			},
+		});
+
+		for (const callNode of callExpressions) {
+			// Find the parent statement (expression_statement) to remove the entire line
+			let parentStatement = callNode.parent();
+
+			// Traverse up to find the expression_statement
+			while (parentStatement && parentStatement.kind() !== 'expression_statement') {
+				parentStatement = parentStatement.parent();
+			}
+
+			if (parentStatement) {
+				// Get the range of the entire statement including semicolon
+				const range = parentStatement.range();
+				linesToRemove.push(range);
+			} else {
+				// Fallback: just remove the call itself if we can't find the statement
+				edits.push(callNode.replace(''));
+			}
+		}
+	}
+
+	// If no changes were made, return null
+	if (edits.length === 0 && linesToRemove.length === 0) return null;
+
+	// Apply edits first if any
+	const sourceCode = edits.length > 0 
+		? rootNode.commitEdits(edits) 
+		: rootNode.text();
+
+	// Then remove the lines
+	return linesToRemove.length > 0
+		? removeLines(sourceCode, linesToRemove)
+		: sourceCode;
+}

recipes/net-setSimultaneousAccepts-migration/tests/expected/file-1.js

@@ -0,0 +1,3 @@
+const net = require("node:net");
+
+const server = net.createServer();

recipes/net-setSimultaneousAccepts-migration/tests/expected/file-2.js

@@ -0,0 +1,7 @@
+const net = require("node:net");
+
+function createServer() {
+  return net.createServer((socket) => {
+    // handle connection
+  });
+}

recipes/net-setSimultaneousAccepts-migration/tests/expected/file-3.js

@@ -0,0 +1,5 @@
+const net = require("node:net");
+
+module.exports = {
+  createServer: () => net.createServer()
+};

recipes/net-setSimultaneousAccepts-migration/tests/expected/file-4.js

@@ -0,0 +1,6 @@
+const net = require("node:net");
+
+function initializeApp() {
+  const server = net.createServer();
+  server.listen(3000);
+}

recipes/net-setSimultaneousAccepts-migration/tests/expected/file-5.js

@@ -0,0 +1,3 @@
+import net from "node:net";
+
+const server = net.createServer();

recipes/net-setSimultaneousAccepts-migration/tests/expected/file-6.js

@@ -0,0 +1,9 @@
+const net = require("node:net");
+
+const config = {
+  port: 8080
+};
+
+function setupServer(config) {
+  return net.createServer().listen(config.port);
+}

recipes/net-setSimultaneousAccepts-migration/tests/input/file-1.js

@@ -0,0 +1,4 @@
+const net = require("node:net");
+
+net._setSimultaneousAccepts(false);
+const server = net.createServer();

recipes/net-setSimultaneousAccepts-migration/tests/input/file-2.js

@@ -0,0 +1,8 @@
+const net = require("node:net");
+
+function createServer() {
+  net._setSimultaneousAccepts(true);
+  return net.createServer((socket) => {
+    // handle connection
+  });
+}

recipes/net-setSimultaneousAccepts-migration/tests/input/file-3.js

@@ -0,0 +1,6 @@
+const net = require("node:net");
+
+net._setSimultaneousAccepts(false);
+module.exports = {
+  createServer: () => net.createServer()
+};

recipes/net-setSimultaneousAccepts-migration/tests/input/file-4.js

@@ -0,0 +1,7 @@
+const net = require("node:net");
+
+function initializeApp() {
+  net._setSimultaneousAccepts(true);
+  const server = net.createServer();
+  server.listen(3000);
+}

recipes/net-setSimultaneousAccepts-migration/tests/input/file-5.js

@@ -0,0 +1,4 @@
+import net from "node:net";
+
+net._setSimultaneousAccepts(false);
+const server = net.createServer();

recipes/net-setSimultaneousAccepts-migration/tests/input/file-6.js

@@ -0,0 +1,11 @@
+const net = require("node:net");
+
+const config = {
+  simultaneousAccepts: false,
+  port: 8080
+};
+
+function setupServer(config) {
+  net._setSimultaneousAccepts(config.simultaneousAccepts);
+  return net.createServer().listen(config.port);
+}

recipes/net-setSimultaneousAccepts-migration/workflow.yaml

@@ -0,0 +1,25 @@
+# yaml-language-server: $schema=https://raw.githubusercontent.com/codemod-com/codemod/refs/heads/main/schemas/workflow.json
+
+version: "1"
+
+nodes:
+  - id: apply-transforms
+    name: Apply AST Transformations
+    type: automatic
+    steps:
+      - name: Handle DEDEP0121 Remove net._setSimultaneousAccepts().
+        js-ast-grep:
+          js_file: src/workflow.ts
+          base_path: .
+          include:
+            - "**/*.js"
+            - "**/*.jsx"
+            - "**/*.mjs"
+            - "**/*.cjs"
+            - "**/*.cts"
+            - "**/*.mts"
+            - "**/*.ts"
+            - "**/*.tsx"
+          exclude:
+            - "**/node_modules/**"
+          language: typescript