trantor 0.17.93 → 0.17.95

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "trantor",
-  "version": "0.17.93",
+  "version": "0.17.95",
   "description": "Trantor \u2014 the hub-world for AI agent crews: live message bus, presence, project Kanban/flow board + crew orchestration for independent AI coding agents (Claude, Codex, Gemini, Kimi, DeepSeek)",
   "mcpServers": {
     "relay": {

package/hooks/lib/handoff.mjs

@@ -187,9 +187,26 @@ function digest(turns, budget = 56_000) {
   return out;
 }
 
-function haveScrooge() {
-  if (process.env.TRANTOR_NO_SCROOGE === "1") return false; // opt out (tests / no-LLM summary)
-  try { execSync("command -v scrooge", { stdio: "ignore" }); return true; } catch { return false; }
+// Where scrooge actually lives. `command -v` alone was the bug: it installs into ~/.local/bin,
+// which is NOT on a default PATH, so every handoff written from a hook, launchd job or precompact
+// silently failed the check and dumped raw transcript instead. Those are precisely the automatic
+// paths, so the summarizer was missing exactly when nobody was watching. Resolve to an absolute
+// path and exec THAT.
+const SCROOGE_DIRS = [
+  join(homedir(), ".local", "bin"),
+  "/opt/homebrew/bin",
+  "/usr/local/bin",
+  "/usr/bin",
+];
+function resolveScrooge() {
+  if (process.env.TRANTOR_NO_SCROOGE === "1") return "";   // opt out (tests / no-LLM summary)
+  if (process.env.TRANTOR_SCROOGE_BIN && existsSync(process.env.TRANTOR_SCROOGE_BIN)) return process.env.TRANTOR_SCROOGE_BIN;
+  try {
+    const p = execSync("command -v scrooge", { encoding: "utf8", stdio: ["ignore", "pipe", "ignore"] }).trim();
+    if (p && existsSync(p)) return p;
+  } catch {}
+  for (const d of SCROOGE_DIRS) { const p = join(d, "scrooge"); if (existsSync(p)) return p; }
+  return "";
 }
 
 export function buildSummary(transcriptPath) {
@@ -198,14 +215,38 @@ export function buildSummary(transcriptPath) {
   try { convo = digest(collectTurns(transcriptPath)); } catch { convo = ""; }
   if (!convo) return "*(transcript unreadable)*";
   const sys = "You are writing a SESSION HANDOFF so a fresh Claude Code session can take over without losing context. The text spans an entire (possibly multi-hour) session: opening turns, an even sample of the middle, and the recent tail. Produce a concise but COMPLETE markdown handoff with these sections: TASK (what we're doing + the goal), STATE (done / in-progress), KEY DECISIONS, OPEN THREADS & NEXT STEPS (concrete actions), KEY FILES & locations (exact paths). Be specific. Cover the whole arc, not just the end. Do not pad.";
-  if (haveScrooge()) {
+  // Cut the raw tail on a TURN boundary. A blind slice(-12000) opens mid-sentence, which is how the
+  // 2026-08-24 handoff began, and a successor cannot tell a truncated thought from a complete one.
+  const tail = (n) => {
+    // Only trim to a turn boundary when we ACTUALLY truncated. When the whole digest fits, trimming
+    // would throw away the session's opening — which is the part a successor needs most, and which
+    // test-handoff.mjs rightly insists on.
+    if (convo.length <= n) return convo;
+    const cut = convo.slice(-n);
+    const b = cut.indexOf("\n\n");
+    return b > 0 && b < 2000 ? cut.slice(b + 2) : cut;
+  };
+  // Say WHICH failure this was. One string for "not installed" and "the call died" meant nobody
+  // could tell a missing tool from a broken one, and the reason only ever reached stderr.
+  const degraded = (why) =>
+    `*(⚠️ DEGRADED HANDOFF — this is a raw transcript tail, not a written summary.*\n`
+    + `*Reason: ${why}. It may open mid-thought and it OMITS anything older than the tail;*\n`
+    + `*treat the project's memory files as the reliable record and re-read them before acting.)*\n\n${tail(12000)}`;
+
+  const bin = resolveScrooge();
+  if (bin) {
     try {
-      return execSync(`scrooge -t summarize -d medium --system ${JSON.stringify(sys)}`, {
-        input: convo, encoding: "utf8", timeout: 60_000, maxBuffer: 8 * 1024 * 1024,
-      }).trim() || `*(empty summary — raw recent tail)*\n\n${convo.slice(-8000)}`;
-    } catch (e) { process.stderr.write(`[trantor] scrooge summarize failed: ${e?.message}\n`); }
+      // 29s observed summarizing a 56KB digest, so 60s left almost no headroom on a slow provider.
+      return execSync(`${JSON.stringify(bin)} -t summarize -d medium --system ${JSON.stringify(sys)}`, {
+        input: convo, encoding: "utf8", timeout: 180_000, maxBuffer: 8 * 1024 * 1024,
+      }).trim() || degraded("the summarizer returned nothing");
+    } catch (e) {
+      const why = `the summarizer failed: ${String(e?.message || e).slice(0, 200)}`;
+      process.stderr.write(`[trantor] scrooge summarize failed: ${e?.message}\n`);
+      return degraded(why);
+    }
   }
-  return `*(no summarizer available — representative transcript digest)*\n\n${convo.slice(-12000)}`;
+  return degraded("no summarizer is installed (scrooge was not found on PATH or in the usual locations)");
 }
 
 // The exact recent exchange, VERBATIM (not summarized/sampled) — so a baton-pass handoff carries the

package/hub.mjs

@@ -388,7 +388,13 @@ function dutyTick() {
   const floor = now() - 24 * 3600 * 1000;                 // never escalate ancient history
   for (const m of state.messages) {
     if (m.ts > cutoff || m.ts < floor) continue;
-    if (!m.to || m.to === "all" || m.to === DUTY_SESSION || m.from === "hub:duty") continue;
+    // `hub:*` is the hub's own pseudo-identity, not a session: nothing polls it and nothing ever
+    // will, so a message addressed there can never be "delivered". Escalating it is a category
+    // error that feeds itself — the duty seat acks the escalation to hub:duty, that ack is
+    // undelivered too, and since dutyEscalated prunes its oldest ids at 5,000 the same ones come
+    // back around. Reported from the seat as "a fresh identical echo every stop-hook cycle".
+    // Skipping the FROM side was already here; the TO side is the half that loops.
+    if (!m.to || m.to === "all" || m.to === DUTY_SESSION || m.from === "hub:duty" || m.to.startsWith("hub:")) continue;
     if (dutyEscalated.has(m.id)) continue;
     if ((state.peers[m.to]?.deliveredUpTo || 0) >= m.id) continue;
     dutyEscalated.add(m.id);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "trantor",
-  "version": "0.17.93",
+  "version": "0.17.95",
   "type": "module",
   "bin": {
     "trantor": "bin/cli.mjs"
@@ -11,7 +11,7 @@
     "zod": "^4.4.3"
   },
   "scripts": {
-    "test": "node bin/slop-gate.mjs --surface desktop/src && node test.mjs && node test-scenarios.mjs && node test-failure.mjs && node test-redelivery.mjs && node test-crew-completion.mjs && node test-contracts.mjs && node test-handoff.mjs && node test-agents.mjs && node test-update.mjs && node test-handoff-guard.mjs && node test-baton-latest.mjs && node test-balances.mjs && node test-subagent-cost.mjs && node test-inbox.mjs && node test-inflight.mjs && node test-notify.mjs && node test-focus.mjs && node test-cardlog.mjs && node test-relay-note.mjs && node test-reaper.mjs && node test-events.mjs && node test-proposals.mjs && node test-scrub.mjs && node test-store-delta.mjs && node test-message-re.mjs && node test-claims.mjs && node test-adopt.mjs && node test-summarize.mjs && node test-identity-core.mjs && node test-identity.mjs && node test-identity-instances.mjs && node test-duty.mjs && node test-duty-seat.mjs && node test-discovery.mjs && node test-doctor.mjs && node test-splitbrain.mjs && node test-overseer.mjs && node test-overseer-lib.mjs && node test-overseer-warn.mjs && node test-provider-keys.mjs && node test-inbox-delivery.mjs && node test-identity-drift.mjs && node test-seats.mjs && node test-seat-identity.mjs && node test-hub-routing.mjs && node test-hook-routing.mjs && node test-relay-wait.mjs && node test-dsh-seat.mjs && node test-kimi-bridge.mjs && node test-kimi-events.mjs && python3 engine/test-routing.py && node test-reconcile.mjs && node test-resources.mjs && node test-patrol.mjs && node test-bridge.mjs && bash test-crew.sh"
+    "test": "node bin/slop-gate.mjs --surface desktop/src && node test.mjs && node test-scenarios.mjs && node test-failure.mjs && node test-redelivery.mjs && node test-crew-completion.mjs && node test-contracts.mjs && node test-handoff.mjs && node test-handoff-summarizer.mjs && node test-agents.mjs && node test-update.mjs && node test-handoff-guard.mjs && node test-baton-latest.mjs && node test-balances.mjs && node test-subagent-cost.mjs && node test-inbox.mjs && node test-inflight.mjs && node test-notify.mjs && node test-focus.mjs && node test-cardlog.mjs && node test-relay-note.mjs && node test-reaper.mjs && node test-events.mjs && node test-proposals.mjs && node test-scrub.mjs && node test-store-delta.mjs && node test-message-re.mjs && node test-claims.mjs && node test-adopt.mjs && node test-summarize.mjs && node test-identity-core.mjs && node test-identity.mjs && node test-identity-instances.mjs && node test-duty.mjs && node test-duty-seat.mjs && node test-discovery.mjs && node test-doctor.mjs && node test-splitbrain.mjs && node test-overseer.mjs && node test-overseer-lib.mjs && node test-overseer-warn.mjs && node test-provider-keys.mjs && node test-inbox-delivery.mjs && node test-identity-drift.mjs && node test-inbox-staleness.mjs && node test-seats.mjs && node test-seat-identity.mjs && node test-hub-routing.mjs && node test-hook-routing.mjs && node test-relay-wait.mjs && node test-dsh-seat.mjs && node test-kimi-bridge.mjs && node test-kimi-events.mjs && python3 engine/test-routing.py && node test-reconcile.mjs && node test-resources.mjs && node test-patrol.mjs && node test-bridge.mjs && bash test-crew.sh"
   },
   "description": "The hub-world for AI agent crews \u2014 orchestrate Claude Code, Codex, GLM, Kimi, DeepSeek & any OpenRouter model as live crews with a plan-aware Advisor, a Kanban/flow command center, a testing gate, and an economics brain (Scrooge).",
   "files": [