@payconductor/houndsman 0.4.0 → 0.6.0

package/src/server/client/FindingCard.tsx

@@ -7,21 +7,22 @@
  */
 
 import { useCallback, useEffect, useMemo, useRef, useState } from "preact/hooks";
-import { api } from "./api";
-import { highlightCode, languageFromFile } from "./code-highlight";
-import { DECISION_LABELS, DECISIONS, SEVERITY_LABELS } from "./constants";
-import { vscodeFileLink } from "./format";
 import {
 	Check,
 	ChevronRight,
 	CodeIcon,
 	ExternalLink,
+	GitHub,
 	Sparkles,
 	Spinner,
 	Watch,
 } from "./icons";
 import { MarkdownText } from "./MarkdownText";
-import type { Finding } from "./types";
+import { api } from "./utils/api";
+import { highlightCode, languageFromFile } from "./utils/code-highlight";
+import { DECISION_LABELS, DECISIONS, SEVERITY_LABELS } from "./utils/constants";
+import { vscodeFileLink } from "./utils/format";
+import type { Finding } from "./utils/types";
 
 /** Debounce do auto-save do comentário (ms). */
 const AUTOSAVE_MS = 700;
@@ -155,17 +156,20 @@ export function FindingCard({
 				{finding.github &&
 					(finding.github.url ? (
 						<a
-							class="sent-tag"
+							class="sent-tag gh"
 							href={finding.github.url}
 							target="_blank"
 							rel="noreferrer"
-							title="Comentário enviado à PR"
+							title={`Comentário enviado ao GitHub (PR #${finding.github.pr}) — abrir`}
 						>
-							<ExternalLink size={12} /> enviado
+							<GitHub size={12} /> enviado ao GitHub
 						</a>
 					) : (
-						<span class="sent-tag" title="Comentário enviado à PR">
-							<ExternalLink size={12} /> enviado
+						<span
+							class="sent-tag gh"
+							title={`Comentário enviado ao GitHub (PR #${finding.github.pr})`}
+						>
+							<GitHub size={12} /> enviado ao GitHub
 						</span>
 					))}
 				<span class="spacer" />

package/src/server/client/FindingsList.tsx

@@ -3,10 +3,10 @@
  */
 
 import { useMemo } from "preact/hooks";
-import { SEVERITY_LABELS, SEVERITY_ORDER } from "./constants";
 import { FindingCard } from "./FindingCard";
 import { Alert } from "./icons";
-import type { Finding, Severity } from "./types";
+import { SEVERITY_LABELS, SEVERITY_ORDER } from "./utils/constants";
+import type { Finding, Severity } from "./utils/types";
 
 export function FindingsList({
 	slug,

package/src/server/client/MarkdownText.tsx

@@ -1,28 +1,48 @@
 /**
- * Render de markdown mínimo para o campo `action` do finding.
+ * Render de markdown mínimo para o campo `action` do finding e artefatos.
  *
  * Sem HTML cru: tudo vira nós do Preact. Blocos de código reutilizam o
- * highlight de código; links passam por `safeUrl`.
+ * highlight de código; links passam por `safeUrl` e caminhos de arquivo ganham
+ * destaque próprio.
  */
 
 import { useMemo } from "preact/hooks";
-import { highlightCode, languageFromName } from "./code-highlight";
-import { type InlineToken, parseInline, parseMarkdown, safeUrl } from "./markdown";
+import { highlightCode, languageFromName } from "./utils/code-highlight";
+import {
+	type InlineToken,
+	isFilePath,
+	type MarkdownBlock,
+	parseInline,
+	parseMarkdown,
+	safeUrl,
+} from "./utils/markdown";
 
 function renderInlineToken(token: InlineToken) {
 	switch (token.type) {
 		case "code":
-			return <code class="md-code">{token.text}</code>;
+			return (
+				<code class={`md-code${isFilePath(token.text) ? " md-path" : ""}`}>
+					{token.text}
+				</code>
+			);
+		case "path":
+			return <code class="md-path">{token.text}</code>;
 		case "strong":
 			return <strong>{token.text}</strong>;
 		case "em":
 			return <em>{token.text}</em>;
 		case "link": {
+			const label = token.text.replace(/^`+|`+$/g, "");
 			const url = safeUrl(token.url);
-			return url ? (
-				<a href={url} target="_blank" rel="noreferrer">
-					{token.text}
-				</a>
+			if (url) {
+				return (
+					<a href={url} target="_blank" rel="noreferrer">
+						{label}
+					</a>
+				);
+			}
+			return isFilePath(label) || isFilePath(token.url) ? (
+				<code class="md-path">{label}</code>
 			) : (
 				token.text
 			);
@@ -59,9 +79,8 @@ function CodeBlock({ code, language }: { code: string; language: string | null }
 	);
 }
 
-/** Renderiza o markdown do action preservando quebras de linha e listas. */
-export function MarkdownText({ text }: { text: string }) {
-	const blocks = useMemo(() => parseMarkdown(text), [text]);
+/** Renderiza blocos markdown já parseados (seções de artefato, ação etc.). */
+export function MarkdownBlocks({ blocks }: { blocks: MarkdownBlock[] }) {
 	return (
 		<>
 			{blocks.map((block) => {
@@ -72,6 +91,21 @@ export function MarkdownText({ text }: { text: string }) {
 								<InlineTokens text={block.text} />
 							</p>
 						);
+					case "heading": {
+						const level = Math.min(Math.max(block.level, 1), 6);
+						const Tag = `h${level}` as
+							| "h1"
+							| "h2"
+							| "h3"
+							| "h4"
+							| "h5"
+							| "h6";
+						return (
+							<Tag class={`md-h md-h${level}`}>
+								<InlineTokens text={block.text} />
+							</Tag>
+						);
+					}
 					case "list": {
 						const items = block.items.map((item) => (
 							<li>
@@ -84,6 +118,44 @@ export function MarkdownText({ text }: { text: string }) {
 							<ul class="md-list">{items}</ul>
 						);
 					}
+					case "table":
+						return (
+							<div class="md-table-wrap">
+								<table class="md-table">
+									<thead>
+										<tr>
+											{block.header.map((cell, column) => (
+												<th
+													style={{
+														textAlign: block.align[column],
+													}}
+												>
+													<InlineTokens text={cell} />
+												</th>
+											))}
+										</tr>
+									</thead>
+									<tbody>
+										{block.rows.map((row) => (
+											<tr key={row.join("|")}>
+												{block.header.map((_, column) => (
+													<td
+														style={{
+															textAlign:
+																block.align[column],
+														}}
+													>
+														<InlineTokens
+															text={row[column] ?? ""}
+														/>
+													</td>
+												))}
+											</tr>
+										))}
+									</tbody>
+								</table>
+							</div>
+						);
 					case "code":
 						return <CodeBlock code={block.code} language={block.language} />;
 					case "quote":
@@ -92,6 +164,8 @@ export function MarkdownText({ text }: { text: string }) {
 								<InlineTokens text={block.text} />
 							</blockquote>
 						);
+					case "hr":
+						return <hr class="md-hr" />;
 					default:
 						return null;
 				}
@@ -99,3 +173,9 @@ export function MarkdownText({ text }: { text: string }) {
 		</>
 	);
 }
+
+/** Renderiza o markdown do action/artefato preservando blocos e listas. */
+export function MarkdownText({ text }: { text: string }) {
+	const blocks = useMemo(() => parseMarkdown(text), [text]);
+	return <MarkdownBlocks blocks={blocks} />;
+}

package/src/server/client/ResumePanel.tsx

@@ -0,0 +1,368 @@
+/**
+ * Aba "Resumo" (inicial): visão principal da revisão.
+ *
+ * O texto narrativo vem do artefato `review-resume.md` (Resumo de negócio); os
+ * números viram cards a partir do `classification.json`/`manifest.json` e o
+ * card de domínios resume os achados por severidade em cada watchdog — assim a
+ * UI mostra os mesmos dados do resumo de forma mais organizada, sem duplicar o
+ * texto.
+ */
+
+import type { ComponentChildren } from "preact";
+import { useEffect, useMemo, useState } from "preact/hooks";
+import { ExternalLink, Layers } from "./icons";
+import { MarkdownBlocks } from "./MarkdownText";
+import { api } from "./utils/api";
+import { SEVERITY_LABELS, SEVERITY_ORDER } from "./utils/constants";
+import { formatRunTime } from "./utils/format";
+import { sectionBlocks } from "./utils/markdown";
+import type {
+	ClassificationArtifact,
+	Finding,
+	RunDetail,
+	RunManifest,
+	Severity,
+	Summary,
+} from "./utils/types";
+
+const STATUS_VIEW = {
+	blocking: { label: "Mudanças necessárias", tone: "blocking" },
+	attention: { label: "Aprovado com ressalvas", tone: "attention" },
+	clean: { label: "Aprovado", tone: "clean" },
+} as const;
+
+const STATUS_LABELS: Record<string, string> = {
+	added: "adicionado(s)",
+	modified: "modificado(s)",
+	deleted: "removido(s)",
+	renamed: "renomeado(s)",
+	copied: "copiado(s)",
+	unknown: "alterado(s)",
+};
+
+/** Soma do churn quando o manifesto não traz os totais. */
+function churnTotals(churn: ClassificationArtifact["churn"]): {
+	additions: number;
+	deletions: number;
+} {
+	let additions = 0;
+	let deletions = 0;
+	for (const entry of Object.values(churn ?? {})) {
+		additions += entry.additions;
+		deletions += entry.deletions;
+	}
+	return { additions, deletions };
+}
+
+function ResumeKpi({
+	label,
+	value,
+	hint,
+	tone,
+}: {
+	label: string;
+	value: ComponentChildren;
+	hint?: string;
+	tone?: "blocking" | "attention" | "total";
+}) {
+	return (
+		<div class={`kpi${tone ? ` kpi-${tone}` : ""}`} title={hint}>
+			<span class="kpi-label">{label}</span>
+			<span class="kpi-value">{value}</span>
+			{hint && <span class="kpi-hint">{hint}</span>}
+		</div>
+	);
+}
+
+export function ResumePanel({
+	slug,
+	branch,
+	findings,
+	summary,
+	onOpenArtifacts,
+}: {
+	slug: string;
+	branch: string;
+	findings: Finding[];
+	summary: Summary;
+	onOpenArtifacts: () => void;
+}) {
+	const [resume, setResume] = useState("");
+	const [detail, setDetail] = useState<RunDetail | null>(null);
+	const [classification, setClassification] = useState<ClassificationArtifact | null>(
+		null,
+	);
+	const [error, setError] = useState("");
+	const [loading, setLoading] = useState(true);
+
+	useEffect(() => {
+		let cancelled = false;
+		setLoading(true);
+		setError("");
+		Promise.all([
+			api<RunDetail>(`/api/runs/${slug}`).catch(() => null),
+			fetch(`/api/runs/${slug}/artifacts/review-resume.md`)
+				.then((res) => (res.ok ? res.text() : ""))
+				.catch(() => ""),
+			api<ClassificationArtifact>(
+				`/api/runs/${slug}/artifacts/classification.json`,
+			).catch(() => null),
+		])
+			.then(([runDetail, markdown, classified]) => {
+				if (cancelled) return;
+				setDetail(runDetail);
+				setResume(markdown);
+				setClassification(classified);
+			})
+			.catch((err) => {
+				if (!cancelled) setError(String(err));
+			})
+			.finally(() => {
+				if (!cancelled) setLoading(false);
+			});
+		return () => {
+			cancelled = true;
+		};
+	}, [slug]);
+
+	const manifest: RunManifest | null = detail?.manifest ?? null;
+	const business = useMemo(() => sectionBlocks(resume, "Resumo de negócio"), [resume]);
+	const status =
+		summary.bySeverity.blocking > 0
+			? STATUS_VIEW.blocking
+			: summary.bySeverity.attention > 0
+				? STATUS_VIEW.attention
+				: STATUS_VIEW.clean;
+
+	const files = classification?.files ?? {};
+	const fileCount = Object.keys(files).length;
+	/** Arquivos com conteúdo para revisar (deletados não entram no escopo). */
+	const reviewableCount = Object.values(files).filter(
+		(file) => file.status !== "deleted",
+	).length;
+	const statuses = useMemo(() => {
+		const counts: Record<string, number> = {};
+		for (const file of Object.values(files)) {
+			counts[file.status] = (counts[file.status] ?? 0) + 1;
+		}
+		return counts;
+	}, [files]);
+	const statusLabel = Object.entries(statuses)
+		.map(([state, total]) => `${total} ${STATUS_LABELS[state] ?? state}`)
+		.join(" · ");
+	const churn = manifest?.churn ?? churnTotals(classification?.churn);
+	const pr = manifest?.context?.source;
+	const reviewedCount = useMemo(() => {
+		const set = new Set<string>();
+		for (const domain of manifest?.watchdogs ?? []) {
+			for (const file of domain.files) set.add(file);
+		}
+		return set.size;
+	}, [manifest]);
+	const coverage =
+		manifest && reviewableCount > 0
+			? `${Math.min(100, Math.round((reviewedCount / reviewableCount) * 100))}%`
+			: "—";
+
+	const domains = useMemo(() => {
+		const base = manifest?.watchdogs?.length
+			? manifest.watchdogs.map((domain) => ({
+					id: domain.id,
+					name: domain.name || domain.id,
+					files: domain.files.length,
+					overflow: domain.overflow,
+				}))
+			: (detail?.watchdogs ?? []).map((id) => ({
+					id,
+					name: id,
+					files: 0,
+					overflow: 0,
+				}));
+
+		return base
+			.map((domain) => {
+				const bySeverity: Record<Severity, number> = {
+					blocking: 0,
+					attention: 0,
+					suggestion: 0,
+					nit: 0,
+				};
+				for (const finding of findings) {
+					if (!finding.watchdogIds.includes(domain.id)) continue;
+					bySeverity[finding.severity] += 1;
+				}
+				const total = SEVERITY_ORDER.reduce(
+					(sum, severity) => sum + bySeverity[severity],
+					0,
+				);
+				return { ...domain, bySeverity, total };
+			})
+			.sort(
+				(a, b) =>
+					b.bySeverity.blocking - a.bySeverity.blocking ||
+					b.bySeverity.attention - a.bySeverity.attention ||
+					b.total - a.total ||
+					a.name.localeCompare(b.name),
+			);
+	}, [manifest, detail, findings]);
+
+	return (
+		<div class="resume">
+			<section class="resume-hero">
+				<div class="resume-hero-text">
+					<span class="resume-kicker">Resumo principal</span>
+					<h2 class="resume-title">{branch}</h2>
+					<p class="resume-meta">
+						{manifest?.generatedAt && (
+							<span>{formatRunTime(manifest.generatedAt)}</span>
+						)}
+						{domains.length > 0 && (
+							<span>
+								{domains.length}{" "}
+								{domains.length === 1 ? "domínio" : "domínios"}
+							</span>
+						)}
+						{pr?.url ? (
+							<a
+								class="resume-pr"
+								href={pr.url}
+								target="_blank"
+								rel="noreferrer"
+							>
+								{pr.number ? `PR #${pr.number}` : "Abrir PR"}
+								<ExternalLink size={12} />
+							</a>
+						) : pr?.number ? (
+							<span>PR #{pr.number}</span>
+						) : null}
+					</p>
+				</div>
+				<div class="resume-hero-side">
+					<span class={`resume-status ${status.tone}`}>{status.label}</span>
+					<button
+						type="button"
+						class="btn"
+						title="Ver os artefatos brutos da revisão"
+						onClick={onOpenArtifacts}
+					>
+						<Layers size={14} />
+						Ver artefatos
+					</button>
+				</div>
+			</section>
+
+			{error && <p class="error">{error}</p>}
+
+			<div class="kpis resume-kpis">
+				<ResumeKpi
+					label="Arquivos"
+					value={fileCount}
+					hint={statusLabel || "sem classificação"}
+				/>
+				<ResumeKpi
+					label="Churn"
+					value={
+						<>
+							<span class="churn-add">+{churn.additions}</span>
+							<span class="churn-sep">/</span>
+							<span class="churn-del">-{churn.deletions}</span>
+						</>
+					}
+					hint="linhas adicionadas / removidas"
+				/>
+				<ResumeKpi
+					label="Cobertura"
+					value={coverage}
+					hint={
+						reviewableCount > 0
+							? `${reviewedCount}/${reviewableCount} revisáveis no escopo`
+							: "sem classificação"
+					}
+				/>
+				<ResumeKpi
+					label="Achados"
+					value={summary.total}
+					hint={`${summary.bySeverity.blocking} bloqueantes · ${summary.bySeverity.attention} atenção`}
+					tone={
+						summary.bySeverity.blocking > 0
+							? "blocking"
+							: summary.bySeverity.attention > 0
+								? "attention"
+								: "total"
+					}
+				/>
+			</div>
+
+			<div class="resume-grid">
+				<section class="resume-card">
+					<h3 class="resume-card-title">Resumo de negócio</h3>
+					{loading ? (
+						<p class="empty">Carregando resumo…</p>
+					) : business.length > 0 ? (
+						<div class="resume-narrative">
+							<MarkdownBlocks blocks={business} />
+						</div>
+					) : (
+						<p class="empty">
+							O resumo narrativo ainda não foi gerado para esta run. Rode o{" "}
+							<code>--finalize</code> para criar{" "}
+							<code>review-resume.md</code>.
+						</p>
+					)}
+				</section>
+
+				<section class="resume-card">
+					<div class="resume-card-head">
+						<h3 class="resume-card-title">Domínios</h3>
+						{domains.length > 0 && (
+							<span class="resume-card-count">{domains.length}</span>
+						)}
+					</div>
+					{domains.length > 0 ? (
+						<ul class="resume-domains">
+							{domains.map((domain) => (
+								<li class="resume-domain" key={domain.id}>
+									<div class="resume-domain-head">
+										<span class="resume-domain-name">
+											{domain.name}
+										</span>
+										<span class="resume-domain-files">
+											{domain.files > 0
+												? `${domain.files} ${domain.files === 1 ? "arquivo" : "arquivos"}`
+												: "—"}
+											{domain.overflow > 0
+												? ` · ${domain.overflow} fora do orçamento`
+												: ""}
+										</span>
+									</div>
+									<div class="resume-domain-sev">
+										{domain.total === 0 ? (
+											<span class="resume-domain-ok">
+												Sem achados
+											</span>
+										) : (
+											SEVERITY_ORDER.filter(
+												(severity) =>
+													domain.bySeverity[severity] > 0,
+											).map((severity) => (
+												<span
+													class={`sev-badge sev-${severity}`}
+													key={severity}
+												>
+													{SEVERITY_LABELS[severity]}{" "}
+													{domain.bySeverity[severity]}
+												</span>
+											))
+										)}
+									</div>
+								</li>
+							))}
+						</ul>
+					) : (
+						<p class="empty">Nenhum domínio ativo na revisão.</p>
+					)}
+				</section>
+			</div>
+		</div>
+	);
+}

package/src/server/client/RunList.tsx

@@ -3,10 +3,10 @@
  * com data/hora, relação de findings presentes e badge de correção pronta.
  */
 
-import { describeRunStatus } from "./constants";
-import { formatRunTime } from "./format";
 import { BranchIcon, Clock, Sparkles } from "./icons";
-import type { RunSummary } from "./types";
+import { describeRunStatus } from "./utils/constants";
+import { formatRunTime } from "./utils/format";
+import type { RunSummary } from "./utils/types";
 
 export function RunList({
 	runs,

package/src/server/client/SubmitModal.tsx

@@ -9,16 +9,16 @@
  */
 
 import { useCallback, useEffect, useMemo, useState } from "preact/hooks";
-import { api } from "./api";
-import { EXEC_STATES, SEVERITY_LABELS, SEVERITY_ORDER } from "./constants";
-import { ExternalLink, Send, Spinner } from "./icons";
+import { GitHub, Send, Spinner } from "./icons";
+import { api } from "./utils/api";
+import { EXEC_STATES, SEVERITY_LABELS, SEVERITY_ORDER } from "./utils/constants";
 import type {
 	ExecState,
 	Severity,
 	SubmitOutcome,
 	SubmitPreview,
 	SubmitResponse,
-} from "./types";
+} from "./utils/types";
 
 const EXEC_LABELS: Record<ExecState, string> = {
 	pending: "Pendente",
@@ -333,6 +333,27 @@ export function SubmitModal({
 															arquivo
 														</span>
 													)}
+													{candidate.sent &&
+														(candidate.sentUrl ? (
+															<a
+																class="sent-tag gh"
+																href={candidate.sentUrl}
+																target="_blank"
+																rel="noreferrer"
+																title="Já enviado ao GitHub — abrir comentário"
+															>
+																<GitHub size={12} />
+																enviado
+															</a>
+														) : (
+															<span
+																class="sent-tag gh"
+																title="Já enviado ao GitHub"
+															>
+																<GitHub size={12} />
+																enviado
+															</span>
+														))}
 												</div>
 											))}
 										</div>
@@ -389,22 +410,44 @@ export function SubmitModal({
 /** Resumo do resultado do envio. */
 function ResultView({ outcome }: { outcome: SubmitOutcome }) {
 	const generalUrl = outcome.general?.url;
+	const review = outcome.review;
+	const reviewUrl = review?.url;
 	return (
 		<>
 			<div class="submit-summary">
 				Enviados <b>{outcome.posted.length}</b> · falhas{" "}
 				<b>{outcome.failed.length}</b> · pulados <b>{outcome.skipped.length}</b>.
 			</div>
+			{review && (
+				<p class="submit-empty">
+					{review.created
+						? review.finalized
+							? `Revisão #${review.id} finalizada (${review.event}).`
+							: `Revisão #${review.id} criada, mas não finalizada.`
+						: `Comentários adicionados à revisão pendente #${review.id}.`}{" "}
+					{reviewUrl && (
+						<a
+							class="sent-tag gh"
+							href={reviewUrl}
+							target="_blank"
+							rel="noreferrer"
+						>
+							<GitHub size={12} /> abrir
+						</a>
+					)}
+					{review.error && <> ❌ {review.error}</>}
+				</p>
+			)}
 			{generalUrl && (
 				<p class="submit-empty">
 					Comentário geral:{" "}
 					<a
-						class="sent-tag"
+						class="sent-tag gh"
 						href={generalUrl}
 						target="_blank"
 						rel="noreferrer"
 					>
-						<ExternalLink size={12} /> abrir
+						<GitHub size={12} /> abrir
 					</a>
 				</p>
 			)}