@sentry/warden 0.1.1 → 0.2.0

package/src/cli/terminal.test.ts

@@ -1,148 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import { writeFileSync, mkdirSync, rmSync } from 'node:fs';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import { renderTerminalReport } from './terminal.js';
-import type { SkillReport, Finding } from '../types/index.js';
-
-describe('renderTerminalReport', () => {
-  let tempDir: string;
-
-  beforeEach(() => {
-    tempDir = join(tmpdir(), `warden-terminal-test-${Date.now()}`);
-    mkdirSync(tempDir, { recursive: true });
-  });
-
-  afterEach(() => {
-    rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  function createFinding(overrides: Partial<Finding> = {}): Finding {
-    return {
-      id: 'test-1',
-      severity: 'medium',
-      title: 'Test Finding',
-      description: 'This is a test finding',
-      ...overrides,
-    };
-  }
-
-  function createReport(overrides: Partial<SkillReport> = {}): SkillReport {
-    return {
-      skill: 'test-skill',
-      summary: 'Test summary',
-      findings: [],
-      ...overrides,
-    };
-  }
-
-  describe('file unavailable indication', () => {
-    it('shows file unavailable when source file cannot be read', () => {
-      const nonExistentPath = join(tempDir, 'nonexistent.ts');
-      const report = createReport({
-        findings: [
-          createFinding({
-            location: {
-              path: nonExistentPath,
-              startLine: 5,
-            },
-          }),
-        ],
-      });
-
-      const output = renderTerminalReport([report], {
-        isTTY: true,
-        supportsColor: false, // Disable color for easier assertions
-        columns: 80,
-      });
-
-      expect(output).toContain('5 │');
-      expect(output).toContain('(file unavailable)');
-    });
-
-    it('shows code line when file exists and is readable', () => {
-      const filePath = join(tempDir, 'test.ts');
-      writeFileSync(
-        filePath,
-        'line 1\nline 2\nline 3\nline 4\nconst important = true;\nline 6'
-      );
-
-      const report = createReport({
-        findings: [
-          createFinding({
-            location: {
-              path: filePath,
-              startLine: 5,
-            },
-          }),
-        ],
-      });
-
-      const output = renderTerminalReport([report], {
-        isTTY: true,
-        supportsColor: false,
-        columns: 80,
-      });
-
-      expect(output).toContain('5 │');
-      expect(output).toContain('const important = true;');
-      expect(output).not.toContain('(file unavailable)');
-    });
-
-    it('shows nothing when line number exceeds file length', () => {
-      const filePath = join(tempDir, 'short.ts');
-      writeFileSync(filePath, 'line 1\nline 2');
-
-      const report = createReport({
-        findings: [
-          createFinding({
-            location: {
-              path: filePath,
-              startLine: 100, // Way past end of file
-            },
-          }),
-        ],
-      });
-
-      const output = renderTerminalReport([report], {
-        isTTY: true,
-        supportsColor: false,
-        columns: 80,
-      });
-
-      // Should not show file unavailable or any code line for out-of-range
-      expect(output).not.toContain('100 │');
-      expect(output).not.toContain('(file unavailable)');
-    });
-  });
-
-  describe('basic rendering', () => {
-    it('renders report with no findings', () => {
-      const report = createReport();
-
-      const output = renderTerminalReport([report], {
-        isTTY: true,
-        supportsColor: false,
-        columns: 80,
-      });
-
-      expect(output).toContain('test-skill');
-      expect(output).toContain('No issues found');
-    });
-
-    it('renders finding without location', () => {
-      const report = createReport({
-        findings: [createFinding()],
-      });
-
-      const output = renderTerminalReport([report], {
-        isTTY: true,
-        supportsColor: false,
-        columns: 80,
-      });
-
-      expect(output).toContain('Test Finding');
-      expect(output).toContain('This is a test finding');
-    });
-  });
-});

package/src/cli/terminal.ts

@@ -1,301 +0,0 @@
-import { readFileSync } from 'node:fs';
-import chalk from 'chalk';
-import type { SkillReport, Finding, Severity, SeverityThreshold } from '../types/index.js';
-import { filterFindingsBySeverity } from '../types/index.js';
-import {
-  formatSeverityBadge,
-  formatSeverityPlain,
-  formatFindingCounts,
-  formatFindingCountsPlain,
-  formatDuration,
-  formatElapsed,
-  countBySeverity,
-} from './output/index.js';
-import { BoxRenderer } from './output/box.js';
-import type { OutputMode } from './output/tty.js';
-
-const SEVERITY_COLORS: Record<Severity, typeof chalk.red> = {
-  critical: chalk.red.bold,
-  high: chalk.red,
-  medium: chalk.yellow,
-  low: chalk.green,
-  info: chalk.blue,
-};
-
-type FileLineResult =
-  | { status: 'ok'; line: string }
-  | { status: 'file_unavailable' }
-  | { status: 'line_not_found' };
-
-/**
- * Read a specific line from a file.
- * Returns a result indicating success, file unavailable, or line not found.
- */
-function readFileLine(filePath: string, lineNumber: number): FileLineResult {
-  try {
-    const content = readFileSync(filePath, 'utf-8');
-    const lines = content.split('\n');
-    const line = lines[lineNumber - 1];
-    if (lineNumber > 0 && lineNumber <= lines.length && line !== undefined) {
-      return { status: 'ok', line };
-    }
-    return { status: 'line_not_found' };
-  } catch {
-    return { status: 'file_unavailable' };
-  }
-}
-
-/**
- * Format a finding for TTY display.
- */
-function formatFindingTTY(finding: Finding): string[] {
-  const lines: string[] = [];
-  const badge = formatSeverityBadge(finding.severity);
-  const color = SEVERITY_COLORS[finding.severity];
-
-  // Title line with severity dot
-  const titleParts = [badge, color(finding.title)];
-  lines.push(titleParts.join(' '));
-
-  // Location with elapsed time
-  if (finding.location) {
-    const locParts = [chalk.dim(`${finding.location.path}:${finding.location.startLine}`)];
-    if (finding.elapsedMs !== undefined) {
-      locParts.push(chalk.dim(formatElapsed(finding.elapsedMs)));
-    }
-    lines.push(`  ${locParts.join('  ')}`);
-  }
-
-  // Code snippet
-  if (finding.location?.startLine) {
-    const result = readFileLine(finding.location.path, finding.location.startLine);
-    const lineNum = chalk.dim(`${finding.location.startLine} │`);
-    if (result.status === 'ok') {
-      lines.push(`  ${lineNum} ${result.line.trimStart()}`);
-    } else if (result.status === 'file_unavailable') {
-      lines.push(`  ${lineNum} ${chalk.dim.italic('(file unavailable)')}`);
-    }
-    // For 'line_not_found', we silently skip - the line may not exist in this version
-  }
-
-  // Blank line, then description
-  lines.push('');
-  lines.push(`  ${chalk.dim(finding.description)}`);
-
-  // Suggested fix diff if available
-  if (finding.suggestedFix?.diff) {
-    lines.push('');
-    lines.push(chalk.dim('  Suggested fix:'));
-    const diffLines = finding.suggestedFix.diff.split('\n').map((line) => {
-      if (line.startsWith('+') && !line.startsWith('+++')) {
-        return chalk.green(`  ${line}`);
-      } else if (line.startsWith('-') && !line.startsWith('---')) {
-        return chalk.red(`  ${line}`);
-      } else if (line.startsWith('@@')) {
-        return chalk.cyan(`  ${line}`);
-      }
-      return `  ${line}`;
-    });
-    lines.push(...diffLines);
-  }
-
-  return lines;
-}
-
-/**
- * Format a finding for CI (non-TTY) display.
- */
-function formatFindingCI(finding: Finding): string[] {
-  const lines: string[] = [];
-  const badge = formatSeverityPlain(finding.severity);
-
-  // Title line with location and elapsed time
-  const titleParts = [badge];
-  if (finding.location) {
-    titleParts.push(`${finding.location.path}:${finding.location.startLine}`);
-  }
-  titleParts.push('-', finding.title);
-  if (finding.elapsedMs !== undefined) {
-    titleParts.push(`(${formatElapsed(finding.elapsedMs)})`);
-  }
-  lines.push(titleParts.join(' '));
-
-  // Description
-  lines.push(`  ${finding.description}`);
-
-  return lines;
-}
-
-/**
- * Render a skill report as a box (TTY mode).
- */
-function renderSkillBoxTTY(report: SkillReport, mode: OutputMode): string[] {
-  const counts = countBySeverity(report.findings);
-  const durationStr = report.durationMs !== undefined ? formatDuration(report.durationMs) : undefined;
-
-  const box = new BoxRenderer({
-    title: report.skill,
-    badge: durationStr,
-    mode,
-  });
-
-  box.header();
-
-  // Finding counts summary line
-  const countStr = formatFindingCounts(counts);
-  box.content(countStr);
-
-  if (report.findings.length === 0) {
-    box.blank();
-    box.content(chalk.green('No issues found.'));
-  } else {
-    // Render each finding
-    for (const [index, finding] of report.findings.entries()) {
-      box.divider();
-      box.blank();
-      const findingLines = formatFindingTTY(finding);
-      box.content(findingLines);
-      // Only add blank after finding if not the last one
-      if (index < report.findings.length - 1) {
-        box.blank();
-      }
-    }
-  }
-
-  box.footer();
-
-  return box.render();
-}
-
-/**
- * Render a skill report for CI (non-TTY) mode.
- */
-function renderSkillCI(report: SkillReport): string[] {
-  const lines: string[] = [];
-  const counts = countBySeverity(report.findings);
-  const durationStr = report.durationMs !== undefined ? formatDuration(report.durationMs) : '';
-
-  // Header
-  lines.push(`=== ${report.skill} (${durationStr}) ===`);
-  lines.push(`${formatFindingCountsPlain(counts)}`);
-
-  if (report.findings.length === 0) {
-    lines.push('No issues found.');
-  } else {
-    lines.push('---');
-    for (const finding of report.findings) {
-      const findingLines = formatFindingCI(finding);
-      lines.push(...findingLines);
-      lines.push('---');
-    }
-  }
-
-  return lines;
-}
-
-/**
- * Render skill reports for terminal output.
- * @param reports - The skill reports to render
- * @param mode - Output mode (TTY vs non-TTY)
- */
-export function renderTerminalReport(reports: SkillReport[], mode?: OutputMode): string {
-  const lines: string[] = [];
-
-  // Default to TTY mode if not specified (for backwards compatibility)
-  const outputMode: OutputMode = mode ?? {
-    isTTY: true,
-    supportsColor: true,
-    columns: 80,
-  };
-
-  if (outputMode.isTTY) {
-    // TTY mode: use boxes
-    for (const report of reports) {
-      lines.push(...renderSkillBoxTTY(report, outputMode));
-      lines.push('');
-    }
-  } else {
-    // CI mode: plain text
-    for (const report of reports) {
-      lines.push(...renderSkillCI(report));
-      lines.push('');
-    }
-  }
-
-  return lines.join('\n');
-}
-
-/**
- * Aggregate usage stats from reports.
- */
-function aggregateUsage(reports: SkillReport[]) {
-  const usages = reports.map((r) => r.usage).filter((u) => u !== undefined);
-  if (usages.length === 0) return undefined;
-
-  return usages.reduce((acc, u) => ({
-    inputTokens: acc.inputTokens + u.inputTokens,
-    outputTokens: acc.outputTokens + u.outputTokens,
-    cacheReadInputTokens: (acc.cacheReadInputTokens ?? 0) + (u.cacheReadInputTokens ?? 0),
-    cacheCreationInputTokens: (acc.cacheCreationInputTokens ?? 0) + (u.cacheCreationInputTokens ?? 0),
-    costUSD: acc.costUSD + u.costUSD,
-  }));
-}
-
-/**
- * Filter reports to only include findings at or above the given severity threshold.
- * Returns new report objects with filtered findings; does not mutate the originals.
- * If commentOn is 'off', returns reports with empty findings.
- */
-export function filterReportsBySeverity(reports: SkillReport[], commentOn?: SeverityThreshold): SkillReport[] {
-  if (!commentOn) return reports;
-  return reports.map((report) => ({
-    ...report,
-    findings: filterFindingsBySeverity(report.findings, commentOn),
-  }));
-}
-
-/**
- * Render skill reports as JSON.
- */
-export function renderJsonReport(reports: SkillReport[]): string {
-  const totalUsage = aggregateUsage(reports);
-
-  const output = {
-    reports: reports.map((r) => ({
-      skill: r.skill,
-      summary: r.summary,
-      findings: r.findings,
-      metadata: r.metadata,
-      durationMs: r.durationMs,
-      usage: r.usage,
-    })),
-    summary: {
-      totalFindings: reports.reduce((sum, r) => sum + r.findings.length, 0),
-      bySeverity: {
-        critical: reports.reduce(
-          (sum, r) => sum + r.findings.filter((f) => f.severity === 'critical').length,
-          0
-        ),
-        high: reports.reduce(
-          (sum, r) => sum + r.findings.filter((f) => f.severity === 'high').length,
-          0
-        ),
-        medium: reports.reduce(
-          (sum, r) => sum + r.findings.filter((f) => f.severity === 'medium').length,
-          0
-        ),
-        low: reports.reduce(
-          (sum, r) => sum + r.findings.filter((f) => f.severity === 'low').length,
-          0
-        ),
-        info: reports.reduce(
-          (sum, r) => sum + r.findings.filter((f) => f.severity === 'info').length,
-          0
-        ),
-      },
-      usage: totalUsage,
-    },
-  };
-
-  return JSON.stringify(output, null, 2);
-}

package/src/config/index.ts

@@ -1,3 +0,0 @@
-export * from './schema.js';
-export * from './loader.js';
-export * from './writer.js';