hilos-agent 0.1.2 → 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hilos-agent",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "Run your own coding agent (Claude Code / Codex / Cursor) as an autonomous teammate in a hilos channel. Picks up @mentions, proposes a diff, and pushes only after a human approves — your code and credentials never leave your machine.",
   "type": "module",
   "bin": {

package/src/handler.mjs

@@ -119,6 +119,31 @@ async function applyDecision({ decision, repoPath, branch, task, cfg, tool, chan
   return { status: "timeout", branch };
 }
 
+/**
+ * Does this mention ask for a code change (→ branch/diff/PR flow), or is it
+ * chat (→ a conversational reply)? Greetings and questions are chat even when
+ * they mention code; an action verb means work. Keeps casual talk in a
+ * repo-linked channel from spawning empty branches.
+ */
+export function looksLikeCodeTask(text) {
+  const t = String(text || "")
+    .toLowerCase()
+    .replace(/@[a-z0-9-]+/g, " ")
+    .trim();
+  if (!t) return false;
+  // Greetings / questions → chat (even if they name code things).
+  if (
+    /^(hi|hey|hello|yo|sup|gm|thanks|thank you|nice|cool|ok|okay|how|what|why|when|who|where|which|is\b|are\b|do you|does|did|could you (tell|explain|show|describe)|can you (tell|explain|show|describe))/.test(
+      t,
+    )
+  ) {
+    return false;
+  }
+  const verb =
+    /\b(fix|add|implement|refactor|change|update|create|remove|delete|rename|write|build|bump|migrate|wire|patch|optimi[sz]e|debug|revert|rebase|configure|improve|replace|extract|split|move|generate|scaffold|make|set up|hook up|adjust|tweak)\b/;
+  return verb.test(t);
+}
+
 /** owner/name from a GitHub remote URL (https or ssh), or null. */
 export function normalizeRemote(url) {
   const m = String(url || "")
@@ -145,12 +170,15 @@ async function respondConversationally({ message, channelId, tool, me, cfg }) {
     `Conversation so far:\n${transcript}`;
 
   const parts = cfg.codingCmd.split(" ").filter(Boolean);
+  console.log(`  chat → running \`${cfg.codingCmd}\`…`);
   const run = spawnSync(parts[0], [...parts.slice(1), prompt], {
     encoding: "utf8",
     timeout: cfg.runTimeoutMs,
     maxBuffer: 50 * 1024 * 1024,
   });
   const reply = (run.stdout || "").trim();
+  if (run.error) console.log(`  ! ${parts[0]} failed to start: ${run.error.message}`);
+  console.log(`  chat → ${reply ? `replied (${reply.length} chars)` : "no output"}; posting`);
   await tool("post_message", {
     channelId,
     body: reply || `(my CLI returned nothing — is \`${cfg.codingCmd}\` installed and on PATH?)`,
@@ -164,9 +192,9 @@ export async function handleTask({ message, channelId, tool, me }, cfg, depsOver
 
   const { links = [] } = await tool("get_links", { channelId }).catch(() => ({ links: [] }));
   const repoLink = links.find((l) => l.repo_full_name);
-  if (!repoLink) {
-    // No repo here → respond conversationally by running YOUR CLI. This is the
-    // agent genuinely answering in chat (the CLI is the brain), not a stub.
+  // Chat (no repo linked, OR a greeting/question even in a repo channel) →
+  // reply via the CLI. Only an actual code request runs the branch/diff/PR flow.
+  if (!repoLink || !looksLikeCodeTask(message.body)) {
     await respondConversationally({ message, channelId, tool, me, cfg });
     return { status: "chat" };
   }
@@ -223,15 +251,21 @@ export async function handleTask({ message, channelId, tool, me }, cfg, depsOver
 
   const parts = cfg.codingCmd.split(" ").filter(Boolean);
   const proposeRound = async (promptText) => {
+    console.log(`  code → running \`${cfg.codingCmd}\` in ${repoPath}…`);
     const run = spawnSync(parts[0], [...parts.slice(1), promptText], {
       cwd: repoPath,
       encoding: "utf8",
       timeout: cfg.runTimeoutMs,
       maxBuffer: 50 * 1024 * 1024,
     });
+    if (run.error) console.log(`  ! ${parts[0]} failed to start: ${run.error.message}`);
     git(repoPath, ["add", "-A"]);
     const diff = git(repoPath, ["diff", "--cached"]).stdout || "";
-    if (!diff.trim()) return { empty: true };
+    if (!diff.trim()) {
+      console.log("  code → no changes produced");
+      return { empty: true };
+    }
+    console.log("  code → diff captured; posting proposal");
     const stat = parseShortstat(git(repoPath, ["diff", "--cached", "--shortstat"]).stdout);
     const { text: diffText, truncated, omittedLines } = truncateDiff(diff);
     const report = buildProposalReport({