@xylabs/ts-scripts-yarn3 6.5.6 → 6.5.8

package/dist/bin/xy.mjs

@@ -20,7 +20,7 @@ var checkResult = (name, result, level = "error", exitOnFail = false) => {
 };
 
 // src/lib/deleteGlob.ts
-import fs from "node:fs";
+import fs from "fs";
 import { glob } from "glob";
 var deleteGlob = (globPath) => {
   const files = glob.sync(globPath);
@@ -30,7 +30,7 @@ var deleteGlob = (globPath) => {
 };
 
 // src/lib/dependencies/detectDuplicateDependencies.ts
-import { execSync } from "node:child_process";
+import { execSync } from "child_process";
 
 // src/lib/processEx.ts
 import chalk2 from "chalk";
@@ -90,7 +90,7 @@ var safeExitAsync = async (func, exitOnFail = true) => {
 };
 
 // src/lib/dependencies/DuplicateDetector.ts
-import { EOL } from "node:os";
+import { EOL } from "os";
 import chalk3 from "chalk";
 
 // src/lib/jsonFormatters.ts
@@ -210,7 +210,7 @@ import {
   existsSync,
   readFileSync,
   writeFileSync
-} from "node:fs";
+} from "fs";
 
 // src/lib/string/empty.ts
 var empty = (value) => value?.trim().length === 0;
@@ -235,7 +235,7 @@ var writeLines = (uri, lines, options = defaultReadFileSyncOptions) => {
 import chalk4 from "chalk";
 
 // src/lib/yarn/isYarnVersionOrGreater.ts
-import { spawnSync } from "node:child_process";
+import { spawnSync } from "child_process";
 var isYarnVersionOrGreater = (major, minor, patch) => {
   const result = spawnSync("yarn", ["-v"], { encoding: "utf8", shell: true });
   const version = result.stdout.toString().replaceAll("\n", "");
@@ -250,7 +250,7 @@ var isYarnVersionOrGreater = (major, minor, patch) => {
 };
 
 // src/lib/yarn/workspace/yarnWorkspaces.ts
-import { spawnSync as spawnSync2 } from "node:child_process";
+import { spawnSync as spawnSync2 } from "child_process";
 var yarnWorkspaces = () => {
   const result = spawnSync2("yarn", ["workspaces", "list", "--json", "--recursive"], { encoding: "utf8", shell: true });
   if (result.error) {
@@ -298,7 +298,7 @@ var generateIgnoreFiles = (filename3, pkg) => {
 };
 
 // src/lib/parsedPackageJSON.ts
-import { readFileSync as readFileSync2 } from "node:fs";
+import { readFileSync as readFileSync2 } from "fs";
 var parsedPackageJSON = (path5) => {
   const pathToPackageJSON = path5 ?? process.env.npm_package_json ?? "";
   const packageJSON = readFileSync2(pathToPackageJSON).toString();
@@ -306,8 +306,8 @@ var parsedPackageJSON = (path5) => {
 };
 
 // src/lib/runSteps.ts
-import { spawnSync as spawnSync3 } from "node:child_process";
-import { existsSync as existsSync2 } from "node:fs";
+import { spawnSync as spawnSync3 } from "child_process";
+import { existsSync as existsSync2 } from "fs";
 import chalk5 from "chalk";
 var runSteps = (name, steps, exitOnFail = true, messages) => {
   return safeExit(() => {
@@ -337,8 +337,8 @@ var runSteps = (name, steps, exitOnFail = true, messages) => {
 };
 
 // src/lib/runStepsAsync.ts
-import { spawn } from "node:child_process";
-import { existsSync as existsSync3 } from "node:fs";
+import { spawn } from "child_process";
+import { existsSync as existsSync3 } from "fs";
 import chalk6 from "chalk";
 var runStepAsync = (name, step, exitOnFail = true, message) => {
   return new Promise((resolve) => {
@@ -424,7 +424,7 @@ var cleanAll = ({ verbose }) => {
 };
 
 // src/actions/clean-docs.ts
-import path from "node:path";
+import path from "path";
 import chalk8 from "chalk";
 var cleanDocs = () => {
   const pkgName = process.env.npm_package_name;
@@ -550,7 +550,7 @@ var compileTypesAll = ({
 };
 
 // src/actions/copy-assets.ts
-import path2 from "node:path/posix";
+import path2 from "path/posix";
 import chalk11 from "chalk";
 import cpy from "cpy";
 var copyPackageTargetAssets = async (target, name, location) => {
@@ -669,25 +669,15 @@ function findFilesByGlob(cwd, pattern) {
 // src/actions/deplint/findFiles.ts
 function findFiles(path5) {
   const allSourceInclude = ["./src/**/*.{ts,tsx}"];
-  const allDistInclude = ["./dist/**/*.d.ts"];
-  const prodExcludeEndswith = [".spec.ts", ".stories.tsx"];
-  const prodExcludeIncludes = ["/spec/", "/stories/", "/scripts/"];
-  const allSourceFiles = allSourceInclude.flatMap((pattern) => findFilesByGlob(path5, pattern));
-  const allDistFiles = allDistInclude.flatMap((pattern) => findFilesByGlob(path5, pattern));
-  const prodSourceFiles = allSourceFiles.filter((file) => !prodExcludeEndswith.some((ext) => file.endsWith(ext)) && !prodExcludeIncludes.some((excl) => file.includes(excl)));
-  const prodDistFiles = allDistFiles.filter((file) => !prodExcludeEndswith.some((ext) => file.endsWith(ext)) && !prodExcludeIncludes.some((excl) => file.includes(excl)));
-  const devSourceFiles = allSourceFiles.filter((file) => !prodSourceFiles.includes(file));
-  return {
-    allSourceFiles,
-    prodSourceFiles,
-    devSourceFiles,
-    prodDistFiles
-  };
+  const allDistInclude = ["./dist/**/*.d.ts", "./dist/**/*.{mjs,js,cjs}"];
+  const srcFiles = allSourceInclude.flatMap((pattern) => findFilesByGlob(path5, pattern));
+  const distFiles = allDistInclude.flatMap((pattern) => findFilesByGlob(path5, pattern));
+  return { srcFiles, distFiles };
 }
 
 // src/actions/deplint/getDependenciesFromPackageJson.ts
-import fs2 from "node:fs";
-import path3 from "node:path";
+import fs2 from "fs";
+import path3 from "path";
 function getDependenciesFromPackageJson(packageJsonPath) {
   const packageJsonFullPath = path3.resolve(packageJsonPath);
   const rawContent = fs2.readFileSync(packageJsonFullPath, "utf8");
@@ -703,27 +693,30 @@ function getDependenciesFromPackageJson(packageJsonPath) {
 }
 
 // src/actions/deplint/getImportsFromFile.ts
-import fs3 from "node:fs";
-import path4 from "node:path";
+import fs3 from "fs";
+import path4 from "path";
 import ts from "typescript";
 
 // src/actions/deplint/getBasePackageName.ts
 function getBasePackageName(importName) {
-  if (importName.startsWith("@")) {
-    const parts = importName.split("/");
-    return parts.length >= 2 ? `${parts[0]}/${parts[1]}` : importName;
+  const importNameScrubbed = importName.replaceAll('"', "").trim();
+  if (importNameScrubbed.startsWith("@")) {
+    const parts = importNameScrubbed.split("/");
+    return parts.length >= 2 ? `${parts[0]}/${parts[1]}` : importNameScrubbed;
   }
-  return importName.split("/")[0];
+  return importNameScrubbed.split("/")[0];
 }
 
 // src/actions/deplint/getImportsFromFile.ts
 function getImportsFromFile(filePath, importPaths, typeImportPaths) {
   const sourceCode = fs3.readFileSync(filePath, "utf8");
+  const isMjsFile = filePath.endsWith(".mjs");
   const sourceFile = ts.createSourceFile(
     path4.basename(filePath),
     sourceCode,
     ts.ScriptTarget.Latest,
-    true
+    true,
+    isMjsFile ? ts.ScriptKind.JS : void 0
   );
   const imports = [];
   const typeImports = [];
@@ -732,8 +725,8 @@ function getImportsFromFile(filePath, importPaths, typeImportPaths) {
     if (ts.isImportDeclaration(node) || ts.isExportDeclaration(node)) {
       const moduleSpecifier = node.moduleSpecifier?.getFullText();
       const isTypeImport = ts.isImportDeclaration(node) ? node.importClause?.isTypeOnly ?? false : false;
-      if (moduleSpecifier) {
-        const trimmed = moduleSpecifier.split("'").at(1) ?? moduleSpecifier;
+      if (typeof moduleSpecifier === "string") {
+        const trimmed = moduleSpecifier.replaceAll("'", "").replaceAll('"', "").trim();
         if (isTypeImport || isDeclarationFile) {
           typeImports.push(trimmed);
         } else {
@@ -754,11 +747,11 @@ function getImportsFromFile(filePath, importPaths, typeImportPaths) {
   const cleanedImports = imports.filter((imp) => !importsStartsWithExcludes.some((exc) => imp.startsWith(exc))).map(getBasePackageName);
   const cleanedTypeImports = typeImports.filter((imp) => !importsStartsWithExcludes.some((exc) => imp.startsWith(exc))).map(getBasePackageName);
   for (const imp of cleanedImports) {
-    importPaths[imp] = importPaths[imp] || [];
+    importPaths[imp] = importPaths[imp] ?? [];
     importPaths[imp].push(filePath);
   }
   for (const imp of cleanedTypeImports) {
-    typeImportPaths[imp] = typeImportPaths[imp] || [];
+    typeImportPaths[imp] = typeImportPaths[imp] ?? [];
     typeImportPaths[imp].push(filePath);
   }
   return [cleanedImports, cleanedTypeImports];
@@ -769,117 +762,115 @@ var internalImportPrefixes = [".", "#", "node:"];
 var removeInternalImports = (imports) => {
   return imports.filter((imp) => !internalImportPrefixes.some((prefix) => imp.startsWith(prefix)));
 };
-function getExternalImportsFromFiles({
-  prodSourceFiles,
-  devSourceFiles,
-  prodDistFiles
-}) {
-  const prodImportPaths = {};
-  const prodTypeImportPaths = {};
-  const prodImportPairs = prodSourceFiles.map((path5) => getImportsFromFile(path5, prodImportPaths, prodTypeImportPaths));
-  const prodDistImportPairs = prodDistFiles.map((path5) => getImportsFromFile(path5, prodImportPaths, prodTypeImportPaths));
-  const prodImports = prodImportPairs.flatMap((pair) => pair[0]);
-  const prodDistImports = prodDistImportPairs.flatMap((pair) => pair[1]);
-  const devImportPaths = {};
-  const devTypeImportPaths = {};
-  const devImportPairs = devSourceFiles.map((path5) => getImportsFromFile(path5, devImportPaths, devTypeImportPaths));
-  const devImports = devImportPairs.flatMap((pair) => pair[0]);
-  const devTypeImports = devImportPairs.flatMap((pair) => pair[1]);
-  const externalProdImports = removeInternalImports(prodImports);
-  const externalProdTypeImports = removeInternalImports(prodDistImports);
-  const externalDevImports = removeInternalImports(devImports);
+function getExternalImportsFromFiles({ srcFiles, distFiles }) {
+  const srcImportPaths = {};
+  const distImportPaths = {};
+  for (const path5 of srcFiles) getImportsFromFile(path5, srcImportPaths, srcImportPaths).flat();
+  for (const path5 of distFiles) getImportsFromFile(path5, distImportPaths, distImportPaths).flat();
+  const srcImports = Object.keys(srcImportPaths);
+  const distImports = Object.keys(distImportPaths);
+  const externalSrcImports = removeInternalImports(srcImports);
+  const externalDistImports = removeInternalImports(distImports);
   return {
-    prodImports,
-    devImports,
-    prodImportPaths,
-    prodTypeImportPaths,
-    devImportPaths,
-    externalProdImports,
-    externalDevImports,
-    prodDistImports,
-    devTypeImports,
-    externalProdTypeImports
+    srcImports,
+    srcImportPaths,
+    externalSrcImports,
+    distImports,
+    distImportPaths,
+    externalDistImports
   };
 }
 
 // src/actions/deplint/checkPackage/getUnlistedDependencies.ts
 import chalk12 from "chalk";
-function getUnlistedDependencies({ name }, { dependencies, peerDependencies }, {
-  externalProdTypeImports,
-  prodTypeImportPaths,
-  externalProdImports,
-  prodImportPaths
-}) {
+function getUnlistedDependencies({ name, location }, { dependencies, peerDependencies }, { externalDistImports, distImportPaths }) {
   let unlistedDependencies = 0;
-  for (const imp of externalProdTypeImports) {
-    if (!dependencies.includes(imp) && !peerDependencies.includes(imp) && !dependencies.includes(`@types/${imp}`) && !peerDependencies.includes(`@types/${imp}`)) {
-      unlistedDependencies++;
-      console.log(`[${chalk12.blue(name)}] Missing dependency in package.json: ${chalk12.red(imp)}`);
-      console.log(`  ${prodTypeImportPaths[imp].join("\n")}`);
-      console.log("");
-    }
-  }
-  for (const imp of externalProdImports) {
+  for (const imp of externalDistImports) {
     if (!dependencies.includes(imp) && !peerDependencies.includes(imp)) {
       unlistedDependencies++;
       console.log(`[${chalk12.blue(name)}] Missing dependency in package.json: ${chalk12.red(imp)}`);
-      console.log(`  ${prodImportPaths[imp].join("\n")}`);
-      console.log("");
+      console.log(`  ${distImportPaths[imp].join("\n ")}`);
     }
   }
+  if (unlistedDependencies > 0) {
+    const packageLocation = `${location}/package.json`;
+    console.log(`  ${chalk12.yellow(packageLocation)}
+`);
+  }
   return unlistedDependencies;
 }
 
 // src/actions/deplint/checkPackage/getUnlistedDevDependencies.ts
 import chalk13 from "chalk";
-function getUnlistedDevDependencies({ name }, {
+function getUnlistedDevDependencies({ name, location }, {
   devDependencies,
   dependencies,
   peerDependencies
-}, { devImportPaths, externalDevImports }) {
+}, {
+  srcImportPaths,
+  externalSrcImports,
+  distImports
+}) {
   let unlistedDevDependencies = 0;
-  for (const imp of externalDevImports) {
-    if (!devDependencies.includes(imp) && !dependencies.includes(imp) && !peerDependencies.includes(imp)) {
+  for (const imp of externalSrcImports) {
+    if (!distImports.includes(imp) && !dependencies.includes(imp) && !peerDependencies.includes(imp) && !devDependencies.includes(imp)) {
       unlistedDevDependencies++;
       console.log(`[${chalk13.blue(name)}] Missing devDependency in package.json: ${chalk13.red(imp)}`);
-      console.log(`  Found in: ${devImportPaths[imp].join(", ")}`);
+      console.log(`  ${srcImportPaths[imp].join("\n ")}`);
     }
   }
+  if (unlistedDevDependencies > 0) {
+    const packageLocation = `${location}/package.json`;
+    console.log(`  ${chalk13.yellow(packageLocation)}
+`);
+  }
   return unlistedDevDependencies;
 }
 
 // src/actions/deplint/checkPackage/getUnusedDependencies.ts
 import chalk14 from "chalk";
 function getUnusedDependencies({ name, location }, { dependencies }, {
-  externalProdImports,
-  externalProdTypeImports
+  externalDistImports,
+  externalSrcImports
 }) {
   let unusedDependencies = 0;
   for (const dep of dependencies) {
-    if (!externalProdImports.includes(dep) && !externalProdTypeImports.includes(dep)) {
+    if (!externalDistImports.includes(dep)) {
       unusedDependencies++;
-      console.log(`[${chalk14.blue(name)}] Unused dependency in package.json: ${chalk14.red(dep)}`);
-      console.log(`  ${location}/package.json
-`);
-      console.log("");
+      if (externalSrcImports.includes(dep)) {
+        console.log(`[${chalk14.blue(name)}] dependency should be devDependency in package.json: ${chalk14.red(dep)}`);
+      } else {
+        console.log(`[${chalk14.blue(name)}] Unused dependency in package.json: ${chalk14.red(dep)}`);
+      }
     }
   }
+  if (unusedDependencies > 0) {
+    const packageLocation = `${location}/package.json`;
+    console.log(`  ${chalk14.yellow(packageLocation)}
+`);
+  }
   return unusedDependencies;
 }
 
 // src/actions/deplint/checkPackage/getUnusedPeerDependencies.ts
 import chalk15 from "chalk";
-function getUnusedPeerDependencies({ name, location }, { peerDependencies }, { externalProdImports, externalProdTypeImports }) {
+function getUnusedPeerDependencies({ name, location }, { peerDependencies, dependencies }, { externalDistImports }) {
   let unusedDependencies = 0;
   for (const dep of peerDependencies) {
-    if (!externalProdImports.includes(dep) && !externalProdTypeImports.includes(dep)) {
+    if (!externalDistImports.includes(dep)) {
       unusedDependencies++;
-      console.log(`[${chalk15.blue(name)}] Unused peerDependency in package.json: ${chalk15.red(dep)}`);
-      console.log(`  ${location}/package.json
-`);
-      console.log("");
+      if (dependencies.includes(dep)) {
+        console.log(`[${chalk15.blue(name)}] Unused peerDependency [already a dependency] in package.json: ${chalk15.red(dep)}`);
+      } else {
+        console.log(`[${chalk15.blue(name)}] Unused peerDependency in package.json: ${chalk15.red(dep)}`);
+      }
     }
   }
+  if (unusedDependencies > 0) {
+    const packageLocation = `${location}/package.json`;
+    console.log(`  ${chalk15.yellow(packageLocation)}
+`);
+  }
   return unusedDependencies;
 }
 
@@ -891,26 +882,17 @@ function checkPackage({
   devDeps = false,
   peerDeps = false
 }) {
-  const {
-    prodSourceFiles,
-    devSourceFiles,
-    prodDistFiles
-  } = findFiles(location);
+  const { srcFiles, distFiles } = findFiles(location);
   const checkDeps = deps || !(deps || devDeps || peerDeps);
   const checkDevDeps = devDeps || !(deps || devDeps || peerDeps);
   const checkPeerDeps = peerDeps;
-  const sourceParams = getExternalImportsFromFiles({
-    prodSourceFiles,
-    devSourceFiles,
-    prodDistFiles
-  });
+  const sourceParams = getExternalImportsFromFiles({ srcFiles, distFiles });
   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 typesInDependencies = 0;
   const unlistedDevDependencies = checkDevDeps ? getUnlistedDevDependencies({ name, location }, packageParams, sourceParams) : 0;
   const unusedPeerDependencies = checkPeerDeps ? getUnusedPeerDependencies({ name, location }, packageParams, sourceParams) : 0;
-  const totalErrors = unlistedDependencies + unlistedDevDependencies + unusedDependencies + typesInDependencies + unusedPeerDependencies;
+  const totalErrors = unlistedDependencies + unlistedDevDependencies + unusedDependencies + unusedPeerDependencies;
   return totalErrors;
 }
 
@@ -953,7 +935,7 @@ var deplint = ({
 };
 
 // src/actions/deploy.ts
-import { readFileSync as readFileSync3 } from "node:fs";
+import { readFileSync as readFileSync3 } from "fs";
 var privatePackageExcludeList = () => {
   const possibleDeployablePackages = yarnWorkspaces().map((workspace) => [
     workspace,
@@ -978,7 +960,7 @@ var deploy = () => {
 };
 
 // src/actions/deploy-major.ts
-import { readFileSync as readFileSync4 } from "node:fs";
+import { readFileSync as readFileSync4 } from "fs";
 var privatePackageExcludeList2 = () => {
   const possibleDeployablePackages = yarnWorkspaces().map((workspace) => [
     workspace,
@@ -1003,7 +985,7 @@ var deployMajor = () => {
 };
 
 // src/actions/deploy-minor.ts
-import { readFileSync as readFileSync5 } from "node:fs";
+import { readFileSync as readFileSync5 } from "fs";
 var privatePackageExcludeList3 = () => {
   const possibleDeployablePackages = yarnWorkspaces().map((workspace) => [
     workspace,
@@ -1028,7 +1010,7 @@ var deployMinor = () => {
 };
 
 // src/actions/deploy-next.ts
-import { readFileSync as readFileSync6 } from "node:fs";
+import { readFileSync as readFileSync6 } from "fs";
 var privatePackageExcludeList4 = () => {
   const possibleDeployablePackages = yarnWorkspaces().map((workspace) => [
     workspace,
@@ -1169,7 +1151,7 @@ Gitlint Start [${process.cwd()}]
 };
 
 // src/actions/gitlint-fix.ts
-import { execSync as execSync2 } from "node:child_process";
+import { execSync as execSync2 } from "child_process";
 import chalk20 from "chalk";
 import ParseGitConfig2 from "parse-git-config";
 var gitlintFix = () => {
@@ -1366,7 +1348,7 @@ import {
   closeSync,
   openSync,
   rmSync
-} from "node:fs";
+} from "fs";
 var reinstall = () => {
   console.log("Reinstall [Clear Lock File]");
   closeSync(openSync("./yarn.lock", "w"));