pi-extended-teams 2.3.2 → 2.4.0

package/extensions/tools/agent-communication-tools.ts

@@ -122,7 +122,7 @@ export function createAgentCommunicationTools(options: AgentCommunicationToolsOp
     {
       name: "read_inbox",
       label: "Read Inbox",
-      description: "Read this agent's inbox in the current Pi session.",
+      description: "Read this agent's inbox with current sender message-admission status. Status is a snapshot, not proof of process liveness; do not send to unavailable senders.",
       parameters: Type.Object({
         unread_only: Type.Optional(Type.Boolean({ default: true })),
         mark_as_read: Type.Optional(Type.Boolean({ default: true, description: "Set false to peek without marking messages read." })),
@@ -131,7 +131,7 @@ export function createAgentCommunicationTools(options: AgentCommunicationToolsOp
         const teamName = requireCurrentSession(options);
         const markAsRead = params.mark_as_read !== false;
         const unreadOnly = params.unread_only !== false;
-        const msgs = await messaging.readInbox(teamName, options.agentName, unreadOnly, markAsRead);
+        const msgs = await messaging.readInboxWithSenderStatus(teamName, options.agentName, unreadOnly, markAsRead);
         const lifecycleRunId = options.getLifecycleRunId();
         // Do not fail after read flags are persisted; telemetry is best-effort.
         if (markAsRead && lifecycleRunId) {

package/extensions/tools/coordination-tools.ts

@@ -333,7 +333,7 @@ export function registerCoordinationTools(pi: any, options: CoordinationToolsOpt
   pi.registerTool({
     name: "read_inbox",
     label: "Read Inbox",
-    description: "Read messages from the current Pi session inbox. Defaults to this agent's inbox.",
+    description: "Read messages from the current Pi session inbox with current sender message-admission status. Defaults to your inbox. Status is a snapshot, not proof of process liveness; do not send to unavailable senders.",
     parameters: Type.Object({
       agent_name: Type.Optional(Type.String({ description: "Whose inbox to read. Defaults to your own." })),
       unread_only: Type.Optional(Type.Boolean({ default: true })),
@@ -345,7 +345,7 @@ export function registerCoordinationTools(pi: any, options: CoordinationToolsOpt
       if (process.env.PI_EXTENDED_TEAMS_HERDR_RESUME === "1" && targetAgent !== options.agentName) throw new Error("A resumed subagent can only read its own inbox.");
       const markAsRead = params.mark_as_read !== false;
       const unreadOnly = params.unread_only !== false;
-      const msgs = await messaging.readInbox(targetTeamName, targetAgent, unreadOnly, markAsRead);
+      const msgs = await messaging.readInboxWithSenderStatus(targetTeamName, targetAgent, unreadOnly, markAsRead);
 
       if (markAsRead && options.isTeammate && targetAgent === options.agentName) {
         const config = await teams.readConfig(targetTeamName).catch(() => null);

package/extensions/ui/agent-follow-view.ts

@@ -28,7 +28,7 @@ export interface AgentFollowTranscriptOptions {
 
 type TranscriptBlock =
   | { kind: "section"; label: "user" | "thinking" | "assistant"; text: string }
-  | { kind: "tool"; id?: string; name: string; args: unknown; result?: string; details?: unknown; isError?: boolean };
+  | { kind: "tool"; id?: string; name: string; args: unknown; result?: string; readSummary?: string; details?: unknown; isError?: boolean };
 
 function stringifyToolArgs(args: unknown): string {
   if (args === undefined) return "";
@@ -42,6 +42,7 @@ function stringifyToolArgs(args: unknown): string {
 function compactToolArgs(name: string, args: unknown): string {
   if (!args || typeof args !== "object") return sanitizeTuiLine(stringifyToolArgs(args));
   const values = args as Record<string, unknown>;
+  if (name === "ls") return compactTranscriptLine(String(values.path ?? "."));
   const primary = name === "bash"
     ? values.command
     : name === "read"
@@ -226,6 +227,14 @@ function renderToolBlock(theme: ExtendedTeamsTheme, block: Extract<TranscriptBlo
   if (compactBlock) return compactBlock;
 
   const header = renderToolHeader(theme, block);
+  if (!expandLargeToolResults && (block.name === "read" || block.name === "bash" || block.name === "ls")) {
+    const state = block.result === undefined
+      ? pendingText(theme, "working")
+      : block.isError ? failureText(theme, "✗") : successText(theme, "✓");
+    const summary = block.name === "read" && !block.isError && block.readSummary
+      ? mutedText(theme, ` · ${block.readSummary}`) : "";
+    return [boundTranscriptLine(`${header}${summary}${mutedText(theme, " · ")}${state}`, width)];
+  }
   if (block.result === undefined) {
     return [header, `${structuralText(theme, "│")} ${pendingText(theme, "waiting for result…")}`, `${structuralText(theme, "╰─")} ${pendingText(theme, "running")}`, ""];
   }
@@ -290,28 +299,40 @@ export function formatAgentFollowTranscript(messages: any[], options: AgentFollo
       const name = sanitizeTuiLine(String(message.toolName || "tool"));
       const matchingTool = (id ? toolsById.get(id) : undefined)
         ?? blocks.slice().reverse().find((block): block is Extract<TranscriptBlock, { kind: "tool" }> => block.kind === "tool" && block.result === undefined && block.name === name);
+      let readSummary: string | undefined;
+      if ((matchingTool?.name ?? name) === "read") {
+        const text = typeof message.content === "string" ? message.content
+          : (Array.isArray(message.content) ? message.content : [])
+            .filter((part: any) => part?.type === "text" && typeof part.text === "string")
+            .map((part: any) => part.text).join("\n");
+        const lineCount = text ? text.replace(/\r\n?/g, "\n").replace(/\n$/, "").split("\n").length : 0;
+        readSummary = `${lineCount} line${lineCount === 1 ? "" : "s"} · ${formatResultSize(text)}`;
+      }
       const result = sanitizeTuiText(extractTextParts(message.content));
       const isError = typeof message.isError === "boolean" ? message.isError : undefined;
       if (matchingTool) {
         matchingTool.result = result;
+        matchingTool.readSummary = readSummary;
         matchingTool.details = message.details;
         matchingTool.isError = isError;
       } else {
-        blocks.push({ kind: "tool", id, name, args: undefined, result, details: message.details, isError });
+        blocks.push({ kind: "tool", id, name, args: undefined, result, readSummary, details: message.details, isError });
       }
     }
   }
 
-  const lines = blocks.flatMap(block => {
+  const lines: string[] = [];
+  for (const block of blocks) {
     if (block.kind === "tool") {
-      return renderToolBlock(theme, block, options.expandLargeToolResults === true, options.width);
+      lines.push(...renderToolBlock(theme, block, options.expandLargeToolResults === true, options.width));
+    } else if (block.label === "thinking") {
+      if (lines.length > 0 && lines[lines.length - 1] !== "") lines.push("");
+      lines.push(theme.fg("thinkingText", block.label), block.text.replace(/\*\*/g, ""), "");
+    } else {
+      const labelToken = block.label === "user" ? "customMessageLabel" : "accent";
+      lines.push(theme.fg(labelToken, block.label), block.text, "");
     }
-    if (block.label === "thinking") {
-      return [theme.fg("thinkingText", block.label), block.text.replace(/\*\*/g, ""), ""];
-    }
-    const labelToken = block.label === "user" ? "customMessageLabel" : "accent";
-    return [theme.fg(labelToken, block.label), block.text, ""];
-  });
+  }
   return lines.length > 0 ? lines : [theme.fg("dim", "Waiting for the agent's first transcript event…")];
 }
 

package/extensions/ui/renderers.ts

@@ -104,7 +104,11 @@ export function summarizeInboxMessage(message: any): string {
   const from = message?.from || "unknown";
   const summary = message?.summary || "message";
   const timestamp = message?.timestamp ? new Date(message.timestamp).toLocaleTimeString() : "";
-  return `${from}${timestamp ? ` ${timestamp}` : ""}: ${summary}`;
+  const sender = message?.senderStatus;
+  const availability = sender
+    ? ` [sender now: ${sender.status}; can message: ${sender.canMessage === null ? "unknown" : sender.canMessage ? "yes" : "no"}]`
+    : "";
+  return `${from}${availability}${timestamp ? ` ${timestamp}` : ""}: ${summary}`;
 }
 
 export function formatInboxMessagesForModel(messages: any[]): string {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-extended-teams",
-  "version": "2.3.2",
+  "version": "2.4.0",
   "description": "Control-first, session-connected subagents for Pi with live navigation and intent-tier routing",
   "repository": {
     "type": "git",
@@ -28,7 +28,7 @@
   "scripts": {
     "typecheck": "tsc --noEmit",
     "test": "vitest run",
-    "test:focused": "vitest run src/utils/claims.test.ts src/utils/tasks.test.ts src/utils/model-resolution.test.ts src/utils/settings.test.ts src/utils/write-queue.test.ts src/utils/shared-memory.test.ts extensions/index.test.ts extensions/events/register-events.test.ts extensions/internal/session-files.test.ts extensions/team/contracts.test.ts extensions/tools/delegation-guard.test.ts extensions/tools/agent-communication-tools.test.ts extensions/tools/read-helper.test.ts extensions/agents/read-agent.test.ts extensions/agents/write-agent.test.ts extensions/tools/team-tools.read-agent.test.ts extensions/ui/agent-follow-view.test.ts extensions/tools/coordination-tools.test.ts",
+    "test:focused": "vitest run --maxWorkers=2 src/utils/claims.test.ts src/utils/tasks.test.ts src/utils/model-resolution.test.ts src/utils/settings.test.ts src/utils/write-queue.test.ts src/utils/shared-memory.test.ts extensions/index.test.ts extensions/events/register-events.test.ts extensions/internal/session-files.test.ts extensions/team/contracts.test.ts extensions/tools/delegation-guard.test.ts extensions/tools/agent-communication-tools.test.ts extensions/tools/read-helper.test.ts extensions/agents/read-agent.test.ts extensions/agents/write-agent.test.ts extensions/tools/team-tools.read-agent.test.ts extensions/ui/agent-follow-view.test.ts extensions/tools/coordination-tools.test.ts",
     "build": "tsc --noEmit"
   },
   "main": "extensions/index.ts",

package/src/utils/messaging.ts

@@ -7,7 +7,12 @@ import { InboxMessage, TeamConfig } from "./models";
 import { withLock } from "./lock";
 import { configPath, inboxPath } from "./paths";
 import { readConfig } from "./teams";
-import { assertLifecycleTombstoneAbsent, withLifecycleTombstoneLock } from "./lifecycle-tombstone";
+import {
+  assertLifecycleTombstoneAbsent,
+  readLifecycleTombstoneSnapshot,
+  withLifecycleTombstoneLock,
+  type LifecycleTombstonePhase,
+} from "./lifecycle-tombstone";
 
 export interface MessageMetadataOptions {
   id?: string;
@@ -272,6 +277,57 @@ export async function readInbox(
   });
 }
 
+export interface InboxSenderStatus {
+  status: "lead" | "active" | "not_running" | "quarantined" | "unknown" | LifecycleTombstonePhase;
+  canMessage: boolean | null;
+}
+
+export async function readInboxWithSenderStatus(
+  teamName: string,
+  agentName: string,
+  unreadOnly = false,
+  markAsRead = true
+): Promise<Array<InboxMessage & { senderStatus: InboxSenderStatus }>> {
+  const messages = await readInbox(teamName, agentName, unreadOnly, markAsRead);
+  if (messages.length === 0) return [];
+
+  // Advisory admission snapshot, not process liveness or a reservation to send.
+  // Enrich only returned copies: persisted inbox history must not cache live status.
+  let members: TeamConfig["members"] | undefined;
+  try {
+    const config = JSON.parse(fs.readFileSync(configPath(teamName), "utf-8")) as TeamConfig;
+    if (Array.isArray(config.members)) members = config.members;
+  } catch (error) {
+    if ((error as NodeJS.ErrnoException).code === "ENOENT") members = [];
+  }
+
+  const senders = new Map<string, InboxSenderStatus>();
+  for (const sender of new Set(messages.map(message => message.from))) {
+    let senderStatus: InboxSenderStatus = { status: "unknown", canMessage: null };
+    try {
+      if (sender === "team-lead") {
+        senderStatus = { status: "lead", canMessage: true };
+      } else {
+        const fence = readLifecycleTombstoneSnapshot(teamName, sender);
+        if (fence.status !== "absent") {
+          senderStatus = {
+            status: fence.status === "corrupt" ? "quarantined" : fence.tombstone.phase,
+            canMessage: false,
+          };
+        } else if (members) {
+          const member = members.find(member => member.name === sender);
+          const canMessage = !!member && member.isActive !== false;
+          senderStatus = { status: canMessage ? "active" : "not_running", canMessage };
+        }
+      }
+    } catch {
+      // Status lookup must not discard messages after read flags were saved.
+    }
+    senders.set(sender, senderStatus);
+  }
+  return messages.map(message => ({ ...message, senderStatus: senders.get(message.from)! }));
+}
+
 export async function markInboxMessagesRead(teamName: string, agentName: string, ids: string[]): Promise<void> {
   const p = inboxPath(teamName, agentName);
   if (!fs.existsSync(p)) throw new Error("Referenced inbox is unavailable.");