@xynogen/pix-core 0.5.1 → 0.5.6

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-core",
-	"version": "0.5.1",
+	"version": "0.5.6",
 	"description": "Pi extension bundle — installs and activates all core pix-* extensions",
 	"type": "module",
 	"main": "src/extension.ts",
@@ -38,31 +38,31 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@xynogen/pix-runtime": "^0.5.2",
-		"@xynogen/pix-data": "^0.4.1",
-		"@xynogen/pix-pretty": "^1.11.0",
-		"@xynogen/pix-welcome": "^0.1.19",
-		"@xynogen/pix-footer": "^0.1.21",
-		"@xynogen/pix-diagnostics": "^0.1.10",
-		"@xynogen/pix-display": "^0.1.14",
-		"@xynogen/pix-prompts": "^0.3.5",
-		"@xynogen/pix-skills": "^0.7.8",
-		"@xynogen/pix-models": "^0.1.27",
-		"@xynogen/pix-update": "^0.1.18",
-		"@xynogen/pix-commands": "^0.3.2",
-		"@xynogen/pix-nudge": "^0.1.13",
-		"@xynogen/pix-read": "^0.1.20",
-		"@xynogen/pix-write": "^0.1.19",
-		"@xynogen/pix-edit": "^0.1.21",
-		"@xynogen/pix-find": "^0.1.20",
-		"@xynogen/pix-grep": "^0.1.20",
-		"@xynogen/pix-ls": "^0.1.20",
-		"@xynogen/pix-bash": "^0.1.19",
-		"@xynogen/pix-todo": "^0.1.21",
-		"@xynogen/pix-ask": "^0.2.16",
-		"@xynogen/pix-optimizer": "^1.1.22",
-		"@xynogen/pix-gate": "^0.2.3",
-		"@xynogen/pix-subagent": "^0.6.5"
+		"@xynogen/pix-runtime": "^0.5.3",
+		"@xynogen/pix-data": "^0.4.3",
+		"@xynogen/pix-pretty": "^1.11.2",
+		"@xynogen/pix-welcome": "^0.1.21",
+		"@xynogen/pix-footer": "^0.1.24",
+		"@xynogen/pix-diagnostics": "^0.1.12",
+		"@xynogen/pix-display": "^0.1.16",
+		"@xynogen/pix-prompts": "^0.3.7",
+		"@xynogen/pix-skills": "^0.7.12",
+		"@xynogen/pix-models": "^0.1.31",
+		"@xynogen/pix-update": "^0.1.20",
+		"@xynogen/pix-commands": "^0.3.5",
+		"@xynogen/pix-nudge": "^0.1.15",
+		"@xynogen/pix-read": "^0.1.22",
+		"@xynogen/pix-write": "^0.1.21",
+		"@xynogen/pix-edit": "^0.1.24",
+		"@xynogen/pix-find": "^0.1.23",
+		"@xynogen/pix-grep": "^0.1.23",
+		"@xynogen/pix-ls": "^0.1.23",
+		"@xynogen/pix-bash": "^0.1.22",
+		"@xynogen/pix-todo": "^0.1.23",
+		"@xynogen/pix-ask": "^0.2.19",
+		"@xynogen/pix-optimizer": "^1.1.24",
+		"@xynogen/pix-gate": "^0.2.6",
+		"@xynogen/pix-subagent": "^0.6.7"
 	},
 	"peerDependencies": {
 		"@earendil-works/pi-ai": "*",

package/src/compaction.ts

@@ -65,7 +65,10 @@ export function compactionThresholdTokens(
 	triggerPercent: number,
 	minimumTokens: number,
 ): number {
-	return Math.max(minimumTokens, Math.ceil((contextWindow * triggerPercent) / 100));
+	const cw = Number.isFinite(contextWindow) ? Math.max(0, Math.floor(contextWindow)) : 0;
+	const pct = Number.isFinite(triggerPercent) ? Math.min(100, Math.max(0, triggerPercent)) : 0;
+	const floor = Number.isFinite(minimumTokens) ? Math.max(0, Math.round(minimumTokens)) : 100_000;
+	return Math.max(floor, Math.ceil((cw * pct) / 100));
 }
 
 /**
@@ -196,14 +199,9 @@ export default function registerCompaction(pi: ExtensionAPI): void {
 		ctx.compact({
 			onComplete: (result) => {
 				compacting = false;
-				const decision = resumeDecisionAfterCompaction({
-					estimatedTokensAfter: result.estimatedTokensAfter,
-					threshold,
-					idle: ctx.isIdle(),
-					hasPending: ctx.hasPendingMessages(),
-				});
-
-				if (decision === "loop") {
+
+				// Loop check is token-based (timing-independent) — decide it now.
+				if ((result.estimatedTokensAfter ?? 0) >= threshold) {
 					// Still over the line (e.g. large minimumTokens floor on a small model).
 					// Latch the auto-trigger off so the resume turn doesn't re-fire
 					// agent_settled and compact every turn. Reset by /compact or new session.
@@ -215,27 +213,48 @@ export default function registerCompaction(pi: ExtensionAPI): void {
 					return;
 				}
 
-				if (decision === "skip") {
-					// User is already driving; skipping avoids colliding with their in-flight
-					// prompt and avoids overriding an explicit redirect. The fresh summary is
-					// already in context, so the agent continues without a nudge.
-					ctx.ui.notify("Compaction: resume nudge skipped (your prompt is in flight)", "info");
-					return;
-				}
+				// Defer the resume/skip decision by one macrotask. A message the user
+				// typed DURING compaction lives in the TUI's own compactionQueuedMessages
+				// buffer, which ctx.hasPendingMessages()/isIdle() cannot see. The host
+				// flushes that buffer on compaction_end by starting a new turn; deferring
+				// lets that turn's prompt() microtask chain flip the session to streaming
+				// first, so the re-read below reports busy and we skip the nudge. Without
+				// this, our nudge races the user's steer, wins the turn, and leaves their
+				// message queued "stuck" behind it — and re-points the agent at the old task.
+				// ponytail: setTimeout(0) drains microtasks; if the flush's prompt() blocks
+				// on a real-async auth check it may still be idle here and the nudge fires
+				// (user steer then queues behind it). Rare; upgrade path is a host API that
+				// exposes the TUI compaction buffer or a post-flush event.
+				setTimeout(() => {
+					const decision = resumeDecisionAfterCompaction({
+						estimatedTokensAfter: 0, // loop already handled above
+						threshold,
+						idle: ctx.isIdle(),
+						hasPending: ctx.hasPendingMessages(),
+					});
 
-				// Idle: nudge the agent to pick its task back up. Visible as a normal user
-				// message in the transcript (no hidden automation). deliverAs "followUp" is
-				// a backstop for the narrow await-window where a prompt starts between the
-				// isIdle() check and this send; pi swallows a residual rejection into its
-				// own emitError, so it can't crash the session. The try/catch only guards a
-				// stale-ctx throw from sendUserMessage.
-				ctx.ui.notify("Compaction: resuming task", "info");
-				try {
-					pi.sendUserMessage(RESUME_NUDGE, { deliverAs: "followUp" });
-				} catch (err) {
-					const message = err instanceof Error ? err.message : String(err);
-					ctx.ui.notify(`Compaction: resume nudge skipped (${message})`, "warning");
-				}
+					if (decision === "skip") {
+						// User is already driving (in-flight or freshly flushed prompt); skipping
+						// avoids colliding with it and avoids overriding an explicit redirect. The
+						// fresh summary is already in context, so the agent continues without a nudge.
+						ctx.ui.notify("Compaction: resume nudge skipped (your prompt is in flight)", "info");
+						return;
+					}
+
+					// Idle: nudge the agent to pick its task back up. Visible as a normal user
+					// message in the transcript (no hidden automation). deliverAs "followUp" is
+					// a backstop for the narrow await-window where a prompt starts between the
+					// isIdle() check and this send; pi swallows a residual rejection into its
+					// own emitError, so it can't crash the session. The try/catch only guards a
+					// stale-ctx throw from sendUserMessage.
+					ctx.ui.notify("Compaction: resuming task", "info");
+					try {
+						pi.sendUserMessage(RESUME_NUDGE, { deliverAs: "followUp" });
+					} catch (err) {
+						const message = err instanceof Error ? err.message : String(err);
+						ctx.ui.notify(`Compaction: resume nudge skipped (${message})`, "warning");
+					}
+				}, 0);
 			},
 			onError: (err) => {
 				compacting = false;