pi-rtk-optimizer 0.5.5 → 0.6.0

package/CHANGELOG.md

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.6.0] - 2026-04-27
+
+### Changed
+- **Breaking:** Command rewriting now delegates rewrite decisions to the installed `rtk rewrite` command, making RTK the source of truth for command support, shell parsing, bypasses, and compound-command behavior instead of the extension's local rewrite rule tables.
+
+### Removed
+- **Breaking:** Removed the rewrite category configuration surface (`rewriteGitGithub`, `rewriteFilesystem`, `rewriteRust`, `rewriteJavaScript`, `rewritePython`, `rewriteGo`, `rewriteContainers`, `rewriteNetwork`, and `rewritePackageManagers`) from configuration normalization, examples, settings UI, and documentation. Configure rewrite policy in RTK itself instead of this extension.
+
 ## [0.5.5] - 2026-04-24
 
 ### Changed

package/README.md

@@ -1,5 +1,7 @@
 # pi-rtk-optimizer
 
+[![npm version](https://img.shields.io/npm/v/pi-rtk-optimizer?style=flat-square)](https://www.npmjs.com/package/pi-rtk-optimizer) [![License](https://img.shields.io/github/license/MasuRii/pi-rtk-optimizer?style=flat-square)](LICENSE)
+
 > RTK command rewriting and tool output compaction extension for the Pi coding agent.
 
 <img width="1360" height="752" alt="image" src="https://github.com/user-attachments/assets/f4536889-62ec-429a-984e-dc0de9f1f709" />
@@ -12,19 +14,9 @@
 ### Command Rewriting
 
 - **Automatic rewriting** or **suggestion-only** mode for common development workflows
-- Supported command categories:
-  - **Git/GitHub** — `git`, `gh` commands
-  - **Filesystem** — `cat`, `head`, `tail`, `grep`, `rg`, `ls`, `tree`, `find`, `diff`, `wc`, `bash`, `cmd`, `powershell`
-  - **Rust** — `cargo` commands
-  - **JavaScript** — `vitest`, `npm`, `yarn`, `pnpm`, `bun` commands
-  - **Python** — `pytest`, `python`, `pip`, `uv` commands
-  - **Go** — `go` commands
-  - **Containers** — `docker`, `docker-compose`, `podman` commands
-  - **Network** — `curl`, `wget` commands
-  - **Package Managers** — `apt`, `brew`, `dnf`, `pacman`, `yum` commands
-- Runtime guard when `rtk` binary is unavailable (falls back to original commands only in rewrite mode)
-- Safe rewrite bypasses for structured `gh` output commands and non-interactive container shell sessions
-- Improved command parsing for `sed`, shell separators, and `pnpm dlx` proxy rewrites
+- Delegates bash command rewrite decisions to the installed `rtk rewrite` command, keeping RTK as the source of truth for supported commands, shell parsing, bypasses, and compound-command behavior
+- Runtime guard when `rtk` binary is unavailable (raw commands run unchanged and repeated missing-binary rewrite probes are avoided)
+- Pi-specific shell safety fixups for rewritten commands on Windows
 
 ### Output Compaction Pipeline
 
@@ -126,19 +118,11 @@ A starter template is included at `config/config.example.json`.
 | `guardWhenRtkMissing` | boolean | `true` | Run original commands when rtk binary unavailable |
 | `showRewriteNotifications` | boolean | `true` | Show rewrite notices in TUI |
 
-#### Rewrite Category Toggles
+#### Rewrite Source
+
+Bash command support is intentionally resolved by the installed `rtk` binary through `rtk rewrite`. The extension does not maintain duplicate rewrite rules or category classifiers; update/configure RTK itself for command support policy.
 
-| Option | Default | Commands Affected |
-|--------|---------|-------------------|
-| `rewriteGitGithub` | `true` | `git`, `gh` |
-| `rewriteFilesystem` | `true` | `cat`, `head`, `tail`, `grep`, `rg`, `ls`, `tree`, `find`, `diff`, `wc` |
-| `rewriteRust` | `true` | `cargo` |
-| `rewriteJavaScript` | `true` | `vitest`, `npm`, `yarn`, `pnpm`, `bun` |
-| `rewritePython` | `true` | `pytest`, `python`, `pip`, `uv` |
-| `rewriteGo` | `true` | `go` |
-| `rewriteContainers` | `true` | `docker`, `docker-compose`, `podman` |
-| `rewriteNetwork` | `true` | `curl`, `wget` |
-| `rewritePackageManagers` | `true` | `apt`, `brew`, `dnf`, `pacman`, `yum` |
+> **Breaking in 0.6.0:** Rewrite category toggles (`rewriteGitGithub`, `rewriteFilesystem`, `rewriteRust`, `rewriteJavaScript`, `rewritePython`, `rewriteGo`, `rewriteContainers`, `rewriteNetwork`, and `rewritePackageManagers`) were removed from the extension config surface. Existing rewrite policy should be configured in RTK because the extension now delegates rewrite ownership to `rtk rewrite`.
 
 #### Output Compaction Settings
 
@@ -185,15 +169,6 @@ Skill-read preservation covers the global Pi skills directory (`~/.pi/agent/skil
   "mode": "rewrite",
   "guardWhenRtkMissing": true,
   "showRewriteNotifications": true,
-  "rewriteGitGithub": true,
-  "rewriteFilesystem": true,
-  "rewriteRust": true,
-  "rewriteJavaScript": true,
-  "rewritePython": true,
-  "rewriteGo": true,
-  "rewriteContainers": true,
-  "rewriteNetwork": true,
-  "rewritePackageManagers": true,
   "outputCompaction": {
     "enabled": true,
     "stripAnsi": true,

package/config/config.example.json

@@ -3,15 +3,6 @@
   "mode": "rewrite",
   "guardWhenRtkMissing": true,
   "showRewriteNotifications": true,
-  "rewriteGitGithub": true,
-  "rewriteFilesystem": true,
-  "rewriteRust": true,
-  "rewriteJavaScript": true,
-  "rewritePython": true,
-  "rewriteGo": true,
-  "rewriteContainers": true,
-  "rewriteNetwork": true,
-  "rewritePackageManagers": true,
   "outputCompaction": {
     "enabled": true,
     "stripAnsi": true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-rtk-optimizer",
-  "version": "0.5.5",
+  "version": "0.6.0",
   "description": "Pi extension that optimizes RTK command rewriting and tool output compaction for the coding agent.",
   "type": "module",
   "main": "./index.ts",
@@ -58,7 +58,7 @@
     ]
   },
   "peerDependencies": {
-    "@mariozechner/pi-coding-agent": "^0.70.0",
-    "@mariozechner/pi-tui": "^0.70.0"
+    "@mariozechner/pi-coding-agent": "^0.70.2",
+    "@mariozechner/pi-tui": "^0.70.2"
   }
 }

package/src/additional-coverage-test.ts

@@ -64,7 +64,7 @@ runTest("config-store normalizes invalid values and clamps numeric ranges", () =
 
 	assert.equal(normalized.enabled, true);
 	assert.equal(normalized.mode, "rewrite");
-	assert.equal(normalized.rewriteGitGithub, false);
+	assert.equal(Object.hasOwn(normalized, "rewriteGitGithub"), false);
 	assert.equal(normalized.outputCompaction.stripAnsi, false);
 	assert.equal(normalized.outputCompaction.sourceCodeFilteringEnabled, true);
 	assert.equal(normalized.outputCompaction.sourceCodeFiltering, "minimal");

package/src/command-rewriter-test.ts

@@ -1,160 +1,118 @@
-import assert from "node:assert/strict";
-
-import { computeRewriteDecision } from "./command-rewriter.ts";
-import { cloneDefaultConfig, runTest } from "./test-helpers.ts";
-
-function expectedProxyExecutable(base: string): string {
-	const windowsExecutables = new Set(["npm", "npx", "pnpm", "yarn"]);
-	return process.platform === "win32" && windowsExecutables.has(base) ? `${base}.cmd` : base;
-}
-
-runTest("pnpm dlx rewrites through RTK proxy instead of generic pnpm wrapper", () => {
-	const config = cloneDefaultConfig();
-	const decision = computeRewriteDecision("pnpm dlx create-vite@latest demo --template react-ts", config);
-
-	assert.equal(decision.changed, true);
-	assert.equal(decision.rule?.id, "pnpm-dlx-proxy");
-	assert.equal(
-		decision.rewrittenCommand,
-		`rtk proxy ${expectedProxyExecutable("pnpm")} dlx create-vite@latest demo --template react-ts`,
-	);
-});
-
-runTest("docker run with shell command bypasses rewrite when interactive flags are missing", () => {
-	const config = cloneDefaultConfig();
-	const decision = computeRewriteDecision("docker run ubuntu bash", config);
-
-	assert.equal(decision.changed, false);
-	assert.equal(decision.rewrittenCommand, "docker run ubuntu bash");
-	assert.equal(decision.reason, "no_match");
-});
-
-runTest("docker run with -it keeps container rewrite enabled", () => {
-	const config = cloneDefaultConfig();
-	const decision = computeRewriteDecision("docker run -it ubuntu bash", config);
-
-	assert.equal(decision.changed, true);
-	assert.equal(decision.rule?.id, "docker");
-	assert.equal(decision.rewrittenCommand, "rtk docker run -it ubuntu bash");
-});
-
-runTest("docker compose exec without -it bypasses interactive shell rewrite", () => {
-	const config = cloneDefaultConfig();
-	const decision = computeRewriteDecision("docker compose exec web bash", config);
-
-	assert.equal(decision.changed, false);
-	assert.equal(decision.rewrittenCommand, "docker compose exec web bash");
-});
-
-runTest("container rewrites stay enabled for scripted shells and non-shell commands", () => {
-	const config = cloneDefaultConfig();
-
-	const scriptedShell = computeRewriteDecision('docker run ubuntu bash -lc "echo hi"', config);
-	assert.equal(scriptedShell.changed, true);
-	assert.equal(scriptedShell.rule?.id, "docker");
-	assert.equal(scriptedShell.rewrittenCommand, 'rtk docker run ubuntu bash -lc "echo hi"');
-
-	const nonShell = computeRewriteDecision("docker run ubuntu python app.py", config);
-	assert.equal(nonShell.changed, true);
-	assert.equal(nonShell.rule?.id, "docker");
-	assert.equal(nonShell.rewrittenCommand, "rtk docker run ubuntu python app.py");
-});
-
-runTest("kubectl exec requires interactive flags before rewriting shell sessions", () => {
-	const config = cloneDefaultConfig();
-	const missingFlagsDecision = computeRewriteDecision("kubectl exec pod-123 -- bash", config);
-	assert.equal(missingFlagsDecision.changed, false);
-	assert.equal(missingFlagsDecision.rewrittenCommand, "kubectl exec pod-123 -- bash");
-
-	const interactiveDecision = computeRewriteDecision("kubectl exec -it pod-123 -- bash", config);
-	assert.equal(interactiveDecision.changed, true);
-	assert.equal(interactiveDecision.rule?.id, "kubectl");
-	assert.equal(interactiveDecision.rewrittenCommand, "rtk kubectl exec -it pod-123 -- bash");
-});
-
-runTest("sed scripts keep internal separators intact while later pipe segments still rewrite", () => {
-	const config = cloneDefaultConfig();
-	const decision = computeRewriteDecision("sed -e s/a/b/;d file.txt | git status", config);
-
-	assert.equal(decision.changed, true);
-	assert.equal(decision.rewrittenCommand, "sed -e s/a/b/;d file.txt | rtk git status");
-	assert.equal(decision.rule?.id, "git-any");
-});
-
-runTest("background operators rewrite both command segments without misreading redirect ampersands", () => {
-	const config = cloneDefaultConfig();
-	const backgroundDecision = computeRewriteDecision("git status & cargo test", config);
-	assert.equal(backgroundDecision.changed, true);
-	assert.equal(backgroundDecision.rewrittenCommand, "rtk git status & rtk cargo test");
-
-	const redirectDecision = computeRewriteDecision("cargo test 2>&1 | head -5", config);
-	assert.equal(redirectDecision.changed, true);
-	assert.equal(redirectDecision.rewrittenCommand, "rtk cargo test 2>&1 | head -5");
-	assert.equal(redirectDecision.rule?.id, "cargo-any");
-});
-
-runTest("gh structured output commands bypass RTK rewrites", () => {
-	const config = cloneDefaultConfig();
-	const structuredCommands = [
-		"gh pr list --json number,title",
-		"gh issue list --jq '.[].title'",
-		"gh pr view 123 --template '{{.title}}'",
-	];
-
-	for (const command of structuredCommands) {
-		const decision = computeRewriteDecision(command, config);
-		assert.equal(decision.changed, false);
-		assert.equal(decision.rewrittenCommand, command);
-		assert.equal(decision.reason, "no_match");
-	}
-});
-
-runTest("compound find and search shell flows bypass rewrites when RTK parity is unsafe", () => {
-	const config = cloneDefaultConfig();
-	const auditedCommands = [
-		"find src -type f | xargs grep todo",
-		"find . -name '*.ts' -exec grep -n FIXME {} +",
-		"grep -R TODO src | head -n 5",
-		"rg TODO src && wc -l",
-	];
-
-	for (const command of auditedCommands) {
-		const decision = computeRewriteDecision(command, config);
-		assert.equal(decision.changed, false, command);
-		assert.equal(decision.rewrittenCommand, command, command);
-		assert.equal(decision.reason, "no_match", command);
-	}
-});
-
-runTest("formatting-sensitive ls flows bypass rewrites", () => {
-	const config = cloneDefaultConfig();
-	const auditedCommands = ["ls -la", "ls -l src | grep command-rewriter"];
-
-	for (const command of auditedCommands) {
-		const decision = computeRewriteDecision(command, config);
-		assert.equal(decision.changed, false, command);
-		assert.equal(decision.rewrittenCommand, command, command);
-		assert.equal(decision.reason, "no_match", command);
-	}
-});
-
-runTest("native bash shell proxy flows bypass rewrites when RTK parity is unsafe", () => {
-	const config = cloneDefaultConfig();
-	const command = 'bash -lc "find src -type f | xargs grep todo"';
-	const decision = computeRewriteDecision(command, config);
-
-	assert.equal(decision.changed, false);
-	assert.equal(decision.rewrittenCommand, command);
-	assert.equal(decision.reason, "no_match");
-});
-
-runTest("python proxy rewrites preserve the original executable token", () => {
-	const config = cloneDefaultConfig();
-	const decision = computeRewriteDecision('python3.11 -c "print(1)"', config);
-
-	assert.equal(decision.changed, true);
-	assert.equal(decision.rule?.id, "python-proxy");
-	assert.equal(decision.rewrittenCommand, `rtk proxy ${expectedProxyExecutable("python3.11")} -c "print(1)"`);
-});
-
-console.log("All command-rewriter tests passed.");
+import assert from "node:assert/strict";
+
+import { computeRewriteDecision } from "./command-rewriter.ts";
+import { cloneDefaultConfig, runTest } from "./test-helpers.ts";
+import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
+
+function createMockPi(execResult: { code: number; stdout?: string; stderr?: string }): ExtensionAPI {
+	return { exec: async () => execResult } as unknown as ExtensionAPI;
+}
+
+runTest("empty command unchanged", async () => {
+	const config = cloneDefaultConfig();
+	const decision = await computeRewriteDecision("", config, createMockPi({ code: 1 }));
+	assert.equal(decision.changed, false);
+	assert.equal(decision.reason, "empty");
+});
+
+runTest("already rtk unchanged", async () => {
+	const config = cloneDefaultConfig();
+	const decision = await computeRewriteDecision("rtk status", config, createMockPi({ code: 1 }));
+	assert.equal(decision.changed, false);
+	assert.equal(decision.reason, "already_rtk");
+});
+
+runTest("rtk unsupported heredoc result leaves command unchanged", async () => {
+	const config = cloneDefaultConfig();
+	const decision = await computeRewriteDecision("cat <<EOF", config, createMockPi({ code: 1 }));
+	assert.equal(decision.changed, false);
+	assert.equal(decision.reason, "no_match");
+});
+
+runTest("quoted heredoc marker is delegated to RTK rewrite", async () => {
+	const config = cloneDefaultConfig();
+	const command = 'echo "<<not heredoc" && git status';
+	const decision = await computeRewriteDecision(
+		command,
+		config,
+		createMockPi({ code: 3, stdout: 'echo "<<not heredoc" && rtk git status' }),
+	);
+	assert.equal(decision.changed, true);
+	assert.equal(decision.rewrittenCommand, 'echo "<<not heredoc" && rtk git status');
+	assert.equal(decision.reason, "ok");
+});
+
+runTest("legacy category toggles do not pre-filter RTK rewrite source of truth", async () => {
+	const config = { ...cloneDefaultConfig(), rewriteGitGithub: false };
+	const decision = await computeRewriteDecision("git status", config, createMockPi({ code: 3, stdout: "rtk git status" }));
+	assert.equal(decision.changed, true);
+	assert.equal(decision.rewrittenCommand, "rtk git status");
+	assert.equal(decision.reason, "ok");
+});
+
+runTest("rtk exit 0 rewrites", async () => {
+	const config = cloneDefaultConfig();
+	const decision = await computeRewriteDecision("git status", config, createMockPi({ code: 0, stdout: "rtk git status" }));
+	assert.equal(decision.changed, true);
+	assert.equal(decision.rewrittenCommand, "rtk git status");
+	assert.equal(decision.reason, "ok");
+});
+
+runTest("rtk exit 3 rewrites", async () => {
+	const config = cloneDefaultConfig();
+	const decision = await computeRewriteDecision("git status", config, createMockPi({ code: 3, stdout: "rtk git status" }));
+	assert.equal(decision.changed, true);
+	assert.equal(decision.rewrittenCommand, "rtk git status");
+	assert.equal(decision.reason, "ok");
+});
+
+runTest("exit 1 leaves unchanged", async () => {
+	const config = cloneDefaultConfig();
+	const decision = await computeRewriteDecision("git status", config, createMockPi({ code: 1 }));
+	assert.equal(decision.changed, false);
+	assert.equal(decision.reason, "no_match");
+});
+
+runTest("exit 2 leaves unchanged", async () => {
+	const config = cloneDefaultConfig();
+	const decision = await computeRewriteDecision("git status", config, createMockPi({ code: 2, stderr: "denied" }));
+	assert.equal(decision.changed, false);
+	assert.equal(decision.reason, "no_match");
+});
+
+runTest("unknown category passes through to RTK", async () => {
+	const config = cloneDefaultConfig();
+	const pi = createMockPi({ code: 0, stdout: "rtk custom" });
+	const decision = await computeRewriteDecision("custom-cmd", config, pi);
+	assert.equal(decision.changed, true);
+	assert.equal(decision.rewrittenCommand, "rtk custom");
+	assert.equal(decision.reason, "ok");
+});
+
+runTest("exec error/timeout leaves unchanged", async () => {
+	const config = cloneDefaultConfig();
+	const pi = {
+		exec: async () => {
+			throw new Error("timeout");
+		},
+	} as unknown as ExtensionAPI;
+	const decision = await computeRewriteDecision("git status", config, pi);
+	assert.equal(decision.changed, false);
+	assert.equal(decision.reason, "no_match");
+});
+
+runTest("compound commands forwarded to RTK", async () => {
+	const config = cloneDefaultConfig();
+	let capturedArgs: string[] = [];
+	const pi = {
+		exec: async (_cmd: string, args: string[]) => {
+			capturedArgs = args;
+			return { code: 0, stdout: "rtk result" };
+		},
+	} as unknown as ExtensionAPI;
+	const decision = await computeRewriteDecision("git status && cargo test", config, pi);
+	assert.equal(decision.changed, true);
+	assert.deepEqual(capturedArgs, ["rewrite", "git status && cargo test"]);
+});
+
+console.log("All command-rewriter tests passed.");