codereview-aia 0.1.2 → 0.1.4

package/src/runtime/reportMerge.ts

@@ -1,102 +0,0 @@
-import type { ReviewTotals } from './reviewPipeline';
-
-export interface StructuredIssue {
-  priority?: string;
-  filePath?: string;
-  lineNumbers?: string;
-  type?: string;
-  description?: string;
-  suggestedFix?: string;
-  title?: string;
-  impact?: string;
-  [key: string]: unknown;
-}
-
-export interface StructuredDataReport {
-  structuredData?: {
-    issues?: StructuredIssue[];
-  };
-  totals?: Record<string, number>;
-  metadata?: Record<string, unknown>;
-  costInfo?: Record<string, unknown>;
-  modelUsed?: string;
-}
-
-interface MergeResult {
-  findings: Record<string, unknown>[];
-  totals: ReviewTotals;
-  metadata?: Record<string, unknown>;
-  costInfo?: Record<string, unknown>;
-  modelUsed?: string;
-}
-
-export function mergeReports(reports: StructuredDataReport[]): MergeResult {
-  if (reports.length === 0) {
-    return { findings: [], totals: { critical: 0, high: 0, medium: 0, low: 0 } };
-  }
-
-  if (reports.length === 1) {
-    return normalizeReport(reports[0]);
-  }
-
-  const allFindings: Record<string, unknown>[] = [];
-  const totals: ReviewTotals = { critical: 0, high: 0, medium: 0, low: 0 };
-
-  for (const report of reports) {
-    const normalized = normalizeReport(report);
-    if (normalized.findings) {
-      allFindings.push(...(normalized.findings as Record<string, unknown>[]));
-    }
-    if (normalized.totals) {
-      totals.critical += normalized.totals.critical || 0;
-      totals.high += normalized.totals.high || 0;
-      totals.medium += normalized.totals.medium || 0;
-      totals.low += normalized.totals.low || 0;
-    }
-  }
-
-  return {
-    findings: allFindings,
-    totals,
-    metadata: {
-      mergedFrom: reports.length,
-      timestamp: new Date().toISOString(),
-    },
-  };
-}
-
-function normalizeReport(report: StructuredDataReport): MergeResult {
-  const findings: Record<string, unknown>[] = [];
-  const totals: ReviewTotals = { critical: 0, high: 0, medium: 0, low: 0 };
-
-  if (report.structuredData?.issues) {
-    for (const issue of report.structuredData.issues) {
-      const priority = issue.priority?.toLowerCase() || 'medium';
-      const finding = {
-        file: issue.filePath,
-        line: parseInt(issue.lineNumbers?.split('-')[0] || '0', 10),
-        rule: issue.type || 'general',
-        severity: priority,
-        msg: issue.description,
-        fix_hint: issue.suggestedFix,
-        title: issue.title,
-        impact: issue.impact,
-        raw: issue,
-      };
-      findings.push(finding);
-
-      if (priority === 'critical') totals.critical += 1;
-      else if (priority === 'high') totals.high += 1;
-      else if (priority === 'medium') totals.medium += 1;
-      else totals.low += 1;
-    }
-  }
-
-  return {
-    findings,
-    totals,
-    metadata: report.metadata,
-    costInfo: report.costInfo,
-    modelUsed: report.modelUsed,
-  };
-}

package/src/runtime/reporting/markdownReportBuilder.ts

@@ -1,138 +0,0 @@
-import type { FileIssueGroup, TokenStats } from './reportDataCollector';
-import type { ReviewTotals } from '../reviewPipeline';
-
-export const SUMMARY_PLACEHOLDER = '<!--REPORT_SUMMARY_PLACEHOLDER-->';
-
-export interface MarkdownReportContext {
-  repoName: string;
-  generatedAt: Date;
-  durationSeconds: number;
-  filesReviewed: number;
-  totals: ReviewTotals;
-  tokenStats?: TokenStats;
-  estimatedCostUSD?: number;
-  issueGroups: FileIssueGroup[];
-}
-
-function pad(num: number): string {
-  return num.toString().padStart(2, '0');
-}
-
-function formatEuropeanTimestamp(date: Date): string {
-  const day = pad(date.getDate());
-  const month = pad(date.getMonth() + 1);
-  const year = date.getFullYear();
-  const hours = pad(date.getHours());
-  const minutes = pad(date.getMinutes());
-  return `${day}.${month}.${year} ${hours}.${minutes}`;
-}
-
-function formatDuration(seconds: number): string {
-  const hrs = Math.floor(seconds / 3600);
-  const mins = Math.floor((seconds % 3600) / 60);
-  const secs = seconds % 60;
-  return `${pad(hrs)}:${pad(mins)}:${pad(secs)}`;
-}
-
-function formatCost(amount?: number): string | undefined {
-  if (typeof amount !== 'number' || Number.isNaN(amount)) {
-    return undefined;
-  }
-  return `$${amount.toFixed(amount < 1 ? 4 : 2)} USD`;
-}
-
-function buildStatsSection(ctx: MarkdownReportContext): string {
-  const stats: string[] = ['### Stats'];
-
-  const cost = formatCost(ctx.estimatedCostUSD);
-  if (cost) {
-    stats.push(`- Spend: ${cost}`);
-  }
-
-  stats.push(`- Duration: ${formatDuration(ctx.durationSeconds)}`);
-  stats.push(`- Files reviewed: ${ctx.filesReviewed}`);
-
-  if (ctx.tokenStats) {
-    stats.push(
-      `- Total tokens: ${ctx.tokenStats.totalTokens.toLocaleString()} (input: ${ctx.tokenStats.inputTokens.toLocaleString()} / output: ${ctx.tokenStats.outputTokens.toLocaleString()})`,
-    );
-  }
-
-  stats.push(
-    `- Issues: Critical ${ctx.totals.critical} · High ${ctx.totals.high} · Medium ${ctx.totals.medium} · Low ${ctx.totals.low}`,
-  );
-
-  return stats.join('\n');
-}
-
-function formatIssueTitle(issue: FileIssueGroup['issues'][number]): string {
-  const capitalized = issue.severity.charAt(0).toUpperCase() + issue.severity.slice(1);
-  const linePart = issue.lineLabel ? ` (lines ${issue.lineLabel})` : '';
-  const title = issue.title || 'Issue';
-  return `##### [${capitalized}] ${title}${linePart}`;
-}
-
-function buildIssueBlock(group: FileIssueGroup): string {
-  const lines: string[] = [`#### ${group.filePath}`];
-
-  for (const issue of group.issues) {
-    lines.push(formatIssueTitle(issue));
-
-    if (issue.description) {
-      lines.push(`- **Problem:** ${issue.description.trim()}`);
-    }
-    if (issue.impact) {
-      lines.push(`- **Impact:** ${issue.impact.trim()}`);
-    }
-    if (issue.suggestedFix) {
-      lines.push(`- **Recommendation:** ${issue.suggestedFix.trim()}`);
-    }
-
-    lines.push('');
-  }
-
-  return lines.join('\n').trimEnd();
-}
-
-function buildIssuesSection(groups: FileIssueGroup[]): string {
-  if (groups.length === 0) {
-    return '### Issues\n_No actionable issues reported in this review._';
-  }
-
-  const blocks = groups.map((group) => buildIssueBlock(group));
-  return `### Issues\n\n${blocks.join('\n\n---\n\n')}`;
-}
-
-export function buildMarkdownReport(ctx: MarkdownReportContext): string {
-  const parts: string[] = [];
-  const timestamp = formatEuropeanTimestamp(ctx.generatedAt);
-
-  parts.push(`# Code Review Report — ${timestamp}`);
-  parts.push(`**Repository:** ${ctx.repoName}`);
-  parts.push('\n---\n');
-
-  parts.push(buildStatsSection(ctx));
-  parts.push('\n\n---\n');
-
-  parts.push('### Summary');
-  parts.push(SUMMARY_PLACEHOLDER);
-  parts.push('\n---\n');
-
-  parts.push(buildIssuesSection(ctx.issueGroups));
-  parts.push('\n\n---\n');
-  parts.push('_Generated by cr-aia_');
-
-  return parts.join('\n\n');
-}
-
-export function injectSummary(markdown: string, summary: string): string {
-  if (!markdown.includes(SUMMARY_PLACEHOLDER)) {
-    return markdown;
-  }
-
-  const formattedSummary = summary.trim() ? summary.trim() : '_Summary unavailable._';
-  return markdown.replace(
-    SUMMARY_PLACEHOLDER,
-    `>${formattedSummary.replace(/\n+/g, ' ')}`,
-  );
-}

package/src/runtime/reporting/reportDataCollector.ts

@@ -1,234 +0,0 @@
-import type { StructuredDataReport, StructuredIssue } from '../reportMerge';
-import type { ReviewTotals } from '../reviewPipeline';
-
-export type IssueSeverity = 'critical' | 'high' | 'medium' | 'low';
-
-export interface NormalizedIssue {
-  filePath: string;
-  severity: IssueSeverity;
-  title?: string;
-  description?: string;
-  impact?: string;
-  suggestedFix?: string;
-  lineLabel?: string;
-  lineStart?: number;
-  lineEnd?: number;
-}
-
-export interface FileIssueGroup {
-  filePath: string;
-  issues: NormalizedIssue[];
-}
-
-export interface TokenStats {
-  totalTokens: number;
-  inputTokens: number;
-  outputTokens: number;
-}
-
-export interface CollectedReportData {
-  totals: ReviewTotals;
-  groupedIssues: FileIssueGroup[];
-  tokenStats?: TokenStats;
-  estimatedCostUSD?: number;
-  issueCount: number;
-}
-
-const SEVERITY_ORDER: Record<IssueSeverity, number> = {
-  critical: 0,
-  high: 1,
-  medium: 2,
-  low: 3,
-};
-
-function normalizeSeverity(priority?: string): IssueSeverity {
-  const normalized = priority?.toLowerCase().trim();
-  if (normalized === 'critical') return 'critical';
-  if (normalized === 'high') return 'high';
-  if (normalized === 'low') return 'low';
-  return 'medium';
-}
-
-function parseLineLabel(lineNumbers?: string): {
-  label?: string;
-  start?: number;
-  end?: number;
-} {
-  if (!lineNumbers) {
-    return {};
-  }
-
-  const trimmed = lineNumbers.trim();
-  if (!trimmed) {
-    return {};
-  }
-
-  const match = trimmed.match(/^(\d+)(?:\s*-\s*(\d+))?$/);
-  if (!match) {
-    return { label: trimmed };
-  }
-
-  const start = Number.parseInt(match[1], 10);
-  const end = match[2] ? Number.parseInt(match[2], 10) : start;
-  const label = start === end ? `${start}` : `${start}-${end}`;
-
-  return { label, start, end };
-}
-
-function toNumber(value: unknown): number | undefined {
-  if (typeof value === 'number' && Number.isFinite(value)) {
-    return value;
-  }
-  if (typeof value === 'string') {
-    const parsed = Number.parseFloat(value);
-    if (!Number.isNaN(parsed)) {
-      return parsed;
-    }
-  }
-  return undefined;
-}
-
-function extractIssues(reports: StructuredDataReport[]): NormalizedIssue[] {
-  const issues: NormalizedIssue[] = [];
-
-  for (const report of reports) {
-    let structuredIssues: StructuredIssue[] | undefined = report.structuredData?.issues;
-
-    if (!structuredIssues && typeof report.structuredData === 'string') {
-      try {
-        const parsed = JSON.parse(report.structuredData);
-        structuredIssues = parsed?.issues;
-      } catch {
-        structuredIssues = undefined;
-      }
-    }
-
-    if (!Array.isArray(structuredIssues)) {
-      continue;
-    }
-
-    for (const issue of structuredIssues) {
-      const filePath = issue.filePath?.trim() || 'Unspecified file';
-      const severity = normalizeSeverity(issue.priority);
-      const { label, start, end } = parseLineLabel(issue.lineNumbers);
-
-      issues.push({
-        filePath,
-        severity,
-        title: issue.title,
-        description: issue.description,
-        impact: issue.impact,
-        suggestedFix: issue.suggestedFix,
-        lineLabel: label,
-        lineStart: start,
-        lineEnd: end,
-      });
-    }
-  }
-
-  return issues;
-}
-
-function groupIssuesByFile(issues: NormalizedIssue[]): FileIssueGroup[] {
-  const groups = new Map<string, NormalizedIssue[]>();
-
-  for (const issue of issues) {
-    if (!groups.has(issue.filePath)) {
-      groups.set(issue.filePath, []);
-    }
-    groups.get(issue.filePath)!.push(issue);
-  }
-
-  const sortedGroups = Array.from(groups.entries())
-    .map(([filePath, fileIssues]) => {
-      fileIssues.sort((a, b) => {
-        const severityDiff = SEVERITY_ORDER[a.severity] - SEVERITY_ORDER[b.severity];
-        if (severityDiff !== 0) {
-          return severityDiff;
-        }
-        if (a.lineStart && b.lineStart) {
-          return a.lineStart - b.lineStart;
-        }
-        return (a.title || '').localeCompare(b.title || '');
-      });
-
-      return { filePath, issues: fileIssues };
-    })
-    .sort((a, b) => a.filePath.localeCompare(b.filePath));
-
-  return sortedGroups;
-}
-
-function aggregateCostInfo(reports: StructuredDataReport[]): {
-  tokenStats?: TokenStats;
-  estimatedCostUSD?: number;
-} {
-  let inputTokens = 0;
-  let outputTokens = 0;
-  let estimatedCost = 0;
-  let hasCostInfo = false;
-
-  for (const report of reports) {
-    const costInfo = (report.costInfo || (report as any).costInfo) as Record<string, unknown> | undefined;
-    if (!costInfo) continue;
-
-    const input = toNumber(costInfo.inputTokens);
-    const output = toNumber(costInfo.outputTokens);
-    const total = toNumber(costInfo.totalTokens);
-    const estimate = toNumber(costInfo.estimatedCost ?? costInfo.cost);
-
-    if (input) {
-      inputTokens += input;
-      hasCostInfo = true;
-    }
-    if (output) {
-      outputTokens += output;
-      hasCostInfo = true;
-    }
-    if (estimate) {
-      estimatedCost += estimate;
-      hasCostInfo = true;
-    }
-
-    if (!input && !output && total && !Number.isNaN(total)) {
-      // If total tokens is provided without split, distribute evenly.
-      inputTokens += total / 2;
-      outputTokens += total / 2;
-      hasCostInfo = true;
-    }
-  }
-
-  if (!hasCostInfo) {
-    return {};
-  }
-
-  return {
-    tokenStats: {
-      inputTokens: Math.round(inputTokens),
-      outputTokens: Math.round(outputTokens),
-      totalTokens: Math.round(inputTokens + outputTokens),
-    },
-    estimatedCostUSD: estimatedCost || undefined,
-  };
-}
-
-export function collectReportData(reports: StructuredDataReport[]): CollectedReportData {
-  const issues = extractIssues(reports);
-  const groupedIssues = groupIssuesByFile(issues);
-
-  // Derive severity totals if missing (default zero).
-  const totals: ReviewTotals = { critical: 0, high: 0, medium: 0, low: 0 };
-  for (const issue of issues) {
-    totals[issue.severity] += 1;
-  }
-
-  const { tokenStats, estimatedCostUSD } = aggregateCostInfo(reports);
-
-  return {
-    totals,
-    groupedIssues,
-    tokenStats,
-    estimatedCostUSD,
-    issueCount: issues.length,
-  };
-}

package/src/runtime/reporting/summaryGenerator.ts

@@ -1,86 +0,0 @@
-import { buildOpenRouterProxyHeaders, resolveOpenRouterProxyUrl, withProxyMetadata } from '../openrouterProxy';
-import type { ReviewTotals } from '../reviewPipeline';
-import logger from '../../utils/logger';
-
-interface SummaryGeneratorOptions {
-  model?: string;
-  totals: ReviewTotals;
-  durationSeconds: number;
-  estimatedCostUSD?: number;
-  issueCount: number;
-}
-
-function buildFallbackSummary(options: SummaryGeneratorOptions): string {
-  const parts: string[] = [];
-  const { totals, durationSeconds, estimatedCostUSD, issueCount } = options;
-  const totalIssues = issueCount || totals.critical + totals.high + totals.medium + totals.low;
-  parts.push(
-    `Review completed in ${Math.round(durationSeconds)}s with ${totalIssues} tracked issues ` +
-      `(critical ${totals.critical}, high ${totals.high}, medium ${totals.medium}, low ${totals.low}).`,
-  );
-
-  if (typeof estimatedCostUSD === 'number') {
-    parts.push(`Approximate spend: $${estimatedCostUSD.toFixed(estimatedCostUSD < 1 ? 4 : 2)} USD.`);
-  }
-
-  return parts.join(' ');
-}
-
-export async function generateReportSummary(
-  markdownDraft: string,
-  options: SummaryGeneratorOptions,
-): Promise<string> {
-  const summaryModel = process.env.CR_AIA_SUMMARY_MODEL || options.model;
-  if (!summaryModel) {
-    return buildFallbackSummary(options);
-  }
-
-  const [, modelName] = summaryModel.includes(':')
-    ? summaryModel.split(':')
-    : ['openrouter', summaryModel];
-
-  const systemPrompt = `You are an engineering lead summarizing a comprehensive code review report.
-Provide 2-3 sentences capturing:
-- Overall health of the reviewed codebase
-- Highest-risk themes or repeating issues
-- Notable spend/time observations if they appear.
-Keep it factual and concise.`;
-
-  const payload = withProxyMetadata({
-    model: modelName,
-    temperature: 0.2,
-    max_tokens: 200,
-    messages: [
-      { role: 'system', content: systemPrompt },
-      { role: 'user', content: markdownDraft },
-    ],
-  });
-
-  try {
-    const response = await fetch(resolveOpenRouterProxyUrl(), {
-      method: 'POST',
-      headers: buildOpenRouterProxyHeaders(),
-      body: JSON.stringify(payload),
-    });
-
-    if (!response.ok) {
-      const errorData = await response.text();
-      logger.warn(`AI summary generation failed (${response.status}): ${errorData}`);
-      return buildFallbackSummary(options);
-    }
-
-    const data = await response.json();
-    const content = data?.choices?.[0]?.message?.content?.trim();
-    if (content) {
-      return content;
-    }
-
-    logger.warn('AI summary response missing content, falling back to deterministic summary.');
-    return buildFallbackSummary(options);
-  } catch (error) {
-    logger.warn(
-      `Unable to generate AI summary: ${error instanceof Error ? error.message : String(error)}`,
-    );
-    return buildFallbackSummary(options);
-  }
-}