pi-harness-runtime 1.1.392 → 1.1.394

package/packages/relay/src/client.ts

@@ -42,6 +42,19 @@ const CONNECT_TIMEOUT_MS = 10000;
 // Client Callbacks
 // ============================================================================
 
+/**
+ * Where the client writes its diagnostics.
+ *
+ * A TUI owns the terminal: a stray console write lands in the prompt input
+ * field. The harness therefore injects a logger that writes to the log file
+ * (the console stays the default for CLI scripts).
+ */
+export interface RelayLogger {
+  log?: (message: string) => void;
+  warn?: (message: string) => void;
+  error?: (message: string) => void;
+}
+
 export interface RelayClientCallbacks {
   /** Called when connected and authenticated */
   onConnected?: (sessionId: string) => void;
@@ -69,6 +82,7 @@ export class RelayClient {
   private ws: WebSocket | null = null;
   private config: RelayConfig;
   private callbacks: RelayClientCallbacks;
+  private logger: RelayLogger;
   
   // State machine
   private state: ConnectionState = { state: 'disconnected' };
@@ -97,9 +111,57 @@ export class RelayClient {
   private snapshotReceived = false;
   private synchronizedReceived = false;
 
-  constructor(config: RelayConfig, callbacks: RelayClientCallbacks = {}) {
+  constructor(config: RelayConfig, callbacks: RelayClientCallbacks = {}, logger: RelayLogger = {}) {
     this.config = config;
     this.callbacks = callbacks;
+    this.logger = logger;
+  }
+
+  // ========================================================================
+  // Diagnostics
+  // ========================================================================
+
+  /**
+   * Write a diagnostic.
+   *
+   * A TUI owns the terminal: the harness injects a logger so that nothing
+   * reaches the console. A partial logger must not reopen that hole - a
+   * missing level falls back to a level the logger does provide.
+   */
+  private log(message: string): void {
+    this.emit("log", message);
+  }
+
+  private logWarn(message: string): void {
+    this.emit("warn", message);
+  }
+
+  private logError(message: string): void {
+    this.emit("error", message);
+  }
+
+  private emit(level: "log" | "warn" | "error", message: string): void {
+    const logger = this.logger;
+    // A logger was injected: route the level to one the logger provides,
+    // never back to the console. The order keeps the severity as close as
+    // possible (error -> warn -> log).
+    const candidates: Array<"log" | "warn" | "error"> =
+      level === "error" ? ["error", "warn", "log"] : level === "warn" ? ["warn", "error", "log"] : ["log", "warn", "error"];
+
+    for (const candidate of candidates) {
+      const sink = candidate === "log" ? logger.log : candidate === "warn" ? logger.warn : logger.error;
+      if (!sink) continue;
+      try {
+        sink(message);
+      } catch {
+        // A broken logger must never break the relay client.
+      }
+      return;
+    }
+
+    if (level === "log") console.log(message);
+    else if (level === "warn") console.warn(message);
+    else console.error(message);
   }
 
   // ========================================================================
@@ -167,7 +229,7 @@ export class RelayClient {
 
     const url = this.config.serverUrl;
     const timeoutMs = this.config.connectTimeout ?? CONNECT_TIMEOUT_MS;
-    console.log(`[RelayClient] Connecting to ${url}...`);
+    this.log(`[RelayClient] Connecting to ${url}...`);
 
     return new Promise<void>((resolve, reject) => {
       const attempt = ++this.connectAttempt;
@@ -176,7 +238,7 @@ export class RelayClient {
       // Deadline covers connection AND authentication (one timer for the whole handshake)
       this.connectTimeoutTimer = setTimeout(() => {
         if (!this.pendingConnect) return;
-        console.error(`[RelayClient] Connection timeout after ${timeoutMs}ms`);
+        this.logError(`[RelayClient] Connection timeout after ${timeoutMs}ms`);
         this.settleConnect(new Error(`Connection timeout after ${timeoutMs}ms`));
         this.closeSocket();
         this.setState({ state: 'disconnected' });
@@ -191,7 +253,7 @@ export class RelayClient {
         // pending: settle it, disarm the deadline and retry like any other
         // failed connect.
         const error = err instanceof Error ? err : new Error(String(err));
-        console.error("[RelayClient] WebSocket construction failed:", error.message);
+        this.logError(`[RelayClient] WebSocket construction failed: ${error.message}`);
         this.settleConnect(error);
         this.setState({ state: 'disconnected' });
         this.scheduleReconnectAfterFailure();
@@ -201,7 +263,7 @@ export class RelayClient {
 
       ws.on("open", () => {
         if (this.ws !== ws) return;
-        console.log("[RelayClient] WebSocket opened");
+        this.log("[RelayClient] WebSocket opened");
         this.setState({ state: 'authenticating' });
 
         // Send authentication
@@ -223,7 +285,7 @@ export class RelayClient {
         // would otherwise corrupt the live connection (state, heartbeat) and
         // schedule a spurious reconnect.
         if (this.ws !== ws) return;
-        console.log("[RelayClient] WebSocket closed");
+        this.log("[RelayClient] WebSocket closed");
         this.ws = null;
         const wasPending = this.pendingConnect !== null;
         this.handleDisconnect();
@@ -235,7 +297,7 @@ export class RelayClient {
 
       ws.on("error", (err) => {
         if (this.ws !== ws) return;
-        console.error("[RelayClient] WebSocket error:", err.message);
+        this.logError(`[RelayClient] WebSocket error: ${err.message}`);
         this.callbacks.onError?.(err);
         const wasPending = this.pendingConnect !== null;
         this.ws = null;
@@ -275,7 +337,7 @@ export class RelayClient {
    * Disconnect from relay server
    */
   disconnect(): void {
-    console.log("[RelayClient] Disconnecting...");
+    this.log("[RelayClient] Disconnecting...");
     // Deliberate stop: do not schedule a reconnect afterwards
     this.shuttingDown = true;
     // Never leave a caller awaiting connect() hanging
@@ -301,7 +363,7 @@ export class RelayClient {
     if (this.ws?.readyState === WebSocket.OPEN) {
       this.ws.send(JSON.stringify(message));
     } else {
-      console.warn("[RelayClient] Cannot send: WebSocket not open");
+      this.logWarn("[RelayClient] Cannot send: WebSocket not open");
     }
   }
 
@@ -344,18 +406,20 @@ export class RelayClient {
 
         default: {
           const unknownMsg = msg as { type: string };
-          console.warn(`[RelayClient] Unknown message type: ${unknownMsg.type}`);
+          this.logWarn(`[RelayClient] Unknown message type: ${unknownMsg.type}`);
         }
       }
     } catch (err) {
-      console.error("[RelayClient] Failed to parse message:", err);
+      const detail = err instanceof Error ? (err.stack ?? err.message) : String(err);
+    // One line per log record: the stack must not break the format.
+    this.logError(`[RelayClient] Failed to parse message: ${detail.replace(/\s*\n\s*/g, " | ")}`);
       this.callbacks.onError?.(err as Error);
     }
   }
 
   private handleConnected(msg: ConnectedMessage): void {
     this.clientId = msg.payload.clientId;
-    console.log(`[RelayClient] Connected, clientId: ${this.clientId}`);
+    this.log(`[RelayClient] Connected, clientId: ${this.clientId}`);
   }
 
   private handleAuth(msg: AuthResponseMessage): void {
@@ -372,13 +436,13 @@ export class RelayClient {
       
       // Notify callback
       this.callbacks.onConnected?.(this.sessionId || '');
-      console.log(`[RelayClient] Authenticated, sessionId: ${this.sessionId}`);
+      this.log(`[RelayClient] Authenticated, sessionId: ${this.sessionId}`);
 
       // Readiness milestone reached - settle the connect() promise
       this.settleConnect();
     } else {
       const message = msg.payload.error || 'Authentication failed';
-      console.error(`[RelayClient] Auth failed: ${message}`);
+      this.logError(`[RelayClient] Auth failed: ${message}`);
       const error = new Error(message);
       this.callbacks.onError?.(error);
       // Reject the caller before tearing the socket down
@@ -392,7 +456,7 @@ export class RelayClient {
   }
 
   private handleSnapshot(msg: SnapshotMessage): void {
-    console.log(`[RelayClient] Snapshot received: ${msg.payload.recentCallbacks.length} recent callbacks`);
+    this.log(`[RelayClient] Snapshot received: ${msg.payload.recentCallbacks.length} recent callbacks`);
     this.snapshotReceived = true;
     
     // Notify callback
@@ -400,7 +464,7 @@ export class RelayClient {
   }
 
   private handleSynchronized(_msg: SynchronizedMessage): void {
-    console.log("[RelayClient] Synchronized - live events starting");
+    this.log("[RelayClient] Synchronized - live events starting");
     this.synchronizedReceived = true;
     
     // Notify callback
@@ -410,7 +474,7 @@ export class RelayClient {
   private handleCallback(msg: CallbackMessage): void {
     // Only process callbacks after synchronized
     if (!this.synchronizedReceived) {
-      console.warn("[RelayClient] Ignoring callback before synchronized");
+      this.logWarn("[RelayClient] Ignoring callback before synchronized");
       return;
     }
     
@@ -418,7 +482,7 @@ export class RelayClient {
   }
 
   private handleStatus(msg: StatusMessage): void {
-    console.log(`[RelayClient] Status [${msg.payload.kind}]: ${msg.payload.code} - ${msg.payload.message}`);
+    this.log(`[RelayClient] Status [${msg.payload.kind}]: ${msg.payload.code} - ${msg.payload.message}`);
     this.callbacks.onStatus?.(msg.payload);
   }
 
@@ -434,7 +498,7 @@ export class RelayClient {
 
     // A deliberate stop must not trigger a reconnect
     if (this.shuttingDown) {
-      console.log("[RelayClient] Disconnected (intentional stop, no reconnect)");
+      this.log("[RelayClient] Disconnected (intentional stop, no reconnect)");
       return;
     }
 
@@ -464,12 +528,12 @@ export class RelayClient {
     this.reconnectAttempt++;
     this.setState({ state: 'reconnecting', attempt: this.reconnectAttempt });
     
-    console.log(`[RelayClient] Scheduling reconnect attempt ${this.reconnectAttempt} in ${this.reconnectDelay}ms`);
+    this.log(`[RelayClient] Scheduling reconnect attempt ${this.reconnectAttempt} in ${this.reconnectDelay}ms`);
     
     this.reconnectTimer = setTimeout(() => {
       this.reconnectTimer = null;
       this.connect().catch((err) => {
-        console.error(`[RelayClient] Reconnect failed: ${err.message}`);
+        this.logError(`[RelayClient] Reconnect failed: ${err.message}`);
       });
     }, this.reconnectDelay);
     
@@ -545,13 +609,13 @@ export class RelayClient {
     ws.removeAllListeners("error");
     // An 'error' without a listener throws inside the ws implementation.
     ws.on("error", (err: Error) => {
-      console.warn("[RelayClient] Socket error while closing:", err.message);
+      this.logWarn(`[RelayClient] Socket error while closing: ${err.message}`);
     });
     ws.on("close", () => {});
     try {
       ws.close();
     } catch (err) {
-      console.warn("[RelayClient] Error while closing socket:", String(err));
+      this.logWarn(`[RelayClient] Error while closing socket: ${String(err)}`);
     }
   }
 

package/packages/relay/src/index.ts

@@ -15,7 +15,7 @@ export type { RelayServerOptions } from './server.js';
 
 // Client
 export { RelayClient } from './client.js';
-export type { RelayClientCallbacks } from './client.js';
+export type { RelayClientCallbacks, RelayLogger } from './client.js';
 
 // Endpoint resolution (relay URL + auth token discovery)
 export {

package/packages/relay/test/client.test.ts

@@ -135,6 +135,140 @@ describe("RelayClient.connect", () => {
 		await fixture.close();
 	});
 
+	it("writes its diagnostics to the injected logger, never to the console", async () => {
+		const fixture = await startFixture("ok");
+		const logged: string[] = [];
+		const consoleWrites: string[] = [];
+		const originalLog = console.log;
+		const originalWarn = console.warn;
+		const originalError = console.error;
+		console.log = (...args: unknown[]) => {
+			consoleWrites.push(args.map(String).join(" "));
+		};
+		console.warn = console.log;
+		console.error = console.log;
+
+		try {
+			const client = new RelayClient(
+				{ serverUrl: fixture.url, token: TEST_TOKEN, connectTimeout: 2000 },
+				{},
+				{
+					log: (message) => logged.push(message),
+					warn: (message) => logged.push(message),
+					error: (message) => logged.push(message),
+				},
+			);
+
+			await client.connect();
+			client.disconnect();
+			await fixture.close();
+		} finally {
+			console.log = originalLog;
+			console.warn = originalWarn;
+			console.error = originalError;
+		}
+
+		// The TUI owns the terminal: nothing may reach the console.
+		expect(consoleWrites.length).toBe(0);
+		expect(logged.some((message) => message.includes("[RelayClient]"))).toBe(true);
+	});
+
+	it("routes a partial logger to the levels it has, never to the console", async () => {
+		const fixture = await startFixture("ok");
+		const logged: string[] = [];
+		const consoleWrites: string[] = [];
+		const originalLog = console.log;
+		const originalWarn = console.warn;
+		const originalError = console.error;
+		console.log = (...args: unknown[]) => {
+			consoleWrites.push(args.map(String).join(" "));
+		};
+		console.warn = console.log;
+		console.error = console.log;
+
+		try {
+			// Only "log" is provided: a warning must NOT reach the console.
+			const client = new RelayClient(
+				{ serverUrl: fixture.url, token: TEST_TOKEN, connectTimeout: 2000 },
+				{},
+				{ log: (message) => logged.push(message) },
+			);
+
+			await client.connect();
+			client.disconnect();
+			await fixture.close();
+		} finally {
+			console.log = originalLog;
+			console.warn = originalWarn;
+			console.error = originalError;
+		}
+
+		expect(consoleWrites.length).toBe(0);
+		expect(logged.length).toBeGreaterThan(0);
+	});
+
+	it("never reaches the console for any partial logger shape", async () => {
+		const shapes: Array<{ name: string; logger: { log?: (m: string) => void; warn?: (m: string) => void; error?: (m: string) => void } }> = [
+			{ name: "only log", logger: { log: () => {} } },
+			{ name: "only warn", logger: { warn: () => {} } },
+			{ name: "only error", logger: { error: () => {} } },
+			{ name: "log + error", logger: { log: () => {}, error: () => {} } },
+		];
+
+		for (const shape of shapes) {
+			const fixture = await startFixture("ok");
+			const consoleWrites: string[] = [];
+			const originalLog = console.log;
+			const originalWarn = console.warn;
+			const originalError = console.error;
+			console.log = (...args: unknown[]) => {
+				consoleWrites.push(args.map(String).join(" "));
+			};
+			console.warn = console.log;
+			console.error = console.log;
+
+			try {
+				const client = new RelayClient(
+					{ serverUrl: fixture.url, token: TEST_TOKEN, connectTimeout: 2000 },
+					{},
+					shape.logger,
+				);
+
+				await client.connect();
+				client.disconnect();
+				await fixture.close();
+			} finally {
+				console.log = originalLog;
+				console.warn = originalWarn;
+				console.error = originalError;
+			}
+
+			expect(consoleWrites.length).toBe(0);
+		}
+	});
+
+	it("survives a logger that throws", async () => {
+		const fixture = await startFixture("ok");
+		let calls = 0;
+		const broken = (): void => {
+			calls++;
+			throw new Error("logger is broken");
+		};
+		const client = new RelayClient(
+			{ serverUrl: fixture.url, token: TEST_TOKEN, connectTimeout: 2000 },
+			{},
+			{ log: broken, warn: broken, error: broken },
+		);
+
+		await client.connect();
+		expect(client.getState().state).toBe("connected");
+		// The logger really ran (the test would be vacuous otherwise).
+		expect(calls).toBeGreaterThan(0);
+
+		client.disconnect();
+		await fixture.close();
+	});
+
 	it("rejects when the server never answers authentication", async () => {
 		const fixture = await startFixture("silent");
 		const client = new RelayClient({

package/packages/release-manager/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/release-manager",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "type": "module",
   "description": "Release management for pi-harness",
   "main": "dist/index.js",

package/packages/requirement-compiler/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/requirement-compiler",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Compiles raw requirements into validated CompiledRequirement with ambiguity detection and risk tagging",
   "type": "module",
   "main": "./dist/index.js",

package/packages/runtime/package.json

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

package/packages/scheduler/package.json

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

package/packages/scheduler-adapter/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/scheduler-adapter",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Scheduler Adapter — cron, systemd, launchd, internal adapters for autonomous runtime (RFC-0101 §6)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/session/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/session",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Session management system for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/session-api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/session-api",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/session-export/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/session-export",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/shared-context/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/shared-context",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "pi-harness-runtime — shared-context",
   "type": "module",
   "main": "./dist/index.js",

package/packages/skill-mcp-client/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/skill-mcp-client",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "MCP client for Skills SaaS backend",
   "type": "module",
   "main": "./dist/index.js",

package/packages/skill-registry/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/skill-registry",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Skill Registry for pi-harness-runtime — dynamic skill discovery and invocation",
   "type": "module",
   "main": "./dist/index.js",

package/packages/skills/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/skills",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "File-based skill system with pi.dev compatibility and Hermes self-improvement",
   "type": "module",
   "main": "src/index.ts",

package/packages/sprint-planner/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/sprint-planner",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "type": "module",
   "description": "Sprint planning engine for pi-harness",
   "main": "dist/index.js",

package/packages/subscription-engine/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/subscription-engine",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "type": "module",
   "main": "src/index.ts",
   "exports": {

package/packages/task-compiler/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/task-compiler",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Task Compiler for pi-harness-runtime — produces deterministic task DAG from CompiledRequirement",
   "type": "module",
   "main": "./dist/index.js",

package/packages/telegram/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/telegram",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "pi-harness-runtime — Telegram inbound/outbound",
   "type": "module",
   "main": "./src/index.js",

package/packages/tencentdb-memory/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moocoding/tencentdb-memory",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "TencentDB Agent Memory client - sync skills to centralized knowledge base",
   "type": "module",
   "main": "src/index.ts",

package/packages/tencentdb-sync/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/tencentdb-sync",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Sync SKILL.md skills to TencentDB-Agent-Memory server (RFC-0105/0106)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/test-data-generator/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi/test-data-generator",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Test data generation for pi-harness-runtime",
   "type": "module",
   "main": "./dist/index.js",

package/packages/test-generator/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/test-generator",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Auto-Test Generator — generate unit and E2E tests from source code (RFC-0013)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/todo-bd-sync/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moocoding/todo-bd-sync",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Two-way sync between rpiv-todo overlay and bd issue tracker",
   "type": "module",
   "main": "src/index.ts",

package/packages/token-estimation/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/token-estimation",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "pi-harness-runtime — token-estimation",
   "type": "module",
   "main": "./dist/index.js",

package/packages/token-optimizer/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/token-optimizer",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Token Budget Optimizer — maximize prompt value within token limits (RFC-0011)",
   "type": "module",
   "main": "./dist/index.js",

package/packages/trajectory/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/trajectory",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Trajectory capture for the write-review loop — persists cycle records for learning",
   "type": "module",
   "main": "./dist/src/index.js",

package/packages/tui/package.json

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

package/packages/types/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-harness/types",
-  "version": "1.1.392",
+  "version": "1.1.394",
   "description": "Shared TypeScript types for pi-harness-runtime packages",
   "type": "module",
   "main": "./src/index.js",