@payconductor/houndsman 0.6.0 → 0.8.0

package/src/harness/hunt/hunt-session.ts

@@ -14,11 +14,9 @@ import {
 	type ModelProvider,
 } from "../../core/ai";
 import type { AstAnalysis } from "../../core/ast/types";
-import { collectDiff } from "../../core/diff-source";
 import type { HoundsmanEngine } from "../../core/engine/houndsman-engine";
 import type { RunClassification } from "../../core/engine/review-scope";
 import { parseEntries } from "../../core/file-classifier";
-import { runGit, tryGit } from "../../core/git-runner";
 import type { LoggerLike } from "../../core/logging";
 import type { PrepareResult } from "../../core/orchestrator";
 import { defaultOptions, type ReviewOptions } from "../../core/orchestrator";
@@ -67,6 +65,8 @@ export class HuntSession {
 			clean: options.clean,
 			noAi: true,
 			finalize: false,
+			pr: options.pr,
+			worktreePath: options.worktreePath,
 		};
 
 		const session = engine.session(reviewOptions);
@@ -137,7 +137,7 @@ export class HuntSession {
 			reporter,
 			config: this.deps.config,
 			ast: engine.ast,
-			readFile: defaultFileReader(fileRoot()),
+			readFile: defaultFileReader(engine.root),
 		});
 		const runner = new AgentRunner(factory);
 		const scheduler = new AgentScheduler(
@@ -265,11 +265,11 @@ export class HuntSession {
 		work: import("../../core/storage/work-area").ReviewWorkArea,
 	) {
 		const { engine, options } = this.deps;
-		const diff = collectDiff(options.base, options.head);
+		const diff = engine.git.collectDiff(options.base, options.head);
 		const entries = parseEntries(
-			runGit(["diff", `${options.base}...${options.head}`, "--name-status"]),
+			engine.git.diffNameStatus(options.base, options.head),
 		);
-		const baseRef = mergeBase(options.base, options.head) ?? options.base;
+		const baseRef = engine.git.mergeBase(options.base, options.head) ?? options.base;
 		const ast: Map<string, AstAnalysis> = options.noAst
 			? new Map()
 			: await engine.ast.analyze(entries, diff, baseRef);
@@ -283,7 +283,7 @@ export class HuntSession {
 			context,
 			ast,
 			impact,
-			repoRoot: process.cwd(),
+			repoRoot: engine.root,
 			diff,
 			branch: review.branch,
 			watchdog: options.watchdog,
@@ -357,16 +357,3 @@ function readContextSummary(
 		return null;
 	}
 }
-
-/** Merge-base best effort. */
-function mergeBase(base: string, head: string): string | null {
-	const result = tryGit(["merge-base", base, head]);
-	if (!result.ok) return null;
-	const sha = result.stdout.trim();
-	return sha.length > 0 ? sha : null;
-}
-
-/** Raiz do repositório para leitura das tools. */
-function fileRoot(): string {
-	return process.cwd();
-}

package/src/index.ts

@@ -89,6 +89,14 @@ export { buildCarryover, mergeCarryover } from "./core/engine/carryover";
 export { HoundsmanEngine } from "./core/engine/houndsman-engine";
 export { ReviewScopeResolver } from "./core/engine/review-scope";
 export { ReviewSession } from "./core/engine/review-session";
+export type {
+	ResolvedWorkspace,
+	WorkspaceRequest,
+} from "./core/engine/review-workspace";
+export {
+	mainRootOf,
+	resolveReviewWorkspace,
+} from "./core/engine/review-workspace";
 export type { HoundsmanEnv } from "./core/env";
 export { ENV_NAMES, env, readEnv } from "./core/env";
 export {
@@ -171,8 +179,26 @@ export { FixBatchBuilder } from "./core/services/fix-batch-builder";
 export type { GhResult, GhRunner } from "./core/services/gh-client";
 export { GhClient } from "./core/services/gh-client";
 export { GitRepository } from "./core/services/git-repository";
+export type {
+	PrRefs,
+	PrResolveResult,
+	PullRequestInfo,
+} from "./core/services/pr-resolver";
+export { PrResolver } from "./core/services/pr-resolver";
 export { ProjectFiles } from "./core/services/project-files";
 export { Verifier } from "./core/services/verifier";
+export type {
+	EnsureWorktreeInput,
+	EnsureWorktreeResult,
+	RemoveWorktreeResult,
+	WorktreeEntry,
+	WorktreeManagerOptions,
+} from "./core/services/worktree-manager";
+export {
+	parseWorktreeList,
+	repoSlug,
+	WorktreeManager,
+} from "./core/services/worktree-manager";
 export {
 	createMemoryStorage,
 	createStorage,
@@ -226,4 +252,9 @@ export {
 } from "./core/watchdog-registry";
 export type { ServerOptions } from "./server";
 export { createRequestHandler, HoundsmanServer, startServer } from "./server";
+export type {
+	DeleteRunResult,
+	RunDetail,
+	RunListItem,
+} from "./server/run-repository";
 export { RunRepository } from "./server/run-repository";

package/src/server/client/App.tsx

@@ -7,18 +7,26 @@
 import { useCallback, useEffect, useMemo, useRef, useState } from "preact/hooks";
 import { ArtifactsPanel } from "./ArtifactsPanel";
 import { AutoRefresh } from "./AutoRefresh";
+import { CommentsPanel } from "./CommentsPanel";
+import { DeleteRunModal } from "./DeleteRunModal";
 import { Filters, type FiltersState } from "./Filters";
 import { FindingsList } from "./FindingsList";
-import { FileText, Layers, ListIcon, Refresh, Send } from "./icons";
+import { FileText, Layers, ListIcon, MessageSquare, Refresh, Send } from "./icons";
 import { ResumePanel } from "./ResumePanel";
-import { RunList } from "./RunList";
+import { Sidebar } from "./Sidebar";
 import { SubmitModal } from "./SubmitModal";
 import { SummaryBar } from "./SummaryBar";
 import { api, computeSummary } from "./utils/api";
 import { AUTO_REFRESH_MS, TAB_CAPTIONS } from "./utils/constants";
-import type { Finding, FindingsResponse, RunSummary } from "./utils/types";
+import type {
+	DeleteRunResponse,
+	EditorContext,
+	Finding,
+	FindingsResponse,
+	RunSummary,
+} from "./utils/types";
 
-type Tab = "resume" | "findings" | "artifacts";
+type Tab = "resume" | "findings" | "artifacts" | "comments";
 
 const TABS: { id: Tab; label: string; hint: string }[] = [
 	{
@@ -36,8 +44,20 @@ const TABS: { id: Tab; label: string; hint: string }[] = [
 		label: "Artefatos",
 		hint: "Arquivos brutos gerados pela revisão (manifesto, classificação, relatório…)",
 	},
+	{
+		id: "comments",
+		label: "Comentários",
+		hint: "Comentários da PR: threads de revisão e comentários gerais (carregados ao abrir a aba)",
+	},
 ];
 
+function TabIcon({ id }: { id: Tab }) {
+	if (id === "resume") return <FileText size={15} />;
+	if (id === "findings") return <ListIcon size={15} />;
+	if (id === "artifacts") return <Layers size={15} />;
+	return <MessageSquare size={15} />;
+}
+
 export function App() {
 	const [runs, setRuns] = useState<RunSummary[]>([]);
 	const [slug, setSlug] = useState<string | null>(null);
@@ -54,6 +74,9 @@ export function App() {
 	const [refreshing, setRefreshing] = useState(false);
 	const [syncing, setSyncing] = useState(false);
 	const [submitOpen, setSubmitOpen] = useState(false);
+	const [deleteOpen, setDeleteOpen] = useState(false);
+	/** Editor detectado pelo servidor — base dos links de código. */
+	const [editor, setEditor] = useState<EditorContext>({ kind: "vscode" });
 	/** Refs espelhando estado volátil — o polling não deve recriar o intervalo. */
 	const lastSyncRef = useRef(lastSync);
 	const slugRef = useRef<string | null>(null);
@@ -116,6 +139,14 @@ export function App() {
 		reloadRuns();
 	}, [reloadRuns]);
 
+	useEffect(() => {
+		api<EditorContext>("/api/editor")
+			.then(setEditor)
+			.catch(() => {
+				// fail-open: mantém o fallback VS Code.
+			});
+	}, []);
+
 	useEffect(() => {
 		if (slug) loadFindings(slug);
 	}, [slug, loadFindings]);
@@ -189,12 +220,37 @@ export function App() {
 
 	const summary = useMemo(() => computeSummary(data?.findings ?? []), [data]);
 
-	const branch =
-		data?.branch ?? runs.find((run) => run.slug === slug)?.branch ?? slug ?? "";
+	const selectedRun = runs.find((run) => run.slug === slug) ?? null;
+	const branch = data?.branch ?? selectedRun?.branch ?? slug ?? "";
+
+	/** Após excluir: limpa a seleção e escolhe a próxima run da lista. */
+	const afterDelete = useCallback((result: DeleteRunResponse) => {
+		setDeleteOpen(false);
+		setData(null);
+		setSlug(null);
+		if (result.warning) {
+			setError(
+				`Execução excluída, mas o worktree não foi removido: ${result.warning}`,
+			);
+		} else {
+			setError("");
+		}
+		api<{ runs: RunSummary[] }>("/api/runs")
+			.then(({ runs: list }) => {
+				setRuns(list);
+				setSlug(list[0]?.slug ?? null);
+			})
+			.catch((err) => setError(String(err)));
+	}, []);
 
 	return (
 		<div class="app">
-			<RunList runs={runs} slug={slug} onSelect={setSlug} />
+			<Sidebar
+				runs={runs}
+				slug={slug}
+				onSelect={setSlug}
+				onRefreshRuns={() => void reloadRuns()}
+			/>
 			<main class="main">
 				<header class="topbar">
 					<div class="segmented">
@@ -206,13 +262,7 @@ export function App() {
 								class={`seg-btn${tab === item.id ? " active" : ""}`}
 								onClick={() => setTab(item.id)}
 							>
-								{item.id === "resume" ? (
-									<FileText size={15} />
-								) : item.id === "findings" ? (
-									<ListIcon size={15} />
-								) : (
-									<Layers size={15} />
-								)}
+								<TabIcon id={item.id} />
 								{item.label}
 							</button>
 						))}
@@ -268,7 +318,7 @@ export function App() {
 							branch={branch}
 							findings={data.findings}
 							summary={summary}
-							onOpenArtifacts={() => setTab("artifacts")}
+							onRequestDelete={() => setDeleteOpen(true)}
 						/>
 					)}
 					{slug && tab === "findings" && data && (
@@ -278,21 +328,35 @@ export function App() {
 							<FindingsList
 								slug={slug}
 								findings={findings}
+								editor={editor}
 								onUpdated={onUpdated}
 							/>
 						</>
 					)}
 					{slug && tab === "artifacts" && <ArtifactsPanel slug={slug} />}
+					{slug && tab === "comments" && (
+						<CommentsPanel slug={slug} editor={editor} />
+					)}
 					{!slug && !error && <p class="empty">Nenhuma execução para triar.</p>}
 				</div>
 				{slug && (
-					<SubmitModal
-						slug={slug}
-						branch={branch}
-						open={submitOpen}
-						onClose={() => setSubmitOpen(false)}
-						onSubmitted={afterSubmit}
-					/>
+					<>
+						<SubmitModal
+							slug={slug}
+							branch={branch}
+							open={submitOpen}
+							onClose={() => setSubmitOpen(false)}
+							onSubmitted={afterSubmit}
+						/>
+						<DeleteRunModal
+							slug={slug}
+							branch={branch}
+							worktreePath={selectedRun?.worktreePath}
+							open={deleteOpen}
+							onClose={() => setDeleteOpen(false)}
+							onDeleted={afterDelete}
+						/>
+					</>
 				)}
 			</main>
 		</div>

package/src/server/client/CommentsPanel.tsx

@@ -0,0 +1,255 @@
+/**
+ * Aba "Comentários": threads de revisão e comentários gerais da PR da run.
+ *
+ * O componente só é montado quando a aba é acessada — o carregamento (e a
+ * chamada ao `gh`) acontece no mount, nunca no carregamento da página.
+ */
+
+import { useCallback, useEffect, useMemo, useState } from "preact/hooks";
+import { Clock, CodeIcon, ExternalLink, GitHub, MessageSquare, Refresh } from "./icons";
+import { MarkdownText } from "./MarkdownText";
+import { api } from "./utils/api";
+import { editorFileLink, editorLabel, formatRunTime } from "./utils/format";
+import type {
+	EditorContext,
+	PrCommentsResponse,
+	PrCommentThread,
+	PrCommentView,
+} from "./utils/types";
+
+function CommentCard({
+	comment,
+	editor,
+	reply,
+}: {
+	comment: PrCommentView;
+	editor: EditorContext;
+	reply: boolean;
+}) {
+	const location = comment.path
+		? comment.line
+			? `${comment.path}:${comment.line}`
+			: comment.path
+		: null;
+	const href = comment.absFile
+		? editorFileLink(comment.absFile, comment.line, editor)
+		: null;
+	return (
+		<article class={`pr-comment${reply ? " reply" : ""}`}>
+			<header class="pr-comment-head">
+				<span class="pr-comment-author">{comment.author}</span>
+				{comment.authorType === "Bot" && <span class="tag">bot</span>}
+				<span class="run-when">
+					<Clock size={12} />
+					{formatRunTime(comment.createdAt)}
+				</span>
+				<span class="spacer" />
+				{comment.url && (
+					<a
+						class="pr-comment-link"
+						href={comment.url}
+						target="_blank"
+						rel="noreferrer"
+						title="Abrir o comentário no GitHub"
+					>
+						<GitHub size={12} /> GitHub
+					</a>
+				)}
+			</header>
+			<div class="pr-comment-body">
+				<MarkdownText text={comment.body} />
+			</div>
+			{location &&
+				(href ? (
+					<a
+						class="loc"
+						href={href}
+						title={`Abrir ${location} no ${editorLabel(editor)}`}
+					>
+						<CodeIcon size={13} />
+						{location}
+						<ExternalLink size={12} class="ext" />
+					</a>
+				) : (
+					<code class="loc static">
+						<CodeIcon size={13} />
+						{location}
+					</code>
+				))}
+		</article>
+	);
+}
+
+function ThreadCard({
+	thread,
+	editor,
+}: {
+	thread: PrCommentThread;
+	editor: EditorContext;
+}) {
+	const location = thread.line
+		? `${thread.path}:${thread.line}`
+		: thread.path || "arquivo";
+	const href = thread.absFile
+		? editorFileLink(thread.absFile, thread.line, editor)
+		: null;
+	return (
+		<section class="comment-thread">
+			<header class="comment-thread-head">
+				<CodeIcon size={13} />
+				<span class="comment-file">{location}</span>
+				<span class="spacer" />
+				{href && (
+					<a
+						class="icon-btn"
+						href={href}
+						title={`Abrir ${location} no ${editorLabel(editor)}`}
+					>
+						<ExternalLink size={13} />
+					</a>
+				)}
+				<span class="count">{thread.comments.length}</span>
+			</header>
+			{thread.comments.map((comment, index) => (
+				<CommentCard
+					key={comment.id}
+					comment={comment}
+					editor={editor}
+					reply={index > 0}
+				/>
+			))}
+		</section>
+	);
+}
+
+export function CommentsPanel({ slug, editor }: { slug: string; editor: EditorContext }) {
+	const [data, setData] = useState<PrCommentsResponse | null>(null);
+	const [loading, setLoading] = useState(true);
+	const [error, setError] = useState("");
+
+	const load = useCallback(
+		async (refresh = false) => {
+			setLoading(true);
+			setError("");
+			try {
+				const result = await api<PrCommentsResponse>(
+					`/api/runs/${slug}/comments${refresh ? "?refresh=1" : ""}`,
+				);
+				setData(result);
+			} catch (err) {
+				setError(String(err));
+			} finally {
+				setLoading(false);
+			}
+		},
+		[slug],
+	);
+
+	useEffect(() => {
+		setData(null);
+		void load();
+	}, [load]);
+
+	const groups = useMemo(() => {
+		const map = new Map<string, PrCommentThread[]>();
+		for (const thread of data?.threads ?? []) {
+			const key = thread.path || "(sem arquivo)";
+			const list = map.get(key) ?? [];
+			list.push(thread);
+			map.set(key, list);
+		}
+		return [...map.entries()];
+	}, [data]);
+
+	const empty =
+		data?.available && data.threads.length === 0 && data.issueComments.length === 0;
+
+	return (
+		<div class="comments">
+			<div class="comments-toolbar">
+				<div class="comments-counts">
+					<MessageSquare size={14} />
+					{data?.available ? (
+						<>
+							<span>
+								<b>{data.counts.threads}</b> threads
+							</span>
+							<span>
+								<b>{data.counts.inline}</b> inline
+							</span>
+							<span>
+								<b>{data.counts.issue}</b> gerais
+							</span>
+						</>
+					) : (
+						<span>Comentários da PR</span>
+					)}
+					{data?.target?.url && (
+						<a
+							class="pr-comment-link"
+							href={data.target.url}
+							target="_blank"
+							rel="noreferrer"
+						>
+							<GitHub size={12} /> PR #{data.target.number}
+						</a>
+					)}
+				</div>
+				<button
+					type="button"
+					class="btn"
+					disabled={loading}
+					title="Recarrega os comentários ignorando o cache de 60s"
+					onClick={() => void load(true)}
+				>
+					<Refresh size={14} class={loading ? "icon spin" : undefined} />
+					Atualizar
+				</button>
+			</div>
+
+			{loading && !data && <p class="empty">Carregando comentários da PR…</p>}
+			{error && <p class="error">{error}</p>}
+			{data && !data.available && (
+				<p class="empty">
+					Comentários indisponíveis
+					{data.reason ? `: ${data.reason}` : "."}
+				</p>
+			)}
+			{empty && <p class="empty">Nenhum comentário nesta PR.</p>}
+
+			{data?.available && groups.length > 0 && (
+				<section class="comment-group">
+					<h2 class="comment-group-title">Comentários de revisão</h2>
+					{groups.map(([path, threads]) => (
+						<div class="comment-file-group" key={path}>
+							<p class="comment-group-label">
+								{path} <span class="count">{threads.length}</span>
+							</p>
+							{threads.map((thread) => (
+								<ThreadCard
+									key={thread.id}
+									thread={thread}
+									editor={editor}
+								/>
+							))}
+						</div>
+					))}
+				</section>
+			)}
+
+			{data?.available && data.issueComments.length > 0 && (
+				<section class="comment-group">
+					<h2 class="comment-group-title">Comentários gerais</h2>
+					{data.issueComments.map((comment) => (
+						<CommentCard
+							key={comment.id}
+							comment={comment}
+							editor={editor}
+							reply={false}
+						/>
+					))}
+				</section>
+			)}
+		</div>
+	);
+}