@nijaru/pi-provider-compaction 0.0.10 → 0.0.12

package/README.md

@@ -39,15 +39,14 @@ The summary request extends the captured logical prompt, rather than serializing
 ```json
 {
   "providerCompaction": {
-    "cachedSummaries": true,
-    "outcomeNotifications": false
+    "cachedSummaries": true
   }
 }
 ```
 
 These are the defaults on hosts exposing Pi's `getSettings` API; older hosts retain native compaction with cached summaries disabled. Optional `cacheRetention` accepts `"short"`, `"long"`, or `"none"`; omission inherits Pi/provider behavior. Retention is applied per request, not through process-wide environment changes. A longer tier can increase cache-write charges and only works where the adapter/backend supports it.
 
-A cache hit is best effort, not a promised saving. Countdown estimates use the model's declared `promptCache` lifetime for the active tier, measured conservatively from request dispatch. Missing lifetimes remain unknown: DeepSeek, for example, [documents best-effort eviction over hours to days](https://api-docs.deepseek.com/guides/kv_cache/), not a predictable expiry. Pi's successful warmer requests refresh the observed cycle and countdown.
+A cache hit is best effort, not a promised saving. Expiry estimates use the model's declared `promptCache` lifetime for the active tier, measured conservatively from request dispatch. Missing lifetimes remain unknown: DeepSeek, for example, [documents best-effort eviction over hours to days](https://api-docs.deepseek.com/guides/kv_cache/), not a predictable expiry. Pi's successful warmer requests refresh the observed cycle and estimated expiry.
 
 With [pi-compactor's opt-in idle expiry policy](https://github.com/nijaru/pi-compactor#cache-aware-idle-compaction), compaction can run before an estimated expiry. Both extensions share a live ownership/input guard, checked before paid summary and native-checkpoint stages regardless of extension order. A failed or revoked cached-only attempt cancels instead of starting an ordinary summary. It does not start a continuation.
 
@@ -77,9 +76,9 @@ Legacy v1/v2 whole-history checkpoints remain inspectable but are never reinterp
 
 ## Diagnose portable fallback
 
-Run `/provider-compaction-status` to see observer/capture eligibility, estimated cache lifetime, the last selected path, elapsed time, reported input/cache/output tokens, and estimated cost (including available companion corrections). Optional outcome notifications show measured usage after a committed compaction; hypothetical savings are not presented as measured. Failed/cancelled expiry attempts remain visible in runtime status; if no checkpoint is saved, their usage cannot be attached to compaction totals through Pi's extension API. Portable fallback also appends a `pi-provider-compaction:diagnostic` session entry containing only a fixed reason string. It is not included in model context. Diagnostics never include request content, credentials, or backend error text.
+Committed compaction entries retain the selected path, elapsed time, reported usage, and estimated cost. Portable fallback appends a `pi-provider-compaction:diagnostic` session entry with a fixed reason. Cancelled attempts with reported usage retain that usage in the same diagnostic record; without a saved checkpoint, Pi's extension API cannot include it in compaction totals. Diagnostics stay out of model context and never include request content, credentials, or backend error text.
 
-These diagnostics require the extension to be loaded when the attempt happens; they cannot explain earlier compactions retroactively. A committed native checkpoint does not by itself prove that a later request replayed it.
+These records describe attempts made while the extension was loaded; they cannot explain earlier compactions retroactively. A committed native checkpoint does not by itself prove that a later request replayed it.
 
 ## Generic compaction-model override
 

package/cached-summary.ts

@@ -17,13 +17,12 @@ export interface CacheState {
 	cachedTokens: number;
 	inputTokens: number;
 }
-export interface CacheConfig { cachedSummaries: boolean; cacheRetention?: CacheRetention; outcomeNotifications: boolean }
+export interface CacheConfig { cachedSummaries: boolean; cacheRetention?: CacheRetention }
 export function cacheConfig(settings: unknown): CacheConfig {
 	const raw = isObject(settings) && isObject(settings.providerCompaction) ? settings.providerCompaction : {};
 	return {
 		cachedSummaries: raw.cachedSummaries !== false,
 		cacheRetention: raw.cacheRetention === "long" || raw.cacheRetention === "short" || raw.cacheRetention === "none" ? raw.cacheRetention : undefined,
-		outcomeNotifications: raw.outcomeNotifications === true,
 	};
 }
 export function supportsCachedSummary(model: Model<any> | undefined): boolean {

package/dist/index.js

@@ -1591,8 +1591,7 @@ function cacheConfig(settings) {
   const raw = isObject(settings) && isObject(settings.providerCompaction) ? settings.providerCompaction : {};
   return {
     cachedSummaries: raw.cachedSummaries !== false,
-    cacheRetention: raw.cacheRetention === "long" || raw.cacheRetention === "short" || raw.cacheRetention === "none" ? raw.cacheRetention : undefined,
-    outcomeNotifications: raw.outcomeNotifications === true
+    cacheRetention: raw.cacheRetention === "long" || raw.cacheRetention === "short" || raw.cacheRetention === "none" ? raw.cacheRetention : undefined
   };
 }
 function supportsCachedSummary(model) {
@@ -2257,7 +2256,7 @@ function supportedRouteOptions(model, options) {
   return !["AZURE_OPENAI_BASE_URL", "AZURE_OPENAI_RESOURCE_NAME", "AZURE_OPENAI_API_VERSION", "AZURE_OPENAI_DEPLOYMENT_NAME_MAP"].some((key) => process.env[key] !== undefined && options.env?.[key] === undefined);
 }
 function pi_provider_compaction_default(pi) {
-  const configuration = () => typeof pi.getSettings === "function" ? cacheConfig(pi.getSettings()) : { cachedSummaries: false, outcomeNotifications: false, cacheRetention: undefined };
+  const configuration = () => typeof pi.getSettings === "function" ? cacheConfig(pi.getSettings()) : { cachedSummaries: false, cacheRetention: undefined };
   const secret = randomBytes(32);
   const routeDigest = (value) => createHmac("sha256", secret).update(JSON.stringify(value)).digest("hex");
   let runtimeId = randomUUID2();
@@ -2274,30 +2273,10 @@ function pi_provider_compaction_default(pi) {
   let unregisterCleanup;
   let generation = 0;
   let requestSequence = 0;
-  let captureStatus = "no request observed";
-  let compactionStatus = "not attempted in this runtime";
   let fallbackReason;
-  const captureRejected = (reason) => {
-    captureStatus = reason;
-  };
   const fallback = (reason) => {
     fallbackReason = reason;
-    compactionStatus = `portable: ${reason}`;
   };
-  pi.registerCommand("provider-compaction-status", {
-    description: "Show cache/native eligibility and measured compaction usage",
-    handler: async (_args, context) => {
-      const state = cacheState(context);
-      const remaining = state.expiresAt === undefined ? "unknown lifetime" : `estimated ${Math.max(0, Math.ceil((state.expiresAt - Date.now()) / 1000))}s remaining (model declaration; best effort)`;
-      const usage = lastOutcome?.usage;
-      context.ui.notify(`Provider compaction
-Observer: ${observer?.isInstalled() ? "installed" : "unavailable for current registration"}
-Capture: ${captureStatus}
-Cache: ${state.reason}; ${remaining}
-Last attempt: ${compactionStatus}${lastOutcome ? `
-Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usage.input}, cache read ${usage.cacheRead}, write ${usage.cacheWrite}, output ${usage.output}; estimated cost $${usage.cost.total.toFixed(6)}` : "; usage unavailable"}${lastOutcome.reason ? `; ${lastOutcome.reason}` : ""}` : ""}`, "info");
-    }
-  });
   const invalidate = () => {
     generation++;
     runtimeId = randomUUID2();
@@ -2306,7 +2285,6 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
     active = undefined;
     pending = undefined;
     expiryAttempt = undefined;
-    captureStatus = "invalidated by lifecycle/history/policy change";
   };
   const retire = () => {
     usageObservers.clear();
@@ -2396,23 +2374,12 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
       return;
     if (compactionRequest)
       return;
-    if (!supportsNativeCompaction(model) && !supportsCachedSummary(model)) {
-      captureRejected("unsupported API");
+    if (!supportsNativeCompaction(model) && !supportsCachedSummary(model))
       return;
-    }
-    if (!options.onPayload) {
-      if (captureStatus !== "eligible prepared request captured")
-        captureRejected("no payload callback");
+    if (!options.onPayload || generic(owner))
       return;
-    }
-    if (generic(owner)) {
-      captureRejected("generic compaction model takes precedence");
-      return;
-    }
-    if (!supportedRouteOptions(model, options)) {
-      captureRejected("unsupported route overrides");
+    if (!supportedRouteOptions(model, options))
       return;
-    }
     const ticket = generation;
     const startedAt = Date.now();
     const sequence = ++requestSequence;
@@ -2443,14 +2410,11 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
       if (!auth.ok || generic(owner) || auth.apiKey !== options.apiKey || (auth.baseUrl ?? model.baseUrl) !== model.baseUrl || !equal(normalizedHeaders(auth.headers), normalizedHeaders(options.headers)) || !equal(auth.env ?? {}, options.env ?? {})) {
         active = undefined;
         cache = undefined;
-        captureRejected("resolved auth/route differs from prepared request, or generic policy changed");
         return payload;
       }
       const provider = owner.modelRegistry.getProvider(model.provider);
-      if (!provider) {
-        captureRejected("provider unavailable");
+      if (!provider)
         return payload;
-      }
       const routeHash = route(model, options);
       if (config.cachedSummaries && supportsCachedSummary(model)) {
         const warming = options.maxTokens === 1 && cache && cache.authHash === routeDigest(auth) && sameCachedPrompt(cache.payload, payload) && equal(cache.context, context);
@@ -2469,11 +2433,9 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
         if (!warming)
           cache = undefined;
       }
-      if (!supportsNativeCompaction(model)) {
-        captureStatus = candidate ? "eligible cached request captured" : "cached request ineligible";
+      if (!supportsNativeCompaction(model))
         return payload;
-      }
-      const prepared = capturePreparedRequest(model, context, canonical, payload, captureRejected);
+      const prepared = capturePreparedRequest(model, context, canonical, payload);
       if (!prepared) {
         active = undefined;
         return payload;
@@ -2502,7 +2464,6 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
           active = undefined;
       } else
         active = undefined;
-      captureStatus = "eligible prepared request captured";
       snapshot = {
         prepared,
         model: structuredClone(model),
@@ -2540,7 +2501,6 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
     if (!supportsNativeCompaction(context.model) && !supportsCachedSummary(context.model))
       return;
     observer = installObserver(pi, context, observe);
-    captureStatus = observer ? "awaiting prepared request" : "unsupported provider registration";
   };
   pi.on("session_start", (_event, context) => {
     retire();
@@ -2581,8 +2541,8 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
       removeInvalidation();
   });
   pi.on("session_before_compact", async (event, context) => {
-    const began = Date.now();
-    compactionStatus = "preparing compaction";
+    const began = Date.now(), ticket = generation;
+    const sessionId = context.sessionManager.getSessionId();
     fallbackReason = undefined;
     const attempt = expiryAttempt;
     const capture = snapshot;
@@ -2591,7 +2551,12 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
     cache = undefined;
     pending = undefined;
     const cancelled = (usage, reason = "compaction ownership or input changed") => {
-      lastOutcome = { path: attempt ? "expiry cancelled" : "cancelled", elapsedMs: Date.now() - began, usage, reason };
+      if (usage && ticket === generation && ctx) {
+        try {
+          if (ctx.sessionManager.getSessionId() === sessionId)
+            pi.appendEntry("pi-provider-compaction:diagnostic", { status: `${attempt ? "expiry cancelled" : "cancelled"}: ${reason}`, usage });
+        } catch {}
+      }
       return { cancel: true };
     };
     const reject = (reason) => {
@@ -2605,7 +2570,6 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
     }
     if (!capture && !cached || !context.model || generic(context) || event.signal.aborted)
       return reject("no eligible snapshot, explicit model override, or aborted attempt");
-    const ticket = generation;
     const branch = context.sessionManager.getBranch();
     const branchHash = digest(branch);
     const source = cached ?? capture;
@@ -2630,7 +2594,6 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
     let summaryPath = "standard";
     let summaryReason;
     if (cached) {
-      compactionStatus = "generating cached portable summary";
       compactionRequest = true;
       try {
         const result = await cachedSummary(cached, context, event.preparation, event.customInstructions, event.signal, current);
@@ -2650,7 +2613,6 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
         return cancelled(attemptedUsage, summaryReason ?? "warm cache unavailable");
       if (!input && !attemptedUsage)
         return reject(summaryReason ?? fallbackReason ?? "native coverage unavailable");
-      compactionStatus = "generating portable summary";
       try {
         portable = await compact(event.preparation, source.model, auth.apiKey, auth.headers, event.customInstructions, event.signal, context.thinkingLevel, source.provider.streamSimple.bind(source.provider), auth.env, undefined, undefined, source.sessionId);
         portable = withFileTracking(portable, event.preparation);
@@ -2669,7 +2631,6 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
       return { compaction: result };
     };
     if (!capture || !protocol || !input) {
-      compactionStatus = `${summaryPath} portable summary prepared`;
       return outcome(summaryPath, portable, summaryReason);
     }
     try {
@@ -2680,7 +2641,6 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
         fallback("auth changed after portable generation");
         return outcome(summaryPath, portable, "native auth changed");
       }
-      compactionStatus = "requesting native checkpoint";
       let native;
       compactionRequest = true;
       try {
@@ -2742,19 +2702,8 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
   pi.on("session_compact", (event, context) => {
     const recorded = isObject(event.compactionEntry.details) ? event.compactionEntry.details.providerCompaction : undefined;
     const owned = isObject(recorded) && equal(recorded, lastOutcome);
-    if (readNativeCompactionDetails(event.compactionEntry.details)?.version === 3)
-      compactionStatus = "native checkpoint committed";
-    else if (fallbackReason)
-      compactionStatus = `portable: ${fallbackReason}`;
-    else if (owned)
-      compactionStatus = `${lastOutcome.path} portable summary committed`;
-    else
+    if (!owned && !fallbackReason && readNativeCompactionDetails(event.compactionEntry.details)?.version !== 3)
       fallback("another handler supplied the committed result");
-    if (owned) {
-      if (configuration().outcomeNotifications && lastOutcome)
-        context.ui.notify(`Compacted via ${lastOutcome.path} in ${(lastOutcome.elapsedMs / 1000).toFixed(1)}s${lastOutcome.usage ? `; cache read ${lastOutcome.usage.cacheRead}, write ${lastOutcome.usage.cacheWrite}; estimated cost $${lastOutcome.usage.cost.total.toFixed(6)}` : ""}`, "info");
-    } else
-      lastOutcome = { path: "Pi/other handler", elapsedMs: 0, usage: event.compactionEntry.usage, reason: compactionStatus };
     if (fallbackReason)
       pi.appendEntry("pi-provider-compaction:diagnostic", { status: `portable: ${fallbackReason}` });
     active = undefined;
@@ -2783,7 +2732,6 @@ Outcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usa
     pending = undefined;
     snapshot = undefined;
     cache = undefined;
-    compactionStatus = "compaction failed or cancelled";
   });
 }
 export {

package/index.ts

@@ -157,7 +157,7 @@ export default function (pi: ExtensionAPI): void {
 	// hook/privacy policy, even if the visible summary text happens to match.
 	// Older hosts in the native adapter's supported range lack getSettings.
 	// Keep their existing native path; cache preferences require that host API.
-	const configuration = () => typeof pi.getSettings === "function" ? cacheConfig(pi.getSettings()) : { cachedSummaries: false, outcomeNotifications: false, cacheRetention: undefined };
+	const configuration = () => typeof pi.getSettings === "function" ? cacheConfig(pi.getSettings()) : { cachedSummaries: false, cacheRetention: undefined };
 	const secret = randomBytes(32);
 	const routeDigest = (value: unknown) => createHmac("sha256", secret).update(JSON.stringify(value)).digest("hex");
 	let runtimeId = randomUUID();
@@ -174,21 +174,9 @@ export default function (pi: ExtensionAPI): void {
 	let unregisterCleanup: (() => void) | undefined;
 	let generation = 0;
 	let requestSequence = 0;
-	let captureStatus = "no request observed";
-	let compactionStatus = "not attempted in this runtime";
 	let fallbackReason: string | undefined;
-	const captureRejected = (reason: string) => { captureStatus = reason; };
-	const fallback = (reason: string) => { fallbackReason = reason; compactionStatus = `portable: ${reason}`; };
-	pi.registerCommand("provider-compaction-status", {
-		description: "Show cache/native eligibility and measured compaction usage",
-		handler: async (_args, context) => {
-			const state = cacheState(context);
-			const remaining = state.expiresAt === undefined ? "unknown lifetime" : `estimated ${Math.max(0, Math.ceil((state.expiresAt - Date.now()) / 1000))}s remaining (model declaration; best effort)`;
-			const usage = lastOutcome?.usage;
-			context.ui.notify(`Provider compaction\nObserver: ${observer?.isInstalled() ? "installed" : "unavailable for current registration"}\nCapture: ${captureStatus}\nCache: ${state.reason}; ${remaining}\nLast attempt: ${compactionStatus}${lastOutcome ? `\nOutcome: ${lastOutcome.path}, ${lastOutcome.elapsedMs}ms${usage ? `; input ${usage.input}, cache read ${usage.cacheRead}, write ${usage.cacheWrite}, output ${usage.output}; estimated cost $${usage.cost.total.toFixed(6)}` : "; usage unavailable"}${lastOutcome.reason ? `; ${lastOutcome.reason}` : ""}` : ""}`, "info");
-		},
-	});
-	const invalidate = () => { generation++; runtimeId = randomUUID(); snapshot = undefined; cache = undefined; active = undefined; pending = undefined; expiryAttempt = undefined; captureStatus = "invalidated by lifecycle/history/policy change"; };
+	const fallback = (reason: string) => { fallbackReason = reason; };
+	const invalidate = () => { generation++; runtimeId = randomUUID(); snapshot = undefined; cache = undefined; active = undefined; pending = undefined; expiryAttempt = undefined; };
 	const retire = () => {
 		usageObservers.clear();
 		invalidate();
@@ -260,17 +248,11 @@ export default function (pi: ExtensionAPI): void {
 		}
 		if (options.sessionId !== manager.getSessionId()) return;
 		if (compactionRequest) return;
-		if (!supportsNativeCompaction(model) && !supportsCachedSummary(model)) { captureRejected("unsupported API"); return; }
-		if (!options.onPayload) {
-			// The portable-summary request re-enters this observer without a payload
-			// callback. Do not overwrite the main request's capture diagnostic.
-			if (captureStatus !== "eligible prepared request captured") captureRejected("no payload callback");
-			return;
-		}
-		if (generic(owner)) { captureRejected("generic compaction model takes precedence"); return; }
+		if (!supportsNativeCompaction(model) && !supportsCachedSummary(model)) return;
+		if (!options.onPayload || generic(owner)) return;
 		// The Azure adapter also accepts API-specific overrides and process-env
 		// fallbacks. Those must not silently select a route absent from our evidence.
-		if (!supportedRouteOptions(model, options)) { captureRejected("unsupported route overrides"); return; }
+		if (!supportedRouteOptions(model, options)) return;
 		const ticket = generation;
 		const startedAt = Date.now();
 		const sequence = ++requestSequence;
@@ -296,11 +278,10 @@ export default function (pi: ExtensionAPI): void {
 			const normalizedHeaders = (headers: ProviderHeaders | undefined) => Object.entries(headers ?? {}).map(([name, value]) => [name.toLowerCase(), value]).sort(([a], [b]) => String(a).localeCompare(String(b)));
 			if (!auth.ok || generic(owner) || auth.apiKey !== options.apiKey || (auth.baseUrl ?? model.baseUrl) !== model.baseUrl || !equal(normalizedHeaders(auth.headers), normalizedHeaders(options.headers)) || !equal(auth.env ?? {}, options.env ?? {})) {
 				active = undefined; cache = undefined;
-				captureRejected("resolved auth/route differs from prepared request, or generic policy changed");
 				return payload;
 			}
 			const provider = owner.modelRegistry.getProvider(model.provider);
-			if (!provider) { captureRejected("provider unavailable"); return payload; }
+			if (!provider) return payload;
 			const routeHash = route(model, options);
 			if (config.cachedSummaries && supportsCachedSummary(model)) {
 				const warming = options.maxTokens === 1 && cache && cache.authHash === routeDigest(auth) && sameCachedPrompt(cache.payload, payload) && equal(cache.context, context);
@@ -310,8 +291,8 @@ export default function (pi: ExtensionAPI): void {
 				}, payload);
 				if (!warming) cache = undefined;
 			}
-			if (!supportsNativeCompaction(model)) { captureStatus = candidate ? "eligible cached request captured" : "cached request ineligible"; return payload; }
-			const prepared = capturePreparedRequest(model, context, canonical, payload, captureRejected);
+			if (!supportsNativeCompaction(model)) return payload;
+			const prepared = capturePreparedRequest(model, context, canonical, payload);
 			if (!prepared) { active = undefined; return payload; }
 			const policyHash = digest(requestPolicy(model, prepared));
 			const latest = latestCompaction(branch);
@@ -334,7 +315,6 @@ export default function (pi: ExtensionAPI): void {
 					} else active = undefined;
 				} else active = undefined;
 			} else active = undefined;
-			captureStatus = "eligible prepared request captured";
 			snapshot = {
 				prepared, model: structuredClone(model), provider,
 				options: { apiKey: options.apiKey, headers: structuredClone(options.headers), env: structuredClone(options.env), fetch: options.fetch, ...(typeof (options as { serviceTier?: unknown }).serviceTier === "string" ? { serviceTier: (options as { serviceTier: string }).serviceTier } : {}) },
@@ -359,7 +339,6 @@ export default function (pi: ExtensionAPI): void {
 		if (observer) { invalidate(); observer.remove(); observer = undefined; }
 		if (!supportsNativeCompaction(context.model) && !supportsCachedSummary(context.model)) return;
 		observer = installObserver(pi, context, observe);
-		captureStatus = observer ? "awaiting prepared request" : "unsupported provider registration";
 	};
 	pi.on("session_start", (_event, context) => {
 		retire();
@@ -386,15 +365,21 @@ export default function (pi: ExtensionAPI): void {
 	pi.on("session_shutdown", () => { if (typeof removeInvalidation === "function") removeInvalidation(); });
 
 	pi.on("session_before_compact", async (event, context) => {
-		const began = Date.now();
-		compactionStatus = "preparing compaction";
+		const began = Date.now(), ticket = generation;
+		const sessionId = context.sessionManager.getSessionId();
 		fallbackReason = undefined;
 		const attempt = expiryAttempt;
 		const capture = snapshot;
 		const cached = cacheState(context).available ? cache : undefined;
 		snapshot = undefined; cache = undefined; pending = undefined;
 		const cancelled = (usage?: Usage, reason = "compaction ownership or input changed") => {
-			lastOutcome = { path: attempt ? "expiry cancelled" : "cancelled", elapsedMs: Date.now() - began, usage, reason };
+			// A cancelled attempt has no checkpoint to carry its known bill. Retain
+			// that usage in the existing session diagnostic, not a command-only cache.
+			if (usage && ticket === generation && ctx) {
+				try {
+					if (ctx.sessionManager.getSessionId() === sessionId) pi.appendEntry("pi-provider-compaction:diagnostic", { status: `${attempt ? "expiry cancelled" : "cancelled"}: ${reason}`, usage });
+				} catch { /* A disposed session cannot accept a diagnostic. */ }
+			}
 			return { cancel: true as const };
 		};
 		const reject = (reason: string) => {
@@ -408,7 +393,6 @@ export default function (pi: ExtensionAPI): void {
 			attempt.signal = event.signal;
 		}
 		if ((!capture && !cached) || !context.model || generic(context) || event.signal.aborted) return reject("no eligible snapshot, explicit model override, or aborted attempt");
-		const ticket = generation;
 		const branch = context.sessionManager.getBranch();
 		const branchHash = digest(branch);
 		const source = cached ?? capture!;
@@ -431,7 +415,6 @@ export default function (pi: ExtensionAPI): void {
 		let summaryPath = "standard";
 		let summaryReason: string | undefined;
 		if (cached) {
-			compactionStatus = "generating cached portable summary";
 			compactionRequest = true;
 			try {
 				const result = await cachedSummary(cached, context, event.preparation, event.customInstructions, event.signal, current);
@@ -446,7 +429,6 @@ export default function (pi: ExtensionAPI): void {
 			if (attempt) return cancelled(attemptedUsage, summaryReason ?? "warm cache unavailable");
 			// Without native coverage or a dispatched cached attempt, Pi can own fallback.
 			if (!input && !attemptedUsage) return reject(summaryReason ?? fallbackReason ?? "native coverage unavailable");
-			compactionStatus = "generating portable summary";
 			try {
 				portable = await compact(event.preparation, source.model, auth.apiKey, auth.headers as Record<string, string> | undefined, event.customInstructions, event.signal, context.thinkingLevel, source.provider.streamSimple.bind(source.provider), auth.env, undefined, undefined, source.sessionId);
 				portable = withFileTracking(portable, event.preparation);
@@ -463,14 +445,12 @@ export default function (pi: ExtensionAPI): void {
 			return { compaction: result! };
 		};
 		if (!capture || !protocol || !input) {
-			compactionStatus = `${summaryPath} portable summary prepared`;
 			return outcome(summaryPath, portable, summaryReason);
 		}
 		try {
 			const freshAuth = await context.modelRegistry.getApiKeyAndHeaders(context.model);
 			if (!current()) return cancelled(portable.usage);
 			if (!freshAuth.ok || routeDigest(freshAuth) !== capture.authHash) { fallback("auth changed after portable generation"); return outcome(summaryPath, portable, "native auth changed"); }
-			compactionStatus = "requesting native checkpoint";
 			let native: Awaited<ReturnType<typeof requestProviderCompaction>>;
 			compactionRequest = true;
 			try {
@@ -512,13 +492,7 @@ export default function (pi: ExtensionAPI): void {
 	pi.on("session_compact", (event, context) => {
 		const recorded = isObject(event.compactionEntry.details) ? event.compactionEntry.details.providerCompaction : undefined;
 		const owned = isObject(recorded) && equal(recorded, lastOutcome);
-		if (readNativeCompactionDetails(event.compactionEntry.details)?.version === 3) compactionStatus = "native checkpoint committed";
-		else if (fallbackReason) compactionStatus = `portable: ${fallbackReason}`;
-		else if (owned) compactionStatus = `${lastOutcome!.path} portable summary committed`;
-		else fallback("another handler supplied the committed result");
-		if (owned) {
-			if (configuration().outcomeNotifications && lastOutcome) context.ui.notify(`Compacted via ${lastOutcome.path} in ${(lastOutcome.elapsedMs / 1000).toFixed(1)}s${lastOutcome.usage ? `; cache read ${lastOutcome.usage.cacheRead}, write ${lastOutcome.usage.cacheWrite}; estimated cost $${lastOutcome.usage.cost.total.toFixed(6)}` : ""}`, "info");
-		} else lastOutcome = { path: "Pi/other handler", elapsedMs: 0, usage: event.compactionEntry.usage, reason: compactionStatus };
+		if (!owned && !fallbackReason && readNativeCompactionDetails(event.compactionEntry.details)?.version !== 3) fallback("another handler supplied the committed result");
 		// Only static diagnostics are persisted, never request/auth data or error text.
 		if (fallbackReason) pi.appendEntry("pi-provider-compaction:diagnostic", { status: `portable: ${fallbackReason}` });
 		active = undefined; snapshot = undefined; cache = undefined;
@@ -538,5 +512,5 @@ export default function (pi: ExtensionAPI): void {
 			...(typeof candidate.capture.prepared.payload.model === "string" ? { payloadModel: candidate.capture.prepared.payload.model } : {}),
 		};
 	});
-	pi.on("session_compact_failed", () => { pending = undefined; snapshot = undefined; cache = undefined; compactionStatus = "compaction failed or cancelled"; });
+	pi.on("session_compact_failed", () => { pending = undefined; snapshot = undefined; cache = undefined; });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nijaru/pi-provider-compaction",
-  "version": "0.0.10",
+  "version": "0.0.12",
   "type": "module",
   "description": "Provider-native context compaction adapters for Pi",
   "keywords": [