@driftdev/cli 0.37.0 → 0.38.0

package/dist/drift.js

@@ -175703,6 +175703,54 @@ function renderBatchExamples(data) {
   return lines.join(`
 `);
 }
+function renderBatchDiff(data) {
+  const lines = [""];
+  const rows = data.packages;
+  const nameW = Math.max(7, ...rows.map((r) => r.name.length));
+  lines.push(indent(`${c.gray(pad("PACKAGE", nameW))}  ${c.gray(pad("BREAKING", 8))}  ${c.gray(pad("ADDED", 5))}  ${c.gray("CHANGED")}`));
+  for (const r of rows) {
+    const bStr = pad(String(r.breaking), 8);
+    const aStr = pad(String(r.added), 5);
+    const cStr = String(r.changed);
+    const breaking = r.breaking === 0 ? c.green(bStr) : c.red(bStr);
+    const added = r.added === 0 ? aStr : c.green(aStr);
+    const changed = r.changed === 0 ? cStr : c.yellow(cStr);
+    lines.push(indent(`${pad(r.name, nameW)}  ${breaking}  ${added}  ${changed}`));
+  }
+  lines.push("");
+  const parts = [];
+  if (data.aggregate.breaking > 0)
+    parts.push(`${data.aggregate.breaking} breaking`);
+  if (data.aggregate.added > 0)
+    parts.push(`${data.aggregate.added} added`);
+  if (data.aggregate.changed > 0)
+    parts.push(`${data.aggregate.changed} changed`);
+  lines.push(indent(`${c.bold("Total")}: ${parts.length > 0 ? parts.join(", ") : "no changes"}`));
+  if (data.skipped && data.skipped.length > 0) {
+    lines.push(indent(`${c.gray(`Skipped ${data.skipped.length} private: ${data.skipped.join(", ")}`)}`));
+  }
+  lines.push("");
+  return lines.join(`
+`);
+}
+function renderBatchBreaking(data) {
+  const lines = [""];
+  const rows = data.packages;
+  const nameW = Math.max(7, ...rows.map((r) => r.name.length));
+  lines.push(indent(`${c.gray(pad("PACKAGE", nameW))}  ${c.gray("BREAKING")}`));
+  for (const r of rows) {
+    const count = r.count === 0 ? c.green("0") : c.red(String(r.count));
+    lines.push(indent(`${pad(r.name, nameW)}  ${count}`));
+  }
+  lines.push("");
+  lines.push(indent(`${c.bold("Total")}: ${data.aggregate.count} breaking change${data.aggregate.count === 1 ? "" : "s"}`));
+  if (data.skipped && data.skipped.length > 0) {
+    lines.push(indent(`${c.gray(`Skipped ${data.skipped.length} private: ${data.skipped.join(", ")}`)}`));
+  }
+  lines.push("");
+  return lines.join(`
+`);
+}
 function pad(s, w) {
   return s + " ".repeat(Math.max(0, w - s.length));
 }
@@ -176127,7 +176175,7 @@ function registerHealthCommand(program) {
 import { readFileSync as readFileSync15 } from "node:fs";
 import * as path19 from "node:path";
 import { fileURLToPath as fileURLToPath2 } from "node:url";
-import { diffSpec as diffSpec3, categorizeBreakingChanges as categorizeBreakingChanges2 } from "@openpkg-ts/spec";
+import { diffSpec as diffSpec3, categorizeBreakingChanges as categorizeBreakingChanges2, normalize as normalize4 } from "@openpkg-ts/spec";
 
 // src/formatters/breaking.ts
 function renderBreaking(data) {
@@ -176150,17 +176198,20 @@ function renderBreaking(data) {
 `);
 }
 
-// src/utils/resolve-specs.ts
-import { readFileSync as readFileSync14 } from "node:fs";
-import * as path18 from "node:path";
-import { normalize as normalize3 } from "@openpkg-ts/spec";
-
 // src/utils/git-extract.ts
 import { execSync } from "node:child_process";
 import { existsSync as existsSync14, mkdirSync as mkdirSync4, mkdtempSync, rmSync as rmSync2, symlinkSync, writeFileSync as writeFileSync4, cpSync } from "node:fs";
 import * as os4 from "node:os";
 import * as path17 from "node:path";
 import { normalize as normalize2 } from "@openpkg-ts/spec";
+function getGitRoot(cwd) {
+  return execSync("git rev-parse --show-toplevel", {
+    encoding: "utf-8",
+    cwd,
+    timeout: 5000,
+    stdio: ["pipe", "pipe", "pipe"]
+  }).trim();
+}
 function listFilesAtRef(ref, prefix, cwd) {
   try {
     const output = execSync(`git ls-tree -r --name-only ${ref} -- ${prefix}`, {
@@ -176200,19 +176251,21 @@ function validateRef(ref, cwd = process.cwd()) {
   }
 }
 async function extractSpecFromRef(ref, entry, cwd = process.cwd()) {
-  const relEntry = path17.relative(cwd, path17.resolve(cwd, entry));
-  const entryDir = path17.dirname(relEntry);
+  const gitRoot = getGitRoot(cwd);
+  const absEntry = path17.resolve(cwd, entry);
+  const repoRelEntry = path17.relative(gitRoot, absEntry);
+  const entryDir = path17.dirname(repoRelEntry);
   const srcPrefix = entryDir.includes("/") ? entryDir.split("/").slice(0, 2).join("/") : ".";
   const tmpDir = mkdtempSync(path17.join(os4.tmpdir(), "drift-git-"));
   try {
     const prefixes = srcPrefix === "." ? [""] : [srcPrefix];
     const rootFiles = ["tsconfig.json", "tsconfig.base.json", "package.json"];
     for (const prefix of prefixes) {
-      const files = listFilesAtRef(ref, prefix || ".", cwd);
+      const files = listFilesAtRef(ref, prefix || ".", gitRoot);
       for (const file of files) {
         if (!file.match(/\.(ts|tsx|json|js|mjs|cjs)$/))
           continue;
-        const content = getFileAtRef(ref, file, cwd);
+        const content = getFileAtRef(ref, file, gitRoot);
         if (content === null)
           continue;
         const destPath = path17.join(tmpDir, file);
@@ -176223,7 +176276,7 @@ async function extractSpecFromRef(ref, entry, cwd = process.cwd()) {
     for (const rootFile of rootFiles) {
       const dest = path17.join(tmpDir, rootFile);
       if (!existsSync14(dest)) {
-        const content = getFileAtRef(ref, rootFile, cwd);
+        const content = getFileAtRef(ref, rootFile, gitRoot);
         if (content) {
           writeFileSync4(dest, content);
         }
@@ -176234,8 +176287,15 @@ async function extractSpecFromRef(ref, entry, cwd = process.cwd()) {
     if (existsSync14(nmSrc) && !existsSync14(nmDest)) {
       symlinkSync(nmSrc, nmDest, "dir");
     }
+    if (cwd !== gitRoot) {
+      const rootNm = path17.join(gitRoot, "node_modules");
+      const rootNmDest = path17.join(tmpDir, "node_modules");
+      if (existsSync14(rootNm) && !existsSync14(rootNmDest)) {
+        symlinkSync(rootNm, rootNmDest, "dir");
+      }
+    }
     if (srcPrefix !== ".") {
-      const nestedNm = path17.join(cwd, srcPrefix, "node_modules");
+      const nestedNm = path17.join(gitRoot, srcPrefix, "node_modules");
       const nestedDest = path17.join(tmpDir, srcPrefix, "node_modules");
       if (existsSync14(nestedNm) && !existsSync14(nestedDest)) {
         symlinkSync(nestedNm, nestedDest, "dir");
@@ -176246,9 +176306,9 @@ async function extractSpecFromRef(ref, entry, cwd = process.cwd()) {
     if (existsSync14(tsconfigSrc) && !existsSync14(tsconfigDest)) {
       cpSync(tsconfigSrc, tsconfigDest);
     }
-    const entryFile = path17.join(tmpDir, relEntry);
+    const entryFile = path17.join(tmpDir, repoRelEntry);
     if (!existsSync14(entryFile)) {
-      throw new Error(`Entry file not found at ref ${ref}: ${relEntry}`);
+      throw new Error(`Entry file not found at ref ${ref}: ${repoRelEntry}`);
     }
     const result = await extract({ entryFile });
     return normalize2(result.spec);
@@ -176258,6 +176318,9 @@ async function extractSpecFromRef(ref, entry, cwd = process.cwd()) {
 }
 
 // src/utils/resolve-specs.ts
+import { readFileSync as readFileSync14 } from "node:fs";
+import * as path18 from "node:path";
+import { normalize as normalize3 } from "@openpkg-ts/spec";
 function loadSpec(filePath) {
   return JSON.parse(readFileSync14(path18.resolve(process.cwd(), filePath), "utf-8"));
 }
@@ -176316,10 +176379,52 @@ function getVersion2() {
   }
 }
 function registerBreakingCommand(program) {
-  program.command("breaking [old] [new]").description("Detect breaking changes between two specs").option("--base <ref>", "Git ref for old spec").option("--head <ref>", "Git ref for new spec (default: working tree)").option("--entry <file>", "Entry file for git ref extraction").action(async (oldPath, newPath, options) => {
+  program.command("breaking [old] [new]").description("Detect breaking changes between two specs").option("--base <ref>", "Git ref for old spec").option("--head <ref>", "Git ref for new spec (default: working tree)").option("--entry <file>", "Entry file for git ref extraction").option("--all", "Run across all workspace packages").option("--private", "Include private packages in --all mode").action(async (oldPath, newPath, options) => {
     const startTime = Date.now();
     const version = getVersion2();
     try {
+      const cwd = process.cwd();
+      const allPackages = options.all ? discoverPackages(cwd) : null;
+      const autoDetected = !options.all && options.base && !options.entry && !oldPath ? discoverPackages(cwd) : null;
+      const batchPackages = allPackages ?? autoDetected;
+      if (batchPackages && batchPackages.length > 0 && options.base) {
+        const skipped = options.private ? [] : batchPackages.filter((p) => p.private).map((p) => p.name);
+        const packages = options.private ? batchPackages : filterPublic(batchPackages);
+        if (packages.length === 0) {
+          formatError("breaking", "No workspace packages found", startTime, version);
+          return;
+        }
+        const rows = [];
+        let totalBreaking = 0;
+        for (const pkg of packages) {
+          const relEntry = path19.relative(cwd, pkg.entry);
+          const oldSpec2 = await extractSpecFromRef(options.base, relEntry, cwd);
+          let newSpec2;
+          if (options.head) {
+            newSpec2 = await extractSpecFromRef(options.head, relEntry, cwd);
+          } else {
+            const result = await extract({ entryFile: pkg.entry });
+            newSpec2 = normalize4(result.spec);
+          }
+          const diff2 = diffSpec3(oldSpec2, newSpec2);
+          const breaking2 = categorizeBreakingChanges2(diff2.breaking, oldSpec2, newSpec2);
+          rows.push({ name: pkg.name, breaking: breaking2, count: breaking2.length });
+          totalBreaking += breaking2.length;
+        }
+        const data2 = { packages: rows, aggregate: { count: totalBreaking }, ...skipped.length > 0 ? { skipped } : {} };
+        formatOutput("breaking", data2, startTime, version, renderBatchBreaking);
+        if (totalBreaking > 0) {
+          if (!shouldRenderHuman()) {
+            process.stderr.write(`${totalBreaking} breaking change${totalBreaking === 1 ? "" : "s"} found
+`);
+          }
+          process.exitCode = 1;
+        } else if (!shouldRenderHuman()) {
+          process.stderr.write(`No breaking changes
+`);
+        }
+        return;
+      }
       const args = [oldPath, newPath].filter(Boolean);
       const { oldSpec, newSpec } = await resolveSpecs({ args, ...options });
       const diff = diffSpec3(oldSpec, newSpec);
@@ -176896,7 +177001,7 @@ function buildPRComment(results, pass, commit) {
 `);
 }
 function registerCiCommand(program) {
-  program.command("ci").description("Run CI checks on changed packages").option("--all", "Check all packages, not just changed ones").option("--private", "Include private packages").action(async (options) => {
+  program.command("ci").description("Run CI checks on changed packages").option("--all", "Check all packages, not just changed ones").option("--private", "Include private packages").option("--min <number>", "Minimum coverage percentage (0-100)").action(async (options) => {
     const startTime = Date.now();
     const version = getVersion5();
     const cwd = process.cwd();
@@ -176913,7 +177018,7 @@ function registerCiCommand(program) {
       }
       if (packageDirs.length === 0)
         packageDirs = allDirs;
-      let minThreshold = config.coverage?.min ?? 0;
+      let minThreshold = options.min ? Number(options.min) : config.coverage?.min ?? 0;
       if (minThreshold > 0 && config.coverage?.ratchet) {
         const ratchet = computeRatchetMin(minThreshold);
         minThreshold = ratchet.effectiveMin;
@@ -177371,7 +177476,7 @@ function registerExamplesCommand(program) {
 import { readFileSync as readFileSync23 } from "node:fs";
 import * as path27 from "node:path";
 import { fileURLToPath as fileURLToPath8 } from "node:url";
-import { diffSpec as diffSpec7, categorizeBreakingChanges as categorizeBreakingChanges5 } from "@openpkg-ts/spec";
+import { diffSpec as diffSpec7, categorizeBreakingChanges as categorizeBreakingChanges5, normalize as normalize5 } from "@openpkg-ts/spec";
 
 // src/formatters/diff.ts
 function renderDiff(data) {
@@ -177431,10 +177536,58 @@ function getVersion8() {
   }
 }
 function registerDiffCommand(program) {
-  program.command("diff [old] [new]").description("Compare two specs and show what changed").option("--base <ref>", "Git ref for old spec").option("--head <ref>", "Git ref for new spec (default: working tree)").option("--entry <file>", "Entry file for git ref extraction").action(async (oldPath, newPath, options) => {
+  program.command("diff [old] [new]").description("Compare two specs and show what changed").option("--base <ref>", "Git ref for old spec").option("--head <ref>", "Git ref for new spec (default: working tree)").option("--entry <file>", "Entry file for git ref extraction").option("--all", "Run across all workspace packages").option("--private", "Include private packages in --all mode").action(async (oldPath, newPath, options) => {
     const startTime = Date.now();
     const version = getVersion8();
     try {
+      const cwd = process.cwd();
+      const allPackages = options.all ? discoverPackages(cwd) : null;
+      const autoDetected = !options.all && options.base && !options.entry && !oldPath ? discoverPackages(cwd) : null;
+      const batchPackages = allPackages ?? autoDetected;
+      if (batchPackages && batchPackages.length > 0 && options.base) {
+        const skipped = options.private ? [] : batchPackages.filter((p) => p.private).map((p) => p.name);
+        const packages = options.private ? batchPackages : filterPublic(batchPackages);
+        if (packages.length === 0) {
+          formatError("diff", "No workspace packages found", startTime, version);
+          return;
+        }
+        const rows = [];
+        let totalBreaking = 0;
+        let totalAdded = 0;
+        let totalChanged = 0;
+        for (const pkg of packages) {
+          const relEntry = path27.relative(cwd, pkg.entry);
+          const oldSpec2 = await extractSpecFromRef(options.base, relEntry, cwd);
+          let newSpec2;
+          if (options.head) {
+            newSpec2 = await extractSpecFromRef(options.head, relEntry, cwd);
+          } else {
+            const result = await extract({ entryFile: pkg.entry });
+            newSpec2 = normalize5(result.spec);
+          }
+          const diff2 = diffSpec7(oldSpec2, newSpec2);
+          rows.push({ name: pkg.name, breaking: diff2.breaking.length, added: diff2.nonBreaking.length, changed: diff2.docsOnly.length });
+          totalBreaking += diff2.breaking.length;
+          totalAdded += diff2.nonBreaking.length;
+          totalChanged += diff2.docsOnly.length;
+        }
+        const data2 = { packages: rows, aggregate: { breaking: totalBreaking, added: totalAdded, changed: totalChanged }, ...skipped.length > 0 ? { skipped } : {} };
+        formatOutput("diff", data2, startTime, version, renderBatchDiff);
+        if (!shouldRenderHuman()) {
+          const parts = [];
+          if (totalBreaking > 0)
+            parts.push(`${totalBreaking} breaking`);
+          if (totalAdded > 0)
+            parts.push(`${totalAdded} added`);
+          if (totalChanged > 0)
+            parts.push(`${totalChanged} changed`);
+          process.stderr.write(`${parts.length > 0 ? parts.join(", ") : "no changes"}
+`);
+        }
+        if (totalBreaking > 0)
+          process.exitCode = 1;
+        return;
+      }
       const args = [oldPath, newPath].filter(Boolean);
       const { oldSpec, newSpec } = await resolveSpecs({ args, ...options });
       const diff = diffSpec7(oldSpec, newSpec);
@@ -177474,7 +177627,7 @@ import { readFileSync as readFileSync24 } from "node:fs";
 import * as path28 from "node:path";
 import { fileURLToPath as fileURLToPath9 } from "node:url";
 import { DocCov } from "@driftdev/sdk";
-import { normalize as normalize4 } from "@openpkg-ts/spec";
+import { normalize as normalize6 } from "@openpkg-ts/spec";
 
 // src/formatters/extract.ts
 function renderExtract(data) {
@@ -177542,7 +177695,7 @@ function registerExtractCommand(program) {
           formatError("extract", "Failed to extract spec", startTime, version);
           return;
         }
-        spec = normalize4(result.spec);
+        spec = normalize6(result.spec);
       } else {
         const result = await cachedExtract(entryFile);
         spec = result.spec;
@@ -178152,7 +178305,7 @@ import { existsSync as existsSync19, mkdirSync as mkdirSync7, readFileSync as re
 import * as path33 from "node:path";
 import { fileURLToPath as fileURLToPath14 } from "node:url";
 init_global();
-import { normalize as normalize5 } from "@openpkg-ts/spec";
+import { normalize as normalize7 } from "@openpkg-ts/spec";
 
 // src/formatters/init.ts
 function renderInit(data) {
@@ -178474,7 +178627,7 @@ async function scanPackage(cwd, pkgDir) {
   try {
     const entryFile = detectEntry(absDir);
     const result = await extract({ entryFile });
-    const spec = normalize5(result.spec);
+    const spec = normalize7(result.spec);
     const exports = spec.exports ?? [];
     const total = exports.length;
     let documented = 0;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@driftdev/cli",
-  "version": "0.37.0",
+  "version": "0.38.0",
   "description": "Drift CLI - Documentation coverage and drift detection for TypeScript",
   "keywords": [
     "typescript",