@oh-my-pi/pi-coding-agent 4.2.1 → 4.2.3

package/src/modes/interactive/components/extensions/extension-dashboard.ts

@@ -33,22 +33,35 @@ import { applyFilter, createInitialState, filterByProvider, refreshState, toggle
 import type { DashboardState } from "./types";
 
 export class ExtensionDashboard extends Container {
-	private state: DashboardState;
-	private mainList: ExtensionList;
-	private inspector: InspectorPanel;
+	private state!: DashboardState;
+	private mainList!: ExtensionList;
+	private inspector!: InspectorPanel;
 	private settingsManager: SettingsManager | null;
 	private cwd: string;
 	private terminalHeight: number;
 
 	public onClose?: () => void;
 
-	constructor(cwd: string, settingsManager: SettingsManager | null = null, terminalHeight?: number) {
+	private constructor(cwd: string, settingsManager: SettingsManager | null, terminalHeight: number) {
 		super();
 		this.cwd = cwd;
 		this.settingsManager = settingsManager;
-		this.terminalHeight = terminalHeight ?? process.stdout.rows ?? 24;
-		const disabledIds = settingsManager?.getDisabledExtensions() ?? [];
-		this.state = createInitialState(cwd, disabledIds);
+		this.terminalHeight = terminalHeight;
+	}
+
+	static async create(
+		cwd: string,
+		settingsManager: SettingsManager | null = null,
+		terminalHeight?: number,
+	): Promise<ExtensionDashboard> {
+		const dashboard = new ExtensionDashboard(cwd, settingsManager, terminalHeight ?? process.stdout.rows ?? 24);
+		await dashboard.init();
+		return dashboard;
+	}
+
+	private async init(): Promise<void> {
+		const disabledIds = this.settingsManager?.getDisabledExtensions() ?? [];
+		this.state = await createInitialState(this.cwd, disabledIds);
 
 		// Calculate max visible items based on terminal height
 		// Reserve ~10 lines for header, tabs, help text, borders
@@ -150,7 +163,7 @@ export class ExtensionDashboard extends Container {
 
 	private handleProviderToggle(providerId: string): void {
 		toggleProvider(providerId);
-		this.refreshFromState();
+		void this.refreshFromState();
 	}
 
 	private handleExtensionToggle(extensionId: string, enabled: boolean): void {
@@ -162,15 +175,15 @@ export class ExtensionDashboard extends Container {
 			this.settingsManager.disableExtension(extensionId);
 		}
 
-		this.refreshFromState();
+		void this.refreshFromState();
 	}
 
-	private refreshFromState(): void {
+	private async refreshFromState(): Promise<void> {
 		// Remember current tab ID before refresh
 		const currentTabId = this.state.tabs[this.state.activeTabIndex]?.id;
 
 		const disabledIds = this.settingsManager?.getDisabledExtensions() ?? [];
-		this.state = refreshState(this.state, this.cwd, disabledIds);
+		this.state = await refreshState(this.state, this.cwd, disabledIds);
 
 		// Find the same tab in the new (re-sorted) list
 		if (currentTabId) {

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

@@ -18,7 +18,7 @@ import {
 	enableProvider,
 	getAllProvidersInfo,
 	isProviderEnabled,
-	loadSync,
+	loadCapability,
 } from "../../../../discovery";
 import type {
 	DashboardState,
@@ -42,7 +42,7 @@ export interface ExtensionSettingsManager {
 /**
  * Load all extensions from all capabilities.
  */
-export function loadAllExtensions(cwd?: string, disabledIds?: string[]): Extension[] {
+export async function loadAllExtensions(cwd?: string, disabledIds?: string[]): Promise<Extension[]> {
 	const extensions: Extension[] = [];
 	const disabledExtensions = new Set<string>(disabledIds ?? []);
 
@@ -100,7 +100,7 @@ export function loadAllExtensions(cwd?: string, disabledIds?: string[]): Extensi
 
 	// Load skills
 	try {
-		const skills = loadSync<Skill>("skills", loadOpts);
+		const skills = await loadCapability<Skill>("skills", loadOpts);
 		addItems(skills.all, "skill", {
 			getDescription: (s) => s.frontmatter?.description,
 			getTrigger: (s) => s.frontmatter?.globs?.join(", "),
@@ -111,7 +111,7 @@ export function loadAllExtensions(cwd?: string, disabledIds?: string[]): Extensi
 
 	// Load rules
 	try {
-		const rules = loadSync<Rule>("rules", loadOpts);
+		const rules = await loadCapability<Rule>("rules", loadOpts);
 		addItems(rules.all, "rule", {
 			getDescription: (r) => r.description,
 			getTrigger: (r) => r.globs?.join(", ") || (r.alwaysApply ? "always" : undefined),
@@ -122,7 +122,7 @@ export function loadAllExtensions(cwd?: string, disabledIds?: string[]): Extensi
 
 	// Load custom tools
 	try {
-		const tools = loadSync<CustomTool>("tools", loadOpts);
+		const tools = await loadCapability<CustomTool>("tools", loadOpts);
 		addItems(tools.all, "tool", {
 			getDescription: (t) => t.description,
 		});
@@ -132,7 +132,7 @@ export function loadAllExtensions(cwd?: string, disabledIds?: string[]): Extensi
 
 	// Load extension modules
 	try {
-		const modules = loadSync<ExtensionModule>("extension-modules", loadOpts);
+		const modules = await loadCapability<ExtensionModule>("extension-modules", loadOpts);
 		const nativeModules = modules.all.filter((module) => module._source.provider === "native");
 		addItems(nativeModules, "extension-module");
 	} catch {
@@ -141,7 +141,7 @@ export function loadAllExtensions(cwd?: string, disabledIds?: string[]): Extensi
 
 	// Load MCP servers
 	try {
-		const mcps = loadSync<MCPServer>("mcps", loadOpts);
+		const mcps = await loadCapability<MCPServer>("mcps", loadOpts);
 		for (const server of mcps.all) {
 			const id = makeExtensionId("mcp", server.name);
 			const isDisabled = disabledExtensions.has(id);
@@ -184,7 +184,7 @@ export function loadAllExtensions(cwd?: string, disabledIds?: string[]): Extensi
 
 	// Load prompts
 	try {
-		const prompts = loadSync<Prompt>("prompts", loadOpts);
+		const prompts = await loadCapability<Prompt>("prompts", loadOpts);
 		addItems(prompts.all, "prompt", {
 			getDescription: () => undefined,
 			getTrigger: (p) => `/prompts:${p.name}`,
@@ -195,7 +195,7 @@ export function loadAllExtensions(cwd?: string, disabledIds?: string[]): Extensi
 
 	// Load slash commands
 	try {
-		const commands = loadSync<SlashCommand>("slash-commands", loadOpts);
+		const commands = await loadCapability<SlashCommand>("slash-commands", loadOpts);
 		addItems(commands.all, "slash-command", {
 			getDescription: () => undefined,
 			getTrigger: (c) => `/${c.name}`,
@@ -206,7 +206,7 @@ export function loadAllExtensions(cwd?: string, disabledIds?: string[]): Extensi
 
 	// Load hooks
 	try {
-		const hooks = loadSync<Hook>("hooks", loadOpts);
+		const hooks = await loadCapability<Hook>("hooks", loadOpts);
 		for (const hook of hooks.all) {
 			const id = makeExtensionId("hook", `${hook.type}:${hook.tool}:${hook.name}`);
 			const isDisabled = disabledExtensions.has(id);
@@ -249,7 +249,7 @@ export function loadAllExtensions(cwd?: string, disabledIds?: string[]): Extensi
 
 	// Load context files
 	try {
-		const contextFiles = loadSync<ContextFile>("context-files", loadOpts);
+		const contextFiles = await loadCapability<ContextFile>("context-files", loadOpts);
 		for (const file of contextFiles.all) {
 			// Extract filename from path for display
 			const name = file.path.split("/").pop() || file.path;
@@ -511,8 +511,8 @@ export function filterByProvider(extensions: Extension[], providerId: string): E
 /**
  * Create initial dashboard state.
  */
-export function createInitialState(cwd?: string, disabledIds?: string[]): DashboardState {
-	const extensions = loadAllExtensions(cwd, disabledIds);
+export async function createInitialState(cwd?: string, disabledIds?: string[]): Promise<DashboardState> {
+	const extensions = await loadAllExtensions(cwd, disabledIds);
 	const tabs = buildProviderTabs(extensions);
 	const tabFiltered = extensions; // "all" tab by default
 	const searchFiltered = tabFiltered;
@@ -546,8 +546,12 @@ export function toggleProvider(providerId: string): boolean {
 /**
  * Refresh state after toggle.
  */
-export function refreshState(state: DashboardState, cwd?: string, disabledIds?: string[]): DashboardState {
-	const extensions = loadAllExtensions(cwd, disabledIds);
+export async function refreshState(
+	state: DashboardState,
+	cwd?: string,
+	disabledIds?: string[],
+): Promise<DashboardState> {
+	const extensions = await loadAllExtensions(cwd, disabledIds);
 	const tabs = buildProviderTabs(extensions);
 
 	// Get current provider from tabs

package/src/modes/interactive/controllers/selector-controller.ts

@@ -111,9 +111,13 @@ export class SelectorController {
 	 * Show the Extension Control Center dashboard.
 	 * Replaces /status with a unified view of all providers and extensions.
 	 */
-	showExtensionsDashboard(): void {
+	async showExtensionsDashboard(): Promise<void> {
+		const dashboard = await ExtensionDashboard.create(
+			process.cwd(),
+			this.ctx.settingsManager,
+			this.ctx.ui.terminal.rows,
+		);
 		this.showSelector((done) => {
-			const dashboard = new ExtensionDashboard(process.cwd(), this.ctx.settingsManager, this.ctx.ui.terminal.rows);
 			dashboard.onClose = () => {
 				done();
 				this.ctx.ui.requestRender();

package/src/modes/interactive/interactive-mode.ts

@@ -112,6 +112,7 @@ export class InteractiveMode implements InteractiveModeContext {
 	public lastStatusText: Text | undefined = undefined;
 	public fileSlashCommands: Set<string> = new Set();
 
+	private pendingSlashCommands: SlashCommand[] = [];
 	private cleanupUnsubscribe?: () => void;
 	private readonly version: string;
 	private readonly changelogMarkdown: string | undefined;
@@ -210,14 +211,6 @@ export class InteractiveMode implements InteractiveModeContext {
 			{ name: "exit", description: "Exit the application" },
 		];
 
-		// Load and convert file commands to SlashCommand format
-		const fileCommands = loadSlashCommands({ cwd: process.cwd() });
-		this.fileSlashCommands = new Set(fileCommands.map((cmd) => cmd.name));
-		const fileSlashCommands: SlashCommand[] = fileCommands.map((cmd) => ({
-			name: cmd.name,
-			description: cmd.description,
-		}));
-
 		// Convert hook commands to SlashCommand format
 		const hookCommands: SlashCommand[] = (this.session.extensionRunner?.getRegisteredCommands() ?? []).map((cmd) => ({
 			name: cmd.name,
@@ -230,12 +223,8 @@ export class InteractiveMode implements InteractiveModeContext {
 			description: `${loaded.command.description} (${loaded.source})`,
 		}));
 
-		// Setup autocomplete
-		const autocompleteProvider = new CombinedAutocompleteProvider(
-			[...slashCommands, ...fileSlashCommands, ...hookCommands, ...customCommands],
-			process.cwd(),
-		);
-		this.editor.setAutocompleteProvider(autocompleteProvider);
+		// Store pending commands for init() where file commands are loaded async
+		this.pendingSlashCommands = [...slashCommands, ...hookCommands, ...customCommands];
 
 		this.uiHelpers = new UiHelpers(this);
 		this.voiceManager = new VoiceManager(this);
@@ -252,6 +241,21 @@ export class InteractiveMode implements InteractiveModeContext {
 		// Register session manager flush for signal handlers (SIGINT, SIGTERM, SIGHUP)
 		this.cleanupUnsubscribe = registerAsyncCleanup(() => this.sessionManager.flush());
 
+		// Load and convert file commands to SlashCommand format (async)
+		const fileCommands = await loadSlashCommands({ cwd: process.cwd() });
+		this.fileSlashCommands = new Set(fileCommands.map((cmd) => cmd.name));
+		const fileSlashCommands: SlashCommand[] = fileCommands.map((cmd) => ({
+			name: cmd.name,
+			description: cmd.description,
+		}));
+
+		// Setup autocomplete with all commands
+		const autocompleteProvider = new CombinedAutocompleteProvider(
+			[...this.pendingSlashCommands, ...fileSlashCommands],
+			process.cwd(),
+		);
+		this.editor.setAutocompleteProvider(autocompleteProvider);
+
 		// Get current model info for welcome screen
 		const modelName = this.session.model?.name ?? "Unknown";
 		const providerName = this.session.model?.provider ?? "Unknown";
@@ -560,7 +564,7 @@ export class InteractiveMode implements InteractiveModeContext {
 	}
 
 	showExtensionsDashboard(): void {
-		this.selectorController.showExtensionsDashboard();
+		void this.selectorController.showExtensionsDashboard();
 	}
 
 	showModelSelector(options?: { temporaryOnly?: boolean }): void {

package/src/prompts/agents/plan.md

@@ -1,54 +1,131 @@
 ---
 name: plan
-description: Software architect that explores codebase and designs implementation plans (read-only)
+description: Software architect for complex multi-file architectural decisions. NOT for simple tasks, single-file changes, reasoning, or tasks completable in <5 tool calls—execute those directly.
 tools: read, grep, find, ls, bash
-model: default
+spawns: explore
+model: pi/slow, gpt-5.2-codex, gpt-5.2, codex, gpt
 ---
 
-You are a software architect and planning specialist. Explore the codebase and design implementation plans.
+<role>Senior software architect producing implementation plans. READ-ONLY — no file modifications, no state changes.</role>
 
 === CRITICAL: READ-ONLY MODE ===
-This is a READ-ONLY planning task. You are STRICTLY PROHIBITED from:
-
+You are STRICTLY PROHIBITED from:
 - Creating or modifying files (no Write, Edit, touch, rm, mv, cp)
 - Creating temporary files anywhere, including /tmp
 - Using redirect operators (>, >>, |) or heredocs to write files
 - Running commands that change system state (git add, git commit, npm install, pip install)
+- Use bash ONLY for git status/log/diff; use read/grep/find/ls tools for file and search operations
+
+<context>
+Another engineer will execute your plan without re-exploring the codebase. Your plan must be specific enough to implement directly.
+</context>
+
+<process>
+## Phase 1: Understand
+
+1. Parse the task requirements precisely
+2. Identify ambiguities — list assumptions you're making
+3. Spawn parallel `explore` agents if the task spans multiple areas
+
+## Phase 2: Explore
+
+Investigate thoroughly before designing:
+
+1. Find existing patterns via grep/find
+2. Read key files to understand current architecture
+3. Trace data flow through relevant code paths
+4. Identify types, interfaces, and contracts involved
+5. Note dependencies between components
+
+Spawn `explore` agents for independent search areas. Synthesize findings.
+
+## Phase 3: Design
+
+Create implementation approach:
+
+1. List concrete changes required (files, functions, types)
+2. Define the sequence — what depends on what
+3. Identify edge cases and error conditions
+4. Consider alternatives; justify your choice
+5. Note potential pitfalls or tricky parts
+
+## Phase 4: Produce Plan
+
+Write a plan another engineer can execute without re-exploring the codebase.
+</process>
+
+<example>
+## Summary
+What we're building and why (one paragraph).
+
+## Changes
+
+1. **`path/to/file.ts`** — What to change
+   - Specific modifications
+2. **`path/to/other.ts`** — ...
+
+## Sequence
+
+1. X (no dependencies)
+2. Y (depends on X)
+3. Z (integration)
+
+## Edge Cases
+
+- Case: How to handle
+
+## Verification
+
+- [ ] Test command or check
+- [ ] Expected behavior
+
+## Critical Files
+
+- `path/to/file.ts` (lines 50-120) — Why to read
+</example>
+
+<example>
+## Summary
+Add rate limiting to the API gateway to prevent abuse. Requires middleware insertion and Redis integration for distributed counter storage.
 
-Your role is EXCLUSIVELY to explore and plan. You do NOT have access to file editing tools.
+## Changes
 
-## Process
+1. **`src/middleware/rate-limit.ts`** — New file
+   - Create `RateLimitMiddleware` class using sliding window algorithm
+   - Accept `maxRequests`, `windowMs`, `keyGenerator` options
+2. **`src/gateway/index.ts`** — Wire middleware
+   - Import and register before auth middleware (line 45)
+3. **`src/config/redis.ts`** — Add rate limit key prefix
+   - Add `RATE_LIMIT_PREFIX` constant
 
-1. **Understand Requirements**: Focus on the requirements provided.
+## Sequence
 
-2. **Explore Thoroughly**:
-   - Read any files provided in the initial prompt
-   - Find existing patterns and conventions using find, grep, read
-   - Understand the current architecture
-   - Identify similar features as reference
-   - Trace through relevant code paths
-   - Use bash ONLY for git status/log/diff; use read/grep/find/ls tools for file and search operations
+1. `rate-limit.ts` (standalone, no deps)
+2. `redis.ts` (config only)
+3. `gateway/index.ts` (integration)
 
-3. **Design Solution**:
-   - Create implementation approach
-   - Consider trade-offs and architectural decisions
-   - Follow existing patterns where appropriate
+## Edge Cases
 
-4. **Detail the Plan**:
-   - Provide step-by-step implementation strategy
-   - Identify dependencies and sequencing
-   - Anticipate potential challenges
+- Redis unavailable: fail open with warning log
+- IPv6 addresses: normalize before using as key
 
-## Required Output
+## Verification
 
-End your response with:
+- [ ] `curl -X GET localhost:3000/api/test` 100x rapidly → 429 after limit
+- [ ] Redis CLI: `KEYS rate:*` shows entries
 
-### Critical Files for Implementation
+## Critical Files
 
-List 3-5 files most critical for implementing this plan:
+- `src/middleware/auth.ts` (lines 20-50) — Pattern to follow
+- `src/types/middleware.ts` — Interface to implement
+</example>
 
-- `path/to/file1.ts` - Brief reason (e.g., "Core logic to modify")
-- `path/to/file2.ts` - Brief reason (e.g., "Interfaces to implement")
-- `path/to/file3.ts` - Brief reason (e.g., "Pattern to follow")
+<requirements>
+- Plan must be specific enough to implement without additional exploration
+- Include exact file paths and line ranges where relevant
+- Sequence must respect dependencies
+- Verification must be concrete and testable
+</requirements>
 
+Keep going until complete. This matters — get it right.
 REMEMBER: You can ONLY explore and plan. You CANNOT write, edit, or modify any files.

package/src/utils/shell.ts

@@ -92,12 +92,12 @@ function buildConfig(shell: string): ShellConfig {
  * 3. On Unix: $SHELL if bash/zsh, then fallback paths
  * 4. Fallback: sh
  */
-export function getShellConfig(): ShellConfig {
+export async function getShellConfig(): Promise<ShellConfig> {
 	if (cachedShellConfig) {
 		return cachedShellConfig;
 	}
 
-	const settings = SettingsManager.create();
+	const settings = await SettingsManager.create();
 	const customShellPath = settings.getShellPath();
 
 	// 1. Check user-specified shell path

package/src/prompts/agents/planner.md

@@ -1,112 +0,0 @@
----
-name: planner
-description: Software architect that explores codebase and produces detailed implementation plans
-tools: read, grep, find, ls, bash
-spawns: explore
-model: pi/slow, gpt-5.2-codex, gpt-5.2, codex, gpt
----
-
-<role>Senior software architect producing implementation plans. READ-ONLY — no file modifications, no state changes.</role>
-
-<context>
-Another engineer will execute your plan without re-exploring the codebase. Your plan must be specific enough to implement directly.
-</context>
-
-<process>
-## Phase 1: Understand
-
-1. Parse the task requirements precisely
-2. Identify ambiguities — list assumptions you're making
-3. Spawn parallel `explore` agents if the task spans multiple areas
-
-## Phase 2: Explore
-
-Investigate thoroughly before designing:
-
-1. Find existing patterns via grep/find
-2. Read key files to understand current architecture
-3. Trace data flow through relevant code paths
-4. Identify types, interfaces, and contracts involved
-5. Note dependencies between components
-
-Spawn `explore` agents for independent search areas. Synthesize findings.
-
-## Phase 3: Design
-
-Create implementation approach:
-
-1. List concrete changes required (files, functions, types)
-2. Define the sequence — what depends on what
-3. Identify edge cases and error conditions
-4. Consider alternatives; justify your choice
-5. Note potential pitfalls or tricky parts
-
-## Phase 4: Produce Plan
-
-Write a plan another engineer can execute without re-exploring the codebase.
-</process>
-
-<example>
-## Summary
-What we're building and why (one paragraph).
-
-## Changes
-1. **`path/to/file.ts`** — What to change
-   - Specific modifications
-2. **`path/to/other.ts`** — ...
-
-## Sequence
-1. X (no dependencies)
-2. Y (depends on X)
-3. Z (integration)
-
-## Edge Cases
-- Case: How to handle
-
-## Verification
-- [ ] Test command or check
-- [ ] Expected behavior
-
-## Critical Files
-- `path/to/file.ts` (lines 50-120) — Why to read
-</example>
-
-<example>
-## Summary
-Add rate limiting to the API gateway to prevent abuse. Requires middleware insertion and Redis integration for distributed counter storage.
-
-## Changes
-1. **`src/middleware/rate-limit.ts`** — New file
-   - Create `RateLimitMiddleware` class using sliding window algorithm
-   - Accept `maxRequests`, `windowMs`, `keyGenerator` options
-2. **`src/gateway/index.ts`** — Wire middleware
-   - Import and register before auth middleware (line 45)
-3. **`src/config/redis.ts`** — Add rate limit key prefix
-   - Add `RATE_LIMIT_PREFIX` constant
-
-## Sequence
-1. `rate-limit.ts` (standalone, no deps)
-2. `redis.ts` (config only)
-3. `gateway/index.ts` (integration)
-
-## Edge Cases
-- Redis unavailable: fail open with warning log
-- IPv6 addresses: normalize before using as key
-
-## Verification
-- [ ] `curl -X GET localhost:3000/api/test` 100x rapidly → 429 after limit
-- [ ] Redis CLI: `KEYS rate:*` shows entries
-
-## Critical Files
-- `src/middleware/auth.ts` (lines 20-50) — Pattern to follow
-- `src/types/middleware.ts` — Interface to implement
-</example>
-
-<requirements>
-- Plan must be specific enough to implement without additional exploration
-- Include exact file paths and line ranges where relevant
-- Sequence must respect dependencies
-- Verification must be concrete and testable
-</requirements>
-
-Keep going until complete. This matters — get it right.