@oh-my-pi/pi-coding-agent 8.0.20 → 8.2.0

package/src/modes/components/extensions/state-manager.ts

@@ -2,25 +2,24 @@
  * State manager for the Extension Control Center.
  * Handles data loading, tree building, filtering, and toggle persistence.
  */
-
-import type { ContextFile } from "@oh-my-pi/pi-coding-agent/capability/context-file";
-import type { ExtensionModule } from "@oh-my-pi/pi-coding-agent/capability/extension-module";
-import type { Hook } from "@oh-my-pi/pi-coding-agent/capability/hook";
-import type { MCPServer } from "@oh-my-pi/pi-coding-agent/capability/mcp";
-import type { Prompt } from "@oh-my-pi/pi-coding-agent/capability/prompt";
-import type { Rule } from "@oh-my-pi/pi-coding-agent/capability/rule";
-import type { Skill } from "@oh-my-pi/pi-coding-agent/capability/skill";
-import type { SlashCommand } from "@oh-my-pi/pi-coding-agent/capability/slash-command";
-import type { CustomTool } from "@oh-my-pi/pi-coding-agent/capability/tool";
-import type { SourceMeta } from "@oh-my-pi/pi-coding-agent/capability/types";
+import { logger } from "@oh-my-pi/pi-utils";
+import type { ContextFile } from "../../../capability/context-file";
+import type { ExtensionModule } from "../../../capability/extension-module";
+import type { Hook } from "../../../capability/hook";
+import type { MCPServer } from "../../../capability/mcp";
+import type { Prompt } from "../../../capability/prompt";
+import type { Rule } from "../../../capability/rule";
+import type { Skill } from "../../../capability/skill";
+import type { SlashCommand } from "../../../capability/slash-command";
+import type { CustomTool } from "../../../capability/tool";
+import type { SourceMeta } from "../../../capability/types";
 import {
 	disableProvider,
 	enableProvider,
 	getAllProvidersInfo,
 	isProviderEnabled,
 	loadCapability,
-} from "@oh-my-pi/pi-coding-agent/discovery";
-import { logger } from "@oh-my-pi/pi-utils";
+} from "../../../discovery";
 import type {
 	DashboardState,
 	Extension,
@@ -103,8 +102,8 @@ export async function loadAllExtensions(cwd?: string, disabledIds?: string[]): P
 	try {
 		const skills = await loadCapability<Skill>("skills", loadOpts);
 		addItems(skills.all, "skill", {
-			getDescription: (s) => s.frontmatter?.description,
-			getTrigger: (s) => s.frontmatter?.globs?.join(", "),
+			getDescription: s => s.frontmatter?.description,
+			getTrigger: s => s.frontmatter?.globs?.join(", "),
 		});
 	} catch (error) {
 		logger.warn("Failed to load skills capability", { error: String(error) });
@@ -114,8 +113,8 @@ export async function loadAllExtensions(cwd?: string, disabledIds?: string[]): P
 	try {
 		const rules = await loadCapability<Rule>("rules", loadOpts);
 		addItems(rules.all, "rule", {
-			getDescription: (r) => r.description,
-			getTrigger: (r) => r.globs?.join(", ") || (r.alwaysApply ? "always" : undefined),
+			getDescription: r => r.description,
+			getTrigger: r => r.globs?.join(", ") || (r.alwaysApply ? "always" : undefined),
 		});
 	} catch (error) {
 		logger.warn("Failed to load rules capability", { error: String(error) });
@@ -125,7 +124,7 @@ export async function loadAllExtensions(cwd?: string, disabledIds?: string[]): P
 	try {
 		const tools = await loadCapability<CustomTool>("tools", loadOpts);
 		addItems(tools.all, "tool", {
-			getDescription: (t) => t.description,
+			getDescription: t => t.description,
 		});
 	} catch (error) {
 		logger.warn("Failed to load tools capability", { error: String(error) });
@@ -134,7 +133,7 @@ export async function loadAllExtensions(cwd?: string, disabledIds?: string[]): P
 	// Load extension modules
 	try {
 		const modules = await loadCapability<ExtensionModule>("extension-modules", loadOpts);
-		const nativeModules = modules.all.filter((module) => module._source.provider === "native");
+		const nativeModules = modules.all.filter(module => module._source.provider === "native");
 		addItems(nativeModules, "extension-module");
 	} catch (error) {
 		logger.warn("Failed to load extension-modules capability", { error: String(error) });
@@ -188,7 +187,7 @@ export async function loadAllExtensions(cwd?: string, disabledIds?: string[]): P
 		const prompts = await loadCapability<Prompt>("prompts", loadOpts);
 		addItems(prompts.all, "prompt", {
 			getDescription: () => undefined,
-			getTrigger: (p) => `/prompts:${p.name}`,
+			getTrigger: p => `/prompts:${p.name}`,
 		});
 	} catch (error) {
 		logger.warn("Failed to load prompts capability", { error: String(error) });
@@ -199,7 +198,7 @@ export async function loadAllExtensions(cwd?: string, disabledIds?: string[]): P
 		const commands = await loadCapability<SlashCommand>("slash-commands", loadOpts);
 		addItems(commands.all, "slash-command", {
 			getDescription: () => undefined,
-			getTrigger: (c) => `/${c.name}`,
+			getTrigger: c => `/${c.name}`,
 		});
 	} catch (error) {
 		logger.warn("Failed to load slash-commands capability", { error: String(error) });
@@ -396,7 +395,7 @@ export function applyFilter(extensions: Extension[], query: string): Extension[]
 		return extensions;
 	}
 
-	return extensions.filter((ext) => {
+	return extensions.filter(ext => {
 		const searchable = [
 			ext.name,
 			ext.displayName,
@@ -408,7 +407,7 @@ export function applyFilter(extensions: Extension[], query: string): Extension[]
 			.join(" ")
 			.toLowerCase();
 
-		return tokens.every((token) => searchable.includes(token));
+		return tokens.every(token => searchable.includes(token));
 	});
 }
 
@@ -506,7 +505,7 @@ export function filterByProvider(extensions: Extension[], providerId: string): E
 	if (providerId === "all") {
 		return extensions;
 	}
-	return extensions.filter((ext) => ext.source.provider === providerId);
+	return extensions.filter(ext => ext.source.provider === providerId);
 }
 
 /**
@@ -564,12 +563,12 @@ export async function refreshState(
 	const searchFiltered = applyFilter(tabFiltered, state.searchQuery);
 
 	// Find new index for current provider (tabs may have reordered)
-	const newActiveTabIndex = tabs.findIndex((t) => t.id === providerId);
+	const newActiveTabIndex = tabs.findIndex(t => t.id === providerId);
 	const activeTabIndex = newActiveTabIndex >= 0 ? newActiveTabIndex : 0;
 
 	// Try to preserve selection
 	const selectedId = state.selected?.id;
-	let selected = selectedId ? searchFiltered.find((e) => e.id === selectedId) : null;
+	let selected = selectedId ? searchFiltered.find(e => e.id === selectedId) : null;
 	if (!selected && searchFiltered.length > 0) {
 		selected = searchFiltered[Math.min(state.listIndex, searchFiltered.length - 1)];
 	}

package/src/modes/components/extensions/types.ts

@@ -1,8 +1,7 @@
 /**
  * Types for the Extension Control Center dashboard.
  */
-
-import type { SourceMeta } from "@oh-my-pi/pi-coding-agent/capability/types";
+import type { SourceMeta } from "../../../capability/types";
 
 /**
  * Extension kinds matching capability types.

package/src/modes/components/footer.ts

@@ -1,10 +1,11 @@
-import { existsSync, type FSWatcher, readFileSync, watch } from "node:fs";
+import * as fs from "node:fs";
+import * as path from "node:path";
 import type { AssistantMessage } from "@oh-my-pi/pi-ai";
-import { theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
-import type { AgentSession } from "@oh-my-pi/pi-coding-agent/session/agent-session";
-import { shortenPath } from "@oh-my-pi/pi-coding-agent/tools/render-utils";
 import { type Component, truncateToWidth, visibleWidth } from "@oh-my-pi/pi-tui";
-import { dirname, join } from "path";
+import { isEnoent } from "@oh-my-pi/pi-utils";
+import { theme } from "../../modes/theme/theme";
+import type { AgentSession } from "../../session/agent-session";
+import { shortenPath } from "../../tools/render-utils";
 
 /**
  * Sanitize text for display in a single-line status.
@@ -18,20 +19,19 @@ function sanitizeStatusText(text: string): string {
 		.trim();
 }
 
-/**
- * Find the git root directory by walking up from cwd.
- * Returns the path to .git/HEAD if found, null otherwise.
- */
-function findGitHeadPath(): string | null {
+/** Find the git root by walking up from cwd. Returns path and content of .git/HEAD if found. */
+async function findGitHeadPath(): Promise<{ path: string; content: string } | null> {
 	let dir = process.cwd();
 	while (true) {
-		const gitHeadPath = join(dir, ".git", "HEAD");
-		if (existsSync(gitHeadPath)) {
-			return gitHeadPath;
+		const gitHeadPath = path.join(dir, ".git", "HEAD");
+		try {
+			const content = await Bun.file(gitHeadPath).text();
+			return { path: gitHeadPath, content };
+		} catch (err) {
+			if (!isEnoent(err)) throw err;
 		}
-		const parent = dirname(dir);
+		const parent = path.dirname(dir);
 		if (parent === dir) {
-			// Reached filesystem root
 			return null;
 		}
 		dir = parent;
@@ -44,7 +44,7 @@ function findGitHeadPath(): string | null {
 export class FooterComponent implements Component {
 	private session: AgentSession;
 	private cachedBranch: string | null | undefined = undefined; // undefined = not checked yet, null = not in git repo, string = branch name
-	private gitWatcher: FSWatcher | null = null;
+	private gitWatcher: fs.FSWatcher | null = null;
 	private onBranchChange: (() => void) | null = null;
 	private autoCompactEnabled: boolean = true;
 	private extensionStatuses: Map<string, string> = new Map();
@@ -88,21 +88,22 @@ export class FooterComponent implements Component {
 			this.gitWatcher = null;
 		}
 
-		const gitHeadPath = findGitHeadPath();
-		if (!gitHeadPath) {
-			return;
-		}
+		findGitHeadPath().then(result => {
+			if (!result) {
+				return;
+			}
 
-		try {
-			this.gitWatcher = watch(gitHeadPath, () => {
-				this.cachedBranch = undefined; // Invalidate cache
-				if (this.onBranchChange) {
-					this.onBranchChange();
-				}
-			});
-		} catch {
-			// Silently fail if we can't watch
-		}
+			try {
+				this.gitWatcher = fs.watch(result.path, () => {
+					this.cachedBranch = undefined; // Invalidate cache
+					if (this.onBranchChange) {
+						this.onBranchChange();
+					}
+				});
+			} catch {
+				// Silently fail if we can't watch
+			}
+		});
 	}
 
 	/**
@@ -130,27 +131,30 @@ export class FooterComponent implements Component {
 			return this.cachedBranch;
 		}
 
-		try {
-			const gitHeadPath = findGitHeadPath();
-			if (!gitHeadPath) {
+		// Note: fire-and-forget async call - will return undefined on first call
+		// This is acceptable since it's a cached value that will update on next render
+		findGitHeadPath().then(result => {
+			if (!result) {
 				this.cachedBranch = null;
-				return null;
+				if (this.onBranchChange) {
+					this.onBranchChange();
+				}
+				return;
 			}
-			const content = readFileSync(gitHeadPath, "utf8").trim();
+			const content = result.content.trim();
 
 			if (content.startsWith("ref: refs/heads/")) {
-				// Normal branch: extract branch name
 				this.cachedBranch = content.slice(16);
 			} else {
-				// Detached HEAD state
 				this.cachedBranch = "detached";
 			}
-		} catch {
-			// Not in a git repo or error reading file
-			this.cachedBranch = null;
-		}
+			if (this.onBranchChange) {
+				this.onBranchChange();
+			}
+		});
 
-		return this.cachedBranch;
+		// Return undefined while loading (will show on next render once loaded)
+		return null;
 	}
 
 	render(width: number): string[] {
@@ -177,7 +181,7 @@ export class FooterComponent implements Component {
 		const lastAssistantMessage = state.messages
 			.slice()
 			.reverse()
-			.find((m) => m.role === "assistant" && m.stopReason !== "aborted") as AssistantMessage | undefined;
+			.find(m => m.role === "assistant" && m.stopReason !== "aborted") as AssistantMessage | undefined;
 
 		// Calculate context percentage from last message (input + output + cacheRead + cacheWrite)
 		const contextTokens = lastAssistantMessage

package/src/modes/components/history-search.ts

@@ -1,5 +1,3 @@
-import { theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
-import type { HistoryEntry, HistoryStorage } from "@oh-my-pi/pi-coding-agent/session/history-storage";
 import {
 	type Component,
 	Container,
@@ -10,6 +8,8 @@ import {
 	truncateToWidth,
 	visibleWidth,
 } from "@oh-my-pi/pi-tui";
+import { theme } from "../../modes/theme/theme";
+import type { HistoryEntry, HistoryStorage } from "../../session/history-storage";
 import { DynamicBorder } from "./dynamic-border";
 
 class HistoryResultsList implements Component {

package/src/modes/components/hook-editor.ts

@@ -2,13 +2,12 @@
  * Multi-line editor component for hooks.
  * Supports Ctrl+G for external editor.
  */
-
-import { rm } from "node:fs/promises";
+import * as fs from "node:fs/promises";
 import * as os from "node:os";
 import * as path from "node:path";
-import { getEditorTheme, theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
 import { Container, Editor, matchesKey, Spacer, Text, type TUI } from "@oh-my-pi/pi-tui";
 import { nanoid } from "nanoid";
+import { getEditorTheme, theme } from "../../modes/theme/theme";
 import { DynamicBorder } from "./dynamic-border";
 
 export class HookEditorComponent extends Container {
@@ -110,7 +109,7 @@ export class HookEditorComponent extends Container {
 			}
 		} finally {
 			try {
-				await rm(tmpFile, { force: true });
+				await fs.rm(tmpFile, { force: true });
 			} catch {
 				// Ignore cleanup errors
 			}

package/src/modes/components/hook-input.ts

@@ -1,9 +1,8 @@
 /**
  * Simple text input component for hooks.
  */
-
-import { theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
 import { Container, Input, matchesKey, Spacer, Text, type TUI } from "@oh-my-pi/pi-tui";
+import { theme } from "../../modes/theme/theme";
 import { CountdownTimer } from "./countdown-timer";
 import { DynamicBorder } from "./dynamic-border";
 
@@ -44,7 +43,7 @@ export class HookInputComponent extends Container {
 			this.countdown = new CountdownTimer(
 				opts.timeout,
 				opts.tui,
-				(s) => this.titleText.setText(theme.fg("accent", `${this.baseTitle} (${s}s)`)),
+				s => this.titleText.setText(theme.fg("accent", `${this.baseTitle} (${s}s)`)),
 				() => this.onCancelCallback(),
 			);
 		}

package/src/modes/components/hook-message.ts

@@ -1,9 +1,9 @@
 import type { TextContent } from "@oh-my-pi/pi-ai";
-import type { HookMessageRenderer } from "@oh-my-pi/pi-coding-agent/extensibility/hooks/types";
-import { getMarkdownTheme, theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
-import type { HookMessage } from "@oh-my-pi/pi-coding-agent/session/messages";
 import type { Component } from "@oh-my-pi/pi-tui";
 import { Box, Container, Markdown, Spacer, Text } from "@oh-my-pi/pi-tui";
+import type { HookMessageRenderer } from "../../extensibility/hooks/types";
+import { getMarkdownTheme, theme } from "../../modes/theme/theme";
+import type { HookMessage } from "../../session/messages";
 
 /**
  * Component that renders a custom message entry from hooks.
@@ -24,7 +24,7 @@ export class HookMessageComponent extends Container {
 		this.addChild(new Spacer(1));
 
 		// Create box with purple background (used for default rendering)
-		this.box = new Box(1, 1, (t) => theme.bg("customMessageBg", t));
+		this.box = new Box(1, 1, t => theme.bg("customMessageBg", t));
 
 		this.rebuild();
 	}
@@ -80,7 +80,7 @@ export class HookMessageComponent extends Container {
 		} else {
 			text = this.message.content
 				.filter((c): c is TextContent => c.type === "text")
-				.map((c) => c.text)
+				.map(c => c.text)
 				.join("\n");
 		}
 

package/src/modes/components/hook-selector.ts

@@ -2,9 +2,8 @@
  * Generic selector component for hooks.
  * Displays a list of string options with keyboard navigation.
  */
-
-import { theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
 import { Container, matchesKey, Spacer, Text, type TUI } from "@oh-my-pi/pi-tui";
+import { theme } from "../../modes/theme/theme";
 import { CountdownTimer } from "./countdown-timer";
 import { DynamicBorder } from "./dynamic-border";
 
@@ -50,7 +49,7 @@ export class HookSelectorComponent extends Container {
 			this.countdown = new CountdownTimer(
 				opts.timeout,
 				opts.tui,
-				(s) => this.titleText.setText(theme.fg("accent", `${this.baseTitle} (${s}s)`)),
+				s => this.titleText.setText(theme.fg("accent", `${this.baseTitle} (${s}s)`)),
 				() => this.onCancelCallback(),
 			);
 		}

package/src/modes/components/keybinding-hints.ts

@@ -1,10 +1,9 @@
 /**
  * Utilities for formatting keybinding hints in the UI.
  */
-
-import type { AppAction, KeybindingsManager } from "@oh-my-pi/pi-coding-agent/config/keybindings";
-import { theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
 import { type EditorAction, getEditorKeybindings, type KeyId } from "@oh-my-pi/pi-tui";
+import type { AppAction, KeybindingsManager } from "../../config/keybindings";
+import { theme } from "../../modes/theme/theme";
 
 /**
  * Format keys array as display string (e.g., ["ctrl+c", "escape"] -> "ctrl+c/escape").

package/src/modes/components/login-dialog.ts

@@ -1,7 +1,7 @@
 import { getOAuthProviders } from "@oh-my-pi/pi-ai";
-import { theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
 import { Container, getEditorKeybindings, Input, Spacer, Text, type TUI } from "@oh-my-pi/pi-tui";
 import { $ } from "bun";
+import { theme } from "../../modes/theme/theme";
 import { DynamicBorder } from "./dynamic-border";
 
 /**
@@ -23,7 +23,7 @@ export class LoginDialogComponent extends Container {
 		super();
 		this.tui = tui;
 
-		const providerInfo = getOAuthProviders().find((p) => p.id === providerId);
+		const providerInfo = getOAuthProviders().find(p => p.id === providerId);
 		const providerName = providerInfo?.name || providerId;
 
 		// Top border

package/src/modes/components/model-selector.ts

@@ -1,9 +1,4 @@
 import { type Model, modelsAreEqual } from "@oh-my-pi/pi-ai";
-import type { ModelRegistry } from "@oh-my-pi/pi-coding-agent/config/model-registry";
-import { parseModelString } from "@oh-my-pi/pi-coding-agent/config/model-resolver";
-import type { SettingsManager } from "@oh-my-pi/pi-coding-agent/config/settings-manager";
-import { type ThemeColor, theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
-import { fuzzyFilter } from "@oh-my-pi/pi-coding-agent/utils/fuzzy";
 import {
 	Container,
 	Input,
@@ -16,6 +11,11 @@ import {
 	type TUI,
 	visibleWidth,
 } from "@oh-my-pi/pi-tui";
+import type { ModelRegistry } from "../../config/model-registry";
+import { parseModelString } from "../../config/model-resolver";
+import type { SettingsManager } from "../../config/settings-manager";
+import { type ThemeColor, theme } from "../../modes/theme/theme";
+import { fuzzyFilter } from "../../utils/fuzzy";
 import { DynamicBorder } from "./dynamic-border";
 
 function makeInvertedBadge(label: string, color: ThemeColor): string {
@@ -190,7 +190,7 @@ export class ModelSelectorComponent extends Container {
 		if (defaultStr) {
 			const parsed = parseModelString(defaultStr);
 			if (parsed) {
-				this.defaultModel = allModels.find((m) => m.provider === parsed.provider && m.id === parsed.id);
+				this.defaultModel = allModels.find(m => m.provider === parsed.provider && m.id === parsed.id);
 			}
 		}
 
@@ -199,7 +199,7 @@ export class ModelSelectorComponent extends Container {
 		if (smolStr) {
 			const parsed = parseModelString(smolStr);
 			if (parsed) {
-				this.smolModel = allModels.find((m) => m.provider === parsed.provider && m.id === parsed.id);
+				this.smolModel = allModels.find(m => m.provider === parsed.provider && m.id === parsed.id);
 			}
 		}
 
@@ -208,7 +208,7 @@ export class ModelSelectorComponent extends Container {
 		if (slowStr) {
 			const parsed = parseModelString(slowStr);
 			if (parsed) {
-				this.slowModel = allModels.find((m) => m.provider === parsed.provider && m.id === parsed.id);
+				this.slowModel = allModels.find(m => m.provider === parsed.provider && m.id === parsed.id);
 			}
 		}
 	}
@@ -218,7 +218,7 @@ export class ModelSelectorComponent extends Container {
 
 		// Use scoped models if provided via --models flag
 		if (this.scopedModels.length > 0) {
-			models = this.scopedModels.map((scoped) => ({
+			models = this.scopedModels.map(scoped => ({
 				provider: scoped.model.provider,
 				id: scoped.model.id,
 				model: scoped.model,
@@ -279,7 +279,7 @@ export class ModelSelectorComponent extends Container {
 	private updateTabBar(): void {
 		this.headerContainer.clear();
 
-		const tabs: Tab[] = this.providers.map((provider) => ({ id: provider, label: provider }));
+		const tabs: Tab[] = this.providers.map(provider => ({ id: provider, label: provider }));
 		const tabBar = new TabBar("Models", tabs, getTabBarTheme(), this.activeTabIndex);
 		tabBar.onTabChange = (_tab, index) => {
 			this.activeTabIndex = index;
@@ -300,7 +300,7 @@ export class ModelSelectorComponent extends Container {
 		// Start with all models or filter by provider
 		let baseModels = this.allModels;
 		if (activeProvider !== ALL_TAB) {
-			baseModels = this.allModels.filter((m) => m.provider.toUpperCase() === activeProvider);
+			baseModels = this.allModels.filter(m => m.provider.toUpperCase() === activeProvider);
 		}
 
 		// Apply fuzzy filter if query is present
@@ -426,7 +426,7 @@ export class ModelSelectorComponent extends Container {
 		const menuWidth = Math.max(
 			visibleWidth(headerText),
 			visibleWidth(hintText),
-			...actionLines.map((line) => visibleWidth(line)),
+			...actionLines.map(line => visibleWidth(line)),
 		);
 
 		// Menu header

package/src/modes/components/oauth-selector.ts

@@ -1,7 +1,7 @@
 import { getOAuthProviders, type OAuthProviderInfo } from "@oh-my-pi/pi-ai";
-import { theme } from "@oh-my-pi/pi-coding-agent/modes/theme/theme";
-import type { AuthStorage } from "@oh-my-pi/pi-coding-agent/session/auth-storage";
 import { Container, matchesKey, Spacer, TruncatedText } from "@oh-my-pi/pi-tui";
+import { theme } from "../../modes/theme/theme";
+import type { AuthStorage } from "../../session/auth-storage";
 import { DynamicBorder } from "./dynamic-border";
 
 /**