hilos-agent 0.11.8 → 0.11.10

package/bin/hilos-agent.mjs

@@ -30,6 +30,7 @@ import { hookMain, hooksMain } from "../src/hook.mjs";
 import { runWebMcpCommand } from "../src/webmcp-bridge.mjs";
 import { detectVendor, fastChatCmd, webCapability } from "../src/progress-emitter.mjs";
 import { commandArgv } from "../src/argv.mjs";
+import { runWithTerminalSignals } from "../src/cli.mjs";
 
 // 1251 — the plugin writes `hook --claude --personal`, so the client flags name
 // the vendor for `hook` the same way they choose a target for `hooks install`.
@@ -318,7 +319,7 @@ async function main() {
   delete cliFlags.joinStdin;
   delete cliFlags.help;
   const cfg = resolveConfig({ flags: cliFlags, join: joinPayload });
-  await run(cfg);
+  await runWithTerminalSignals((signal) => run(cfg, { signal }));
 }
 
 main().catch((e) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hilos-agent",
-  "version": "0.11.8",
+  "version": "0.11.10",
   "description": "Run your own coding agent (Claude Code, Codex, Cursor, OpenCode, Hermes, or any command) as a teammate in a hilos room. The checkout and credentials stay local; changes go to your configured Git remote as a PR for human review, and bounded progress and reports go to hilos.",
   "type": "module",
   "bin": {

package/skills/hilos-handoff/SKILL.md

@@ -5,8 +5,15 @@ description: Hand work from this session to the room's agents — one ask, or a
 
 # Handoff
 
-Give work to the room instead of doing all of it here. Everything below is
-posted as the person running this session; they are the author of record.
+Give work to the room instead of doing all of it here. Two names are involved,
+and the difference matters: the ask is posted as the person running this
+session, so they are the author of record; the work is done by the named agent
+they chose, under that agent's own name. Never present an agent's result as
+the person's, and never let this session do the delegated work itself.
+
+Needs the hilos MCP server (`list_room_roster`, `ask_agent`, `propose_plan`).
+If those tools are not in this session, say so and point to
+https://hilos.sh/docs/share-from-your-terminal.
 
 1. Call `list_room_roster` for the room. Offer the agents it returns as the
    options, with their state, so nobody hands work to an agent already busy.

package/skills/hilos-room/SKILL.md

@@ -8,6 +8,10 @@ description: Show who is in the hilos room this session is attached to, and what
 Who is at the table right now, and what is waiting on the person running this
 session. Read-only: this skill never posts anything.
 
+Needs the hilos MCP server (`list_channels`, `list_room_roster`,
+`list_mentions`). If those tools are not in this session, say so and point to
+https://hilos.sh/docs/share-from-your-terminal; never invent a roster.
+
 1. Find the room. Use the channel this session has already posted into. If
    there is none, call `list_channels` and ask which room to read.
 2. Call `list_room_roster` with that channel id.

package/skills/hilos-share/SKILL.md

@@ -6,7 +6,12 @@ description: Post a short summary, a diff, or a link from this session into the
 # Share
 
 Put what just happened in front of the people in the room. The message is
-posted as the person running this session, under their name.
+posted as the person running this session, under their name, with a small
+"via Claude Code" (or Codex, Cursor) marker.
+
+Needs the hilos MCP server (`list_channels`, `post_message`). If those tools
+are not in this session, say so and point to
+https://hilos.sh/docs/share-from-your-terminal instead of pretending to post.
 
 1. Find the thread. Use the thread this session last posted into. If there is
    none, call `list_channels`, ask which room, and post at top level there.

package/src/cli.mjs

@@ -7,6 +7,30 @@
 
 import { spawn } from "node:child_process";
 
+/**
+ * Let the daemon finish its existing abort/cleanup path before exiting.
+ * @param {(signal: AbortSignal) => Promise<any>} task
+ * @param {import("node:events").EventEmitter & { exitCode?: number | string }} signalSource
+ */
+export async function runWithTerminalSignals(task, signalSource = process) {
+  const controller = new AbortController();
+  const interrupt = () => stop("SIGINT", 130);
+  const terminate = () => stop("SIGTERM", 143);
+  const stop = (name, code) => {
+    if (controller.signal.aborted) return;
+    signalSource.exitCode = code;
+    controller.abort(new Error(`Daemon received ${name}`));
+  };
+  signalSource.once("SIGINT", interrupt);
+  signalSource.once("SIGTERM", terminate);
+  try {
+    return await task(controller.signal);
+  } finally {
+    signalSource.removeListener("SIGINT", interrupt);
+    signalSource.removeListener("SIGTERM", terminate);
+  }
+}
+
 // ── Environment isolation ────────────────────────────────────────────────────
 // The coding/chat CLI we spawn (`claude -p`, `codex`, …) is a model with tool
 // use: it can run `env`/`printenv` and echo whatever it sees into the channel.

package/src/handler.mjs

@@ -4975,6 +4975,11 @@ export async function handleTask({ message, channelId, tool, me, caps = {}, iter
       body,
     });
     await updateChannelMarker(compactRunMarker("cancelled", branch));
+    // 1270 — a process interruption has no server-side Stop action to settle
+    // the row. Iterate claims keep their fenced recovery path in run().
+    if (runId && caps.runs && !iterateClaimId) {
+      await tool("update_run", { runId, status: "failed", reason: "daemon-stopped" }).catch(() => {});
+    }
     // Every repo-lane cancel funnels through here, so one settle covers them
     // all: a stopped run still spent tokens (0787).
     await settleRunUsage();
@@ -5254,8 +5259,19 @@ export async function handleTask({ message, channelId, tool, me, caps = {}, iter
       compactRunMarker(staged.failed ? "run-failed" : "no-changes", branch),
     );
     await settleRunUsage(); // "no changes" is not "no spend" (0787)
-    // 1182 — no turn is left in this pass: what is held is answered now.
+    // 1182 — retry direction receipts before terminal settlement drops any
+    // still-unacknowledged input. No turn is left in this pass.
     if (directions) await directions.close();
+    // A terminal progress card is presentation; it does not settle this row.
+    // Keep an existing PR at its review gate when feedback produces no diff.
+    if (runId && caps.runs && !iterateClaimId) {
+      await tool("update_run", {
+        runId,
+        status: continuingPrUrl ? "awaiting_review" : staged.failed ? "failed" : "succeeded",
+        ...(continuingPrUrl ? { prUrl: continuingPrUrl } : {}),
+        ...(!continuingPrUrl ? { reason: staged.failed ? "coding-tool-failed" : "no-changes" } : {}),
+      });
+    }
     return { status: staged.failed ? "run-failed" : "no-changes" };
   };
   if (staged.empty) return await finishEmptyTree({ afterDirection: afterCoding.advanced });