@oh-my-pi/pi-coding-agent 12.12.2 → 12.13.0

package/CHANGELOG.md

@@ -2,6 +2,36 @@
 
 ## [Unreleased]
 
+## [12.13.0] - 2026-02-19
+### Breaking Changes
+
+- Removed automatic line relocation when hash references become stale; edits with mismatched line hashes now fail with an error instead of silently relocating to matching lines elsewhere in the file
+
+### Added
+
+- Added `ssh` command for managing SSH host configurations (add, list, remove)
+- Added `/ssh` slash command in interactive mode to manage SSH hosts with subcommands
+- Added support for SSH host configuration at project and user scopes (.omp/ssh.json and ~/.omp/agent/ssh.json)
+- Added `--host`, `--user`, `--port`, `--key`, `--desc`, `--compat`, and `--scope` flags for SSH host configuration
+- Added discovery of SSH hosts from project configuration files alongside manually configured hosts
+- Added NanoGPT as a login provider (`/login nanogpt`) with API key prompt flow linking to `https://nano-gpt.com/api` ([#111](https://github.com/can1357/oh-my-pi/issues/111))
+
+### Changed
+
+- Updated hashline reference format from `LINE:HASH` to `LINE#ID` throughout the codebase for improved clarity
+- Renamed hashline edit operations: `set_line` → `set`, `replace_lines` → `set_range`, `insert_after` → `insert` with support for `before` and `between` anchors
+- Changed hashline edit `body` field from string to array of strings for clearer multiline handling
+- Updated handlebars helpers: renamed `hashline` to `hlineref` and added `hlinefull` for formatted line output
+- Improved insert operation to support `before`, `after`, and `between` (both anchors) positioning modes
+- Made autocorrect heuristics (boundary echo stripping, indent restoration) conditional on `PI_HL_AUTOCORRECT` environment variable
+- Updated SSH host discovery to load from managed omp config paths (.omp/ssh.json and ~/.omp/agent/ssh.json) in addition to legacy root-level ssh.json and .ssh.json files
+- Improved terminal output handling in interactive bash sessions to ensure all queued writes complete before returning results
+
+### Fixed
+
+- Fixed insert-between operation to properly validate adjacent anchor lines and strip boundary echoes from both sides
+- Fixed terminal output handling to properly queue and serialize writes, preventing dropped or corrupted output in interactive bash sessions
+
 ## [12.12.1] - 2026-02-19
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "12.12.2",
+	"version": "12.13.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"bin": {
@@ -84,12 +84,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "0.6.0",
-		"@oh-my-pi/omp-stats": "12.12.2",
-		"@oh-my-pi/pi-agent-core": "12.12.2",
-		"@oh-my-pi/pi-ai": "12.12.2",
-		"@oh-my-pi/pi-natives": "12.12.2",
-		"@oh-my-pi/pi-tui": "12.12.2",
-		"@oh-my-pi/pi-utils": "12.12.2",
+		"@oh-my-pi/omp-stats": "12.13.0",
+		"@oh-my-pi/pi-agent-core": "12.13.0",
+		"@oh-my-pi/pi-ai": "12.13.0",
+		"@oh-my-pi/pi-natives": "12.13.0",
+		"@oh-my-pi/pi-tui": "12.13.0",
+		"@oh-my-pi/pi-utils": "12.13.0",
 		"@sinclair/typebox": "^0.34.48",
 		"@xterm/headless": "^6.0.0",
 		"ajv": "^8.18.0",

package/src/cli/ssh-cli.ts

@@ -0,0 +1,179 @@
+/**
+ * SSH CLI command handlers.
+ *
+ * Handles `omp ssh <command>` subcommands for SSH host configuration management.
+ */
+
+import { getSSHConfigPath } from "@oh-my-pi/pi-utils/dirs";
+import chalk from "chalk";
+import { addSSHHost, readSSHConfigFile, removeSSHHost, type SSHHostConfig } from "../ssh/config-writer";
+
+// =============================================================================
+// Types
+// =============================================================================
+
+export type SSHAction = "add" | "remove" | "list";
+
+export interface SSHCommandArgs {
+	action: SSHAction;
+	args: string[];
+	flags: {
+		json?: boolean;
+		host?: string;
+		user?: string;
+		port?: string;
+		key?: string;
+		desc?: string;
+		compat?: boolean;
+		scope?: "project" | "user";
+	};
+}
+
+// =============================================================================
+// Main dispatcher
+// =============================================================================
+
+export async function runSSHCommand(cmd: SSHCommandArgs): Promise<void> {
+	switch (cmd.action) {
+		case "add":
+			await handleAdd(cmd);
+			break;
+		case "remove":
+			await handleRemove(cmd);
+			break;
+		case "list":
+			await handleList(cmd);
+			break;
+		default:
+			process.stdout.write(chalk.red(`Unknown action: ${cmd.action}\n`));
+			process.stdout.write(`Valid actions: add, remove, list\n`);
+			process.exitCode = 1;
+	}
+}
+
+// =============================================================================
+// Handlers
+// =============================================================================
+
+async function handleAdd(cmd: SSHCommandArgs): Promise<void> {
+	const name = cmd.args[0];
+	if (!name) {
+		process.stdout.write(chalk.red("Error: Host name required\n"));
+		process.stdout.write(
+			chalk.dim("Usage: omp ssh add <name> --host <address> [--user <user>] [--port <port>] [--key <path>]\n"),
+		);
+		process.exitCode = 1;
+		return;
+	}
+
+	const host = cmd.flags.host;
+	if (!host) {
+		process.stdout.write(chalk.red("Error: --host is required\n"));
+		process.stdout.write(chalk.dim("Usage: omp ssh add <name> --host <address>\n"));
+		process.exitCode = 1;
+		return;
+	}
+
+	// Validate port if provided
+	if (cmd.flags.port !== undefined) {
+		const port = Number.parseInt(cmd.flags.port, 10);
+		if (Number.isNaN(port) || port < 1 || port > 65535) {
+			process.stdout.write(chalk.red("Error: Port must be an integer between 1 and 65535\n"));
+			process.exitCode = 1;
+			return;
+		}
+	}
+
+	const hostConfig: SSHHostConfig = { host };
+	if (cmd.flags.user) hostConfig.username = cmd.flags.user;
+	if (cmd.flags.port) hostConfig.port = Number.parseInt(cmd.flags.port, 10);
+	if (cmd.flags.key) hostConfig.keyPath = cmd.flags.key;
+	if (cmd.flags.desc) hostConfig.description = cmd.flags.desc;
+	if (cmd.flags.compat) hostConfig.compat = true;
+
+	const scope = cmd.flags.scope ?? "project";
+	const filePath = getSSHConfigPath(scope);
+
+	try {
+		await addSSHHost(filePath, name, hostConfig);
+		process.stdout.write(chalk.green(`Added SSH host "${name}" to ${scope} config\n`));
+	} catch (err) {
+		process.stdout.write(chalk.red(`Error: ${err instanceof Error ? err.message : String(err)}\n`));
+		process.exitCode = 1;
+	}
+}
+
+async function handleRemove(cmd: SSHCommandArgs): Promise<void> {
+	const name = cmd.args[0];
+	if (!name) {
+		process.stdout.write(chalk.red("Error: Host name required\n"));
+		process.stdout.write(chalk.dim("Usage: omp ssh remove <name> [--scope project|user]\n"));
+		process.exitCode = 1;
+		return;
+	}
+
+	const scope = cmd.flags.scope ?? "project";
+	const filePath = getSSHConfigPath(scope);
+
+	try {
+		await removeSSHHost(filePath, name);
+		process.stdout.write(chalk.green(`Removed SSH host "${name}" from ${scope} config\n`));
+	} catch (err) {
+		process.stdout.write(chalk.red(`Error: ${err instanceof Error ? err.message : String(err)}\n`));
+		process.exitCode = 1;
+	}
+}
+
+async function handleList(cmd: SSHCommandArgs): Promise<void> {
+	const projectPath = getSSHConfigPath("project");
+	const userPath = getSSHConfigPath("user");
+
+	const [projectConfig, userConfig] = await Promise.all([readSSHConfigFile(projectPath), readSSHConfigFile(userPath)]);
+
+	const projectHosts = projectConfig.hosts ?? {};
+	const userHosts = userConfig.hosts ?? {};
+
+	if (cmd.flags.json) {
+		process.stdout.write(JSON.stringify({ project: projectHosts, user: userHosts }, null, 2));
+		process.stdout.write("\n");
+		return;
+	}
+
+	const hasProject = Object.keys(projectHosts).length > 0;
+	const hasUser = Object.keys(userHosts).length > 0;
+
+	if (!hasProject && !hasUser) {
+		process.stdout.write(chalk.dim("No SSH hosts configured\n"));
+		process.stdout.write(chalk.dim("Add one with: omp ssh add <name> --host <address>\n"));
+		return;
+	}
+
+	if (hasProject) {
+		process.stdout.write(chalk.bold("Project SSH Hosts (.omp/ssh.json):\n"));
+		printHosts(projectHosts);
+	}
+
+	if (hasProject && hasUser) {
+		process.stdout.write("\n");
+	}
+
+	if (hasUser) {
+		process.stdout.write(chalk.bold("User SSH Hosts (~/.omp/agent/ssh.json):\n"));
+		printHosts(userHosts);
+	}
+}
+
+// =============================================================================
+// Helpers
+// =============================================================================
+
+function printHosts(hosts: Record<string, SSHHostConfig>): void {
+	for (const [name, config] of Object.entries(hosts)) {
+		const parts = [chalk.cyan(name), config.host];
+		if (config.username) parts.push(chalk.dim(config.username));
+		if (config.port && config.port !== 22) parts.push(chalk.dim(`port:${config.port}`));
+		if (config.keyPath) parts.push(chalk.dim(config.keyPath));
+		if (config.description) parts.push(chalk.dim(`- ${config.description}`));
+		process.stdout.write(`  ${parts.join("  ")}\n`);
+	}
+}

package/src/cli.ts

@@ -23,6 +23,7 @@ const commands: CommandEntry[] = [
 	{ name: "plugin", load: () => import("./commands/plugin").then(m => m.default) },
 	{ name: "setup", load: () => import("./commands/setup").then(m => m.default) },
 	{ name: "shell", load: () => import("./commands/shell").then(m => m.default) },
+	{ name: "ssh", load: () => import("./commands/ssh").then(m => m.default) },
 	{ name: "stats", load: () => import("./commands/stats").then(m => m.default) },
 	{ name: "update", load: () => import("./commands/update").then(m => m.default) },
 	{ name: "search", load: () => import("./commands/web-search").then(m => m.default), aliases: ["q"] },

package/src/commands/ssh.ts

@@ -0,0 +1,60 @@
+/**
+ * Manage SSH host configurations.
+ */
+import { Args, Command, Flags } from "@oh-my-pi/pi-utils/cli";
+import { runSSHCommand, type SSHAction, type SSHCommandArgs } from "../cli/ssh-cli";
+import { initTheme } from "../modes/theme/theme";
+
+const ACTIONS: SSHAction[] = ["add", "remove", "list"];
+
+export default class SSH extends Command {
+	static description = "Manage SSH host configurations";
+
+	static args = {
+		action: Args.string({
+			description: "SSH action",
+			required: false,
+			options: ACTIONS,
+		}),
+		targets: Args.string({
+			description: "Host name or arguments",
+			required: false,
+			multiple: true,
+		}),
+	};
+
+	static flags = {
+		json: Flags.boolean({ description: "Output JSON" }),
+		host: Flags.string({ description: "Host address" }),
+		user: Flags.string({ description: "Username" }),
+		port: Flags.string({ description: "Port number" }),
+		key: Flags.string({ description: "Identity key path" }),
+		desc: Flags.string({ description: "Host description" }),
+		compat: Flags.boolean({ description: "Enable compatibility mode" }),
+		scope: Flags.string({ description: "Config scope (project|user)", options: ["project", "user"] }),
+	};
+
+	async run(): Promise<void> {
+		const { args, flags } = await this.parse(SSH);
+		const action = (args.action ?? "list") as SSHAction;
+		const targets = Array.isArray(args.targets) ? args.targets : args.targets ? [args.targets] : [];
+
+		const cmd: SSHCommandArgs = {
+			action,
+			args: targets,
+			flags: {
+				json: flags.json,
+				host: flags.host,
+				user: flags.user,
+				port: flags.port,
+				key: flags.key,
+				desc: flags.desc,
+				compat: flags.compat,
+				scope: flags.scope as "project" | "user" | undefined,
+			},
+		};
+
+		await initTheme();
+		await runSSHCommand(cmd);
+	}
+}

package/src/commit/prompts/analysis-system.md

@@ -76,7 +76,6 @@ Call create_conventional_analysis with:
 }
 </output_format>
 
-<examples>
 <example name="feature-with-api">
 {
   "type": "feat",
@@ -146,5 +145,4 @@ Call create_conventional_analysis with:
   "details": [],
   "issue_refs": []
 }
-</example>
-</examples>
+</example>

package/src/config/prompt-templates.ts

@@ -230,13 +230,28 @@ handlebars.registerHelper("jtdToTypeScript", (schema: unknown): string => jtdToT
 handlebars.registerHelper("jsonStringify", (value: unknown): string => JSON.stringify(value));
 
 /**
- * {{hashline lineNum "content"}} — compute a real hashline ref for prompt examples.
- * Returns `"lineNum:hash"` using the actual hash algorithm.
+ * {{hlineref lineNum "content"}} — compute a real hashline ref for prompt examples.
+ * Returns `"lineNum#hash"` using the actual hash algorithm.
  */
-handlebars.registerHelper("hashline", (lineNum: unknown, content: unknown): string => {
+function formatHashlineRef(lineNum: unknown, content: unknown): { num: number; text: string; ref: string } {
 	const num = typeof lineNum === "number" ? lineNum : Number.parseInt(String(lineNum), 10);
-	const str = typeof content === "string" ? content : String(content ?? "");
-	return `${num}:${computeLineHash(num, str)}`;
+	const text = typeof content === "string" ? content : String(content ?? "");
+	const ref = `${num}#${computeLineHash(num, text)}`;
+	return { num, text, ref };
+}
+
+handlebars.registerHelper("hlineref", (lineNum: unknown, content: unknown): string => {
+	const { ref } = formatHashlineRef(lineNum, content);
+	return ref;
+});
+
+/**
+ * {{hlinefull lineNum "content"}} — format a full read-style line with prefix.
+ * Returns `"lineNum#hash|content"`.
+ */
+handlebars.registerHelper("hlinefull", (lineNum: unknown, content: unknown): string => {
+	const { ref, text } = formatHashlineRef(lineNum, content);
+	return `${ref}|${text}`;
 });
 
 export function renderPromptTemplate(template: string, context: TemplateContext = {}): string {

package/src/config/settings-schema.ts

@@ -287,7 +287,7 @@ export const SETTINGS_SCHEMA = {
 		ui: {
 			tab: "config",
 			label: "Read hash lines",
-			description: "Include line hashes in read output for hashline edit mode (LINE:HASH|content)",
+			description: "Include line hashes in read output for hashline edit mode (LINE#ID|content)",
 		},
 	},
 	showHardwareCursor: {

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,
 });