@oh-my-pi/pi-coding-agent 13.3.2 → 13.3.4

package/CHANGELOG.md

@@ -2,6 +2,15 @@
 
 ## [Unreleased]
 
+## [13.3.3] - 2026-02-26
+### Added
+
+- Support for `move` parameter in `computeHashlineDiff` to enable file move operations alongside content edits
+
+### Changed
+
+- Modified no-op detection logic to allow move-only operations when file content remains unchanged
+
 ## [13.3.1] - 2026-02-26
 
 ### Added

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "13.3.2",
+	"version": "13.3.4",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -41,12 +41,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "^0.6",
-		"@oh-my-pi/omp-stats": "13.3.2",
-		"@oh-my-pi/pi-agent-core": "13.3.2",
-		"@oh-my-pi/pi-ai": "13.3.2",
-		"@oh-my-pi/pi-natives": "13.3.2",
-		"@oh-my-pi/pi-tui": "13.3.2",
-		"@oh-my-pi/pi-utils": "13.3.2",
+		"@oh-my-pi/omp-stats": "13.3.4",
+		"@oh-my-pi/pi-agent-core": "13.3.4",
+		"@oh-my-pi/pi-ai": "13.3.4",
+		"@oh-my-pi/pi-natives": "13.3.4",
+		"@oh-my-pi/pi-tui": "13.3.4",
+		"@oh-my-pi/pi-utils": "13.3.4",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/cursor.ts

@@ -198,7 +198,7 @@ export class CursorExecHandlers implements ICursorExecHandlers {
 		const timeoutSeconds = args.timeout && args.timeout > 0 ? args.timeout : undefined;
 		const toolResultMessage = await executeTool(this.options, "bash", toolCallId, {
 			command: args.command,
-			workdir: args.workingDirectory || undefined,
+			cwd: args.workingDirectory || undefined,
 			timeout: timeoutSeconds,
 		});
 		return toolResultMessage;

package/src/modes/components/tool-execution.ts

@@ -204,12 +204,13 @@ export class ToolExecutionComponent extends Container {
 			return;
 		}
 		const edits = this.#args?.edits;
+		const move = this.#args?.move;
 		if (path && Array.isArray(edits)) {
-			const argsKey = JSON.stringify({ path, edits });
+			const argsKey = JSON.stringify({ path, edits, move });
 			if (this.#editDiffArgsKey === argsKey) return;
 			this.#editDiffArgsKey = argsKey;
 
-			computeHashlineDiff({ path, edits }, this.#cwd).then(result => {
+			computeHashlineDiff({ path, edits, move }, this.#cwd).then(result => {
 				if (this.#editDiffArgsKey === argsKey) {
 					this.#editDiffPreview = result;
 					this.#updateDisplay();

package/src/patch/diff.ts

@@ -386,10 +386,10 @@ export async function computePatchDiff(
  * Used for preview rendering in the TUI before hashline-mode edits execute.
  */
 export async function computeHashlineDiff(
-	input: { path: string; edits: HashlineEdit[] },
+	input: { path: string; edits: HashlineEdit[]; move?: string },
 	cwd: string,
 ): Promise<DiffResult | DiffError> {
-	const { path, edits } = input;
+	const { path, edits, move } = input;
 	const absolutePath = resolveToCwd(path, cwd);
 
 	try {
@@ -414,7 +414,7 @@ export async function computeHashlineDiff(
 		const normalizedContent = normalizeToLF(content);
 
 		const result = applyHashlineEdits(normalizedContent, edits);
-		if (normalizedContent === result.lines) {
+		if (normalizedContent === result.lines && !move) {
 			return { error: `No changes would be made to ${path}. The edits produce identical content.` };
 		}
 

package/src/prompts/tools/bash.md

@@ -22,7 +22,20 @@ Returns the output, and an exit code from command execution.
 </output>
 
 <critical>
+You **MUST** use specialized tools instead of bash for ALL file operations:
+
+|Instead of (WRONG)|Use (CORRECT)|
+|---|---|
+|`cat file`, `head -n N file`|`read(path="file", limit=N)`|
+|`cat -n file \|sed -n '50,150p'`|`read(path="file", offset=50, limit=100)`|
+|`grep -A 20 'pat' file`|`grep(pattern="pat", path="file", post=20)`|
+|`grep -rn 'pat' dir/`|`grep(pattern="pat", path="dir/")`|
+|`rg 'pattern' dir/`|`grep(pattern="pattern", path="dir/")`|
+|`find dir -name '*.ts'`|`find(pattern="dir/**/*.ts")`|
+|`ls dir/`|`read(path="dir/")`|
+|`cat <<'EOF' > file`|`write(path="file", content="...")`|
+|`sed -i 's/old/new/' file`|`edit(path="file", edits=[...])`|
 - You **MUST NOT** use Bash for these operations like read, grep, find, edit, write, where specialized tools exist.
-- You **MUST NOT** use `2>&1` pattern, stdout and stderr are already merged.
+- You **MUST NOT** use `2>&1` | `2>/dev/null` pattern, stdout and stderr are already merged.
 - You **MUST NOT** use `| head -n 50` or `| tail -n 100` pattern, use `head` and `tail` parameters instead.
 </critical>

package/src/prompts/tools/read.md

@@ -18,4 +18,11 @@ Reads files from local filesystem or internal URLs.
 <output>
 - Returns file content as text; images return visual content; PDFs return extracted text
 - Missing files: returns closest filename matches for correction
-</output>
+</output>
+
+<critical>
+- You **MUST** use `read` instead of bash for ALL file reading: `cat`, `head`, `tail`, `less`, `more` are FORBIDDEN.
+- You **MUST** use `read(path="dir/")` instead of `ls dir/` for directory listings.
+- You **MUST** always include the `path` parameter — NEVER call `read` with empty arguments `{}`.
+- When reading specific line ranges, use `offset` and `limit`: `read(path="file", offset=50, limit=100)` not `cat -n file | sed`.
+</critical>

package/src/ssh/connection-manager.ts

@@ -92,12 +92,12 @@ function buildCommonArgs(host: SSHConnectionTarget): string[] {
 }
 
 async function runSshSync(args: string[]): Promise<{ exitCode: number | null; stderr: string }> {
-	const result = await $`ssh ${args}`.nothrow();
+	const result = await $`ssh ${args}`.quiet().nothrow();
 	return { exitCode: result.exitCode, stderr: result.stderr.toString().trim() };
 }
 
 async function runSshCaptureSync(args: string[]): Promise<{ exitCode: number | null; stdout: string; stderr: string }> {
-	const result = await $`ssh ${args}`.nothrow();
+	const result = await $`ssh ${args}`.quiet().nothrow();
 	return {
 		exitCode: result.exitCode,
 		stdout: result.stdout.toString().trim(),

package/src/ssh/ssh-executor.ts

@@ -79,6 +79,7 @@ export async function executeSSH(
 	using child = ptree.spawn(["ssh", ...(await buildRemoteCommand(host, resolvedCommand))], {
 		signal: options?.signal,
 		timeout: options?.timeout,
+		stdin: "pipe",
 		stderr: "full",
 	});