@nghyane/arcane 0.1.12 → 0.1.14

package/src/web/search/render.ts

@@ -5,63 +5,15 @@
  */
 
 import type { Component } from "@nghyane/arcane-tui";
-import { Text, visibleWidth, wrapTextWithAnsi } from "@nghyane/arcane-tui";
+import { Text } from "@nghyane/arcane-tui";
 import type { RenderResultOptions } from "../../extensibility/custom-tools/types";
-import type { Theme } from "../../modes/theme/theme";
-import {
-	formatAge,
-	formatCount,
-	formatExpandHint,
-	formatMoreItems,
-	formatStatusIcon,
-	getDomain,
-	getPreviewLines,
-	PREVIEW_LIMITS,
-	TRUNCATE_LENGTHS,
-	truncateToWidth,
-} from "../../tools/render-utils";
-import { renderStatusLine, renderTreeList } from "../../tui";
-import { CachedOutputBlock } from "../../tui/output-block";
+import type { Theme } from "../../theme/theme";
+import { renderStatusLine } from "../../tui";
+import { formatCount, formatMoreItems, getDomain, truncateToWidth } from "../../ui/render-utils";
 import { getSearchProvider } from "./provider";
 import type { SearchResponse } from "./types";
 
-const MAX_COLLAPSED_ANSWER_LINES = PREVIEW_LIMITS.COLLAPSED_LINES;
-const MAX_EXPANDED_ANSWER_LINES = PREVIEW_LIMITS.EXPANDED_LINES;
-const MAX_ANSWER_LINE_LEN = TRUNCATE_LENGTHS.LINE;
-const MAX_SNIPPET_LINES = 2;
-const MAX_SNIPPET_LINE_LEN = TRUNCATE_LENGTHS.LINE;
-const MAX_COLLAPSED_ITEMS = PREVIEW_LIMITS.COLLAPSED_ITEMS;
-const MAX_QUERY_PREVIEW = 2;
-const MAX_QUERY_LEN = 90;
-const MAX_REQUEST_ID_LEN = 36;
-
-function renderFallbackText(contentText: string, expanded: boolean, theme: Theme): Component {
-	const lines = contentText.split("\n").filter(line => line.trim());
-	const maxLines = expanded ? lines.length : 6;
-	const displayLines = lines.slice(0, maxLines).map(line => truncateToWidth(line.trim(), 110));
-	const remaining = lines.length - displayLines.length;
-
-	const headerIcon = formatStatusIcon("warning", theme);
-	const expandHint = formatExpandHint(theme, expanded, remaining > 0);
-	let text = `${headerIcon} ${theme.fg("dim", "Response")}${expandHint}`;
-
-	if (displayLines.length === 0) {
-		text += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg("muted", "No response data")}`;
-		return new Text(text, 0, 0);
-	}
-
-	for (let i = 0; i < displayLines.length; i++) {
-		const isLast = i === displayLines.length - 1 && remaining === 0;
-		const branch = isLast ? theme.tree.last : theme.tree.branch;
-		text += `\n ${theme.fg("dim", branch)} ${theme.fg("dim", displayLines[i])}`;
-	}
-
-	if (!expanded && remaining > 0) {
-		text += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg("muted", formatMoreItems(remaining, "line"))}`;
-	}
-
-	return new Text(text, 0, 0);
-}
+const MAX_COLLAPSED_SOURCES = 5;
 
 export interface SearchRenderDetails {
 	response: SearchResponse;
@@ -81,203 +33,70 @@ export function renderSearchResult(
 	},
 ): Component {
 	const details = result.details;
-
-	// Handle error case
-	if (details?.error) {
-		return new Text(theme.fg("error", `Error: ${details.error}`), 0, 0);
-	}
-
-	const rawText = result.content?.find(block => block.type === "text")?.text?.trim() ?? "";
 	const response = details?.response;
-	if (!response) {
-		return renderFallbackText(rawText, options.expanded, theme);
-	}
-
-	const sources = Array.isArray(response.sources) ? response.sources : [];
+	const sources = Array.isArray(response?.sources) ? response.sources : [];
 	const sourceCount = sources.length;
-	const citations = Array.isArray(response.citations) ? response.citations : [];
-	const citationCount = citations.length;
-	const searchQueries = Array.isArray(response.searchQueries)
+	const searchQueries = Array.isArray(response?.searchQueries)
 		? response.searchQueries.filter(item => typeof item === "string")
 		: [];
-	const provider = response.provider;
-
-	// Get answer text
-	const answerText = typeof response.answer === "string" ? response.answer.trim() : "";
-	const contentText = answerText || rawText;
-	const answerLines = contentText
-		? contentText
-				.split("\n")
-				.filter(l => l.trim())
-				.map(l => l.trim())
-		: [];
-	const totalAnswerLines = answerLines.length;
-
-	const providerLabel = provider !== "none" ? getSearchProvider(provider).label : "None";
+	const provider = response?.provider;
+
+	const providerLabel = provider
+		? provider === "none"
+			? "None"
+			: provider === "grep"
+				? "grep.app"
+				: getSearchProvider(provider).label
+		: "auto";
 	const queryPreview = args?.query
 		? truncateToWidth(args.query, 80)
 		: searchQueries[0]
 			? truncateToWidth(searchQueries[0], 80)
 			: undefined;
+
 	const header = renderStatusLine(
 		{
-			icon: sourceCount > 0 ? "success" : "warning",
+			icon: sourceCount > 0 ? "success" : details?.error ? "error" : "warning",
 			title: "Web Search",
-			description: providerLabel,
-			meta: [formatCount("source", sourceCount)],
+			description: queryPreview,
+			meta: [formatCount("source", sourceCount), providerLabel],
 		},
 		theme,
 	);
 
-	const metaLines: string[] = [];
-	metaLines.push(`${theme.fg("muted", "Provider:")} ${theme.fg("text", providerLabel)}`);
-	if (response.authMode)
-		metaLines.push(
-			`${theme.fg("muted", "Auth:")} ${theme.fg("text", response.authMode === "oauth" ? "OAuth" : response.authMode === "api_key" ? "API key" : response.authMode)}`,
-		);
-	if (response.model) metaLines.push(`${theme.fg("muted", "Model:")} ${theme.fg("text", response.model)}`);
-	metaLines.push(`${theme.fg("muted", "Sources:")} ${theme.fg("text", String(sourceCount))}`);
-	if (citationCount > 0)
-		metaLines.push(`${theme.fg("muted", "Citations:")} ${theme.fg("text", String(citationCount))}`);
-	if (response.usage) {
-		const usageParts: string[] = [];
-		if (response.usage.inputTokens !== undefined) usageParts.push(`in ${response.usage.inputTokens}`);
-		if (response.usage.outputTokens !== undefined) usageParts.push(`out ${response.usage.outputTokens}`);
-		if (response.usage.totalTokens !== undefined) usageParts.push(`total ${response.usage.totalTokens}`);
-		if (response.usage.searchRequests !== undefined) usageParts.push(`search ${response.usage.searchRequests}`);
-		if (usageParts.length > 0)
-			metaLines.push(`${theme.fg("muted", "Usage:")} ${theme.fg("text", usageParts.join(theme.sep.dot))}`);
-	}
-	if (response.requestId) {
-		metaLines.push(
-			`${theme.fg("muted", "Request:")} ${theme.fg("text", truncateToWidth(response.requestId, MAX_REQUEST_ID_LEN))}`,
-		);
-	}
-	if (searchQueries.length > 0) {
-		const queriesPreview = searchQueries.slice(0, MAX_QUERY_PREVIEW);
-		const queryList = queriesPreview.map(q => truncateToWidth(q, MAX_QUERY_LEN));
-		const suffix = searchQueries.length > queriesPreview.length ? "…" : "";
-		metaLines.push(`${theme.fg("muted", "Queries:")} ${theme.fg("text", queryList.join("; "))}${suffix}`);
-	}
+	let text = header;
 
-	const outputBlock = new CachedOutputBlock();
-
-	return {
-		render(width: number): string[] {
-			// Read mutable state at render time
-			const { expanded } = options;
+	if (details?.error) {
+		text += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg("error", details.error)}`;
+		return new Text(text, 0, 0);
+	}
 
-			// Expanded-dependent computations
-			const answerLimit = expanded ? MAX_EXPANDED_ANSWER_LINES : MAX_COLLAPSED_ANSWER_LINES;
-			const answerPreview = contentText
-				? args?.allowLongAnswer
-					? answerLines.slice(0, args.maxAnswerLines ?? answerLines.length)
-					: getPreviewLines(contentText, answerLimit, MAX_ANSWER_LINE_LEN)
-				: [];
-			const remainingAnswer = totalAnswerLines - answerPreview.length;
+	const { expanded } = options;
+	const maxItems = expanded ? sources.length : Math.min(sources.length, MAX_COLLAPSED_SOURCES);
+	const remaining = sources.length - maxItems;
 
-			const sourceTree = renderTreeList(
-				{
-					items: sources,
-					expanded,
-					maxCollapsed: MAX_COLLAPSED_ITEMS,
-					itemType: "source",
-					renderItem: src => {
-						const titleText =
-							typeof src.title === "string" && src.title.trim()
-								? src.title
-								: typeof src.url === "string" && src.url.trim()
-									? src.url
-									: "Untitled";
-						const title = truncateToWidth(titleText, 70);
-						const url = typeof src.url === "string" ? src.url : "";
-						const domain = url ? getDomain(url) : "";
-						const age =
-							formatAge(src.ageSeconds) || (typeof src.publishedDate === "string" ? src.publishedDate : "");
-						const metaParts: string[] = [];
-						if (domain) metaParts.push(theme.fg("dim", `(${domain})`));
-						if (typeof src.author === "string" && src.author.trim())
-							metaParts.push(theme.fg("muted", src.author));
-						if (age) metaParts.push(theme.fg("muted", age));
-						const metaSep = theme.fg("dim", theme.sep.dot);
-						const metaSuffix = metaParts.length > 0 ? ` ${metaParts.join(metaSep)}` : "";
-						const srcLines: string[] = [`${theme.fg("accent", title)}${metaSuffix}`];
-						const snippetText = typeof src.snippet === "string" ? src.snippet : "";
-						if (snippetText.trim()) {
-							const snippetLines = getPreviewLines(snippetText, MAX_SNIPPET_LINES, MAX_SNIPPET_LINE_LEN);
-							for (const snippetLine of snippetLines) {
-								srcLines.push(theme.fg("muted", `${theme.format.dash} ${snippetLine}`));
-							}
-						}
-						if (url) srcLines.push(theme.fg("mdLinkUrl", url));
-						return srcLines;
-					},
-				},
-				theme,
-			);
+	for (let i = 0; i < maxItems; i++) {
+		const src = sources[i];
+		const isLast = i === maxItems - 1 && remaining === 0;
+		const branch = isLast ? theme.tree.last : theme.tree.branch;
+		const titleText =
+			typeof src.title === "string" && src.title.trim()
+				? src.title
+				: typeof src.url === "string" && src.url.trim()
+					? src.url
+					: "Untitled";
+		const title = truncateToWidth(titleText, 70);
+		const url = typeof src.url === "string" ? src.url : "";
+		const domain = url ? getDomain(url) : "";
+		const domainPart = domain ? ` ${theme.fg("dim", `(${domain})`)}` : "";
+		text += `\n ${theme.fg("dim", branch)} ${theme.fg("accent", title)}${domainPart}`;
+	}
 
-			// Build answer section
-			const answerState = sourceCount > 0 ? "success" : "warning";
-			const borderColor: "warning" | "dim" = answerState === "warning" ? "warning" : "dim";
-			const border = (t: string) => theme.fg(borderColor, t);
-			const contentPrefix = border(`${theme.boxSharp.vertical} `);
-			const contentSuffix = border(theme.boxSharp.vertical);
-			const contentWidth = Math.max(0, width - visibleWidth(contentPrefix) - visibleWidth(contentSuffix));
-			const answerTreeLines = answerPreview.length > 0 ? answerPreview : ["No answer text returned"];
-			const answerTree = renderTreeList(
-				{
-					items: answerTreeLines,
-					expanded: true,
-					maxCollapsed: answerTreeLines.length,
-					itemType: "line",
-					renderItem: (line, context) => {
-						const coloredLine =
-							line === "No answer text returned" ? theme.fg("muted", line) : theme.fg("dim", line);
-						if (!args?.allowLongAnswer) {
-							return coloredLine;
-						}
-						const prefixWidth = visibleWidth(context.continuePrefix);
-						const wrapWidth = Math.max(10, contentWidth - prefixWidth);
-						return wrapTextWithAnsi(coloredLine, wrapWidth);
-					},
-				},
-				theme,
-			);
-			if (remainingAnswer > 0) {
-				answerTree.push(theme.fg("muted", formatMoreItems(remainingAnswer, "line")));
-			}
+	if (remaining > 0) {
+		text += `\n ${theme.fg("dim", theme.tree.last)} ${theme.fg("muted", formatMoreItems(remaining, "source"))}`;
+	}
 
-			return outputBlock.render(
-				{
-					header,
-					state: sourceCount > 0 ? "success" : "warning",
-					sections: [
-						...(queryPreview
-							? [
-									{
-										lines: [`${theme.fg("muted", "Query:")} ${theme.fg("text", queryPreview)}`],
-									},
-								]
-							: []),
-						{
-							label: theme.fg("toolTitle", "Answer"),
-							lines: answerTree,
-						},
-						{
-							label: theme.fg("toolTitle", "Sources"),
-							lines: sourceTree.length > 0 ? sourceTree : [theme.fg("muted", "No sources returned")],
-						},
-						{ label: theme.fg("toolTitle", "Metadata"), lines: metaLines },
-					],
-					width,
-				},
-				theme,
-			);
-		},
-		invalidate() {
-			outputBlock.invalidate();
-		},
-	};
+	return new Text(text, 0, 0);
 }
 
 /** Render web search call (query preview) */
@@ -291,9 +110,3 @@ export function renderSearchCall(
 	const text = renderStatusLine({ icon: "pending", title: "Web Search", description: query, meta: [provider] }, theme);
 	return new Text(text, 0, 0);
 }
-
-export const webSearchToolRenderer = {
-	renderCall: renderSearchCall,
-	renderResult: renderSearchResult,
-	mergeCallAndResult: true,
-};

package/src/web/search/types.ts

@@ -48,7 +48,7 @@ export interface SearchUsage {
 
 /** Unified response across providers */
 export interface SearchResponse {
-	provider: SearchProviderId | "none";
+	provider: SearchProviderId | "grep" | "none";
 	/** Synthesized answer text (anthropic, perplexity) */
 	answer?: string;
 	/** Search result sources */

package/src/commands/commit.ts

@@ -1,36 +0,0 @@
-/**
- * Generate and optionally push a commit with changelog updates.
- */
-import { Command, Flags } from "@nghyane/arcane-utils/cli";
-import { runCommitCommand } from "../commit";
-import type { CommitCommandArgs } from "../commit/types";
-import { initTheme } from "../modes/theme/theme";
-
-export default class Commit extends Command {
-	static description = "Generate a commit message and update changelogs";
-
-	static flags = {
-		push: Flags.boolean({ description: "Push after committing" }),
-		"dry-run": Flags.boolean({ description: "Preview without committing" }),
-		"no-changelog": Flags.boolean({ description: "Skip changelog updates" }),
-		legacy: Flags.boolean({ description: "Use legacy deterministic pipeline" }),
-		context: Flags.string({ char: "c", description: "Additional context for the model" }),
-		model: Flags.string({ char: "m", description: "Override model selection" }),
-	};
-
-	async run(): Promise<void> {
-		const { flags } = await this.parse(Commit);
-
-		const cmd: CommitCommandArgs = {
-			push: flags.push ?? false,
-			dryRun: flags["dry-run"] ?? false,
-			noChangelog: flags["no-changelog"] ?? false,
-			legacy: flags.legacy,
-			context: flags.context,
-			model: flags.model,
-		};
-
-		await initTheme();
-		await runCommitCommand(cmd);
-	}
-}

package/src/commit/agentic/agent.ts

@@ -1,311 +0,0 @@
-import type { Api, Model } from "@nghyane/arcane-ai";
-import { Markdown } from "@nghyane/arcane-tui";
-import chalk from "chalk";
-import type { ControlledGit } from "../../commit/git";
-import typesDescriptionPrompt from "../../commit/prompts/types-description.md" with { type: "text" };
-import type { ModelRegistry } from "../../config/model-registry";
-import { renderPromptTemplate } from "../../config/prompt-templates";
-import type { Settings } from "../../config/settings";
-import { getMarkdownTheme } from "../../modes/theme/theme";
-import { createAgentSession } from "../../sdk";
-import type { AgentSessionEvent } from "../../session/agent-session";
-import type { AuthStorage } from "../../session/auth-storage";
-import agentUserPrompt from "./prompts/session-user.md" with { type: "text" };
-import agentSystemPrompt from "./prompts/system.md" with { type: "text" };
-import type { CommitAgentState } from "./state";
-import { createCommitTools } from "./tools";
-
-export interface CommitAgentInput {
-	cwd: string;
-	git: ControlledGit;
-	model: Model<Api>;
-	settings: Settings;
-	modelRegistry: ModelRegistry;
-	authStorage: AuthStorage;
-	userContext?: string;
-	contextFiles?: Array<{ path: string; content: string }>;
-	changelogTargets: string[];
-	requireChangelog: boolean;
-	diffText?: string;
-	existingChangelogEntries?: ExistingChangelogEntries[];
-}
-
-export interface ExistingChangelogEntries {
-	path: string;
-	sections: Array<{ name: string; items: string[] }>;
-}
-
-export async function runCommitAgentSession(input: CommitAgentInput): Promise<CommitAgentState> {
-	const typesDescription = renderPromptTemplate(typesDescriptionPrompt);
-	const systemPrompt = renderPromptTemplate(agentSystemPrompt, {
-		types_description: typesDescription,
-	});
-	const state: CommitAgentState = { diffText: input.diffText };
-	const spawns = "quick_task";
-	const tools = createCommitTools({
-		cwd: input.cwd,
-		git: input.git,
-		authStorage: input.authStorage,
-		modelRegistry: input.modelRegistry,
-		settings: input.settings,
-		spawns,
-		state,
-		changelogTargets: input.changelogTargets,
-		enableAnalyzeFiles: true,
-	});
-
-	const { session } = await createAgentSession({
-		cwd: input.cwd,
-		authStorage: input.authStorage,
-		modelRegistry: input.modelRegistry,
-		settings: input.settings,
-		model: input.model,
-		systemPrompt,
-		customTools: tools,
-		enableLsp: false,
-		enableMCP: false,
-		hasUI: false,
-		spawns,
-		toolNames: ["__none__"],
-		contextFiles: input.contextFiles,
-		disableExtensionDiscovery: true,
-		skills: [],
-		promptTemplates: [],
-		slashCommands: [],
-	});
-	let toolCalls = 0;
-	let messageCount = 0;
-	let isThinking = false;
-	let thinkingLineActive = false;
-	const toolArgsById = new Map<string, { name: string; args?: Record<string, unknown> }>();
-	const writeThinkingLine = (text: string) => {
-		const line = chalk.dim(`… ${text}`);
-		process.stdout.write(`\r\x1b[2K${line}`);
-		thinkingLineActive = true;
-	};
-	const clearThinkingLine = () => {
-		if (!thinkingLineActive) return;
-		process.stdout.write("\r\x1b[2K");
-		thinkingLineActive = false;
-	};
-	const unsubscribe = session.subscribe((event: AgentSessionEvent) => {
-		switch (event.type) {
-			case "message_start":
-				if (event.message.role === "assistant") {
-					isThinking = true;
-					thinkingLineActive = false;
-				}
-				break;
-			case "message_update": {
-				if (event.message?.role !== "assistant") break;
-				const preview = extractMessagePreview(event.message?.content ?? []);
-				if (!preview) break;
-				writeThinkingLine(preview);
-				break;
-			}
-			case "tool_execution_start":
-				toolCalls += 1;
-				toolArgsById.set(event.toolCallId, { name: event.toolName, args: event.args });
-				break;
-			case "message_end": {
-				const role = event.message?.role;
-				if (role === "assistant") {
-					messageCount += 1;
-					isThinking = false;
-					clearThinkingLine();
-					const assistantMessage = event.message as { stopReason?: string; errorMessage?: string };
-					if (assistantMessage.stopReason === "error" && assistantMessage.errorMessage) {
-						writeStdout(`● Error: ${assistantMessage.errorMessage}`);
-					}
-					const messageText = extractMessageText(event.message?.content ?? []);
-					if (messageText) {
-						writeAssistantMessage(messageText);
-					}
-				}
-				break;
-			}
-			case "tool_execution_end": {
-				const stored = toolArgsById.get(event.toolCallId) ?? { name: event.toolName };
-				toolArgsById.delete(event.toolCallId);
-				clearThinkingLine();
-				const toolLabel = formatToolLabel(stored.name);
-				const symbol = event.isError ? "" : "";
-				writeStdout(`${symbol} ${toolLabel}`);
-				const argsLines = formatToolArgs(stored.args);
-				if (argsLines.length > 0) {
-					writeStdout(formatToolArgsBlock(argsLines));
-				}
-				break;
-			}
-			case "agent_end":
-				if (isThinking) {
-					isThinking = false;
-				}
-				writeStdout(`● agent finished (${messageCount} messages, ${toolCalls} tools)`);
-				break;
-			default:
-				break;
-		}
-	});
-
-	try {
-		const prompt = renderPromptTemplate(agentUserPrompt, {
-			user_context: input.userContext,
-			changelog_targets: input.changelogTargets.length > 0 ? input.changelogTargets.join("\n") : undefined,
-			existing_changelog_entries: input.existingChangelogEntries,
-		});
-		const MAX_RETRIES = 3;
-		let retryCount = 0;
-		const needsChangelog = input.requireChangelog && input.changelogTargets.length > 0;
-
-		await session.prompt(prompt, { expandPromptTemplates: false });
-		while (retryCount < MAX_RETRIES && !isProposalComplete(state, needsChangelog)) {
-			retryCount += 1;
-			const reminder = buildReminderMessage(state, needsChangelog, retryCount, MAX_RETRIES);
-			await session.prompt(reminder, { expandPromptTemplates: false });
-		}
-
-		return state;
-	} finally {
-		unsubscribe();
-		await session.dispose();
-	}
-}
-
-function writeStdout(message: string): void {
-	process.stdout.write(`${message}\n`);
-}
-
-function extractMessagePreview(content: Array<{ type: string; text?: string }>): string | null {
-	const textBlocks = content
-		.filter(block => block.type === "text" && typeof block.text === "string")
-		.map(block => block.text?.trim())
-		.filter((value): value is string => Boolean(value));
-	if (textBlocks.length === 0) return null;
-	const combined = textBlocks.join(" ").replace(/\s+/g, " ").trim();
-	return truncateToolArg(combined);
-}
-
-function extractMessageText(content: Array<{ type: string; text?: string }>): string | null {
-	const textBlocks = content
-		.filter(block => block.type === "text" && typeof block.text === "string")
-		.map(block => block.text ?? "")
-		.filter(value => value.trim().length > 0);
-	if (textBlocks.length === 0) return null;
-	return textBlocks.join("\n").trim();
-}
-
-function writeAssistantMessage(message: string): void {
-	const lines = renderMarkdownLines(message);
-	if (lines.length === 0) return;
-	let firstContentIndex = lines.findIndex(line => line.trim().length > 0);
-	if (firstContentIndex === -1) {
-		firstContentIndex = 0;
-	}
-	for (const [index, line] of lines.entries()) {
-		const prefix = index === firstContentIndex ? "● " : "  ";
-		writeStdout(`${prefix}${line}`.trimEnd());
-	}
-}
-
-function renderMarkdownLines(message: string): string[] {
-	const width = Math.max(40, process.stdout.columns ?? 100);
-	const markdown = new Markdown(message, 0, 0, getMarkdownTheme());
-	return markdown.render(width);
-}
-
-function formatToolLabel(toolName: string): string {
-	const displayName = toolName
-		.split(/[_-]/)
-		.map(segment => segment.charAt(0).toUpperCase() + segment.slice(1))
-		.join("");
-	return displayName;
-}
-
-function formatToolArgs(args?: Record<string, unknown>): string[] {
-	if (!args || Object.keys(args).length === 0) return [];
-	const lines: string[] = [];
-	const visit = (value: unknown, keyPath: string) => {
-		if (value === null || value === undefined) return;
-		if (Array.isArray(value)) {
-			if (value.length === 0) return;
-			const rendered = value.map(item => renderPrimitive(item)).filter(Boolean);
-			if (rendered.length > 0) {
-				lines.push(`${keyPath}: ${rendered.join(", ")}`);
-			}
-			return;
-		}
-		if (typeof value === "object") {
-			const entries = Object.entries(value as Record<string, unknown>);
-			if (entries.length === 0) return;
-			for (const [childKey, childValue] of entries) {
-				visit(childValue, `${keyPath}.${childKey}`);
-			}
-			return;
-		}
-		const rendered = renderPrimitive(value);
-		if (rendered) {
-			lines.push(`${keyPath}: ${rendered}`);
-		}
-	};
-	for (const [key, value] of Object.entries(args)) {
-		visit(value, key);
-	}
-	return lines;
-}
-
-function renderPrimitive(value: unknown): string | null {
-	if (value === null || value === undefined) return null;
-	if (typeof value === "string") {
-		const trimmed = value.trim();
-		return trimmed.length > 0 ? trimmed : null;
-	}
-	if (typeof value === "number" || typeof value === "boolean") {
-		return String(value);
-	}
-	return null;
-}
-
-function formatToolArgsBlock(lines: string[]): string {
-	return lines
-		.map((line, index) => {
-			if (index === 0) return `  ⎿ ${line}`;
-			const branch = index === lines.length - 1 ? "└" : "├";
-			return `    ${branch} ${line}`;
-		})
-		.join("\n");
-}
-
-function isProposalComplete(state: CommitAgentState, requireChangelog: boolean): boolean {
-	const hasCommit = Boolean(state.proposal ?? state.splitProposal);
-	const hasChangelog = !requireChangelog || Boolean(state.changelogProposal);
-	return hasCommit && hasChangelog;
-}
-
-function buildReminderMessage(
-	state: CommitAgentState,
-	requireChangelog: boolean,
-	retryCount: number,
-	maxRetries: number,
-): string {
-	const missing: string[] = [];
-	if (!state.proposal && !state.splitProposal) {
-		missing.push("commit proposal (propose_commit or split_commit)");
-	}
-	if (requireChangelog && !state.changelogProposal) {
-		missing.push("changelog entries (propose_changelog)");
-	}
-	return `<system-reminder>
-CRITICAL: You must call the required tools before finishing.
-
-Missing: ${missing.join(", ") || "none"}.
-Reminder ${retryCount} of ${maxRetries}.
-
-Call the missing tool(s) now.
-</system-reminder>`;
-}
-
-function truncateToolArg(value: string): string {
-	if (value.length <= 40) return value;
-	return `${value.slice(0, 39)}…`;
-}