@ipation/specbridge 1.0.3 → 1.0.4

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") {
@@ -3101,7 +3111,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 +3120,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 +3246,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}`);
     }