viberails 0.5.3 → 0.5.5

package/dist/index.js

@@ -33,7 +33,7 @@ import * as clack5 from "@clack/prompts";
 // src/utils/prompt-integrations.ts
 import { spawnSync } from "child_process";
 import * as clack from "@clack/prompts";
-async function promptHookManagerInstall(projectRoot, packageManager) {
+async function promptHookManagerInstall(projectRoot, packageManager, isWorkspace) {
   const choice = await clack.select({
     message: "No git hook manager detected. Install Lefthook for shareable pre-commit hooks?",
     options: [
@@ -52,7 +52,7 @@ async function promptHookManagerInstall(projectRoot, packageManager) {
   assertNotCancelled(choice);
   if (choice !== "install") return void 0;
   const pm = packageManager || "npm";
-  const installCmd = pm === "yarn" ? "yarn add -D lefthook" : pm === "pnpm" ? "pnpm add -D lefthook" : "npm install -D lefthook";
+  const installCmd = pm === "yarn" ? "yarn add -D lefthook" : pm === "pnpm" ? `pnpm add -D${isWorkspace ? " -w" : ""} lefthook` : "npm install -D lefthook";
   const s = clack.spinner();
   s.start("Installing Lefthook...");
   const result = spawnSync(installCmd, {
@@ -80,7 +80,8 @@ async function promptIntegrations(projectRoot, hookManager, tools) {
   if (!resolvedHookManager) {
     resolvedHookManager = await promptHookManagerInstall(
       projectRoot,
-      tools?.packageManager ?? "npm"
+      tools?.packageManager ?? "npm",
+      tools?.isWorkspace
     );
   }
   const isBareHook = !resolvedHookManager;
@@ -97,7 +98,7 @@ async function promptIntegrations(projectRoot, hookManager, tools) {
     options.push({
       value: "typecheck",
       label: "Typecheck (tsc --noEmit)",
-      hint: "catches type errors before commit"
+      hint: "pre-commit hook + CI check"
     });
   }
   if (tools?.linter) {
@@ -105,7 +106,7 @@ async function promptIntegrations(projectRoot, hookManager, tools) {
     options.push({
       value: "lint",
       label: `Lint check (${linterName})`,
-      hint: "runs linter on staged files before commit"
+      hint: "pre-commit hook + CI check"
     });
   }
   options.push(
@@ -1420,7 +1421,8 @@ function formatPackageSummary(pkg) {
   if (pkg.stack.styling) {
     parts.push(formatItem(pkg.stack.styling, STYLING_NAMES));
   }
-  const files = `${pkg.statistics.totalFiles} files`;
+  const n = pkg.statistics.totalFiles;
+  const files = `${n} ${n === 1 ? "file" : "files"}`;
   const detail = parts.length > 0 ? `${parts.join(", ")} (${files})` : `(${files})`;
   return `  ${pkg.relativePath} \u2014 ${detail}`;
 }
@@ -1476,7 +1478,8 @@ function formatPackageSummaryPlain(pkg) {
   if (pkg.stack.styling) {
     parts.push(formatItem(pkg.stack.styling, STYLING_NAMES));
   }
-  const files = `${pkg.statistics.totalFiles} files`;
+  const n = pkg.statistics.totalFiles;
+  const files = `${n} ${n === 1 ? "file" : "files"}`;
   const detail = parts.length > 0 ? `${parts.join(", ")} (${files})` : `(${files})`;
   return `  ${pkg.relativePath} \u2014 ${detail}`;
 }
@@ -2479,7 +2482,8 @@ function checkCoveragePrereqs(projectRoot, scanResult) {
       return hasDependency(pkgDir, "@vitest/coverage-v8") || hasDependency(pkgDir, "@vitest/coverage-istanbul");
     });
   }
-  const addCmd = pm === "yarn" ? "yarn add -D" : pm === "npm" ? "npm install -D" : `${pm} add -D`;
+  const isWorkspace = scanResult.packages.length > 1;
+  const addCmd = pm === "yarn" ? "yarn add -D" : pm === "pnpm" && isWorkspace ? "pnpm add -D -w" : pm === "npm" ? "npm install -D" : `${pm} add -D`;
   const affectedPackages = vitestPackages.length > 1 ? vitestPackages : void 0;
   const reason = affectedPackages ? `Required for coverage in: ${affectedPackages.join(", ")}` : "Required for coverage percentage checks with vitest";
   return [
@@ -2680,7 +2684,6 @@ function addLefthookPreCommit(lefthookPath) {
 function detectHookManager(projectRoot) {
   if (fs16.existsSync(path16.join(projectRoot, "lefthook.yml"))) return "Lefthook";
   if (fs16.existsSync(path16.join(projectRoot, ".husky"))) return "Husky";
-  if (fs16.existsSync(path16.join(projectRoot, ".git"))) return "git hook";
   return void 0;
 }
 function setupClaudeCodeHook(projectRoot) {
@@ -2734,7 +2737,7 @@ function setupClaudeMdReference(projectRoot) {
   fs16.writeFileSync(claudeMdPath, prefix + ref);
   console.log(`  ${chalk9.green("\u2713")} CLAUDE.md \u2014 added @.viberails/context.md reference`);
 }
-function setupGithubAction(projectRoot, packageManager) {
+function setupGithubAction(projectRoot, packageManager, options) {
   const workflowDir = path16.join(projectRoot, ".github", "workflows");
   const workflowPath = path16.join(workflowDir, "viberails.yml");
   if (fs16.existsSync(workflowPath)) {
@@ -2770,7 +2773,16 @@ function setupGithubAction(projectRoot, packageManager) {
     "          node-version: 22",
     pm !== "npm" ? `          cache: ${pm}` : "",
     "",
-    `      - run: ${installCmd}`,
+    `      - run: ${installCmd}`
+  );
+  if (options?.typecheck) {
+    lines.push(`      - run: ${runPrefix} tsc --noEmit`);
+  }
+  if (options?.linter) {
+    const lintCmd = options.linter === "biome" ? "biome check ." : "eslint .";
+    lines.push(`      - run: ${runPrefix} ${lintCmd}`);
+  }
+  lines.push(
     `      - run: ${runPrefix} viberails check --enforce --diff-base origin/\${{ github.event.pull_request.base.ref }}`,
     ""
   );
@@ -2891,7 +2903,10 @@ function setupSelectedIntegrations(projectRoot, integrations, opts) {
     created.push("CLAUDE.md \u2014 added @.viberails/context.md reference");
   }
   if (integrations.githubAction) {
-    const t = setupGithubAction(projectRoot, opts.packageManager ?? "npm");
+    const t = setupGithubAction(projectRoot, opts.packageManager ?? "npm", {
+      linter: integrations.lintHook ? opts.linter : void 0,
+      typecheck: integrations.typecheckHook
+    });
     if (t) created.push(`${t} \u2014 blocks PRs on violations`);
   }
   return created;
@@ -2959,11 +2974,15 @@ async function initNonInteractive(projectRoot, configPath) {
   setupClaudeMdReference(projectRoot);
   const rootPkg = config.packages[0];
   const rootPkgPm = rootPkg?.stack?.packageManager ?? "npm";
-  const actionTarget = setupGithubAction(projectRoot, rootPkgPm);
+  const linter = rootPkg?.stack?.linter?.split("@")[0];
+  const isTypeScript = rootPkg?.stack?.language === "typescript";
+  const actionTarget = setupGithubAction(projectRoot, rootPkgPm, {
+    linter,
+    typecheck: isTypeScript
+  });
   const hookManager = detectHookManager(projectRoot);
   const hasHookManager = hookManager === "Lefthook" || hookManager === "Husky";
   const preCommitTarget = hasHookManager ? setupPreCommitHook(projectRoot) : void 0;
-  const linter = rootPkg?.stack?.linter?.split("@")[0];
   const ok = chalk11.green("\u2713");
   const created = [
     `${ok} ${path18.basename(configPath)}`,
@@ -2996,13 +3015,6 @@ async function initInteractive(projectRoot, configPath, options) {
   const scanResult = await scan2(projectRoot);
   const config = generateConfig(scanResult);
   s.stop("Scan complete");
-  const prereqResult = await promptMissingPrereqs(
-    projectRoot,
-    checkCoveragePrereqs(projectRoot, scanResult)
-  );
-  if (prereqResult.disableCoverage) {
-    config.rules.testCoverage = 0;
-  }
   if (scanResult.statistics.totalFiles === 0) {
     clack8.log.warn(
       "No source files detected. Try running from the project root,\nor check that source files exist. Run viberails sync after adding files."
@@ -3043,20 +3055,29 @@ async function initInteractive(projectRoot, configPath, options) {
       if (denyCount > 0) {
         config.boundaries = inferred;
         config.rules.enforceBoundaries = true;
-        bs.stop(`Inferred ${denyCount} boundary rules`);
-        const boundaryLines = Object.entries(inferred.deny).map(([pkg, denied]) => `${pkg} must NOT import from: ${denied.join(", ")}`).join("\n");
-        clack8.note(boundaryLines, "Boundary rules");
+        const pkgCount = Object.keys(inferred.deny).length;
+        bs.stop(`Inferred ${denyCount} boundary rules across ${pkgCount} packages`);
       } else {
         bs.stop("No boundary rules inferred");
       }
     }
   }
   const hookManager = detectHookManager(projectRoot);
+  const coveragePrereqs = checkCoveragePrereqs(projectRoot, scanResult);
+  const hasMissingPrereqs = coveragePrereqs.some((p) => !p.installed) || !hookManager;
+  if (hasMissingPrereqs) {
+    clack8.log.info("Some dependencies are needed for full functionality.");
+  }
+  const prereqResult = await promptMissingPrereqs(projectRoot, coveragePrereqs);
+  if (prereqResult.disableCoverage) {
+    config.rules.testCoverage = 0;
+  }
   const rootPkgStack = (config.packages.find((p) => p.path === ".") ?? config.packages[0])?.stack;
   const integrations = await promptIntegrations(projectRoot, hookManager, {
     isTypeScript: rootPkgStack?.language === "typescript",
     linter: rootPkgStack?.linter?.split("@")[0],
-    packageManager: rootPkgStack?.packageManager
+    packageManager: rootPkgStack?.packageManager,
+    isWorkspace: config.packages.length > 1
   });
   const shouldWrite = await confirm3("Write configuration and set up selected integrations?");
   if (!shouldWrite) {
@@ -3068,17 +3089,14 @@ async function initInteractive(projectRoot, configPath, options) {
 `);
   writeGeneratedFiles(projectRoot, config, scanResult);
   updateGitignore(projectRoot);
-  const createdFiles = [
-    path18.basename(configPath),
-    ".viberails/context.md",
-    ".viberails/scan-result.json",
-    ...setupSelectedIntegrations(projectRoot, integrations, {
-      linter: rootPkgStack?.linter?.split("@")[0],
-      packageManager: rootPkgStack?.packageManager
-    })
-  ];
-  clack8.log.success(`Created:
-${createdFiles.map((f) => `  ${f}`).join("\n")}`);
+  const ok = chalk11.green("\u2713");
+  clack8.log.step(`${ok} ${path18.basename(configPath)}`);
+  clack8.log.step(`${ok} .viberails/context.md`);
+  clack8.log.step(`${ok} .viberails/scan-result.json`);
+  setupSelectedIntegrations(projectRoot, integrations, {
+    linter: rootPkgStack?.linter?.split("@")[0],
+    packageManager: rootPkgStack?.packageManager
+  });
   clack8.outro(
     `Done! Next: review viberails.config.json, then run viberails check
   ${chalk11.dim("Tip: use")} ${chalk11.cyan("viberails check --enforce")} ${chalk11.dim("in CI to block PRs on violations.")}`
@@ -3192,7 +3210,7 @@ ${chalk12.bold("Synced:")}`);
 }
 
 // src/index.ts
-var VERSION = "0.5.3";
+var VERSION = "0.5.5";
 var program = new Command();
 program.name("viberails").description("Guardrails for vibe coding").version(VERSION);
 program.command("init", { isDefault: true }).description("Scan your project and set up enforcement guardrails").option("-y, --yes", "Non-interactive mode (use defaults, high-confidence only)").option("-f, --force", "Re-initialize, replacing existing config").action(async (options) => {