pi-harness-runtime 0.5.0 → 0.6.3-beta.13

package/packages/code-review/src/reports/markdown.ts

@@ -0,0 +1,197 @@
+/**
+ * Code Review Engine - Markdown Reporter
+ *
+ * Generate Markdown format reports.
+ */
+
+import type { CodeIssue, ReportOptions, ReviewResult } from "../types.js";
+
+export class MarkdownReporter {
+	/**
+	 * Generate Markdown report
+	 */
+	generate(result: ReviewResult, options: ReportOptions): string {
+		const lines: string[] = [];
+
+		// Header
+		lines.push("# Code Review Report");
+		lines.push("");
+		lines.push(`**Review ID:** ${result.id}`);
+		lines.push(`**Timestamp:** ${result.timestamp}`);
+		lines.push(`**Duration:** ${result.durationMs}ms`);
+		lines.push("");
+
+		// Summary
+		if (options.includeSummary !== false) {
+			lines.push("## Summary");
+			lines.push("");
+			lines.push(`| Metric | Value |`);
+			lines.push(`|--------|-------|`);
+			lines.push(`| Files Reviewed | ${result.stats.filesReviewed} |`);
+			lines.push(`| Lines Reviewed | ${result.stats.linesReviewed} |`);
+			lines.push(`| Total Issues | ${result.summary.total} |`);
+			lines.push(`| Errors | ${result.summary.errors} |`);
+			lines.push(`| Warnings | ${result.summary.warnings} |`);
+			lines.push(`| Info | ${result.summary.info} |`);
+			lines.push(`| Hints | ${result.summary.hints} |`);
+			lines.push("");
+		}
+
+		// Category breakdown
+		if (Object.keys(result.summary.byCategory).length > 0) {
+			lines.push("### Issues by Category");
+			lines.push("");
+			lines.push(`| Category | Count |`);
+			lines.push(`|----------|-------|`);
+			for (const [category, count] of Object.entries(
+				result.summary.byCategory,
+			)) {
+				lines.push(`| ${category} | ${count} |`);
+			}
+			lines.push("");
+		}
+
+		// Issues
+		if (result.issues.length > 0) {
+			lines.push("## Issues");
+			lines.push("");
+
+			// Group issues
+			const groupedIssues = options.groupByFile
+				? this.groupByFile(result.issues)
+				: options.groupBySeverity
+					? this.groupBySeverity(result.issues)
+					: [{ header: "", issues: result.issues }];
+
+			for (const group of groupedIssues) {
+				if (group.header) {
+					lines.push(`### ${group.header}`);
+					lines.push("");
+				}
+
+				for (const issue of group.issues) {
+					lines.push(this.formatIssue(issue, options));
+				}
+			}
+		} else {
+			lines.push("## Issues");
+			lines.push("");
+			lines.push(
+				"🎉 **No issues found!** Your code passed all automated checks.",
+			);
+			lines.push("");
+		}
+
+		// Footer
+		lines.push("---");
+		lines.push("");
+		lines.push("*Generated by pi-harness-runtime Code Review Engine*");
+
+		return lines.join("\n");
+	}
+
+	/**
+	 * Format a single issue
+	 */
+	private formatIssue(issue: CodeIssue, options: ReportOptions): string {
+		const lines: string[] = [];
+
+		const severityEmoji = this.getSeverityEmoji(issue.severity);
+		lines.push(`#### ${severityEmoji} ${issue.title}`);
+		lines.push("");
+		lines.push(`- **Severity:** ${issue.severity}`);
+		lines.push(`- **Category:** ${issue.category}`);
+		lines.push(
+			`- **Location:** ${issue.file}${issue.line ? `:${issue.line}` : ""}`,
+		);
+
+		if (issue.rule) {
+			lines.push(`- **Rule:** \`${issue.rule}\``);
+		}
+
+		lines.push("");
+		lines.push(`${issue.message}`);
+		lines.push("");
+
+		if (issue.suggestion && options.includeSuggestions !== false) {
+			lines.push(`> **Suggestion:** ${issue.suggestion}`);
+			lines.push("");
+		}
+
+		if (issue.code && options.includeCode !== false) {
+			lines.push("```");
+			lines.push(issue.code);
+			lines.push("```");
+			lines.push("");
+		}
+
+		return lines.join("\n");
+	}
+
+	/**
+	 * Get severity emoji
+	 */
+	private getSeverityEmoji(severity: string): string {
+		switch (severity) {
+			case "error":
+				return "🔴";
+			case "warning":
+				return "🟡";
+			case "info":
+				return "🔵";
+			case "hint":
+				return "💡";
+			default:
+				return "⚪";
+		}
+	}
+
+	/**
+	 * Group issues by file
+	 */
+	private groupByFile(
+		issues: CodeIssue[],
+	): { header: string; issues: CodeIssue[] }[] {
+		const groups = new Map<string, CodeIssue[]>();
+
+		for (const issue of issues) {
+			const existing = groups.get(issue.file);
+			if (existing) {
+				existing.push(issue);
+			} else {
+				groups.set(issue.file, [issue]);
+			}
+		}
+
+		return Array.from(groups.entries()).map(([file, fileIssues]) => ({
+			header: file,
+			issues: fileIssues,
+		}));
+	}
+
+	/**
+	 * Group issues by severity
+	 */
+	private groupBySeverity(
+		issues: CodeIssue[],
+	): { header: string; issues: CodeIssue[] }[] {
+		const groups = new Map<string, CodeIssue[]>();
+		const order = ["error", "warning", "info", "hint"];
+
+		for (const issue of issues) {
+			const existing = groups.get(issue.severity);
+			if (existing) {
+				existing.push(issue);
+			} else {
+				groups.set(issue.severity, [issue]);
+			}
+		}
+
+		return order
+			.filter((s) => groups.has(s))
+			.map((severity) => ({
+				header: `${severity.charAt(0).toUpperCase() + severity.slice(1)}s`,
+				issues: groups.get(severity)!,
+			}));
+	}
+}

package/packages/code-review/src/reports/text.ts

@@ -0,0 +1,171 @@
+/**
+ * Code Review Engine - Text Reporter
+ *
+ * Generate text format reports.
+ */
+
+import type { CodeIssue, ReportOptions, ReviewResult } from "../types.js";
+
+export class TextReporter {
+	/**
+	 * Generate text report
+	 */
+	generate(result: ReviewResult, options: ReportOptions): string {
+		const lines: string[] = [];
+
+		// Header
+		lines.push("=".repeat(60));
+		lines.push("CODE REVIEW REPORT");
+		lines.push("=".repeat(60));
+		lines.push(`Review ID: ${result.id}`);
+		lines.push(`Timestamp: ${result.timestamp}`);
+		lines.push(`Duration: ${result.durationMs}ms`);
+		lines.push("");
+
+		// Summary
+		if (options.includeSummary !== false) {
+			lines.push("-".repeat(60));
+			lines.push("SUMMARY");
+			lines.push("-".repeat(60));
+			lines.push(`Files Reviewed: ${result.stats.filesReviewed}`);
+			lines.push(`Lines Reviewed: ${result.stats.linesReviewed}`);
+			lines.push(`Total Issues: ${result.summary.total}`);
+			lines.push(`  - Errors: ${result.summary.errors}`);
+			lines.push(`  - Warnings: ${result.summary.warnings}`);
+			lines.push(`  - Info: ${result.summary.info}`);
+			lines.push(`  - Hints: ${result.summary.hints}`);
+			lines.push("");
+		}
+
+		// Issues
+		if (result.issues.length > 0) {
+			lines.push("-".repeat(60));
+			lines.push("ISSUES");
+			lines.push("-".repeat(60));
+
+			// Group issues
+			const groupedIssues = options.groupByFile
+				? this.groupByFile(result.issues)
+				: options.groupBySeverity
+					? this.groupBySeverity(result.issues)
+					: [{ header: "", issues: result.issues }];
+
+			for (const group of groupedIssues) {
+				if (group.header) {
+					lines.push("");
+					lines.push(`${group.header}:`);
+					lines.push("~".repeat(40));
+				}
+
+				for (const issue of group.issues) {
+					lines.push("");
+					lines.push(this.formatIssue(issue, options));
+				}
+			}
+		}
+
+		lines.push("");
+		lines.push("=".repeat(60));
+		lines.push("END OF REPORT");
+		lines.push("=".repeat(60));
+
+		return lines.join("\n");
+	}
+
+	/**
+	 * Format a single issue
+	 */
+	private formatIssue(issue: CodeIssue, options: ReportOptions): string {
+		const lines: string[] = [];
+
+		const severityIcon = this.getSeverityIcon(issue.severity);
+		lines.push(
+			`${severityIcon} [${issue.severity.toUpperCase()}] ${issue.title}`,
+		);
+		lines.push(`    Category: ${issue.category}`);
+		lines.push(`    File: ${issue.file}${issue.line ? `:${issue.line}` : ""}`);
+
+		if (issue.rule) {
+			lines.push(`    Rule: ${issue.rule}`);
+		}
+
+		lines.push(`    ${issue.message}`);
+
+		if (issue.suggestion && options.includeSuggestions !== false) {
+			lines.push(`    Suggestion: ${issue.suggestion}`);
+		}
+
+		if (issue.code && options.includeCode !== false) {
+			lines.push(`    Code: ${issue.code}`);
+		}
+
+		return lines.join("\n");
+	}
+
+	/**
+	 * Get severity icon
+	 */
+	private getSeverityIcon(severity: string): string {
+		switch (severity) {
+			case "error":
+				return "✗";
+			case "warning":
+				return "⚠";
+			case "info":
+				return "ℹ";
+			case "hint":
+				return "💡";
+			default:
+				return "•";
+		}
+	}
+
+	/**
+	 * Group issues by file
+	 */
+	private groupByFile(
+		issues: CodeIssue[],
+	): { header: string; issues: CodeIssue[] }[] {
+		const groups = new Map<string, CodeIssue[]>();
+
+		for (const issue of issues) {
+			const existing = groups.get(issue.file);
+			if (existing) {
+				existing.push(issue);
+			} else {
+				groups.set(issue.file, [issue]);
+			}
+		}
+
+		return Array.from(groups.entries()).map(([file, fileIssues]) => ({
+			header: file,
+			issues: fileIssues,
+		}));
+	}
+
+	/**
+	 * Group issues by severity
+	 */
+	private groupBySeverity(
+		issues: CodeIssue[],
+	): { header: string; issues: CodeIssue[] }[] {
+		const groups = new Map<string, CodeIssue[]>();
+		const order = ["error", "warning", "info", "hint"];
+
+		for (const issue of issues) {
+			const existing = groups.get(issue.severity);
+			if (existing) {
+				existing.push(issue);
+			} else {
+				groups.set(issue.severity, [issue]);
+			}
+		}
+
+		return order
+			.filter((s) => groups.has(s))
+			.map((severity) => ({
+				header: severity.toUpperCase(),
+				issues: groups.get(severity)!,
+			}));
+	}
+}

package/packages/code-review/src/types.ts

@@ -0,0 +1,324 @@
+/**
+ * Code Review Engine - Types
+ *
+ * Core types for automated code review.
+ */
+
+// ─── SDK Version ────────────────────────────────────────────────────────────
+
+/**
+ * SDK version for compatibility checks
+ */
+export const SDK_VERSION = "1.0.0";
+
+// ─── Severity & Category ───────────────────────────────────────────────────
+
+/**
+ * Issue severity
+ */
+export type IssueSeverity = "error" | "warning" | "info" | "hint";
+
+/**
+ * Issue category
+ */
+export type IssueCategory =
+	| "security"
+	| "performance"
+	| "correctness"
+	| "maintainability"
+	| "style"
+	| "best-practice"
+	| "typescript"
+	| "lint";
+
+/**
+ * Focus area for AI review
+ */
+export type FocusArea =
+	| "security"
+	| "performance"
+	| "correctness"
+	| "best-practices"
+	| "readability"
+	| "architecture";
+
+// ─── Issue Types ───────────────────────────────────────────────────────────
+
+/**
+ * Code issue
+ */
+export interface CodeIssue {
+	id: string;
+	severity: IssueSeverity;
+	category: IssueCategory;
+	title: string;
+	message: string;
+	file: string;
+	line?: number;
+	column?: number;
+	endLine?: number;
+	endColumn?: number;
+	rule?: string;
+	ruleUrl?: string;
+	suggestion?: string;
+	code?: string;
+	fix?: string;
+}
+
+/**
+ * Issue summary
+ */
+export interface IssueSummary {
+	total: number;
+	errors: number;
+	warnings: number;
+	info: number;
+	hints: number;
+	byCategory: Record<IssueCategory, number>;
+	byFile: Record<string, number>;
+}
+
+// ─── Review Types ──────────────────────────────────────────────────────────
+
+/**
+ * File content for review
+ */
+export interface ReviewFile {
+	path: string;
+	content: string;
+	language?: string;
+	ast?: unknown;
+}
+
+/**
+ * Diff change
+ */
+export interface DiffChange {
+	type: "added" | "removed" | "modified";
+	path: string;
+	oldContent?: string;
+	newContent: string;
+	oldLines?: number;
+	newLines?: number;
+}
+
+/**
+ * Review request
+ */
+export interface ReviewRequest {
+	files?: ReviewFile[];
+	diff?: DiffChange[];
+	rules?: ReviewRule[];
+	config?: ReviewConfig;
+	aiProvider?: AIProviderConfig;
+	includePatterns?: string[];
+	excludePatterns?: string[];
+}
+
+/**
+ * Review result
+ */
+export interface ReviewResult {
+	id: string;
+	timestamp: string;
+	durationMs: number;
+	issues: CodeIssue[];
+	summary: IssueSummary;
+	aiSuggestions?: AISuggestion[];
+	stats: ReviewStats;
+	metadata?: Record<string, unknown>;
+}
+
+/**
+ * Review statistics
+ */
+export interface ReviewStats {
+	filesReviewed: number;
+	linesReviewed: number;
+	issuesFound: number;
+	aiSuggestions: number;
+	rulesMatched: number;
+}
+
+/**
+ * Review configuration
+ */
+export interface ReviewConfig {
+	severityThreshold?: IssueSeverity;
+	categories?: IssueCategory[];
+	failOnErrors?: boolean;
+	failOnWarnings?: boolean;
+	maxIssuesPerFile?: number;
+	parallelFiles?: boolean;
+	parallelRules?: boolean;
+	timeoutMs?: number;
+	includeAiReview?: boolean;
+	aiFocusAreas?: FocusArea[];
+	aiModel?: string;
+}
+
+// ─── Rule Types ────────────────────────────────────────────────────────────
+
+/**
+ * Review rule
+ */
+export interface ReviewRule {
+	id: string;
+	name: string;
+	severity: IssueSeverity;
+	category: IssueCategory;
+	description: string;
+	pattern?: string | RegExp;
+	patternFlags?: string;
+	ASTPattern?: string;
+	message?: string;
+	suggestion?: string;
+	url?: string;
+	enabled?: boolean;
+	tags?: string[];
+}
+
+/**
+ * Rule set
+ */
+export interface RuleSet {
+	name: string;
+	description: string;
+	rules: ReviewRule[];
+	extends?: string[];
+	tags?: string[];
+}
+
+// ─── AI Types ──────────────────────────────────────────────────────────────
+
+/**
+ * AI provider configuration
+ */
+export interface AIProviderConfig {
+	provider: "openai" | "anthropic" | "ollama";
+	model?: string;
+	apiKey?: string;
+	baseUrl?: string;
+	maxTokens?: number;
+	temperature?: number;
+}
+
+/**
+ * AI suggestion
+ */
+export interface AISuggestion {
+	id: string;
+	severity: IssueSeverity;
+	category: IssueCategory;
+	title: string;
+	explanation: string;
+	suggestion: string;
+	file: string;
+	line?: number;
+	code?: string;
+	confidence: number;
+	focusArea: FocusArea;
+}
+
+/**
+ * AI review request
+ */
+export interface AIReviewRequest {
+	files: ReviewFile[];
+	focusAreas: FocusArea[];
+	maxSuggestions?: number;
+	model?: string;
+}
+
+/**
+ * AI review response
+ */
+export interface AIReviewResponse {
+	suggestions: AISuggestion[];
+	overallAssessment: string;
+	model: string;
+	tokensUsed: number;
+}
+
+// ─── Report Types ──────────────────────────────────────────────────────────
+
+/**
+ * Report format
+ */
+export type ReportFormat = "text" | "json" | "html" | "markdown";
+
+/**
+ * Report options
+ */
+export interface ReportOptions {
+	format: ReportFormat;
+	includeCode?: boolean;
+	includeSuggestions?: boolean;
+	includeStats?: boolean;
+	includeSummary?: boolean;
+	groupByFile?: boolean;
+	groupBySeverity?: boolean;
+	color?: boolean;
+	template?: string;
+}
+
+// ─── History Types ─────────────────────────────────────────────────────────
+
+/**
+ * Review history entry
+ */
+export interface ReviewHistoryEntry {
+	id: string;
+	timestamp: string;
+	branch?: string;
+	commit?: string;
+	result: ReviewResult;
+	projectPath: string;
+}
+
+// ─── Linter Types ──────────────────────────────────────────────────────────
+
+/**
+ * Linter result
+ */
+export interface LinterResult {
+	success: boolean;
+	issues: CodeIssue[];
+	output?: string;
+	error?: string;
+	durationMs: number;
+}
+
+/**
+ * Linter configuration
+ */
+export interface LinterConfig {
+	name: string;
+	enabled: boolean;
+	configPath?: string;
+	rules?: Record<string, unknown>;
+	extensions?: string[];
+}
+
+// ─── Git Types ─────────────────────────────────────────────────────────────
+
+/**
+ * Git diff
+ */
+export interface GitDiff {
+	files: DiffChange[];
+	from: string;
+	to: string;
+	baseBranch?: string;
+	headBranch?: string;
+}
+
+/**
+ * Changed file
+ */
+export interface ChangedFile {
+	path: string;
+	status: "added" | "modified" | "deleted" | "renamed";
+	additions: number;
+	deletions: number;
+}