@payconductor/houndsman 0.8.0 → 0.11.0

package/src/server/client/PrsSection.tsx

@@ -0,0 +1,296 @@
+/**
+ * Seção `PRs` da sidebar: lista as PRs abertas via `gh`, abre a run local da
+ * branch, cria worktree dedicado e faz checkout quando a árvore está limpa.
+ *
+ * A seção fica montada mesmo quando outra está ativa — o fetch só ocorre na
+ * primeira ativação (`active`), preservando filtros e estados ao alternar.
+ */
+
+import { useCallback, useEffect, useRef, useState } from "preact/hooks";
+import { BranchIcon, Clock, GitHub, GitPullRequest, Refresh, Spinner } from "./icons";
+import type { SidebarSectionContext } from "./sidebar-sections";
+import { api } from "./utils/api";
+import { formatRunTime } from "./utils/format";
+import type {
+	CheckoutResponse,
+	PrListItem,
+	PrListResponse,
+	WorktreeState,
+} from "./utils/types";
+
+const REVIEW_LABELS: Record<string, string> = {
+	APPROVED: "aprovada",
+	CHANGES_REQUESTED: "mudanças",
+	REVIEW_REQUIRED: "revisão pendente",
+};
+
+function PrItem({
+	pr,
+	currentBranch,
+	worktree,
+	busy,
+	confirming,
+	onConfirm,
+	onCancel,
+	onCheckout,
+	onSelectRun,
+}: {
+	pr: PrListItem;
+	currentBranch: string | null;
+	worktree: WorktreeState;
+	busy: boolean;
+	confirming: boolean;
+	onConfirm: (number: number) => void;
+	onCancel: () => void;
+	onCheckout: (pr: PrListItem) => void;
+	onSelectRun: (slug: string) => void;
+}) {
+	const isCurrent = currentBranch === pr.headRefName;
+	const runSlug = pr.runSlug;
+	const review = pr.reviewDecision ? REVIEW_LABELS[pr.reviewDecision] : undefined;
+	// O título da PR às vezes repete o nome da branch; nesse caso só o
+	// indicador `head → base` fica visível.
+	const showTitle = pr.title.trim() !== pr.headRefName.trim();
+	return (
+		<article class={`pr${isCurrent ? " current" : ""}`}>
+			<div class="pr-head">
+				<GitPullRequest size={14} class="pr-icon" />
+				<span class="pr-number">#{pr.number}</span>
+				{pr.isDraft && <span class="tag">draft</span>}
+				{isCurrent && <span class="pr-current">atual</span>}
+				{pr.worktreePath && (
+					<span
+						class="pr-worktree-badge"
+						title={`Worktree: ${pr.worktreePath}`}
+					>
+						<BranchIcon size={10} />
+						worktree
+					</span>
+				)}
+				<span class="spacer" />
+				{pr.updatedAt && (
+					<span class="run-when">
+						<Clock size={12} />
+						{formatRunTime(pr.updatedAt)}
+					</span>
+				)}
+			</div>
+			{showTitle && <p class="pr-title">{pr.title}</p>}
+			<p class="pr-meta">
+				<span class="pr-branch">{pr.headRefName}</span>
+				<span class="pr-arrow">→</span>
+				<span>{pr.baseRefName || "—"}</span>
+				<span>· {pr.author}</span>
+			</p>
+			{review && <p class={`pr-review pr-review-${pr.reviewDecision}`}>{review}</p>}
+			<div class="pr-actions">
+				{runSlug && (
+					<button
+						type="button"
+						class="pr-btn"
+						title="Abrir a run local desta branch"
+						onClick={() => onSelectRun(runSlug)}
+					>
+						Abrir run
+					</button>
+				)}
+				{pr.url && (
+					<a
+						class="pr-btn"
+						href={pr.url}
+						target="_blank"
+						rel="noreferrer"
+						title="Abrir a PR no GitHub"
+					>
+						<GitHub size={12} /> GitHub
+					</a>
+				)}
+				{confirming ? (
+					<>
+						<button
+							type="button"
+							class="pr-btn danger"
+							disabled={busy}
+							title="Confirmar checkout da PR"
+							onClick={() => onCheckout(pr)}
+						>
+							{busy ? <Spinner size={12} class="icon spin" /> : "Confirmar"}
+						</button>
+						<button
+							type="button"
+							class="pr-btn"
+							disabled={busy}
+							onClick={onCancel}
+						>
+							Cancelar
+						</button>
+					</>
+				) : (
+					<button
+						type="button"
+						class="pr-btn"
+						disabled={!worktree.clean || busy}
+						title={
+							worktree.clean
+								? `Fazer checkout da branch ${pr.headRefName}`
+								: "Há alterações não commitadas — faça commit ou stash antes"
+						}
+						onClick={() => onConfirm(pr.number)}
+					>
+						Checkout
+					</button>
+				)}
+			</div>
+		</article>
+	);
+}
+
+export function PrsSection({ active, onSelect, onRefreshRuns }: SidebarSectionContext) {
+	const [data, setData] = useState<PrListResponse | null>(null);
+	const [loading, setLoading] = useState(false);
+	const [error, setError] = useState("");
+	const [notice, setNotice] = useState("");
+	const [busy, setBusy] = useState<number | null>(null);
+	const [confirming, setConfirming] = useState<number | null>(null);
+	const requested = useRef(false);
+
+	const loadPrs = useCallback(async () => {
+		setLoading(true);
+		setError("");
+		try {
+			setData(await api<PrListResponse>("/api/prs"));
+		} catch (err) {
+			setError(String(err));
+		} finally {
+			setLoading(false);
+		}
+	}, []);
+
+	useEffect(() => {
+		if (!active || requested.current) return;
+		requested.current = true;
+		void loadPrs();
+	}, [active, loadPrs]);
+
+	const checkout = useCallback(
+		async (pr: PrListItem) => {
+			setBusy(pr.number);
+			setError("");
+			setNotice("");
+			try {
+				const result = await api<CheckoutResponse>("/api/prs/checkout", {
+					method: "POST",
+					headers: { "content-type": "application/json" },
+					body: JSON.stringify({ number: pr.number }),
+				});
+				if (!result.ok) {
+					if (result.reason === "dirty") {
+						setData((current) =>
+							current
+								? {
+										...current,
+										worktree: {
+											...current.worktree,
+											clean: false,
+											changes:
+												result.changes?.length ??
+												current.worktree.changes,
+											sample:
+												result.changes ?? current.worktree.sample,
+										},
+									}
+								: current,
+						);
+					}
+					setError(
+						result.reason === "dirty"
+							? `Checkout bloqueado: ${result.detail}`
+							: result.detail,
+					);
+					return;
+				}
+				setConfirming(null);
+				setNotice(
+					`Checkout de #${pr.number} concluído${
+						result.branch ? ` — ${result.branch}` : ""
+					}.`,
+				);
+				await loadPrs();
+				onRefreshRuns();
+				if (pr.runSlug) onSelect(pr.runSlug);
+			} catch (err) {
+				setError(String(err));
+			} finally {
+				setBusy(null);
+			}
+		},
+		[loadPrs, onRefreshRuns, onSelect],
+	);
+
+	const openCount = data?.available ? data.prs.length : null;
+
+	return (
+		<>
+			<div class="pr-toolbar">
+				<p class="section-label">
+					{openCount === null
+						? "PRs abertas"
+						: `${openCount} ${openCount === 1 ? "aberta" : "abertas"}`}
+				</p>
+				<button
+					type="button"
+					class="icon-btn"
+					title="Atualizar lista de PRs"
+					disabled={loading}
+					onClick={() => void loadPrs()}
+				>
+					<Refresh size={13} class={loading ? "icon spin" : undefined} />
+				</button>
+			</div>
+
+			{error && <p class="error">{error}</p>}
+			{notice && <p class="pr-notice">{notice}</p>}
+			{data && !data.available && (
+				<p class="empty">
+					GitHub indisponível
+					{data.reason ? `: ${data.reason}` : "."}
+				</p>
+			)}
+			{data && !data.worktree.available && (
+				<p class="empty">Não foi possível ler o estado do git.</p>
+			)}
+			{data?.worktree.available && !data.worktree.clean && (
+				<p class="pr-warning">
+					{data.worktree.changes} alteração(ões) não commitada(s) — checkout
+					bloqueado até limpar a árvore.
+				</p>
+			)}
+			{data?.currentBranch && (
+				<p class="pr-current-branch">
+					Branch atual: <code>{data.currentBranch}</code>
+				</p>
+			)}
+
+			<nav class="prs" aria-label="Pull requests abertas">
+				{loading && !data && <p class="empty">Carregando PRs…</p>}
+				{data?.available && data.prs.length === 0 && (
+					<p class="empty">Nenhuma PR aberta.</p>
+				)}
+				{data?.prs.map((pr) => (
+					<PrItem
+						key={pr.number}
+						pr={pr}
+						currentBranch={data.currentBranch}
+						worktree={data.worktree}
+						busy={busy !== null}
+						confirming={confirming === pr.number}
+						onConfirm={setConfirming}
+						onCancel={() => setConfirming(null)}
+						onCheckout={checkout}
+						onSelectRun={onSelect}
+					/>
+				))}
+			</nav>
+		</>
+	);
+}

package/src/server/client/ResumePanel.tsx

@@ -1,21 +1,19 @@
 /**
  * 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.
+ * O texto narrativo vem do artefato `review-resume.md` e é exibido por inteiro
+ * (resumo de negócio + pontos técnicos); 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.
  */
 
 import type { ComponentChildren } from "preact";
 import { useEffect, useMemo, useState } from "preact/hooks";
-import { ExternalLink, Layers, Trash } from "./icons";
-import { MarkdownBlocks } from "./MarkdownText";
+import { ExternalLink, Trash } from "./icons";
+import { MarkdownText } 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,
@@ -126,7 +124,6 @@ export function ResumePanel({
 	}, [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
@@ -295,16 +292,16 @@ export function ResumePanel({
 
 			<div class="resume-grid">
 				<section class="resume-card">
-					<h3 class="resume-card-title">Resumo de negócio</h3>
+					<h3 class="resume-card-title">Resumo da revisão</h3>
 					{loading ? (
 						<p class="empty">Carregando resumo…</p>
-					) : business.length > 0 ? (
+					) : resume.trim().length > 0 ? (
 						<div class="resume-narrative">
-							<MarkdownBlocks blocks={business} />
+							<MarkdownText text={resume} />
 						</div>
 					) : (
 						<p class="empty">
-							O resumo narrativo ainda não foi gerado para esta run. Rode o{" "}
+							O resumo ainda não foi gerado para esta run. Rode o{" "}
 							<code>--finalize</code> para criar{" "}
 							<code>review-resume.md</code>.
 						</p>

package/src/server/client/RunsSection.tsx

@@ -0,0 +1,73 @@
+/**
+ * Seção `Execuções` da sidebar: lista das runs revisadas, com status, data e
+ * contagem de findings. Selecionar uma run troca apenas o conteúdo principal.
+ */
+
+import { BranchIcon, Clock, Sparkles } from "./icons";
+import type { SidebarSectionContext } from "./sidebar-sections";
+import { describeRunStatus } from "./utils/constants";
+import { formatRunTime } from "./utils/format";
+import type { RunSummary } from "./utils/types";
+
+function RunItem({
+	run,
+	active,
+	onSelect,
+}: {
+	run: RunSummary;
+	active: boolean;
+	onSelect: (slug: string) => void;
+}) {
+	const status = describeRunStatus(run);
+	const fixReady = status?.kind === "fix_done";
+	return (
+		<button
+			type="button"
+			title={`Abrir a run da branch ${run.branch}`}
+			class={`run${active ? " active" : ""}`}
+			onClick={() => onSelect(run.slug)}
+		>
+			<span class="run-top">
+				<BranchIcon size={14} class="run-icon" />
+				<span class="run-branch">{run.branch}</span>
+			</span>
+			<span class="run-meta">
+				<span class="run-when">
+					<Clock size={12} />
+					{formatRunTime(run.updatedAt) || "sem data"}
+				</span>
+				<span
+					class="run-count"
+					title={`${run.summary.present} de ${run.summary.total} findings presentes`}
+				>
+					{run.summary.present}/{run.summary.total}
+				</span>
+			</span>
+			{fixReady ? (
+				<span class="run-badge">
+					<Sparkles size={12} /> Correção pronta
+				</span>
+			) : (
+				status && (
+					<span class={`run-status status-${status.kind}`}>{status.label}</span>
+				)
+			)}
+		</button>
+	);
+}
+
+export function RunsSection({ runs, slug, onSelect }: SidebarSectionContext) {
+	return (
+		<nav class="runs" aria-label="Execuções">
+			{runs.length === 0 && <p class="empty">Nenhuma run encontrada.</p>}
+			{runs.map((run) => (
+				<RunItem
+					key={run.slug}
+					run={run}
+					active={run.slug === slug}
+					onSelect={onSelect}
+				/>
+			))}
+		</nav>
+	);
+}