@ipation/specbridge 1.0.2 → 1.0.4

package/dist/index.js

@@ -1323,9 +1323,7 @@ var ErrorsAnalyzer = class {
     );
     if (errorClasses.length < 2) return null;
     const files = scanner.getFiles();
-    let extendsError = 0;
-    let extendsCustomBase = 0;
-    let customBaseName = null;
+    const baseCount = /* @__PURE__ */ new Map();
     for (const errorClass of errorClasses) {
       const file = files.find((f) => f.path === errorClass.file);
       if (!file) continue;
@@ -1334,22 +1332,27 @@ var ErrorsAnalyzer = class {
       const extendClause = classDecl.getExtends();
       if (extendClause) {
         const baseName = extendClause.getText();
-        if (baseName === "Error") {
-          extendsError++;
-        } else if (baseName.endsWith("Error")) {
-          extendsCustomBase++;
-          customBaseName = customBaseName || baseName;
+        if (baseName !== "Error" && baseName.endsWith("Error")) {
+          baseCount.set(baseName, (baseCount.get(baseName) || 0) + 1);
         }
       }
     }
-    if (extendsCustomBase >= 2 && customBaseName) {
-      const confidence = calculateConfidence(extendsCustomBase, errorClasses.length);
+    let customBaseName = null;
+    let maxCount = 0;
+    for (const [baseName, count] of baseCount.entries()) {
+      if (count > maxCount) {
+        maxCount = count;
+        customBaseName = baseName;
+      }
+    }
+    if (maxCount >= 3 && customBaseName) {
+      const confidence = calculateConfidence(maxCount, errorClasses.length);
       return createPattern(this.id, {
         id: "errors-custom-base",
         name: "Custom Error Base Class",
         description: `Custom errors extend a common base class (${customBaseName})`,
         confidence,
-        occurrences: extendsCustomBase,
+        occurrences: maxCount,
         examples: errorClasses.slice(0, 3).map((c) => ({
           file: c.file,
           line: c.line,
@@ -1360,7 +1363,7 @@ var ErrorsAnalyzer = class {
           rule: `Custom error classes should extend ${customBaseName}`,
           severity: "medium",
           scope: "src/**/*.ts",
-          verifier: "error-hierarchy"
+          verifier: "errors"
         }
       });
     }
@@ -1431,10 +1434,11 @@ var ErrorsAnalyzer = class {
             } else if (text.startsWith("new ") && text.includes("Error")) {
               throwCustom++;
               if (examples.length < 3) {
+                const snippet = text.length > 50 ? text.slice(0, 50) + "..." : text;
                 examples.push({
                   file: path,
                   line: node.getStartLineNumber(),
-                  snippet: `throw ${text.slice(0, 50)}${text.length > 50 ? "..." : ""}`
+                  snippet: `throw ${snippet}`
                 });
               }
             }
@@ -1456,7 +1460,7 @@ var ErrorsAnalyzer = class {
           rule: "Throw custom error classes instead of generic Error",
           severity: "medium",
           scope: "src/**/*.ts",
-          verifier: "custom-errors-only"
+          verifier: "errors"
         }
       });
     }
@@ -2030,9 +2034,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,
@@ -2048,17 +2054,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") {
@@ -2753,7 +2767,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}% |`
       );
@@ -2762,15 +2776,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}%`;
 }
 
@@ -2830,7 +2844,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}`);
     }