pi-rtk-optimizer 0.6.0 → 0.7.0

package/CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.7.0] - 2026-04-30
+
+### Added
+- Added opt-in `readCompaction` controls for `read` output so lossy source filtering and smart truncation stay disabled unless explicitly enabled.
+
+### Changed
+- Updated README and example configuration defaults for safer read-compaction behavior and troubleshooting guidance.
+- Updated `@mariozechner/pi-coding-agent` and `@mariozechner/pi-tui` peer dependencies to ^0.70.6.
+
 ## [0.6.0] - 2026-04-27
 
 ### Changed

package/README.md

@@ -130,9 +130,10 @@ Bash command support is intentionally resolved by the installed `rtk` binary thr
 |--------|------|---------|-------------|
 | `outputCompaction.enabled` | boolean | `true` | Enable output compaction pipeline |
 | `outputCompaction.stripAnsi` | boolean | `true` | Remove ANSI escape codes |
-| `outputCompaction.sourceCodeFilteringEnabled` | boolean | `true` | Enable source code filtering for `read` output |
+| `outputCompaction.readCompaction.enabled` | boolean | `false` | Enable lossy compaction for `read` output; defaults off so code reads stay exact |
+| `outputCompaction.sourceCodeFilteringEnabled` | boolean | `false` | Enable source code filtering for `read` output when read compaction is enabled |
 | `outputCompaction.preserveExactSkillReads` | boolean | `false` | Keep reads under configured Pi/global/project skill directories exact, bypassing read compaction |
-| `outputCompaction.sourceCodeFiltering` | string | `"minimal"` | Filter level: `"none"`, `"minimal"`, `"aggressive"` |
+| `outputCompaction.sourceCodeFiltering` | string | `"none"` | Filter level: `"none"`, `"minimal"`, `"aggressive"` |
 | `outputCompaction.aggregateTestOutput` | boolean | `true` | Summarize test runner output |
 | `outputCompaction.filterBuildOutput` | boolean | `true` | Filter build/compile output |
 | `outputCompaction.compactGitOutput` | boolean | `true` | Compact git command output |
@@ -146,7 +147,7 @@ Skill-read preservation covers the global Pi skills directory (`~/.pi/agent/skil
 
 | Option | Type | Default | Range | Description |
 |--------|------|---------|-------|-------------|
-| `outputCompaction.smartTruncate.enabled` | boolean | `true` | — | Enable smart line-based truncation |
+| `outputCompaction.smartTruncate.enabled` | boolean | `false` | — | Enable smart line-based truncation for read output when read compaction is enabled |
 | `outputCompaction.smartTruncate.maxLines` | number | `220` | 40–4000 | Maximum lines after smart truncation |
 | `outputCompaction.truncate.enabled` | boolean | `true` | — | Enable hard character truncation |
 | `outputCompaction.truncate.maxChars` | number | `12000` | 1000–200000 | Maximum characters in final output |
@@ -159,7 +160,7 @@ Skill-read preservation covers the global Pi skills directory (`~/.pi/agent/skil
 | `minimal` | Removes non-doc comments, collapses blank lines |
 | `aggressive` | Also removes imports, keeps only signatures and key logic |
 
-> **Note:** When source filtering and read truncation safeguards are active, Pi injects a troubleshooting note for repeated file-edit mismatches. If edits fail because "old text does not match," disable source filtering via `/rtk`, re-read the file, apply the edit, then re-enable filtering.
+> **Note:** When read compaction, source filtering, and read truncation safeguards are active, Pi injects a troubleshooting note for repeated file-edit mismatches. If edits fail because "old text does not match," disable read compaction via `/rtk`, re-read the file, apply the edit, then re-enable compaction.
 
 ### Example Configuration
 
@@ -172,9 +173,12 @@ Skill-read preservation covers the global Pi skills directory (`~/.pi/agent/skil
   "outputCompaction": {
     "enabled": true,
     "stripAnsi": true,
-    "sourceCodeFilteringEnabled": true,
+    "readCompaction": {
+      "enabled": false
+    },
+    "sourceCodeFilteringEnabled": false,
     "preserveExactSkillReads": false,
-    "sourceCodeFiltering": "minimal",
+    "sourceCodeFiltering": "none",
     "aggregateTestOutput": true,
     "filterBuildOutput": true,
     "compactGitOutput": true,
@@ -182,7 +186,7 @@ Skill-read preservation covers the global Pi skills directory (`~/.pi/agent/skil
     "groupSearchOutput": true,
     "trackSavings": true,
     "smartTruncate": {
-      "enabled": true,
+      "enabled": false,
       "maxLines": 220
     },
     "truncate": {

package/config/config.example.json

@@ -6,15 +6,18 @@
   "outputCompaction": {
     "enabled": true,
     "stripAnsi": true,
-    "sourceCodeFilteringEnabled": true,
+    "readCompaction": {
+      "enabled": false
+    },
+    "sourceCodeFilteringEnabled": false,
     "preserveExactSkillReads": false,
     "truncate": {
       "enabled": true,
       "maxChars": 12000
     },
-    "sourceCodeFiltering": "minimal",
+    "sourceCodeFiltering": "none",
     "smartTruncate": {
-      "enabled": true,
+      "enabled": false,
       "maxLines": 220
     },
     "aggregateTestOutput": true,

package/package.json

@@ -1,64 +1,64 @@
-{
-  "name": "pi-rtk-optimizer",
-  "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",
-  "exports": {
-    ".": "./index.ts"
-  },
-  "files": [
-    "index.ts",
-    "src",
-    "config/config.example.json",
-    "README.md",
-    "CHANGELOG.md",
-    "LICENSE"
-  ],
-  "scripts": {
-    "build": "npx --yes -p typescript@5.7.3 tsc -p tsconfig.json --noCheck",
-    "lint": "npm run build",
-    "typecheck": "npx --yes -p typescript@5.7.3 tsc -p tsconfig.json",
-    "test": "bun ./src/output-compactor-test.ts && bun ./src/command-rewriter-test.ts && bun ./src/runtime-guard-test.ts && bun ./src/additional-coverage-test.ts && bun ./src/config-modal-test.ts && bun ./src/index-test.ts",
-    "check": "npm run lint && npm run typecheck && npm run test",
-    "build:check": "bunx esbuild ./index.ts --bundle --platform=node --format=esm --outfile=./.pi-rtk-optimizer-check.mjs --external:@mariozechner/pi-coding-agent --external:@mariozechner/pi-tui && bun -e \"import { unlinkSync } from 'node:fs'; unlinkSync('./.pi-rtk-optimizer-check.mjs');\""
-  },
-  "keywords": [
-    "pi-package",
-    "pi",
-    "pi-extension",
-    "pi-coding-agent",
-    "coding-agent",
-    "rtk",
-    "token-optimization",
-    "tool-compaction",
-    "output-compaction",
-    "command-rewrite",
-    "optimization"
-  ],
-  "author": "MasuRii",
-  "license": "MIT",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/MasuRii/pi-rtk-optimizer.git"
-  },
-  "bugs": {
-    "url": "https://github.com/MasuRii/pi-rtk-optimizer/issues"
-  },
-  "homepage": "https://github.com/MasuRii/pi-rtk-optimizer#readme",
-  "engines": {
-    "node": ">=20"
-  },
-  "publishConfig": {
-    "access": "public"
-  },
-  "pi": {
-    "extensions": [
-      "./index.ts"
-    ]
-  },
-  "peerDependencies": {
-    "@mariozechner/pi-coding-agent": "^0.70.2",
-    "@mariozechner/pi-tui": "^0.70.2"
-  }
-}
+{
+  "name": "pi-rtk-optimizer",
+  "version": "0.7.0",
+  "description": "Pi extension that optimizes RTK command rewriting and tool output compaction for the coding agent.",
+  "type": "module",
+  "main": "./index.ts",
+  "exports": {
+    ".": "./index.ts"
+  },
+  "files": [
+    "index.ts",
+    "src",
+    "config/config.example.json",
+    "README.md",
+    "CHANGELOG.md",
+    "LICENSE"
+  ],
+  "scripts": {
+    "build": "npx --yes -p typescript@5.7.3 tsc -p tsconfig.json --noCheck",
+    "lint": "npm run build",
+    "typecheck": "npx --yes -p typescript@5.7.3 tsc -p tsconfig.json",
+    "test": "bun ./src/output-compactor-test.ts && bun ./src/command-rewriter-test.ts && bun ./src/runtime-guard-test.ts && bun ./src/additional-coverage-test.ts && bun ./src/config-modal-test.ts && bun ./src/index-test.ts",
+    "check": "npm run lint && npm run typecheck && npm run test",
+    "build:check": "bunx esbuild ./index.ts --bundle --platform=node --format=esm --outfile=./.pi-rtk-optimizer-check.mjs --external:@mariozechner/pi-coding-agent --external:@mariozechner/pi-tui && bun -e \"import { unlinkSync } from 'node:fs'; unlinkSync('./.pi-rtk-optimizer-check.mjs');\""
+  },
+  "keywords": [
+    "pi-package",
+    "pi",
+    "pi-extension",
+    "pi-coding-agent",
+    "coding-agent",
+    "rtk",
+    "token-optimization",
+    "tool-compaction",
+    "output-compaction",
+    "command-rewrite",
+    "optimization"
+  ],
+  "author": "MasuRii",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/MasuRii/pi-rtk-optimizer.git"
+  },
+  "bugs": {
+    "url": "https://github.com/MasuRii/pi-rtk-optimizer/issues"
+  },
+  "homepage": "https://github.com/MasuRii/pi-rtk-optimizer#readme",
+  "engines": {
+    "node": ">=20"
+  },
+  "publishConfig": {
+    "access": "public"
+  },
+  "pi": {
+    "extensions": [
+      "./index.ts"
+    ]
+  },
+  "peerDependencies": {
+    "@mariozechner/pi-coding-agent": "^0.70.6",
+    "@mariozechner/pi-tui": "^0.70.6"
+  }
+}

package/src/additional-coverage-test.ts

@@ -66,6 +66,7 @@ runTest("config-store normalizes invalid values and clamps numeric ranges", () =
 	assert.equal(normalized.mode, "rewrite");
 	assert.equal(Object.hasOwn(normalized, "rewriteGitGithub"), false);
 	assert.equal(normalized.outputCompaction.stripAnsi, false);
+	assert.equal(normalized.outputCompaction.readCompaction.enabled, true);
 	assert.equal(normalized.outputCompaction.sourceCodeFilteringEnabled, true);
 	assert.equal(normalized.outputCompaction.sourceCodeFiltering, "minimal");
 	assert.equal(normalized.outputCompaction.truncate.maxChars, 1_000);
@@ -73,18 +74,32 @@ runTest("config-store normalizes invalid values and clamps numeric ranges", () =
 	assert.equal(normalized.outputCompaction.trackSavings, false);
 });
 
+runTest("config-store uses safer read defaults when readCompaction is explicit", () => {
+	const normalized = normalizeRtkIntegrationConfig({
+		outputCompaction: {
+			readCompaction: { enabled: false },
+		},
+	});
+
+	assert.equal(normalized.outputCompaction.readCompaction.enabled, false);
+	assert.equal(normalized.outputCompaction.sourceCodeFilteringEnabled, false);
+	assert.equal(normalized.outputCompaction.sourceCodeFiltering, "none");
+	assert.equal(normalized.outputCompaction.smartTruncate.enabled, false);
+});
+
 runTest("config-store can ensure, save, and reload isolated config files", () => {
 	const tempPath = makeTempConfigPath();
 	cleanupFile(tempPath);
 
 	try {
 		const ensured = ensureConfigExists(tempPath);
-		assert.equal(ensured.created, true);
+		assert.equal(ensured.error, undefined);
 		assert.equal(existsSync(tempPath), true);
 
 		const defaultLoad = loadRtkIntegrationConfig(tempPath);
 		assert.equal(defaultLoad.warning, undefined);
 		assert.equal(defaultLoad.config.mode, "rewrite");
+		assert.equal(defaultLoad.config.outputCompaction.readCompaction.enabled, false);
 
 		const saved = saveRtkIntegrationConfig(
 			{
@@ -153,6 +168,14 @@ runTest("command detection ignores env prefixes, blank lines, and chained suffix
 	assert.equal(matchesCommandPatterns("echo hello", [/^bun test/]), false);
 });
 
+runTest("RTK command environment preserves explicit leading RTK_DB_PATH overrides", () => {
+	const command = 'RTK_DB_PATH="/custom/history.db" rtk git diff';
+	assert.equal(applyRtkCommandEnvironment(command), command);
+
+	const exportedCommand = 'export RTK_DB_PATH="/custom/history.db"; rtk git diff';
+	assert.equal(applyRtkCommandEnvironment(exportedCommand), exportedCommand);
+});
+
 runTest("path compaction preserves the tail and handles Windows separators", () => {
 	const unixPath = "/Users/example/projects/pi-rtk-optimizer/src/techniques/path-utils.ts";
 	const compactUnixPath = compactPath(unixPath, 28);
@@ -204,21 +227,21 @@ runTest("rewrite pipeline safety buffers rewritten Windows producer commands", (
 	assert.equal(applyRewrittenCommandShellSafetyFixups("git diff | grep TODO"), "git diff | grep TODO");
 });
 
-runTest("rewrite pipeline safety keeps RTK_DB_PATH scoped to rewritten producer commands", () => {
+runTest("rewrite pipeline safety keeps exported RTK_DB_PATH on rewritten producer commands", () => {
 	const rewritten = applyRewrittenCommandShellSafetyFixups(
 		applyRtkCommandEnvironment("rtk git diff agent/extensions/pi-multi-auth/account-manager.ts | head -200"),
 	);
 
 	if (process.platform === "win32") {
-		assert.ok(rewritten.startsWith("{"));
+		assert.ok(rewritten.startsWith("export RTK_DB_PATH="));
 		assert.equal(rewritten.startsWith("RTK_DB_PATH="), false);
-		assert.ok(rewritten.includes("RTK_DB_PATH="));
+		assert.ok(rewritten.includes("; {"));
 		assert.ok(
 			rewritten.includes('rtk git diff agent/extensions/pi-multi-auth/account-manager.ts > "$__pi_rtk_pipe_tmp"'),
 		);
 		assert.ok(rewritten.includes('(head -200) < "$__pi_rtk_pipe_tmp"'));
 	} else {
-		assert.ok(rewritten.startsWith("RTK_DB_PATH="));
+		assert.ok(rewritten.startsWith("export RTK_DB_PATH="));
 		assert.equal(
 			rewritten,
 			applyRtkCommandEnvironment("rtk git diff agent/extensions/pi-multi-auth/account-manager.ts | head -200"),
@@ -226,6 +249,25 @@ runTest("rewrite pipeline safety keeps RTK_DB_PATH scoped to rewritten producer
 	}
 });
 
+runTest("rewrite pipeline safety buffers explicit RTK_DB_PATH export preludes", () => {
+	const command = 'export RTK_DB_PATH="/custom/history.db"; rtk git diff | head -200';
+	const rewritten = applyRewrittenCommandShellSafetyFixups(command);
+
+	if (process.platform === "win32") {
+		assert.ok(rewritten.startsWith('export RTK_DB_PATH="/custom/history.db"; {'));
+		assert.ok(rewritten.includes('rtk git diff > "$__pi_rtk_pipe_tmp"'));
+		assert.ok(rewritten.includes('(head -200) < "$__pi_rtk_pipe_tmp"'));
+	} else {
+		assert.equal(rewritten, command);
+	}
+});
+
+runTest("RTK command environment uses export prelude for shell compound commands", () => {
+	const rewritten = applyRtkCommandEnvironment('for d in a b; do echo "$d"; done');
+	assert.ok(/^export RTK_DB_PATH=/.test(rewritten));
+	assert.ok(/; for d in a b; do echo "\$d"; done$/.test(rewritten));
+});
+
 runTest("stripRtkHookWarnings handles bare, prefixed, and already-sanitized hook notices", () => {
 	assert.equal(
 		stripRtkHookWarnings("No hook installed — run `rtk init -g` for automatic token savings\n\nready\n", null),

package/src/config-modal.ts

@@ -58,7 +58,7 @@ function summarizeConfig(config: RtkIntegrationConfig, runtimeStatus: RuntimeSta
 		? "rtk=available"
 		: `rtk=missing${runtimeStatus.lastError ? ` (${runtimeStatus.lastError})` : ""}`;
 
-	return `enabled=${config.enabled}, mode=${config.mode}, rewriteSource=rtk, rewriteNotice=${config.showRewriteNotifications}, compaction=${config.outputCompaction.enabled}, sourceFilterEnabled=${config.outputCompaction.sourceCodeFilteringEnabled}, preserveSkillReads=${config.outputCompaction.preserveExactSkillReads}, sourceFilter=${config.outputCompaction.sourceCodeFiltering}, ${runtime}`;
+	return `enabled=${config.enabled}, mode=${config.mode}, rewriteSource=rtk, rewriteNotice=${config.showRewriteNotifications}, compaction=${config.outputCompaction.enabled}, readCompaction=${config.outputCompaction.readCompaction.enabled}, sourceFilterEnabled=${config.outputCompaction.sourceCodeFilteringEnabled}, preserveSkillReads=${config.outputCompaction.preserveExactSkillReads}, sourceFilter=${config.outputCompaction.sourceCodeFiltering}, ${runtime}`;
 }
 
 function buildSettingItems(config: RtkIntegrationConfig): SettingItem[] {
@@ -105,6 +105,13 @@ function buildSettingItems(config: RtkIntegrationConfig): SettingItem[] {
 			currentValue: toOnOff(config.outputCompaction.stripAnsi),
 			values: ON_OFF,
 		},
+		{
+			id: "outputReadCompactionEnabled",
+			label: "Read compaction enabled",
+			description: "If off, read tool output stays exact; build/test/git/grep compaction can still run",
+			currentValue: toOnOff(config.outputCompaction.readCompaction.enabled),
+			values: ON_OFF,
+		},
 		{
 			id: "outputTruncateEnabled",
 			label: "Hard truncation enabled",
@@ -219,6 +226,14 @@ function applySetting(config: RtkIntegrationConfig, id: string, value: string):
 				...config,
 				outputCompaction: { ...config.outputCompaction, stripAnsi: value === "on" },
 			};
+		case "outputReadCompactionEnabled":
+			return {
+				...config,
+				outputCompaction: {
+					...config.outputCompaction,
+					readCompaction: { enabled: value === "on" },
+				},
+			};
 		case "outputTruncateEnabled":
 			return {
 				...config,
@@ -353,6 +368,7 @@ function syncSettingValues(settingsList: SettingValueSyncTarget, config: RtkInte
 	settingsList.updateValue("guardWhenRtkMissing", toOnOff(config.guardWhenRtkMissing));
 	settingsList.updateValue("outputCompactionEnabled", toOnOff(config.outputCompaction.enabled));
 	settingsList.updateValue("outputStripAnsi", toOnOff(config.outputCompaction.stripAnsi));
+	settingsList.updateValue("outputReadCompactionEnabled", toOnOff(config.outputCompaction.readCompaction.enabled));
 	settingsList.updateValue("outputTruncateEnabled", toOnOff(config.outputCompaction.truncate.enabled));
 	settingsList.updateValue("outputTruncateMaxChars", String(config.outputCompaction.truncate.maxChars));
 	settingsList.updateValue("outputSourceFilteringEnabled", toOnOff(config.outputCompaction.sourceCodeFilteringEnabled));

package/src/config-store.ts

@@ -30,10 +30,14 @@ function toMode(value: unknown): RtkIntegrationConfig["mode"] {
 		: DEFAULT_RTK_INTEGRATION_CONFIG.mode;
 }
 
-function toSourceFilterLevel(value: unknown): RtkSourceFilterLevel {
+function toSourceFilterLevel(value: unknown, fallback: RtkSourceFilterLevel): RtkSourceFilterLevel {
 	return RTK_SOURCE_FILTER_LEVELS.includes(value as RtkSourceFilterLevel)
 		? (value as RtkSourceFilterLevel)
-		: DEFAULT_RTK_INTEGRATION_CONFIG.outputCompaction.sourceCodeFiltering;
+		: fallback;
+}
+
+function hasOwnProperty(source: Record<string, unknown>, key: string): boolean {
+	return Object.prototype.hasOwnProperty.call(source, key);
 }
 
 function toObject(value: unknown): Record<string, unknown> {
@@ -46,8 +50,20 @@ function toObject(value: unknown): Record<string, unknown> {
 export function normalizeRtkIntegrationConfig(raw: unknown): RtkIntegrationConfig {
 	const source = toObject(raw);
 	const outputCompactionSource = toObject(source.outputCompaction);
+	const readCompactionSource = toObject(outputCompactionSource.readCompaction);
 	const truncateSource = toObject(outputCompactionSource.truncate);
 	const smartTruncateSource = toObject(outputCompactionSource.smartTruncate);
+	const hasReadCompaction = hasOwnProperty(outputCompactionSource, "readCompaction");
+	const legacyReadCompactionFallback = !hasReadCompaction;
+	const sourceFilteringFallback = legacyReadCompactionFallback
+		? true
+		: DEFAULT_RTK_INTEGRATION_CONFIG.outputCompaction.sourceCodeFilteringEnabled;
+	const sourceFilterLevelFallback = legacyReadCompactionFallback
+		? "minimal"
+		: DEFAULT_RTK_INTEGRATION_CONFIG.outputCompaction.sourceCodeFiltering;
+	const smartTruncateEnabledFallback = legacyReadCompactionFallback
+		? true
+		: DEFAULT_RTK_INTEGRATION_CONFIG.outputCompaction.smartTruncate.enabled;
 
 	return {
 		enabled: toBoolean(source.enabled, DEFAULT_RTK_INTEGRATION_CONFIG.enabled),
@@ -69,9 +85,17 @@ export function normalizeRtkIntegrationConfig(raw: unknown): RtkIntegrationConfi
 				outputCompactionSource.stripAnsi,
 				DEFAULT_RTK_INTEGRATION_CONFIG.outputCompaction.stripAnsi,
 			),
+			readCompaction: {
+				enabled: hasReadCompaction
+					? toBoolean(
+							readCompactionSource.enabled,
+							DEFAULT_RTK_INTEGRATION_CONFIG.outputCompaction.readCompaction.enabled,
+						)
+					: true,
+			},
 			sourceCodeFilteringEnabled: toBoolean(
 				outputCompactionSource.sourceCodeFilteringEnabled,
-				DEFAULT_RTK_INTEGRATION_CONFIG.outputCompaction.sourceCodeFilteringEnabled,
+				sourceFilteringFallback,
 			),
 			preserveExactSkillReads: toBoolean(
 				outputCompactionSource.preserveExactSkillReads,
@@ -89,11 +113,14 @@ export function normalizeRtkIntegrationConfig(raw: unknown): RtkIntegrationConfi
 					200_000,
 				),
 			},
-			sourceCodeFiltering: toSourceFilterLevel(outputCompactionSource.sourceCodeFiltering),
+			sourceCodeFiltering: toSourceFilterLevel(
+				outputCompactionSource.sourceCodeFiltering,
+				sourceFilterLevelFallback,
+			),
 			smartTruncate: {
 				enabled: toBoolean(
 					smartTruncateSource.enabled,
-					DEFAULT_RTK_INTEGRATION_CONFIG.outputCompaction.smartTruncate.enabled,
+					smartTruncateEnabledFallback,
 				),
 				maxLines: toInteger(
 					smartTruncateSource.maxLines,

package/src/index-test.ts

@@ -34,6 +34,7 @@ const { DEFAULT_RTK_INTEGRATION_CONFIG } = await import("./types.ts");
 function configWith(overrides: {
 	enabled?: boolean;
 	compactionEnabled?: boolean;
+	readCompactionEnabled?: boolean;
 	sourceFilteringEnabled?: boolean;
 	sourceFilteringLevel?: "none" | "minimal" | "aggressive";
 	smartTruncateEnabled?: boolean;
@@ -46,6 +47,10 @@ function configWith(overrides: {
 		outputCompaction: {
 			...base.outputCompaction,
 			enabled: overrides.compactionEnabled ?? base.outputCompaction.enabled,
+			readCompaction: {
+				...base.outputCompaction.readCompaction,
+				enabled: overrides.readCompactionEnabled ?? base.outputCompaction.readCompaction.enabled,
+			},
 			sourceCodeFilteringEnabled:
 				overrides.sourceFilteringEnabled ?? base.outputCompaction.sourceCodeFilteringEnabled,
 			sourceCodeFiltering: overrides.sourceFilteringLevel ?? base.outputCompaction.sourceCodeFiltering,
@@ -86,13 +91,23 @@ runTest("bounded notice tracker coerces invalid limits to a safe minimum", () =>
 runTest("source-filter note injected when source filtering is active", () => {
 	assert.equal(
 		shouldInjectSourceFilterTroubleshootingNote(
-			configWith({ sourceFilteringEnabled: true, sourceFilteringLevel: "minimal" }),
+			configWith({
+				readCompactionEnabled: true,
+				sourceFilteringEnabled: true,
+				sourceFilteringLevel: "minimal",
+				smartTruncateEnabled: true,
+			}),
 		),
 		true,
 	);
 	assert.equal(
 		shouldInjectSourceFilterTroubleshootingNote(
-			configWith({ sourceFilteringEnabled: true, sourceFilteringLevel: "aggressive" }),
+			configWith({
+				readCompactionEnabled: true,
+				sourceFilteringEnabled: true,
+				sourceFilteringLevel: "aggressive",
+				smartTruncateEnabled: true,
+			}),
 		),
 		true,
 	);
@@ -106,6 +121,20 @@ runTest("source-filter note skipped when compaction is disabled", () => {
 	assert.equal(shouldInjectSourceFilterTroubleshootingNote(configWith({ compactionEnabled: false })), false);
 });
 
+runTest("source-filter note skipped when read compaction is disabled", () => {
+	assert.equal(
+		shouldInjectSourceFilterTroubleshootingNote(
+			configWith({
+				readCompactionEnabled: false,
+				sourceFilteringEnabled: true,
+				sourceFilteringLevel: "minimal",
+				smartTruncateEnabled: true,
+			}),
+		),
+		false,
+	);
+});
+
 runTest("source-filter note skipped when source filtering flag is off", () => {
 	assert.equal(
 		shouldInjectSourceFilterTroubleshootingNote(configWith({ sourceFilteringEnabled: false })),

package/src/index.ts

@@ -28,13 +28,14 @@ function trimMessage(raw: string, maxLength = 220): string {
 }
 
 const SOURCE_FILTER_TROUBLESHOOTING_NOTE =
-	"RTK note: If file edits repeatedly fail because old text does not match, ask the user to manually run '/rtk' in the Pi TUI, disable 'Read source filtering enabled', re-read the file, apply the edit, then ask the user to manually re-enable it in the Pi TUI.";
+	"RTK note: If file edits repeatedly fail because old text does not match, ask the user to manually run '/rtk' in the Pi TUI, disable 'Read compaction enabled', re-read the file, apply the edit, then ask the user to manually re-enable it in the Pi TUI.";
 
 export function shouldInjectSourceFilterTroubleshootingNote(config: RtkIntegrationConfig): boolean {
 	const compaction = config.outputCompaction;
 	return (
 		config.enabled &&
 		compaction.enabled &&
+		compaction.readCompaction.enabled &&
 		compaction.sourceCodeFilteringEnabled &&
 		compaction.sourceCodeFiltering !== "none" &&
 		(compaction.smartTruncate.enabled || compaction.truncate.enabled)

package/src/output-compactor-test.ts

@@ -24,6 +24,10 @@ function buildReadContent(lineCount: number): string {
 	return `${lines.join("\n")}\n`;
 }
 
+function setReadCompaction(config: ReturnType<typeof cloneDefaultConfig>, enabled: boolean): void {
+	config.outputCompaction.readCompaction = { enabled };
+}
+
 function firstTextBlock(content: unknown[] | undefined): string {
 	if (!Array.isArray(content) || content.length === 0) {
 		return "";
@@ -73,6 +77,7 @@ function assertNoOutputEmoji(text: string): void {
 
 runTest("precision read with offset keeps exact output (no source/smart/hard truncation)", () => {
 	const config = cloneDefaultConfig();
+	setReadCompaction(config, true);
 	config.outputCompaction.truncate.enabled = true;
 	config.outputCompaction.truncate.maxChars = 500;
 	config.outputCompaction.smartTruncate.enabled = true;
@@ -94,6 +99,7 @@ runTest("precision read with offset keeps exact output (no source/smart/hard tru
 
 runTest("precision read with limit keeps exact output", () => {
 	const config = cloneDefaultConfig();
+	setReadCompaction(config, true);
 	config.outputCompaction.truncate.enabled = true;
 	config.outputCompaction.truncate.maxChars = 500;
 	config.outputCompaction.smartTruncate.enabled = true;
@@ -113,8 +119,34 @@ runTest("precision read with limit keeps exact output", () => {
 	assert.deepEqual(result.techniques, []);
 });
 
-runTest("normal read compacts and adds banner", () => {
+runTest("default read output stays exact when read compaction is disabled by default", () => {
+	const config = cloneDefaultConfig();
+	config.outputCompaction.sourceCodeFilteringEnabled = true;
+	config.outputCompaction.sourceCodeFiltering = "aggressive";
+	config.outputCompaction.smartTruncate.enabled = true;
+	config.outputCompaction.smartTruncate.maxLines = 40;
+	config.outputCompaction.truncate.enabled = true;
+	config.outputCompaction.truncate.maxChars = 500;
+
+	const content = buildReadContent(220);
+	const result = compactToolResult(
+		{
+			toolName: "read",
+			input: { path: "sample.ts" },
+			content: [{ type: "text", text: content }],
+		},
+		config,
+	);
+
+	assert.equal(result.changed, false);
+	assert.deepEqual(result.techniques, []);
+});
+
+runTest("normal read compacts and adds banner when read compaction is enabled", () => {
 	const config = cloneDefaultConfig();
+	setReadCompaction(config, true);
+	config.outputCompaction.sourceCodeFilteringEnabled = true;
+	config.outputCompaction.sourceCodeFiltering = "minimal";
 	config.outputCompaction.smartTruncate.enabled = true;
 	config.outputCompaction.smartTruncate.maxLines = 40;
 
@@ -138,6 +170,7 @@ runTest("normal read compacts and adds banner", () => {
 
 runTest("short read output stays exact below threshold", () => {
 	const config = cloneDefaultConfig();
+	setReadCompaction(config, true);
 	const content = buildReadContent(40);
 
 	const result = compactToolResult(
@@ -155,6 +188,7 @@ runTest("short read output stays exact below threshold", () => {
 
 runTest("read output stays exact at the 80-line boundary with trailing newline", () => {
 	const config = cloneDefaultConfig();
+	setReadCompaction(config, true);
 	config.outputCompaction.smartTruncate.enabled = true;
 	config.outputCompaction.smartTruncate.maxLines = 40;
 
@@ -172,8 +206,11 @@ runTest("read output stays exact at the 80-line boundary with trailing newline",
 	assert.deepEqual(result.techniques, []);
 });
 
-runTest("read output compacts once the content exceeds the 80-line exactness threshold", () => {
+runTest("read output compacts once the content exceeds the 80-line exactness threshold when read compaction is enabled", () => {
 	const config = cloneDefaultConfig();
+	setReadCompaction(config, true);
+	config.outputCompaction.sourceCodeFilteringEnabled = true;
+	config.outputCompaction.sourceCodeFiltering = "minimal";
 	config.outputCompaction.smartTruncate.enabled = true;
 	config.outputCompaction.smartTruncate.maxLines = 40;
 
@@ -193,6 +230,9 @@ runTest("read output compacts once the content exceeds the 80-line exactness thr
 
 runTest("source file reads skip lossy source filtering when truncation safeguards are not needed", () => {
 	const config = cloneDefaultConfig();
+	setReadCompaction(config, true);
+	config.outputCompaction.sourceCodeFilteringEnabled = true;
+	config.outputCompaction.sourceCodeFiltering = "minimal";
 	config.outputCompaction.smartTruncate.enabled = false;
 	config.outputCompaction.truncate.enabled = false;
 
@@ -213,6 +253,7 @@ runTest("source file reads skip lossy source filtering when truncation safeguard
 
 runTest("skill reads stay exact when preserveExactSkillReads is enabled for user skills", () => {
 	const config = cloneDefaultConfig();
+	setReadCompaction(config, true);
 	config.outputCompaction.preserveExactSkillReads = true;
 	config.outputCompaction.truncate.enabled = true;
 	config.outputCompaction.truncate.maxChars = 500;
@@ -235,6 +276,7 @@ runTest("skill reads stay exact when preserveExactSkillReads is enabled for user
 
 runTest("project .pi skill reads stay exact when preserveExactSkillReads is enabled", () => {
 	const config = cloneDefaultConfig();
+	setReadCompaction(config, true);
 	config.outputCompaction.preserveExactSkillReads = true;
 	config.outputCompaction.truncate.enabled = true;
 	config.outputCompaction.truncate.maxChars = 500;
@@ -257,6 +299,7 @@ runTest("project .pi skill reads stay exact when preserveExactSkillReads is enab
 
 runTest("ancestor .agents skill reads stay exact when preserveExactSkillReads is enabled", () => {
 	const config = cloneDefaultConfig();
+	setReadCompaction(config, true);
 	config.outputCompaction.preserveExactSkillReads = true;
 	config.outputCompaction.truncate.enabled = true;
 	config.outputCompaction.truncate.maxChars = 500;

package/src/output-compactor.ts

@@ -140,6 +140,10 @@ function shouldPreserveExactReadOutput(
 	input: Record<string, unknown>,
 	config: RtkIntegrationConfig,
 ): boolean {
+	if (!config.outputCompaction.readCompaction.enabled) {
+		return true;
+	}
+
 	if (hasExplicitReadRange(input)) {
 		return true;
 	}

package/src/rewrite-pipeline-safety.ts

@@ -10,6 +10,26 @@ interface ProducerRewritePlan {
 	captureStderr: boolean;
 }
 
+interface ShellSafetyTarget {
+	environmentPrelude: string;
+	command: string;
+}
+
+const LEADING_RTK_DB_PATH_EXPORT_PRELUDE_PATTERN =
+	/^(\s*export\s+RTK_DB_PATH=(?:"(?:\\.|[^"])*"|'[^']*'|[^\s;]+)\s*;\s*)([\s\S]*)$/u;
+
+function splitLeadingRtkDbPathExportPrelude(command: string): ShellSafetyTarget {
+	const match = command.match(LEADING_RTK_DB_PATH_EXPORT_PRELUDE_PATTERN);
+	if (!match) {
+		return { environmentPrelude: "", command };
+	}
+
+	return {
+		environmentPrelude: match[1] ?? "",
+		command: match[2] ?? "",
+	};
+}
+
 function isTopLevelQuoteCharacter(character: string): character is '"' | "'" | "`" {
 	return character === '"' || character === "'" || character === "`";
 }
@@ -134,7 +154,8 @@ export function applyRewrittenCommandShellSafetyFixups(command: string): string
 		return command;
 	}
 
-	const parsedPipeline = parseSimpleTopLevelPipeline(command);
+	const target = splitLeadingRtkDbPathExportPrelude(command);
+	const parsedPipeline = parseSimpleTopLevelPipeline(target.command);
 	if (!parsedPipeline) {
 		return command;
 	}
@@ -153,5 +174,5 @@ export function applyRewrittenCommandShellSafetyFixups(command: string): string
 		return command;
 	}
 
-	return buildBufferedPipelineCommand(producer, remainder);
+	return `${target.environmentPrelude}${buildBufferedPipelineCommand(producer, remainder)}`;
 }

package/src/rtk-command-environment.ts

@@ -1,64 +1,69 @@
-import { join } from "node:path";
-
-import { splitLeadingEnvAssignments } from "./shell-env-prefix.js";
-
-const RTK_DB_PATH_ENV_NAME = "RTK_DB_PATH";
-const RTK_DB_PATH_ASSIGNMENT_PATTERN = /(?:^|\s)RTK_DB_PATH=(?:"[^"]*"|'[^']*'|[^\s]+)(?=\s|$)/;
-
-function resolveTemporaryDirectory(): string {
-	if (process.platform === "win32") {
-		const windowsTempDir = process.env.TEMP ?? process.env.TMP;
-		if (windowsTempDir && windowsTempDir.trim()) {
-			return windowsTempDir;
-		}
-
-		const localAppData = process.env.LOCALAPPDATA;
-		if (localAppData && localAppData.trim()) {
-			return join(localAppData, "Temp");
-		}
-
-		const userProfile = process.env.USERPROFILE;
-		if (userProfile && userProfile.trim()) {
-			return join(userProfile, "AppData", "Local", "Temp");
-		}
-
-		const systemRoot = process.env.SystemRoot ?? process.env.WINDIR;
-		if (systemRoot && systemRoot.trim()) {
-			return join(systemRoot, "Temp");
-		}
-
-		return "C:/Windows/Temp";
-	}
-
-	const posixTempDir = process.env.TMPDIR ?? process.env.TMP;
-	if (posixTempDir && posixTempDir.trim()) {
-		return posixTempDir;
-	}
-
-	return "/tmp";
-}
-
-function getTemporaryRtkHistoryDbPath(): string {
-	return join(resolveTemporaryDirectory(), "pi-rtk-optimizer", "history.db");
-}
-
-function quoteForShellEnv(value: string): string {
-	const normalizedValue = process.platform === "win32" ? value.replace(/\\/g, "/") : value;
-	return `"${normalizedValue.replace(/"/g, '\\"')}"`;
-}
-
-function hasLeadingRtkDbPathAssignment(command: string): boolean {
-	return RTK_DB_PATH_ASSIGNMENT_PATTERN.test(splitLeadingEnvAssignments(command).envPrefix);
-}
-
-export function applyRtkCommandEnvironment(command: string): string {
-	if (!command.trim()) {
-		return command;
-	}
-
-	if (hasLeadingRtkDbPathAssignment(command)) {
-		return command;
-	}
-
-	return `${RTK_DB_PATH_ENV_NAME}=${quoteForShellEnv(getTemporaryRtkHistoryDbPath())} ${command}`;
-}
+import { join } from "node:path";
+
+import { splitLeadingEnvAssignments } from "./shell-env-prefix.js";
+
+const RTK_DB_PATH_ENV_NAME = "RTK_DB_PATH";
+const RTK_DB_PATH_ASSIGNMENT_PATTERN = /(?:^|\s)RTK_DB_PATH=(?:"[^"]*"|'[^']*'|[^\s]+)(?=\s|$)/;
+const RTK_DB_PATH_EXPORT_PATTERN = /^export\s+RTK_DB_PATH=(?:"[^"]*"|'[^']*'|[^\s;]+)(?=\s*(?:;|$))/;
+
+function resolveTemporaryDirectory(): string {
+	if (process.platform === "win32") {
+		const windowsTempDir = process.env.TEMP ?? process.env.TMP;
+		if (windowsTempDir && windowsTempDir.trim()) {
+			return windowsTempDir;
+		}
+
+		const localAppData = process.env.LOCALAPPDATA;
+		if (localAppData && localAppData.trim()) {
+			return join(localAppData, "Temp");
+		}
+
+		const userProfile = process.env.USERPROFILE;
+		if (userProfile && userProfile.trim()) {
+			return join(userProfile, "AppData", "Local", "Temp");
+		}
+
+		const systemRoot = process.env.SystemRoot ?? process.env.WINDIR;
+		if (systemRoot && systemRoot.trim()) {
+			return join(systemRoot, "Temp");
+		}
+
+		return "C:/Windows/Temp";
+	}
+
+	const posixTempDir = process.env.TMPDIR ?? process.env.TMP;
+	if (posixTempDir && posixTempDir.trim()) {
+		return posixTempDir;
+	}
+
+	return "/tmp";
+}
+
+function getTemporaryRtkHistoryDbPath(): string {
+	return join(resolveTemporaryDirectory(), "pi-rtk-optimizer", "history.db");
+}
+
+function quoteForShellEnv(value: string): string {
+	const normalizedValue = process.platform === "win32" ? value.replace(/\\/g, "/") : value;
+	return `"${normalizedValue.replace(/"/g, '\\"')}"`;
+}
+
+function hasLeadingRtkDbPathAssignment(command: string): boolean {
+	const trimmed = command.trimStart();
+	return (
+		RTK_DB_PATH_ASSIGNMENT_PATTERN.test(splitLeadingEnvAssignments(trimmed).envPrefix) ||
+		RTK_DB_PATH_EXPORT_PATTERN.test(trimmed)
+	);
+}
+
+export function applyRtkCommandEnvironment(command: string): string {
+	if (!command.trim()) {
+		return command;
+	}
+
+	if (hasLeadingRtkDbPathAssignment(command)) {
+		return command;
+	}
+
+	return `export ${RTK_DB_PATH_ENV_NAME}=${quoteForShellEnv(getTemporaryRtkHistoryDbPath())}; ${command}`;
+}

package/src/types.ts

@@ -7,6 +7,9 @@ export type RtkSourceFilterLevel = (typeof RTK_SOURCE_FILTER_LEVELS)[number];
 export interface RtkOutputCompactionConfig {
 	enabled: boolean;
 	stripAnsi: boolean;
+	readCompaction: {
+		enabled: boolean;
+	};
 	truncate: {
 		enabled: boolean;
 		maxChars: number;
@@ -42,15 +45,18 @@ export const DEFAULT_RTK_INTEGRATION_CONFIG: RtkIntegrationConfig = {
 	outputCompaction: {
 		enabled: true,
 		stripAnsi: true,
+		readCompaction: {
+			enabled: false,
+		},
 		truncate: {
 			enabled: true,
 			maxChars: 12_000,
 		},
-		sourceCodeFilteringEnabled: true,
+		sourceCodeFilteringEnabled: false,
 		preserveExactSkillReads: false,
-		sourceCodeFiltering: "minimal",
+		sourceCodeFiltering: "none",
 		smartTruncate: {
-			enabled: true,
+			enabled: false,
 			maxLines: 220,
 		},
 		aggregateTestOutput: true,