refactor: drop unused/replaceable deps (clone, mkdirp, shelljs) (#1994)

azu · claude · web-flow · commit a3f50faa53b9 · 2026-04-09T00:14:55.000+09:00
## Summary

Removes unused or easily replaceable dependencies (`clone`, `mkdirp`,
`shelljs`) across the monorepo. Replaces their usages with Node.js
built-ins (`structuredClone`, `fs.mkdirSync({ recursive: true })`,
`fs`/`child_process`) to slim down the dependency graph.

## Changes

- Drop `clone`, `mkdirp`, and `shelljs` from affected `package.json`
files and `pnpm-lock.yaml`
- Replace `clone()` with `structuredClone()` in tests
- Replace `mkdirp` with `fs.mkdirSync(path, { recursive: true })`
- Replace `shelljs` usages in scripts/tests with Node.js built-ins
- Remove the obsolete
`packages/@textlint/kernel/tools/migrate-core-to-kernel.js` script
- Adjust `pnpm-workspace.yaml` accordingly

## Breaking Changes

None. Only internal scripts, tests, and dev/runtime dependencies are
affected.

## Test Plan

- [ ] `pnpm install`
- [ ] `pnpm -r run build`
- [ ] `pnpm -r run test`
- [ ] CI passes

---------

Co-authored-by: Claude &lt;noreply@anthropic.com&gt;
diff --git a/examples/perf/package.json b/examples/perf/package.json
@@ -15,7 +15,6 @@
     "perf": "node perf.js"
   },
   "devDependencies": {
-    "shelljs": "catalog:",
     "textlint": "workspace:*",
     "textlint-rule-max-ten": "catalog:",
     "textlint-rule-no-mix-dearu-desumasu": "catalog:",
diff --git a/examples/perf/perf.js b/examples/perf/perf.js
@@ -1,8 +1,8 @@
 // LICENSE : MIT
 "use strict";
-require("shelljs/make");
 const os = require("node:os");
-/* eslint-env shelljs */
+const path = require("node:path");
+const { execFile } = require("node:child_process");
 /*
  * A little bit fuzzy. My computer has a first CPU speed of 3093 and the perf test
  * always completes in < 2000ms. However, Travis is less predictable due to
@@ -12,25 +12,27 @@ const os = require("node:os");
 const PERF_MULTIPLIER = 7.5e6;
 /**
  * Calculates the time for each run for performance
- * @param {string} cmd cmd
+ * @param {string} file executable to run
+ * @param {string[]} args arguments to pass to the executable
  * @param {int} runs Total number of runs to do
  * @param {int} runNumber Current run number
  * @param {int[]} results Collection results from each run
  * @param {function} cb Function to call when everything is done
  * @returns {int[]} calls the cb with all the results
  * @private
  */
-function time(cmd, runs, runNumber, results, cb) {
+function time(file, args, runs, runNumber, results, cb) {
     const start = process.hrtime();
-    // eslint-disable-next-line no-undef
-    exec(cmd, { silent: true }, function () {
+    // textlint exits with a non-zero code when lint errors are found; this perf
+    // test only measures execution time, so the exit code is intentionally ignored.
+    execFile(file, args, function () {
         const diff = process.hrtime(start),
             actual = diff[0] * 1e3 + diff[1] / 1e6; // ms
 
         results.push(actual);
-        echo(`Performance Run #${runNumber}:  %dms`, actual); // eslint-disable-line no-undef
+        console.log(`Performance Run #${runNumber}:  %dms`, actual);
         if (runs > 1) {
-            time(cmd, runs - 1, runNumber + 1, results, cb);
+            time(file, args, runs - 1, runNumber + 1, results, cb);
         } else {
             return cb(results);
         }
@@ -40,10 +42,9 @@ function time(cmd, runs, runNumber, results, cb) {
 function run() {
     const cpuSpeed = os.cpus()[0].speed;
     const max = PERF_MULTIPLIER / cpuSpeed;
-    const TEXTLINT = `node ${__dirname}/node_modules/.bin/textlint`;
-    const target = `${__dirname}/md/`;
-    const cmd = `${TEXTLINT} ${target}`;
-    time(cmd, 5, 1, [], function (results) {
+    const textlintBin = path.join(__dirname, "node_modules", ".bin", "textlint");
+    const target = path.join(__dirname, "md");
+    time("node", [textlintBin, target], 5, 1, [], function (results) {
         results.sort(function (a, b) {
             return a - b;
         });
diff --git a/packages/@textlint/kernel/package.json b/packages/@textlint/kernel/package.json
@@ -53,7 +53,6 @@
     "@types/debug": "catalog:",
     "@types/node": "catalog:",
     "rimraf": "catalog:",
-    "shelljs": "catalog:",
     "textlint-rule-preset-ja-spacing": "catalog:",
     "typescript": "catalog:",
     "unist-util-select": "catalog:",
diff --git a/packages/@textlint/kernel/tools/migrate-core-to-kernel.js b/packages/@textlint/kernel/tools/migrate-core-to-kernel.js
diff --git a/packages/@textlint/markdown-to-ast/package.json b/packages/@textlint/markdown-to-ast/package.json
@@ -50,7 +50,6 @@
     "@types/debug": "catalog:",
     "@types/node": "catalog:",
     "@types/unist": "catalog:",
-    "mkdirp": "catalog:",
     "rimraf": "catalog:",
     "typescript": "catalog:",
     "vitest": "catalog:"
diff --git a/packages/@textlint/markdown-to-ast/tools/create-fixtures.js b/packages/@textlint/markdown-to-ast/tools/create-fixtures.js
@@ -7,7 +7,6 @@
  */
 const fs = require("node:fs");
 const path = require("node:path");
-const mkdirp = require("mkdirp");
 const parse = require("../lib/src/index").parse;
 const testDir = path.join(__dirname, "..", "test");
 // remark_fixtures to fixtures
@@ -20,7 +19,7 @@ fs.readdirSync(remarkFixtures).forEach(function (filePath) {
     const originalPath = path.join(remarkFixtures, filePath);
     const dirName = path.basename(filePath);
     const testCaseDir = path.join(fixtureDir, dirName);
-    mkdirp.sync(testCaseDir);
+    fs.mkdirSync(testCaseDir, { recursive: true });
     const testInputFilePath = path.join(fixtureDir, dirName, "input.md");
     fs.renameSync(originalPath, testInputFilePath);
     const input = fs.readFileSync(testInputFilePath, "utf-8");
diff --git a/packages/textlint-scripts/package.json b/packages/textlint-scripts/package.json
@@ -60,8 +60,7 @@
     },
     "devDependencies": {
         "lint-staged": "catalog:",
-        "prettier": "catalog:",
-        "shelljs": "catalog:"
+        "prettier": "catalog:"
     },
     "peerDependencies": {
         "ts-node": "*",
diff --git a/packages/textlint-scripts/test/test-example.js b/packages/textlint-scripts/test/test-example.js
@@ -1,45 +1,25 @@
 const path = require("path");
-const shell = require("shelljs");
+const fs = require("node:fs");
+const { execSync } = require("child_process");
 
 const exampleDir = path.join(__dirname, "../examples/example");
 const exampleTsDir = path.join(__dirname, "../examples/example-ts");
 const exampleDynamicImport = path.join(__dirname, "../examples/example-dynamic-import");
 const examples = [exampleDir, exampleTsDir, exampleDynamicImport];
+
 const exec = (command, { cwd }) => {
     // eslint-disable-next-line no-console
     console.log(`$ ${command}`);
-    if (
-        shell.exec(command, {
-            cwd
-        }).code !== 0
-    ) {
-        shell.exit(1);
-    }
-};
-const cd = (command) => {
-    // eslint-disable-next-line no-console
-    console.log(`$ ${command}`);
-    if (shell.cd(exampleDir).code !== 0) {
-        shell.exit(1);
+    try {
+        execSync(command, { cwd, stdio: "inherit" });
+    } catch {
+        process.exit(1);
     }
 };
+
 for (const example of examples) {
-    // example
-    cd(exampleDir);
-    // installed in monorepo root
-    // exec("npm install");
-    exec("rm -f ./README.md", {
-        cwd: exampleDir
-    });
-    exec("npm run init-readme", {
-        cwd: exampleDir
-    });
-    exec("npm run build", {
-        cwd: exampleDir
-    });
-    exec("npm test", {
-        cwd: exampleDir
-    });
+    fs.rmSync(path.join(example, "README.md"), { force: true });
+    exec("npm run init-readme", { cwd: example });
+    exec("npm run build", { cwd: example });
+    exec("npm test", { cwd: example });
 }
-// exit
-shell.exit(0);
diff --git a/packages/textlint/package.json b/packages/textlint/package.json
@@ -75,13 +75,9 @@
   },
   "devDependencies": {
     "@textlint/legacy-textlint-core": "workspace:*",
-    "@types/clone": "catalog:",
     "@types/debug": "catalog:",
     "@types/node": "catalog:",
-    "@types/shelljs": "catalog:",
-    "clone": "catalog:",
     "rimraf": "catalog:",
-    "shelljs": "catalog:",
     "textlint-plugin-html": "catalog:",
     "textlint-rule-helper": "catalog:",
     "textlint-rule-no-todo": "catalog:",
diff --git a/packages/textlint/test/config-initializer/config-initializer.test.ts b/packages/textlint/test/config-initializer/config-initializer.test.ts
@@ -5,7 +5,6 @@ import { afterEach, beforeEach, describe, it } from "vitest";
 import { Logger } from "../../src/util/logger.js";
 import path from "node:path";
 import os from "node:os";
-import sh from "shelljs";
 import assert from "node:assert";
 import fs from "node:fs";
 
@@ -18,22 +17,22 @@ describe("config-initializer-test", function () {
         const timestamp = Date.now();
         const random = Math.random().toString(36).substring(7);
         configDir = `${os.tmpdir()}/textlint-config-${timestamp}-${random}`;
-        sh.mkdir("-p", configDir);
+        fs.mkdirSync(configDir, { recursive: true });
     });
 
     afterEach(function () {
         if (fs.existsSync(configDir)) {
-            sh.rm("-rf", configDir);
+            fs.rmSync(configDir, { recursive: true, force: true });
         }
     });
     describe("when package.json has textlint-rule-* packages", function () {
         beforeEach(function () {
             // Ensure directory exists and is accessible
             if (!fs.existsSync(configDir)) {
-                sh.mkdir("-p", configDir);
+                fs.mkdirSync(configDir, { recursive: true });
             }
             const packageFilePath = path.resolve(process.cwd(), "test/config-initializer/fixtures/package.json");
-            sh.cp(packageFilePath, configDir);
+            fs.cpSync(packageFilePath, path.join(configDir, path.basename(packageFilePath)));
         });
         it("should create new file with packages", function () {
             return createConfigFile({
diff --git a/packages/textlint/test/execute-file-backers/cache-backer.test.ts b/packages/textlint/test/execute-file-backers/cache-backer.test.ts
@@ -4,7 +4,6 @@ import assert from "node:assert";
 import { describe, it, beforeAll, afterAll } from "vitest";
 import path from "node:path";
 import os from "node:os";
-import sh from "shelljs";
 import fs from "node:fs";
 import { CacheBacker, CacheBackerOptions } from "../../src/engine/execute-file-backers/cache-backer.js";
 import { TextlintMessage } from "@textlint/types";
@@ -13,15 +12,15 @@ describe("CacheBacker", function () {
     let configDir: string;
     beforeAll(function () {
         configDir = path.join(os.tmpdir(), "textlint-config");
-        sh.mkdir("-p", configDir);
+        fs.mkdirSync(configDir, { recursive: true });
     });
     afterAll(function () {
-        sh.rm("-r", configDir);
+        fs.rmSync(configDir, { recursive: true, force: true });
     });
     describe("when previously have success result", function () {
         it("shouldExecute return false", async () => {
             const testCacheDir = path.join(configDir, "success-test");
-            sh.mkdir("-p", testCacheDir);
+            fs.mkdirSync(testCacheDir, { recursive: true });
             const config: CacheBackerOptions = {
                 cache: true,
                 cacheLocation: path.resolve(testCacheDir, ".cache"),
@@ -54,7 +53,7 @@ describe("CacheBacker", function () {
     describe("when previously have failure result", function () {
         it("shouldExecute return true", () => {
             const testCacheDir = path.join(configDir, "failure-test");
-            sh.mkdir("-p", testCacheDir);
+            fs.mkdirSync(testCacheDir, { recursive: true });
             const config: CacheBackerOptions = {
                 cache: true,
                 cacheLocation: path.resolve(testCacheDir, ".cache"),
@@ -77,7 +76,7 @@ describe("CacheBacker", function () {
     describe("when specify `cacheLocation` options", function () {
         it("should save the specific path", () => {
             const testCacheDir = path.join(configDir, "location-test");
-            sh.mkdir("-p", testCacheDir);
+            fs.mkdirSync(testCacheDir, { recursive: true });
             const cacheFilePath = path.resolve(testCacheDir, ".cache");
             const config: CacheBackerOptions = {
                 cache: true,
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/pnpm-workspace.yaml b/pnpm-workspace.yaml
diff --git a/test/integration-test/package.json b/test/integration-test/package.json
diff --git a/test/integration-test/run_lint.js b/test/integration-test/run_lint.js
