pi-soly 2.3.1 → 2.3.3

package/commands/_helpers.ts

@@ -40,12 +40,9 @@ export interface CommandsDeps {
 	getConfig: () => SolyConfig;
 	reloadConfig: () => void;
 	getIntentDocs: () => IntentDoc[];
-	/** Record a non-error soly event. Rendered as a sub-line under the
-	 *  Working indicator (└─ prefixed, dim/yellow by level). Use this for
-	 *  informational events (e.g. "reloaded 47 rules") — they shouldn't be
-	 *  popups. Errors must still go through `ui.notify(text, "error")` for
-	 *  the popup surface. */
-	recordEvent: (text: string, level?: "info" | "warning") => void;
+	/** Record a soly event (any level). Everything goes through the
+	 *  Working sub-line — no popups at all. */
+	recordEvent: (text: string, level?: "info" | "warning" | "error") => void;
 }
 
 /** Open a focused list modal (overlay) for the given grouped items. */

package/commands/docs.ts

@@ -67,7 +67,7 @@ export function registerDocsCommand(pi: ExtensionAPI, deps: DocsDeps): void {
 				return;
 			}
 
-			ui.notify("Usage: /docs <list|stats>", "error");
+			recordEvent("Usage: /docs <list|stats>", "error");
 			void solyDirFor; // kept available for future subcommands
 		},
 	});

package/commands/rules.ts

@@ -91,12 +91,12 @@ export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
 
 			if (sub === "show") {
 				if (!target) {
-					ui.notify("Usage: /rules show <relPath>", "error");
+					recordEvent("Usage: /rules show <relPath>", "error");
 					return;
 				}
 				const r = getRules().find((x) => x.relPath === target);
 				if (!r) {
-					ui.notify(`Rule not found: ${target}`, "error");
+					recordEvent(`Rule not found: ${target}`, "error");
 					return;
 				}
 				const text = `---\n${r.meta.description ? `description: ${r.meta.description}\n` : ""}source: ${r.sourceLabel}\nenabled: ${r.enabled}\n---\n\n${r.body}`;
@@ -119,12 +119,12 @@ export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
 
 			if (sub === "enable" || sub === "disable") {
 				if (!target) {
-					ui.notify(`Usage: /rules ${sub} <relPath>`, "error");
+					recordEvent(`Usage: /rules ${sub} <relPath>`, "error");
 					return;
 				}
 				const r = getRules().find((x) => x.relPath === target);
 				if (!r) {
-					ui.notify(`Rule not found: ${target}`, "error");
+					recordEvent(`Rule not found: ${target}`, "error");
 					return;
 				}
 				r.enabled = sub === "enable";
@@ -141,7 +141,7 @@ export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
 			}
 
 			if (sub === "add") {
-				ui.notify(
+				recordEvent(
 					"Use /rulewizard add <url> (or /rules add for the rule-creation guide).",
 					"info",
 				);
@@ -149,7 +149,7 @@ export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
 			}
 
 			if (sub === "new") {
-				ui.notify(
+				recordEvent(
 					"Use /rulewizard to scaffold a new rule (it guides the rule-vs-editorconfig-vs-linter decision).",
 					"info",
 				);
@@ -165,7 +165,7 @@ export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
 					path.join(require("node:os").homedir(), ".agents", "rules"),
 				];
 				const found = dirs.filter((d) => fs.existsSync(d));
-				ui.notify(
+				recordEvent(
 					"Rules sources (existing first):\n" +
 						(found.length ? found.map((d) => `  ✓ ${d}`).join("\n") : "  (none)") +
 						"\n" +
@@ -178,7 +178,7 @@ export function registerRulesCommand(pi: ExtensionAPI, deps: RulesDeps): void {
 				return;
 			}
 
-			ui.notify(
+			recordEvent(
 				`Usage: /rules <list|show|stats|analytics|enable|disable|reload|add|new|path> [target]`,
 				"error",
 			);

package/commands/rulewizard.ts

@@ -1,3 +1,4 @@
+import { emit } from "../visual/event-sink.ts";
 // =============================================================================
 // commands/rulewizard.ts — /rulewizard command (interactive guide)
 // =============================================================================
@@ -11,7 +12,7 @@ export function registerRulewizardCommand(pi: ExtensionAPI, deps: CommandsDeps):
 		description:
 			"interactive guide: decide whether a constraint should be a soly rule, an .editorconfig entry, or a linter config (eslint/biome/prettier). Use this BEFORE writing a new rule to avoid duplicating what linters already enforce.",
 		handler: async (_args, ctx) => {
-			ctx.ui.notify(
+			emit(
 				[
 					"soly-rule-wizard:",
 					"",

package/commands/soly.ts

@@ -68,7 +68,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 
 			const showFile = (label: string, content: string | null) => {
 				if (!content) {
-					ui.notify(`${label}: not found`, "error");
+					recordEvent(`${label}: not found`, "error");
 					return;
 				}
 				const MAX = 4000;
@@ -176,7 +176,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 						const s = getState();
 						if (s.position) {
 							// Brief status; full inspector lives in `/soly inspect`.
-							ui.notify(
+							recordEvent(
 								`pos: ${s.position.phase} · ${s.position.plan} (${s.position.status}) · ${s.progress.percent}%`,
 								"info",
 							);
@@ -199,7 +199,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 						if (parts.length <= 1) {
 							const s = getState();
 							if (!s.currentPlanPath) {
-								ui.notify("soly: no current plan", "error");
+								recordEvent("soly: no current plan", "error");
 								return;
 							}
 							showFile(
@@ -216,7 +216,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 					run: () => {
 						const s = getState();
 						if (!s.currentPhase) {
-							ui.notify("soly: no current phase", "error");
+							recordEvent("soly: no current phase", "error");
 							return;
 						}
 						const p = path.join(s.currentPhase.dir, `${s.currentPhase.slug}-CONTEXT.md`);
@@ -228,7 +228,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 					run: () => {
 						const s = getState();
 						if (!s.currentPhase) {
-							ui.notify("soly: no current phase", "error");
+							recordEvent("soly: no current phase", "error");
 							return;
 						}
 						const p = path.join(s.currentPhase.dir, `${s.currentPhase.slug}-RESEARCH.md`);
@@ -296,13 +296,13 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 					run: (parts: string[]) => {
 						const id = (parts[1] ?? "").trim();
 						if (!id) {
-							ui.notify("Usage: /soly task <task-id>", "error");
+							recordEvent("Usage: /soly task <task-id>", "error");
 							return;
 						}
 						const s = getState();
 						const task = s.tasks.find((t) => t.id === id);
 						if (!task) {
-							ui.notify(
+							recordEvent(
 								`soly: task ${id} not found.\nKnown: ${s.tasks.map((t) => t.id).join(", ") || "(none)"}`,
 								"error",
 							);
@@ -357,7 +357,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 								return;
 							}
 						}
-						ui.notify(
+						recordEvent(
 							`soly: no milestone file found. tried:\n  ${candidates.map((c) => path.relative(process.cwd(), c)).join("\n  ")}`,
 							"error",
 						);
@@ -536,7 +536,7 @@ export function registerSolyCommand(pi: ExtensionAPI, deps: SolyDeps): void {
 			}
 
 			if (!(subcommands as Record<string, unknown>)[sub]) {
-				ui.notify(`soly: unknown subcommand '${sub}'`, "error");
+				recordEvent(`soly: unknown subcommand '${sub}'`, "error");
 				return openMenu();
 			}
 

package/commands/why.ts

@@ -97,7 +97,7 @@ export function registerWhyCommand(pi: ExtensionAPI, deps: WhyDeps): void {
 					"If a behavior surprises you, look here first for the basis.",
 			);
 
-			ctx.ui.notify(lines.join("\n"), "info");
+			recordEvent(lines.join("\n"), "info");
 
 			// Suppress unused arg
 			void args;

package/index.ts

@@ -77,6 +77,7 @@ import piAskExtension from "./ask/index.ts";
 import piDeckExtension from "./deck/index.ts";
 import piArtifactExtension from "./artifact/index.ts";
 import { getArtifactServer } from "./artifact/session.ts";
+import { emit } from "./visual/event-sink.ts";
 
 /** Compact phase label for the chrome top bar, e.g. "plan 2/5". Null when idle. */
 function phaseLabelFromState(s: SolyState): string | null {
@@ -387,7 +388,7 @@ export default function solyExtension(pi: ExtensionAPI) {
 		// Non-error soly events land in the Working sub-line (└─ prefixed).
 		// The route goes through the chrome so the sub-line auto-clears on
 		// the next agent_start.
-		recordEvent: (text, level) => chrome.recordEvent(text, level),
+		recordEvent: (text, level) => chrome.emit(text, level),
 	});
 
 	registerTools(pi, {
@@ -397,7 +398,7 @@ export default function solyExtension(pi: ExtensionAPI) {
 	});
 
 	registerWorkflows(pi, {
-		recordEvent: (text, level) => chrome.recordEvent(text, level),
+		recordEvent: (text, level) => chrome.emit(text, level),
 		getState: () => state,
 		getInteractiveRules: () =>
 			combinedRules()
@@ -463,20 +464,20 @@ export default function solyExtension(pi: ExtensionAPI) {
 		const cfgResult = loadConfig(ctx.cwd);
 		activeConfig = cfgResult.config;
 		for (const w of cfgResult.warnings) {
-			ctx.ui.notify(`soly config: ${w}`, "warning");
+			emit(`soly config: ${w}`, "warning");
 		}
 		if (cfgResult.sources.global || cfgResult.sources.project) {
 			const sources = [
 				cfgResult.sources.global ? `global: ${cfgResult.sources.global}` : null,
 				cfgResult.sources.project ? `project: ${cfgResult.sources.project}` : null,
 			].filter(Boolean).join(", ");
-			ctx.ui.notify(`soly config loaded (${sources})`, "info");
+			emit(`soly config loaded (${sources})`, "info");
 		}
 		// Auto-prune old iteration files per retention config
 		if (state.exists) {
 			const r = pruneOldIterations(state.solyDir, activeConfig.iteration.retentionDays);
 			if (r.pruned > 0) {
-				ctx.ui.notify(
+				emit(
 					`soly: pruned ${r.pruned} old iteration file(s) (retention ${activeConfig.iteration.retentionDays}d)`,
 					"info",
 				);
@@ -540,7 +541,7 @@ export default function solyExtension(pi: ExtensionAPI) {
 		// Working indicator (└─ reloaded 47 rules). Errors here are real
 		// (disk I/O failed) and stay as popups.
 		hotReload.setNotifyHandler((reason) => {
-			chrome.recordEvent(`reloaded rules (${reason})`);
+			chrome.emit(`reloaded rules (${reason})`);
 		});
 
 		// Notifications (one-shot at startup)
@@ -556,10 +557,10 @@ export default function solyExtension(pi: ExtensionAPI) {
 			if (phaseRules.length > 0) {
 				summary += ` + ${phaseRules.length} phase-${state.currentPhase?.number}`;
 			}
-			ctx.ui.notify(summary, "info");
+			emit(summary, "info");
 
 			if (overriddenRulePaths.length > 0) {
-				ctx.ui.notify(
+				emit(
 					`soly: ${overriddenRulePaths.length} rule(s) overridden by project (${overriddenRulePaths.join(", ")})`,
 					"info",
 				);
@@ -583,23 +584,23 @@ export default function solyExtension(pi: ExtensionAPI) {
 				if (added.length) parts.push(`+${added.length}`);
 				if (removed.length) parts.push(`-${removed.length}`);
 				if (changed.length) parts.push(`~${changed.length}`);
-				ctx.ui.notify(`soly: rules changed since last session (${parts.join(" ")})`, "info");
+				emit(`soly: rules changed since last session (${parts.join(" ")})`, "info");
 			}
 
 			// Rule budget analytics
 			const analytics = analyzeRules(alwaysOnRules, CONTEXT_WINDOW_TOKENS);
 			if (analytics.contextBudgetPct > 5) {
-				ctx.ui.notify(
+				emit(
 					`soly: rules use ${analytics.contextBudgetPct.toFixed(1)}% of context window (${formatTok(analytics.totalTokens)} across ${analytics.fileCount} files)`,
 					"info",
 				);
 			}
 		} else {
-			ctx.ui.notify("soly rules: none found in .agents/rules.local, .agents/rules, or ~/.agents/rules", "info");
+			emit("soly rules: none found in .agents/rules.local, .agents/rules, or ~/.agents/rules", "info");
 		}
 
 		if (state.exists) {
-			ctx.ui.notify(`soly state: ${state.milestone} (${state.phases.length} phases)`, "info");
+			emit(`soly state: ${state.milestone} (${state.phases.length} phases)`, "info");
 		}
 
 		updateStatus(ctx);

package/init.ts

@@ -35,6 +35,7 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
 import { SOLY_DIRNAME } from "./core.js";
+import { emit } from "./visual/event-sink.ts";
 
 export type InitTemplate = "minimal" | "web-app" | "library" | "cli";
 
@@ -207,7 +208,7 @@ export async function initSolyProject(
 
 	// Preconditions
 	if (fs.existsSync(agentsDir)) {
-		ui.notify(
+		emit(
 			`soly init: project already initialized (found ${SOLY_DIRNAME}/). ` +
 				`Aborting to avoid overwriting.`,
 			"error",

package/mcp/commands.ts

@@ -20,6 +20,7 @@ import { getAuthForUrl } from "./mcp-auth.ts";
 import { notifyReconnectFailed } from "./notify.ts";
 import { loadOnboardingState, markSetupCompleted as persistSetupCompleted, markSharedConfigHintShown } from "./onboarding-state.ts";
 import { openPath } from "./utils.ts";
+import { emit } from "../visual/event-sink.ts";
 
 export async function showStatus(state: McpExtensionState, ctx: ExtensionContext): Promise<void> {
   if (!ctx.hasUI) return;
@@ -70,7 +71,7 @@ export async function showStatus(state: McpExtensionState, ctx: ExtensionContext
     );
   }
 
-  ctx.ui.notify(lines.join("\n"), "info");
+  emit(lines.join("\n"), "info");
 }
 
 export async function showTools(state: McpExtensionState, ctx: ExtensionContext): Promise<void> {
@@ -79,7 +80,7 @@ export async function showTools(state: McpExtensionState, ctx: ExtensionContext)
   const allTools = [...state.toolMetadata.values()].flat().map(m => m.name);
 
   if (allTools.length === 0) {
-    ctx.ui.notify("No MCP tools available", "info");
+    emit("No MCP tools available", "info");
     return;
   }
 
@@ -91,7 +92,7 @@ export async function showTools(state: McpExtensionState, ctx: ExtensionContext)
     `Total: ${allTools.length} tools`,
   ];
 
-  ctx.ui.notify(lines.join("\n"), "info");
+  emit(lines.join("\n"), "info");
 }
 
 export async function reconnectServers(
@@ -101,7 +102,7 @@ export async function reconnectServers(
 ): Promise<void> {
   if (targetServer && !state.config.mcpServers[targetServer]) {
     if (ctx.hasUI) {
-      ctx.ui.notify(`Server "${targetServer}" not found in config`, "error");
+      emit(`Server "${targetServer}" not found in config`, "error");
     }
     return;
   }
@@ -117,7 +118,7 @@ export async function reconnectServers(
       const connection = await state.manager.connect(name, definition);
       if (connection.status === "needs-auth") {
         if (ctx.hasUI) {
-          ctx.ui.notify(`MCP: ${name} requires OAuth. Run /mcp-auth ${name} first.`, "warning");
+          emit(`MCP: ${name} requires OAuth. Run /mcp-auth ${name} first.`, "warning");
         }
         continue;
       }
@@ -129,12 +130,12 @@ export async function reconnectServers(
       state.failureTracker.delete(name);
 
       if (ctx.hasUI) {
-        ctx.ui.notify(
+        emit(
           `MCP: Reconnected to ${name} (${connection.tools.length} tools, ${connection.resources.length} resources)`,
           "info"
         );
         if (failedTools.length > 0) {
-          ctx.ui.notify(`MCP: ${name} - ${failedTools.length} tools skipped`, "warning");
+          emit(`MCP: ${name} - ${failedTools.length} tools skipped`, "warning");
         }
       }
     } catch (error) {
@@ -159,13 +160,13 @@ export async function authenticateServer(
   const definition = config.mcpServers[serverName];
   if (!definition) {
     const message = `Server "${serverName}" not found in config`;
-    ctx.ui.notify(message, "error");
+    emit(message, "error");
     return { ok: false, message };
   }
 
   if (!supportsOAuth(definition)) {
     const message = `Server "${serverName}" does not use OAuth authentication. Set "auth": "oauth" or omit auth for auto-detection.`;
-    ctx.ui.notify(
+    emit(
       `Server "${serverName}" does not use OAuth authentication.\n` +
       `Set "auth": "oauth" or omit auth for auto-detection.`,
       "error"
@@ -175,7 +176,7 @@ export async function authenticateServer(
 
   if (!definition.url) {
     const message = `Server "${serverName}" has no URL configured (OAuth requires HTTP transport)`;
-    ctx.ui.notify(message, "error");
+    emit(message, "error");
     return { ok: false, message };
   }
 
@@ -185,7 +186,7 @@ export async function authenticateServer(
 
     if (status === "authenticated") {
       const message = `OAuth authentication successful for "${serverName}"! Run /mcp reconnect ${serverName} to connect with the new token.`;
-      ctx.ui.notify(
+      emit(
         `OAuth authentication successful for "${serverName}"!\n` +
         `Run /mcp reconnect ${serverName} to connect with the new token.`,
         "info"
@@ -194,11 +195,11 @@ export async function authenticateServer(
     }
 
     const message = `OAuth authentication failed for "${serverName}".`;
-    ctx.ui.notify(message, "error");
+    emit(message, "error");
     return { ok: false, message };
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
-    ctx.ui.notify(`Failed to authenticate "${serverName}": ${message}`, "error");
+    emit(`Failed to authenticate "${serverName}": ${message}`, "error");
     return { ok: false, message };
   } finally {
     ctx.ui.setStatus("mcp-auth", undefined);
@@ -213,7 +214,7 @@ export async function logoutServer(
   const definition = state.config.mcpServers[serverName];
   if (!definition) {
     const message = `Server "${serverName}" not found in config`;
-    if (ctx.hasUI) ctx.ui.notify(message, "error");
+    if (ctx.hasUI) emit(message, "error");
     return { ok: false, message };
   }
 
@@ -222,7 +223,7 @@ export async function logoutServer(
   updateStatusBar(state);
 
   const message = `OAuth credentials cleared for "${serverName}". Run /mcp-auth ${serverName} to authenticate again.`;
-  if (ctx.hasUI) ctx.ui.notify(message, "info");
+  if (ctx.hasUI) emit(message, "info");
   return { ok: true, message };
 }
 
@@ -376,7 +377,7 @@ export async function openMcpPanel(
           if (!result.cancelled && result.changes.size > 0) {
             writeDirectToolsConfig(result.changes, provenanceMap, config);
             configChanged = true;
-            ctx.ui.notify("Direct tools updated. Pi will reload after this panel closes.", "info");
+            emit("Direct tools updated. Pi will reload after this panel closes.", "info");
           }
           done(undefined);
           resolve();
@@ -404,7 +405,7 @@ export async function openMcpAuthPanel(
   const config = state.config;
   const oauthServers = Object.entries(config.mcpServers).filter(([, definition]) => supportsOAuth(definition));
   if (oauthServers.length === 0) {
-    ctx.ui.notify("No OAuth-capable MCP servers are configured.", "warning");
+    emit("No OAuth-capable MCP servers are configured.", "warning");
     return { configChanged: false };
   }
 

package/mcp/elicitation-handler.ts

@@ -12,6 +12,7 @@ import {
 import { AjvJsonSchemaValidator } from "@modelcontextprotocol/sdk/validation/ajv";
 import type { JsonSchemaType } from "@modelcontextprotocol/sdk/validation/types.js";
 import open from "open";
+import { emit } from "../visual/event-sink.ts";
 
 export type ElicitationValue = string | number | boolean | string[] | undefined;
 type FormProperty = ElicitRequestFormParams["requestedSchema"]["properties"][string];
@@ -104,7 +105,7 @@ async function collectValidField(
       }, { [name]: result.value });
       return result;
     } catch (error) {
-      ui.notify(error instanceof Error ? error.message : String(error), "error");
+      emit(error instanceof Error ? error.message : String(error), "error");
       current = result.value;
     }
   }

package/mcp/index.ts

@@ -14,6 +14,7 @@ import { initializeOAuth, shutdownOAuth } from "./mcp-auth-flow.ts";
 import { createMcpDirectToolCallRenderer, renderMcpProxyToolCall, renderMcpToolResult } from "./tool-result-renderer.ts";
 import { ToolCache, cacheKey as makeCacheKey } from "./tool-cache.ts";
 import { preloadAppBridge } from "./ext-apps-bridge.ts";
+import { emit } from "../visual/event-sink.ts";
 
 /** Default TTL for cached MCP tool results (60s). Tools that hit a stable
  *  server benefit; volatile ones are penalized for 60s — call sites can
@@ -190,12 +191,12 @@ export default function mcpAdapter(pi: ExtensionAPI) {
           state = await initPromise;
         } catch (error) {
           const message = error instanceof Error ? error.message : String(error);
-          if (ctx.hasUI) ctx.ui.notify(`MCP initialization failed: ${message}`, "error");
+          if (ctx.hasUI) emit(`MCP initialization failed: ${message}`, "error");
           return;
         }
       }
       if (!state) {
-        if (ctx.hasUI) ctx.ui.notify("MCP not initialized", "error");
+        if (ctx.hasUI) emit("MCP not initialized", "error");
         return;
       }
 
@@ -222,7 +223,7 @@ export default function mcpAdapter(pi: ExtensionAPI) {
         case "logout": {
           const serverName = rest;
           if (!serverName) {
-            if (ctx.hasUI) ctx.ui.notify("Usage: /mcp logout <server>", "error");
+            if (ctx.hasUI) emit("Usage: /mcp logout <server>", "error");
             return;
           }
           await logoutServer(serverName, state, ctx);
@@ -258,12 +259,12 @@ export default function mcpAdapter(pi: ExtensionAPI) {
           state = await initPromise;
         } catch (error) {
           const message = error instanceof Error ? error.message : String(error);
-          if (ctx.hasUI) ctx.ui.notify(`MCP initialization failed: ${message}`, "error");
+          if (ctx.hasUI) emit(`MCP initialization failed: ${message}`, "error");
           return;
         }
       }
       if (!state) {
-        if (ctx.hasUI) ctx.ui.notify("MCP not initialized", "error");
+        if (ctx.hasUI) emit("MCP not initialized", "error");
         return;
       }
 

package/mcp/init.ts

@@ -22,6 +22,7 @@ import { UiResourceHandler } from "./ui-resource-handler.ts";
 import { openUrl, parallelLimit } from "./utils.ts";
 import { logger } from "./logger.ts";
 import { getMissingConfiguredDirectToolServers } from "./direct-tools.ts";
+import { emit } from "../visual/event-sink.ts";
 import {
 	notifySessionRecovered,
 	notifySessionRecoveryFailed,
@@ -158,7 +159,7 @@ export async function initializeMcp(
   for (const { name, definition, connection, error } of results) {
     if (error || !connection) {
       if (ctx.hasUI) {
-        ctx.ui.notify(`MCP: Failed to connect to ${name}: ${error}`, "error");
+        emit(`MCP: Failed to connect to ${name}: ${error}`, "error");
       }
       console.error(`MCP: Failed to connect to ${name}: ${error}`);
       continue;
@@ -169,7 +170,7 @@ export async function initializeMcp(
     updateMetadataCache(state, name);
 
     if (failedTools.length > 0 && ctx.hasUI) {
-      ctx.ui.notify(
+      emit(
         `MCP: ${name} - ${failedTools.length} tools skipped`,
         "warning"
       );
@@ -183,7 +184,7 @@ export async function initializeMcp(
     const msg = failedCount > 0
       ? `MCP: ${connectedCount}/${startupServers.length} servers connected (${totalTools} tools)`
       : `MCP: ${connectedCount} servers connected (${totalTools} tools)`;
-    ctx.ui.notify(msg, "info");
+    emit(msg, "info");
   }
 
   const envDirect = process.env.MCP_DIRECT_TOOLS;
@@ -215,7 +216,7 @@ export async function initializeMcp(
       );
       const bootstrapped = bootstrapResults.filter(r => r.ok).map(r => r.name);
       if (bootstrapped.length > 0 && ctx.hasUI) {
-        ctx.ui.notify(`MCP: direct tools for ${bootstrapped.join(", ")} will be available after restart`, "info");
+        emit(`MCP: direct tools for ${bootstrapped.join(", ")} will be available after restart`, "info");
       }
     }
   }

package/mcp/notify.ts

@@ -14,6 +14,7 @@
 
 import { notifyFramed, type NotifBg } from "../notification.ts";
 import type { ExtensionUIContext } from "@earendil-works/pi-coding-agent";
+import { emit } from "../visual/event-sink.ts";
 
 const MCP_KEY_PREFIX = "mcp-";
 
@@ -36,7 +37,7 @@ function notifyBox(
 		// with a short title-only message. Don't crash the MCP handler.
 		const plain = lines.length > 0 ? `${title} — ${lines.join(" ")}` : title;
 		try {
-			ui.notify(plain, bg === "toolErrorBg" ? "error" : bg === "toolPendingBg" ? "warning" : "info");
+			emit(plain, bg === "toolErrorBg" ? "error" : bg === "toolPendingBg" ? "warning" : "info");
 		} catch {
 			// no UI at all — silent
 		}

package/notification.ts

@@ -6,7 +6,7 @@
 // soft background — the same pattern pi itself uses for branch summaries
 // and compaction messages (see pi-coding-agent's BranchSummaryMessage).
 //
-// Why a widget, not `ui.notify()`?
+// Why a widget, not `emit()`?
 //   - `notify()` is plain text, no background, no styled box
 //   - `setWidget(key, factory, opts)` accepts a factory returning a
 //     `Box` / `Text` component, which gives full theme support
@@ -29,6 +29,7 @@
 import { Box, Spacer, Text, type TUI } from "@earendil-works/pi-tui";
 import type { ExtensionUIContext, Theme } from "@earendil-works/pi-coding-agent";
 import { appendNotification } from "./notifications-log.ts";
+import { emit } from "./visual/event-sink.ts";
 
 /** Theme background color names (subset of ThemeBg). */
 export type NotifBg =

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-soly",
-  "version": "2.3.1",
+  "version": "2.3.3",
   "description": "Workflow + project management for pi-coding-agent. Plans, state, MANDATORY rules, self-review, multi-question picker. One npm install, zero config. LLM drives the workflow inline via the soly_workflow tool — no external subagent plugin.",
   "type": "module",
   "main": "index.ts",

package/visual/chrome.ts

@@ -20,6 +20,7 @@ import { SolyFooter } from "./footer.ts";
 import { SolyTopBar } from "./topbar.ts";
 import { SolyHeader, type WelcomeInput } from "./welcome.ts";
 import { buildWorkingMessage } from "./working.ts";
+import { emit } from "./event-sink.ts";
 
 /** Subset of soly config that controls the chrome (see config.ts `chrome`). */
 export type ChromeConfig = {
@@ -61,11 +62,11 @@ export type Chrome = {
 	updateWorking(ui: ExtensionUIContext, outputTokens: number): void;
 	/** Stop the telemetry line and restore the default message (agent_end). */
 	stopWorking(ui: ExtensionUIContext): void;
-	/** Record a non-error soly event. Rendered as a sub-line under the
-	 *  Working indicator (└─ prefixed, level-marker glyph). Auto-cleared on
-	 *  the next `startWorking`. Errors are still surfaced via
-	 *  `ui.notify(text, "error")` — call that directly for the popup. */
-	recordEvent(text: string, level?: "info" | "warning"): void;
+	/** Record a soly event (any level). Rendered as a sub-line under the
+	 *  Working indicator. No popups — everything goes through the sub-line.
+	 *  └─ for info, └─ ⚠ for warning, └─ ✗ for error. Auto-cleared on
+	 *  the next `startWorking`. */
+	emit(text: string, level?: "info" | "warning" | "error"): void;
 	/** Restore pi's native footer/widgets/indicator (session_shutdown / disable). */
 	dispose(ui?: ExtensionUIContext): void;
 };
@@ -94,7 +95,7 @@ export function createChrome(getConfig: () => ChromeConfig): Chrome {
 			},
 			workingWidth(),
 		);
-		try { working.ui.setWorkingMessage(message); } catch { /* session may have ended */ }
+		try { working.ui.setWorkingMessage(message + (data.recentEvent ? `\n    ${data.recentEventLevel === "error" ? "└─ ✗" : data.recentEventLevel === "warning" ? "└─ ⚠" : "└─"} ${data.recentEvent}` : "")); } catch { /* session may have ended */ }
 	};
 
 	const clearWorking = (): void => {
@@ -103,9 +104,12 @@ export function createChrome(getConfig: () => ChromeConfig): Chrome {
 	};
 
 	return {
-		recordEvent(text: string, level: "info" | "warning" = "info"): void {
+		emit(text: string, level: "info" | "warning" | "error" = "info"): void {
 			data.recentEvent = text;
 			data.recentEventLevel = level;
+			// Force the working message to re-render so the sub-line appears
+			// immediately, even mid-turn.
+			if (working) renderWorking();
 			try { tui?.requestRender(); } catch { /* not mounted yet */ }
 		},
 

package/visual/event-sink.ts

@@ -0,0 +1,30 @@
+// visual/event-sink.ts — global event sink for soly notifications
+//
+// All soly modules (commands, workflows, mcp, init) route their events
+// through this sink instead of ctx.ui.notify. The sink is set once at
+// session_start by index.ts (wired to chrome.recordEvent). Before it's
+// set, events are silently dropped (no chrome = no UI).
+//
+// Why a module-level global instead of threading `recordEvent` through
+// every function signature? The mcp/ module has 30+ call sites in
+// deeply-nested handlers; threading a new parameter through all of them
+// is pure churn with no architectural benefit — every call would pass
+// the same value. A global sink is the pragmatic choice.
+
+type EventLevel = "info" | "warning" | "error";
+type EventSink = (text: string, level?: EventLevel) => void;
+
+let sink: EventSink | null = null;
+
+/** Wire the event sink to the chrome's recordEvent. Called once at
+ *  session_start. Safe to call multiple times (idempotent). */
+export function setEventSink(fn: EventSink): void {
+	sink = fn;
+}
+
+/** Record a soly event. Routes to chrome.recordEvent if the sink is
+ *  wired (TUI mode); silently drops if not (RPC/print mode or before
+ *  session_start). */
+export function emit(text: string, level: EventLevel = "info"): void {
+	sink?.(text, level);
+}

package/visual/footer.ts

@@ -124,15 +124,6 @@ export class SolyFooter implements Component {
 	}
 
 	render(width: number): string[] {
-		const out: string[] = [buildFooterLine(this.data, this.fd, width, { ascii: this.getAscii(), styler: themeStyler(this.theme) })];
-		const event = this.data.recentEvent;
-		if (event) {
-			const glyph = this.data.recentEventLevel === "warning" ? "└─ ⚠" : "└─";
-			const styled = this.data.recentEventLevel === "warning"
-				? themeStyler(this.theme).fg("warning", `${glyph} ${event}`)
-				: themeStyler(this.theme).dim(`${glyph} ${event}`);
-			out.push("    " + styled);
-		}
-		return out;
+		return [buildFooterLine(this.data, this.fd, width, { ascii: this.getAscii(), styler: themeStyler(this.theme) })];
 	}
 }

package/workflows/index.ts

@@ -30,6 +30,7 @@ import { createVerifyLoop, type VerifyState } from "./verify.ts";
 import type { ContextManager } from "../context-manager.ts";
 import type { SolyState } from "../core.js";
 import type { SolyConfig } from "../config.js";
+import { emit } from "../visual/event-sink.ts";
 
 export interface WorkflowsDeps {
 	recordEvent: (text: string, level?: "info" | "warning") => void;
@@ -277,10 +278,10 @@ State inspection lives on the slash form — \`/soly <sub>\`:
 				"and .agents/.continue-here.md. Preserve milestone/phase/plan position and key " +
 				"decisions in the summary. Drop implementation-detail noise.",
 			onComplete: () => {
-				recordEvent("session compacted — use 'soly resume' to pick up");
+				emit("session compacted — use 'soly resume' to pick up");
 			},
 			onError: (err) => {
-				ctx.ui.notify(`soly: compact failed — ${err.message}`, "error");
+				emit(`soly: compact failed — ${err.message}`, "error");
 			},
 		});
 	});

package/workflows/inspect.ts

@@ -9,6 +9,7 @@ import * as path from "node:path";
 import { solyDirFor } from "../core.js";
 import type { SolyState } from "../core.js";
 import type { SolyConfig } from "../config.js";
+import { emit } from "../visual/event-sink.ts";
 
 interface InspectUI {
 	notify: (text: string, kind?: "info" | "warning" | "error") => void;
@@ -282,7 +283,7 @@ export function showTodos(
 	ui: InspectUI,
 ): void {
 	if (!state.exists) {
-		ui.notify("soly todos: no .agents/ directory in cwd", "error");
+		emit("soly todos: no .agents/ directory in cwd", "error");
 		return;
 	}
 	const file = findTodosFile(state.solyDir);
@@ -294,7 +295,7 @@ export function showTodos(
 	try {
 		parsed = JSON.parse(fs.readFileSync(file, "utf-8"));
 	} catch {
-		ui.notify(`soly todos: failed to parse ${path.basename(file)} (corrupt JSON?)`, "error");
+		emit(`soly todos: failed to parse ${path.basename(file)} (corrupt JSON?)`, "error");
 		return;
 	}
 	if (!parsed || !Array.isArray(parsed.todos) || parsed.todos.length === 0) {
@@ -327,7 +328,7 @@ export function showIterations(
 	limitDefault: number = 10,
 ): void {
 	if (!state.exists) {
-		ui.notify("soly iterations: no .agents/ directory in cwd", "error");
+		emit("soly iterations: no .agents/ directory in cwd", "error");
 		return;
 	}
 	const iterDir = path.join(state.solyDir, "iterations");
@@ -342,7 +343,7 @@ export function showIterations(
 	if (nArg) {
 		const parsed = parseInt(nArg, 10);
 		if (!Number.isFinite(parsed) || parsed <= 0) {
-			ui.notify(`soly iterations: invalid count "${nArg}"`, "error");
+			emit(`soly iterations: invalid count "${nArg}"`, "error");
 			return;
 		}
 		limit = parsed;
@@ -396,12 +397,12 @@ export function showDiffIterations(
 	ui: InspectUI,
 ): void {
 	if (!state.exists) {
-		ui.notify("soly diff iterations: no .agents/ directory in cwd", "error");
+		emit("soly diff iterations: no .agents/ directory in cwd", "error");
 		return;
 	}
 	const iterDir = path.join(state.solyDir, "iterations");
 	if (cmd.args.length < 2) {
-		ui.notify(
+		emit(
 			`soly diff iterations: need two file arguments (e.g. "soly diff iterations 05-02-exec-T1.md 05-02-exec-T2.md")`,
 			"error",
 		);
@@ -412,11 +413,11 @@ export function showDiffIterations(
 	const pathB = path.isAbsolute(b) ? b : path.join(iterDir, b);
 
 	if (!fs.existsSync(pathA)) {
-		ui.notify(`soly diff iterations: file not found: ${a}`, "error");
+		emit(`soly diff iterations: file not found: ${a}`, "error");
 		return;
 	}
 	if (!fs.existsSync(pathB)) {
-		ui.notify(`soly diff iterations: file not found: ${b}`, "error");
+		emit(`soly diff iterations: file not found: ${b}`, "error");
 		return;
 	}
 
@@ -454,23 +455,23 @@ export function showPhaseDelete(
 	ui: InspectUI,
 ): void {
 	if (!state.exists) {
-		ui.notify("soly phase delete: no .agents/ directory in cwd", "error");
+		emit("soly phase delete: no .agents/ directory in cwd", "error");
 		return;
 	}
 	if (cmd.args.length < 1) {
-		ui.notify("soly phase delete: need a phase number (e.g. `soly phase delete 5`)", "error");
+		emit("soly phase delete: need a phase number (e.g. `soly phase delete 5`)", "error");
 		return;
 	}
 	const phaseNum = parseInt(cmd.args[0]!, 10);
 	if (!Number.isFinite(phaseNum)) {
-		ui.notify(`soly phase delete: invalid phase number "${cmd.args[0]}"`, "error");
+		emit(`soly phase delete: invalid phase number "${cmd.args[0]}"`, "error");
 		return;
 	}
 
 	const phase = state.phases.find((p) => p.number === phaseNum);
 	if (!phase) {
 		const known = state.phases.map((p) => p.number).join(", ") || "(none)";
-		ui.notify(`soly phase delete: phase ${phaseNum} not found. Known: ${known}`, "error");
+		emit(`soly phase delete: phase ${phaseNum} not found. Known: ${known}`, "error");
 		return;
 	}
 
@@ -482,7 +483,7 @@ export function showPhaseDelete(
 		fs.mkdirSync(trashDir, { recursive: true });
 		fs.renameSync(phase.dir, dest);
 	} catch (e) {
-		ui.notify(`soly phase delete: failed to move phase (${(e as Error).message})`, "error");
+		emit(`soly phase delete: failed to move phase (${(e as Error).message})`, "error");
 		return;
 	}
 

package/workflows/new.ts

@@ -18,6 +18,7 @@ import * as fs from "node:fs";
 import { execFileSync } from "node:child_process";
 import { parsePlanName, type SolyCommand } from "./parser.ts";
 import type { SolyState } from "../core.js";
+import { emit } from "../visual/event-sink.ts";
 
 export interface NewResult {
 	handled: boolean;
@@ -152,7 +153,7 @@ export function buildNewTransform(
 		}
 		try {
 			git(["checkout", baseBranch], { cwd: projectRoot });
-			ui.notify(
+			emit(
 				`auto-checkout ${baseBranch} (was on ${currentBranch})`,
 				"info",
 			);

package/workflows/quick.ts

@@ -17,6 +17,7 @@ import * as path from "node:path";
 import { readIfExists, buildProgressBar, type SolyState } from "../core.js";
 import type { SolyConfig } from "../config.js";
 import type { SolyCommand } from "./parser.ts";
+import { emit } from "../visual/event-sink.ts";
 
 const execFileAsync = promisify(execFile);
 
@@ -36,7 +37,7 @@ export function showStatus(
 	config?: SolyConfig,
 ): void {
 	if (!state.exists) {
-		ui.notify("soly: no .agents/ directory in cwd", "error");
+		emit("soly: no .agents/ directory in cwd", "error");
 		return;
 	}
 	const maxPhases = config?.display.maxPhasesInStatus ?? 20;
@@ -122,14 +123,14 @@ const DECISIONS_TABLE_ROW = /^\|\s*([^|]+)\s*\|\s*([^|]+)\s*\|\s*([^|]+)\s*\|\s*
 
 export function showLog(cmd: SolyCommand, state: SolyState, ui: QuickUI): void {
 	if (!state.exists) {
-		ui.notify("soly log: no .agents/ directory in cwd", "error");
+		emit("soly log: no .agents/ directory in cwd", "error");
 		return;
 	}
 
 	const statePath = path.join(state.solyDir, "STATE.md");
 	const raw = readIfExists(statePath);
 	if (!raw) {
-		ui.notify("soly log: STATE.md not found", "error");
+		emit("soly log: STATE.md not found", "error");
 		return;
 	}
 
@@ -160,7 +161,7 @@ export function showLog(cmd: SolyCommand, state: SolyState, ui: QuickUI): void {
 	if (limitArg) {
 		const parsed = parseInt(limitArg, 10);
 		if (!Number.isFinite(parsed) || parsed <= 0) {
-			ui.notify(
+			emit(
 				`soly log: invalid limit "${limitArg}" (must be a positive integer)`,
 				"error",
 			);

package/workflows/settings-ui.ts

@@ -27,6 +27,7 @@ import type { Theme } from "@earendil-works/pi-coding-agent";
 import { ListPanel, type ListItem, type ListAction, type ListGroup, type PanelKeybindings } from "../visual/list-panel.ts";
 
 import { DEFAULT_CONFIG, type SolyConfig } from "../config.ts";
+import { emit } from "../visual/event-sink.ts";
 
 // ---------------------------------------------------------------------------
 // Setting descriptors
@@ -442,12 +443,12 @@ export function openSettingsUI(deps: SettingsUIDeps): void {
 						try {
 							const diff = diffVsDefaults(working);
 							saveConfigFile(solyDir, diff);
-							ui.notify(`settings saved → ${path.basename(solyDir)}/soly.json`, "info");
+							emit(`settings saved → ${path.basename(solyDir)}/soly.json`, "info");
 							// Re-read the config from disk so the active config matches
 							// what we just wrote.
 							reloadConfig();
 						} catch (e) {
-							ui.notify(`save failed: ${(e as Error).message}`, "error");
+							emit(`save failed: ${(e as Error).message}`, "error");
 						}
 					}
 					done();

package/workflows/verify.ts

@@ -18,6 +18,7 @@
 
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import type { AgentMessageLike, ContextManager, ContextRewriter } from "../context-manager.ts";
+import { emit } from "../visual/event-sink.ts";
 
 /** Resolved verify settings (from soly config `verify`). */
 export type VerifyConfig = {
@@ -121,7 +122,7 @@ export function createVerifyLoop(pi: ExtensionAPI, deps: VerifyDeps): VerifyLoop
 	let exitRes: RegExp[] = [];
 	let fixedRes: RegExp[] = [];
 
-	const emit = () => deps.onState({ active, iteration, max, fresh });
+	const emitState = () => deps.onState({ active, iteration, max, fresh });
 
 	// Captures the boundary on the first call, then strips prior iterations.
 	const rewriter: ContextRewriter = (messages) => {
@@ -137,8 +138,7 @@ export function createVerifyLoop(pi: ExtensionAPI, deps: VerifyDeps): VerifyLoop
 		active = false;
 		boundary = -1;
 		deps.contextManager.setRewriter(null);
-		emit();
-		ctx?.ui.notify(`soly verify: ${reason}`, "info");
+		emit(`soly verify: ${reason}`);
 	};
 
 	pi.on("agent_end", async (event, ctx) => {
@@ -158,7 +158,7 @@ export function createVerifyLoop(pi: ExtensionAPI, deps: VerifyDeps): VerifyLoop
 			stop(ctx, `stopped after ${max} iterations`);
 			return;
 		}
-		emit();
+		emit(`soly verify: loop iteration ${iteration}`);
 		pi.sendUserMessage(deps.getConfig().prompt, { deliverAs: "followUp" });
 	});
 
@@ -173,7 +173,7 @@ export function createVerifyLoop(pi: ExtensionAPI, deps: VerifyDeps): VerifyLoop
 		isActive: () => active,
 		start(ctx, opts = {}): void {
 			if (active) {
-				ctx.ui.notify("soly verify: already running", "info");
+				emit("soly verify: already running");
 				return;
 			}
 			const cfg = deps.getConfig();
@@ -185,8 +185,7 @@ export function createVerifyLoop(pi: ExtensionAPI, deps: VerifyDeps): VerifyLoop
 			exitRes = compilePatterns(cfg.exitPatterns);
 			fixedRes = compilePatterns(cfg.issuesFixedPatterns);
 			if (fresh) deps.contextManager.setRewriter(rewriter);
-			emit();
-			ctx.ui.notify(`soly verify: review mode on (max ${max}${fresh ? ", fresh context" : ""})`, "info");
+			emit("verify started");
 			pi.sendUserMessage(cfg.prompt);
 		},
 		stop,