pi-harness-runtime 1.1.184 → 1.1.185

package/index.ts

@@ -158,18 +158,87 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 		let autoContinueTimer: NodeJS.Timeout | null = null;
 		let _inferenceInProgress = false; // Track for overlap prevention
 		let _sessionId = Date.now().toString(36); // Track session for cancellation
+		let _lastNotifiedQuestion = ""; // Track last question to prevent repeat notifications
+
+		// UI status for auto-continue (captured from session_start ctx)
+		let _setAutoContinueStatus: ((text: string | undefined) => void) | null = null;
+		let _currentAutoContinueState = ""; // Track current state for deduplication
+
+		// Auto-continue state labels
+		const AC_STATE = {
+			IDLE: "",
+			WAITING: "WAIT",
+			CHECKING: "CHECK",
+			CONTINUING: "RUN",
+			ERROR: "WARN",
+		} as const;
+
+		// Status message for each state
+		const AC_STATE_MESSAGES = {
+			IDLE: "",
+			WAITING: "Awaiting your response",
+			CHECKING: "Evaluating continuation",
+			CONTINUING: "Continuing autonomously",
+			ERROR: "Auto-continue unavailable",
+		} as const;
+
+		// Update auto-continue status in TUI footer
+		function updateAutoContinueStatus(state: keyof typeof AC_STATE_MESSAGES, requestId?: string): void {
+			if (!_setAutoContinueStatus) return;
+
+			const message = AC_STATE_MESSAGES[state];
+			if (state === "IDLE") {
+				// Clear status
+				_setAutoContinueStatus(undefined);
+				_currentAutoContinueState = "";
+			} else {
+				const statusText = requestId
+					? `[${AC_STATE[state]}] ${message} | ${requestId}`
+					: `[${AC_STATE[state]}] ${message}`;
+
+				// Only update if state changed (prevent flicker)
+				if (statusText !== _currentAutoContinueState) {
+					_setAutoContinueStatus(statusText);
+					_currentAutoContinueState = statusText;
+				}
+			}
+		}
 
-		// Detect when agent asks for continuation
-		pi.on("message_end", async (event) => {
-			const msg = event.message as { role?: string; content?: string | Array<{ type?: string; text?: string }> } | undefined;
-			if (!msg || msg.role !== "assistant") return;
+		// Capture UI context from session_start
+		pi.on("session_start", (_event, ctx) => {
+			if (ctx.hasUI && ctx.ui) {
+				_setAutoContinueStatus = ctx.ui.setStatus.bind(ctx.ui, "harness-auto-continue");
+			}
+		});
+
+		// Clear status and cancel background work on session shutdown
+		pi.on("session_shutdown", () => {
+			// Cancel the auto-continue interval
+			if (autoContinueTimer) {
+				clearInterval(autoContinueTimer);
+				autoContinueTimer = null;
+			}
 
-			// Extract text from array content (TextContent blocks) or legacy string
+			// Clear UI binding and status
+			if (_setAutoContinueStatus) {
+				_setAutoContinueStatus(undefined);
+				_setAutoContinueStatus = null;
+			}
+			_currentAutoContinueState = "";
+
+			// Reset state
+			waitingForUserSince = null;
+			_lastNotifiedQuestion = "";
+			_inferenceInProgress = false;
+		});
+
+		// Extract assistant message text and detect actual confirmation requests
+		function extractAssistantText(msg: { role?: string; content?: string | Array<{ type?: string; text?: string }> } | undefined): string {
+			if (!msg) return "";
 			let content = "";
 			if (typeof msg.content === "string") {
 				content = msg.content;
 			} else if (Array.isArray(msg.content)) {
-				// Extract text from TextContent blocks only; ignore thinking/tool calls
 				content = msg.content
 					.filter((block): block is { type: "text"; text: string } =>
 						block.type === "text" && typeof block.text === "string"
@@ -177,15 +246,65 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 					.map(block => block.text)
 					.join("\n");
 			}
-			const hasContinuation = /continue|proceed|next phase|next step/i.test(content);
+			return content;
+		}
+
+		// Detect actual confirmation requests (questions, not progress statements)
+		// Must be a question ending with ? AND contain both a request word AND user intent
+		function detectConfirmationRequest(text: string): string | null {
+			// Find questions ending with ? and check each
+			// Use regex to match question sentences (ending with ?)
+			const questionPattern = /([^?]*\?)/g;
+			const matches = text.matchAll(questionPattern);
+
+			for (const match of matches) {
+				const sentence = match[1].trim();
+				const lower = sentence.toLowerCase();
+
+				// Must contain request words AND user intent words
+				const hasRequest = /\b(continue|proceed|next|wait|pause|stop|hold|resume|repeat|rephrase|clarify|explain)\b/i.test(lower);
+				const hasUserIntent = /\b(should|would|could|shall|may i|do you|would you|should i|could we|can i|can we|would like)\b/i.test(lower);
+
+				if (hasRequest && hasUserIntent) {
+					// Return the question for deduplication (with the ? preserved)
+					return sentence;
+				}
+			}
 
-			if (hasContinuation && !waitingForUserSince) {
+			return null;
+		}
+
+		// Detect when agent asks for continuation
+		pi.on("message_end", async (event) => {
+			const msg = event.message as { role?: string; content?: string | Array<{ type?: string; text?: string }> } | undefined;
+			if (!msg || msg.role !== "assistant") return;
+
+			const content = extractAssistantText(msg);
+			const detectedQuestion = detectConfirmationRequest(content);
+
+			// Only trigger if: we found a question AND it's different from last notification
+			if (detectedQuestion && detectedQuestion !== _lastNotifiedQuestion && !waitingForUserSince) {
+				_lastNotifiedQuestion = detectedQuestion;
 				waitingForUserSince = new Date();
-				console.log("[auto-continue] Agent waiting for continuation confirmation");
-				// Emit WaitingForUserInput to Telegram so user knows to respond
-				nc("WaitingForUserInput", {
-					requirement: "Agent awaiting your response",
-					error: "Continue / proceed confirmation needed",
+
+				// Create stable jobId from question content hash (callback-safe: digits only)
+				const questionHash = Math.abs(
+					detectedQuestion.split("").reduce((a, c) => {
+						a = ((a << 5) - a) + c.charCodeAt(0);
+						return a & a;
+					}, 0)
+				).toString(36);
+
+				// Update TUI status - use short ID for display
+				updateAutoContinueStatus("WAITING", questionHash);
+
+				// Emit WaitingForUserInput to Telegram with Yes/No buttons
+				_nc?.notifyWithApproval("WaitingForUserInput", {
+					jobId: `waiting-user-${questionHash}`,
+					requirement: detectedQuestion,
+				}, {
+					approveLabel: "Continue",
+					rejectLabel: "Not yet",
 				});
 			}
 		});
@@ -196,8 +315,9 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 			if (msg?.role === "user") {
 				lastUserMessage = new Date();
 				if (waitingForUserSince) {
-					console.log("[auto-continue] User responded, cancelling wait");
 					waitingForUserSince = null;
+					_lastNotifiedQuestion = "";
+					updateAutoContinueStatus("IDLE");
 				}
 			}
 		});
@@ -209,7 +329,6 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 		const checkAndAutoContinue = async () => {
 			// Prevent overlap if inference is already in progress
 			if (_inferenceInProgress) {
-				console.log("[auto-continue] Inference in progress, skipping this check");
 				return;
 			}
 
@@ -217,11 +336,19 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 
 			// Capture session ID at check time for post-inference validation
 			const checkSessionId = _sessionId;
+			// Use same hash algorithm as detection
+			const questionHash = _lastNotifiedQuestion
+				? Math.abs(_lastNotifiedQuestion.split("").reduce((a, c) => {
+						a = ((a << 5) - a) + c.charCodeAt(0);
+						return a & a;
+					}, 0)).toString(36)
+				: "";
 
 			const waitMinutes = (Date.now() - waitingForUserSince.getTime()) / 60000;
 			if (waitMinutes < MIN_WAIT_MINUTES) return;
 
-			console.log(`[auto-continue] Checking after ${waitMinutes.toFixed(0)} minutes wait...`);
+			// Update status to checking
+			updateAutoContinueStatus("CHECKING", questionHash);
 
 			try {
 				// Mark inference as in progress to prevent overlap
@@ -238,7 +365,6 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 
 				// Guard against empty task list
 				if (taskContext.total === 0) {
-					console.log("[auto-continue] No tasks to decide on — skipping");
 					_inferenceInProgress = false;
 					return;
 				}
@@ -254,49 +380,47 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 
 				// Check session validity and user response before applying
 				if (checkSessionId !== _sessionId) {
-					console.log("[auto-continue] Session changed during inference — cancelling");
 					_inferenceInProgress = false;
+					updateAutoContinueStatus("IDLE");
 					return;
 				}
 
 				if (!waitingForUserSince) {
-					console.log("[auto-continue] User responded during inference — cancelling");
 					_inferenceInProgress = false;
+					updateAutoContinueStatus("IDLE");
 					return;
 				}
 
-				console.log(
-					`[auto-continue] Decision: ${decision.action} (${(decision.probability * 100).toFixed(0)}% confidence)`,
-				);
+				if (decision.action === "proceed" || decision.action === "proceed_with_caution") {
+					// Update status to continuing
+					updateAutoContinueStatus("CONTINUING", questionHash);
 
-				if (decision.action === "proceed") {
 					// Send steer message to continue
 					pi.sendUserMessage(
-						"User is unavailable. Based on task progress and low risk, proceeding autonomously.\n" +
-							`Decision: ${decision.reasoning}\n` +
-							"Continue with remaining tasks.",
-						{ deliverAs: "steer" },
-					);
-					waitingForUserSince = null;
-				} else if (decision.action === "proceed_with_caution") {
-					pi.sendUserMessage(
-						"User is unavailable. Proceeding with caution - will log each significant step.\n" +
-						`Risk level: ${decision.riskLevel}\n` +
-						"Continue with remaining tasks, reporting progress.",
+						decision.action === "proceed"
+							? "User is unavailable. Based on task progress and low risk, proceeding autonomously.\n" +
+								  `Decision: ${decision.reasoning}\n` +
+								  "Continue with remaining tasks."
+							: "User is unavailable. Proceeding with caution - will log each significant step.\n" +
+								  `Risk level: ${decision.riskLevel}\n` +
+								  "Continue with remaining tasks, reporting progress.",
 						{ deliverAs: "steer" },
 					);
 					waitingForUserSince = null;
+					_lastNotifiedQuestion = "";
+
+					// Clear status after short delay
+					setTimeout(() => updateAutoContinueStatus("IDLE"), 2000);
 				}
-				// If action is "wait", do nothing and check again later
-			} catch (error) {
-				console.error("[auto-continue] Error:", error);
+				_inferenceInProgress = false;
+			} catch {
+				_inferenceInProgress = false;
+				updateAutoContinueStatus("ERROR");
 			}
 		};
 
 		// Start periodic check
 		autoContinueTimer = setInterval(checkAndAutoContinue, AUTO_CONTINUE_INTERVAL_MS);
-
-		console.error("[auto-continue] Jev Auto-Continue initialized");
 	} catch {
 		// auto-continue not available
 	}
@@ -403,6 +527,11 @@ async function initTelegram(): Promise<void> {
 			center,
 			notify: (event: string, ctx: Record<string, unknown>) =>
 				center.notify(event as never, ctx as never),
+			notifyWithApproval: (
+				event: string,
+				ctx: Record<string, unknown>,
+				options?: { approveLabel?: string; rejectLabel?: string },
+			) => center.notifyWithApproval(event as never, ctx as never, options as never),
 		};
 
 		// Wire to GLMQuotaCountdown so it sends Telegram alerts
@@ -430,13 +559,11 @@ async function initTelegram(): Promise<void> {
 // - Deduplication by message ID
 // - Bounded context injection with provenance
 
-/** Module-level Honcho memory instance */
-let _honchoMemory: import("./harness/honcho-memory.js").HonchoMemory | null = null;
-
 async function initHoncho(pi: ExtensionAPI): Promise<void> {
-	// Import HonchoMemory module to trigger auto-creation of MCP config
-	await import("./harness/honcho-memory.js");
+	// Import new Honcho integration module
+	const { initHoncho: honchoInit, registerHonchoLifecycle } = await import("./harness/honcho-init.js");
 
+	// Check for API key
 	const { readFileSync, existsSync } = await import("node:fs");
 	const home = process.env.HOME || process.env.USERPROFILE || "/home/frappe";
 	const keyPath = `${home}/.pi-harness-runtime/keys/honcho-api-key.txt`;
@@ -455,26 +582,19 @@ async function initHoncho(pi: ExtensionAPI): Promise<void> {
 	}
 
 	try {
-		// Register Honcho MCP server with pi-mcp-adapter at session start
-		pi.on("session_start", () => {
-			const request = {
-				version: 1 as const,
-				name: "honcho",
-				definition: {
-					url: "https://mcp.honcho.dev",
-					auth: "bearer" as const,
-					bearerToken: apiKey,
-				},
-			};
+		// Register lifecycle hooks using the new integration
+		// SAFETY: pi.on accepts these event names at runtime
+		registerHonchoLifecycle(pi as unknown as { on: (event: string, handler: (...args: unknown[]) => void | Promise<void>) => void });
 
-			// Emit registration request — pi-mcp-adapter will respond with
-			// pi-mcp-adapter:runtime-register-result event on completion
-			pi.events.emit("pi-mcp-adapter:runtime-register:v1", request);
-			console.log("[honcho] MCP registration emitted (awaiting adapter ack)");
-		});
+		// Initialize Honcho with the API key and pi.events
+		// SAFETY: pi.events implements the required on/emit interface
+		const result = await honchoInit({ apiKey, piEvents: pi.events });
 
-		// Report pending state — actual memory active requires explicit lifecycle
-		logStartup("[pi-harness] Honcho registered (awaiting adapter connection)");
+		if (result) {
+			logStartup("[pi-harness] Honcho integration ready");
+		} else {
+			logStartup("[pi-harness] Honcho: initialization returned null");
+		}
 	} catch (err) {
 		logStartup("[pi-harness] Honcho: init failed:", err instanceof Error ? err.message : String(err));
 	}
@@ -1023,7 +1143,15 @@ interface HarnessSession {
 let currentSession: HarnessSession | null = null;
 
 /** Singleton notification center — set by initTelegram(), used by all emit helpers */
-let _nc: { center: unknown; notify: (event: string, ctx: Record<string, unknown>) => Promise<unknown> } | null = null;
+let _nc: {
+	center: unknown;
+	notify: (event: string, ctx: Record<string, unknown>) => Promise<unknown>;
+	notifyWithApproval: (
+		event: string,
+		ctx: Record<string, unknown>,
+		options?: { approveLabel?: string; rejectLabel?: string },
+	) => Promise<unknown>;
+} | null = null;
 
 /**
  * Fire a Telegram notification event. Silently skips if Telegram is not configured.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-harness-runtime",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "[BETA] Codex-style /usage status + autonomous coding harness for pi. Not production ready — expect breaking changes.",
   "type": "module",
   "scripts": {

package/packages/a2a-adapter/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/a2a-adapter",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "description": "Google Agent-to-Agent (A2A) protocol adapter for pi-harness",
   "main": "dist/index.js",

package/packages/agent-policy/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/agent-policy",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Agent policy runtime for pi-harness - enforces mandatory rule delivery and mutation gating",
   "type": "module",
   "main": "./dist/index.js",

package/packages/architecture-generator/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/architecture-generator",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Generate ADR and Mermaid diagrams from project analysis",
   "type": "module",
   "main": "./dist/index.js",

package/packages/auth/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/auth",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "pi-harness-runtime — auth",
   "type": "module",
   "main": "./dist/index.js",

package/packages/autonomous-refactor/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/autonomous-refactor",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "description": "Autonomous refactoring engine",
   "main": "dist/index.js",

package/packages/autonomous-runtime/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/autonomous-runtime",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Autonomous Operations Runtime — task inbox, lease management, and worker lifecycle (RFC-0101)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/cache-strategy/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/cache-strategy",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "description": "Smart caching with TTL and invalidation strategies (RFC-0022)",
   "main": "dist/index.js",

package/packages/capability-registry/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/capability-registry",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Capability Registry for pi-harness-runtime — tracks model capabilities and enables capability-based routing",
   "type": "module",
   "main": "./dist/index.js",

package/packages/checkpoint/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/checkpoint",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Enhanced checkpoint system with incremental snapshots, diff-based storage, and recovery strategies",
   "type": "module",
   "main": "./dist/index.js",

package/packages/cli-plugin-sdk/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/cli-plugin-sdk",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "description": "CLI Plugin SDK for pi-harness — install, list, remove, update, invoke plugins",
   "main": "dist/index.js",

package/packages/clipboard/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/clipboard",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/clipboard-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/clipboard-plugin",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/code-generation/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/code-generation",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Code generation pipeline for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/code-review/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/code-review",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Automated code review engine for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/codex-adapter/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/codex-adapter",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "description": "OpenAI Codex provider adapter for pi-harness",
   "main": "dist/index.js",

package/packages/codex-watcher/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/codex-watcher",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Watches Codex CLI sessions and emits visible messages for PING-PONG monitoring",
   "type": "module",
   "main": "./dist/src/index.js",

package/packages/config-capture/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moocoding/config-capture",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Auto-capture API-fetched configuration values to AGENTS.md",
   "type": "module",
   "main": "src/index.ts",

package/packages/context-compiler/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/context-compiler",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Bounded context selection engine for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/context-discovery/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/context-discovery",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/context-manager/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/context-manager",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Context Window Manager — prevents model context overflow via compaction and resume prompts (RFC-0010)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/cookie-sanitizer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/cookie-sanitizer",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "pi-harness-runtime — cookie sanitizer (provider-quota cookie file normaliser)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/cost-optimizer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/cost-optimizer",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Cost Optimizer for pi-harness-runtime — budget tracking, forecasting, and model selection optimization",
   "type": "module",
   "main": "./dist/index.js",

package/packages/dependency-analyzer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/dependency-analyzer",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/django-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/django-plugin",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Django framework analysis — models, management commands, settings",
   "type": "module",
   "main": "./dist/index.js",

package/packages/doc-generator/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/doc-generator",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Auto-Doc Generator — generate documentation from source code (RFC-0014)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/e2e-tester/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/e2e-tester",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "WebMCP-powered E2E testing for pi-harness - natural language test scenarios",
   "type": "module",
   "main": "./dist/index.js",

package/packages/evaluation-engine/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/evaluation-engine",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "description": "Evaluation engine for pi-harness runtime (RFC-0057)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/evaluation-runner/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/evaluation-runner",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "description": "Framework-agnostic evaluation framework for AI outputs (RFC-0020)",
   "main": "dist/index.js",

package/packages/event-bus/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/event-bus",
-  "version": "1.1.184",
+  "version": "1.1.185",
   "type": "module",
   "main": "src/index.ts",
   "exports": {