diffing 0.18.3 → 0.18.4

package/dist/cli.mjs

@@ -711,6 +711,7 @@ startupLease?.release();
 startupLease = null;
 const localUrl = appendSessionToken(`http://${host === "0.0.0.0" ? "127.0.0.1" : host}:${actualPort}${prMode ? "/gh/pr" : ""}`, authToken ?? void 0);
 console.log(`diffing server running at ${localUrl}`);
+console.error(`DIFFING_READY ${localUrl} mode=${prMode ? "gh-pr" : "web"} pid=${process.pid}`);
 if (openModulePromise) try {
 	const settings = loadSettings();
 	const openUrl = appendSessionToken(`http://${host === "0.0.0.0" ? "127.0.0.1" : host}:${actualPort}${prMode ? "/gh/pr" : ""}`, authToken ?? void 0);

package/extensions/pi/index.ts

@@ -194,6 +194,45 @@ async function refreshStatus(ctx: ExtensionContext): Promise<void> {
 	}
 }
 
+// ────────────────────────────────────────────────────────────────────────────
+// Verdict notices (herdr-visible)
+//
+// After a review/plan/mockup await returns a real verdict, surface a compact,
+// greppable `DIFFING_VERDICT …` line in the pi pane so a sibling herdr pane can
+// observe review state via `herdr pane read` / `herdr wait output`. We use pi's
+// own UI hooks (notify + widget) rather than `herdr pane send-text`, which would
+// type raw bytes into the pi TUI input box instead of the rendered scrollback.
+// ────────────────────────────────────────────────────────────────────────────
+
+interface VerdictNotice {
+	kind: "plan" | "mockup" | "review";
+	decision: string;
+}
+
+function verdictNotice(stderr: string): VerdictNotice | null {
+	const plan = stderr.match(/DIFFING_PLAN_DECISION=(\S+)/);
+	if (plan) return { kind: "plan", decision: plan[1] };
+	const mockup = stderr.match(/DIFFING_MOCKUP_DECISION=(\S+)/);
+	if (mockup) return { kind: "mockup", decision: mockup[1] };
+	const review = stderr.match(/DIFFING_REVIEW_ROUND=(\S+)/);
+	if (review) return { kind: "review", decision: `released round=${review[1]}` };
+	return null;
+}
+
+function emitVerdictNotice(ctx: ExtensionContext, stderr: string): void {
+	const notice = verdictNotice(stderr);
+	if (!notice) return;
+	const line = `DIFFING_VERDICT ${notice.kind} decision=${notice.decision}`;
+	try {
+		if (ctx.hasUI) {
+			ctx.ui.notify(line, "info");
+			ctx.ui.setWidget("diffing-verdict", [line]);
+		}
+	} catch {
+		// UI hooks are best-effort; never break the tool result.
+	}
+}
+
 // ────────────────────────────────────────────────────────────────────────────
 // Canonical skill root + self-heal
 // ────────────────────────────────────────────────────────────────────────────
@@ -560,6 +599,7 @@ export default function (pi: ExtensionAPI) {
 					{ parked: true, exitCode: result.exitCode },
 				);
 			}
+			emitVerdictNotice(ctx, result.stderr);
 			return textResult(describe(result, "diffing await-review"), {
 				parked: false,
 				exitCode: result.exitCode,
@@ -667,6 +707,7 @@ export default function (pi: ExtensionAPI) {
 					{ parked: true, exitCode: result.exitCode },
 				);
 			}
+			emitVerdictNotice(ctx, result.stderr);
 			return textResult(describe(result, "diffing plan await"), {
 				parked: false,
 				exitCode: result.exitCode,
@@ -920,6 +961,7 @@ export default function (pi: ExtensionAPI) {
 					{ parked: true, exitCode: result.exitCode },
 				);
 			}
+			emitVerdictNotice(ctx, result.stderr);
 			return textResult(describe(result, "diffing mockup await"), {
 				parked: false,
 				exitCode: result.exitCode,

package/extensions/pi/pi-modules.d.ts

@@ -26,7 +26,7 @@ declare module "@earendil-works/pi-coding-agent" {
 				text: string,
 				level?: "info" | "error" | "success" | "warning",
 			): void;
-			setWidget?(id: string, lines: string[]): void;
+			setWidget(id: string, lines: string[] | undefined): void;
 		};
 	}
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "diffing",
-  "version": "0.18.3",
+  "version": "0.18.4",
   "description": "local-first CLI for reviewing, navigating, and discussing git diffs with AI",
   "type": "module",
   "license": "MIT",