@oh-my-pi/pi-coding-agent 13.18.0 → 14.0.2

package/src/extensibility/hooks/types.ts

@@ -9,9 +9,9 @@ import type { ImageContent, Message, Model, TextContent, ToolResultMessage } fro
 import type { Component, TUI } from "@oh-my-pi/pi-tui";
 import type { Rule } from "../../capability/rule";
 import type { ModelRegistry } from "../../config/model-registry";
+import type { EditToolDetails } from "../../edit";
 import type { ExecOptions, ExecResult } from "../../exec/exec";
 import type { Theme } from "../../modes/theme/theme";
-import type { EditToolDetails } from "../../patch";
 import type { CompactionPreparation, CompactionResult } from "../../session/compaction";
 import type { HookMessage } from "../../session/messages";
 import type {
@@ -394,7 +394,7 @@ export interface TurnEndEvent {
 /** Event data for auto_compaction_start event. */
 export interface AutoCompactionStartEvent {
 	type: "auto_compaction_start";
-	reason: "threshold" | "overflow";
+	reason: "threshold" | "overflow" | "idle";
 	action: "context-full" | "handoff";
 }
 

package/src/extensibility/plugins/doctor.ts

@@ -1,3 +1,4 @@
+import { $which } from "@oh-my-pi/pi-utils";
 import { theme } from "../../modes/theme/theme";
 import type { DoctorCheck } from "./types";
 
@@ -12,7 +13,7 @@ export async function runDoctorChecks(): Promise<DoctorCheck[]> {
 	];
 
 	for (const tool of tools) {
-		const path = Bun.which(tool.name);
+		const path = $which(tool.name);
 		checks.push({
 			name: tool.name,
 			status: path ? "ok" : "warning",

package/src/extensibility/plugins/marketplace/fetcher.ts

@@ -8,7 +8,7 @@ import * as fs from "node:fs/promises";
 import * as os from "node:os";
 import * as path from "node:path";
 import { isEnoent, logger } from "@oh-my-pi/pi-utils";
-import { $ } from "bun";
+import * as git from "../../../utils/git";
 
 import type { MarketplaceCatalog, MarketplaceSourceType } from "./types";
 import { isValidNameSegment } from "./types";
@@ -274,21 +274,11 @@ export async function fetchMarketplace(source: string, cacheDir: string): Promis
  * `promoteCloneToCache` after any duplicate/drift checks pass.
  */
 async function cloneAndReadCatalog(url: string, cacheDir: string): Promise<FetchResult> {
-	if (!Bun.which("git")) {
-		throw new Error("git is not installed. Install git to use git-based marketplace sources.");
-	}
-
 	const tmpDir = path.join(cacheDir, `.tmp-clone-${Date.now()}`);
 	await fs.mkdir(cacheDir, { recursive: true });
 
 	logger.debug(`[marketplace] cloning ${url} → ${tmpDir}`);
-
-	const result = await $`git clone --depth 1 --single-branch ${url} ${tmpDir}`.quiet().nothrow();
-	if (result.exitCode !== 0) {
-		await fs.rm(tmpDir, { recursive: true, force: true });
-		const stderr = result.stderr.toString().trim();
-		throw new Error(`git clone failed (exit ${result.exitCode}): ${stderr || "unknown error"}`);
-	}
+	await git.clone(url, tmpDir);
 
 	const catalogPath = path.join(tmpDir, CATALOG_RELATIVE_PATH);
 	let content: string;
@@ -325,48 +315,3 @@ export async function promoteCloneToCache(tmpDir: string, cacheDir: string, name
 	await fs.rename(tmpDir, finalDir);
 	return finalDir;
 }
-
-/**
- * Clone a git repository to a target directory. Shared by fetcher (marketplace clones)
- * and source-resolver (plugin source clones).
- *
- * @param url - Git clone URL (HTTPS, SSH, or GitHub shorthand expanded to HTTPS)
- * @param targetDir - Directory to clone into (must not exist)
- * @param options.ref - Optional branch/tag to clone
- * @param options.sha - Optional commit SHA to checkout after clone
- */
-export async function cloneGitRepo(
-	url: string,
-	targetDir: string,
-	options?: { ref?: string; sha?: string },
-): Promise<void> {
-	if (!Bun.which("git")) {
-		throw new Error("git is not installed. Install git to use git-based plugin sources.");
-	}
-
-	const cloneArgs = ["git", "clone", "--depth", "1"];
-	if (options?.ref) {
-		cloneArgs.push("--branch", options.ref, "--single-branch");
-	} else {
-		cloneArgs.push("--single-branch");
-	}
-	cloneArgs.push(url, targetDir);
-
-	logger.debug("[marketplace] cloning plugin source", { url, targetDir });
-
-	const result = await $`${cloneArgs}`.quiet().nothrow();
-	if (result.exitCode !== 0) {
-		await fs.rm(targetDir, { recursive: true, force: true });
-		const stderr = result.stderr.toString().trim();
-		throw new Error(`git clone failed (exit ${result.exitCode}): ${stderr || "unknown error"}`);
-	}
-
-	// If a specific SHA is requested, checkout that commit
-	if (options?.sha) {
-		const checkout = await $`git -C ${targetDir} checkout ${options.sha}`.quiet().nothrow();
-		if (checkout.exitCode !== 0) {
-			await fs.rm(targetDir, { recursive: true, force: true });
-			throw new Error(`Failed to checkout SHA ${options.sha} — shallow clone may not contain this commit`);
-		}
-	}
-}

package/src/extensibility/plugins/marketplace/source-resolver.ts

@@ -14,8 +14,8 @@ import * as fs from "node:fs/promises";
 import * as path from "node:path";
 
 import { isEnoent, pathIsWithin } from "@oh-my-pi/pi-utils";
+import * as git from "../../../utils/git";
 
-import { cloneGitRepo } from "./fetcher";
 import type { MarketplaceCatalogMetadata, MarketplacePluginEntry, PluginSource } from "./types";
 
 export interface ResolveContext {
@@ -87,7 +87,7 @@ async function resolveObjectSource(
 			// { source: "url", url: "https://github.com/owner/repo.git" }
 			// Despite the name, this is typically a git clone URL
 			const targetDir = path.join(context.tmpDir, `plugin-${crypto.randomUUID()}`);
-			await cloneGitRepo(source.url, targetDir, { ref: source.ref, sha: source.sha });
+			await git.clone(source.url, targetDir, { ref: source.ref, sha: source.sha });
 			return { dir: targetDir, tempCloneRoot: targetDir };
 		}
 
@@ -95,7 +95,7 @@ async function resolveObjectSource(
 			// { source: "github", repo: "owner/repo" }
 			const url = `https://github.com/${source.repo}.git`;
 			const targetDir = path.join(context.tmpDir, `plugin-${crypto.randomUUID()}`);
-			await cloneGitRepo(url, targetDir, { ref: source.ref, sha: source.sha });
+			await git.clone(url, targetDir, { ref: source.ref, sha: source.sha });
 			return { dir: targetDir, tempCloneRoot: targetDir };
 		}
 
@@ -106,7 +106,7 @@ async function resolveObjectSource(
 					? source.url
 					: `https://github.com/${source.url}.git`;
 			const cloneDir = path.join(context.tmpDir, `plugin-repo-${crypto.randomUUID()}`);
-			await cloneGitRepo(url, cloneDir, { ref: source.ref, sha: source.sha });
+			await git.clone(url, cloneDir, { ref: source.ref, sha: source.sha });
 
 			const subdirPath = path.resolve(cloneDir, source.path);
 			if (!pathIsWithin(cloneDir, subdirPath)) {

package/src/extensibility/slash-commands.ts

@@ -1,10 +1,7 @@
 import type { AutocompleteItem } from "@oh-my-pi/pi-tui";
+import { parseFrontmatter, prompt } from "@oh-my-pi/pi-utils";
 import { slashCommandCapability } from "../capability/slash-command";
-import {
-	appendInlineArgsFallback,
-	renderPromptTemplate,
-	templateUsesInlineArgPlaceholders,
-} from "../config/prompt-templates";
+import { appendInlineArgsFallback, templateUsesInlineArgPlaceholders } from "../config/prompt-templates";
 import type { SlashCommand } from "../discovery";
 import { loadCapability } from "../discovery";
 import {
@@ -14,7 +11,6 @@ import {
 } from "../slash-commands/builtin-registry";
 import { EMBEDDED_COMMAND_TEMPLATES } from "../task/commands";
 import { parseCommandArgs, substituteArgs } from "../utils/command-args";
-import { parseFrontmatter } from "../utils/frontmatter";
 
 export type SlashCommandSource = "extension" | "prompt" | "skill";
 
@@ -223,7 +219,7 @@ export function expandSlashCommand(text: string, fileCommands: FileSlashCommand[
 		const argsText = args.join(" ");
 		const usesInlineArgPlaceholders = templateUsesInlineArgPlaceholders(fileCommand.content);
 		const substituted = substituteArgs(fileCommand.content, args);
-		const rendered = renderPromptTemplate(substituted, { args, ARGUMENTS: argsText, arguments: argsText });
+		const rendered = prompt.render(substituted, { args, ARGUMENTS: argsText, arguments: argsText });
 		return appendInlineArgsFallback(rendered, argsText, usesInlineArgPlaceholders);
 	}
 

package/src/index.ts

@@ -16,6 +16,7 @@ export type * from "./config/prompt-templates";
 export * from "./config/prompt-templates";
 export type { RetrySettings, SkillsSettings } from "./config/settings";
 export { Settings, settings } from "./config/settings";
+export * from "./edit/modes/hashline";
 // Custom commands
 export type * from "./extensibility/custom-commands/types";
 export type * from "./extensibility/custom-tools";
@@ -37,7 +38,6 @@ export * from "./modes";
 export * from "./modes/components";
 // Theme utilities for custom tools
 export * from "./modes/theme/theme";
-export * from "./patch/hashline";
 // SDK for programmatic usage
 export * from "./sdk";
 export * from "./session/agent-session";
@@ -46,11 +46,13 @@ export * from "./session/auth-storage";
 // Compaction
 export * from "./session/compaction";
 export * from "./session/messages";
+export * from "./session/session-dump-format";
 export * from "./session/session-manager";
 export * from "./task/executor";
 export type * from "./task/types";
 // Tools (detail types and utilities)
 export * from "./tools";
+export * from "./utils/git";
 // UI components for extensions
 export {
 	HookEditorComponent as ExtensionEditorComponent,