hilos-agent 0.1.11 → 0.1.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hilos-agent",
-  "version": "0.1.11",
+  "version": "0.1.12",
   "description": "Run your own coding agent (Claude Code / Codex / Cursor) as an autonomous teammate in a hilos channel. Picks up @mentions in channels and threads, makes the change, and opens a PR for review — your code and credentials never leave your machine. (Approve-before-push is available via gate:true.)",
   "type": "module",
   "bin": {

package/src/handler.mjs

@@ -280,20 +280,38 @@ async function fetchContext({ channelId, tool, parentId }) {
   return { rows, transcript: rows.map((m) => `${m.author}: ${m.body}`).join("\n").slice(-6000) };
 }
 
+/**
+ * Prompt for the fast "here's my plan" ack. Carries the distilled task AND the
+ * conversation as background, so the ack is specific ("On it — I'll add X…")
+ * instead of asking for context it was already handed. (The old bug: fed only the
+ * bare mention, it replied "can't see the channel message yet.")
+ * @param {{ task?: string, transcript?: string, repoFullName?: string }} [o]
+ */
+export function planAckPrompt(o) {
+  const { task, transcript, repoFullName } = o || {};
+  const where = repoFullName ? ` in the repo ${repoFullName}` : "";
+  let p =
+    `A teammate asked you to do this${where}:\n"${(task || "").trim()}"\n\n` +
+    `Reply with ONE or TWO short sentences, first person: acknowledge, state your ` +
+    `plan, and say you'll open a PR and report back. No preamble, no lists, no code. ` +
+    `Tone: "On it — I'll add X by doing Y. I'll open a PR and report back."`;
+  const t = (transcript || "").trim();
+  if (t) {
+    p += `\n\nConversation so far (background — resolve references like "this" against it):\n${t}`;
+  }
+  return p;
+}
+
 /**
  * A 1–2 sentence "here's my plan" line for the instant ack, via the FAST chat
  * CLI. Bounded (≤45s) + best-effort: returns trimmed text, or null on
  * empty/timeout/error so the run keeps the instant template and never stalls.
  */
-async function proposePlanAck({ task, repoFullName, cfg, signal }) {
+async function proposePlanAck({ task, transcript, repoFullName, cfg, signal }) {
   const cmd = cfg.chatCmd;
   if (!cmd) return null;
   const parts = cmd.split(" ").filter(Boolean);
-  const prompt =
-    `A teammate asked you to do this in the repo ${repoFullName}:\n"${task}"\n\n` +
-    `Reply with ONE or TWO short sentences, first person: acknowledge, state your ` +
-    `plan, and say you'll open a PR and report back. No preamble, no lists, no code. ` +
-    `Tone: "On it — I'll add X by doing Y. I'll open a PR and report back."`;
+  const prompt = planAckPrompt({ task, transcript, repoFullName });
   const run = await runCli({
     cmd: parts[0],
     args: [...parts.slice(1), prompt],
@@ -518,7 +536,15 @@ export async function handleTask({ message, channelId, tool, me, caps = {} }, cf
   const ack = await tool("post_message", { channelId, parentId, body: ackText(repoFullName) });
   const ackId = ack?.messageId ?? null;
   if (ackId && caps.editMessage && cfg.chatCmd) {
-    const plan = await proposePlanAck({ task: message.body, repoFullName, cfg, signal }).catch(() => null);
+    // Feed the ack the router's distilled brief AND the conversation — not the raw
+    // mention — so it states a real plan instead of "what's the task?".
+    const plan = await proposePlanAck({
+      task: routed.task || message.body,
+      transcript: context.transcript,
+      repoFullName,
+      cfg,
+      signal,
+    }).catch(() => null);
     if (plan && !signal?.aborted) await tool("edit_message", { messageId: ackId, body: plan }).catch(() => {});
   }