codereview-aia 0.1.3 → 0.1.4

package/src/runtime/preprod/webCheck.ts

@@ -1,104 +0,0 @@
-const DEFAULT_AIA_BASE_URL = 'https://ai.enki.si';
-
-function resolveAiaBaseUrl(): string {
-  const configured = process.env.AIA_BASE_URL || process.env.CR_AIA_AIA_BASE_URL;
-  const base = (configured || DEFAULT_AIA_BASE_URL).trim();
-  return base.replace(/\/$/, '');
-}
-
-function summarizeErrorBody(body: string): string {
-  if (!body.trim()) {
-    return 'No response body';
-  }
-
-  if (/<html/i.test(body)) {
-    const sanitized = body
-      .replace(/<script[\s\S]*?<\/script>/gi, ' ')
-      .replace(/<style[\s\S]*?<\/style>/gi, ' ')
-      .replace(/<[^>]+>/g, ' ')
-      .replace(/\s+/g, ' ')
-      .trim();
-    return sanitized ? `${sanitized.slice(0, 200)}${sanitized.length > 200 ? '…' : ''}` : 'HTML error response';
-  }
-
-  const normalized = body.replace(/\s+/g, ' ').trim();
-  if (!normalized) {
-    return 'Empty error body';
-  }
-  return normalized.length > 200 ? `${normalized.slice(0, 200)}…` : normalized;
-}
-
-class WebCheckRequestError extends Error {
-  status?: number;
-  detail?: string;
-
-  constructor(message: string, options?: { status?: number; detail?: string; cause?: unknown }) {
-    super(message, options?.cause ? { cause: options.cause } : undefined);
-    this.name = 'WebCheckRequestError';
-    this.status = options?.status;
-    this.detail = options?.detail;
-  }
-}
-
-async function extractErrorDetail(response: Response): Promise<string | undefined> {
-  const contentType = response.headers.get('content-type') || '';
-  if (contentType.includes('application/json')) {
-    try {
-      const payload = (await response.json()) as { error?: { message?: string }; message?: string };
-      return payload?.error?.message || payload?.message || JSON.stringify(payload);
-    } catch {
-      return undefined;
-    }
-  }
-
-  try {
-    const textBody = await response.text();
-    return summarizeErrorBody(textBody);
-  } catch {
-    return undefined;
-  }
-}
-
-async function sendWebCheckRequest(endpoint: string, payload: Record<string, unknown>): Promise<void> {
-  let response: Response;
-  try {
-    response = await fetch(endpoint, {
-      method: 'POST',
-      headers: {
-        'Content-Type': 'application/json',
-      },
-      body: JSON.stringify(payload),
-    });
-  } catch (networkError) {
-    const message =
-      networkError instanceof Error ? networkError.message : 'Website check request failed due to an unknown error';
-    throw new WebCheckRequestError(`Website check request failed: ${message}`, { cause: networkError });
-  }
-
-  if (!response.ok) {
-    const detail = await extractErrorDetail(response);
-    const suffix = detail ? ` ${detail}` : '';
-    throw new WebCheckRequestError(`Website check failed (${response.status}).${suffix}`, {
-      status: response.status,
-      detail,
-    });
-  }
-}
-
-export async function triggerManualWebCheck(targetUrl: string): Promise<void> {
-  const baseUrl = resolveAiaBaseUrl();
-  const manualEndpoint = `${baseUrl}/api/manual/check`;
-  const legacyEndpoint = `${baseUrl}/api/check-website`;
-  const missingEndpointStatuses = new Set([404, 405, 501]);
-
-  try {
-    await sendWebCheckRequest(manualEndpoint, { url: targetUrl });
-    return;
-  } catch (error) {
-    if (!(error instanceof WebCheckRequestError) || !error.status || !missingEndpointStatuses.has(error.status)) {
-      throw error;
-    }
-  }
-
-  await sendWebCheckRequest(legacyEndpoint, { url: targetUrl, options: {} });
-}

package/src/runtime/proxyConfig.ts

@@ -1,94 +0,0 @@
-import { existsSync, readFileSync } from 'node:fs';
-import { join } from 'node:path';
-import yaml from 'js-yaml';
-
-export interface ProxySettings {
-  baseUrl: string;
-  httpReferer?: string;
-  xTitle?: string;
-}
-
-const CR_AIA_CONFIG_CANDIDATES = ['.cr-aia.yml'];
-const DEFAULT_PROXY_BASE_URL = 'https://cr.ai.enki.si';
-
-export function loadProxySettings(repoRoot: string): ProxySettings {
-  const config = loadCrAiaConfig(repoRoot);
-  const scopedConfig = config && typeof config === 'object' ? getScopedConfig(config) : undefined;
-
-  const envBase = sanitize(process.env.PROXY_BASE_URL);
-  const fileBase = getFirstDefined<string>(config, scopedConfig, ['proxy_base_url', 'proxyBaseUrl']);
-
-  const httpReferer =
-    sanitize(process.env.PROXY_HTTP_REFERER) ||
-    getFirstDefined<string>(config, scopedConfig, ['http_referer', 'httpReferer']);
-
-  const xTitle =
-    sanitize(process.env.PROXY_X_TITLE) ||
-    getFirstDefined<string>(config, scopedConfig, ['x_title', 'xTitle']);
-
-  const baseUrl = envBase || fileBase || DEFAULT_PROXY_BASE_URL;
-
-  const settings: ProxySettings = {
-    baseUrl,
-  };
-
-  if (httpReferer) settings.httpReferer = httpReferer;
-  if (xTitle) settings.xTitle = xTitle;
-  return settings;
-}
-
-function sanitize(value: string | undefined): string | undefined {
-  return value && value.trim() ? value.trim() : undefined;
-}
-
-function loadCrAiaConfig(repoRoot: string): any | undefined {
-  for (const candidate of CR_AIA_CONFIG_CANDIDATES) {
-    const filePath = join(repoRoot, candidate);
-    if (!existsSync(filePath)) continue;
-
-    try {
-      const raw = readFileSync(filePath, 'utf-8');
-      if (candidate.endsWith('.json')) {
-        return JSON.parse(raw);
-      }
-      return yaml.load(raw);
-    } catch {
-      return undefined;
-    }
-  }
-  return undefined;
-}
-
-function getScopedConfig(config: Record<string, any>): Record<string, any> | undefined {
-  if (!config) return undefined;
-  if (typeof config.cr_aia === 'object') return config.cr_aia as Record<string, any>;
-  if (typeof config.crAia === 'object') return config.crAia as Record<string, any>;
-  if (typeof config['cr-aia'] === 'object') return config['cr-aia'] as Record<string, any>;
-  return undefined;
-}
-
-function getFirstDefined<T>(
-  rootConfig: Record<string, any> | undefined,
-  scopedConfig: Record<string, any> | undefined,
-  keys: string[],
-): T | undefined {
-  for (const key of keys) {
-    const scopedValue = scopedConfig ? getNested(scopedConfig, key) : undefined;
-    if (scopedValue !== undefined) return scopedValue as T;
-
-    const rootValue = rootConfig ? getNested(rootConfig, key) : undefined;
-    if (rootValue !== undefined) return rootValue as T;
-  }
-  return undefined;
-}
-
-function getNested(obj: Record<string, any>, pathKey: string): any {
-  const segments = pathKey.split('.');
-  let current: any = obj;
-  for (const segment of segments) {
-    if (!current || typeof current !== 'object') return undefined;
-    if (!(segment in current)) return undefined;
-    current = current[segment];
-  }
-  return current;
-}

package/src/runtime/proxyEnvironment.ts

@@ -1,71 +0,0 @@
-import type { ProxySettings } from './proxyConfig';
-import { loadProxySettings } from './proxyConfig';
-import { getSessionToken } from './auth/session';
-
-let cachedSettings: ProxySettings | null = null;
-
-function proxySuppliesApiKey(settings: ProxySettings): boolean {
-  const override = process.env.CR_AIA_PROXY_SUPPLIES_API_KEY;
-  if (override && override.trim()) {
-    return override.trim().toLowerCase() !== 'false';
-  }
-  return settings.baseUrl.toLowerCase().includes('cr.ai.enki.si');
-}
-
-function ensureApiKeyPlaceholder(settings: ProxySettings): void {
-  if (!proxySuppliesApiKey(settings)) {
-    return;
-  }
-
-  const placeholder = process.env.CR_AIA_PROXY_API_KEY_PLACEHOLDER || 'provided-by-proxy';
-
-  if (!process.env.AI_CODE_REVIEW_OPENROUTER_API_KEY) {
-    process.env.AI_CODE_REVIEW_OPENROUTER_API_KEY = placeholder;
-  }
-
-  if (!process.env.OPENROUTER_API_KEY) {
-    process.env.OPENROUTER_API_KEY = placeholder;
-  }
-}
-
-function applyProxySettings(settings: ProxySettings): void {
-  process.env.CR_AIA_PROXY_BASE_URL = settings.baseUrl;
-
-  if (settings.httpReferer) {
-    process.env.CR_AIA_PROXY_HTTP_REFERER = settings.httpReferer;
-  } else {
-    delete process.env.CR_AIA_PROXY_HTTP_REFERER;
-  }
-
-  if (settings.xTitle) {
-    process.env.CR_AIA_PROXY_X_TITLE = settings.xTitle;
-  } else {
-    delete process.env.CR_AIA_PROXY_X_TITLE;
-  }
-
-  ensureApiKeyPlaceholder(settings);
-  ensureSessionToken();
-}
-
-function ensureSessionToken(): void {
-  if (process.env.CR_AIA_PROXY_SESSION_TOKEN && process.env.CR_AIA_PROXY_SESSION_TOKEN.trim()) {
-    return;
-  }
-
-  const token = getSessionToken();
-  if (token) {
-    process.env.CR_AIA_PROXY_SESSION_TOKEN = token;
-  }
-}
-
-export function ensureProxyEnvironmentInitialized(repoRoot: string): ProxySettings {
-  if (!cachedSettings) {
-    cachedSettings = loadProxySettings(repoRoot);
-    applyProxySettings(cachedSettings);
-  }
-  return cachedSettings;
-}
-
-export function getActiveProxySettings(): ProxySettings | null {
-  return cachedSettings;
-}

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, ' ')}`,
-  );
-}