@pob/codemods 0.10.10 → 0.11.0

package/CHANGELOG.md

@@ -3,6 +3,21 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [0.11.0](https://github.com/christophehurpeau/pob/compare/@pob/codemods@0.10.11...@pob/codemods@0.11.0) (2026-04-05)
+
+### ⚠ BREAKING CHANGES
+
+* node 24
+
+### Features
+
+* node 24
+
+## [0.10.11](https://github.com/christophehurpeau/pob/compare/@pob/codemods@0.10.10...@pob/codemods@0.10.11) (2026-03-21)
+
+Note: no notable changes
+
+
 ## [0.10.10](https://github.com/christophehurpeau/pob/compare/@pob/codemods@0.10.9...@pob/codemods@0.10.10) (2026-03-12)
 
 Note: no notable changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pob/codemods",
-  "version": "0.10.10",
+  "version": "0.11.0",
   "description": "codemods for pob",
   "keywords": [],
   "author": "Christophe Hurpeau <302891+christophehurpeau@users.noreply.github.com> (https://christophe.hurpeau.com)",
@@ -13,7 +13,7 @@
   "homepage": "https://github.com/christophehurpeau/pob",
   "type": "module",
   "engines": {
-    "node": ">=22.18.0"
+    "node": ">=24.0.0"
   },
   "bin": {
     "pob-migrate": "./bin/migrate.mjs"
@@ -26,10 +26,16 @@
   "publishConfig": {
     "access": "public"
   },
+  "files": [
+    "src",
+    "build",
+    "bin",
+    "lib/index.js"
+  ],
   "scripts": {
     "install-bin": "ln -s $PWD/bin/migrate.mjs /usr/local/bin/pob-migrate",
     "lint": "yarn run lint:eslint",
-    "lint:eslint": "yarn ../.. run eslint --quiet '@pob/codemods'"
+    "lint:eslint": "yarn '../..' run eslint --quiet '@pob/codemods'"
   },
   "pob": {},
   "prettier": "@pob/root/prettier-config",
@@ -38,6 +44,6 @@
   },
   "devDependencies": {
     "@types/jscodeshift": "17.3.0",
-    "@types/node": "22.19.13"
+    "@types/node": "24.12.0"
   }
 }

package/.editorconfig

@@ -1,13 +0,0 @@
-# EditorConfig helps developers define and maintain consistent
-# coding styles between different editors and IDEs
-# http://editorconfig.org
-
-root = true
-
-[*]
-indent_style = space
-indent_size = 2
-end_of_line = lf
-charset = utf-8
-trim_trailing_whitespace = true
-insert_final_newline = true

package/.yo-rc.json

@@ -1,11 +0,0 @@
-{
-  "pob": {
-    "app": {
-      "type": "node",
-      "testing": false
-    },
-    "project": {
-      "type": "app"
-    }
-  }
-}

package/lib/runJscodeshift.cjs

@@ -1,7 +0,0 @@
-"use strict";
-
-const { run: jscodeshift } = require("jscodeshift/src/Runner");
-
-module.exports = (...args) => {
-  return jscodeshift(...args);
-};

package/lib/transforms/jest-to-node-runner.cjs

@@ -1,146 +0,0 @@
-/* eslint-disable unicorn/prefer-array-some */
-/* eslint-disable unicorn/no-array-method-this-argument */
-
-"use strict";
-
-const { prependImport } = require("../utils/prependImport.cjs");
-
-const jestToNodeGlobalMethods = {
-  test: "test",
-  describe: "describe",
-  it: "it",
-  beforeAll: "before",
-  afterAll: "after",
-  beforeEach: "beforeEach",
-  afterEach: "afterEach",
-};
-const propertiesMap = {
-  toBe: { default: "equal", not: "notEqual" },
-  toStrictEqual: { default: "equal", not: "notEqual" },
-  toEqual: { default: "equal", not: "notEqual" },
-};
-
-// const mocks = {
-//   toHaveBeenCalledTimes: {
-//     default: 'mock.calls.length',
-//   },
-// };
-
-module.exports = (file, api, options) => {
-  if (
-    !(
-      file.path.endsWith(".test.js") ||
-      file.path.endsWith(".test.ts") ||
-      file.path.endsWith(".test.tsx")
-    )
-  ) {
-    throw new Error(`Invalid file: "${file.path}"`);
-  }
-
-  const j = api.jscodeshift.withParser("tsx");
-  const root = j(file.source);
-
-  function updateTopLevelMethods() {
-    const topLevelMethodsUsed = new Set();
-
-    root
-      .find(j.CallExpression, {
-        callee: {
-          type: j.Identifier.name,
-          name: (name) => Object.keys(jestToNodeGlobalMethods).includes(name),
-        },
-      })
-      .replaceWith((p) => {
-        const { value } = p;
-        const replaceValue = jestToNodeGlobalMethods[value.callee.name];
-        topLevelMethodsUsed.add(replaceValue);
-
-        return j.callExpression(j.identifier(replaceValue), value.arguments);
-      });
-
-    if (topLevelMethodsUsed.size > 0) {
-      prependImport(
-        j,
-        root,
-        j.importDeclaration(
-          [...topLevelMethodsUsed].map((method) =>
-            j.importSpecifier(j.identifier(method)),
-          ),
-          j.stringLiteral("node:test"),
-        ),
-      );
-    }
-  }
-
-  function updateAssertions() {
-    let hasAssertions = false;
-
-    root
-      .find(j.MemberExpression, {
-        object: {
-          callee: { name: "expect" },
-        },
-      })
-      .forEach((memberExpressionPath) => {
-        const { value: memberExpression, parentPath } = memberExpressionPath;
-        let memberExpressionPathToReplace = memberExpressionPath;
-        let propertyName = memberExpression.property.name;
-        let not = false;
-        if (propertyName === "not") {
-          not = true;
-          memberExpressionPathToReplace = parentPath;
-          propertyName = parentPath.value.property.name;
-        }
-
-        if (!propertiesMap[propertyName]) {
-          console.warn(
-            `Unsupported expect property: ${propertyName} in ${file.path}`,
-          );
-          return;
-        }
-
-        const assertPropertyName =
-          propertiesMap[propertyName][not ? "not" : "default"];
-        const callExpressionPath = memberExpressionPathToReplace.parentPath;
-
-        const value = memberExpression.object.arguments[0];
-        const expected = callExpressionPath.value.arguments[0];
-
-        hasAssertions = true;
-        callExpressionPath.value.callee = j.memberExpression(
-          j.identifier("assert"),
-          j.identifier(assertPropertyName),
-        );
-        callExpressionPath.value.arguments = [value, expected];
-      });
-
-    if (hasAssertions) {
-      root
-        .find(j.ImportDeclaration, { source: { value: "node:assert" } })
-        .remove();
-      if (
-        // eslint-disable-next-line no-constant-condition
-        (!root.find(j.ImportDeclaration),
-        { source: { value: "node:assert/strict" } })
-      ) {
-        prependImport(
-          j,
-          root,
-          j.importDeclaration(
-            [j.importDefaultSpecifier(j.identifier("assert"))],
-            j.stringLiteral("node:assert/strict"),
-          ),
-        );
-      }
-    }
-  }
-
-  updateTopLevelMethods();
-  updateAssertions();
-
-  root
-    .find(j.ImportDeclaration, { source: { value: "@jest/globals" } })
-    .remove();
-
-  return root.toSource(options);
-};

package/lib/transforms/update-function-component-return-type.cjs

@@ -1,89 +0,0 @@
-"use strict";
-
-/* eslint-disable complexity */
-module.exports = (file, api, options) => {
-  const j = api.jscodeshift.withParser("tsx");
-  const root = j(file.source);
-  let hasModifications = false;
-
-  function processFunctionComponent(path) {
-    const node = path.value;
-    const { returnType } = node;
-    if (returnType == null) return;
-    if (returnType.type !== "TSTypeAnnotation") return;
-    const { typeAnnotation } = returnType;
-    if (typeAnnotation == null) return;
-
-    if (
-      typeAnnotation.type === "TSTypeReference" &&
-      typeAnnotation.typeName &&
-      typeAnnotation.typeName.type === "Identifier"
-    ) {
-      if (typeAnnotation.typeName.name === "ReactElement") {
-        typeAnnotation.typeName.name = "ReactNode";
-        hasModifications = true;
-      }
-    } else if (
-      typeAnnotation.type === "TSUnionType" &&
-      typeAnnotation.types.every((unionTypeItem) => {
-        if (unionTypeItem.type === "TSTypeReference") {
-          return (
-            unionTypeItem.typeName &&
-            unionTypeItem.typeName.type === "Identifier" &&
-            unionTypeItem.typeName.name === "ReactElement"
-          );
-        }
-
-        if (unionTypeItem.type === "TSNullKeyword") {
-          return true;
-        }
-
-        return false;
-      })
-    ) {
-      returnType.typeAnnotation = j.tsTypeReference(j.identifier("ReactNode"));
-      hasModifications = true;
-    }
-  }
-
-  root.find(j.FunctionDeclaration).forEach(processFunctionComponent);
-
-  if (hasModifications) {
-    root.find(j.ImportDeclaration).forEach((importDeclaration) => {
-      const { node } = importDeclaration;
-      if (node.source.value !== "react") return;
-      if (node.importKind !== "type") return;
-
-      node.specifiers = node.specifiers.filter((importSpecifier) => {
-        if (importSpecifier.local.name !== "ReactElement") return true;
-        if (importSpecifier.importKind !== "value") return true;
-        return (
-          j(importDeclaration)
-            .closestScope()
-            // eslint-disable-next-line unicorn/no-array-method-this-argument -- not an array
-            .find(j.Identifier, { name: "ReactElement" })
-            .filter((p) => {
-              if (p.value.start === importSpecifier.local.start) return false;
-              if (p.parentPath.value.type === "Property" && p.name === "key") {
-                return false;
-              }
-              if (p.name === "property") return false;
-              return true;
-            })
-            .size() > 0
-        );
-      });
-      if (
-        !node.specifiers.some(
-          (specifier) =>
-            specifier.type === "ImportSpecifier" &&
-            specifier.imported.name === "ReactNode",
-        )
-      ) {
-        node.specifiers.push(j.importSpecifier(j.identifier("ReactNode")));
-      }
-    });
-  }
-
-  return hasModifications ? root.toSource(options) : null;
-};

package/lib/utils/prependImport.cjs

@@ -1,35 +0,0 @@
-"use strict";
-
-exports.prependImport = (j, source, importDeclaration) => {
-  const existingImports = source.find(j.ImportDeclaration);
-  if (existingImports.length > 0) {
-    const firstImport = existingImports.at(0);
-    const firstImportNode = firstImport.nodes()[0];
-    if (firstImportNode?.comments) {
-      importDeclaration.comments = firstImportNode.comments;
-      firstImportNode.comments = null;
-    }
-
-    firstImport.insertBefore(importDeclaration);
-    return;
-  }
-
-  const firstNode = source.find(j.Program).get("body", 0).node;
-  const { comments } = firstNode;
-  if (comments?.length) {
-    const comment = comments[0];
-
-    // Only move comments that look like file-level comments. Ignore
-    // line-level and JSDoc-style comments because these probably belong
-    // to the first node, rather than the file.
-    if (
-      (comment.type === "Block" || comment.type === "CommentBlock") &&
-      !comment.value.startsWith("*")
-    ) {
-      importDeclaration.comments = comments;
-      firstNode.comments = null;
-    }
-  }
-
-  source.get("program", "body").unshift(importDeclaration);
-};