@payconductor/houndsman 0.3.0 → 0.5.0

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",
@@ -122,8 +122,13 @@ export function SubmitModal({
 		() => selected.filter((candidate) => includeSent || !candidate.sent),
 		[selected, includeSent],
 	);
-	const onLine = toSend.filter((candidate) => candidate.kind === "line").length;
-	const onFile = toSend.filter((candidate) => candidate.kind === "file").length;
+	const totals = useMemo(
+		() => ({
+			line: toSend.filter((candidate) => candidate.kind === "line").length,
+			file: toSend.filter((candidate) => candidate.kind === "file").length,
+		}),
+		[toSend],
+	);
 	const skipped = selected.length - toSend.length;
 
 	const submit = useCallback(async () => {
@@ -277,8 +282,8 @@ export function SubmitModal({
 
 								<div class="submit-summary">
 									Serão disparados <b>{toSend.length}</b> comentário(s)
-									— <b>{onLine}</b> na linha · <b>{onFile}</b> no
-									arquivo
+									— <b>{totals.line}</b> na linha · <b>{totals.file}</b>{" "}
+									no arquivo
 									{skipped > 0 ? (
 										<>
 											{" "}
@@ -328,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>
@@ -384,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>
 			)}

package/src/server/client/SummaryBar.tsx

@@ -3,7 +3,7 @@
  * progresso da correção (feitos ÷ total).
  */
 
-import type { Summary } from "./types";
+import type { Summary } from "./utils/types";
 
 function Kpi({
 	label,

package/src/server/client/icons.tsx

@@ -51,6 +51,14 @@ export const ExternalLink = (p: IconProps) => (
 	</Svg>
 );
 
+/** Marca do GitHub (contorno) — usada em achados já publicados na PR. */
+export const GitHub = (p: IconProps) => (
+	<Svg {...p}>
+		<path d="M9 19c-4.3 1.3-4.3-2.2-6-2.7" />
+		<path d="M15 22v-3.9a3.4 3.4 0 0 0-.9-2.6c3-.3 6.1-1.5 6.1-6.8a5.3 5.3 0 0 0-1.4-3.7 4.9 4.9 0 0 0-.1-3.6s-1.1-.4-3.7 1.4a12.7 12.7 0 0 0-6.8 0C5.6 1 4.5 1.4 4.5 1.4a4.9 4.9 0 0 0-.1 3.6A5.3 5.3 0 0 0 3 8.7c0 5.2 3.1 6.4 6.1 6.8a3.4 3.4 0 0 0-.9 2.6V22" />
+	</Svg>
+);
+
 export const Sparkles = (p: IconProps) => (
 	<Svg {...p}>
 		<path d="M11 3.5l1.7 4.3 4.3 1.7-4.3 1.7L11 15.5 9.3 11.2 5 9.5l4.3-1.7z" />
@@ -126,6 +134,15 @@ export const Layers = (p: IconProps) => (
 	</Svg>
 );
 
+/** Documento com linhas — aba de resumo. */
+export const FileText = (p: IconProps) => (
+	<Svg {...p}>
+		<path d="M14 3H7a2 2 0 0 0-2 2v14a2 2 0 0 0 2 2h10a2 2 0 0 0 2-2V8z" />
+		<path d="M14 3v5h5" />
+		<path d="M9 13h6M9 17h4" />
+	</Svg>
+);
+
 /** Olho — "quem auditou o trecho" (watchdogs envolvidos). */
 export const Watch = (p: IconProps) => (
 	<Svg {...p}>