@ipation/specbridge 1.0.3 → 1.0.5

package/CHANGELOG.md

@@ -7,6 +7,74 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.0.4] - 2026-01-30
+
+### Fixed
+
+#### Critical Fixes
+- **Verification Timeout Bug** (src/verification/engine.ts)
+  - Fixed event loop hanging issue where verification would wait for full 60-second timeout even after completing
+  - Added proper timeout handle cleanup with try-finally block
+  - Used `unref()` to prevent blocking process exit
+  - **Impact**: Integration tests now complete in ~97 seconds instead of hanging for 10+ minutes
+  - CLI verify command exits immediately (~600ms) instead of waiting 60 seconds
+
+#### Inference Issues
+- **Non-Existent Verifier References** (src/inference/analyzers/errors.ts)
+  - Fixed `error-hierarchy` verifier reference → `errors` (line 90)
+  - Fixed `custom-errors-only` verifier reference → `errors` (line 205)
+  - Updated corresponding unit tests to match
+  - **Impact**: Inferred constraints now reference valid verifiers that exist in the registry
+
+#### CLI Command Issues
+- **`infer --output` Bug** (src/cli/commands/infer.ts)
+  - Fixed early return preventing file save when no patterns detected
+  - Moved file saving logic before pattern check
+  - **Impact**: Output file is always created when `--output` flag is used, even with empty results
+
+#### Visual Formatting
+- **Empty Placeholder Characters** (src/reporting/formats/markdown.ts, src/agent/context.generator.ts)
+  - Added compliance emojis: ✅ (≥90%), ⚠️ (70-89%), ❌ (<70%)
+  - Added progress bar characters: █ (filled), ░ (empty)
+  - Added constraint type emojis: 🔒 (invariant), 📋 (convention), 💡 (guideline)
+  - **Impact**: Reports and agent context now display with proper visual indicators
+
+### Changed
+
+#### Documentation
+- **Version Consistency**
+  - Archived outdated `IMPLEMENTATION_STATUS.md` → `IMPLEMENTATION_STATUS_v0.2.2_ARCHIVED.md`
+  - Added archive notice indicating current version is 1.0.4
+  - **Impact**: No misleading version information
+
+- **GitHub URL Standardization**
+  - Standardized all repository references to `nouatzi/specbridge`
+  - Fixed docs/README.md, docs/troubleshooting.md, src/reporting/formats/markdown.ts
+  - Corrected wrong `anthropics/claude-code` reference in archived file
+  - **Impact**: Consistent repository URLs throughout documentation and code
+
+### Improved
+
+#### Test Suite
+- **Integration Tests** (tests/integration/cli.test.ts)
+  - Fixed `decision create` test syntax (positional argument instead of `--id` flag)
+  - Added valid constraints to manually created decision files (schema compliance)
+  - Fixed 3 previously failing CLI integration tests
+  - **Result**: All 30 integration tests now pass (100% success rate)
+  - **Duration**: Tests complete in ~97 seconds (down from 10+ minutes)
+
+### Test Results Summary
+- ✅ Type Checking: Passed
+- ✅ Unit Tests: 762/762 passed (24 test files)
+- ✅ Integration Tests: 30/30 passed (2 test files)
+- ✅ Total Duration: ~97 seconds
+
+### Files Modified
+- 9 source/test files updated
+- 2 documentation files updated
+- 1 file archived
+- All changes backward compatible (no breaking changes)
+
 ## [1.0.3] - 2026-01-30
 
 ### Improved

package/dist/cli.js

@@ -1045,7 +1045,7 @@ var ErrorsAnalyzer = class {
           rule: `Custom error classes should extend ${customBaseName}`,
           severity: "medium",
           scope: "src/**/*.ts",
-          verifier: "error-hierarchy"
+          verifier: "errors"
         }
       });
     }
@@ -1142,7 +1142,7 @@ var ErrorsAnalyzer = class {
           rule: "Throw custom error classes instead of generic Error",
           severity: "medium",
           scope: "src/**/*.ts",
-          verifier: "custom-errors-only"
+          verifier: "errors"
         }
       });
     }
@@ -1281,6 +1281,12 @@ var inferCommand = new Command2("infer").description("Analyze codebase and detec
       cwd
     });
     spinner.succeed(`Scanned ${result.filesScanned} files in ${result.duration}ms`);
+    if (options.save || options.output) {
+      const outputPath = options.output || join3(getInferredDir(cwd), "patterns.json");
+      await writeTextFile(outputPath, JSON.stringify(result, null, 2));
+      console.log(chalk2.green(`
+Results saved to: ${outputPath}`));
+    }
     if (result.patterns.length === 0) {
       console.log(chalk2.yellow("\nNo patterns detected above confidence threshold."));
       console.log(chalk2.dim(`Try lowering --min-confidence (current: ${minConfidence})`));
@@ -1291,12 +1297,6 @@ var inferCommand = new Command2("infer").description("Analyze codebase and detec
     } else {
       printPatterns(result.patterns);
     }
-    if (options.save || options.output) {
-      const outputPath = options.output || join3(getInferredDir(cwd), "patterns.json");
-      await writeTextFile(outputPath, JSON.stringify(result, null, 2));
-      console.log(chalk2.green(`
-Results saved to: ${outputPath}`));
-    }
     if (!options.json) {
       console.log("");
       console.log(chalk2.cyan("Next steps:"));
@@ -2128,9 +2128,11 @@ var VerificationEngine = class {
     let passed = 0;
     let failed = 0;
     const skipped = 0;
-    const timeoutPromise = new Promise(
-      (resolve) => setTimeout(() => resolve("timeout"), timeout)
-    );
+    let timeoutHandle = null;
+    const timeoutPromise = new Promise((resolve) => {
+      timeoutHandle = setTimeout(() => resolve("timeout"), timeout);
+      timeoutHandle.unref();
+    });
     const verificationPromise = this.verifyFiles(
       filesToVerify,
       decisions,
@@ -2146,17 +2148,25 @@ var VerificationEngine = class {
         }
       }
     );
-    const result = await Promise.race([verificationPromise, timeoutPromise]);
-    if (result === "timeout") {
-      return {
-        success: false,
-        violations: allViolations,
-        checked,
-        passed,
-        failed,
-        skipped: filesToVerify.length - checked,
-        duration: timeout
-      };
+    let result;
+    try {
+      result = await Promise.race([verificationPromise, timeoutPromise]);
+      if (result === "timeout") {
+        return {
+          success: false,
+          violations: allViolations,
+          checked,
+          passed,
+          failed,
+          skipped: filesToVerify.length - checked,
+          duration: timeout
+        };
+      }
+    } finally {
+      if (timeoutHandle) {
+        clearTimeout(timeoutHandle);
+        timeoutHandle = null;
+      }
     }
     const hasBlockingViolations = allViolations.some((v) => {
       if (level === "commit") {
@@ -2752,38 +2762,22 @@ npx specbridge hook run --level commit --files "$STAGED_FILES"
 
 exit $?
 `;
-var hookCommand = new Command9("hook").description("Manage Git hooks for verification");
-hookCommand.command("install").description("Install Git pre-commit hook").option("-f, --force", "Overwrite existing hook").option("--husky", "Install for husky").option("--lefthook", "Install for lefthook").action(async (options) => {
-  const cwd = process.cwd();
-  if (!await pathExists(getSpecBridgeDir(cwd))) {
-    throw new NotInitializedError();
-  }
-  const spinner = ora5("Detecting hook system...").start();
-  try {
-    let hookPath;
-    let hookContent;
-    if (options.husky) {
-      hookPath = join7(cwd, ".husky", "pre-commit");
-      hookContent = HOOK_SCRIPT;
-      spinner.text = "Installing husky pre-commit hook...";
-    } else if (options.lefthook) {
-      spinner.succeed("Lefthook detected");
-      console.log("");
-      console.log(chalk8.cyan("Add this to your lefthook.yml:"));
-      console.log("");
-      console.log(chalk8.dim(`pre-commit:
-  commands:
-    specbridge:
-      glob: "*.{ts,tsx}"
-      run: npx specbridge hook run --level commit --files {staged_files}
-`));
-      return;
-    } else {
-      if (await pathExists(join7(cwd, ".husky"))) {
+function createHookCommand() {
+  const hookCommand2 = new Command9("hook").description("Manage Git hooks for verification");
+  hookCommand2.command("install").description("Install Git pre-commit hook").option("-f, --force", "Overwrite existing hook").option("--husky", "Install for husky").option("--lefthook", "Install for lefthook").action(async (options) => {
+    const cwd = process.cwd();
+    if (!await pathExists(getSpecBridgeDir(cwd))) {
+      throw new NotInitializedError();
+    }
+    const spinner = ora5("Detecting hook system...").start();
+    try {
+      let hookPath;
+      let hookContent;
+      if (options.husky) {
         hookPath = join7(cwd, ".husky", "pre-commit");
         hookContent = HOOK_SCRIPT;
         spinner.text = "Installing husky pre-commit hook...";
-      } else if (await pathExists(join7(cwd, "lefthook.yml"))) {
+      } else if (options.lefthook) {
         spinner.succeed("Lefthook detected");
         console.log("");
         console.log(chalk8.cyan("Add this to your lefthook.yml:"));
@@ -2796,97 +2790,117 @@ hookCommand.command("install").description("Install Git pre-commit hook").option
 `));
         return;
       } else {
-        hookPath = join7(cwd, ".git", "hooks", "pre-commit");
-        hookContent = HOOK_SCRIPT;
-        spinner.text = "Installing Git pre-commit hook...";
+        if (await pathExists(join7(cwd, ".husky"))) {
+          hookPath = join7(cwd, ".husky", "pre-commit");
+          hookContent = HOOK_SCRIPT;
+          spinner.text = "Installing husky pre-commit hook...";
+        } else if (await pathExists(join7(cwd, "lefthook.yml"))) {
+          spinner.succeed("Lefthook detected");
+          console.log("");
+          console.log(chalk8.cyan("Add this to your lefthook.yml:"));
+          console.log("");
+          console.log(chalk8.dim(`pre-commit:
+  commands:
+    specbridge:
+      glob: "*.{ts,tsx}"
+      run: npx specbridge hook run --level commit --files {staged_files}
+`));
+          return;
+        } else {
+          hookPath = join7(cwd, ".git", "hooks", "pre-commit");
+          hookContent = HOOK_SCRIPT;
+          spinner.text = "Installing Git pre-commit hook...";
+        }
       }
+      if (await pathExists(hookPath) && !options.force) {
+        spinner.fail("Hook already exists");
+        console.log(chalk8.yellow(`Use --force to overwrite: ${hookPath}`));
+        return;
+      }
+      await writeTextFile(hookPath, hookContent);
+      const { execSync } = await import("child_process");
+      try {
+        execSync(`chmod +x "${hookPath}"`, { stdio: "ignore" });
+      } catch {
+      }
+      spinner.succeed("Pre-commit hook installed");
+      console.log(chalk8.dim(`  Path: ${hookPath}`));
+      console.log("");
+      console.log(chalk8.cyan("The hook will run on each commit and verify staged files."));
+    } catch (error) {
+      spinner.fail("Failed to install hook");
+      throw error;
     }
-    if (await pathExists(hookPath) && !options.force) {
-      spinner.fail("Hook already exists");
-      console.log(chalk8.yellow(`Use --force to overwrite: ${hookPath}`));
-      return;
+  });
+  hookCommand2.command("run").description("Run verification (called by hook)").option("-l, --level <level>", "Verification level", "commit").option("-f, --files <files>", "Space or comma-separated file list").action(async (options) => {
+    const cwd = process.cwd();
+    if (!await pathExists(getSpecBridgeDir(cwd))) {
+      throw new NotInitializedError();
     }
-    await writeTextFile(hookPath, hookContent);
-    const { execSync } = await import("child_process");
     try {
-      execSync(`chmod +x "${hookPath}"`, { stdio: "ignore" });
-    } catch {
-    }
-    spinner.succeed("Pre-commit hook installed");
-    console.log(chalk8.dim(`  Path: ${hookPath}`));
-    console.log("");
-    console.log(chalk8.cyan("The hook will run on each commit and verify staged files."));
-  } catch (error) {
-    spinner.fail("Failed to install hook");
-    throw error;
-  }
-});
-hookCommand.command("run").description("Run verification (called by hook)").option("-l, --level <level>", "Verification level", "commit").option("-f, --files <files>", "Space or comma-separated file list").action(async (options) => {
-  const cwd = process.cwd();
-  if (!await pathExists(getSpecBridgeDir(cwd))) {
-    throw new NotInitializedError();
-  }
-  try {
-    const config = await loadConfig(cwd);
-    const level = options.level || "commit";
-    const files = options.files ? options.files.split(/[\s,]+/).filter((f) => f.length > 0) : void 0;
-    if (!files || files.length === 0) {
-      process.exit(0);
-    }
-    const engine = createVerificationEngine();
-    const result = await engine.verify(config, {
-      level,
-      files,
-      cwd
-    });
-    if (result.violations.length === 0) {
-      console.log(chalk8.green("\u2713 SpecBridge: All checks passed"));
-      process.exit(0);
-    }
-    console.log(chalk8.red(`\u2717 SpecBridge: ${result.violations.length} violation(s) found`));
-    console.log("");
-    for (const v of result.violations) {
-      const location = v.line ? `:${v.line}` : "";
-      console.log(`  ${v.file}${location}: ${v.message}`);
-      console.log(chalk8.dim(`    [${v.severity}] ${v.decisionId}/${v.constraintId}`));
+      const config = await loadConfig(cwd);
+      const level = options.level || "commit";
+      const files = options.files ? options.files.split(/[\s,]+/).filter((f) => f.length > 0) : void 0;
+      if (!files || files.length === 0) {
+        process.exit(0);
+      }
+      const engine = createVerificationEngine();
+      const result = await engine.verify(config, {
+        level,
+        files,
+        cwd
+      });
+      if (result.violations.length === 0) {
+        console.log(chalk8.green("\u2713 SpecBridge: All checks passed"));
+        process.exit(0);
+      }
+      console.log(chalk8.red(`\u2717 SpecBridge: ${result.violations.length} violation(s) found`));
+      console.log("");
+      for (const v of result.violations) {
+        const location = v.line ? `:${v.line}` : "";
+        console.log(`  ${v.file}${location}: ${v.message}`);
+        console.log(chalk8.dim(`    [${v.severity}] ${v.decisionId}/${v.constraintId}`));
+      }
+      console.log("");
+      console.log(chalk8.yellow("Run `specbridge verify` for full details."));
+      process.exit(result.success ? 0 : 1);
+    } catch (error) {
+      console.error(chalk8.red("SpecBridge verification failed"));
+      console.error(error instanceof Error ? error.message : String(error));
+      process.exit(1);
     }
-    console.log("");
-    console.log(chalk8.yellow("Run `specbridge verify` for full details."));
-    process.exit(result.success ? 0 : 1);
-  } catch (error) {
-    console.error(chalk8.red("SpecBridge verification failed"));
-    console.error(error instanceof Error ? error.message : String(error));
-    process.exit(1);
-  }
-});
-hookCommand.command("uninstall").description("Remove Git pre-commit hook").action(async () => {
-  const cwd = process.cwd();
-  const spinner = ora5("Removing hook...").start();
-  try {
-    const hookPaths = [
-      join7(cwd, ".husky", "pre-commit"),
-      join7(cwd, ".git", "hooks", "pre-commit")
-    ];
-    let removed = false;
-    for (const hookPath of hookPaths) {
-      if (await pathExists(hookPath)) {
-        const content = await readTextFile(hookPath);
-        if (content.includes("SpecBridge")) {
-          const { unlink } = await import("fs/promises");
-          await unlink(hookPath);
-          spinner.succeed(`Removed hook: ${hookPath}`);
-          removed = true;
+  });
+  hookCommand2.command("uninstall").description("Remove Git pre-commit hook").action(async () => {
+    const cwd = process.cwd();
+    const spinner = ora5("Removing hook...").start();
+    try {
+      const hookPaths = [
+        join7(cwd, ".husky", "pre-commit"),
+        join7(cwd, ".git", "hooks", "pre-commit")
+      ];
+      let removed = false;
+      for (const hookPath of hookPaths) {
+        if (await pathExists(hookPath)) {
+          const content = await readTextFile(hookPath);
+          if (content.includes("SpecBridge")) {
+            const { unlink } = await import("fs/promises");
+            await unlink(hookPath);
+            spinner.succeed(`Removed hook: ${hookPath}`);
+            removed = true;
+          }
         }
       }
+      if (!removed) {
+        spinner.info("No SpecBridge hooks found");
+      }
+    } catch (error) {
+      spinner.fail("Failed to remove hook");
+      throw error;
     }
-    if (!removed) {
-      spinner.info("No SpecBridge hooks found");
-    }
-  } catch (error) {
-    spinner.fail("Failed to remove hook");
-    throw error;
-  }
-});
+  });
+  return hookCommand2;
+}
+var hookCommand = createHookCommand();
 
 // src/cli/commands/report.ts
 import { Command as Command10 } from "commander";
@@ -3101,7 +3115,7 @@ function formatMarkdownReport(report) {
     lines.push("| Decision | Status | Constraints | Violations | Compliance |");
     lines.push("|----------|--------|-------------|------------|------------|");
     for (const dec of report.byDecision) {
-      const complianceEmoji = dec.compliance >= 90 ? "" : dec.compliance >= 70 ? "" : "";
+      const complianceEmoji = dec.compliance >= 90 ? "\u2705" : dec.compliance >= 70 ? "\u26A0\uFE0F" : "\u274C";
       lines.push(
         `| ${dec.title} | ${dec.status} | ${dec.constraints} | ${dec.violations} | ${complianceEmoji} ${dec.compliance}% |`
       );
@@ -3110,15 +3124,15 @@ function formatMarkdownReport(report) {
   }
   lines.push("---");
   lines.push("");
-  lines.push("*Generated by [SpecBridge](https://github.com/specbridge/specbridge)*");
+  lines.push("*Generated by [SpecBridge](https://github.com/nouatzi/specbridge)*");
   return lines.join("\n");
 }
 function formatProgressBar(percentage) {
   const width = 20;
   const filled = Math.round(percentage / 100 * width);
   const empty = width - filled;
-  const filledChar = "";
-  const emptyChar = "";
+  const filledChar = "\u2588";
+  const emptyChar = "\u2591";
   return `\`${filledChar.repeat(filled)}${emptyChar.repeat(empty)}\` ${percentage}%`;
 }
 
@@ -3236,7 +3250,7 @@ function formatContextAsMarkdown(context) {
     lines.push("### Constraints");
     lines.push("");
     for (const constraint of decision.constraints) {
-      const typeEmoji = constraint.type === "invariant" ? "" : constraint.type === "convention" ? "" : "";
+      const typeEmoji = constraint.type === "invariant" ? "\u{1F512}" : constraint.type === "convention" ? "\u{1F4CB}" : "\u{1F4A1}";
       const severityBadge = `[${constraint.severity.toUpperCase()}]`;
       lines.push(`- ${typeEmoji} **${severityBadge}** ${constraint.rule}`);
     }
@@ -3293,7 +3307,7 @@ var contextCommand = new Command11("context").description("Generate architectura
     const config = await loadConfig(cwd);
     const output = await generateFormattedContext(file, config, {
       format: options.format,
-      includeRationale: options.noRationale !== true,
+      includeRationale: options.rationale !== false,
       cwd
     });
     if (options.output) {