@oh-my-pi/pi-coding-agent 12.18.3 → 12.19.2

package/src/commit/agentic/agent.ts

@@ -115,7 +115,7 @@ export async function runCommitAgentSession(input: CommitAgentInput): Promise<Co
 					clearThinkingLine();
 					const assistantMessage = event.message as { stopReason?: string; errorMessage?: string };
 					if (assistantMessage.stopReason === "error" && assistantMessage.errorMessage) {
-						writeStdout(`● Error: ${assistantMessage.errorMessage}`);
+						process.stdout.write(`● Error: ${assistantMessage.errorMessage}\n`);
 					}
 					const messageText = extractMessageText(event.message?.content ?? []);
 					if (messageText) {
@@ -130,10 +130,10 @@ export async function runCommitAgentSession(input: CommitAgentInput): Promise<Co
 				clearThinkingLine();
 				const toolLabel = formatToolLabel(stored.name);
 				const symbol = event.isError ? "" : "";
-				writeStdout(`${symbol} ${toolLabel}`);
+				process.stdout.write(`${symbol} ${toolLabel}\n`);
 				const argsLines = formatToolArgs(stored.args);
 				if (argsLines.length > 0) {
-					writeStdout(formatToolArgsBlock(argsLines));
+					process.stdout.write(`${formatToolArgsBlock(argsLines)}\n`);
 				}
 				break;
 			}
@@ -141,7 +141,7 @@ export async function runCommitAgentSession(input: CommitAgentInput): Promise<Co
 				if (isThinking) {
 					isThinking = false;
 				}
-				writeStdout(`● agent finished (${messageCount} messages, ${toolCalls} tools)`);
+				process.stdout.write(`● agent finished (${messageCount} messages, ${toolCalls} tools)\n`);
 				break;
 			default:
 				break;
@@ -172,10 +172,6 @@ export async function runCommitAgentSession(input: CommitAgentInput): Promise<Co
 	}
 }
 
-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")
@@ -204,7 +200,7 @@ function writeAssistantMessage(message: string): void {
 	}
 	for (const [index, line] of lines.entries()) {
 		const prefix = index === firstContentIndex ? "● " : "  ";
-		writeStdout(`${prefix}${line}`.trimEnd());
+		process.stdout.write(`${`${prefix}${line}`.trimEnd()}\n`);
 	}
 }
 
@@ -249,6 +245,7 @@ function formatToolArgs(args?: Record<string, unknown>): string[] {
 		}
 	};
 	for (const [key, value] of Object.entries(args)) {
+		if (key === "agent__intent") continue;
 		visit(value, key);
 	}
 	return lines;

package/src/commit/agentic/index.ts

@@ -1,6 +1,6 @@
 import * as path from "node:path";
 import { createInterface } from "node:readline/promises";
-import { $env } from "@oh-my-pi/pi-utils";
+import { $env, isEnoent } from "@oh-my-pi/pi-utils";
 import { getProjectDir } from "@oh-my-pi/pi-utils/dirs";
 import { applyChangelogProposals } from "../../commit/changelog";
 import { detectChangelogBoundaries } from "../../commit/changelog/detect";
@@ -31,13 +31,13 @@ export async function runAgenticCommit(args: CommitCommandArgs): Promise<void> {
 	const git = new ControlledGit(cwd);
 	const [settings, authStorage] = await Promise.all([Settings.init({ cwd }), discoverAuthStorage()]);
 
-	writeStdout("● Resolving model...");
+	process.stdout.write("● Resolving model...\n");
 	const modelRegistry = new ModelRegistry(authStorage);
 	await modelRegistry.refresh();
 	const stagedFilesPromise = (async () => {
 		let stagedFiles = await git.getStagedFiles();
 		if (stagedFiles.length === 0) {
-			writeStdout("No staged changes detected, staging all changes...");
+			process.stdout.write("No staged changes detected, staging all changes...\n");
 			await git.stageAll();
 			stagedFiles = await git.getStagedFiles();
 		}
@@ -47,17 +47,17 @@ export async function runAgenticCommit(args: CommitCommandArgs): Promise<void> {
 	const primaryModelPromise = resolvePrimaryModel(args.model, settings, modelRegistry);
 	const [primaryModelResult, stagedFiles] = await Promise.all([primaryModelPromise, stagedFilesPromise]);
 	const { model: primaryModel, apiKey: primaryApiKey } = primaryModelResult;
-	writeStdout(`  └─ ${primaryModel.name}`);
+	process.stdout.write(`  └─ ${primaryModel.name}\n`);
 
 	const { model: agentModel } = await resolveSmolModel(settings, modelRegistry, primaryModel, primaryApiKey);
 
 	if (stagedFiles.length === 0) {
-		writeStderr("No changes to commit.");
+		process.stderr.write("No changes to commit.\n");
 		return;
 	}
 
 	if (!args.noChangelog) {
-		writeStdout("● Detecting changelog targets...");
+		process.stdout.write("● Detecting changelog targets...\n");
 	}
 	const [changelogBoundaries, contextFiles, numstat, diff] = await Promise.all([
 		args.noChangelog ? [] : detectChangelogBoundaries(cwd, stagedFiles),
@@ -69,25 +69,25 @@ export async function runAgenticCommit(args: CommitCommandArgs): Promise<void> {
 	if (!args.noChangelog) {
 		if (changelogTargets.length > 0) {
 			for (const path of changelogTargets) {
-				writeStdout(`  └─ ${path}`);
+				process.stdout.write(`  └─ ${path}\n`);
 			}
 		} else {
-			writeStdout("  └─ (none found)");
+			process.stdout.write("  └─ (none found)\n");
 		}
 	}
 
-	writeStdout("● Discovering context files...");
+	process.stdout.write("● Discovering context files...\n");
 	const agentsMdFiles = contextFiles.filter(file => file.path.endsWith("AGENTS.md"));
 	if (agentsMdFiles.length > 0) {
 		for (const file of agentsMdFiles) {
-			writeStdout(`  └─ ${file.path}`);
+			process.stdout.write(`  └─ ${file.path}\n`);
 		}
 	} else {
-		writeStdout("  └─ (none found)");
+		process.stdout.write("  └─ (none found)\n");
 	}
 	const forceFallback = $env.PI_COMMIT_TEST_FALLBACK?.toLowerCase() === "true";
 	if (forceFallback) {
-		writeStdout("● Forcing fallback commit generation...");
+		process.stdout.write("● Forcing fallback commit generation...\n");
 		const fallbackProposal = generateFallbackProposal(numstat);
 		await runSingleCommit(fallbackProposal, { git, dryRun: args.dryRun, push: args.push });
 		return;
@@ -95,7 +95,7 @@ export async function runAgenticCommit(args: CommitCommandArgs): Promise<void> {
 
 	const trivialChange = detectTrivialChange(diff);
 	if (trivialChange) {
-		writeStdout(`● Detected trivial change: ${trivialChange.summary}`);
+		process.stdout.write(`● Detected trivial change: ${trivialChange.summary}\n`);
 		const trivialProposal: CommitProposal = {
 			analysis: {
 				type: trivialChange.type,
@@ -118,7 +118,7 @@ export async function runAgenticCommit(args: CommitCommandArgs): Promise<void> {
 		}
 	}
 
-	writeStdout("● Starting commit agent...");
+	process.stdout.write("● Starting commit agent...\n");
 	let commitState: CommitAgentState;
 	let usedFallback = false;
 
@@ -139,18 +139,18 @@ export async function runAgenticCommit(args: CommitCommandArgs): Promise<void> {
 		});
 	} catch (error) {
 		const errorMessage = error instanceof Error ? error.message : String(error);
-		writeStderr(`Agent error: ${errorMessage}`);
+		process.stderr.write(`Agent error: ${errorMessage}\n`);
 		if (error instanceof Error && error.stack && $env.DEBUG) {
-			writeStderr(error.stack);
+			process.stderr.write(`${error.stack}\n`);
 		}
-		writeStdout("● Using fallback commit generation...");
+		process.stdout.write("● Using fallback commit generation...\n");
 		commitState = { proposal: generateFallbackProposal(numstat) };
 		usedFallback = true;
 	}
 
 	if (!usedFallback && !commitState.proposal && !commitState.splitProposal) {
 		if ($env.PI_COMMIT_NO_FALLBACK?.toLowerCase() !== "true") {
-			writeStdout("● Agent did not provide proposal, using fallback...");
+			process.stdout.write("● Agent did not provide proposal, using fallback...\n");
 			commitState.proposal = generateFallbackProposal(numstat);
 			usedFallback = true;
 		}
@@ -159,26 +159,26 @@ export async function runAgenticCommit(args: CommitCommandArgs): Promise<void> {
 	let updatedChangelogFiles: string[] = [];
 	if (!args.noChangelog && changelogTargets.length > 0 && !usedFallback) {
 		if (!commitState.changelogProposal) {
-			writeStderr("Commit agent did not provide changelog entries.");
+			process.stderr.write("Commit agent did not provide changelog entries.\n");
 			return;
 		}
-		writeStdout("● Applying changelog entries...");
+		process.stdout.write("● Applying changelog entries...\n");
 		const updated = await applyChangelogProposals({
 			git,
 			cwd,
 			proposals: commitState.changelogProposal.entries,
 			dryRun: args.dryRun,
 			onProgress: message => {
-				writeStdout(`  ├─ ${message}`);
+				process.stdout.write(`  ├─ ${message}\n`);
 			},
 		});
 		updatedChangelogFiles = updated.map(filePath => path.relative(cwd, filePath));
 		if (updated.length > 0) {
 			for (const filePath of updated) {
-				writeStdout(`  └─ ${filePath}`);
+				process.stdout.write(`  └─ ${filePath}\n`);
 			}
 		} else {
-			writeStdout("  └─ (no changes)");
+			process.stdout.write("  └─ (no changes)\n");
 		}
 	}
 
@@ -197,24 +197,24 @@ export async function runAgenticCommit(args: CommitCommandArgs): Promise<void> {
 		return;
 	}
 
-	writeStderr("Commit agent did not provide a proposal.");
+	process.stderr.write("Commit agent did not provide a proposal.\n");
 }
 
 async function runSingleCommit(proposal: CommitProposal, ctx: CommitExecutionContext): Promise<void> {
 	if (proposal.warnings.length > 0) {
-		writeStdout(formatWarnings(proposal.warnings));
+		process.stdout.write(formatWarnings(proposal.warnings));
 	}
 	const commitMessage = formatCommitMessage(proposal.analysis, proposal.summary);
 	if (ctx.dryRun) {
-		writeStdout("\nGenerated commit message:\n");
-		writeStdout(commitMessage);
+		process.stdout.write("\nGenerated commit message:\n");
+		process.stdout.write(`${commitMessage}\n`);
 		return;
 	}
 	await ctx.git.commit(commitMessage);
-	writeStdout("Commit created.");
+	process.stdout.write("Commit created.\n");
 	if (ctx.push) {
 		await ctx.git.push();
-		writeStdout("Pushed to remote.");
+		process.stdout.write("Pushed to remote.\n");
 	}
 }
 
@@ -223,7 +223,7 @@ async function runSplitCommit(
 	ctx: CommitExecutionContext & { additionalFiles?: string[] },
 ): Promise<void> {
 	if (plan.warnings.length > 0) {
-		writeStdout(formatWarnings(plan.warnings));
+		process.stdout.write(formatWarnings(plan.warnings));
 	}
 	if (ctx.additionalFiles && ctx.additionalFiles.length > 0) {
 		appendFilesToLastCommit(plan, ctx.additionalFiles);
@@ -232,12 +232,12 @@ async function runSplitCommit(
 	const plannedFiles = new Set(plan.commits.flatMap(commit => commit.changes.map(change => change.path)));
 	const missingFiles = stagedFiles.filter(file => !plannedFiles.has(file));
 	if (missingFiles.length > 0) {
-		writeStderr(`Split commit plan missing staged files: ${missingFiles.join(", ")}`);
+		process.stderr.write(`Split commit plan missing staged files: ${missingFiles.join(", ")}\n`);
 		return;
 	}
 
 	if (ctx.dryRun) {
-		writeStdout("\nSplit commit plan (dry run):\n");
+		process.stdout.write("\nSplit commit plan (dry run):\n");
 		for (const [index, commit] of plan.commits.entries()) {
 			const analysis: ConventionalAnalysis = {
 				type: commit.type,
@@ -246,17 +246,17 @@ async function runSplitCommit(
 				issueRefs: commit.issueRefs,
 			};
 			const message = formatCommitMessage(analysis, commit.summary);
-			writeStdout(`Commit ${index + 1}:\n${message}\n`);
+			process.stdout.write(`Commit ${index + 1}:\n${message}\n`);
 			const changeSummary = commit.changes
 				.map(change => formatFileChangeSummary(change.path, change.hunks))
 				.join(", ");
-			writeStdout(`Changes: ${changeSummary}\n`);
+			process.stdout.write(`Changes: ${changeSummary}\n`);
 		}
 		return;
 	}
 
 	if (!(await confirmSplitCommitPlan(plan))) {
-		writeStdout("Split commit aborted by user.");
+		process.stdout.write("Split commit aborted by user.\n");
 		return;
 	}
 
@@ -279,10 +279,10 @@ async function runSplitCommit(
 		await ctx.git.commit(message);
 		await ctx.git.resetStaging();
 	}
-	writeStdout("Split commits created.");
+	process.stdout.write("Split commits created.\n");
 	if (ctx.push) {
 		await ctx.git.push();
-		writeStdout("Pushed to remote.");
+		process.stdout.write("Pushed to remote.\n");
 	}
 }
 
@@ -312,15 +312,7 @@ async function confirmSplitCommitPlan(plan: SplitCommitPlan): Promise<boolean> {
 }
 
 function formatWarnings(warnings: string[]): string {
-	return `Warnings:\n${warnings.map(warning => `- ${warning}`).join("\n")}`;
-}
-
-function writeStdout(message: string): void {
-	process.stdout.write(`${message}\n`);
-}
-
-function writeStderr(message: string): void {
-	process.stderr.write(`${message}\n`);
+	return `Warnings:\n${warnings.map(warning => `- ${warning}`).join("\n")}\n`;
 }
 
 function formatFileChangeSummary(path: string, hunks: HunkSelector): string {
@@ -336,11 +328,13 @@ function formatFileChangeSummary(path: string, hunks: HunkSelector): string {
 async function loadExistingChangelogEntries(paths: string[]): Promise<ExistingChangelogEntries[]> {
 	const entries = await Promise.all(
 		paths.map(async path => {
-			const file = Bun.file(path);
-			if (!(await file.exists())) {
-				return null;
+			let content: string;
+			try {
+				content = await Bun.file(path).text();
+			} catch (err) {
+				if (isEnoent(err)) return null;
+				throw err;
 			}
-			const content = await file.text();
 			try {
 				const unreleased = parseUnreleasedSection(content);
 				const sections = Object.entries(unreleased.entries)

package/src/commit/agentic/state.ts

@@ -16,15 +16,6 @@ export interface CommitProposal {
 	warnings: string[];
 }
 
-export interface FileObservation {
-	file: string;
-	summary: string;
-	highlights: string[];
-	risks: string[];
-	additions: number;
-	deletions: number;
-}
-
 export type HunkSelector =
 	| { type: "all" }
 	| { type: "indices"; indices: number[] }

package/src/commit/agentic/tools/propose-commit.ts

@@ -12,36 +12,7 @@ import { validateAnalysis } from "../../../commit/analysis/validation";
 import type { ControlledGit } from "../../../commit/git";
 import type { CommitType, ConventionalAnalysis, ConventionalDetail } from "../../../commit/types";
 import type { CustomTool } from "../../../extensibility/custom-tools/types";
-
-const commitTypeSchema = Type.Union([
-	Type.Literal("feat"),
-	Type.Literal("fix"),
-	Type.Literal("refactor"),
-	Type.Literal("perf"),
-	Type.Literal("docs"),
-	Type.Literal("test"),
-	Type.Literal("build"),
-	Type.Literal("ci"),
-	Type.Literal("chore"),
-	Type.Literal("style"),
-	Type.Literal("revert"),
-]);
-
-const detailSchema = Type.Object({
-	text: Type.String(),
-	changelog_category: Type.Optional(
-		Type.Union([
-			Type.Literal("Added"),
-			Type.Literal("Changed"),
-			Type.Literal("Fixed"),
-			Type.Literal("Deprecated"),
-			Type.Literal("Removed"),
-			Type.Literal("Security"),
-			Type.Literal("Breaking Changes"),
-		]),
-	),
-	user_visible: Type.Optional(Type.Boolean()),
-});
+import { commitTypeSchema, detailSchema } from "./schemas.js";
 
 const proposeCommitSchema = Type.Object({
 	type: commitTypeSchema,

package/src/commit/agentic/tools/schemas.ts

@@ -0,0 +1,31 @@
+import { Type } from "@sinclair/typebox";
+
+export const commitTypeSchema = Type.Union([
+	Type.Literal("feat"),
+	Type.Literal("fix"),
+	Type.Literal("refactor"),
+	Type.Literal("perf"),
+	Type.Literal("docs"),
+	Type.Literal("test"),
+	Type.Literal("build"),
+	Type.Literal("ci"),
+	Type.Literal("chore"),
+	Type.Literal("style"),
+	Type.Literal("revert"),
+]);
+
+export const detailSchema = Type.Object({
+	text: Type.String(),
+	changelog_category: Type.Optional(
+		Type.Union([
+			Type.Literal("Added"),
+			Type.Literal("Changed"),
+			Type.Literal("Fixed"),
+			Type.Literal("Deprecated"),
+			Type.Literal("Removed"),
+			Type.Literal("Security"),
+			Type.Literal("Breaking Changes"),
+		]),
+	),
+	user_visible: Type.Optional(Type.Boolean()),
+});

package/src/commit/agentic/tools/split-commit.ts

@@ -13,36 +13,7 @@ import { validateScope } from "../../../commit/analysis/validation";
 import type { ControlledGit } from "../../../commit/git";
 import type { ConventionalDetail } from "../../../commit/types";
 import type { CustomTool } from "../../../extensibility/custom-tools/types";
-
-const commitTypeSchema = Type.Union([
-	Type.Literal("feat"),
-	Type.Literal("fix"),
-	Type.Literal("refactor"),
-	Type.Literal("perf"),
-	Type.Literal("docs"),
-	Type.Literal("test"),
-	Type.Literal("build"),
-	Type.Literal("ci"),
-	Type.Literal("chore"),
-	Type.Literal("style"),
-	Type.Literal("revert"),
-]);
-
-const detailSchema = Type.Object({
-	text: Type.String(),
-	changelog_category: Type.Optional(
-		Type.Union([
-			Type.Literal("Added"),
-			Type.Literal("Changed"),
-			Type.Literal("Fixed"),
-			Type.Literal("Deprecated"),
-			Type.Literal("Removed"),
-			Type.Literal("Security"),
-			Type.Literal("Breaking Changes"),
-		]),
-	),
-	user_visible: Type.Optional(Type.Boolean()),
-});
+import { commitTypeSchema, detailSchema } from "./schemas.js";
 
 const hunkSelectorSchema = Type.Union([
 	Type.Object({ type: Type.Literal("all") }),

package/src/commit/agentic/validation.ts

@@ -1,6 +1,7 @@
 import { stripTypePrefix } from "../../commit/analysis/summary";
 import { validateSummary } from "../../commit/analysis/validation";
 import type { CommitType, ConventionalDetail } from "../../commit/types";
+import { normalizeUnicode } from "../../patch/normalize";
 
 export const SUMMARY_MAX_CHARS = 72;
 export const MAX_DETAIL_ITEMS = 6;
@@ -61,29 +62,11 @@ const pastTenseVerbs = new Set([
 ]);
 const pastTenseEdExceptions = new Set(["hundred", "red", "bed"]);
 
-const unicodeReplacements: Array<[RegExp, string]> = [
-	[/[\u2018\u2019]/g, "'"],
-	[/[\u201C\u201D]/g, '"'],
-	[/[\u2013\u2014\u2212]/g, "-"],
-	[/\u2260/g, "!="],
-	[/\u00BD/g, "1/2"],
-	[/\u03BB/g, "lambda"],
-	[/[\u200B-\u200D\uFEFF]/g, ""],
-];
-
 export function normalizeSummary(summary: string, type: CommitType, scope: string | null): string {
 	const stripped = stripTypePrefix(summary, type, scope);
 	return normalizeUnicode(stripped).replace(/\s+/g, " ").trim();
 }
 
-export function normalizeUnicode(text: string): string {
-	let result = text;
-	for (const [pattern, replacement] of unicodeReplacements) {
-		result = result.replace(pattern, replacement);
-	}
-	return result.normalize("NFC");
-}
-
 export function validateSummaryRules(summary: string): { errors: string[]; warnings: string[] } {
 	const errors: string[] = [];
 	const warnings: string[] = [];

package/src/commit/analysis/conventional.ts

@@ -1,10 +1,11 @@
-import type { Api, AssistantMessage, Model, ToolCall } from "@oh-my-pi/pi-ai";
+import type { Api, AssistantMessage, Model } from "@oh-my-pi/pi-ai";
 import { completeSimple, validateToolCall } from "@oh-my-pi/pi-ai";
 import { Type } from "@sinclair/typebox";
 import analysisSystemPrompt from "../../commit/prompts/analysis-system.md" with { type: "text" };
 import analysisUserPrompt from "../../commit/prompts/analysis-user.md" with { type: "text" };
-import type { ChangelogCategory, ConventionalAnalysis, ConventionalDetail } from "../../commit/types";
+import type { ChangelogCategory, ConventionalAnalysis } from "../../commit/types";
 import { renderPromptTemplate } from "../../config/prompt-templates";
+import { extractTextContent, extractToolCall, normalizeAnalysis, parseJsonPayload } from "../utils";
 
 const ConventionalAnalysisTool = {
 	name: "create_conventional_analysis",
@@ -115,51 +116,3 @@ function parseAnalysisFromResponse(message: AssistantMessage): ConventionalAnaly
 	};
 	return normalizeAnalysis(parsed);
 }
-
-function normalizeAnalysis(parsed: {
-	type: ConventionalAnalysis["type"];
-	scope: string | null;
-	details: Array<{ text: string; changelog_category?: ChangelogCategory; user_visible?: boolean }>;
-	issue_refs: string[];
-}): ConventionalAnalysis {
-	const details: ConventionalDetail[] = parsed.details.map(detail => ({
-		text: detail.text.trim(),
-		changelogCategory: detail.user_visible ? detail.changelog_category : undefined,
-		userVisible: detail.user_visible ?? false,
-	}));
-	return {
-		type: parsed.type,
-		scope: parsed.scope?.trim() || null,
-		details,
-		issueRefs: parsed.issue_refs ?? [],
-	};
-}
-
-function extractToolCall(message: AssistantMessage, name: string): ToolCall | undefined {
-	for (const content of message.content) {
-		if (content.type === "toolCall" && content.name === name) {
-			return content;
-		}
-	}
-	return undefined;
-}
-
-function extractTextContent(message: AssistantMessage): string {
-	return message.content
-		.filter(content => content.type === "text")
-		.map(content => content.text)
-		.join("")
-		.trim();
-}
-
-function parseJsonPayload(text: string): unknown {
-	const trimmed = text.trim();
-	if (trimmed.startsWith("{") && trimmed.endsWith("}")) {
-		return JSON.parse(trimmed) as unknown;
-	}
-	const match = trimmed.match(/\{[\s\S]*\}/);
-	if (!match) {
-		throw new Error("No JSON payload found in analysis response");
-	}
-	return JSON.parse(match[0]) as unknown;
-}

package/src/commit/analysis/summary.ts

@@ -1,10 +1,11 @@
-import type { Api, AssistantMessage, Model, ToolCall } from "@oh-my-pi/pi-ai";
+import type { Api, AssistantMessage, Model } from "@oh-my-pi/pi-ai";
 import { completeSimple, validateToolCall } from "@oh-my-pi/pi-ai";
 import { Type } from "@sinclair/typebox";
 import summarySystemPrompt from "../../commit/prompts/summary-system.md" with { type: "text" };
 import summaryUserPrompt from "../../commit/prompts/summary-user.md" with { type: "text" };
 import type { CommitSummary } from "../../commit/types";
 import { renderPromptTemplate } from "../../config/prompt-templates";
+import { extractTextContent, extractToolCall } from "../utils";
 
 const SummaryTool = {
 	name: "create_commit_summary",
@@ -85,18 +86,6 @@ function parseSummaryFromResponse(message: AssistantMessage, commitType: string,
 	return { summary: stripTypePrefix(text, commitType, scope) };
 }
 
-function extractToolCall(message: AssistantMessage, name: string): ToolCall | undefined {
-	return message.content.find(content => content.type === "toolCall" && content.name === name) as ToolCall | undefined;
-}
-
-function extractTextContent(message: AssistantMessage): string {
-	return message.content
-		.filter(content => content.type === "text")
-		.map(content => content.text)
-		.join("")
-		.trim();
-}
-
 export function stripTypePrefix(summary: string, commitType: string, scope: string | null): string {
 	const trimmed = summary.trim();
 	const scopePart = scope ? `(${scope})` : "";

package/src/commit/changelog/detect.ts

@@ -26,8 +26,11 @@ async function findNearestChangelog(cwd: string, filePath: string): Promise<stri
 	const root = path.resolve(cwd);
 	while (true) {
 		const candidate = path.resolve(current, CHANGELOG_NAME);
-		if (fs.existsSync(candidate)) {
+		try {
+			await fs.promises.access(candidate);
 			return candidate;
+		} catch {
+			// not found, continue traversal
 		}
 		if (current === root) return null;
 		const parent = path.dirname(current);

package/src/commit/changelog/generate.ts

@@ -1,10 +1,11 @@
-import type { Api, AssistantMessage, Model, ToolCall } from "@oh-my-pi/pi-ai";
+import type { Api, AssistantMessage, Model } from "@oh-my-pi/pi-ai";
 import { completeSimple, validateToolCall } from "@oh-my-pi/pi-ai";
 import { Type } from "@sinclair/typebox";
 import changelogSystemPrompt from "../../commit/prompts/changelog-system.md" with { type: "text" };
 import changelogUserPrompt from "../../commit/prompts/changelog-user.md" with { type: "text" };
 import type { ChangelogGenerationResult } from "../../commit/types";
 import { renderPromptTemplate } from "../../config/prompt-templates";
+import { extractTextContent, extractToolCall, parseJsonPayload } from "../utils";
 
 const ChangelogTool = {
 	name: "create_changelog_entries",
@@ -66,30 +67,6 @@ function parseChangelogResponse(message: AssistantMessage): ChangelogGenerationR
 	return { entries: parsed.entries ?? {} };
 }
 
-function extractToolCall(message: AssistantMessage, name: string): ToolCall | undefined {
-	return message.content.find(content => content.type === "toolCall" && content.name === name) as ToolCall | undefined;
-}
-
-function extractTextContent(message: AssistantMessage): string {
-	return message.content
-		.filter(content => content.type === "text")
-		.map(content => content.text)
-		.join("")
-		.trim();
-}
-
-function parseJsonPayload(text: string): unknown {
-	const trimmed = text.trim();
-	if (trimmed.startsWith("{") && trimmed.endsWith("}")) {
-		return JSON.parse(trimmed) as unknown;
-	}
-	const match = trimmed.match(/\{[\s\S]*\}/);
-	if (!match) {
-		throw new Error("No JSON payload found in changelog response");
-	}
-	return JSON.parse(match[0]) as unknown;
-}
-
 function dedupeEntries(entries: Record<string, string[]>): Record<string, string[]> {
 	const result: Record<string, string[]> = {};
 	for (const [category, values] of Object.entries(entries)) {

package/src/commit/changelog/index.ts

@@ -1,4 +1,3 @@
-import * as fs from "node:fs";
 import * as path from "node:path";
 import type { Api, Model } from "@oh-my-pi/pi-ai";
 import { logger } from "@oh-my-pi/pi-utils";
@@ -109,7 +108,7 @@ export async function applyChangelogProposals({
 		)
 			continue;
 		onProgress?.(`Applying entries for ${proposal.path}…`);
-		const exists = fs.existsSync(proposal.path);
+		const exists = await Bun.file(proposal.path).exists();
 		if (!exists) {
 			logger.warn("commit changelog path missing", { path: proposal.path });
 			continue;