@oh-my-pi/pi-coding-agent 12.12.3 → 12.14.0

package/src/discovery/builtin.ts

@@ -656,7 +656,7 @@ async function loadTools(ctx: LoadContext): Promise<LoadResult<CustomTool>> {
 
 		fileLoadPromises.push(
 			loadFilesFromDir<CustomTool>(ctx, toolsDir, PROVIDER_ID, level, {
-				extensions: ["json", "md"],
+				extensions: ["json", "md", "ts", "js", "sh", "bash", "py"],
 				transform: (name, content, path, source) => {
 					if (name.endsWith(".json")) {
 						const data = parseJSON<{ name?: string; description?: string }>(content);
@@ -668,11 +668,21 @@ async function loadTools(ctx: LoadContext): Promise<LoadResult<CustomTool>> {
 							_source: source,
 						};
 					}
-					const { frontmatter } = parseFrontmatter(content, { source: path });
+					if (name.endsWith(".md")) {
+						const { frontmatter } = parseFrontmatter(content, { source: path });
+						return {
+							name: (frontmatter.name as string) || name.replace(/\.md$/, ""),
+							path,
+							description: frontmatter.description as string | undefined,
+							level,
+							_source: source,
+						};
+					}
+					// Executable tool files (.ts, .js, .sh, .bash, .py)
+					const toolName = name.replace(/\.(ts|js|sh|bash|py)$/, "");
 					return {
-						name: (frontmatter.name as string) || name.replace(/\.md$/, ""),
+						name: toolName,
 						path,
-						description: frontmatter.description as string | undefined,
 						level,
 						_source: source,
 					};

package/src/discovery/ssh.ts

@@ -1,10 +1,11 @@
 /**
  * SSH JSON Provider
  *
- * Discovers SSH hosts from ssh.json or .ssh.json in the project root.
- * Priority: 5 (low, project-level only)
+ * Discovers SSH hosts from managed omp config paths and legacy root ssh.json files.
+ * Priority: 5 (low, project/user config discovery)
  */
 import * as path from "node:path";
+import { getSSHConfigPath } from "@oh-my-pi/pi-utils/dirs";
 import { registerProvider } from "../capability";
 import { readFile } from "../capability/fs";
 import { type SSHHost, sshCapability } from "../capability/ssh";
@@ -90,38 +91,39 @@ function normalizeHost(
 	};
 }
 
-async function loadSshJsonFile(_ctx: LoadContext, path: string): Promise<LoadResult<SSHHost>> {
+async function loadSshJsonFile(
+	ctx: LoadContext,
+	filePath: string,
+	level: "user" | "project",
+): Promise<LoadResult<SSHHost>> {
 	const items: SSHHost[] = [];
 	const warnings: string[] = [];
-
-	const content = await readFile(path);
+	const content = await readFile(filePath);
 	if (content === null) {
 		return { items, warnings };
 	}
-
 	const parsed = parseJSON<SSHConfigFile>(content);
 	if (!parsed) {
-		warnings.push(`Failed to parse JSON in ${path}`);
+		warnings.push(`Failed to parse JSON in ${filePath}`);
 		return { items, warnings };
 	}
-
 	const config = expandEnvVarsDeep(parsed);
 	if (!config.hosts || typeof config.hosts !== "object") {
-		warnings.push(`Missing hosts in ${path}`);
+		warnings.push(`Missing hosts in ${filePath}`);
 		return { items, warnings };
 	}
 
-	const source = createSourceMeta(PROVIDER_ID, path, "project");
+	const source = createSourceMeta(PROVIDER_ID, filePath, level);
 	for (const [name, rawHost] of Object.entries(config.hosts)) {
 		if (!name.trim()) {
-			warnings.push(`Invalid SSH host name in ${path}`);
+			warnings.push(`Invalid SSH host name in ${filePath}`);
 			continue;
 		}
 		if (!rawHost || typeof rawHost !== "object") {
-			warnings.push(`Invalid host entry in ${path}: ${name}`);
+			warnings.push(`Invalid host entry in ${filePath}: ${name}`);
 			continue;
 		}
-		const host = normalizeHost(name, rawHost, source, _ctx.home, warnings);
+		const host = normalizeHost(name, rawHost, source, ctx.home, warnings);
 		if (host) items.push(host);
 	}
 
@@ -130,14 +132,19 @@ async function loadSshJsonFile(_ctx: LoadContext, path: string): Promise<LoadRes
 		warnings: warnings.length > 0 ? warnings : undefined,
 	};
 }
-
 async function load(ctx: LoadContext): Promise<LoadResult<SSHHost>> {
-	const filenames = ["ssh.json", ".ssh.json"];
-	const results = await Promise.all(filenames.map(filename => loadSshJsonFile(ctx, path.join(ctx.cwd, filename))));
-
+	const candidateSources: Array<{ path: string; level: "user" | "project" }> = [
+		{ path: getSSHConfigPath("project", ctx.cwd), level: "project" },
+		{ path: getSSHConfigPath("user", ctx.cwd), level: "user" },
+		{ path: path.join(ctx.cwd, "ssh.json"), level: "project" },
+		{ path: path.join(ctx.cwd, ".ssh.json"), level: "project" },
+	];
+	const uniqueSources = candidateSources.filter(
+		(source, index, arr) => arr.findIndex(candidate => candidate.path === source.path) === index,
+	);
+	const results = await Promise.all(uniqueSources.map(source => loadSshJsonFile(ctx, source.path, source.level)));
 	const allItems = results.flatMap(r => r.items);
 	const allWarnings = results.flatMap(r => r.warnings ?? []);
-
 	return {
 		items: allItems,
 		warnings: allWarnings.length > 0 ? allWarnings : undefined,
@@ -147,7 +154,7 @@ async function load(ctx: LoadContext): Promise<LoadResult<SSHHost>> {
 registerProvider(sshCapability.id, {
 	id: PROVIDER_ID,
 	displayName: DISPLAY_NAME,
-	description: "Load SSH hosts from ssh.json or .ssh.json in the project root",
+	description: "Load SSH hosts from managed omp paths and legacy ssh.json/.ssh.json files",
 	priority: 5,
 	load,
 });

package/src/extensibility/extensions/types.ts

@@ -503,6 +503,7 @@ export interface ToolExecutionStartEvent {
 	toolCallId: string;
 	toolName: string;
 	args: unknown;
+	intent?: string;
 }
 
 /** Fired during tool execution with partial/streaming output */

package/src/internal-urls/docs-index.generated.ts

@@ -0,0 +1,101 @@
+// Auto-generated by scripts/generate-docs-index.ts - DO NOT EDIT
+
+import bashToolRuntimeMd from "../../../../docs/bash-tool-runtime.md" with { type: "text" };
+import blobArtifactArchitectureMd from "../../../../docs/blob-artifact-architecture.md" with { type: "text" };
+import compactionMd from "../../../../docs/compaction.md" with { type: "text" };
+import configUsageMd from "../../../../docs/config-usage.md" with { type: "text" };
+import customToolsMd from "../../../../docs/custom-tools.md" with { type: "text" };
+import environmentVariablesMd from "../../../../docs/environment-variables.md" with { type: "text" };
+import extensionLoadingMd from "../../../../docs/extension-loading.md" with { type: "text" };
+import extensionsMd from "../../../../docs/extensions.md" with { type: "text" };
+import fsScanCacheArchitectureMd from "../../../../docs/fs-scan-cache-architecture.md" with { type: "text" };
+import geminiManifestExtensionsMd from "../../../../docs/gemini-manifest-extensions.md" with { type: "text" };
+import handoffGenerationPipelineMd from "../../../../docs/handoff-generation-pipeline.md" with { type: "text" };
+import hooksMd from "../../../../docs/hooks.md" with { type: "text" };
+import mcpProtocolTransportsMd from "../../../../docs/mcp-protocol-transports.md" with { type: "text" };
+import mcpRuntimeLifecycleMd from "../../../../docs/mcp-runtime-lifecycle.md" with { type: "text" };
+import mcpServerToolAuthoringMd from "../../../../docs/mcp-server-tool-authoring.md" with { type: "text" };
+import modelsMd from "../../../../docs/models.md" with { type: "text" };
+import nativesAddonLoaderRuntimeMd from "../../../../docs/natives-addon-loader-runtime.md" with { type: "text" };
+import nativesArchitectureMd from "../../../../docs/natives-architecture.md" with { type: "text" };
+import nativesBindingContractMd from "../../../../docs/natives-binding-contract.md" with { type: "text" };
+import nativesBuildReleaseDebuggingMd from "../../../../docs/natives-build-release-debugging.md" with { type: "text" };
+import nativesMediaSystemUtilsMd from "../../../../docs/natives-media-system-utils.md" with { type: "text" };
+import nativesRustTaskCancellationMd from "../../../../docs/natives-rust-task-cancellation.md" with { type: "text" };
+import nativesShellPtyProcessMd from "../../../../docs/natives-shell-pty-process.md" with { type: "text" };
+import nativesTextSearchPipelineMd from "../../../../docs/natives-text-search-pipeline.md" with { type: "text" };
+import nonCompactionRetryPolicyMd from "../../../../docs/non-compaction-retry-policy.md" with { type: "text" };
+import notebookToolRuntimeMd from "../../../../docs/notebook-tool-runtime.md" with { type: "text" };
+import pluginManagerInstallerPlumbingMd from "../../../../docs/plugin-manager-installer-plumbing.md" with { type: "text" };
+import portingFromPiMonoMd from "../../../../docs/porting-from-pi-mono.md" with { type: "text" };
+import portingToNativesMd from "../../../../docs/porting-to-natives.md" with { type: "text" };
+import providerStreamingInternalsMd from "../../../../docs/provider-streaming-internals.md" with { type: "text" };
+import pythonReplMd from "../../../../docs/python-repl.md" with { type: "text" };
+import rpcMd from "../../../../docs/rpc.md" with { type: "text" };
+import rulebookMatchingPipelineMd from "../../../../docs/rulebook-matching-pipeline.md" with { type: "text" };
+import sdkMd from "../../../../docs/sdk.md" with { type: "text" };
+import secretsMd from "../../../../docs/secrets.md" with { type: "text" };
+import sessionMd from "../../../../docs/session.md" with { type: "text" };
+import sessionOperationsExportShareForkResumeMd from "../../../../docs/session-operations-export-share-fork-resume.md" with { type: "text" };
+import sessionSwitchingAndRecentListingMd from "../../../../docs/session-switching-and-recent-listing.md" with { type: "text" };
+import sessionTreePlanMd from "../../../../docs/session-tree-plan.md" with { type: "text" };
+import skillsMd from "../../../../docs/skills.md" with { type: "text" };
+import slashCommandInternalsMd from "../../../../docs/slash-command-internals.md" with { type: "text" };
+import taskAgentDiscoveryMd from "../../../../docs/task-agent-discovery.md" with { type: "text" };
+import themeMd from "../../../../docs/theme.md" with { type: "text" };
+import treeMd from "../../../../docs/tree.md" with { type: "text" };
+import ttsrInjectionLifecycleMd from "../../../../docs/ttsr-injection-lifecycle.md" with { type: "text" };
+import tuiMd from "../../../../docs/tui.md" with { type: "text" };
+import tuiRuntimeInternalsMd from "../../../../docs/tui-runtime-internals.md" with { type: "text" };
+
+export const EMBEDDED_DOCS: Readonly<Record<string, string>> = {
+	"bash-tool-runtime.md": bashToolRuntimeMd,
+	"blob-artifact-architecture.md": blobArtifactArchitectureMd,
+	"compaction.md": compactionMd,
+	"config-usage.md": configUsageMd,
+	"custom-tools.md": customToolsMd,
+	"environment-variables.md": environmentVariablesMd,
+	"extension-loading.md": extensionLoadingMd,
+	"extensions.md": extensionsMd,
+	"fs-scan-cache-architecture.md": fsScanCacheArchitectureMd,
+	"gemini-manifest-extensions.md": geminiManifestExtensionsMd,
+	"handoff-generation-pipeline.md": handoffGenerationPipelineMd,
+	"hooks.md": hooksMd,
+	"mcp-protocol-transports.md": mcpProtocolTransportsMd,
+	"mcp-runtime-lifecycle.md": mcpRuntimeLifecycleMd,
+	"mcp-server-tool-authoring.md": mcpServerToolAuthoringMd,
+	"models.md": modelsMd,
+	"natives-addon-loader-runtime.md": nativesAddonLoaderRuntimeMd,
+	"natives-architecture.md": nativesArchitectureMd,
+	"natives-binding-contract.md": nativesBindingContractMd,
+	"natives-build-release-debugging.md": nativesBuildReleaseDebuggingMd,
+	"natives-media-system-utils.md": nativesMediaSystemUtilsMd,
+	"natives-rust-task-cancellation.md": nativesRustTaskCancellationMd,
+	"natives-shell-pty-process.md": nativesShellPtyProcessMd,
+	"natives-text-search-pipeline.md": nativesTextSearchPipelineMd,
+	"non-compaction-retry-policy.md": nonCompactionRetryPolicyMd,
+	"notebook-tool-runtime.md": notebookToolRuntimeMd,
+	"plugin-manager-installer-plumbing.md": pluginManagerInstallerPlumbingMd,
+	"porting-from-pi-mono.md": portingFromPiMonoMd,
+	"porting-to-natives.md": portingToNativesMd,
+	"provider-streaming-internals.md": providerStreamingInternalsMd,
+	"python-repl.md": pythonReplMd,
+	"rpc.md": rpcMd,
+	"rulebook-matching-pipeline.md": rulebookMatchingPipelineMd,
+	"sdk.md": sdkMd,
+	"secrets.md": secretsMd,
+	"session.md": sessionMd,
+	"session-operations-export-share-fork-resume.md": sessionOperationsExportShareForkResumeMd,
+	"session-switching-and-recent-listing.md": sessionSwitchingAndRecentListingMd,
+	"session-tree-plan.md": sessionTreePlanMd,
+	"skills.md": skillsMd,
+	"slash-command-internals.md": slashCommandInternalsMd,
+	"task-agent-discovery.md": taskAgentDiscoveryMd,
+	"theme.md": themeMd,
+	"tree.md": treeMd,
+	"ttsr-injection-lifecycle.md": ttsrInjectionLifecycleMd,
+	"tui.md": tuiMd,
+	"tui-runtime-internals.md": tuiRuntimeInternalsMd,
+};
+
+export const EMBEDDED_DOC_FILENAMES = Object.keys(EMBEDDED_DOCS).sort();

package/src/internal-urls/docs-protocol.ts

@@ -0,0 +1,84 @@
+/**
+ * Protocol handler for docs:// URLs.
+ *
+ * Serves statically embedded documentation files bundled at build time.
+ *
+ * URL forms:
+ * - docs:// - Lists all available documentation files
+ * - docs://<file>.md - Reads a specific documentation file
+ */
+import * as path from "node:path";
+import { EMBEDDED_DOC_FILENAMES, EMBEDDED_DOCS } from "./docs-index.generated";
+import type { InternalResource, InternalUrl, ProtocolHandler } from "./types";
+
+/**
+ * Handler for docs:// URLs.
+ *
+ * Resolves documentation file names to their content, or lists available docs.
+ */
+export class DocsProtocolHandler implements ProtocolHandler {
+	readonly scheme = "docs";
+
+	async resolve(url: InternalUrl): Promise<InternalResource> {
+		// Extract filename from host + path
+		const host = url.rawHost || url.hostname;
+		const pathname = url.rawPathname ?? url.pathname;
+		const filename = host ? (pathname && pathname !== "/" ? host + pathname : host) : "";
+
+		if (!filename) {
+			return this.#listDocs(url);
+		}
+
+		return this.#readDoc(filename, url);
+	}
+
+	async #listDocs(url: InternalUrl): Promise<InternalResource> {
+		if (EMBEDDED_DOC_FILENAMES.length === 0) {
+			throw new Error("No documentation files found");
+		}
+
+		const listing = EMBEDDED_DOC_FILENAMES.map(f => `- [${f}](docs://${f})`).join("\n");
+		const content = `# Documentation\n\n${EMBEDDED_DOC_FILENAMES.length} files available:\n\n${listing}\n`;
+
+		return {
+			url: url.href,
+			content,
+			contentType: "text/markdown",
+			size: Buffer.byteLength(content, "utf-8"),
+			sourcePath: "docs://",
+		};
+	}
+
+	async #readDoc(filename: string, url: InternalUrl): Promise<InternalResource> {
+		// Validate: no traversal, no absolute paths
+		if (path.isAbsolute(filename)) {
+			throw new Error("Absolute paths are not allowed in docs:// URLs");
+		}
+
+		const normalized = path.posix.normalize(filename.replaceAll("\\", "/"));
+		if (normalized === ".." || normalized.startsWith("../") || normalized.includes("/../")) {
+			throw new Error("Path traversal (..) is not allowed in docs:// URLs");
+		}
+
+		const content = EMBEDDED_DOCS[normalized];
+		if (content === undefined) {
+			const lookup = normalized.replace(/\.md$/, "");
+			const suggestions = EMBEDDED_DOC_FILENAMES.filter(
+				f => f.includes(lookup) || lookup.includes(f.replace(/\.md$/, "")),
+			).slice(0, 5);
+			const suffix =
+				suggestions.length > 0
+					? `\nDid you mean: ${suggestions.join(", ")}`
+					: "\nUse docs:// to list available files.";
+			throw new Error(`Documentation file not found: ${filename}${suffix}`);
+		}
+
+		return {
+			url: url.href,
+			content,
+			contentType: "text/markdown",
+			size: Buffer.byteLength(content, "utf-8"),
+			sourcePath: `docs://${normalized}`,
+		};
+	}
+}

package/src/internal-urls/index.ts

@@ -22,6 +22,7 @@
 
 export { AgentProtocolHandler, type AgentProtocolOptions } from "./agent-protocol";
 export { ArtifactProtocolHandler, type ArtifactProtocolOptions } from "./artifact-protocol";
+export { DocsProtocolHandler } from "./docs-protocol";
 export { applyQuery, parseQuery, pathToQuery } from "./json-query";
 export { MemoryProtocolHandler, type MemoryProtocolOptions, resolveMemoryUrlToPath } from "./memory-protocol";
 export { PlanProtocolHandler, type PlanProtocolOptions, resolvePlanUrlToPath } from "./plan-protocol";

package/src/internal-urls/router.ts

@@ -1,5 +1,5 @@
 /**
- * Internal URL router for internal protocols (agent://, artifact://, plan://, memory://, skill://, rule://).
+ * Internal URL router for internal protocols (agent://, artifact://, plan://, memory://, skill://, rule://, docs://).
  */
 import type { InternalResource, InternalUrl, ProtocolHandler } from "./types";
 

package/src/modes/controllers/event-controller.ts

@@ -1,3 +1,4 @@
+import { INTENT_FIELD } from "@oh-my-pi/pi-agent-core";
 import { Loader, TERMINAL, Text } from "@oh-my-pi/pi-tui";
 import { settings } from "../../config/settings";
 import { AssistantMessageComponent } from "../../modes/components/assistant-message";
@@ -14,6 +15,7 @@ export class EventController {
 	#lastReadGroup: ReadToolGroupComponent | undefined = undefined;
 	#lastThinkingCount = 0;
 	#renderedCustomMessages = new Set<string>();
+	#lastIntent: string | undefined = undefined;
 
 	constructor(private ctx: InteractiveModeContext) {}
 
@@ -32,6 +34,13 @@ export class EventController {
 		return this.#lastReadGroup;
 	}
 
+	#updateWorkingMessageFromIntent(intent: string | undefined): void {
+		const trimmed = intent?.trim();
+		if (!trimmed || trimmed === this.#lastIntent) return;
+		this.#lastIntent = trimmed;
+		this.ctx.setWorkingMessage(`${trimmed} (esc to interrupt)`);
+	}
+
 	subscribeToAgent(): void {
 		this.ctx.unsubscribe = this.ctx.session.subscribe(async (event: AgentSessionEvent) => {
 			await this.handleEvent(event);
@@ -48,6 +57,7 @@ export class EventController {
 
 		switch (event.type) {
 			case "agent_start":
+				this.#lastIntent = undefined;
 				if (this.ctx.retryEscapeHandler) {
 					this.ctx.editor.onEscape = this.ctx.retryEscapeHandler;
 					this.ctx.retryEscapeHandler = undefined;
@@ -155,6 +165,15 @@ export class EventController {
 							}
 						}
 					}
+
+					// Update working message with intent from streamed tool arguments
+					for (const content of this.ctx.streamingMessage.content) {
+						if (content.type !== "toolCall") continue;
+						const args = content.arguments;
+						if (!args || typeof args !== "object" || !(INTENT_FIELD in args)) continue;
+						this.#updateWorkingMessageFromIntent(args[INTENT_FIELD] as string | undefined);
+					}
+
 					this.ctx.ui.requestRender();
 				}
 				break;
@@ -196,6 +215,7 @@ export class EventController {
 				break;
 
 			case "tool_execution_start": {
+				this.#updateWorkingMessageFromIntent(event.intent);
 				if (!this.ctx.pendingTools.has(event.toolCallId)) {
 					if (event.toolName === "read") {
 						const group = this.#getReadGroup();