pi-herdsman 0.19.0 → 0.19.1

package/dist/index.js

@@ -31,7 +31,7 @@ import { fileURLToPath as fileURLToPath2 } from "node:url";
 // package.json
 var package_default = {
   name: "pi-herdsman",
-  version: "0.19.0",
+  version: "0.19.1",
   type: "module",
   description: "Asynchronous Pi subagents and agent fleet orchestration for parallel coding agents with nested delegation, background work, and supervision in herdr.",
   license: "Apache-2.0",
@@ -5140,10 +5140,9 @@ function projectSupervision(options) {
     const provenance = options.workspaceProvenance?.get(agent.workspaceId) ?? (agent.workspaceCwd ? { workspaceCwd: agent.workspaceCwd } : void 0);
     const workspaceLabel = provenance?.repoName && provenance.branch ? `${provenance.repoName}/${provenance.branch}` : provenance?.workspaceLabel || (provenance?.workspaceCwd ? basename3(provenance.workspaceCwd) : void 0) || agent.workspaceId;
     const tokensName = typeof agent.tokens?.pi_herdsman_name === "string" ? agent.tokens.pi_herdsman_name.trim() : "";
-    const sessionName = typeof agent.sessionName === "string" ? agent.sessionName.trim() : "";
     const herdrName = agent.herdrName?.trim();
     const tabLabel = agent.tabLabel?.trim();
-    const name = tokensName || sessionName || (herdrName && herdrName !== agent.sessionId && !/_[0-9a-f]{16}$/iu.test(herdrName) ? herdrName : void 0) || tabLabel || `lead-${agent.sessionId.slice(0, 8)}`;
+    const name = tokensName || (herdrName && herdrName !== agent.sessionId && !/_[0-9a-f]{16}$/iu.test(herdrName) ? herdrName : void 0) || tabLabel || `lead-${agent.sessionId.slice(0, 8)}`;
     const herdRunStartedAt = metadataInteger(
       agent.tokens,
       "pi_herdsman_herd_run_started_at"
@@ -8815,30 +8814,6 @@ function supervisedSessionFile(agent, sessionId) {
     return void 0;
   }
 }
-function persistedSessionName(agent) {
-  const session = sessionIdentity(agent?.agent_session);
-  if (session?.kind !== "path") return void 0;
-  try {
-    const manager = SessionManager.open(session.value);
-    const name = manager.getSessionName();
-    return typeof name === "string" && name.trim() ? name.trim() : void 0;
-  } catch {
-    return void 0;
-  }
-}
-function isLeadSessionBoundary(agent, pane, ownerSessionId) {
-  if (!isPiAgent(agent) || pane?.agent !== "pi") return false;
-  const session = sessionIdentity(agent?.agent_session);
-  if (session?.kind !== "path") return false;
-  try {
-    if (SessionManager.open(session.value).getSessionId() !== ownerSessionId)
-      return false;
-    const manager = SessionManager.open(session.value);
-    return !sessionAgentIdentity(manager.getEntries(), manager.getSessionId());
-  } catch {
-    return false;
-  }
-}
 function herdrAliasMatchesIfReported(agent, expectedAlias) {
   const aliases = [agent?.name].filter(
     (value2) => typeof value2 === "string" && value2.length > 0
@@ -9499,7 +9474,7 @@ function assertUniqueDurableIdentities(states) {
     seen.add(key);
   }
 }
-async function managedAgentSnapshots(pi, ctx, signal, proveLead = false, allWorkspaces = false, suppliedInventory) {
+async function managedAgentSnapshots(pi, ctx, signal, proveLead = false, allWorkspaces = false, suppliedInventory, allowTranscriptDefinitionFallback = true) {
   const inventory = suppliedInventory ?? await herdrSessionSnapshot(pi, ctx, signal);
   const currentWorkspaceId = process.env.HERDR_WORKSPACE_ID ?? ctx.cwd;
   const allMailboxes = listAgentStates();
@@ -9514,7 +9489,8 @@ async function managedAgentSnapshots(pi, ctx, signal, proveLead = false, allWork
       agentDefinition = agentDefinitionForRuntime(
         agent,
         state,
-        runtimes.get(state.agentLabel)
+        runtimes.get(state.agentLabel),
+        allowTranscriptDefinitionFallback
       );
     } catch {
       agentDefinition = "unknown";
@@ -9604,8 +9580,17 @@ async function managedAgentSnapshots(pi, ctx, signal, proveLead = false, allWork
           const ownerPanes = panes.filter(
             (pane) => pane?.workspace_id === currentWorkspaceId && pane?.pane_id === ownerAgent.pane_id
           );
-          if (ownerPanes.length === 1 && isLeadSessionBoundary(ownerAgent, ownerPanes[0], ownerSessionId))
-            leadSessionIds.push(ownerSessionId);
+          if (ownerPanes.length !== 1 || !isPiAgent(ownerAgent) || ownerPanes[0]?.agent !== "pi")
+            continue;
+          try {
+            const state = readLeadCoordinationState(
+              supervisionRuntime(),
+              ownerSessionId
+            );
+            if (state && (state.role ?? "lead") === "lead")
+              leadSessionIds.push(ownerSessionId);
+          } catch {
+          }
         }
       }
     } catch {
@@ -9749,8 +9734,16 @@ function listedAgentRecord(view, snapshot, ownerSessionId, scope, unresolvedMail
     ...parentLabel ? { parent_label: parentLabel } : {}
   };
 }
-async function agentSnapshotView(pi, ctx, scope, signal, proveLead = false) {
-  const snapshot = await managedAgentSnapshots(pi, ctx, signal, proveLead);
+async function agentSnapshotView(pi, ctx, scope, signal, proveLead = false, allowTranscriptDefinitionFallback = true) {
+  const snapshot = await managedAgentSnapshots(
+    pi,
+    ctx,
+    signal,
+    proveLead,
+    false,
+    void 0,
+    allowTranscriptDefinitionFallback
+  );
   return {
     ...snapshot,
     visible: visibleAgentSnapshots(
@@ -10577,9 +10570,11 @@ function invalidateCachedRuntime(label) {
 function runtimeIdentityMatches(runtime, state, agent) {
   return runtime.runId === state.runId && runtime.ownerSessionId === state.ownerSessionId && runtime.workspaceId === agent.workspace_id && runtime.paneId === agent.pane_id && runtime.piSessionId === state.piSessionId && sameSessionPath(runtime.piSessionFile, state.piSessionFile) && sameCwd(runtime.cwd, agent.cwd);
 }
-function agentDefinitionForRuntime(agent, state, cached) {
+function agentDefinitionForRuntime(agent, state, cached, allowTranscriptFallback = true) {
   if (state.agentDefinition !== void 0) return stateAgentDefinition(state);
-  return cached && runtimeIdentityMatches(cached, state, agent) ? cached.agentDefinition : stateAgentDefinition(state);
+  if (cached && runtimeIdentityMatches(cached, state, agent))
+    return cached.agentDefinition;
+  return allowTranscriptFallback ? stateAgentDefinition(state) : "unknown";
 }
 function guardMailboxOccupancy(mailbox, label, operation, explicit) {
   const retainedRequest = (current) => {
@@ -10722,9 +10717,9 @@ async function resolveRuntime(pi, ctx, agentLabel, operation, signal) {
   await validateIntegration(pi, runtime, ctx, { signal });
   return { runtime, agent, controlState: agent.state };
 }
-function runtimeForListedAgent(agent, state, cached) {
+function runtimeForListedAgent(agent, state, cached, allowTranscriptFallback = true) {
   const useCached = cached !== void 0 && runtimeIdentityMatches(cached, state, agent);
-  const agentDefinition = state.agentDefinition !== void 0 ? stateAgentDefinition(state) : useCached ? cached.agentDefinition : stateAgentDefinition(state);
+  const agentDefinition = state.agentDefinition !== void 0 ? stateAgentDefinition(state) : useCached ? cached.agentDefinition : allowTranscriptFallback ? stateAgentDefinition(state) : "unknown";
   const runtime = (useCached ? cached : void 0) ?? {
     label: agent.label,
     herdrAgent: herdrAgentAlias(agent.workspace_id, agent.label, state.runId),
@@ -12919,6 +12914,7 @@ ${globalInstructions}
   const publishLeadRole = (ctx, mode, generation) => {
     const paneId = process.env.HERDR_PANE_ID;
     if (!paneId) return Promise.resolve();
+    const leadName = mode === "inactive" && activeRole() === "lead" ? pi.getSessionName()?.trim() : void 0;
     const args = [
       "pane",
       "report-metadata",
@@ -12926,9 +12922,15 @@ ${globalInstructions}
       "--source",
       "pi-herdsman:lead",
       "--title",
-      mode === "active" ? "chief" : activeRole() === "manager" ? "Pi Herdsman manager" : "Pi Herdsman lead",
+      mode === "active" ? "chief" : activeRole() === "manager" ? "Pi Herdsman manager" : leadName || "Pi Herdsman lead",
       ...mode === "active" ? ["--token", "pi_herdsman_role=chief"] : mode === "inactive" ? ["--token", `pi_herdsman_role=${activeRole()}`] : ["--clear-token", "pi_herdsman_role"]
     ];
+    if (mode === "inactive" && activeRole() === "lead") {
+      args.push(
+        leadName ? "--token" : "--clear-token",
+        leadName ? `pi_herdsman_name=${leadName}` : "pi_herdsman_name"
+      );
+    }
     if (mode !== "inactive" || activeRole() !== "lead")
       args.push(
         "--clear-token",
@@ -14629,11 +14631,17 @@ ${summary}` : "Herd run settled.",
     let healthTimer;
     let healthGeneration = 0;
     let supervisionTimer;
+    let supervisionRefreshInFlight;
+    let pendingSupervisionRefresh;
     let supervisionOverviewGeneration = 0;
     let activeSupervisionRender;
     const loadSupervisionSnapshot = async (ctx, suppliedInventory, suppliedAgents, includeAll = false) => {
       if (activeRole() === "chief" && !includeAll) {
-        const reports = await directReports(ctx);
+        const reports = await directReports(
+          ctx,
+          suppliedInventory,
+          suppliedAgents
+        );
         const managers = reports.filter(
           (report) => report.role === "manager"
         );
@@ -14711,7 +14719,6 @@ ${summary}` : "Herd run settled.",
       const agents = live.flatMap((agent) => {
         const sessionId = herdrSessionId(agent);
         if (!isPiAgent(agent) || !sessionId) return [];
-        const sessionName = persistedSessionName(agent);
         const candidateSessionFile = supervisedSessionFile(agent, sessionId);
         const piSessionFile = candidateSessionFile && persistedTranscriptReady({
           piSessionId: sessionId,
@@ -14726,7 +14733,6 @@ ${summary}` : "Herd run settled.",
             tabId: agent.tab_id,
             workspaceCwd: agent.cwd,
             herdrName: agent.name,
-            ...sessionName ? { sessionName } : {},
             ...piSessionFile ? { piSessionFile } : {},
             tokens: agent.tokens,
             runtimeState: normalizeHerdrLifecycleState(agent)
@@ -14805,8 +14811,20 @@ ${summary}` : "Herd run settled.",
       if (!chiefAuthority)
         throw new Error("Staff is available only to an active supervisor");
       const managers = [];
-      const allLeads = (await loadSupervisionSnapshot(ctx, void 0, void 0, true)).leads;
-      const managedAgents = (await managedAgentSnapshots(pi, ctx, ctx.signal, false, true)).agents;
+      const allLeads = (await loadSupervisionSnapshot(
+        ctx,
+        suppliedInventory,
+        suppliedAgents,
+        true
+      )).leads;
+      const managedAgents = (suppliedAgents ?? await managedAgentSnapshots(
+        pi,
+        ctx,
+        ctx.signal,
+        false,
+        true,
+        suppliedInventory
+      )).agents;
       const managerLeadSessions = /* @__PURE__ */ new Set();
       const descriptors = listManagerDescriptors(supervisionRuntime());
       for (const descriptor of descriptors) {
@@ -15485,12 +15503,11 @@ ${summary}` : "Herd run settled.",
         throw error2;
       }
     };
-    const refreshSupervision = async (ctx, isCurrent) => {
+    const refreshSupervisionOnce = async (ctx) => {
       if (chiefMode !== "active" && !(activeRole() === "manager" && !roleSuspended))
-        return false;
+        return;
       const generation = currentSupervisionGeneration(ctx);
-      const refreshIsCurrent = () => currentSupervisionGeneration(ctx) === generation && (isCurrent?.() ?? true);
-      let refreshed = false;
+      const refreshIsCurrent = () => currentSupervisionGeneration(ctx) === generation;
       try {
         const inventory = await herdrSessionSnapshot(pi, ctx, ctx.signal);
         const agents = await managedAgentSnapshots(
@@ -15499,15 +15516,15 @@ ${summary}` : "Herd run settled.",
           ctx.signal,
           false,
           true,
-          inventory
+          inventory,
+          false
         );
         const snapshot = await loadSupervisionSnapshot(ctx, inventory, agents);
-        if (!refreshIsCurrent()) return false;
+        if (!refreshIsCurrent()) return;
         supervisionSnapshot = snapshot;
         supervisionSnapshotKnown = true;
         supervisionSnapshotGeneration = generation;
         supervisionStale = false;
-        refreshed = true;
         activeSupervisionRender?.();
       } catch {
         if (refreshIsCurrent())
@@ -15519,7 +15536,20 @@ ${summary}` : "Herd run settled.",
         } catch {
         }
       }
-      return refreshed;
+    };
+    const refreshSupervision = (ctx) => {
+      pendingSupervisionRefresh = ctx;
+      if (supervisionRefreshInFlight) return supervisionRefreshInFlight;
+      supervisionRefreshInFlight = (async () => {
+        while (pendingSupervisionRefresh) {
+          const next = pendingSupervisionRefresh;
+          pendingSupervisionRefresh = void 0;
+          await refreshSupervisionOnce(next);
+        }
+      })().finally(() => {
+        supervisionRefreshInFlight = void 0;
+      });
+      return supervisionRefreshInFlight;
     };
     const latestCustomMessageText = (ctx, customType) => {
       const entry = [
@@ -15541,7 +15571,7 @@ ${summary}` : "Herd run settled.",
       const generation = currentSupervisionGeneration(ctx);
       const isCurrent = () => (isCurrentChief(ctx) || activeRole() === "manager") && currentSupervisionGeneration(ctx) === generation;
       try {
-        await refreshSupervision(ctx, isCurrent);
+        await refreshSupervision(ctx);
         if (!isCurrent()) return;
         const status = supervisionSnapshotStatus(ctx);
         const content = formatSupervisionContext(
@@ -15647,6 +15677,7 @@ This is the session-start definition snapshot. Use agent_list for live Agent sta
       void refreshSupervision(ctx);
     };
     clearSupervisionUI = (removeWidget = true) => {
+      pendingSupervisionRefresh = void 0;
       activeSupervisionRender = void 0;
       requestSupervisionWidgetRender = void 0;
       if (supervisionTimer) clearInterval(supervisionTimer);
@@ -16002,13 +16033,14 @@ This is the session-start definition snapshot. Use agent_list for live Agent sta
       breadcrumb: initialStatusBreadcrumb,
       ...ownToolsSnapshot2()
     };
-    const loadStatusSnapshot = async (ctx, signal) => {
+    const loadStatusSnapshot = async (ctx, signal, allowTranscriptDefinitionFallback = true) => {
       const view = await agentSnapshotView(
         pi,
         ctx,
         controllerScope,
         signal,
-        true
+        true,
+        allowTranscriptDefinitionFallback
       );
       const ownerSessionId = ctx.sessionManager.getSessionId();
       const unresolvedMailboxState = controllerScope?.kind === "lead" && listAgentStateIssues().length > 0;
@@ -16097,7 +16129,8 @@ This is the session-start definition snapshot. Use agent_list for live Agent sta
       try {
         lastValidStatus = await loadStatusSnapshot(
           ctx,
-          controllerAbortController?.signal
+          controllerAbortController?.signal,
+          false
         );
         if (generation !== statusGeneration || ctx !== statusContext) return;
         reconcilePendingStarts(lastValidStatus);
@@ -17586,7 +17619,7 @@ Active work or pending results may be discarded.`
       controllerScope,
       unresolvedMailboxState
     ).available_tools ?? []).map((tool) => tool.slice("agent_".length));
-    const publishAgentLoss = (ctx, state2, availableActions, nextReminderMs, signal) => {
+    const publishAgentLoss = (ctx, state2, snapshotAgentDefinition, availableActions, nextReminderMs, signal) => {
       const mailbox = agentMailboxPath(state2.workspaceId, state2.agentLabel);
       const current = readAgentState(mailbox);
       if (!current || !sameManagedAgentIdentity(current, state2) || current.ownerSessionId !== ctx.sessionManager.getSessionId())
@@ -17648,7 +17681,7 @@ Active work or pending results may be discarded.`
               paneId: current.paneId,
               piSessionId: current.piSessionId,
               piSessionFile: current.piSessionFile,
-              agentDefinition: stateAgentDefinition(current),
+              agentDefinition: current.agentDefinition ?? snapshotAgentDefinition ?? "unknown",
               requestId: latestRequestId,
               availableActions,
               nextReminderMs
@@ -17665,7 +17698,15 @@ Active work or pending results may be discarded.`
       if (signal.aborted || generation !== healthGeneration || !ctx.isIdle())
         return;
       const ownerSessionId = ctx.sessionManager.getSessionId();
-      const snapshot = await managedAgentSnapshots(pi, ctx, signal);
+      const snapshot = await managedAgentSnapshots(
+        pi,
+        ctx,
+        signal,
+        false,
+        false,
+        void 0,
+        false
+      );
       if (signal.aborted || generation !== healthGeneration || !ctx.isIdle())
         return;
       const view = {
@@ -17746,7 +17787,14 @@ Active work or pending results may be discarded.`
           const episode2 = "lost";
           if (published || !attentionDue(state2.runId, episode2, now)) continue;
           const intervalMs2 = nextAttentionInterval(state2.runId, episode2);
-          if (publishAgentLoss(ctx, state2, availableActions, intervalMs2, signal)) {
+          if (publishAgentLoss(
+            ctx,
+            state2,
+            agent.agentDefinition,
+            availableActions,
+            intervalMs2,
+            signal
+          )) {
             recordAttention(state2.runId, episode2, intervalMs2);
             published = true;
           }
@@ -17994,7 +18042,8 @@ Active work or pending results may be discarded.`
               runtimeForListedAgent(
                 agent.listed,
                 current,
-                runtimes.get(current.agentLabel)
+                runtimes.get(current.agentLabel),
+                false
               ),
               diagnosticSignal
             );
@@ -18005,7 +18054,15 @@ Active work or pending results may be discarded.`
         if (attemptedDiagnostic && !diagnostic) {
           let refreshed;
           try {
-            refreshed = await managedAgentSnapshots(pi, ctx, signal);
+            refreshed = await managedAgentSnapshots(
+              pi,
+              ctx,
+              signal,
+              false,
+              false,
+              void 0,
+              false
+            );
           } catch {
             continue;
           }
@@ -18896,7 +18953,10 @@ Active work or pending results may be discarded.`
         pi,
         ctx,
         metadataAbortController?.signal,
-        true
+        true,
+        false,
+        void 0,
+        false
       );
       if (generation !== leafStatusGeneration || ctx !== leafStatusContext)
         return;