faf-taf-git 2.4.1 → 2.4.2

package/src/parsers/jest.ts

@@ -1,104 +0,0 @@
-/**
- * Jest output parser
- *
- * Extracts test counts from Jest CLI output
- */
-
-export interface TestResults {
-  total: number;
-  passed: number;
-  failed: number;
-  skipped?: number;
-  result: 'PASSED' | 'FAILED' | 'IMPROVED' | 'DEGRADED';
-}
-
-/**
- * Parse Jest output to extract test results
- *
- * Jest output formats:
- * - "Tests: 1 failed, 172 passed, 173 total"
- * - "Tests: 173 passed, 173 total"
- * - "Test Suites: 2 failed, 16 passed, 18 total"
- * - "Tests:       9 skipped, 799 passed, 808 total" (with padding)
- */
-export function parseJestOutput(output: string): TestResults | null {
-  // Strip ANSI color codes and other control characters
-  // eslint-disable-next-line no-control-regex
-  let cleanOutput = output.replace(/\x1b\[[0-9;]*m/g, '');
-  // Also strip carriage returns and normalize line endings
-  cleanOutput = cleanOutput.replace(/\r\n/g, '\n').replace(/\r/g, '\n');
-
-  // Find the Jest test summary line
-  // CRITICAL: Must match "Tests:" line that contains "total" to avoid matching
-  // test descriptions like "Brake Tests: Critical dogfooding scenarios"
-  const testSummaryPattern = /Tests:\s+(.+?\d+\s+total)/im;
-  const testLineMatch = cleanOutput.match(testSummaryPattern);
-
-  if (!testLineMatch || !testLineMatch[1]) {
-    return null;
-  }
-
-  const testLine = testLineMatch[1];
-
-  // Extract counts using flexible patterns
-  let total = 0;
-  let passed = 0;
-  let failed = 0;
-  let skipped = 0;
-
-  // Parse numbers - use global search on the whole output near the Tests: line
-  const totalMatch = testLine.match(/(\d+)\s+total/i);
-  if (totalMatch) {
-    total = parseInt(totalMatch[1], 10);
-  }
-
-  const passedMatch = testLine.match(/(\d+)\s+passed/i);
-  if (passedMatch) {
-    passed = parseInt(passedMatch[1], 10);
-  }
-
-  const failedMatch = testLine.match(/(\d+)\s+failed/i);
-  if (failedMatch) {
-    failed = parseInt(failedMatch[1], 10);
-  }
-
-  const skippedMatch = testLine.match(/(\d+)\s+skipped/i);
-  if (skippedMatch) {
-    skipped = parseInt(skippedMatch[1], 10);
-  }
-
-  // Validate
-  if (total === 0) {
-    return null;
-  }
-
-  // Determine result
-  let result: 'PASSED' | 'FAILED' | 'IMPROVED' | 'DEGRADED' = 'PASSED';
-  if (failed > 0) {
-    result = 'FAILED';
-  }
-
-  return {
-    total,
-    passed,
-    failed,
-    skipped: skipped > 0 ? skipped : undefined,
-    result,
-  };
-}
-
-/**
- * Example Jest outputs for testing:
- *
- * All passing:
- * "Tests: 173 passed, 173 total"
- *
- * Some failing:
- * "Tests: 1 failed, 172 passed, 173 total"
- *
- * With skipped:
- * "Tests: 1 failed, 2 skipped, 170 passed, 173 total"
- *
- * With padding (CI):
- * "Tests:       9 skipped, 799 passed, 808 total"
- */

package/src/parsers/junit.ts

@@ -1,110 +0,0 @@
-/**
- * JUnit XML parser
- *
- * Extracts test counts from a JUnit XML report — the common format nearly every
- * test runner can write: pytest (--junitxml), cargo (nextest, cargo2junit), Go
- * (go-junit-report, gotestsum), Maven/Gradle, .NET, PHPUnit, Jest (jest-junit).
- */
-
-import { TestResults } from './jest';
-
-const SUITE_TAG = /<(\/?)testsuite\b([^>]*?)(\/?)>/g;
-
-/** Read one numeric attribute; undefined when the attribute is absent. */
-function attr(attrs: string, name: string): number | undefined {
-  const match = attrs.match(new RegExp(`\\b${name}\\s*=\\s*["'](\\d+)["']`));
-  return match ? parseInt(match[1], 10) : undefined;
-}
-
-interface Counts {
-  tests: number;
-  failed: number;
-  skipped: number;
-}
-
-/** Counts from a <testsuite> or <testsuites> attribute list, or null without a `tests` count. */
-function countsFromAttrs(attrs: string): Counts | null {
-  const tests = attr(attrs, 'tests');
-  if (tests === undefined) return null;
-  return {
-    tests,
-    failed: (attr(attrs, 'failures') ?? 0) + (attr(attrs, 'errors') ?? 0),
-    skipped: (attr(attrs, 'skipped') ?? 0) + (attr(attrs, 'disabled') ?? 0),
-  };
-}
-
-/** Fallback when suites carry no counts: walk the <testcase> elements themselves. */
-function countsFromTestcases(xml: string): Counts {
-  const counts: Counts = { tests: 0, failed: 0, skipped: 0 };
-  const testcase = /<testcase\b[^>]*?(?:\/>|>([\s\S]*?)<\/testcase>)/g;
-  let match: RegExpExecArray | null;
-  while ((match = testcase.exec(xml)) !== null) {
-    counts.tests++;
-    const body = match[1] ?? '';
-    if (/<(failure|error)\b/.test(body)) counts.failed++;
-    else if (/<skipped\b/.test(body)) counts.skipped++;
-  }
-  return counts;
-}
-
-/**
- * Parse a JUnit XML report to extract test results
- *
- * Handles:
- * - a <testsuites> root that carries the totals (gotestsum, Maven)
- * - a <testsuites> root without totals, around counted <testsuite>s (pytest)
- * - a single <testsuite> root
- * - nested <testsuite>s (only the outermost level is counted, never twice)
- * - suites with no counts at all (the <testcase> elements are counted instead)
- *
- * failures + errors count as failed; skipped + disabled count as skipped.
- */
-export function parseJunitOutput(output: string): TestResults | null {
-  const xml = output.replace(/<!--[\s\S]*?-->/g, '');
-  if (!/<testsuites?\b/.test(xml)) return null;
-
-  let counts: Counts | null = null;
-
-  const root = xml.match(/<testsuites\b([^>]*)>/);
-  if (root) counts = countsFromAttrs(root[1]);
-
-  if (!counts) {
-    // Sum the outermost <testsuite>s; a suite without counts sends us to the testcases.
-    let depth = 0;
-    let summed: Counts | null = { tests: 0, failed: 0, skipped: 0 };
-    let seen = 0;
-    let match: RegExpExecArray | null;
-    SUITE_TAG.lastIndex = 0;
-    while ((match = SUITE_TAG.exec(xml)) !== null) {
-      const [, closing, attrs, selfClosing] = match;
-      if (closing) {
-        depth--;
-        continue;
-      }
-      if (depth === 0 && summed) {
-        seen++;
-        const suite = countsFromAttrs(attrs);
-        if (suite) {
-          summed.tests += suite.tests;
-          summed.failed += suite.failed;
-          summed.skipped += suite.skipped;
-        } else {
-          summed = null;
-        }
-      }
-      if (!selfClosing) depth++;
-    }
-    counts = seen > 0 && summed ? summed : countsFromTestcases(xml);
-  }
-
-  if (counts.tests === 0) return null;
-
-  const passed = Math.max(0, counts.tests - counts.failed - counts.skipped);
-  return {
-    total: counts.tests,
-    passed,
-    failed: counts.failed,
-    skipped: counts.skipped > 0 ? counts.skipped : undefined,
-    result: counts.failed > 0 ? 'FAILED' : 'PASSED',
-  };
-}

package/src/parsers/vitest.ts

@@ -1,65 +0,0 @@
-/**
- * Vitest output parser
- *
- * Extracts test counts from Vitest CLI output
- */
-
-import { TestResults } from './jest';
-
-/**
- * Parse Vitest output to extract test results
- *
- * Vitest output formats:
- * - " Tests  8 passed (8)"
- * - " Tests  2 failed | 6 passed (8)"
- * - " Tests  1 skipped | 7 passed (8)"
- * - " Tests  1 failed | 2 skipped | 5 passed (8)"
- */
-export function parseVitestOutput(output: string): TestResults | null {
-  // Strip ANSI color codes
-  // eslint-disable-next-line no-control-regex
-  let cleanOutput = output.replace(/\x1b\[[0-9;]*m/g, '');
-  cleanOutput = cleanOutput.replace(/\r\n/g, '\n').replace(/\r/g, '\n');
-
-  // Vitest summary pattern: " Tests  ... (N)" where N is total in parens
-  const testSummaryPattern = /^\s*Tests\s+(.+?\(\d+\))/m;
-  const testLineMatch = cleanOutput.match(testSummaryPattern);
-
-  if (!testLineMatch || !testLineMatch[1]) {
-    return null;
-  }
-
-  const testLine = testLineMatch[1];
-
-  // Extract total from parentheses at end
-  const totalMatch = testLine.match(/\((\d+)\)\s*$/);
-  if (!totalMatch) return null;
-  const total = parseInt(totalMatch[1], 10);
-  if (total === 0) return null;
-
-  let passed = 0;
-  let failed = 0;
-  let skipped = 0;
-
-  const passedMatch = testLine.match(/(\d+)\s+passed/i);
-  if (passedMatch) passed = parseInt(passedMatch[1], 10);
-
-  const failedMatch = testLine.match(/(\d+)\s+failed/i);
-  if (failedMatch) failed = parseInt(failedMatch[1], 10);
-
-  const skippedMatch = testLine.match(/(\d+)\s+skipped/i);
-  if (skippedMatch) skipped = parseInt(skippedMatch[1], 10);
-
-  const todoMatch = testLine.match(/(\d+)\s+todo/i);
-  if (todoMatch) skipped += parseInt(todoMatch[1], 10);
-
-  let result: 'PASSED' | 'FAILED' = failed > 0 ? 'FAILED' : 'PASSED';
-
-  return {
-    total,
-    passed,
-    failed,
-    skipped: skipped > 0 ? skipped : undefined,
-    result,
-  };
-}

package/src/parsers/wjttc.ts

@@ -1,160 +0,0 @@
-/**
- * WJTTC output parser
- *
- * Extracts test counts from the FAF / pc-ai WJTTC bar runner
- * (tests/wjttc/run-wjttc.mjs and sibling suite runners).
- *
- * Two shapes:
- *
- * 1) Human log (default):
- * ```
- * pc-ai WJTTC
- * ----------
- * PASS  VAL-01  valid-simple portfolio accepts
- * FAIL  OPT-01  …
- * ----------
- * 31/33 passed
- * ```
- *
- * 2) Machine JSON (`--json`):
- * ```
- * {
- *   "suite": "pc-ai-wjttc",
- *   "total": 33,
- *   "passed": 31,
- *   "failed": 2,
- *   "results": [ … ]
- * }
- * ```
- *
- * Anchors are WJTTC-specific so this never false-matches Jest/Vitest/Bun.
- * Prefer JSON when both appear in a teed stream (structured totals win).
- */
-
-import { TestResults } from './jest';
-
-function stripAnsi(output: string): string {
-  // eslint-disable-next-line no-control-regex
-  return output
-    .replace(/\x1b\[[0-9;]*m/g, '')
-    .replace(/\r\n/g, '\n')
-    .replace(/\r/g, '\n');
-}
-
-/**
- * Try to parse a WJTTC --json summary object from output.
- * Handles pure JSON or JSON embedded after build noise in a tee file.
- */
-function parseJsonSuite(clean: string): TestResults | null {
-  // Prefer a top-level object that names the suite.
-  const candidates: string[] = [];
-
-  const trimmed = clean.trim();
-  if (trimmed.startsWith('{')) {
-    candidates.push(trimmed);
-  }
-
-  // Last {...} block often is the suite when logs precede JSON.
-  const lastBrace = clean.lastIndexOf('{');
-  if (lastBrace !== -1) {
-    candidates.push(clean.slice(lastBrace));
-  }
-
-  for (const raw of candidates) {
-    try {
-      const body = JSON.parse(raw);
-      if (!body || typeof body !== 'object') continue;
-      if (body.suite !== 'pc-ai-wjttc') continue;
-
-      const total = Number(body.total);
-      const passed = Number(body.passed);
-      const failed = Number(body.failed);
-      if (!Number.isFinite(total) || total <= 0) continue;
-      if (!Number.isFinite(passed) || !Number.isFinite(failed)) continue;
-
-      // Prefer explicit fields; fall back to results[] length if needed.
-      let p = passed;
-      let f = failed;
-      let t = total;
-      if (Array.isArray(body.results) && body.results.length > 0) {
-        const fromRows = body.results.length;
-        if (t !== fromRows && fromRows > 0) {
-          // Trust results array as the executed set (partition honesty).
-          t = fromRows;
-          p = body.results.filter((r: { ok?: boolean }) => r && r.ok === true).length;
-          f = t - p;
-        }
-      }
-
-      return {
-        total: t,
-        passed: p,
-        failed: f,
-        result: f > 0 ? 'FAILED' : 'PASSED',
-      };
-    } catch {
-      // try next candidate
-    }
-  }
-
-  return null;
-}
-
-/**
- * Human WJTTC log: header + optional PASS/FAIL lines + "N/N passed".
- */
-function parseHumanLog(clean: string): TestResults | null {
-  // Require the suite banner so we do not steal "12/12 passed" from other tools.
-  if (!/pc-ai\s+WJTTC/i.test(clean)) {
-    return null;
-  }
-
-  const summary = clean.match(/(\d+)\s*\/\s*(\d+)\s+passed\b/i);
-  if (!summary) return null;
-
-  const passedFromSummary = parseInt(summary[1], 10);
-  const totalFromSummary = parseInt(summary[2], 10);
-  if (totalFromSummary <= 0) return null;
-
-  // Cross-check with PASS/FAIL lines when present (stronger than summary alone).
-  const passLines = clean.match(/^PASS\s+\S+/gm);
-  const failLines = clean.match(/^FAIL\s+\S+/gm);
-  const linePassed = passLines ? passLines.length : -1;
-  const lineFailed = failLines ? failLines.length : -1;
-
-  let total = totalFromSummary;
-  let passed = passedFromSummary;
-  let failed = total - passed;
-
-  if (linePassed >= 0 && lineFailed >= 0) {
-    const lineTotal = linePassed + lineFailed;
-    if (lineTotal === totalFromSummary) {
-      passed = linePassed;
-      failed = lineFailed;
-      total = lineTotal;
-    }
-    // If lines disagree with summary, still use summary (runner is source of truth)
-    // but failed must be non-negative.
-    if (failed < 0) failed = 0;
-  }
-
-  return {
-    total,
-    passed,
-    failed,
-    result: failed > 0 ? 'FAILED' : 'PASSED',
-  };
-}
-
-/**
- * Parse WJTTC (pc-ai bar) output to TestResults, or null if not WJTTC.
- */
-export function parseWjttcOutput(output: string): TestResults | null {
-  const clean = stripAnsi(output);
-
-  // JSON first — structured totals when --json was used.
-  const fromJson = parseJsonSuite(clean);
-  if (fromJson) return fromJson;
-
-  return parseHumanLog(clean);
-}

package/src/taf-core.ts

@@ -1,139 +0,0 @@
-/**
- * TAF Core - Simplified for GitHub Action
- *
- * Pure functions for reading, parsing, and updating .taf files
- * Derived from faf-cli TAF implementation (MCP-portable)
- */
-
-import * as fs from 'fs';
-import * as yaml from 'yaml';
-import { TestResults } from './parsers/jest';
-
-interface TAFFile {
-  format_version?: string;
-  project?: string;
-  created?: string;
-  last_updated?: string;
-  test_history: TestRun[];
-  [key: string]: any; // Allow other fields
-}
-
-interface TestRun {
-  timestamp: string;
-  result: 'PASSED' | 'FAILED' | 'IMPROVED' | 'DEGRADED';
-  tests: {
-    total: number;
-    passed: number;
-    failed: number;
-    skipped?: number;
-  };
-  command?: string;
-  trigger?: string;
-}
-
-/**
- * Update .taf file with new test results
- * Creates the file if it doesn't exist
- */
-export function updateTafFile(
-  tafPath: string,
-  testResults: TestResults
-): boolean {
-  try {
-    let taf: TAFFile;
-
-    // Check if .taf exists, create if not
-    if (fs.existsSync(tafPath)) {
-      const content = fs.readFileSync(tafPath, 'utf-8');
-      taf = parseTAF(content);
-    } else {
-      // Create new .taf file
-      const projectName = require('path').basename(require('path').dirname(tafPath));
-      taf = createTAF(projectName);
-    }
-
-    // Create test run entry
-    const run: TestRun = {
-      timestamp: new Date().toISOString(),
-      result: testResults.result,
-      tests: {
-        total: testResults.total,
-        passed: testResults.passed,
-        failed: testResults.failed,
-        skipped: testResults.skipped,
-      },
-      trigger: 'github-actions',
-    };
-
-    // Append to history
-    taf.test_history.push(run);
-
-    // Serialize and write back
-    const updated = serializeTAF(taf);
-    fs.writeFileSync(tafPath, updated, 'utf-8');
-
-    return true;
-  } catch (error) {
-    console.error('Failed to update .taf file:', error);
-    return false;
-  }
-}
-
-/**
- * Create a new TAF file structure
- */
-function createTAF(projectName: string): TAFFile {
-  return {
-    format_version: '1.0.0',
-    project: projectName,
-    created: new Date().toISOString(),
-    last_updated: new Date().toISOString(),
-    test_history: [],
-  };
-}
-
-/**
- * Parse .taf file content
- */
-function parseTAF(content: string): TAFFile {
-  // Split on document separator and take only first document
-  const lines = content.split('\n');
-  const yamlLines: string[] = [];
-
-  for (const line of lines) {
-    // Stop at document separator (but skip initial one if present)
-    if (line.trim() === '---' && yamlLines.length > 0) {
-      break;
-    }
-    // Skip initial document separator
-    if (line.trim() === '---' && yamlLines.length === 0) {
-      continue;
-    }
-    yamlLines.push(line);
-  }
-
-  const parsed = yaml.parse(yamlLines.join('\n'));
-
-  // Basic validation
-  if (!parsed || typeof parsed !== 'object') {
-    throw new Error('Invalid .taf file: not a valid YAML object');
-  }
-
-  if (!parsed.test_history || !Array.isArray(parsed.test_history)) {
-    throw new Error('Invalid .taf file: missing or invalid test_history');
-  }
-
-  return parsed as TAFFile;
-}
-
-/**
- * Serialize TAF object back to YAML
- * Uses yaml library to maintain existing format
- */
-function serializeTAF(taf: TAFFile): string {
-  // Update last_updated timestamp
-  taf.last_updated = new Date().toISOString();
-
-  // Use yaml.stringify to preserve format
-  return yaml.stringify(taf);
-}

package/test-mcp.ts

@@ -1,83 +0,0 @@
-/**
- * Test action with claude-faf-mcp repository
- */
-
-import * as fs from 'fs';
-import * as path from 'path';
-import { parseJestOutput } from './src/parsers/jest';
-import { updateTafFile } from './src/taf-core';
-
-async function testMCP() {
-  console.log('Testing action with claude-faf-mcp repo...\n');
-
-  // Simulate Jest output (based on typical MCP test output)
-  const jestOutput = `
-PASS tests/tools.test.ts
-PASS tests/validation.test.ts
-
-Test Suites: 2 passed, 2 total
-Tests:       15 passed, 15 total
-Snapshots:   0 total
-Time:        1.234 s
-`;
-
-  console.log('1. Parsing Jest output...');
-  const testResults = parseJestOutput(jestOutput);
-
-  if (!testResults) {
-    console.error('❌ Failed to parse Jest output');
-    process.exit(1);
-  }
-
-  console.log('✅ Parsed successfully:');
-  console.log(`   - Total: ${testResults.total}`);
-  console.log(`   - Passed: ${testResults.passed}`);
-  console.log(`   - Failed: ${testResults.failed}`);
-  console.log(`   - Result: ${testResults.result}\n`);
-
-  // Test with MCP .taf file
-  const tafPath = path.join('/Users/wolfejam/FAF/claude-faf-mcp', '.taf');
-
-  console.log('2. Checking .taf file...');
-  if (!fs.existsSync(tafPath)) {
-    console.error('❌ No .taf file found at:', tafPath);
-    process.exit(1);
-  }
-  console.log('✅ Found .taf file\n');
-
-  console.log('3. Reading current .taf file...');
-  const beforeContent = fs.readFileSync(tafPath, 'utf-8');
-  const beforeLines = beforeContent.split('\n').filter(line => line.includes('timestamp')).length;
-  console.log(`✅ Current test runs: ${beforeLines}\n`);
-
-  console.log('4. Updating .taf file...');
-  const updated = await updateTafFile(tafPath, testResults);
-
-  if (!updated) {
-    console.error('❌ Failed to update .taf file');
-    process.exit(1);
-  }
-  console.log('✅ .taf file updated\n');
-
-  console.log('5. Verifying update...');
-  const afterContent = fs.readFileSync(tafPath, 'utf-8');
-  const afterLines = afterContent.split('\n').filter(line => line.includes('timestamp')).length;
-  console.log(`✅ Test runs after update: ${afterLines}`);
-
-  if (afterLines !== beforeLines + 1) {
-    console.error('❌ Expected one additional run');
-    process.exit(1);
-  }
-
-  console.log('\n✅ MCP test PASSED');
-  console.log('\nThe action successfully:');
-  console.log('  - Parsed Jest output (15/15 tests)');
-  console.log('  - Updated .taf file');
-  console.log('  - Added new test run entry');
-  console.log('  - Maintained proper format');
-}
-
-testMCP().catch(error => {
-  console.error('❌ Test failed:', error);
-  process.exit(1);
-});