@oh-my-pi/pi-coding-agent 3.1.1337 → 3.3.1337

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## [Unreleased]
 
+## [3.3.1337] - 2026-01-03
+
+### Changed
+
+- Improved `/status` command output formatting to use consistent column alignment across all sections
+- Updated version update notification to suggest `omp update` instead of manual npm install command
+
 ## [3.1.1337] - 2026-01-03
 ### Added
 

package/docs/extension-loading.md

@@ -775,7 +775,7 @@ Update argument parsing:
   result.noHooks = true;
 ```
 
-Add subcommand handling for `pi install`, `pi remove`, `pi update`.
+Add subcommand handling for `omp install`, `omp remove`, `omp update`.
 
 ### `src/core/hooks/loader.ts`
 
@@ -949,7 +949,7 @@ if (settings.skills?.customDirectories) {
 5. **Phase 5: CLI updates**
    - Add new flags to args.ts
    - Update help text
-   - Add `pi install`, `pi remove`, `pi update` subcommands
+   - Add `omp install`, `omp remove`, `omp update` subcommands
 
 6. **Phase 6: Integration**
    - Update sdk.ts

package/docs/sdk.md

@@ -54,7 +54,7 @@ The main factory function. Creates an `AgentSession` with configurable options.
 
 **Philosophy:** "Omit to discover, provide to override."
 
-- Omit an option → pi discovers/loads from standard locations
+- Omit an option → omp discovers/loads from standard locations
 - Provide an option → your value is used, discovery skipped for that option
 
 ```typescript
@@ -405,7 +405,7 @@ const { session } = await createAgentSession({
 
 **When you don't need factories:**
 
-- If you omit `tools`, pi automatically creates them with the correct `cwd`
+- If you omit `tools`, omp automatically creates them with the correct `cwd`
 - If you use `process.cwd()` as your `cwd`, the pre-built instances work fine
 
 **When you must use factories:**

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "3.1.1337",
+	"version": "3.3.1337",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -39,9 +39,9 @@
 		"prepublishOnly": "bun run generate-template && bun run clean && bun run build"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-agent-core": "3.1.1337",
-		"@oh-my-pi/pi-ai": "3.1.1337",
-		"@oh-my-pi/pi-tui": "3.1.1337",
+		"@oh-my-pi/pi-agent-core": "3.3.1337",
+		"@oh-my-pi/pi-ai": "3.3.1337",
+		"@oh-my-pi/pi-tui": "3.3.1337",
 		"@sinclair/typebox": "^0.34.46",
 		"ajv": "^8.17.1",
 		"chalk": "^5.5.0",

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

@@ -1617,7 +1617,7 @@ export class InteractiveMode {
 				theme.bold(theme.fg("warning", "Update Available")) +
 					"\n" +
 					theme.fg("muted", `New version ${newVersion} is available. Run: `) +
-					theme.fg("accent", "npm install -g @oh-my-pi/pi-coding-agent"),
+					theme.fg("accent", "omp update"),
 				1,
 				0,
 			),
@@ -2391,14 +2391,26 @@ export class InteractiveMode {
 	}
 
 	private handleStatusCommand(): void {
-		const sections: string[] = [];
-
 		type StatusSource =
 			| { provider: string; level: string }
 			| { mcpServer: string; provider?: string }
 			| "builtin"
 			| "unknown";
 
+		type StatusLine = {
+			name: string;
+			sourceText: string;
+			nameWithSource: string;
+			desc?: string;
+		};
+
+		type LineSection = {
+			title: string;
+			lines: StatusLine[];
+		};
+
+		type Section = { kind: "lines"; section: LineSection } | { kind: "text"; text: string };
+
 		const capitalize = (value: string): string => value.charAt(0).toUpperCase() + value.slice(1);
 
 		const resolveSourceText = (source: StatusSource): string => {
@@ -2440,29 +2452,28 @@ export class InteractiveMode {
 			return `${acc}...`;
 		};
 
-		// Helper to format a section with consistent column alignment
-		const formatSection = <T>(
+		const buildLineSection = <T>(
 			title: string,
 			items: readonly T[],
 			getName: (item: T) => string,
 			getDesc: (item: T) => string | undefined,
 			getSource: (item: T) => StatusSource,
-		): string => {
-			if (items.length === 0) return "";
+		): LineSection | null => {
+			if (items.length === 0) return null;
 
-			const lineItems = items.map((item) => {
+			const lines = items.map((item) => {
 				const name = getName(item);
-				const desc = getDesc(item);
+				const desc = getDesc(item)?.trim();
 				const sourceText = resolveSourceText(getSource(item));
 				const nameWithSource = sourceText ? `${name} ${sourceText}` : name;
 				return { name, sourceText, nameWithSource, desc };
 			});
 
-			const maxNameWidth = Math.min(
-				60,
-				Math.max(...lineItems.map((line) => visibleWidth(line.nameWithSource))),
-			);
-			const formattedLines = lineItems.map((line) => {
+			return { title, lines };
+		};
+
+		const renderLineSection = (section: LineSection, maxNameWidth: number): string => {
+			const formattedLines = section.lines.map((line) => {
 				let nameText = line.name;
 				let sourceText = line.sourceText;
 
@@ -2471,103 +2482,101 @@ export class InteractiveMode {
 					sourceText = truncateText(sourceText, maxSourceWidth);
 				}
 				const sourceWidth = sourceText ? visibleWidth(sourceText) : 0;
-				const availableForName = sourceText
-					? Math.max(1, maxNameWidth - sourceWidth - 1)
-					: maxNameWidth;
+				const availableForName = sourceText ? Math.max(1, maxNameWidth - sourceWidth - 1) : maxNameWidth;
 				nameText = truncateText(nameText, availableForName);
 
 				const nameWithSourcePlain = sourceText ? `${nameText} ${sourceText}` : nameText;
 				const sourceRendered = sourceText ? renderSourceText(sourceText) : "";
 				const nameRendered = sourceText ? `${theme.bold(nameText)} ${sourceRendered}` : theme.bold(nameText);
 				const pad = Math.max(0, maxNameWidth - visibleWidth(nameWithSourcePlain));
-				const desc = line.desc?.trim();
-				const descPart = desc ? `  ${theme.fg("dim", desc.slice(0, 50) + (desc.length > 50 ? "..." : ""))}` : "";
+				const desc = line.desc;
+				const descPart = desc
+					? `  ${theme.fg("dim", desc.slice(0, 50) + (desc.length > 50 ? "..." : ""))}`
+					: "";
 				return `  ${nameRendered}${" ".repeat(pad)}${descPart}`;
 			});
 
-			return `${theme.bold(theme.fg("accent", title))}\n${formattedLines.join("\n")}`;
+			return `${theme.bold(theme.fg("accent", section.title))}\n${formattedLines.join("\n")}`;
+		};
+
+		const sections: Section[] = [];
+		const pushLineSection = <T>(
+			title: string,
+			items: readonly T[],
+			getName: (item: T) => string,
+			getDesc: (item: T) => string | undefined,
+			getSource: (item: T) => StatusSource,
+		): void => {
+			const section = buildLineSection(title, items, getName, getDesc, getSource);
+			if (section) {
+				sections.push({ kind: "lines", section });
+			}
 		};
 
 		// Loaded context files
 		const contextFilesResult = loadSync(contextFileCapability.id, { cwd: process.cwd() });
 		const contextFiles = contextFilesResult.items as ContextFile[];
-		if (contextFiles.length > 0) {
-			sections.push(
-				formatSection(
-					"Context Files",
-					contextFiles,
-					(f) => basename(f.path),
-					() => undefined,
-					(f) => ({ provider: f._source.providerName, level: f.level }),
-				),
-			);
-		}
+		pushLineSection(
+			"Context Files",
+			contextFiles,
+			(f) => basename(f.path),
+			() => undefined,
+			(f) => ({ provider: f._source.providerName, level: f.level }),
+		);
 
 		// Loaded skills
 		const skillsSettings = this.session.skillsSettings;
 		if (skillsSettings?.enabled !== false) {
 			const { skills, warnings: skillWarnings } = loadSkills(skillsSettings ?? {});
-			if (skills.length > 0) {
-				sections.push(
-					formatSection(
-						"Skills",
-						skills,
-						(s) => s.name,
-						(s) => s.description,
-						(s) => (s._source ? { provider: s._source.providerName, level: s._source.level } : "unknown"),
-					),
-				);
-			}
+			pushLineSection(
+				"Skills",
+				skills,
+				(s) => s.name,
+				(s) => s.description,
+				(s) => (s._source ? { provider: s._source.providerName, level: s._source.level } : "unknown"),
+			);
 			if (skillWarnings.length > 0) {
 				sections.push(
-					theme.bold(theme.fg("warning", "Skill Warnings")) +
-						"\n" +
-						skillWarnings.map((w) => theme.fg("warning", `  ${w.skillPath}: ${w.message}`)).join("\n"),
+					{
+						kind: "text",
+						text:
+							theme.bold(theme.fg("warning", "Skill Warnings")) +
+							"\n" +
+							skillWarnings.map((w) => theme.fg("warning", `  ${w.skillPath}: ${w.message}`)).join("\n"),
+					},
 				);
 			}
 		}
 
 		// Loaded rules
 		const rulesResult = loadSync<Rule>(ruleCapability.id, { cwd: process.cwd() });
-		if (rulesResult.items.length > 0) {
-			sections.push(
-				formatSection(
-					"Rules",
-					rulesResult.items,
-					(r) => r.name,
-					(r) => r.description,
-					(r) => ({ provider: r._source.providerName, level: r._source.level }),
-				),
-			);
-		}
+		pushLineSection(
+			"Rules",
+			rulesResult.items,
+			(r) => r.name,
+			(r) => r.description,
+			(r) => ({ provider: r._source.providerName, level: r._source.level }),
+		);
 
 		// Loaded prompts
 		const promptsResult = loadSync<Prompt>(promptCapability.id, { cwd: process.cwd() });
-		if (promptsResult.items.length > 0) {
-			sections.push(
-				formatSection(
-					"Prompts",
-					promptsResult.items,
-					(p) => p.name,
-					() => undefined,
-					(p) => ({ provider: p._source.providerName, level: p._source.level }),
-				),
-			);
-		}
+		pushLineSection(
+			"Prompts",
+			promptsResult.items,
+			(p) => p.name,
+			() => undefined,
+			(p) => ({ provider: p._source.providerName, level: p._source.level }),
+		);
 
 		// Loaded instructions
 		const instructionsResult = loadSync<Instruction>(instructionCapability.id, { cwd: process.cwd() });
-		if (instructionsResult.items.length > 0) {
-			sections.push(
-				formatSection(
-					"Instructions",
-					instructionsResult.items,
-					(i) => i.name,
-					(i) => (i.applyTo ? `applies to: ${i.applyTo}` : undefined),
-					(i) => ({ provider: i._source.providerName, level: i._source.level }),
-				),
-			);
-		}
+		pushLineSection(
+			"Instructions",
+			instructionsResult.items,
+			(i) => i.name,
+			(i) => (i.applyTo ? `applies to: ${i.applyTo}` : undefined),
+			(i) => ({ provider: i._source.providerName, level: i._source.level }),
+		);
 
 		// Loaded custom tools - split MCP from non-MCP
 		if (this.customTools.size > 0) {
@@ -2577,55 +2586,47 @@ export class InteractiveMode {
 
 			// MCP Tools section
 			if (mcpTools.length > 0) {
-				sections.push(
-					formatSection(
-						"MCP Tools",
-						mcpTools,
-						(ct) => ct.tool.label || ct.tool.name,
-						() => undefined,
-						(ct) => {
-							const match = ct.path.match(/^mcp:(.+?) via (.+)$/);
-							if (match) {
-								const [, serverName, providerName] = match;
-								return { mcpServer: serverName, provider: providerName };
-							}
-							return ct.path.startsWith("mcp:") ? { mcpServer: ct.path.slice(4) } : "unknown";
-						},
-					),
+				pushLineSection(
+					"MCP Tools",
+					mcpTools,
+					(ct) => ct.tool.label || ct.tool.name,
+					() => undefined,
+					(ct) => {
+						const match = ct.path.match(/^mcp:(.+?) via (.+)$/);
+						if (match) {
+							const [, serverName, providerName] = match;
+							return { mcpServer: serverName, provider: providerName };
+						}
+						return ct.path.startsWith("mcp:") ? { mcpServer: ct.path.slice(4) } : "unknown";
+					},
 				);
 			}
 
 			// Custom Tools section
 			if (customTools.length > 0) {
-				sections.push(
-					formatSection(
-						"Custom Tools",
-						customTools,
-						(ct) => ct.tool.label || ct.tool.name,
-						(ct) => ct.tool.description,
-						(ct) => {
-							if (ct.source?.provider === "builtin") return "builtin";
-							if (ct.path === "<exa>") return "builtin";
-							return ct.source ? { provider: ct.source.providerName, level: ct.source.level } : "unknown";
-						},
-					),
+				pushLineSection(
+					"Custom Tools",
+					customTools,
+					(ct) => ct.tool.label || ct.tool.name,
+					(ct) => ct.tool.description,
+					(ct) => {
+						if (ct.source?.provider === "builtin") return "builtin";
+						if (ct.path === "<exa>") return "builtin";
+						return ct.source ? { provider: ct.source.providerName, level: ct.source.level } : "unknown";
+					},
 				);
 			}
 		}
 
 		// Loaded slash commands (file-based)
 		const fileCommands = this.session.fileCommands;
-		if (fileCommands.length > 0) {
-			sections.push(
-				formatSection(
-					"Slash Commands",
-					fileCommands,
-					(cmd) => `/${cmd.name}`,
-					(cmd) => cmd.description,
-					(cmd) => (cmd._source ? { provider: cmd._source.providerName, level: cmd._source.level } : "unknown"),
-				),
-			);
-		}
+		pushLineSection(
+			"Slash Commands",
+			fileCommands,
+			(cmd) => `/${cmd.name}`,
+			(cmd) => cmd.description,
+			(cmd) => (cmd._source ? { provider: cmd._source.providerName, level: cmd._source.level } : "unknown"),
+		);
 
 		// Loaded hooks
 		const hookRunner = this.session.hookRunner;
@@ -2633,12 +2634,30 @@ export class InteractiveMode {
 			const hookPaths = hookRunner.getHookPaths();
 			if (hookPaths.length > 0) {
 				sections.push(
-					`${theme.bold(theme.fg("accent", "Hooks"))}\n${hookPaths.map((p) => `  ${theme.bold(basename(p))} ${theme.fg("dim", "hook")}`).join("\n")}`,
+					{
+						kind: "text",
+						text:
+							`${theme.bold(theme.fg("accent", "Hooks"))}\n` +
+							hookPaths.map((p) => `  ${theme.bold(basename(p))} ${theme.fg("dim", "hook")}`).join("\n"),
+					},
 				);
 			}
 		}
 
-		if (sections.length === 0) {
+		const lineSections = sections.filter((section): section is { kind: "lines"; section: LineSection } => {
+			return section.kind === "lines";
+		});
+		const allLines = lineSections.flatMap((section) => section.section.lines);
+		const maxNameWidth = allLines.length
+			? Math.min(60, Math.max(...allLines.map((line) => visibleWidth(line.nameWithSource))))
+			: 0;
+		const renderedSections = sections
+			.map((section) =>
+				section.kind === "lines" ? renderLineSection(section.section, maxNameWidth) : section.text,
+			)
+			.filter((section) => section.length > 0);
+
+		if (renderedSections.length === 0) {
 			this.chatContainer.addChild(new Spacer(1));
 			this.chatContainer.addChild(new Text(theme.fg("muted", "No extensions loaded."), 1, 0));
 		} else {
@@ -2646,7 +2665,7 @@ export class InteractiveMode {
 			this.chatContainer.addChild(new DynamicBorder());
 			this.chatContainer.addChild(new Text(theme.bold(theme.fg("accent", "Loaded Extensions")), 1, 0));
 			this.chatContainer.addChild(new Spacer(1));
-			for (const section of sections) {
+			for (const section of renderedSections) {
 				this.chatContainer.addChild(new Text(section, 1, 0));
 				this.chatContainer.addChild(new Spacer(1));
 			}

package/src/modes/print-mode.ts

@@ -2,8 +2,8 @@
  * Print mode (single-shot): Send prompts, output result, exit.
  *
  * Used for:
- * - `pi -p "prompt"` - text output
- * - `pi --mode json "prompt"` - JSON event stream
+ * - `omp -p "prompt"` - text output
+ * - `omp --mode json "prompt"` - JSON event stream
  */
 
 import type { AssistantMessage, ImageContent } from "@oh-my-pi/pi-ai";