@xylabs/ts-scripts-yarn3 7.3.2 → 7.4.1

package/dist/actions/deplint/index.mjs

@@ -1,5 +1,5 @@
 // src/actions/deplint/deplint.ts
-import chalk5 from "chalk";
+import chalk6 from "chalk";
 
 // src/lib/yarn/workspace/yarnWorkspaces.ts
 import { spawnSync } from "child_process";
@@ -27,12 +27,18 @@ function findFilesByGlob(cwd, pattern) {
 }
 
 // src/actions/deplint/findFiles.ts
-function findFiles(path3) {
-  const allSourceInclude = ["./src/**/*.{ts,tsx}"];
+function findFiles(path5) {
+  const allSourceInclude = ["./src/**/*.{ts,tsx,mts,cts,js,mjs,cjs}"];
   const allDistInclude = ["./dist/**/*.d.ts", "./dist/**/*.{mjs,js,cjs}"];
-  const srcFiles = allSourceInclude.flatMap((pattern) => findFilesByGlob(path3, pattern));
-  const distFiles = allDistInclude.flatMap((pattern) => findFilesByGlob(path3, pattern));
-  return { srcFiles, distFiles };
+  const allConfigInclude = ["./*.config.{ts,mts,mjs,js}"];
+  const srcFiles = allSourceInclude.flatMap((pattern) => findFilesByGlob(path5, pattern));
+  const distFiles = allDistInclude.flatMap((pattern) => findFilesByGlob(path5, pattern));
+  const configFiles = allConfigInclude.flatMap((pattern) => findFilesByGlob(path5, pattern));
+  return {
+    srcFiles,
+    distFiles,
+    configFiles
+  };
 }
 
 // src/actions/deplint/getDependenciesFromPackageJson.ts
@@ -52,10 +58,9 @@ function getDependenciesFromPackageJson(packageJsonPath) {
   };
 }
 
-// src/actions/deplint/getImportsFromFile.ts
+// src/actions/deplint/getExtendsFromTsconfigs.ts
 import fs2 from "fs";
-import path2 from "path";
-import ts from "typescript";
+import { globSync as globSync2 } from "glob";
 
 // src/actions/deplint/getBasePackageName.ts
 function getBasePackageName(importName) {
@@ -67,7 +72,37 @@ function getBasePackageName(importName) {
   return importNameScrubbed.split("/")[0];
 }
 
+// src/actions/deplint/getExtendsFromTsconfigs.ts
+var isExternalReference = (ref) => !ref.startsWith(".") && !ref.startsWith("/");
+function parseExtendsField(value) {
+  if (typeof value === "string") return [value];
+  if (Array.isArray(value)) return value.filter((v) => typeof v === "string");
+  return [];
+}
+function getExtendsFromTsconfigs(location) {
+  const tsconfigFiles = globSync2("./tsconfig*.json", { cwd: location, absolute: true });
+  const packages = /* @__PURE__ */ new Set();
+  for (const file of tsconfigFiles) {
+    try {
+      const content = fs2.readFileSync(file, "utf8");
+      const cleaned = content.replaceAll(/\/\/.*/g, "").replaceAll(/,\s*([}\]])/g, "$1");
+      const parsed = JSON.parse(cleaned);
+      const refs = parseExtendsField(parsed.extends);
+      for (const ref of refs) {
+        if (isExternalReference(ref)) {
+          packages.add(getBasePackageName(ref));
+        }
+      }
+    } catch {
+    }
+  }
+  return [...packages];
+}
+
 // src/actions/deplint/getImportsFromFile.ts
+import fs3 from "fs";
+import path2 from "path";
+import ts from "typescript";
 function isTypeOnlyImportClause(clause) {
   if (clause === void 0) {
     return false;
@@ -80,7 +115,7 @@ function isTypeOnlyImportClause(clause) {
   return clause.isTypeOnly;
 }
 function getImportsFromFile(filePath, importPaths, typeImportPaths) {
-  const sourceCode = fs2.readFileSync(filePath, "utf8");
+  const sourceCode = fs3.readFileSync(filePath, "utf8");
   const isMjsFile = filePath.endsWith(".mjs");
   const sourceFile = ts.createSourceFile(
     path2.basename(filePath),
@@ -133,24 +168,38 @@ var internalImportPrefixes = [".", "#", "node:"];
 var removeInternalImports = (imports) => {
   return imports.filter((imp) => !internalImportPrefixes.some((prefix) => imp.startsWith(prefix)));
 };
-function getExternalImportsFromFiles({ srcFiles, distFiles }) {
+function getExternalImportsFromFiles({
+  srcFiles,
+  distFiles,
+  configFiles = [],
+  tsconfigExtends = []
+}) {
   const srcImportPaths = {};
   const distImportPaths = {};
   const distTypeImportPaths = {};
-  for (const path3 of srcFiles) getImportsFromFile(path3, srcImportPaths, srcImportPaths).flat();
+  const configImportPaths = {};
+  for (const path5 of srcFiles) getImportsFromFile(path5, srcImportPaths, srcImportPaths).flat();
+  for (const path5 of configFiles) getImportsFromFile(path5, configImportPaths, configImportPaths).flat();
   const distTypeFiles = distFiles.filter((file) => file.endsWith(".d.ts") || file.endsWith(".d.cts") || file.endsWith(".d.mts"));
   const distCodeFiles = distFiles.filter((file) => !(file.endsWith(".d.ts") || file.endsWith(".d.cts") || file.endsWith(".d.mts")));
-  for (const path3 of distCodeFiles) getImportsFromFile(path3, distImportPaths, distImportPaths).flat();
-  for (const path3 of distTypeFiles) getImportsFromFile(path3, distTypeImportPaths, distTypeImportPaths).flat();
+  for (const path5 of distCodeFiles) getImportsFromFile(path5, distImportPaths, distImportPaths).flat();
+  for (const path5 of distTypeFiles) getImportsFromFile(path5, distTypeImportPaths, distTypeImportPaths).flat();
   const srcImports = Object.keys(srcImportPaths);
   const distImports = Object.keys(distImportPaths);
   const distTypeImports = Object.keys(distTypeImportPaths);
   const externalSrcImports = removeInternalImports(srcImports);
   const externalDistImports = removeInternalImports(distImports);
   const externalDistTypeImports = removeInternalImports(distTypeImports);
+  const externalConfigImports = removeInternalImports(Object.keys(configImportPaths));
+  for (const ext of tsconfigExtends) {
+    if (!externalSrcImports.includes(ext)) externalSrcImports.push(ext);
+    if (!externalConfigImports.includes(ext)) externalConfigImports.push(ext);
+  }
   return {
+    configImportPaths,
     srcImports,
     srcImportPaths,
+    externalConfigImports,
     externalSrcImports,
     distImports,
     distImportPaths,
@@ -162,6 +211,15 @@ function getExternalImportsFromFiles({ srcFiles, distFiles }) {
 // src/actions/deplint/checkPackage/getUnlistedDependencies.ts
 import { builtinModules } from "module";
 import chalk from "chalk";
+function isListedOrBuiltin(imp, name, dependencies, peerDependencies) {
+  return dependencies.includes(imp) || imp === name || dependencies.includes(`@types/${imp}`) || peerDependencies.includes(imp) || peerDependencies.includes(`@types/${imp}`) || builtinModules.includes(imp) || builtinModules.includes(`@types/${imp}`);
+}
+function logMissing(name, imp, importPaths) {
+  console.log(`[${chalk.blue(name)}] Missing dependency in package.json: ${chalk.red(imp)}`);
+  if (importPaths[imp]) {
+    console.log(`  ${importPaths[imp].join("\n ")}`);
+  }
+}
 function getUnlistedDependencies({ name, location }, { dependencies, peerDependencies }, {
   externalDistImports,
   externalDistTypeImports,
@@ -169,17 +227,15 @@ function getUnlistedDependencies({ name, location }, { dependencies, peerDepende
 }) {
   let unlistedDependencies = 0;
   for (const imp of externalDistImports) {
-    if (!dependencies.includes(imp) && imp !== name && !dependencies.includes(`@types/${imp}`) && !peerDependencies.includes(imp) && !peerDependencies.includes(`@types/${imp}`) && !builtinModules.includes(imp) && !builtinModules.includes(`@types/${imp}`)) {
+    if (!isListedOrBuiltin(imp, name, dependencies, peerDependencies)) {
       unlistedDependencies++;
-      console.log(`[${chalk.blue(name)}] Missing dependency in package.json: ${chalk.red(imp)}`);
-      console.log(`  ${distImportPaths[imp].join("\n ")}`);
+      logMissing(name, imp, distImportPaths);
     }
   }
   for (const imp of externalDistTypeImports) {
-    if (!dependencies.includes(imp) && imp !== name && dependencies.includes(`@types/${imp}`) && !peerDependencies.includes(imp) && peerDependencies.includes(`@types/${imp}`) && !builtinModules.includes(imp) && builtinModules.includes(`@types/${imp}`)) {
+    if (!isListedOrBuiltin(imp, name, dependencies, peerDependencies)) {
       unlistedDependencies++;
-      console.log(`[${chalk.blue(name)}] Missing dependency in package.json: ${chalk.red(imp)}`);
-      console.log(`  ${distImportPaths[imp].join("\n ")}`);
+      logMissing(name, imp, distImportPaths);
     }
   }
   if (unlistedDependencies > 0) {
@@ -207,7 +263,9 @@ function getUnlistedDevDependencies({ name, location }, {
     if (!distImports.includes(imp) && imp !== name && !dependencies.includes(imp) && !dependencies.includes(`@types/${imp}`) && !peerDependencies.includes(imp) && !peerDependencies.includes(`@types/${imp}`) && !devDependencies.includes(imp) && !devDependencies.includes(`@types/${imp}`) && !builtinModules2.includes(imp)) {
       unlistedDevDependencies++;
       console.log(`[${chalk2.blue(name)}] Missing devDependency in package.json: ${chalk2.red(imp)}`);
-      console.log(`  ${srcImportPaths[imp].join("\n ")}`);
+      if (srcImportPaths[imp]) {
+        console.log(`  ${srcImportPaths[imp].join("\n ")}`);
+      }
     }
   }
   if (unlistedDevDependencies > 0) {
@@ -244,29 +302,243 @@ function getUnusedDependencies({ name, location }, { dependencies }, {
   return unusedDependencies;
 }
 
-// src/actions/deplint/checkPackage/getUnusedPeerDependencies.ts
+// src/actions/deplint/checkPackage/getUnusedDevDependencies.ts
 import chalk4 from "chalk";
+
+// src/actions/deplint/getRequiredPeerDependencies.ts
+import fs4 from "fs";
+import path3 from "path";
+function findDepPackageJson(location, dep) {
+  let dir = location;
+  while (true) {
+    const candidate = path3.join(dir, "node_modules", dep, "package.json");
+    if (fs4.existsSync(candidate)) return candidate;
+    const parent = path3.dirname(dir);
+    if (parent === dir) return void 0;
+    dir = parent;
+  }
+}
+function getRequiredPeerDependencies(location, allDeps) {
+  const required = /* @__PURE__ */ new Set();
+  for (const dep of allDeps) {
+    const depPkgPath = findDepPackageJson(location, dep);
+    if (!depPkgPath) continue;
+    try {
+      const raw = fs4.readFileSync(depPkgPath, "utf8");
+      const pkg = JSON.parse(raw);
+      if (pkg.peerDependencies) {
+        for (const peer of Object.keys(pkg.peerDependencies)) {
+          required.add(peer);
+        }
+      }
+    } catch {
+    }
+  }
+  return required;
+}
+
+// src/actions/deplint/getScriptReferencedPackages.ts
+import fs5 from "fs";
+import path4 from "path";
+function getBinNames(location, dep) {
+  const depPkgPath = findDepPackageJson(location, dep);
+  if (!depPkgPath) return [];
+  try {
+    const raw = fs5.readFileSync(depPkgPath, "utf8");
+    const pkg = JSON.parse(raw);
+    if (!pkg.bin) return [];
+    if (typeof pkg.bin === "string") return [pkg.name?.split("/").pop() ?? dep];
+    return Object.keys(pkg.bin);
+  } catch {
+    return [];
+  }
+}
+function tokenizeScript(script) {
+  return script.split(/[&|;$()"`\s]+/).map((t) => t.trim()).filter(Boolean);
+}
+function getScriptReferencedPackages(location, allDeps) {
+  const pkgPath = path4.join(location, "package.json");
+  let scripts = {};
+  try {
+    const raw = fs5.readFileSync(pkgPath, "utf8");
+    const pkg = JSON.parse(raw);
+    scripts = pkg.scripts ?? {};
+  } catch {
+    return /* @__PURE__ */ new Set();
+  }
+  const scriptText = Object.values(scripts).join(" ");
+  const tokens = new Set(tokenizeScript(scriptText));
+  const binToPackage = /* @__PURE__ */ new Map();
+  for (const dep of allDeps) {
+    const bins = getBinNames(location, dep);
+    for (const bin of bins) {
+      binToPackage.set(bin, dep);
+    }
+  }
+  const referenced = /* @__PURE__ */ new Set();
+  for (const token of tokens) {
+    const baseName = getBasePackageName(token);
+    if (allDeps.includes(baseName)) {
+      referenced.add(baseName);
+    }
+    const pkg = binToPackage.get(token);
+    if (pkg) {
+      referenced.add(pkg);
+    }
+  }
+  return referenced;
+}
+
+// src/actions/deplint/implicitDevDependencies.ts
+import fs6 from "fs";
+var hasFileWithExtension = (files, extensions) => files.some((f) => extensions.some((ext) => f.endsWith(ext)));
+var tsExtensions = [".ts", ".tsx", ".mts", ".cts"];
+var hasTypescriptFiles = ({ srcFiles, configFiles }) => hasFileWithExtension([...srcFiles, ...configFiles], tsExtensions);
+var decoratorPattern = /^\s*@[A-Z]\w*/m;
+var hasDecorators = ({ srcFiles }) => srcFiles.filter((f) => tsExtensions.some((ext) => f.endsWith(ext))).some((file) => {
+  try {
+    const content = fs6.readFileSync(file, "utf8");
+    return decoratorPattern.test(content);
+  } catch {
+    return false;
+  }
+});
+var importPlugins = /* @__PURE__ */ new Set(["eslint-plugin-import-x", "eslint-plugin-import"]);
+function hasImportPlugin({ location, allDependencies }) {
+  if (allDependencies.some((d) => importPlugins.has(d))) return true;
+  for (const dep of allDependencies) {
+    const pkgPath = findDepPackageJson(location, dep);
+    if (!pkgPath) continue;
+    try {
+      const pkg = JSON.parse(fs6.readFileSync(pkgPath, "utf8"));
+      const transitiveDeps = [
+        ...Object.keys(pkg.dependencies ?? {}),
+        ...Object.keys(pkg.peerDependencies ?? {})
+      ];
+      if (transitiveDeps.some((d) => importPlugins.has(d))) return true;
+    } catch {
+    }
+  }
+  return false;
+}
+var rules = [
+  {
+    package: "typescript",
+    isNeeded: hasTypescriptFiles
+  },
+  {
+    package: "eslint-import-resolver-typescript",
+    isNeeded: (context) => hasTypescriptFiles(context) && context.allDependencies.includes("eslint") && hasImportPlugin(context)
+  },
+  {
+    package: "tslib",
+    isNeeded: hasDecorators
+  }
+];
+function getImplicitDevDependencies(context) {
+  const implicit = /* @__PURE__ */ new Set();
+  for (const rule of rules) {
+    if (rule.isNeeded(context)) {
+      implicit.add(rule.package);
+    }
+  }
+  return implicit;
+}
+
+// src/actions/deplint/checkPackage/getUnusedDevDependencies.ts
+var allExternalImports = ({
+  externalSrcImports,
+  externalDistImports,
+  externalDistTypeImports,
+  externalConfigImports
+}) => {
+  const all = /* @__PURE__ */ new Set([
+    ...externalSrcImports,
+    ...externalDistImports,
+    ...externalDistTypeImports,
+    ...externalConfigImports
+  ]);
+  return all;
+};
+function isDevDepUsed(dep, allImports, implicitDeps, requiredPeers, scriptRefs) {
+  if (implicitDeps.has(dep)) return true;
+  if (requiredPeers.has(dep)) return true;
+  if (scriptRefs.has(dep)) return true;
+  if (dep.startsWith("@types/")) {
+    const baseName = dep.replace(/^@types\//, "");
+    return allImports.has(baseName) || allImports.has(dep) || implicitDeps.has(baseName);
+  }
+  return allImports.has(dep);
+}
+function getUnusedDevDependencies({ name, location }, {
+  devDependencies,
+  dependencies,
+  peerDependencies
+}, sourceParams, fileContext) {
+  const allImports = allExternalImports(sourceParams);
+  const allDeps = [...dependencies, ...devDependencies, ...peerDependencies];
+  const implicitDeps = getImplicitDevDependencies({
+    ...fileContext,
+    allDependencies: allDeps,
+    location
+  });
+  const requiredPeers = getRequiredPeerDependencies(location, allDeps);
+  const scriptRefs = getScriptReferencedPackages(location, allDeps);
+  let unusedDevDependencies = 0;
+  for (const dep of devDependencies) {
+    if (dependencies.includes(dep) || peerDependencies.includes(dep)) continue;
+    if (!isDevDepUsed(dep, allImports, implicitDeps, requiredPeers, scriptRefs)) {
+      unusedDevDependencies++;
+      console.log(`[${chalk4.blue(name)}] Unused devDependency in package.json: ${chalk4.red(dep)}`);
+    }
+  }
+  if (unusedDevDependencies > 0) {
+    const packageLocation = `${location}/package.json`;
+    console.log(`  ${chalk4.yellow(packageLocation)}
+`);
+  }
+  return unusedDevDependencies;
+}
+
+// src/actions/deplint/checkPackage/getUnusedPeerDependencies.ts
+import chalk5 from "chalk";
 function getUnusedPeerDependencies({ name, location }, { peerDependencies, dependencies }, { externalDistImports, externalDistTypeImports }) {
   let unusedDependencies = 0;
   for (const dep of peerDependencies) {
     if (!externalDistImports.includes(dep) && !externalDistImports.includes(dep.replace(/^@types\//, "")) && !externalDistTypeImports.includes(dep) && !externalDistTypeImports.includes(dep.replace(/^@types\//, ""))) {
       unusedDependencies++;
       if (dependencies.includes(dep)) {
-        console.log(`[${chalk4.blue(name)}] Unused peerDependency [already a dependency] in package.json: ${chalk4.red(dep)}`);
+        console.log(`[${chalk5.blue(name)}] Unused peerDependency [already a dependency] in package.json: ${chalk5.red(dep)}`);
       } else {
-        console.log(`[${chalk4.blue(name)}] Unused peerDependency in package.json: ${chalk4.red(dep)}`);
+        console.log(`[${chalk5.blue(name)}] Unused peerDependency in package.json: ${chalk5.red(dep)}`);
       }
     }
   }
   if (unusedDependencies > 0) {
     const packageLocation = `${location}/package.json`;
-    console.log(`  ${chalk4.yellow(packageLocation)}
+    console.log(`  ${chalk5.yellow(packageLocation)}
 `);
   }
   return unusedDependencies;
 }
 
 // src/actions/deplint/checkPackage/checkPackage.ts
+function logVerbose(name, location, srcFiles, distFiles, configFiles, tsconfigExtends) {
+  console.info(`Checking package: ${name} at ${location}`);
+  console.info(`Source files: ${srcFiles.length}, Distribution files: ${distFiles.length}, Config files: ${configFiles.length}`);
+  for (const file of srcFiles) {
+    console.info(`Source file: ${file}`);
+  }
+  for (const file of distFiles) {
+    console.info(`Distribution file: ${file}`);
+  }
+  for (const file of configFiles) {
+    console.info(`Config file: ${file}`);
+  }
+  for (const ext of tsconfigExtends) {
+    console.info(`Tsconfig extends: ${ext}`);
+  }
+}
 function checkPackage({
   name,
   location,
@@ -275,27 +547,36 @@ function checkPackage({
   peerDeps = false,
   verbose = false
 }) {
-  const { srcFiles, distFiles } = findFiles(location);
+  const {
+    srcFiles,
+    distFiles,
+    configFiles
+  } = findFiles(location);
+  const tsconfigExtends = getExtendsFromTsconfigs(location);
   if (verbose) {
-    console.info(`Checking package: ${name} at ${location}`);
-    console.info(`Source files: ${srcFiles.length}, Distribution files: ${distFiles.length}`);
-    for (const file of srcFiles) {
-      console.info(`Source file: ${file}`);
-    }
-    for (const file of distFiles) {
-      console.info(`Distribution file: ${file}`);
-    }
+    logVerbose(name, location, srcFiles, distFiles, configFiles, tsconfigExtends);
   }
   const checkDeps = deps || !(deps || devDeps || peerDeps);
   const checkDevDeps = devDeps || !(deps || devDeps || peerDeps);
   const checkPeerDeps = peerDeps;
-  const sourceParams = getExternalImportsFromFiles({ srcFiles, distFiles });
+  const sourceParams = getExternalImportsFromFiles({
+    srcFiles,
+    distFiles,
+    configFiles,
+    tsconfigExtends
+  });
   const packageParams = getDependenciesFromPackageJson(`${location}/package.json`);
   const unlistedDependencies = checkDeps ? getUnlistedDependencies({ name, location }, packageParams, sourceParams) : 0;
   const unusedDependencies = checkDeps ? getUnusedDependencies({ name, location }, packageParams, sourceParams) : 0;
   const unlistedDevDependencies = checkDevDeps ? getUnlistedDevDependencies({ name, location }, packageParams, sourceParams) : 0;
+  const fileContext = {
+    configFiles,
+    distFiles,
+    srcFiles
+  };
+  const unusedDevDependencies = checkDevDeps ? getUnusedDevDependencies({ name, location }, packageParams, sourceParams, fileContext) : 0;
   const unusedPeerDependencies = checkPeerDeps ? getUnusedPeerDependencies({ name, location }, packageParams, sourceParams) : 0;
-  const totalErrors = unlistedDependencies + unlistedDevDependencies + unusedDependencies + unusedPeerDependencies;
+  const totalErrors = unlistedDependencies + unlistedDevDependencies + unusedDependencies + unusedDevDependencies + unusedPeerDependencies;
   return totalErrors;
 }
 
@@ -333,9 +614,9 @@ var deplint = ({
     });
   }
   if (totalErrors > 0) {
-    console.warn(`Deplint: Found ${chalk5.red(totalErrors)} dependency problems. ${chalk5.red("\u2716")}`);
+    console.warn(`Deplint: Found ${chalk6.red(totalErrors)} dependency problems. ${chalk6.red("\u2716")}`);
   } else {
-    console.info(`Deplint: Found no dependency problems. ${chalk5.green("\u2714")}`);
+    console.info(`Deplint: Found no dependency problems. ${chalk6.green("\u2714")}`);
   }
   return 0;
 };