@davesheffer/hunch 1.47.1 → 1.47.2

package/dist/cli/index.js

@@ -23,7 +23,8 @@ import { Command } from "commander";
 import { hunchPaths, hunchPathsForDir, findRoot, toPosixTarget, isDir, realpathNorm, repoRelativeTarget } from "../core/paths.js";
 import { writeFileAtomic } from "../core/io.js";
 import { boundedSpawnSync } from "../core/boundedSpawn.js";
-import { looksLikeCorrection, CORRECTION_NUDGE } from "../core/correction.js";
+import { looksLikeCorrection, CORRECTION_NUDGE, stripHostBlocks } from "../core/correction.js";
+import { roleHasHunchTools } from "../core/subagentrole.js";
 import { HUNCH_VERSION } from "../core/version.js";
 import { registerIntegrationCommands } from "./integrations.js";
 import { registerTaskReportCommands } from "./taskReport.js";
@@ -5456,9 +5457,11 @@ program
                     return flat.length > max ? `${flat.slice(0, max - 1).trimEnd()}…` : flat;
                 };
                 const type = (evt.agent_type ?? "").toLowerCase();
+                // A role whose tool list leaves out Hunch's MCP tools is not told to call them.
+                const tools = roleHasHunchTools(root, evt.agent_type ?? "");
                 const L = [];
                 const served = [];
-                const route = routedCwd
+                const route = routedCwd && tools
                     ? `Worktree routing: call hunch_context first with cwd: ${JSON.stringify(routedCwd)}, and pass the same cwd to hunch_task, hunch_report, and every Hunch capture/write call in this delegated task.`
                     : null;
                 const activeProvider = provider;
@@ -5478,8 +5481,9 @@ program
                         served.push({ kind: "components", record_id: c.id, token_cost: Math.ceil([...line].length / 4) });
                     }
                     if (components.length > 12)
-                        L.push(`…and ${components.length - 12} more — hunch_structure() for the full map.`);
-                    L.push("Orient: hunch_structure(target) · hunch_why(target) · hunch_context(task).");
+                        L.push(`…and ${components.length - 12} more${tools ? " — hunch_structure() for the full map" : ""}.`);
+                    if (tools)
+                        L.push("Orient: hunch_structure(target) · hunch_why(target) · hunch_context(task).");
                 }
                 else if (/plan|architect|design/.test(type)) {
                     // A plan drafted blind re-proposes what the graph already rejected.
@@ -5496,7 +5500,8 @@ program
                         L.push(line);
                         served.push({ kind: "decisions", record_id: d.id, token_cost: Math.ceil([...line].length / 4) });
                     }
-                    L.push("Before finalizing a plan: hunch_why(target) · hunch_current_decision(topic) · hunch_check_constraints(scope).");
+                    if (tools)
+                        L.push("Before finalizing a plan: hunch_why(target) · hunch_current_decision(topic) · hunch_check_constraints(scope).");
                 }
                 else {
                     const sevRank = { blocking: 0, warning: 1, advisory: 2 };
@@ -5514,8 +5519,9 @@ program
                         served.push({ kind: "constraints", record_id: c.id, token_cost: Math.ceil([...line].length / 4) });
                     }
                     if (constraints.length > 8)
-                        L.push(`…and ${constraints.length - 8} more — hunch_check_constraints(scope) for your files.`);
-                    L.push("Before editing: hunch_check_constraints(scope) · hunch_why(target). Orient: hunch_context(task).");
+                        L.push(`…and ${constraints.length - 8} more${tools ? " — hunch_check_constraints(scope) for your files" : "; the edit hook shows the ones your files touch"}.`);
+                    if (tools)
+                        L.push("Before editing: hunch_check_constraints(scope) · hunch_why(target). Orient: hunch_context(task).");
                 }
                 if (route)
                     L.push(route);
@@ -7768,19 +7774,21 @@ function promptTaskSelection(root, store, provider, evt) {
     const taskId = hookReportTaskId(root, provider, evt);
     if (!taskId)
         return "";
+    // Host-injected blocks (reminders, browser instructions) are not the task the human named.
+    const humanText = stripHostBlocks(evt.prompt ?? "");
     const selection = selectForTask(liveSelectionRecords({
         decisions: store.recs("decisions"),
         bugs: store.recs("bugs"),
         constraints: store.recs("constraints"),
         findings: store.recs("findings"),
-    }), evt.prompt ?? "", { root, pathExists: (p) => existsSync(join(root, p)) });
+    }), humanText, { root, pathExists: (p) => existsSync(join(root, p)) });
     // Same task id again (a notification alias) keeps its own earlier selection; a
     // new task that CONTINUES the session's previous one inherits that task's
     // selection only on a bare follow-up ("continue", "go on"), so it is not left
     // ungrounded. A substantive prompt names its own task: a same-session prompt
     // inside the continuation window must not hide memory behind an old selection.
     let previous = loadTaskSelection(taskId);
-    if (!previous && isBareFollowUp(evt.prompt ?? "")) {
+    if (!previous && isBareFollowUp(humanText)) {
         try {
             const continues = readTaskReport(root, taskId).task.continues;
             previous = continues ? loadTaskSelection(continues) : null;

package/dist/core/correction.js

@@ -30,11 +30,12 @@ const CORRECTION_PATTERNS = [
     /\b(not like that|don'?t do (that|this)( again)?|stop doing (that|this))\b/i,
 ];
 /** Host-generated blocks a prompt can carry: a background task's completion
- *  (`<task-notification>`, which also wraps a subagent's hand-back report) and
- *  injected context (`<system-reminder>`, hook output). Their words are not the
+ *  (`<task-notification>`, which also wraps a subagent's hand-back report),
+ *  injected context (`<system-reminder>`, hook output) and the browser
+ *  extension's standing instructions (`<browser_instruction>`). Their words are not the
  *  human's, so "never use X" quoted from a constraint is not a correction. A tag
  *  left unclosed drops only the rest of its own line, never the human's text below. */
-const HOST_TAGS = "task-notification|system-reminder|local-command-stdout|local-command-stderr|command-message|command-name|command-args";
+const HOST_TAGS = "task-notification|system-reminder|local-command-stdout|local-command-stderr|command-message|command-name|command-args|browser_instruction";
 const HOST_BLOCK = new RegExp(`<(${HOST_TAGS})>[\\s\\S]*?<\\/\\1>`, "gi");
 const HOST_OPEN_LINE = new RegExp(`<(?:${HOST_TAGS})>[^\\n]*`, "gi");
 /** The prompt with host-generated blocks removed: what the human actually typed. */

package/dist/core/subagentrole.d.ts

@@ -0,0 +1,6 @@
+/** Whether a delegated agent can call Hunch's MCP tools. A named role defined in
+ *  `.claude/agents/<type>.md` (project first, then user) with a `tools:` list that
+ *  names no `mcp__hunch` tool and no wildcard cannot: telling it to call
+ *  hunch_why only costs it a failed lookup. Built-in and unknown types inherit
+ *  the parent's tools. */
+export declare function roleHasHunchTools(root: string, agentType: string, home?: string): boolean;

package/dist/core/subagentrole.js

@@ -0,0 +1,32 @@
+import { readFileSync } from "node:fs";
+import { homedir } from "node:os";
+import { join } from "node:path";
+/** Whether a delegated agent can call Hunch's MCP tools. A named role defined in
+ *  `.claude/agents/<type>.md` (project first, then user) with a `tools:` list that
+ *  names no `mcp__hunch` tool and no wildcard cannot: telling it to call
+ *  hunch_why only costs it a failed lookup. Built-in and unknown types inherit
+ *  the parent's tools. */
+export function roleHasHunchTools(root, agentType, home = homedir()) {
+    const type = agentType.trim();
+    if (!type || !/^[\w.-]+$/.test(type))
+        return true;
+    for (const dir of [join(root, ".claude", "agents"), join(home, ".claude", "agents")]) {
+        let text;
+        try {
+            text = readFileSync(join(dir, `${type}.md`), "utf8");
+        }
+        catch {
+            continue;
+        }
+        const front = /^?---\r?\n([\s\S]*?)\r?\n---/.exec(text)?.[1];
+        if (!front)
+            return true;
+        // A block list continues on indented "- x" lines; a flow list runs to its "]".
+        const tools = /^tools:[ \t]*(\[[^\]]*\]?|.*(?:\r?\n[ \t]+-.*)*)/m.exec(front)?.[1];
+        if (tools === undefined || !tools.trim())
+            return true;
+        return /\*|mcp__hunch/.test(tools);
+    }
+    return true;
+}
+//# sourceMappingURL=subagentrole.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@davesheffer/hunch",
-  "version": "1.47.1",
+  "version": "1.47.2",
   "mcpName": "io.github.davesheffer/hunch",
   "license": "Apache-2.0",
   "author": "Dave Sheffer <dave.sheffer1@gmail.com>",

package/server.json

@@ -7,13 +7,13 @@
     "source": "github"
   },
   "websiteUrl": "https://www.hunchmemory.com",
-  "version": "1.47.1",
+  "version": "1.47.2",
   "packages": [
     {
       "registryType": "npm",
       "registryBaseUrl": "https://registry.npmjs.org",
       "identifier": "@davesheffer/hunch",
-      "version": "1.47.1",
+      "version": "1.47.2",
       "runtimeHint": "npx",
       "packageArguments": [
         {