fix(test-summary): parse node:test (node --test) TAP output (#52)

detectRunner did not recognise `node --test`, so its output fell through to
the generic parser. node:test emits a TAP footer ("# pass N" / "# fail N",
numbers after the word) and "ok N - name" points (ok before the number), which
the generic "<N> passed" regex never matches — so a green 2/2 run reported 0
passed.

Add a 'node' runner: detect it by command (`node --test` / `node:test`) or by
the TAP footer, and parse pass/fail/skipped/tests from the footer with a
fallback to counting ok/not ok lines. Failure names come from "not ok N - name".
Purely additive — no existing parser is touched.

Co-authored-by: shahinyanm <mher.shahinyan@12go.asia>

Author: Shahinyanm

src/handlers/test-summary.ts

@@ -101,6 +101,7 @@ export function detectRunner(command: string, output: string): string {
   if (cmd.includes('go test')) return 'go';
   if (cmd.includes('rspec')) return 'rspec';
   if (cmd.includes('mocha')) return 'mocha';
+  if (cmd.includes('node --test') || cmd.includes('node:test')) return 'node';
 
   // Detect from output
   const lower = output.toLowerCase();
@@ -109,6 +110,8 @@ export function detectRunner(command: string, output: string): string {
   if (lower.includes('pytest') || (lower.includes('=== ') && lower.includes(' passed'))) return 'pytest';
   if (lower.includes('phpunit')) return 'phpunit';
   if (lower.includes('--- fail:') || lower.includes('--- pass:') || lower.includes('ok  \t')) return 'go';
+  // node:test prints a TAP summary footer: "# tests N" + "# pass N" + "# fail N".
+  if (/^#\s*tests\s+\d+/m.test(output) && /^#\s*pass\s+\d+/m.test(output)) return 'node';
 
   return 'generic';
 }
@@ -130,6 +133,8 @@ export function parseTestOutput(output: string, runner: string): TestResult {
       return parseGoTest(output);
     case 'cargo':
       return parseCargoTest(output);
+    case 'node':
+      return parseNodeTest(output);
     default:
       return parseGeneric(output);
   }
@@ -293,6 +298,45 @@ function parseGoTest(output: string): TestResult {
   return result;
 }
 
+function parseNodeTest(output: string): TestResult {
+  const result: TestResult = { total: 0, passed: 0, failed: 0, skipped: 0, failures: [] };
+
+  // node:test (`node --test`) prints a TAP summary footer:
+  //   # tests 2
+  //   # pass 2
+  //   # fail 0
+  //   # skipped 0
+  const num = (re: RegExp): number | null => {
+    const m = output.match(re);
+    return m ? parseInt(m[1], 10) : null;
+  };
+  const pass = num(/^#\s*pass\s+(\d+)/m);
+  const fail = num(/^#\s*fail\s+(\d+)/m);
+  const skip = num(/^#\s*skipped\s+(\d+)/m);
+  const tests = num(/^#\s*tests\s+(\d+)/m);
+
+  if (pass !== null || fail !== null) {
+    result.passed = pass ?? 0;
+    result.failed = fail ?? 0;
+    result.skipped = skip ?? 0;
+    result.total = tests ?? result.passed + result.failed + result.skipped;
+  } else {
+    // No footer (truncated output) — count the TAP point lines instead.
+    result.passed = (output.match(/^ok\s+\d+/gm) ?? []).length;
+    result.failed = (output.match(/^not ok\s+\d+/gm) ?? []).length;
+    result.total = result.passed + result.failed + result.skipped;
+  }
+
+  // Failure names come from the TAP point: "not ok 3 - the test name".
+  const failPattern = /^not ok\s+\d+\s*-\s*(.+)$/gm;
+  let match: RegExpExecArray | null;
+  while ((match = failPattern.exec(output)) !== null) {
+    result.failures.push({ name: match[1].trim(), error: '' });
+  }
+
+  return result;
+}
+
 function parseCargoTest(output: string): TestResult {
   const result: TestResult = { total: 0, passed: 0, failed: 0, skipped: 0, failures: [] };
 

tests/handlers/test-summary.test.ts

@@ -182,3 +182,52 @@ describe('handleTestSummary', () => {
     expect(text).toContain('boom');
   });
 });
+
+describe('parseTestOutput — node:test (TAP)', () => {
+  // Real `node --test` output footer.
+  const passing = [
+    'ok 1 - a',
+    'ok 2 - b',
+    '1..2',
+    '# tests 2',
+    '# suites 0',
+    '# pass 2',
+    '# fail 0',
+    '# cancelled 0',
+    '# skipped 0',
+    '# todo 0',
+    '# duration_ms 41.305043',
+  ].join('\n');
+
+  it('detects node:test from command', () => {
+    expect(detectRunner('node --test', '')).toBe('node');
+    expect(detectRunner('node --test test/*.mjs', '')).toBe('node');
+  });
+
+  it('detects node:test from the TAP footer when the command is generic', () => {
+    expect(detectRunner('npm test', passing)).toBe('node');
+  });
+
+  it('parses all-passing node:test output (regression: was 0/2)', () => {
+    const r = parseTestOutput(passing, 'node');
+    expect(r.passed).toBe(2);
+    expect(r.failed).toBe(0);
+    expect(r.total).toBe(2);
+  });
+
+  it('parses failures with their names', () => {
+    const failing = [
+      'ok 1 - a',
+      'not ok 2 - b',
+      '1..2',
+      '# tests 2',
+      '# pass 1',
+      '# fail 1',
+      '# skipped 0',
+    ].join('\n');
+    const r = parseTestOutput(failing, 'node');
+    expect(r.passed).toBe(1);
+    expect(r.failed).toBe(1);
+    expect(r.failures.map((f) => f.name)).toContain('b');
+  });
+});