@side-quest/bun-runner 1.0.2 → 1.0.3

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @side-quest/bun-runner
 
+## 1.0.3
+
+### Patch Changes
+
+- [#12](https://github.com/nathanvale/side-quest-runners/pull/12) [`aa20a3f`](https://github.com/nathanvale/side-quest-runners/commit/aa20a3f190a86e978f9bd7039e525a1321b6e4dd) Thanks [@nathanvale](https://github.com/nathanvale)! - Fix console.error false positive in bun_runTests parser. Tests emitting console.error output as part of expected behavior no longer create spurious failures when the summary line shows 0 fail.
+
 ## 1.0.2
 
 ### Patch Changes

package/dist/index.d.ts

@@ -16,7 +16,17 @@ interface TestSummary {
 	failures: TestFailure[];
 }
 /**
-* Parse bun test output to extract test results
+* Parse bun test output to extract test results.
+*
+* The summary line (e.g., "3 pass\n0 fail") is the source of truth.
+* Console.error output from tests can create false positives if we
+* start failure blocks from orphan "error:" lines without a (fail) marker.
+*
+* Strategy:
+* 1. Extract pass/fail counts from summary FIRST
+* 2. If summary shows 0 failures, trust it and skip failure parsing
+* 3. Only parse failure details when summary indicates failures > 0
+* 4. In v1.3+ format, only create failures that have a (fail) marker
 */
 declare function parseBunTestOutput(output: string): TestSummary;
 export { parseBunTestOutput, TestSummary, TestFailure };

package/dist/index.js

@@ -20,66 +20,15 @@ import {
 
 // mcp/parse-utils.ts
 function parseBunTestOutput(output) {
-  const failures = [];
-  const lines = output.split(`
-`);
-  let currentFailure = null;
-  let currentTestName;
-  for (const line of lines) {
-    if (!line)
-      continue;
-    const failMatch = line.match(/\(fail\)\s+(.+?)\s+\[/);
-    if (failMatch) {
-      if (currentFailure) {
-        currentTestName = failMatch[1];
-        currentFailure.message = `${currentTestName}: ${currentFailure.message}`;
-        failures.push(currentFailure);
-        currentFailure = null;
-      }
-      continue;
-    }
-    if (line.includes("\u2717") || line.startsWith("FAIL ")) {
-      if (currentFailure)
-        failures.push(currentFailure);
-      currentFailure = {
-        file: "unknown",
-        message: line.trim()
-      };
-      continue;
-    }
-    if (line.trim().startsWith("error:")) {
-      if (currentFailure) {
-        currentFailure.message += `
-${line.trim()}`;
-      } else {
-        currentFailure = {
-          file: "unknown",
-          message: line.trim()
-        };
-      }
-      continue;
-    }
-    if (currentFailure) {
-      if (line.trim().startsWith("at ")) {
-        const match = line.match(/\((.+):(\d+):(\d+)\)/) || line.match(/at (.+):(\d+):(\d+)/);
-        if (match?.[1] && match[2]) {
-          currentFailure.file = match[1];
-          currentFailure.line = Number.parseInt(match[2], 10);
-        }
-        currentFailure.stack = `${currentFailure.stack || ""}${line}
-`;
-      } else if (line.trim() && !line.match(/^\d+ \| /)) {
-        currentFailure.message += `
-${line.trim()}`;
-      }
-    }
-  }
-  if (currentFailure)
-    failures.push(currentFailure);
   const passMatch = output.match(/(\d+) pass/);
-  const failMatchNum = output.match(/(\d+) fail/);
+  const failMatch = output.match(/(\d+) fail/);
   const passed = passMatch?.[1] ? Number.parseInt(passMatch[1], 10) : 0;
-  const failed = failMatchNum?.[1] ? Number.parseInt(failMatchNum[1], 10) : failures.length;
+  const failedFromSummary = failMatch?.[1] ? Number.parseInt(failMatch[1], 10) : null;
+  if (failedFromSummary === 0) {
+    return { passed, failed: 0, total: passed, failures: [] };
+  }
+  const failures = parseFailureDetails(output);
+  const failed = failedFromSummary ?? failures.length;
   return {
     passed,
     failed,
@@ -87,15 +36,7 @@ ${line.trim()}`;
     failures
   };
 }
-
-// mcp/index.ts
-var { initLogger, getSubsystemLogger } = createPluginLogger({
-  name: "bun-runner",
-  subsystems: ["mcp"]
-});
-initLogger().catch(console.error);
-var mcpLogger = getSubsystemLogger("mcp");
-function parseBunTestOutputImpl(output) {
+function parseFailureDetails(output) {
   const failures = [];
   const lines = output.split(`
 `);
@@ -150,19 +91,19 @@ ${line.trim()}`;
       }
     }
   }
-  if (currentFailure)
+  if (currentFailure && (currentFailure.message.includes("\u2717") || currentFailure.message.startsWith("FAIL "))) {
     failures.push(currentFailure);
-  const passMatch = output.match(/(\d+) pass/);
-  const failMatchNum = output.match(/(\d+) fail/);
-  const passed = passMatch?.[1] ? Number.parseInt(passMatch[1], 10) : 0;
-  const failed = failMatchNum?.[1] ? Number.parseInt(failMatchNum[1], 10) : failures.length;
-  return {
-    passed,
-    failed,
-    total: passed + failed,
-    failures
-  };
+  }
+  return failures;
 }
+
+// mcp/index.ts
+var { initLogger, getSubsystemLogger } = createPluginLogger({
+  name: "bun-runner",
+  subsystems: ["mcp"]
+});
+initLogger().catch(console.error);
+var mcpLogger = getSubsystemLogger("mcp");
 async function runBunTests(pattern) {
   const cmd = pattern ? ["bun", "test", pattern] : ["bun", "test"];
   const TIMEOUT_MS = 30000;
@@ -192,7 +133,7 @@ ${stderr}`;
       failures: []
     };
   }
-  return parseBunTestOutputImpl(output);
+  return parseBunTestOutput(output);
 }
 async function runBunTestCoverage() {
   const TIMEOUT_MS = 60000;
@@ -216,7 +157,7 @@ ${stderr}`;
       coverage: { percent: 0, uncovered: [] }
     };
   }
-  const summary = exitCode === 0 ? parseBunTestOutputImpl(stdout) : parseBunTestOutputImpl(output);
+  const summary = exitCode === 0 ? parseBunTestOutput(stdout) : parseBunTestOutput(output);
   const coverageMatch = output.match(/(\d+(?:\.\d+)?)\s*%/);
   const percent = coverageMatch?.[1] ? Number.parseFloat(coverageMatch[1]) : 0;
   const uncovered = [];

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@side-quest/bun-runner",
-	"version": "1.0.2",
+	"version": "1.0.3",
 	"description": "Bun test runner MCP server — structured, token-efficient test output for Claude Code",
 	"author": {
 		"name": "Nathan Vale",