pi-crew 0.2.14 → 0.2.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-crew",
-  "version": "0.2.14",
+  "version": "0.2.16",
   "description": "Pi extension for coordinated AI teams, workflows, worktrees, and async task orchestration",
   "author": "baphuongna",
   "license": "MIT",

package/src/extension/registration/compaction-guard.ts

@@ -67,7 +67,6 @@ function formatCrewArtifactIndex(entries: CrewArtifactIndexEntry[]): string {
 }
 
 export function registerCompactionGuard(pi: ExtensionAPI, options: RegisterCompactionGuardOptions): void {
-	let pendingCompactReason: string | null = null;
 	let compactionInProgress = false;
 
 	const startCompact = (ctx: ExtensionContext, reason: string): void => {
@@ -94,34 +93,22 @@ export function registerCompactionGuard(pi: ExtensionAPI, options: RegisterCompa
 		});
 	};
 
-	// Phase 1.2: Defer compaction during foreground runs unless context is critically full.
+	// Phase 1.2: Allow compaction during foreground runs.
+	// Previously this deferred compaction, but that conflicts with context-mode's PreCompact hook
+	// which needs compaction to run to build resume snapshots.
+	// Instead, we now prioritize crew state in the compaction summary (see startCompact).
 	pi.on("session_before_compact", async (_event, ctx) => {
-		if (options.foregroundControllers.size === 0 && options.foregroundTeamRunControllers.size === 0) return;
-		const ratio = usageRatio(ctx);
-		if (ratio !== undefined && ratio >= HARD_RATIO) {
-			ctx.ui.notify("Compaction allowed despite foreground run: context is critically full", "warning");
-			return;
-		}
-		pendingCompactReason = "deferred-during-foreground-run";
-		ctx.ui.notify("Compaction deferred: foreground team run in progress", "info");
-		return { cancel: true };
+		// Always allow compaction - context-mode needs it for resume snapshots
+		// pi-crew state is preserved via customInstructions and artifactIndex
+		return;
 	});
 
 	// Phase 2.1: Proactive compaction with dynamic threshold based on model context window.
 	pi.on("turn_end", (_event, ctx) => {
 		if (compactionInProgress) return;
-		const hasActiveForeground = options.foregroundControllers.size > 0 || options.foregroundTeamRunControllers.size > 0;
-		if (!hasActiveForeground && pendingCompactReason) {
-			pendingCompactReason = null;
-			startCompact(ctx, "deferred");
-			return;
-		}
 		const ratio = usageRatio(ctx);
 		if (ratio === undefined || ratio < TRIGGER_RATIO) return;
-		if (hasActiveForeground) {
-			pendingCompactReason = "threshold-during-foreground-run";
-			return;
-		}
-		startCompact(ctx, "threshold");
+		// Always allow compaction during foreground runs - context-mode needs it
+		startCompact(ctx, ratio >= HARD_RATIO ? "critical" : "threshold");
 	});
 }

package/src/ui/crew-widget.ts

@@ -156,9 +156,15 @@ function agentStats(agent: CrewAgentRecord, liveHandle?: LiveAgentHandle): strin
 		const rawCompleted = act.completedAtMs || 0;
 		const nowMs = Date.now();
 		const nowSec = Math.floor(nowMs / 1000);
-		// Detect if value is in seconds (Unix timestamp) vs milliseconds
-		// If value looks like Unix seconds (within range of ±2 years from now), convert to ms
-		const isSeconds = (v: number) => v > 1000000000 && v < 2000000000 + 31536000 * 2;
+		// Detect if value is in seconds vs milliseconds by comparing distance to current time
+		// If value is closer to nowSec (within ±2 years) AND not close to nowMs, treat as seconds
+		const isSeconds = (v: number) => {
+			if (v <= 0) return false;
+			const distToSec = Math.abs(v - nowSec);
+			const distToMs = Math.abs(v - nowMs);
+			// If distance to seconds is much smaller AND value is in valid seconds range
+			return distToSec < distToMs && distToSec < 31536000 * 2 && v > 1000000000;
+		};
 		const startedMs = isSeconds(rawStarted) ? rawStarted * 1000 : rawStarted;
 		const completedMs = isSeconds(rawCompleted) ? rawCompleted * 1000 : rawCompleted;
 		// Validate: startedAtMs should be within reasonable bounds

package/src/ui/live-conversation-overlay.ts

@@ -67,8 +67,14 @@ export class LiveConversationOverlay {
 		const rawStarted = act.startedAtMs || 0;
 		const rawCompleted = act.completedAtMs || 0;
 		const nowMs = Date.now();
-		// Detect if value is in seconds vs milliseconds
-		const isSeconds = (v: number) => v > 1000000000 && v < 2000000000 + 31536000 * 2;
+		const nowSec = Math.floor(nowMs / 1000);
+		// Detect if value is in seconds vs milliseconds by comparing distance to current time
+		const isSeconds = (v: number) => {
+			if (v <= 0) return false;
+			const distToSec = Math.abs(v - nowSec);
+			const distToMs = Math.abs(v - nowMs);
+			return distToSec < distToMs && distToSec < 31536000 * 2 && v > 1000000000;
+		};
 		const startedMs = isSeconds(rawStarted) ? rawStarted * 1000 : rawStarted;
 		const completedMs = isSeconds(rawCompleted) ? rawCompleted * 1000 : rawCompleted;
 		const isValidStarted = startedMs > 0 && startedMs < nowMs + 60000 && startedMs > nowMs - 3155692600000;