pi-rtk-optimizer 0.5.5 → 0.7.0

package/src/config-modal.ts

@@ -54,25 +54,11 @@ function parseIntegerInRange(value: string, min: number, max: number): number |
 }
 
 function summarizeConfig(config: RtkIntegrationConfig, runtimeStatus: RuntimeStatus): string {
-	const categories = [
-		config.rewriteGitGithub ? "git" : "",
-		config.rewriteFilesystem ? "files" : "",
-		config.rewriteRust ? "rust" : "",
-		config.rewriteJavaScript ? "js" : "",
-		config.rewritePython ? "python" : "",
-		config.rewriteGo ? "go" : "",
-		config.rewriteContainers ? "containers" : "",
-		config.rewriteNetwork ? "network" : "",
-		config.rewritePackageManagers ? "packages" : "",
-	]
-		.filter((value) => value.length > 0)
-		.join(",");
-
 	const runtime = runtimeStatus.rtkAvailable
 		? "rtk=available"
 		: `rtk=missing${runtimeStatus.lastError ? ` (${runtimeStatus.lastError})` : ""}`;
 
-	return `enabled=${config.enabled}, mode=${config.mode}, rewriteNotice=${config.showRewriteNotifications}, compaction=${config.outputCompaction.enabled}, sourceFilterEnabled=${config.outputCompaction.sourceCodeFilteringEnabled}, preserveSkillReads=${config.outputCompaction.preserveExactSkillReads}, sourceFilter=${config.outputCompaction.sourceCodeFiltering}, categories=[${categories || "none"}], ${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[] {
@@ -119,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",
@@ -210,69 +203,6 @@ function buildSettingItems(config: RtkIntegrationConfig): SettingItem[] {
 			currentValue: toOnOff(config.outputCompaction.trackSavings),
 			values: ON_OFF,
 		},
-		{
-			id: "rewriteGitGithub",
-			label: "Rewrite git / gh",
-			description: "git status/log/diff and gh pr/issue/run/api/release",
-			currentValue: toOnOff(config.rewriteGitGithub),
-			values: ON_OFF,
-		},
-		{
-			id: "rewriteFilesystem",
-			label: "Rewrite filesystem commands",
-			description: "cat, head, rg/grep, ls, tree, find, diff",
-			currentValue: toOnOff(config.rewriteFilesystem),
-			values: ON_OFF,
-		},
-		{
-			id: "rewriteRust",
-			label: "Rewrite Rust commands",
-			description: "cargo test/build/clippy/check/install/nextest/fmt",
-			currentValue: toOnOff(config.rewriteRust),
-			values: ON_OFF,
-		},
-		{
-			id: "rewriteJavaScript",
-			label: "Rewrite JavaScript/TypeScript",
-			description: "vitest, npm/npx/next, tsc, lint, prettier, playwright, prisma",
-			currentValue: toOnOff(config.rewriteJavaScript),
-			values: ON_OFF,
-		},
-		{
-			id: "rewritePython",
-			label: "Rewrite Python",
-			description: "pytest, ruff, pip, uv pip",
-			currentValue: toOnOff(config.rewritePython),
-			values: ON_OFF,
-		},
-		{
-			id: "rewriteGo",
-			label: "Rewrite Go",
-			description: "go test/build/vet and golangci-lint",
-			currentValue: toOnOff(config.rewriteGo),
-			values: ON_OFF,
-		},
-		{
-			id: "rewriteContainers",
-			label: "Rewrite containers",
-			description: "docker compose/ps/images/logs/run/build/exec and kubectl core ops",
-			currentValue: toOnOff(config.rewriteContainers),
-			values: ON_OFF,
-		},
-		{
-			id: "rewriteNetwork",
-			label: "Rewrite network",
-			description: "curl and wget",
-			currentValue: toOnOff(config.rewriteNetwork),
-			values: ON_OFF,
-		},
-		{
-			id: "rewritePackageManagers",
-			label: "Rewrite package managers",
-			description: "pnpm list/ls/outdated/build/typecheck and npm list/outdated",
-			currentValue: toOnOff(config.rewritePackageManagers),
-			values: ON_OFF,
-		},
 	];
 }
 
@@ -296,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,
@@ -418,24 +356,6 @@ function applySetting(config: RtkIntegrationConfig, id: string, value: string):
 					trackSavings: value === "on",
 				},
 			};
-		case "rewriteGitGithub":
-			return { ...config, rewriteGitGithub: value === "on" };
-		case "rewriteFilesystem":
-			return { ...config, rewriteFilesystem: value === "on" };
-		case "rewriteRust":
-			return { ...config, rewriteRust: value === "on" };
-		case "rewriteJavaScript":
-			return { ...config, rewriteJavaScript: value === "on" };
-		case "rewritePython":
-			return { ...config, rewritePython: value === "on" };
-		case "rewriteGo":
-			return { ...config, rewriteGo: value === "on" };
-		case "rewriteContainers":
-			return { ...config, rewriteContainers: value === "on" };
-		case "rewriteNetwork":
-			return { ...config, rewriteNetwork: value === "on" };
-		case "rewritePackageManagers":
-			return { ...config, rewritePackageManagers: value === "on" };
 		default:
 			return config;
 	}
@@ -448,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));
@@ -461,15 +382,6 @@ function syncSettingValues(settingsList: SettingValueSyncTarget, config: RtkInte
 	settingsList.updateValue("outputAggregateLinterOutput", toOnOff(config.outputCompaction.aggregateLinterOutput));
 	settingsList.updateValue("outputGroupSearchOutput", toOnOff(config.outputCompaction.groupSearchOutput));
 	settingsList.updateValue("outputTrackSavings", toOnOff(config.outputCompaction.trackSavings));
-	settingsList.updateValue("rewriteGitGithub", toOnOff(config.rewriteGitGithub));
-	settingsList.updateValue("rewriteFilesystem", toOnOff(config.rewriteFilesystem));
-	settingsList.updateValue("rewriteRust", toOnOff(config.rewriteRust));
-	settingsList.updateValue("rewriteJavaScript", toOnOff(config.rewriteJavaScript));
-	settingsList.updateValue("rewritePython", toOnOff(config.rewritePython));
-	settingsList.updateValue("rewriteGo", toOnOff(config.rewriteGo));
-	settingsList.updateValue("rewriteContainers", toOnOff(config.rewriteContainers));
-	settingsList.updateValue("rewriteNetwork", toOnOff(config.rewriteNetwork));
-	settingsList.updateValue("rewritePackageManagers", toOnOff(config.rewritePackageManagers));
 }
 
 async function openSettingsModal(ctx: ExtensionCommandContext, controller: RtkIntegrationController): Promise<void> {

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),
@@ -60,30 +76,6 @@ export function normalizeRtkIntegrationConfig(raw: unknown): RtkIntegrationConfi
 			source.showRewriteNotifications,
 			DEFAULT_RTK_INTEGRATION_CONFIG.showRewriteNotifications,
 		),
-		rewriteGitGithub: toBoolean(
-			source.rewriteGitGithub,
-			DEFAULT_RTK_INTEGRATION_CONFIG.rewriteGitGithub,
-		),
-		rewriteFilesystem: toBoolean(
-			source.rewriteFilesystem,
-			DEFAULT_RTK_INTEGRATION_CONFIG.rewriteFilesystem,
-		),
-		rewriteRust: toBoolean(source.rewriteRust, DEFAULT_RTK_INTEGRATION_CONFIG.rewriteRust),
-		rewriteJavaScript: toBoolean(
-			source.rewriteJavaScript,
-			DEFAULT_RTK_INTEGRATION_CONFIG.rewriteJavaScript,
-		),
-		rewritePython: toBoolean(source.rewritePython, DEFAULT_RTK_INTEGRATION_CONFIG.rewritePython),
-		rewriteGo: toBoolean(source.rewriteGo, DEFAULT_RTK_INTEGRATION_CONFIG.rewriteGo),
-		rewriteContainers: toBoolean(
-			source.rewriteContainers,
-			DEFAULT_RTK_INTEGRATION_CONFIG.rewriteContainers,
-		),
-		rewriteNetwork: toBoolean(source.rewriteNetwork, DEFAULT_RTK_INTEGRATION_CONFIG.rewriteNetwork),
-		rewritePackageManagers: toBoolean(
-			source.rewritePackageManagers,
-			DEFAULT_RTK_INTEGRATION_CONFIG.rewritePackageManagers,
-		),
 		outputCompaction: {
 			enabled: toBoolean(
 				outputCompactionSource.enabled,
@@ -93,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,
@@ -113,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)
@@ -123,7 +124,6 @@ export default function rtkIntegrationExtension(pi: ExtensionAPI): void {
 			return;
 		}
 
-		console.warn(`[${EXTENSION_NAME}] ${message}`);
 		if (ctx.hasUI) {
 			ctx.ui.notify(message, level);
 		}
@@ -222,7 +222,7 @@ export default function rtkIntegrationExtension(pi: ExtensionAPI): void {
 	};
 
 	const maybeWarnRtkMissing = (ctx: ExtensionContext): void => {
-		if (!config.enabled || config.mode !== "rewrite" || !config.guardWhenRtkMissing) {
+		if (!config.enabled || !config.guardWhenRtkMissing) {
 			return;
 		}
 
@@ -237,7 +237,8 @@ export default function rtkIntegrationExtension(pi: ExtensionAPI): void {
 
 		missingRtkWarningShown = true;
 		const reason = runtimeStatus.lastError ? ` (${runtimeStatus.lastError})` : "";
-		warnOnce(ctx, `${EXTENSION_NAME}: rtk binary unavailable, command rewrite bypassed${reason}.`);
+		const handling = config.mode === "suggest" ? "rewrite suggestions" : "command rewrite";
+		warnOnce(ctx, `${EXTENSION_NAME}: rtk binary unavailable, ${handling} bypassed${reason}.`);
 	};
 
 	const ensureRuntimeStatusFresh = async (): Promise<void> => {
@@ -361,8 +362,8 @@ export default function rtkIntegrationExtension(pi: ExtensionAPI): void {
 			return {};
 		}
 
-		const decision = computeRewriteDecision(event.input.command, config);
-		if (!decision.changed || !decision.rule) {
+		const decision = await computeRewriteDecision(event.input.command, config, pi);
+		if (!decision.changed) {
 			return {};
 		}
 
@@ -376,7 +377,7 @@ export default function rtkIntegrationExtension(pi: ExtensionAPI): void {
 		}
 
 		if (config.mode === "suggest") {
-			const suggestionKey = `${decision.rule.id}:${decision.rewrittenCommand}`;
+			const suggestionKey = `${decision.originalCommand}:${decision.rewrittenCommand}`;
 			if (suggestionNotices.remember(suggestionKey) && ctx.hasUI) {
 				ctx.ui.notify(`RTK suggestion: ${decision.rewrittenCommand}`, "info");
 			}

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