@lumi.ai/runner 0.5.6 → 0.5.7

package/dist/cli.js

@@ -9,7 +9,7 @@ import {
   collection as collection5,
   deleteField as deleteField2,
   doc as doc7,
-  getDoc as getDoc5,
+  getDoc as getDoc6,
   getDocs as getDocs4,
   onSnapshot as onSnapshot2,
   orderBy as orderBy4,
@@ -242,6 +242,9 @@ function clampStepText(text, max) {
 function firstLine(text) {
   return text.split("\n").find((l) => l.trim()) ?? "";
 }
+function step(kind, label, detail) {
+  return detail ? { kind, label, detail } : { kind, label };
+}
 function str(input, key) {
   if (!input || typeof input !== "object")
     return void 0;
@@ -354,36 +357,28 @@ function builtinDetail(tool, input) {
 function describeToolUse(name, input, opts) {
   if (name.startsWith(WORKSPACE_PREFIX)) {
     const tool = name.slice(WORKSPACE_PREFIX.length);
-    return {
-      kind: "tool",
-      label: WORKSPACE_LABELS[tool] ?? "Working",
-      detail: workspaceDetail(tool, input)
-    };
+    return step("tool", WORKSPACE_LABELS[tool] ?? "Working", workspaceDetail(tool, input));
   }
   if (name.startsWith(MCP_PREFIX)) {
     const rest = name.slice(MCP_PREFIX.length);
     const split = rest.indexOf("__");
     const key = split < 0 ? rest : rest.slice(0, split);
     const tool = split < 0 ? void 0 : rest.slice(split + 2);
-    return {
-      kind: "tool",
-      label: `Using ${opts?.mcpNames?.[key] ?? key}`,
-      detail: tool || void 0
-    };
+    return step("tool", `Using ${opts?.mcpNames?.[key] ?? key}`, tool);
   }
   const label = BUILTIN_LABELS[name];
   if (label)
-    return { kind: "tool", label, detail: builtinDetail(name, input) };
-  return { kind: "tool", label: "Working" };
+    return step("tool", label, builtinDetail(name, input));
+  return step("tool", "Working");
 }
 function describeAssistantText(text) {
-  return { kind: "saying", label: "Working it out", detail: firstLine(text) || void 0 };
+  return step("saying", "Working it out", firstLine(text));
 }
-function pushJobStep(progress, step, max = MAX_JOB_STEPS) {
-  const steps = [...progress?.steps ?? [], step];
+function pushJobStep(progress, step2, max = MAX_JOB_STEPS) {
+  const steps = [...progress?.steps ?? [], step2];
   return {
     seq: (progress?.seq ?? 0) + 1,
-    updatedAt: step.at,
+    updatedAt: step2.at,
     // Keep the NEWEST `max`, oldest first — a reader renders top to bottom.
     steps: steps.slice(Math.max(0, steps.length - max))
   };
@@ -739,7 +734,7 @@ function mcpUrl(config2) {
 }
 
 // src/version.ts
-var RUNNER_VERSION = true ? "0.5.6" : "0.0.0-dev";
+var RUNNER_VERSION = true ? "0.5.7" : "0.0.0-dev";
 
 // src/auth.ts
 import { signInWithCustomToken } from "firebase/auth";
@@ -1777,7 +1772,7 @@ async function runSession(input, bin, dirs) {
           if (event.type === "result") resultEvent = event;
           if (event.type === "assistant") input.log("claude: assistant turn");
           if (input.onStep) {
-            for (const step of stepsFromClaudeEvent(event, { mcpNames })) input.onStep(step);
+            for (const step2 of stepsFromClaudeEvent(event, { mcpNames })) input.onStep(step2);
           }
         } catch {
         }
@@ -2054,6 +2049,7 @@ import {
   collection as collection4,
   deleteField,
   doc as doc6,
+  getDoc as getDoc5,
   getDocs as getDocs3,
   limit as fsLimit,
   orderBy as orderBy3,
@@ -2190,16 +2186,27 @@ ${error.slice(0, 800)}
 }
 async function markTaskStopped(db, shipId, job, stoppedBy) {
   const taskRef = doc6(db, COLLECTIONS.ships, shipId, COLLECTIONS.tasks, job.taskId);
+  const who = await actorName(db, shipId, stoppedBy);
   await addDoc(collection4(taskRef, COLLECTIONS.activity), {
     author: { type: "agent", id: job.agentId },
     createdAt: Date.now(),
     kind: "comment",
-    content: `This run was stopped by ${stoppedBy} before it finished. The task is unchanged.`
+    content: `This run was stopped by ${who} before it finished. The task is unchanged.`
   });
 }
+async function actorName(db, shipId, actorId) {
+  try {
+    const snap = await getDoc5(doc6(db, COLLECTIONS.ships, shipId, COLLECTIONS.members, actorId));
+    const m = snap.data();
+    return m?.displayName || m?.email || actorId;
+  } catch {
+    return actorId;
+  }
+}
 async function markChatStopped(db, shipId, job, stoppedBy) {
   const chatRef = doc6(db, COLLECTIONS.ships, shipId, COLLECTIONS.chats, job.chatId);
-  const content = `Stopped by ${stoppedBy} before I finished. Write again to start a fresh run.`;
+  const who = await actorName(db, shipId, stoppedBy);
+  const content = `Stopped by ${who} before I finished. Write again to start a fresh run.`;
   await addDoc(collection4(chatRef, COLLECTIONS.chatMessages), {
     author: { type: "agent", id: job.agentId },
     content,
@@ -2264,16 +2271,24 @@ async function ensureChatReply(db, shipId, job, resultText) {
 // src/jobs/progress.ts
 var PROGRESS_FLUSH_MS = 3e3;
 var PROGRESS_DENIALS_BEFORE_PROBE = 2;
-function redactStep(step, secrets) {
-  const label = clampStepText(redactTranscript(step.label, secrets), MAX_STEP_LABEL);
-  if (step.detail === void 0) return { ...step, label };
-  const detail = clampStepText(redactTranscript(step.detail, secrets), MAX_STEP_DETAIL);
-  return detail ? { ...step, label, detail } : { at: step.at, kind: step.kind, label };
+function redactStep(step2, secrets) {
+  const label = clampStepText(redactTranscript(step2.label, secrets), MAX_STEP_LABEL);
+  const base = { at: step2.at, kind: step2.kind, label };
+  if (step2.detail === void 0) return base;
+  const detail = clampStepText(redactTranscript(step2.detail, secrets), MAX_STEP_DETAIL);
+  return detail ? { ...base, detail } : base;
+}
+function errorCode(error) {
+  return error?.code ?? "";
 }
 function isDenied(error) {
-  const code = error?.code ?? "";
+  const code = errorCode(error);
   return code === "permission-denied" || code.endsWith("/permission-denied");
 }
+function isUnwritable(error) {
+  const code = errorCode(error);
+  return code === "invalid-argument" || code.endsWith("/invalid-argument");
+}
 function createProgressWriter(deps) {
   const now = deps.now ?? Date.now;
   const flushMs = deps.flushMs ?? PROGRESS_FLUSH_MS;
@@ -2292,12 +2307,18 @@ function createProgressWriter(deps) {
     }
     if (closed || inFlight || pending.length === 0) return;
     const secrets = deps.secrets();
-    for (const step of pending) progress = pushJobStep(progress, redactStep(step, secrets));
+    for (const step2 of pending) progress = pushJobStep(progress, redactStep(step2, secrets));
     pending = [];
     lastWriteAt = now();
     inFlight = true;
     const snapshot = progress;
-    void deps.write(snapshot).then(() => {
+    let written;
+    try {
+      written = deps.write(snapshot);
+    } catch (e) {
+      written = Promise.reject(e);
+    }
+    void written.then(() => {
       denials = 0;
     }).catch((e) => {
       if (isDenied(e)) {
@@ -2309,9 +2330,12 @@ function createProgressWriter(deps) {
         return;
       }
       denials = 0;
+      if (isUnwritable(e)) closed = true;
       if (!loggedFailure) {
         loggedFailure = true;
-        deps.log(`progress write failed (further ones silent): ${e instanceof Error ? e.message : e}`);
+        deps.log(
+          `progress write failed${closed ? " \u2014 live steps are off for this job" : " (further ones silent)"}: ${e instanceof Error ? e.message : e}`
+        );
       }
     }).finally(() => {
       inFlight = false;
@@ -2329,9 +2353,9 @@ function createProgressWriter(deps) {
     timer.unref?.();
   }
   return {
-    push(step) {
+    push(step2) {
       if (closed) return;
-      pending.push({ ...step, at: now() });
+      pending.push({ ...step2, at: now() });
       schedule();
     },
     stop() {
@@ -2415,7 +2439,7 @@ async function startDaemon() {
     try {
       for (const shipId of [...serving]) {
         try {
-          const snap = await getDoc5(shipRunnerRef(shipId));
+          const snap = await getDoc6(shipRunnerRef(shipId));
           if (!snap.exists()) {
             forgetShipLocally(shipId, "a captain removed this machine on the Daemons page");
             continue;
@@ -2741,7 +2765,7 @@ async function startDaemon() {
       void (async () => {
         if (slot.stop) return;
         try {
-          const snap = await getDoc5(
+          const snap = await getDoc6(
             doc7(sess(shipId).fb.db, COLLECTIONS.ships, shipId, COLLECTIONS.jobs, job.id)
           );
           const fresh = snap.data();
@@ -2796,7 +2820,7 @@ async function startDaemon() {
       onDenied: () => {
         void (async () => {
           try {
-            const snap = await getDoc5(
+            const snap = await getDoc6(
               doc7(sess(shipId).fb.db, COLLECTIONS.ships, shipId, COLLECTIONS.jobs, job.id)
             );
             const fresh = snap.data();
@@ -2889,7 +2913,7 @@ async function startDaemon() {
         // §15.36. The driver emits Crew's step vocabulary; the writer coalesces, redacts and
         // persists. A throw here must never reach the driver's stdout loop, which is why `push`
         // swallows everything.
-        onStep: (step) => progress.push(step)
+        onStep: (step2) => progress.push(step2)
       });
       transcript = redactTranscript(session.transcript, knownSecrets);
       usage = session.usage;
@@ -3335,7 +3359,7 @@ function setParallel(config2, value, ship2) {
 
 // src/cli/commands/doctor.ts
 import { spawnSync as spawnSync2 } from "node:child_process";
-import { collection as collection6, doc as doc8, getDoc as getDoc6, getDocs as getDocs5 } from "firebase/firestore";
+import { collection as collection6, doc as doc8, getDoc as getDoc7, getDocs as getDocs5 } from "firebase/firestore";
 
 // src/service.ts
 import { spawnSync } from "node:child_process";
@@ -3754,7 +3778,7 @@ async function checkShips(config2) {
       continue;
     }
     try {
-      const snap = await getDoc6(
+      const snap = await getDoc7(
         doc8(fb.db, COLLECTIONS.ships, shipId, COLLECTIONS.runners, config2.runnerId)
       );
       if (!snap.exists()) {
@@ -4179,7 +4203,7 @@ async function runUninstall(options) {
 }
 
 // src/cli/commands/ship.ts
-import { doc as doc9, getDoc as getDoc7 } from "firebase/firestore";
+import { doc as doc9, getDoc as getDoc8 } from "firebase/firestore";
 async function listMyShips() {
   const config2 = requireConfig();
   const ids = Object.keys(config2.shipKeys ?? {});
@@ -4187,7 +4211,7 @@ async function listMyShips() {
     ids.map(async (id) => {
       try {
         const { fb } = await openShipSession(id);
-        const snap = await getDoc7(doc9(fb.db, COLLECTIONS.ships, id));
+        const snap = await getDoc8(doc9(fb.db, COLLECTIONS.ships, id));
         return { id, ...snap.data() };
       } catch {
         return null;
@@ -4314,7 +4338,7 @@ import {
   collection as collection7,
   doc as doc10,
   getCountFromServer as getCountFromServer2,
-  getDoc as getDoc8,
+  getDoc as getDoc9,
   getDocs as getDocs6,
   query as query5,
   where as where5
@@ -4352,7 +4376,7 @@ async function runStatus() {
       continue;
     }
     const shipRef = doc10(fb.db, COLLECTIONS.ships, shipId);
-    const mirrorSnap = await getDoc8(doc10(shipRef, COLLECTIONS.runners, config2.runnerId));
+    const mirrorSnap = await getDoc9(doc10(shipRef, COLLECTIONS.runners, config2.runnerId));
     const mirror = mirrorSnap.data();
     let queued = 0;
     try {
@@ -4363,7 +4387,7 @@ async function runStatus() {
     } catch {
       queued = 0;
     }
-    const usageSnap = await getDoc8(doc10(shipRef, COLLECTIONS.usageDaily, utcDay(Date.now())));
+    const usageSnap = await getDoc9(doc10(shipRef, COLLECTIONS.usageDaily, utcDay(Date.now())));
     const today = { ...EMPTY_USAGE_TOTALS, ...usageSnap.data()?.totals ?? {} };
     const now = Date.now();
     const limitsSnap = await getDocs6(collection7(shipRef, COLLECTIONS.engineLimits));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lumi.ai/runner",
-  "version": "0.5.6",
+  "version": "0.5.7",
   "type": "module",
   "description": "Lumi Crew runner daemon — claims jobs from your Ships and executes them as headless Claude sessions on your own machine.",
   "//name": "The ONLY package in this monorepo published to the public registry, so it is the one that does not follow the internal @lumi/crew-* convention: `@lumi` is not a scope we own, `@lumi.ai` is (the npm org). The workspace DIRECTORY stays packages/crew/runner — renaming the package is not renaming the folder.",