@henryqw/pi-auto-compact 1.1.10 → 2.0.1

package/README.md

@@ -30,9 +30,9 @@ Restart Pi after install or settings changes. Trusted project settings in `.pi/s
 
 This package requires `@henryqw/pi-task-models` for shared compaction routes.
 
-`~/.pi/agent/config/pi-task-models.json` is shared and owned by `@henryqw/pi-task-models`. The local `pi-auto-compact/autoCompact` declaration defaults to `fast`. A task entry is an explicit user override.
+`~/.pi/agent/config/pi-task-models/config.json` is shared and owned by `@henryqw/pi-task-models`. The local `pi-auto-compact/autoCompact` declaration defaults to `fast`. A task entry is an explicit user override.
 
-Malformed shared task-model config is reported and left unchanged. Compaction then uses the current session model.
+Malformed shared config is reported and left unchanged. Compaction then uses the current session model.
 
 ## Use
 
@@ -46,16 +46,18 @@ Use `/auto-compact` to set the compaction threshold.
 
 ## Config
 
-Package-owned: `~/.pi/agent/config/pi-auto-compact.json`
+Package-owned: `~/.pi/agent/config/pi-auto-compact/config.json`
 
 ```json
 {
-  "autoCompactThreshold": 50
+  "autoCompactThreshold": 70
 }
 ```
 
-`autoCompactThreshold` is optional. It must be a number that is at least 25 and below 100. The default is `50`.
+`autoCompactThreshold` is optional. It must be a number that is at least 25 and below 100. The default is `70`.
 
 Unknown fields are ignored. Legacy model fields are obsolete. `/auto-compact` writes this file.
 
-A missing file uses the default. A malformed or invalid file fails visibly at session start, falls back to 50%, and stays unchanged.
+A missing file uses 70%. Reads do not create it. A malformed or invalid file fails visibly at session start, falls back to 70%, and stays unchanged.
+
+Only `/auto-compact` writes this file. Its write is atomic.

package/extensions/auto-compact.ts

@@ -1,11 +1,10 @@
-import { mkdirSync, readFileSync, writeFileSync } from "node:fs";
-import { dirname, join } from "node:path";
 import {
 	compact,
 	estimateTokens,
 	getAgentDir,
 	SettingsManager,
 } from "@earendil-works/pi-coding-agent";
+import { createConfigStore } from "@henryqw/pi-config-store";
 import type {
 	ExtensionAPI,
 	ExtensionContext,
@@ -30,7 +29,7 @@ type AgentMessage = Parameters<typeof estimateTokens>[0];
  * Pi's ctx.compact() aborts active low-level run internally. Mid-task
  * compaction sends a follow-up user message to resume work after summary.
  */
-const DEFAULT_COMPACT_THRESHOLD_PERCENT = 50;
+const DEFAULT_COMPACT_THRESHOLD_PERCENT = 70;
 const MIN_COMPACT_THRESHOLD_PERCENT = 25;
 export const AUTO_COMPACT_TASK = {
 	id: "pi-auto-compact/autoCompact",
@@ -38,22 +37,13 @@ export const AUTO_COMPACT_TASK = {
 	purpose: "Compact session context before it is exhausted.",
 	defaultProfile: "fast",
 } as const satisfies ModelTask;
-const configPath = () => join(getAgentDir(), "config", "pi-auto-compact.json");
+type AutoCompactConfig = { autoCompactThreshold: number };
 
 function isValidThreshold(value: unknown): value is number {
 	return typeof value === "number" && Number.isFinite(value) && value >= MIN_COMPACT_THRESHOLD_PERCENT && value < 100;
 }
 
-function readConfig(): number {
-	let value: unknown;
-	try {
-		value = JSON.parse(readFileSync(configPath(), "utf8"));
-	} catch (error) {
-		if (error && typeof error === "object" && "code" in error && error.code === "ENOENT") {
-			return DEFAULT_COMPACT_THRESHOLD_PERCENT;
-		}
-		throw error;
-	}
+function parseAutoCompactConfig(value: unknown): AutoCompactConfig {
 	if (!value || typeof value !== "object" || Array.isArray(value)) {
 		throw new Error("Config must be an object.");
 	}
@@ -61,13 +51,15 @@ function readConfig(): number {
 	if (!isValidThreshold(threshold)) {
 		throw new Error(`autoCompactThreshold must be at least ${MIN_COMPACT_THRESHOLD_PERCENT} and below 100.`);
 	}
-	return threshold;
+	return { autoCompactThreshold: threshold };
 }
 
-function writeConfig(threshold: number): void {
-	const file = configPath();
-	mkdirSync(dirname(file), { recursive: true });
-	writeFileSync(file, `${JSON.stringify({ autoCompactThreshold: threshold }, null, 2)}\n`);
+function createAutoCompactConfigStore() {
+	return createConfigStore<AutoCompactConfig>({
+		extensionId: "pi-auto-compact",
+		defaults: () => ({ autoCompactThreshold: DEFAULT_COMPACT_THRESHOLD_PERCENT }),
+		parse: parseAutoCompactConfig,
+	});
 }
 
 function configuredTaskRoutes(ctx: ExtensionContext): ResolvedTaskRoute[] {
@@ -75,6 +67,7 @@ function configuredTaskRoutes(ctx: ExtensionContext): ResolvedTaskRoute[] {
 		return resolveConfiguredTaskRoutes(ctx, AUTO_COMPACT_TASK);
 	} catch (error) {
 		const { taskRouteCode, profileName } = error as TaskRouteError;
+		if (taskRouteCode === "config-missing") return [];
 		const cause = taskRouteCode === "profile-missing"
 			? `Task model profile ${profileName} is not configured`
 			: taskRouteCode === "no-route"
@@ -155,6 +148,7 @@ function hasToolCall(message: AgentMessage): boolean {
 
 export default function (pi: ExtensionAPI) {
 	registerModelTask(pi, AUTO_COMPACT_TASK);
+	const configStore = createAutoCompactConfigStore();
 	let active = false;
 	let autoCompactThreshold = DEFAULT_COMPACT_THRESHOLD_PERCENT;
 	// Prevent lifecycle hooks from starting duplicate summaries.
@@ -256,7 +250,7 @@ export default function (pi: ExtensionAPI) {
 
 			let currentThreshold: number;
 			try {
-				currentThreshold = readConfig();
+				currentThreshold = configStore.loadSync().value.autoCompactThreshold;
 			} catch {
 				ctx.ui.notify("Couldn't read pi-auto-compact config.", "error");
 				return;
@@ -275,7 +269,7 @@ export default function (pi: ExtensionAPI) {
 			}
 
 			try {
-				writeConfig(threshold);
+				await configStore.save({ autoCompactThreshold: threshold });
 			} catch {
 				ctx.ui.notify("Couldn't save pi-auto-compact config.", "error");
 				return;
@@ -289,10 +283,11 @@ export default function (pi: ExtensionAPI) {
 	// activation unless effective global/project settings disable it.
 	pi.on("session_start", (event, ctx) => {
 		try {
-			autoCompactThreshold = readConfig();
+			const config = configStore.loadSync();
+			autoCompactThreshold = config.value.autoCompactThreshold;
 		} catch {
 			autoCompactThreshold = DEFAULT_COMPACT_THRESHOLD_PERCENT;
-			ctx.ui.notify("Couldn't read pi-auto-compact config; using 50%.", "error");
+			ctx.ui.notify(`Couldn't read pi-auto-compact config; using ${DEFAULT_COMPACT_THRESHOLD_PERCENT}%.`, "error");
 		}
 
 		active = !SettingsManager.create(ctx.cwd, getAgentDir(), {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-auto-compact",
-  "version": "1.1.10",
+  "version": "2.0.1",
   "description": "Proactively compact Pi context at a configurable threshold and resume the current task.",
   "keywords": [
     "pi-package",
@@ -44,6 +44,7 @@
     ]
   },
   "dependencies": {
-    "@henryqw/pi-task-models": "^3.0.0"
+    "@henryqw/pi-config-store": "^0.1.0",
+    "@henryqw/pi-task-models": "^4.0.0"
   }
 }