@dorokuma/herdsman-pi 0.11.4 → 0.11.5

package/LICENSE

@@ -0,0 +1,23 @@
+MIT License
+
+Copyright (c) 2026 Ryo Nakae
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.
+
+Copyright (c) 2026 dorokuma (herdsman)

package/README.md

@@ -12,7 +12,7 @@ pi install npm:@dorokuma/herdsman-pi
 herdsman daemon start
 ```
 
-When Pi runs inside Herdr, this extension connects to the Herdsman daemon and registers its exact Pi session path as presence identity. It does not send per-turn tool-result or final-message telemetry.
+When Pi runs inside Herdr, this extension connects to the Herdsman daemon and registers its exact Pi session path as presence identity. After register it sends `agent.ping` at least every 30 seconds so an idle owner is not dropped by the daemon heartbeat. It does not send per-turn tool-result or final-message telemetry.
 
 Enter these commands in Pi, not in a shell:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dorokuma/herdsman-pi",
-  "version": "0.11.4",
+  "version": "0.11.5",
   "description": "Pi extension bridge for Herdsman agent history. Forked from @ryonakae/herdsman.",
   "type": "module",
   "keywords": [
@@ -11,7 +11,7 @@
   "license": "MIT",
   "files": [
     "src",
-    "../LICENSE"
+    "LICENSE"
   ],
   "publishConfig": {
     "access": "public"

package/src/daemon-client.ts

@@ -245,11 +245,12 @@ export class ReconnectingDaemonClient {
         code: message.error.code,
       });
       pending.reject(error);
-      // Optional methods (turn completion signaling) are rejected like any other
-      // failure but must not mark the daemon incompatible: an older daemon
-      // answers "Unknown method" and the connection has to stay alive.
+      // Optional methods (turn completion signaling, keepalive ping) are rejected
+      // like any other failure but must not mark the daemon incompatible: an older
+      // daemon answers "Unknown method" and the connection has to stay alive.
       if (
         pending.method !== "agent.turn.completed" &&
+        pending.method !== "agent.ping" &&
         /unknown|not found|unsupported|method/i.test(error.message)
       ) {
         const incompatible = new Error("Herdsman daemon version is incompatible; restart the session");

package/src/index.ts

@@ -3,6 +3,7 @@ import { sanitizeText } from "./sanitize-text.js";
 import { appendFileSync, mkdirSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, isAbsolute, join } from "node:path";
+import { stripVTControlCharacters } from "node:util";
 
 export type HerdsmanPiLogLevel = "info" | "warn" | "error";
 
@@ -180,6 +181,7 @@ const HERDR_REQUIRED_MESSAGE = "Herdsman requires a Herdr workspace";
 const RECONNECTING_MESSAGE = "Herdsman is reconnecting · try again shortly";
 export const MAX_ACK_ATTEMPTS = 5;
 export const ACK_BACKOFF_CAP_MS = 30_000;
+const KEEPALIVE_INTERVAL_MS = 30_000;
 
 type AckFailureClass = "terminal" | "resync" | "transient";
 
@@ -267,8 +269,23 @@ export function createHerdsmanPiExtension(options: ExtensionOptions = {}) {
     };
     options.onStateExposed?.(state);
     let activeContext: PiContext | undefined;
+    let keepaliveTimer: ReturnType<typeof setInterval> | undefined;
     let wakeGeneration = 0;
 
+    const stopKeepalive = () => {
+      if (!keepaliveTimer) return;
+      clearInterval(keepaliveTimer);
+      keepaliveTimer = undefined;
+    };
+
+    const startKeepalive = (client: HerdsmanDaemonClient) => {
+      stopKeepalive();
+      keepaliveTimer = setInterval(() => {
+        void client.request("agent.ping", {}).catch(() => undefined);
+      }, KEEPALIVE_INTERVAL_MS);
+      keepaliveTimer.unref?.();
+    };
+
     const setHerdsmanUi = (ctx: PiContext | undefined) => {
       if (!ctx) return;
       const footerState: HerdsmanFooterState = state.reconnectingFromOn
@@ -744,12 +761,14 @@ export function createHerdsmanPiExtension(options: ExtensionOptions = {}) {
             snapshot
               ? {
                   ...snapshot,
-                  agents: snapshot.agents.filter((agent) => {
-                    const nextAgents = next?.agents.filter((candidate) => candidate.paneId === agent.paneId) ?? [];
-                    if (nextAgents.length === 0) return false;
-                    return nextAgents.some(
-                      (nextAgent) => !agent.id || !nextAgent.id || agent.id === nextAgent.id,
+                  agents: snapshot.agents.flatMap((agent) => {
+                    const nextAgents =
+                      next?.agents.filter((candidate) => candidate.paneId === agent.paneId) ?? [];
+                    if (nextAgents.length === 0) return [];
+                    const nextAgent = nextAgents.find(
+                      (candidate) => !agent.id || !candidate.id || agent.id === candidate.id,
                     );
+                    return nextAgent ? [nextAgent] : [];
                   }),
                 }
               : undefined;
@@ -782,11 +801,13 @@ export function createHerdsmanPiExtension(options: ExtensionOptions = {}) {
         })
         .then((response) => {
           state.connected = true;
+          startKeepalive(client);
           applyConnectionStateResponse(response as ConnectionStateResponse, ctx, {
             notifyReconnectLoss: true,
           });
         })
         .catch((error) => {
+          stopKeepalive();
           state.connected = false;
           const incompatibleMessage =
             error instanceof Error && /incompatible/i.test(error.message)
@@ -865,12 +886,19 @@ export function createHerdsmanPiExtension(options: ExtensionOptions = {}) {
         loseRole(ctx);
         return;
       }
+      stopKeepalive();
       state.client?.close();
       const client = options.clientFactory?.() ?? new ReconnectingDaemonClient({ socketPath: defaultSocketPath() });
       client.resetForSession?.();
+      const closeClient = client.close.bind(client);
+      client.close = () => {
+        stopKeepalive();
+        closeClient();
+      };
       state.client = client;
       client.onConnected = () => registerPresence(ctx);
       client.onDisconnected = () => {
+        stopKeepalive();
         state.connected = false;
         markDisconnected(activeContext);
       };
@@ -878,6 +906,7 @@ export function createHerdsmanPiExtension(options: ExtensionOptions = {}) {
     });
 
     pi.on("session_shutdown", () => {
+      stopKeepalive();
       state.connected = false;
       loseRole(activeContext);
       state.deliveredBatch = undefined;
@@ -1161,11 +1190,44 @@ export function formatHiddenAgentContext(input: {
         agent: agent.agent ?? "unknown",
         name: agent.name,
       });
-      return [
-        `- ${identity} ${agent.paneId ?? "unknown"} ${agent.agentStatus ?? "unknown"}`,
-        `  last user: ${oneLine(sanitizeText(history.lastUserMessage?.text ?? "").text)}`,
-        `  last assistant: ${oneLine(sanitizeText(history.lastAssistantMessage?.text ?? "").text)}`,
-      ].join("\n");
+      const paneId = agent.paneId ?? "unknown";
+      const status = agent.agentStatus ?? "unknown";
+      const prefix = `- ${identity} ${paneId} ${status}`;
+
+      const rawAgent = record(agent);
+      const tabTitleCandidate =
+        stringValue(rawAgent.terminalTitle) ?? stringValue(rawAgent.label);
+      const tabTitleCleaned =
+        tabTitleCandidate !== null ? sanitizeAndCleanContextText(tabTitleCandidate) : "";
+      const tabTitle =
+        tabTitleCleaned.length > 0 ? truncateSummary(tabTitleCleaned, 60) : null;
+
+      const rawHistory = record(history);
+      const lastAssistantRecord = record(history.lastAssistantMessage);
+      const timeCandidate =
+        lastAssistantRecord.timestamp ??
+        history.updatedAt ??
+        rawHistory.updatedAt ??
+        rawAgent.updatedAt ??
+        rawAgent.time;
+      const formattedTime = formatTimestamp(timeCandidate);
+
+      const assistantRaw = history.lastAssistantMessage?.text;
+      const assistantCleaned =
+        assistantRaw !== undefined && assistantRaw !== null
+          ? sanitizeAndCleanContextText(assistantRaw)
+          : "";
+      const assistantSummary =
+        assistantCleaned.length > 0 ? truncateSummary(assistantCleaned, 100) : null;
+
+      const segments = [
+        prefix,
+        tabTitle,
+        formattedTime,
+        assistantSummary,
+      ].filter((segment): segment is string => segment !== null && segment.length > 0);
+
+      return segments.join(" · ");
     }),
     "Use herdsman agent get/read if details are needed.",
   ].join("\n");
@@ -1247,3 +1309,50 @@ function record(value: unknown): Record<string, unknown> {
 function oneLine(value: string): string {
   return value.replace(/\s+/g, " ");
 }
+
+function cleanContextText(value: string): string {
+  return stripVTControlCharacters(value)
+    .replace(
+      /[\u0000-\u0008\u000b\u000c\u000e-\u001f\u007f-\u009f\u00ad\u180e\u200b-\u200f\u202a-\u202e\u2060-\u2064\u2066-\u2069\ufeff]/g,
+      "",
+    )
+    .replace(/\s+/g, " ")
+    .trim();
+}
+
+function sanitizeAndCleanContextText(value: string): string {
+  const preCleaned = cleanContextText(value);
+  if (!preCleaned) return "";
+  const sanitized = sanitizeText(preCleaned).text;
+  return cleanContextText(sanitized);
+}
+
+function truncateSummary(value: string, limit = 100): string {
+  if (value.length <= limit) return value;
+  const maxSearch = Math.min(value.length, limit + 8);
+  const searchSlice = value.slice(0, maxSearch);
+  const lastSpaceIndex = searchSlice.search(/\s[^\s]*$/);
+  if (lastSpaceIndex >= limit - 20) {
+    return `${value.slice(0, lastSpaceIndex)}…`;
+  }
+  return `${value.slice(0, limit)}…`;
+}
+
+function formatTimestamp(value: unknown): string | null {
+  if (typeof value === "string") {
+    const trimmed = value.trim();
+    if (/^\d{2}:\d{2}:\d{2}$/.test(trimmed)) return trimmed;
+    if (/^\d{2}-\d{2} \d{2}:\d{2}:\d{2}$/.test(trimmed)) return trimmed;
+  }
+  if (!value) return null;
+  const date = value instanceof Date ? value : new Date(value as string | number);
+  if (Number.isNaN(date.getTime())) return null;
+  const pad = (n: number) => String(n).padStart(2, "0");
+  const month = pad(date.getMonth() + 1);
+  const day = pad(date.getDate());
+  const hours = pad(date.getHours());
+  const minutes = pad(date.getMinutes());
+  const seconds = pad(date.getSeconds());
+  return `${month}-${day} ${hours}:${minutes}:${seconds}`;
+}
+