pi-harness-runtime 1.1.346 → 1.1.349

package/index.ts

@@ -134,6 +134,8 @@ const sharedState = {
 	todoContinuationController: null as unknown,
 	// eslint-disable-next-line @typescript-eslint/no-explicit-any
 	updateAutoContinueStatus: null as any,
+	// eslint-disable-next-line @typescript-eslint/no-explicit-any
+	pqController: null as any,
 };
 
 async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
@@ -168,6 +170,9 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 			readFileSync(`${keysDir}/jev-api-key.txt`, "utf8").trim();
 
 		const mod = await import("./packages/jev-judge/src/auto-continue.js");
+		const { createPendingQuestionController } = await import("./packages/notification/dist/index.js");
+		const pqController = createPendingQuestionController();
+		sharedState.pqController = pqController;
 
 		// Track waiting state
 		let waitingForUserSince: Date | null = null;
@@ -176,6 +181,7 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 		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
+		let _completedNotifiedFor: string | null = null; // Track completion notification deduplication
 
 		// UI status for auto-continue (captured from session_start ctx)
 		let _setAutoContinueStatus: ((text: string | undefined) => void) | null = null;
@@ -271,50 +277,20 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 		}
 
 		// Detect actual confirmation requests (questions, not progress statements)
-		// Must be a question ending with ? AND contain both a request word AND user intent
+		// Uses the pending question controller for detection with expanded patterns
 		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;
-				}
+			const result = pqController.detectQuestion(text);
+			if (result.detected && result.questionText) {
+				return result.questionText;
 			}
-
 			return null;
 		}
 
 		// Detect routine continuation questions ("Should I continue?")
 		// These should NOT block immediate continuation - let Jev/todo controller decide
 		function isRoutineContinuation(question: string): boolean {
-			const lower = question.toLowerCase();
-			// Match ALL routine continuation patterns
-			const routinePatterns = [
-				/\b(should i|should we|should the|can i continue|can we continue)\b/,
-				/\b(would you like me to continue|shall i proceed)\b/,
-				/\b(can i proceed|may i continue)\b/,
-				/\b(continue|proceed)\s*(with|to|on|with the next|to the next)\b/,
-				/\bshall\s+(i|we)\s+(proceed|continue)\b/,
-				/\bshould\s+(i|we)\s+(continue|proceed|move on)\b/,
-				/\b(continue|proceed)\s+\?/,
-			];
-			for (const pattern of routinePatterns) {
-				if (pattern.test(lower)) {
-					return true;
-				}
-			}
-			return false;
+			const result = pqController.detectQuestion(question);
+			return result.isRoutine ?? false;
 		}
 
 		// Detect when agent asks for continuation
@@ -488,6 +464,9 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 					waitingForUserSince = null;
 					_lastNotifiedQuestion = "";
 					updateAutoContinueStatus("IDLE");
+					// Cancel pending question on user response
+					// (User answered via Pi instead of button)
+					sharedState.pqController?.cancel("User responded via Pi");
 				}
 			}
 		});
@@ -550,6 +529,8 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 
 				// Check session validity and user response before applying
 				if (checkSessionId !== _sessionId) {
+					// Session changed - cancel pending question
+					sharedState.pqController?.cancel("Session switched");
 					_inferenceInProgress = false;
 					updateAutoContinueStatus("IDLE");
 					return;
@@ -792,26 +773,50 @@ async function initAutoContinue(pi: ExtensionAPI): Promise<void> {
 					if (result.action === "blocked") {
 						updateAutoContinueStatus("ERROR", "blocked");
 						logToFile("[auto-continue]", "Tasks blocked:", result.question ?? "unknown");
-				} else if (result.action === "stop") {
-					logToFile("[auto-continue]", "decide returned stop:", result.reason ?? "unknown");
-					// Get full todo list for notification
-					const snapshot = await todoContinuationController.getSnapshot();
-					const todoList = snapshot.items
-						.map((item) => {
-							const status = item.status === "completed" ? "[DONE]" : "[PENDING]";
-						const subject = item.subject.length > 50 ? item.subject.slice(0, 47) + "..." : item.subject;
-							return `${status} ${subject}`;
-						})
-						.join("\n");
-						const message = todoList || "All tasks completed";
-						// Notify user that all work is complete with todo list
-						_nc?.notify("ReadyForClient", {
-							jobId: `done-${Date.now()}`,
-							requirement: message,
-						}).catch((e) => logToFile("[Notification]", "ReadyForClient failed:", String(e)));
+					} else if (result.action === "stop") {
+						const reason = result.reason ?? "unknown";
+						logToFile("[auto-continue]", "decide returned stop:", reason);
+
+						// Only send completion notification for "No pending tasks"
+						// Other stop reasons (paused, unavailable, max attempts) are not completion
+						const isNoPendingTasks = reason === "No pending tasks";
+
+						if (isNoPendingTasks) {
+							// Check if there's a pending question - suppress completion if so
+							if (pqController.hasPendingQuestion()) {
+								logToFile("[auto-continue]", "Suppressing completion - pending question exists");
+								updateAutoContinueStatus("WAITING", "pending");
+								return;
+							}
+
+							// Deduplication: only notify once per work generation
+							// Use a static flag for this session's settled cycle
+							const completionKey = `done-${reason}`;
+							if (_completedNotifiedFor === completionKey) {
+								logToFile("[auto-continue]", "Completion already notified for:", reason);
+								return;
+							}
+							_completedNotifiedFor = completionKey;
+
+							// Get full todo list for notification
+							const snapshot = await todoContinuationController.getSnapshot();
+							const todoList = snapshot.items
+								.map((item) => {
+									const status = item.status === "completed" ? "[DONE]" : "[PENDING]";
+									const subject = item.subject.length > 50 ? item.subject.slice(0, 47) + "..." : item.subject;
+									return `${status} ${subject}`;
+								})
+								.join("\n");
+							const message = todoList || "All tasks completed";
+							// Notify user that all work is complete with todo list
+							_nc?.notify("ReadyForClient", {
+								jobId: `done-${reason.replace(/\s+/g, "-")}`,
+								requirement: message,
+							}).catch((e) => logToFile("[Notification]", "ReadyForClient failed:", String(e)));
 						}
-				}
-			} catch (err) {
+					}
+					}
+				} catch (err) {
 				logToFile("[auto-continue]", "agent_settled handler error:", String(err));
 				todoContinuationController?.clearQueue();
 			}
@@ -1202,12 +1207,16 @@ async function initTelegram(pi: ExtensionAPI): Promise<void> {
 					logToFile("[TelegramCallback] Approved - user clicked Continue, resuming work");
 					sharedState.waitingForUserSince = null;
 					sharedState.updateAutoContinueStatus?.("IDLE");
+					// Resolve pending question with yes
+					sharedState.pqController?.resolve({ resolution: "yes", by: "user" });
 					// Also resume continuation controller
 					(sharedState.todoContinuationController as any)?.resume?.();
 					break;
 				case CallbackActions.REJECT:
 					// User clicked Not yet - just dismiss, don't resume
 					logToFile("[TelegramCallback] Rejected - user clicked Not yet, pausing");
+					// Resolve pending question with no
+					sharedState.pqController?.resolve({ resolution: "no", by: "user" });
 					// Explicitly pause the continuation
 					(sharedState.todoContinuationController as any)?.explicitPause?.();
 					break;
@@ -1231,30 +1240,29 @@ async function initTelegram(pi: ExtensionAPI): Promise<void> {
 
 		// Register message handler to forward Telegram messages to Pi
 		center.setMessageHandler(async (text: string, _chatId: string) => {
-			// Parse /notify command for channel management
-			// Usage: /notify on telegram, /notify off line, /notify on all, /notify (show status)
-			const notifyRegex = /^\/notify(?:\s+(on|off)(?:\s+(telegram|line|all))?/i;
-			const notifyMatch = text.match(notifyRegex);
-			if (notifyMatch) {
-				const [, action, channel] = notifyMatch;
-				const isEnabled = action === "on";
-				const channels = channel === "all" ? ["telegram", "line"] : [channel || "telegram"];
-
-				for (const ch of channels) {
-					(center as any).setChannelEnabled?.(ch, isEnabled);
-					logToFile("[NotifyCommand]", `Set ${ch} to ${isEnabled ? "enabled" : "disabled"}`);
+			// Parse /notify-settings command for channel management
+			// Usage: /notify-settings - show current status
+			//        /notify-settings telegram on - enable Telegram
+			//        /notify-settings line off - disable LINE
+			//        /notify-settings all on - enable all channels
+			const settingsMatch = text.match(/^\/notify-settings(?:\s+(\w+)\s+(on|off))?/i);
+			if (settingsMatch || text === "/notify-settings") {
+				const [, channel, action] = settingsMatch || [];
+
+				if (action && channel) {
+					const isEnabled = action === "on";
+					const targetChannels = channel === "all" ? ["telegram", "line"] : [channel];
+					for (const ch of targetChannels) {
+						(center as any).setChannelEnabled?.(ch, isEnabled);
+						logToFile("[NotifySettings]", `Set ${ch} to ${isEnabled ? "enabled" : "disabled"}`);
+					}
 				}
-				const telegramStatus = (center as any).isChannelEnabled?.("telegram") ? "ON" : "OFF";
-				const lineStatus = (center as any).isChannelEnabled?.("line") ? "ON" : "OFF";
-				logToFile("[NotifyCommand]", `Current: Telegram=${telegramStatus}, LINE=${lineStatus}`);
-				return;
-			}
 
-			// Show help for /notify without args
-			if (text === "/notify") {
+				// Show current status
 				const telegramStatus = (center as any).isChannelEnabled?.("telegram") ? "ON" : "OFF";
 				const lineStatus = (center as any).isChannelEnabled?.("line") ? "ON" : "OFF";
-				logToFile("[NotifyCommand]", `Channel Status: Telegram=${telegramStatus}, LINE=${lineStatus}`);
+				logToFile("[NotifySettings]", `Current: Telegram=${telegramStatus}, LINE=${lineStatus}`);
+				logToFile("[NotifySettings]", `Commands: /notify-settings telegram on|off, /notify-settings line on|off, /notify-settings all on|off`);
 				return;
 			}
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-harness-runtime",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/clipboard-plugin/package.json

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

package/packages/code-generation/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/code-generation",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "description": "Auto-capture API-fetched configuration values to AGENTS.md",
   "type": "module",
   "main": "src/index.ts",

package/packages/connection-runtime/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/connection-runtime",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "type": "module",
   "description": "Connection management with jittered exponential backoff for pi-harness",
   "main": "./dist/index.js",

package/packages/context-compiler/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/context-compiler",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "description": "Context Window Manager — prevents model context overflow via compaction and resume prompts (RFC-0010)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/contracts/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/contracts",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "type": "module",
   "description": "RPC schema definitions for pi-harness runtime",
   "main": "./dist/index.js",

package/packages/cookie-sanitizer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/cookie-sanitizer",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/django-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/django-plugin",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "description": "Auto-Doc Generator — generate documentation from source code (RFC-0014)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/durable-review/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moocoding/durable-review",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "description": "SQLite-backed durable reviewer using @earendil-works/pi-durable",
   "type": "module",
   "main": "src/index.ts",

package/packages/e2e-tester/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/e2e-tester",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "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.346",
+  "version": "1.1.349",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/event-sink/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/event-sink",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "type": "module",
   "description": "Atomic event sourcing with SQLite transactions for pi-harness",
   "main": "./dist/index.js",

package/packages/event-store/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/event-store",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/experience-replay/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/experience-replay",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "description": "Experience replay for debugging and reproducibility (RFC-0059)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/feedback-collector/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/feedback-collector",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "type": "module",
   "description": "Collect and aggregate human/automated feedback (RFC-0021)",
   "main": "dist/index.js",

package/packages/file-copy-helper/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moocoding/file-copy-helper",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "description": "Injects cp/sudo cp rule when user asks to mimic/copy files",
   "type": "module",
   "main": "dist/index.js",

package/packages/framework-detector/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/framework-detector",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "description": "Framework detection system for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

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

@@ -1,6 +1,6 @@
 {
   "name": "@pi/framework-plugin-sdk",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "description": "Framework plugin SDK for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/frappe-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/frappe-plugin",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "description": "Frappe/ERPNext framework analysis — DocTypes, hooks, bench sites, site config",
   "type": "module",
   "main": "./dist/index.js",

package/packages/generic-web-plugin/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/generic-web-plugin",
-  "version": "1.1.346",
+  "version": "1.1.349",
   "description": "Generic web framework analysis — REST/GraphQL endpoints, OpenAPI, auth",
   "type": "module",
   "main": "./dist/index.js",