hilos-agent 0.11.17 → 0.11.18

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hilos-agent",
-  "version": "0.11.17",
+  "version": "0.11.18",
   "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/src/config.mjs

@@ -181,8 +181,9 @@ const DEFAULTS = {
   // on timeout we post an honest "taking longer than expected" line.
   chatTimeoutMs: 90000,
   // Work queue: mentions run one at a time — parallel CLI runs on one checkout
-  // would collide on git state. queueAcks posts "queued behind the current task"
-  // when a mention lands during a run; set false to keep it quiet.
+  // would collide on git state. queueAcks posts a short receipt when a mention
+  // lands during a run (replaced by the reply once it starts, 1354); set false
+  // to keep it quiet.
   queueAcks: true,
   runTimeoutMs: 600000,
   decisionTimeoutMs: 1800000,

package/src/handler.mjs

@@ -1509,7 +1509,7 @@ function agentPreamble(workspaceMemory, cfg) {
  * (threads are where conversations live), else the channel tail. Returns the raw
  * rows (for intent routing) and a transcript string (for the prompt).
  */
-async function fetchContext({ channelId, tool, parentId }) {
+async function fetchContext({ channelId, tool, parentId, excludeIds = [] }) {
   let rows = [];
   if (parentId) {
     const t = await tool("get_thread", { parentId }).catch(() => null);
@@ -1520,6 +1520,10 @@ async function fetchContext({ channelId, tool, parentId }) {
     }));
     rows = messages;
   }
+  // 1354 — this agent's own queue receipt for THIS mention is plumbing, not
+  // conversation: read back as context it became "I'm waiting on a current
+  // task", and the model repeated it as fact.
+  if (excludeIds.length) rows = rows.filter((m) => !excludeIds.includes(m?.id));
   // 0779: the rows carry resolved `attachments`, and the body carries the
   // unresolvable `![name](attachment:<uuid>)` pill token. Flatten the token and
   // name the file, so the daemon's chat replies and routing decisions stop
@@ -1617,7 +1621,8 @@ async function respondConversationally({
   pathExistsFn = existsSync,
 }) {
   const implicitDm = message?.implicit === "dm";
-  const { transcript } = context || (await fetchContext({ channelId, tool, parentId }));
+  const { transcript } =
+    context || (await fetchContext({ channelId, tool, parentId, excludeIds: contextExclusions(message) }));
   // 1249 — the room can ask agents to answer in a thread instead of in the
   // room. The server owns the setting and marks the mention; the daemon only
   // has to post under the message it names. Other mentions retain their
@@ -1685,7 +1690,13 @@ async function respondConversationally({
   // If the reply is slow, post ONE "still thinking…" ping and then edit it into
   // the answer (single message, no dead air). Needs edit_message; otherwise we
   // just post the reply when it's ready. A fast reply never trips the timer.
-  let thinkingId = null;
+  // 1354 — a mention that waited in the queue already has such a line: the
+  // queue receipt, posted where the reply goes. Reuse it, so the reply lands
+  // in place of "picking this up now" instead of under it.
+  let thinkingId =
+    testCapabilities.editMessage && message?.queueAckId && replyParentId === (parentId ?? null)
+      ? message.queueAckId
+      : null;
   let beatStopped = false;
   let beatBusy = false;
   const beatMs = Math.min(cfg.chatTimeoutMs || 90000, 30000);
@@ -1768,6 +1779,11 @@ async function respondConversationally({
   await deliver(body);
 }
 
+/** Rows a mention's context must not carry: its own queue receipt (1354). */
+function contextExclusions(message) {
+  return message?.queueAckId ? [message.queueAckId] : [];
+}
+
 /** True when this mention is a request for THIS agent to review a PR (0286/0288). */
 export function isReviewRequest(message) {
   return (
@@ -2779,7 +2795,7 @@ export async function handleTask({ message, channelId, tool, me, testCapabilitie
   // route AND (for a code run) handed to the coding prompt. Context is always
   // crucial: a reply like "yeah, do it" only means something against what was
   // just said.
-  const context = await fetchContext({ channelId, tool, parentId });
+  const context = await fetchContext({ channelId, tool, parentId, excludeIds: contextExclusions(message) });
   const mode = agentMode(message);
 
   // REVIEW EXECUTION (0288): a review-request routes to the read-only reviewer

package/src/run.mjs

@@ -24,6 +24,11 @@ import {
 
 const MAX_IMMEDIATE_SEQUENCE_PAGES = 8;
 
+/** The queue receipt a mention gets while another job runs (1354). */
+export const QUEUE_ACK_BODY = "Got it — I'll pick this up as soon as I finish what I'm on.";
+/** What that receipt becomes the moment its job starts (1354). */
+export const QUEUE_ACK_STARTED_BODY = "Picking this up now.";
+
 // 1290: the hilos.sh contract. Missing-tool paths exist only for injected tests;
 // tools/list remains the token's prompt inventory and the recall scope check.
 const HILOS_DAEMON_TOOLS = Object.freeze({
@@ -519,6 +524,15 @@ export async function run(
     const withImages = images.files.length
       ? { ...message, imageDir: images.dir, images: images.files }
       : message;
+    // 1354 — retire the queue receipt as the job starts. The chat path then
+    // edits this same message into the reply (one line, in place); a code
+    // run leaves it as this truthful "starting" line under its own cards.
+    if (message.queueAckId && testCapabilities.editMessage) {
+      await taskTool("edit_message", {
+        messageId: message.queueAckId,
+        body: QUEUE_ACK_STARTED_BODY,
+      }).catch(() => {});
+    }
     // 0782 — a person pressed Stop on this run's live card. The server already
     // settled the run row (status canceled); this end says so in the room and
     // tears the work down through the SAME cancel path a chat "stop" uses, so a
@@ -660,11 +674,20 @@ export async function run(
     // agent, so "Got it — queued" over a line that may deserve no reply at all
     // is exactly the noise the implicit lane promises not to make.
     if (liveCfg.queueAcks && !r.startedImmediately && !m.deployRequest && !m.implicit) {
-      await tool("post_message", {
+      // 1354 — the ack is a queue receipt, not a status. It used to read
+      // "queued behind the current task", and every model in the room (this
+      // agent's own included, since the ack sits in its transcript) took the
+      // "current task" literally: Ada told Hilo she was "waiting for current
+      // task to clear" for an hour over a queue that had drained in seconds,
+      // and the work never started. The receipt now says what is true, its id
+      // rides on the job, and safeHandle replaces it the moment the job
+      // starts, so nothing stale is left for anyone to reason about.
+      const ack = await tool("post_message", {
         channelId,
         parentId: m.parentId ?? null,
-        body: "Got it — queued behind the current task.",
-      }).catch(() => {});
+        body: QUEUE_ACK_BODY,
+      }).catch(() => null);
+      if (ack?.messageId) m.queueAckId = ack.messageId;
     }
     return { accepted: true, settled: false };
   }
@@ -837,7 +860,7 @@ export async function run(
         await tool("post_message", {
           channelId: bridge.binding.channelId,
           parentId: bridge.binding.threadRootId,
-          body: "Got it — this will continue in the same local session after the current task.",
+          body: "Got it — this continues in the same local session as soon as I finish what I'm on.",
         }).catch(() => {});
       }
     }