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

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "13.3.3",
+	"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.3",
-		"@oh-my-pi/pi-agent-core": "13.3.3",
-		"@oh-my-pi/pi-ai": "13.3.3",
-		"@oh-my-pi/pi-natives": "13.3.3",
-		"@oh-my-pi/pi-tui": "13.3.3",
-		"@oh-my-pi/pi-utils": "13.3.3",
+		"@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/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",
 	});