pi-advisor-flow 0.2.7 → 0.2.8

package/CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to this project are documented here.
 
 The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## 0.2.8
+
+### Fixed
+
+- Advisor session blocks are reported to Herdr again. Herdr grants each pane a single lifecycle status authority, which for Pi is Herdr's own `herdr-agent-state` integration, and silently drops semantic state from any other source. The Advisor block now signals that integration over Pi's in-process event bus (`herdr:blocked`) instead of relying on display metadata alone, so a blocked pane shows `blocked` rather than `working`. Only the unblocked-to-blocked edge emits, because the listener refcounts; clears are always delivered.
+
 ## 0.2.7
 
 ### Security

package/extensions/index.ts

@@ -1,5 +1,6 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { registerCommands } from "../src/commands.js";
+import { setHerdrBlockedEmitter } from "../src/herdr.js";
 import {
   consultAdvisor as consultAdvisorImplementation,
   parseAutomaticDecision as parseAutomaticDecisionImplementation,
@@ -28,6 +29,9 @@ export const runAdvisorGate = (
 ) => runAdvisorGateImplementation(...args);
 
 export default function (pi: ExtensionAPI) {
+  setHerdrBlockedEmitter((active, label) =>
+    pi.events.emit("herdr:blocked", { active, label })
+  );
   registerAdvisorTool(pi);
   registerCommands(pi);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-advisor-flow",
-  "version": "0.2.7",
+  "version": "0.2.8",
   "author": "Philip Brembeck",
   "repository": {
     "type": "git",

package/src/herdr.ts

@@ -38,6 +38,25 @@ export interface HerdrNotificationRequest {
 export type HerdrRequest = HerdrMetadataRequest | HerdrNotificationRequest;
 type Report = (request: HerdrRequest) => void;
 
+// Herdr only accepts semantic state from the pane's lifecycle authority, which
+// for pi is herdr's own herdr-agent-state extension (source "herdr:pi"). Socket
+// reports from any other source are dropped, so the blocked state has to be
+// signalled through the in-process pi event bus that integration listens on.
+type BlockedEmitter = (active: boolean, label: string) => void;
+let emitBlocked: BlockedEmitter | undefined;
+
+export const setHerdrBlockedEmitter = (emitter: BlockedEmitter | undefined) => {
+  emitBlocked = emitter;
+};
+
+const safeEmitBlocked = (active: boolean, label = "Advisor blocked") => {
+  try {
+    emitBlocked?.(active, label);
+  } catch {
+    /* Herdr is optional. */
+  }
+};
+
 const isControlCharacter = (character: string) =>
   character <= "\u001f" || character === "\u007f";
 
@@ -167,8 +186,14 @@ export class HerdrAdvisorBlock {
     if (!this.enabled()) {
       return;
     }
+    const label = cleanNotification(reason, 200);
+    const wasBlocked: boolean = this.#blocked;
+    if (!wasBlocked) {
+      // The listener refcounts, so only the false → true edge may emit.
+      safeEmitBlocked(true, label);
+    }
     this.#blocked = true;
-    this.safeReport({ blocked: cleanNotification(reason, 200) });
+    this.safeReport({ blocked: label });
   }
 
   clear() {
@@ -179,6 +204,7 @@ export class HerdrAdvisorBlock {
     }
     // Clearing previously reported state is a de-escalation and must still be
     // delivered if integration was disabled after the block was reported.
+    safeEmitBlocked(false);
     try {
       this.report({
         id: `${BLOCK_SOURCE}:${nextSequence()}`,