@kb-labs/quality-entry 2.109.0 → 2.111.0

package/dist/cli/commands/index.js

@@ -34,11 +34,11 @@ var build_order_default = defineCommand({
             }))
           });
         }
-        return { exitCode: 1 };
+        return { ok: false, error: "Command failed" };
       }
       if (flags.json) {
         ctx.ui?.json?.(result);
-        return { exitCode: 0 };
+        return { ok: true };
       }
       if (flags.script) {
         for (let i = 0; i < result.layers.length; i++) {
@@ -46,7 +46,7 @@ var build_order_default = defineCommand({
           if (!layer) continue;
           ctx.ui?.success?.(`# Layer ${i + 1}`, { sections: [{ header: "", items: layer.map((p) => `pnpm --filter "${p}" run build`) }] });
         }
-        return { exitCode: 0 };
+        return { ok: true };
       }
       const sections = flags.layers ? result.layers.map((layer, i) => ({
         header: `Layer ${i + 1} \u2014 ${layer.length} package(s) (parallel)`,
@@ -57,7 +57,7 @@ var build_order_default = defineCommand({
         items: [`${result.packageCount} packages in ${result.layerCount} layers`]
       });
       ctx.ui?.success?.(flags.package ? `Build order for ${flags.package}` : "Monorepo build order", { sections });
-      return { exitCode: 0 };
+      return { ok: true };
     }
   }
 });
@@ -73,7 +73,7 @@ var check_builds_default = defineCommand({
         const cached = await platform.cache.get(cacheKey);
         if (cached) {
           outputBuildCheck({ ...cached, cached: true }, flags, ui);
-          return { exitCode: cached.failing > 0 ? 1 : 0, result: cached };
+          return cached.failing > 0 ? { ok: false, error: "Build failures found", result: cached } : { ok: true, result: cached };
         }
       }
       const result = await checkBuilds(ctx.cwd, {
@@ -90,10 +90,7 @@ var check_builds_default = defineCommand({
         packageSpecific: !!flags.package
       });
       outputBuildCheck({ ...result, cached: false }, flags, ui);
-      return {
-        exitCode: result.failing > 0 ? 1 : 0,
-        result
-      };
+      return result.failing > 0 ? { ok: false, error: "Build failures found", result } : { ok: true, result };
     }
   }
 });
@@ -157,7 +154,7 @@ var check_tests_default = defineCommand({
         const cached = await platform.cache.get(cacheKey);
         if (cached) {
           outputTestResults({ ...cached, cached: true }, flags, ui);
-          return { exitCode: cached.failing > 0 ? 1 : 0, result: cached };
+          return cached.failing > 0 ? { ok: false, error: "Test failures found", result: cached } : { ok: true, result: cached };
         }
       }
       const result = await runTests(ctx.cwd, {
@@ -176,11 +173,7 @@ var check_tests_default = defineCommand({
         cached: false
       });
       outputTestResults({ ...result, cached: false }, flags, ui);
-      return {
-        exitCode: result.failing > 0 ? 1 : 0,
-        result,
-        meta: { cached: false }
-      };
+      return result.failing > 0 ? { ok: false, error: "Test failures found", result, meta: { cached: false } } : { ok: true, result, meta: { cached: false } };
     }
   }
 });
@@ -262,7 +255,7 @@ var check_types_default = defineCommand({
         const cached = await platform.cache.get(cacheKey);
         if (cached) {
           outputTypeAnalysis({ ...cached, cached: true }, flags, ui);
-          return { exitCode: cached.totalErrors > 0 ? 1 : 0, result: cached };
+          return cached.totalErrors > 0 ? { ok: false, error: "Type errors found", result: cached } : { ok: true, result: cached };
         }
       }
       const result = await analyzeTypes(ctx.cwd, {
@@ -278,11 +271,7 @@ var check_types_default = defineCommand({
         cached: false
       });
       outputTypeAnalysis({ ...result, cached: false }, flags, ui);
-      return {
-        exitCode: result.totalErrors > 0 ? 1 : 0,
-        result,
-        meta: { cached: false }
-      };
+      return result.totalErrors > 0 ? { ok: false, error: "Type errors found", result, meta: { cached: false } } : { ok: true, result, meta: { cached: false } };
     }
   }
 });
@@ -368,11 +357,11 @@ var check_layers_default = defineCommand({
       } : report;
       if (flags.json) {
         ctx.ui?.json?.(filtered);
-        return { exitCode: filtered.totalViolations > 0 ? 1 : 0 };
+        return filtered.totalViolations > 0 ? { ok: false, error: "Layer violations found", result: filtered } : { ok: true, result: filtered };
       }
       if (filtered.totalViolations === 0) {
         ctx.ui?.success?.("No layering violations found");
-        return { exitCode: 0 };
+        return { ok: true };
       }
       const byPackage = /* @__PURE__ */ new Map();
       for (const v of filtered.violations) {
@@ -385,7 +374,7 @@ var check_layers_default = defineCommand({
         items: vs.map((v) => `\u2192 ${v.toPackage} (Layer ${v.toLayer})  ${v.importSpecifier}`)
       }));
       ctx.ui?.error?.(`${filtered.totalViolations} layering violation(s) in ${filtered.affectedPackages.length} package(s)`, { sections });
-      return { exitCode: 1 };
+      return { ok: false, error: "Command failed" };
     }
   }
 });
@@ -400,7 +389,7 @@ var coupling_default = defineCommand({
       const report = analyzeCoupling({ rootDir: cwd, layerMap: config?.layers, topN: flags.top });
       if (flags.json) {
         ctx.ui?.json?.(report);
-        return { exitCode: 0 };
+        return { ok: true };
       }
       const sorted = flags.sort === "coupled" ? [...report.packages].sort((a, b) => b.afferent + b.efferent - (a.afferent + a.efferent)) : [...report.packages].sort((a, b) => b.instability - a.instability);
       const top = sorted.slice(0, flags.top ?? 10);
@@ -412,7 +401,7 @@ var coupling_default = defineCommand({
       ctx.ui?.success?.(`Coupling \u2014 avg instability: ${report.avgInstability.toFixed(2)}`, {
         sections: [{ header: `Top ${top.length} by instability`, items }]
       });
-      return { exitCode: 0 };
+      return { ok: true };
     }
   }
 });
@@ -426,10 +415,7 @@ var cycles_default = defineCommand({
       const graph = buildDependencyGraph(ctx.cwd);
       const cycles = findCircularDependencies(graph);
       outputCycles(cycles, flags, ui);
-      return {
-        exitCode: cycles.length > 0 ? 1 : 0,
-        cycles
-      };
+      return cycles.length > 0 ? { ok: false, error: "Circular dependencies found", result: { cycles } } : { ok: true, result: { cycles } };
     }
   }
 });
@@ -504,16 +490,16 @@ var dead_code_default = defineCommand({
       const cfg = config ?? defaultQualityConfig;
       if (!cfg.knip.enabled) {
         ctx.ui?.success?.("knip is disabled in config (knip.enabled = false)");
-        return { exitCode: 0 };
+        return { ok: true };
       }
       const report = await runKnip({ shell: ctx.api.shell, rootDir: cwd });
       if (flags.json) {
         ctx.ui?.json?.(report);
-        return { exitCode: report.totalIssues > 0 ? 1 : 0 };
+        return report.totalIssues > 0 ? { ok: false, error: "Dead code found", result: report } : { ok: true, result: report };
       }
       if (report.totalIssues === 0) {
         ctx.ui?.success?.("No dead code detected");
-        return { exitCode: 0 };
+        return { ok: true };
       }
       const sections = [];
       if (report.unusedFiles.length > 0) {
@@ -541,7 +527,7 @@ var dead_code_default = defineCommand({
         });
       }
       ctx.ui?.error?.(`${report.totalIssues} dead code issue(s) found`, { sections });
-      return { exitCode: 1 };
+      return { ok: false, error: "Command failed" };
     }
   }
 });
@@ -589,11 +575,11 @@ var fix_deps_default = defineCommand({
       if (showStats) {
         const stats = await getDependencyStats(ctx.cwd);
         outputStats(stats, flags, ui);
-        return { exitCode: 0 };
+        return { ok: true };
       }
       if (!removeUnused && !addMissing && !alignVersions) {
         validationError(ctx, "No fix options specified", "Use --remove-unused, --add-missing, --align-versions, or --all", flags.json);
-        return { exitCode: 1 };
+        return { ok: false, error: "Command failed" };
       }
       const loader = useLoader("Scanning packages...");
       loader.start();
@@ -609,7 +595,7 @@ var fix_deps_default = defineCommand({
         alignedCount: result.alignedDeps.length
       });
       outputResults({ ...result, dryRun: false }, flags, ui);
-      return { exitCode: 0, result };
+      return { ok: true, result };
     }
   }
 });
@@ -929,7 +915,7 @@ var health_default = defineCommand({
       if (cached && !flags.refresh) {
         const health2 = cached;
         outputHealth(health2, flags, ctx);
-        return { exitCode: health2.score < cfg.thresholds.health ? 1 : 0 };
+        return health2.score < cfg.thresholds.health ? { ok: false, error: "Health score is below threshold", result: health2 } : { ok: true, result: health2 };
       }
       const [layering, types] = await Promise.all([
         analyzeLayering({ rootDir: cwd, layerMap: cfg.layers }),
@@ -951,7 +937,7 @@ var health_default = defineCommand({
       });
       await ctx.platform.cache.set(CACHE_KEYS.HEALTH, health, CACHE_TTLS.FAST);
       outputHealth(health, flags, ctx);
-      return { exitCode: health.score < cfg.thresholds.health ? 1 : 0 };
+      return health.score < cfg.thresholds.health ? { ok: false, error: "Health score is below threshold", result: health } : { ok: true, result: health };
     }
   }
 });
@@ -980,13 +966,13 @@ var history_default = defineCommand({
       const { snapshots, delta, latest } = store.history();
       if (snapshots.length === 0) {
         ctx.ui?.success?.("No snapshots yet. Run `kb quality snapshot` first.");
-        return { exitCode: 0 };
+        return { ok: true };
       }
       const limit = flags.limit ?? 10;
       const recent = snapshots.slice(-limit).reverse();
       if (flags.json) {
         ctx.ui?.json?.({ snapshots: recent, delta, latest });
-        return { exitCode: 0 };
+        return { ok: true };
       }
       const items = recent.map((s) => {
         const date = new Date(s.timestamp).toLocaleString();
@@ -1010,7 +996,7 @@ var history_default = defineCommand({
         });
       }
       ctx.ui?.success?.(`Quality history (${recent.length} snapshots)`, { sections });
-      return { exitCode: 0 };
+      return { ok: true };
     }
   }
 });
@@ -1092,7 +1078,7 @@ var snapshot_default = defineCommand({
           }]
         });
       }
-      return { exitCode: 0 };
+      return { ok: true };
     }
   }
 });
@@ -1106,7 +1092,7 @@ var stats_default = defineCommand({
       const stats = await calculateStats(cwd);
       if (flags.json) {
         ctx.ui?.json?.(stats);
-        return { exitCode: 0 };
+        return { ok: true };
       }
       ctx.ui?.success?.("Monorepo statistics", {
         sections: [{
@@ -1118,7 +1104,7 @@ var stats_default = defineCommand({
           ]
         }]
       });
-      return { exitCode: 0 };
+      return { ok: true };
     }
   }
 });
@@ -1143,7 +1129,7 @@ var visualize_default = defineCommand({
       } else {
         outputStats2(graph, flags, ui);
       }
-      return { exitCode: 0 };
+      return { ok: true, result: { graph } };
     }
   }
 });