viberails 0.2.1 → 0.2.3

package/dist/index.cjs

@@ -256,6 +256,23 @@ function resolveIgnoreForFile(relPath, config) {
 var import_node_child_process = require("child_process");
 var fs4 = __toESM(require("fs"), 1);
 var path4 = __toESM(require("path"), 1);
+var ALWAYS_SKIP_DIRS = /* @__PURE__ */ new Set([
+  "node_modules",
+  ".git",
+  "dist",
+  "build",
+  ".next",
+  ".expo",
+  ".output",
+  ".svelte-kit",
+  ".turbo",
+  "coverage",
+  "public",
+  "vendor",
+  "__generated__",
+  "generated",
+  ".viberails"
+]);
 var SOURCE_EXTS = /* @__PURE__ */ new Set([
   ".ts",
   ".tsx",
@@ -275,12 +292,19 @@ var NAMING_PATTERNS = {
 };
 function isIgnored(relPath, ignorePatterns) {
   for (const pattern of ignorePatterns) {
-    if (pattern.endsWith("/**")) {
+    const startsGlob = pattern.startsWith("**/");
+    const endsGlob = pattern.endsWith("/**");
+    if (startsGlob && endsGlob) {
+      const middle = pattern.slice(3, -3);
+      if (relPath.startsWith(`${middle}/`) || relPath.includes(`/${middle}/`) || relPath === middle) {
+        return true;
+      }
+    } else if (endsGlob) {
       const prefix = pattern.slice(0, -3);
       if (relPath.startsWith(`${prefix}/`) || relPath === prefix) return true;
-    } else if (pattern.startsWith("**/")) {
+    } else if (startsGlob) {
       const suffix = pattern.slice(3);
-      if (relPath.endsWith(suffix)) return true;
+      if (relPath.endsWith(suffix) || relPath === suffix) return true;
     } else if (relPath === pattern || relPath.startsWith(`${pattern}/`)) {
       return true;
     }
@@ -304,7 +328,7 @@ function checkNaming(relPath, conventions) {
   const filename = path4.basename(relPath);
   const ext = path4.extname(filename);
   if (!SOURCE_EXTS.has(ext)) return void 0;
-  if (filename.startsWith("index.") || filename.includes(".config.") || filename.includes(".test.") || filename.includes(".spec.") || filename.startsWith(".")) {
+  if (filename.startsWith("index.") || filename.includes(".config.") || filename.includes(".test.") || filename.includes(".spec.") || filename.startsWith(".") || filename.startsWith("_") || filename.startsWith("+") || filename.startsWith("$") || filename.startsWith("[")) {
     return void 0;
   }
   const bare = filename.slice(0, filename.indexOf("."));
@@ -337,7 +361,7 @@ function getAllSourceFiles(projectRoot, config) {
     for (const entry of entries) {
       const rel = path4.relative(projectRoot, path4.join(dir, entry.name));
       if (entry.isDirectory()) {
-        if (entry.name === "node_modules" || entry.name === ".git" || entry.name === "dist") {
+        if (ALWAYS_SKIP_DIRS.has(entry.name)) {
           continue;
         }
         if (isIgnored(rel, config.ignore)) continue;
@@ -400,13 +424,13 @@ function checkMissingTests(projectRoot, config, severity) {
   const testSuffix = testPattern.replace("*", "");
   const sourceFiles = collectSourceFiles(srcPath, projectRoot);
   for (const relFile of sourceFiles) {
-    const basename6 = path5.basename(relFile);
-    if (basename6.includes(".test.") || basename6.includes(".spec.") || basename6.startsWith("index.") || basename6.endsWith(".d.ts")) {
+    const basename7 = path5.basename(relFile);
+    if (basename7.includes(".test.") || basename7.includes(".spec.") || basename7.startsWith("index.") || basename7.endsWith(".d.ts")) {
       continue;
     }
-    const ext = path5.extname(basename6);
+    const ext = path5.extname(basename7);
     if (!SOURCE_EXTS2.has(ext)) continue;
-    const stem = basename6.slice(0, basename6.indexOf("."));
+    const stem = basename7.slice(0, basename7.indexOf("."));
     const expectedTestFile = `${stem}${testSuffix}`;
     const dir = path5.dirname(path5.join(projectRoot, relFile));
     const colocatedTest = path5.join(dir, expectedTestFile);
@@ -427,6 +451,50 @@ function checkMissingTests(projectRoot, config, severity) {
 
 // src/commands/check.ts
 var CONFIG_FILE2 = "viberails.config.json";
+function isTestFile(relPath) {
+  const filename = path6.basename(relPath);
+  return filename.includes(".test.") || filename.includes(".spec.") || filename.startsWith("test.") || filename.startsWith("spec.") || relPath.includes("__tests__/") || relPath.includes("__test__/");
+}
+function printGroupedViolations(violations, limit) {
+  const groups = /* @__PURE__ */ new Map();
+  for (const v of violations) {
+    const existing = groups.get(v.rule) ?? [];
+    existing.push(v);
+    groups.set(v.rule, existing);
+  }
+  const ruleOrder = ["file-size", "file-naming", "missing-test", "boundary-violation"];
+  const sortedKeys = [...groups.keys()].sort(
+    (a, b) => (ruleOrder.indexOf(a) === -1 ? 99 : ruleOrder.indexOf(a)) - (ruleOrder.indexOf(b) === -1 ? 99 : ruleOrder.indexOf(b))
+  );
+  let totalShown = 0;
+  const totalLimit = limit ?? Number.POSITIVE_INFINITY;
+  for (const rule of sortedKeys) {
+    const group = groups.get(rule);
+    if (!group) continue;
+    const remaining = totalLimit - totalShown;
+    if (remaining <= 0) break;
+    const toShow = group.slice(0, remaining);
+    const hidden = group.length - toShow.length;
+    for (const v of toShow) {
+      const icon = v.severity === "error" ? import_chalk2.default.red("\u2717") : import_chalk2.default.yellow("!");
+      console.log(`${icon} ${import_chalk2.default.dim(v.rule)} ${v.file}: ${v.message}`);
+    }
+    totalShown += toShow.length;
+    if (hidden > 0) {
+      console.log(import_chalk2.default.dim(`  ... and ${hidden} more ${rule} violations`));
+    }
+  }
+}
+function printSummary(violations) {
+  const counts = /* @__PURE__ */ new Map();
+  for (const v of violations) {
+    counts.set(v.rule, (counts.get(v.rule) ?? 0) + 1);
+  }
+  const word = violations.length === 1 ? "violation" : "violations";
+  const parts = [...counts.entries()].map(([rule, count]) => `${count} ${rule}`);
+  console.log(`
+${violations.length} ${word} found (${parts.join(", ")}).`);
+}
 async function checkCommand(options, cwd) {
   const startDir = cwd ?? process.cwd();
   const projectRoot = findProjectRoot(startDir);
@@ -463,13 +531,15 @@ async function checkCommand(options, cwd) {
     if (isIgnored(relPath, effectiveIgnore)) continue;
     if (!fs6.existsSync(absPath)) continue;
     const resolved = resolveConfigForFile(relPath, config);
-    if (resolved.rules.maxFileLines > 0) {
+    const testFile = isTestFile(relPath);
+    const maxLines = testFile ? resolved.rules.maxTestFileLines : resolved.rules.maxFileLines;
+    if (maxLines > 0) {
       const lines = countFileLines(absPath);
-      if (lines !== null && lines > resolved.rules.maxFileLines) {
+      if (lines !== null && lines > maxLines) {
         violations.push({
           file: relPath,
           rule: "file-size",
-          message: `${lines} lines (max ${resolved.rules.maxFileLines}). Split into focused modules.`,
+          message: `${lines} lines (max ${maxLines}). Split into focused modules.`,
           severity
         });
       }
@@ -517,13 +587,10 @@ async function checkCommand(options, cwd) {
     console.log(`${import_chalk2.default.green("\u2713")} ${filesToCheck.length} files checked \u2014 no violations`);
     return 0;
   }
-  for (const v of violations) {
-    const icon = v.severity === "error" ? import_chalk2.default.red("\u2717") : import_chalk2.default.yellow("!");
-    console.log(`${icon} ${import_chalk2.default.dim(v.rule)} ${v.file}: ${v.message}`);
+  if (!options.quiet) {
+    printGroupedViolations(violations, options.limit);
   }
-  const word = violations.length === 1 ? "violation" : "violations";
-  console.log(`
-${violations.length} ${word} found.`);
+  printSummary(violations);
   if (config.enforcement === "enforce") {
     console.log(import_chalk2.default.red("Fix violations before committing."));
     return 1;
@@ -771,8 +838,8 @@ var path9 = __toESM(require("path"), 1);
 function generateTestStub(sourceRelPath, config, projectRoot) {
   const { testPattern } = config.structure;
   if (!testPattern) return null;
-  const basename6 = path9.basename(sourceRelPath);
-  const stem = basename6.slice(0, basename6.indexOf("."));
+  const basename7 = path9.basename(sourceRelPath);
+  const stem = basename7.slice(0, basename7.indexOf("."));
   const testSuffix = testPattern.replace("*", "");
   const testFilename = `${stem}${testSuffix}`;
   const dir = path9.dirname(path9.join(projectRoot, sourceRelPath));
@@ -812,7 +879,7 @@ async function fixCommand(options, cwd) {
     return 1;
   }
   const config = await (0, import_config3.loadConfig)(configPath);
-  if (!options.yes && !options.dryRun) {
+  if (!options.dryRun) {
     const isDirty = checkGitDirty(projectRoot);
     if (isDirty) {
       console.log(
@@ -1343,7 +1410,7 @@ ${import_chalk9.default.bold("Synced:")}`);
 }
 
 // src/index.ts
-var VERSION = "0.2.1";
+var VERSION = "0.2.3";
 var program = new import_commander.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)").action(async (options) => {
@@ -1364,19 +1431,21 @@ program.command("sync").description("Re-scan and update generated files").action
     process.exit(1);
   }
 });
-program.command("check").description("Check files against enforced rules").option("--staged", "Check only staged files (for pre-commit hooks)").option("--files <files...>", "Check specific files").option("--no-boundaries", "Skip boundary checking").action(async (options) => {
-  try {
-    const exitCode = await checkCommand({
-      ...options,
-      noBoundaries: options.boundaries === false
-    });
-    process.exit(exitCode);
-  } catch (err) {
-    const message = err instanceof Error ? err.message : String(err);
-    console.error(`${import_chalk10.default.red("Error:")} ${message}`);
-    process.exit(1);
+program.command("check").description("Check files against enforced rules").option("--staged", "Check only staged files (for pre-commit hooks)").option("--files <files...>", "Check specific files").option("--no-boundaries", "Skip boundary checking").option("--quiet", "Show only summary counts, not individual violations").option("--limit <n>", "Maximum number of violations to display", Number.parseInt).action(
+  async (options) => {
+    try {
+      const exitCode = await checkCommand({
+        ...options,
+        noBoundaries: options.boundaries === false
+      });
+      process.exit(exitCode);
+    } catch (err) {
+      const message = err instanceof Error ? err.message : String(err);
+      console.error(`${import_chalk10.default.red("Error:")} ${message}`);
+      process.exit(1);
+    }
   }
-});
+);
 program.command("fix").description("Auto-fix file naming violations and generate missing test stubs").option("--dry-run", "Show planned fixes without applying them").option("--rule <rules...>", "Fix only specific rules (file-naming, missing-test)").option("-y, --yes", "Skip confirmation prompt").action(async (options) => {
   try {
     const exitCode = await fixCommand(options);