@gethmy/mcp 2.25.0 → 3.0.0

package/README.md

@@ -5,7 +5,7 @@ Claude Code, OpenAI Codex, Cursor, and any MCP client claim cards, report progre
 
 ## Features
 
-- **78 MCP Tools** for full board control, knowledge graph, and workflow plans
+- **77 MCP Tools** for full board control, knowledge graph, and workflow plans
 - **Global Skills** — installable in one command, served from the DB-backed [skill hub](../../docs/skills.md) with auto-update and admin-managed versioning
 - **Knowledge Graph Memory** — Phase 1 surface: hybrid retrieval (vector + lexical + RRF), session-scoped working memory, activity feed. See [docs/memory.md](../../docs/memory.md)
 - **GSD Workflow Plans** - plan/execute/verify/done lifecycle with auto card creation
@@ -89,7 +89,7 @@ If you prefer to configure manually (e.g., in Claude.ai's UI):
 1. Get an API key from [Harmony](https://gethmy.com/user/keys)
 2. In Claude.ai, add a remote MCP server with URL `https://mcp.gethmy.com/mcp`
 3. Set the Authorization header to `Bearer hmy_your_key_here`
-4. All 78 Harmony tools become available in your conversation
+4. All 77 Harmony tools become available in your conversation
 
 **Session management** is automatic - sessions have a 1-hour TTL and are created/renewed transparently.
 

package/dist/cli.js

@@ -1520,6 +1520,9 @@ function declaredGateMetricsFromAgents(agents) {
   }
   return { names, known };
 }
+// ../harmony-shared/dist/fanoutSource.js
+var FANOUT_KEY_MARKER = "harmony:fanout-item";
+var FANOUT_KEY_RE = new RegExp(`^\\[${FANOUT_KEY_MARKER}\\]:\\s*#(\\S+)\\s*$`, "m");
 // ../harmony-shared/dist/gateConfigError.js
 var GATE_CONFIG_ERROR_KEY = "configError";
 var GATE_CONFIG_ERROR_MARK = Object.freeze({ [GATE_CONFIG_ERROR_KEY]: true });
@@ -1947,8 +1950,12 @@ class HarmonyApiClient {
   async updateCard(cardId, updates) {
     return this.request("PATCH", `/cards/${cardId}`, updates);
   }
-  async claimCard(cardId, agentId) {
-    const res = await this.request("PATCH", `/cards/${cardId}`, { assignedAgentId: agentId, ifAssignedAgentNull: true });
+  async claimCard(cardId, agentId, opts) {
+    const res = await this.request("PATCH", `/cards/${cardId}`, {
+      assignedAgentId: agentId,
+      ifAssignedAgentNull: true,
+      ...opts?.requireUnassigned ? { ifAssigneeNull: true } : {}
+    });
     return { claimed: res.claimed !== false };
   }
   async moveCard(cardId, columnId, position) {
@@ -2046,9 +2053,6 @@ class HarmonyApiClient {
       expires_at: expiresAt
     });
   }
-  async classifyCard(cardId) {
-    return this.request("POST", `/cards/${cardId}/classify`);
-  }
   async createColumn(projectId, name) {
     return this.request("POST", "/columns", { projectId, name });
   }
@@ -3773,7 +3777,8 @@ function warnDeprecatedTool(oldName) {
 }
 var DEPRECATED_REMOVED_TOOLS = {
   harmony_run_playbook: "Playbooks are stage models run by the board and agent daemon as a card moves through their stages — there is no server-side macro to trigger. This tool no longer runs anything.",
-  harmony_save_card_as_playbook: "The steps_version=1 automation macro was removed. Build a stage playbook with harmony_create_playbook instead."
+  harmony_save_card_as_playbook: "The steps_version=1 automation macro was removed. Build a stage playbook with harmony_create_playbook instead.",
+  harmony_classify_card: "Card classification was retired. Run sizing happens at daemon pickup and is run-scoped, and the cards.intent / complexity_score / model_tier / classified_at columns it wrote have been dropped. The feature/bug/idea label — the only output anything read — is applied at card creation, so there is nothing left to trigger."
 };
 var _warnedRemovedTools = new Set;
 function deprecatedRemovedToolResult(name) {
@@ -4481,16 +4486,6 @@ var TOOLS = {
       required: ["target", "storagePath"]
     }
   },
-  harmony_classify_card: {
-    description: "DEPRECATED — run sizing now happens at daemon pickup and is run-scoped, so nothing reads `model_tier`, `intent` or `complexity_score` any more; this tool still writes them, but only the type label has an effect. Prefer letting card creation apply the type label. Sets `intent` (plan/think/implement/review), `complexity_score` (0-10), `model_tier` (simple/advanced/research), stamps `classified_at`, and applies the type label (feature/bug/idea). Idempotent; never touches the user-owned `model_override`.",
-    inputSchema: {
-      type: "object",
-      properties: {
-        cardId: { type: "string", description: "Card UUID" }
-      },
-      required: ["cardId"]
-    }
-  },
   harmony_share_artifact: {
     description: "Create a public, unauthenticated share link for a hosted artifact. Anyone with the link can view the rendered HTML without a Harmony account. Returns the share token and the full public URL. Only needed for a recipient OUTSIDE the workspace — for a teammate, hand over the `app_url` the upload already returned instead of exposing the document publicly.",
     inputSchema: {
@@ -5556,7 +5551,7 @@ var TOOLS = {
         },
         autoBind: {
           type: "object",
-          description: "Auto-bind rule: {priority?: number, mode?: 'all'|'any', when: [{path, op, value}]}. Conditions are evaluated against the card's labels (lowercased), intent, complexity_score and priority with the gate operators eq/neq/gte/gt/lte/lt/contains/exists; 'contains' on labels is membership. Stored even while triggerType is 'manual', so a rule can be armed later without re-authoring it."
+          description: "Auto-bind rule: {priority?: number, mode?: 'all'|'any', when: [{path, op, value}]}. Conditions are evaluated against the card's labels (lowercased) and priority with the gate operators eq/neq/gte/gt/lte/lt/contains/exists; 'contains' on labels is membership. `intent` and `complexity_score` are NOT addressable — run sizing is run-scoped and those card columns were dropped, so a rule naming either fails closed and never binds. Stored even while triggerType is 'manual', so a rule can be armed later without re-authoring it."
         },
         catalogId: {
           type: "string",
@@ -6362,11 +6357,8 @@ ${list}
       const result = await client3.addExternalLink(cardId, url, title);
       return { success: true, ...result };
     }
-    case "harmony_classify_card": {
-      const cardId = z.string().uuid().parse(args.cardId);
-      const result = await client3.classifyCard(cardId);
-      return result;
-    }
+    case "harmony_classify_card":
+      return deprecatedRemovedToolResult("harmony_classify_card");
     case "harmony_create_subtask": {
       const cardId = z.string().uuid().parse(args.cardId);
       const title = z.string().min(1).max(500).parse(args.title);

package/dist/index.js

@@ -1515,6 +1515,9 @@ function declaredGateMetricsFromAgents(agents) {
   }
   return { names, known };
 }
+// ../harmony-shared/dist/fanoutSource.js
+var FANOUT_KEY_MARKER = "harmony:fanout-item";
+var FANOUT_KEY_RE = new RegExp(`^\\[${FANOUT_KEY_MARKER}\\]:\\s*#(\\S+)\\s*$`, "m");
 // ../harmony-shared/dist/gateConfigError.js
 var GATE_CONFIG_ERROR_KEY = "configError";
 var GATE_CONFIG_ERROR_MARK = Object.freeze({ [GATE_CONFIG_ERROR_KEY]: true });
@@ -1942,8 +1945,12 @@ class HarmonyApiClient {
   async updateCard(cardId, updates) {
     return this.request("PATCH", `/cards/${cardId}`, updates);
   }
-  async claimCard(cardId, agentId) {
-    const res = await this.request("PATCH", `/cards/${cardId}`, { assignedAgentId: agentId, ifAssignedAgentNull: true });
+  async claimCard(cardId, agentId, opts) {
+    const res = await this.request("PATCH", `/cards/${cardId}`, {
+      assignedAgentId: agentId,
+      ifAssignedAgentNull: true,
+      ...opts?.requireUnassigned ? { ifAssigneeNull: true } : {}
+    });
     return { claimed: res.claimed !== false };
   }
   async moveCard(cardId, columnId, position) {
@@ -2041,9 +2048,6 @@ class HarmonyApiClient {
       expires_at: expiresAt
     });
   }
-  async classifyCard(cardId) {
-    return this.request("POST", `/cards/${cardId}/classify`);
-  }
   async createColumn(projectId, name) {
     return this.request("POST", "/columns", { projectId, name });
   }
@@ -3768,7 +3772,8 @@ function warnDeprecatedTool(oldName) {
 }
 var DEPRECATED_REMOVED_TOOLS = {
   harmony_run_playbook: "Playbooks are stage models run by the board and agent daemon as a card moves through their stages — there is no server-side macro to trigger. This tool no longer runs anything.",
-  harmony_save_card_as_playbook: "The steps_version=1 automation macro was removed. Build a stage playbook with harmony_create_playbook instead."
+  harmony_save_card_as_playbook: "The steps_version=1 automation macro was removed. Build a stage playbook with harmony_create_playbook instead.",
+  harmony_classify_card: "Card classification was retired. Run sizing happens at daemon pickup and is run-scoped, and the cards.intent / complexity_score / model_tier / classified_at columns it wrote have been dropped. The feature/bug/idea label — the only output anything read — is applied at card creation, so there is nothing left to trigger."
 };
 var _warnedRemovedTools = new Set;
 function deprecatedRemovedToolResult(name) {
@@ -4476,16 +4481,6 @@ var TOOLS = {
       required: ["target", "storagePath"]
     }
   },
-  harmony_classify_card: {
-    description: "DEPRECATED — run sizing now happens at daemon pickup and is run-scoped, so nothing reads `model_tier`, `intent` or `complexity_score` any more; this tool still writes them, but only the type label has an effect. Prefer letting card creation apply the type label. Sets `intent` (plan/think/implement/review), `complexity_score` (0-10), `model_tier` (simple/advanced/research), stamps `classified_at`, and applies the type label (feature/bug/idea). Idempotent; never touches the user-owned `model_override`.",
-    inputSchema: {
-      type: "object",
-      properties: {
-        cardId: { type: "string", description: "Card UUID" }
-      },
-      required: ["cardId"]
-    }
-  },
   harmony_share_artifact: {
     description: "Create a public, unauthenticated share link for a hosted artifact. Anyone with the link can view the rendered HTML without a Harmony account. Returns the share token and the full public URL. Only needed for a recipient OUTSIDE the workspace — for a teammate, hand over the `app_url` the upload already returned instead of exposing the document publicly.",
     inputSchema: {
@@ -5551,7 +5546,7 @@ var TOOLS = {
         },
         autoBind: {
           type: "object",
-          description: "Auto-bind rule: {priority?: number, mode?: 'all'|'any', when: [{path, op, value}]}. Conditions are evaluated against the card's labels (lowercased), intent, complexity_score and priority with the gate operators eq/neq/gte/gt/lte/lt/contains/exists; 'contains' on labels is membership. Stored even while triggerType is 'manual', so a rule can be armed later without re-authoring it."
+          description: "Auto-bind rule: {priority?: number, mode?: 'all'|'any', when: [{path, op, value}]}. Conditions are evaluated against the card's labels (lowercased) and priority with the gate operators eq/neq/gte/gt/lte/lt/contains/exists; 'contains' on labels is membership. `intent` and `complexity_score` are NOT addressable — run sizing is run-scoped and those card columns were dropped, so a rule naming either fails closed and never binds. Stored even while triggerType is 'manual', so a rule can be armed later without re-authoring it."
         },
         catalogId: {
           type: "string",
@@ -6357,11 +6352,8 @@ ${list}
       const result = await client3.addExternalLink(cardId, url, title);
       return { success: true, ...result };
     }
-    case "harmony_classify_card": {
-      const cardId = z.string().uuid().parse(args.cardId);
-      const result = await client3.classifyCard(cardId);
-      return result;
-    }
+    case "harmony_classify_card":
+      return deprecatedRemovedToolResult("harmony_classify_card");
     case "harmony_create_subtask": {
       const cardId = z.string().uuid().parse(args.cardId);
       const title = z.string().min(1).max(500).parse(args.title);

package/dist/lib/api-client.js

@@ -951,6 +951,9 @@ var TIMINGS = {
   QUERY_STALE_TIME: 1000 * 60 * 5,
   QUERY_GC_TIME: 1000 * 60 * 60 * 24
 };
+// ../harmony-shared/dist/fanoutSource.js
+var FANOUT_KEY_MARKER = "harmony:fanout-item";
+var FANOUT_KEY_RE = new RegExp(`^\\[${FANOUT_KEY_MARKER}\\]:\\s*#(\\S+)\\s*$`, "m");
 // ../harmony-shared/dist/gateConfigError.js
 var GATE_CONFIG_ERROR_KEY = "configError";
 var GATE_CONFIG_ERROR_MARK = Object.freeze({ [GATE_CONFIG_ERROR_KEY]: true });
@@ -1312,8 +1315,12 @@ class HarmonyApiClient {
   async updateCard(cardId, updates) {
     return this.request("PATCH", `/cards/${cardId}`, updates);
   }
-  async claimCard(cardId, agentId) {
-    const res = await this.request("PATCH", `/cards/${cardId}`, { assignedAgentId: agentId, ifAssignedAgentNull: true });
+  async claimCard(cardId, agentId, opts) {
+    const res = await this.request("PATCH", `/cards/${cardId}`, {
+      assignedAgentId: agentId,
+      ifAssignedAgentNull: true,
+      ...opts?.requireUnassigned ? { ifAssigneeNull: true } : {}
+    });
     return { claimed: res.claimed !== false };
   }
   async moveCard(cardId, columnId, position) {
@@ -1411,9 +1418,6 @@ class HarmonyApiClient {
       expires_at: expiresAt
     });
   }
-  async classifyCard(cardId) {
-    return this.request("POST", `/cards/${cardId}/classify`);
-  }
   async createColumn(projectId, name) {
     return this.request("POST", "/columns", { projectId, name });
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gethmy/mcp",
-  "version": "2.25.0",
+  "version": "3.0.0",
   "description": "MCP server for Harmony, the shared surface for human–agent teams — agents claim cards, report progress, and move work on your board.",
   "publishConfig": {
     "access": "public"
@@ -80,7 +80,7 @@
   },
   "devDependencies": {
     "@harmony/memory": "workspace:*",
-    "@types/bun": "^1.3.13",
+    "@types/bun": "^1.4.0",
     "@types/node": "^25.5.0",
     "typescript": "^6.0.1"
   }

package/src/api-client.ts

@@ -227,18 +227,6 @@ export type ResolveCardApiResult =
       candidates: ResolveCardCandidate[];
     };
 
-/** Result of the classify-card classifier (card #415). Any field may be null
- * if the LLM didn't return a usable value. `model_override` is never touched. */
-export interface CardClassificationResult {
-  type: "feature" | "bug" | "idea" | null;
-  intent: "plan" | "think" | "implement" | "review" | null;
-  complexity_score: number | null;
-  model_tier: "simple" | "advanced" | "research" | null;
-  classified_at: string;
-  applied_type_label_id: string | null;
-  reasoning: string | null;
-}
-
 export class HarmonyApiClient {
   private apiKey: string;
   private apiUrl: string;
@@ -724,15 +712,27 @@ export class HarmonyApiClient {
    * Compare-and-set claim of a card for a virtual agent: sets assigned_agent_id
    * only if it is currently NULL (server-side guard). Returns `{ claimed: false }`
    * when another daemon already owns it. Used by the agent-agnostic review pickup.
+   *
+   * `requireUnassigned` additionally guards `assignee_id IS NULL`, so the claim
+   * also loses to a PERSON who took the card. Self-selection (sweep, card #978)
+   * needs it: this write clears `assignee_id` as a side effect, so the agent-only
+   * guard does not merely let the daemon co-own a person's card — it lets the
+   * daemon erase their assignment. Default false keeps the review-strand claim
+   * (which reaches only cards no one holds) byte-unchanged.
    */
   async claimCard(
     cardId: string,
     agentId: string,
+    opts?: { requireUnassigned?: boolean },
   ): Promise<{ claimed: boolean }> {
     const res = await this.request<{ card?: unknown; claimed?: boolean }>(
       "PATCH",
       `/cards/${cardId}`,
-      { assignedAgentId: agentId, ifAssignedAgentNull: true },
+      {
+        assignedAgentId: agentId,
+        ifAssignedAgentNull: true,
+        ...(opts?.requireUnassigned ? { ifAssigneeNull: true } : {}),
+      },
     );
     // Success body is `{ card }` (no `claimed` key) → won. Lost-race body is
     // `{ claimed: false }`.
@@ -974,12 +974,6 @@ export class HarmonyApiClient {
     });
   }
 
-  async classifyCard(
-    cardId: string,
-  ): Promise<{ classification: CardClassificationResult }> {
-    return this.request("POST", `/cards/${cardId}/classify`);
-  }
-
   // ============ COLUMN OPERATIONS ============
 
   async createColumn(
@@ -1121,6 +1115,19 @@ export class HarmonyApiClient {
        * NULL until the terminal cost frame reported the observed model.
        */
       modelName?: string;
+      /**
+       * Which side of the daemon's pipeline opened this session (card #950):
+       * `implement` (`worker.ts`) or `review` (`review-worker.ts`). Persisted to
+       * `card_agent_context.session_kind`, which is what lets the board name the
+       * model the card's WORK ran on rather than the newest model reported —
+       * review always runs last, so without this the review model (often the
+       * bare alias `sonnet`) passed as the card's model.
+       *
+       * Omit it from an `/hmy` or other MCP session: NULL means "unattributed"
+       * and reads as "not a review" everywhere, which is the right default for a
+       * session that is doing the card's actual work.
+       */
+      sessionKind?: "implement" | "review";
       /**
        * Who is calling (Task 5, "the drivers"): `daemon` (the local agent
        * daemon), `interactive` (a human-driven `/hmy` session), or `script`

package/src/server.ts

@@ -167,6 +167,8 @@ const DEPRECATED_REMOVED_TOOLS: Record<string, string> = {
     "Playbooks are stage models run by the board and agent daemon as a card moves through their stages — there is no server-side macro to trigger. This tool no longer runs anything.",
   harmony_save_card_as_playbook:
     "The steps_version=1 automation macro was removed. Build a stage playbook with harmony_create_playbook instead.",
+  harmony_classify_card:
+    "Card classification was retired. Run sizing happens at daemon pickup and is run-scoped, and the cards.intent / complexity_score / model_tier / classified_at columns it wrote have been dropped. The feature/bug/idea label — the only output anything read — is applied at card creation, so there is nothing left to trigger.",
 };
 const _warnedRemovedTools = new Set<string>();
 function deprecatedRemovedToolResult(name: string): {
@@ -1205,17 +1207,6 @@ export const TOOLS = {
       required: ["target", "storagePath"],
     },
   },
-  harmony_classify_card: {
-    description:
-      "DEPRECATED — run sizing now happens at daemon pickup and is run-scoped, so nothing reads `model_tier`, `intent` or `complexity_score` any more; this tool still writes them, but only the type label has an effect. Prefer letting card creation apply the type label. Sets `intent` (plan/think/implement/review), `complexity_score` (0-10), `model_tier` (simple/advanced/research), stamps `classified_at`, and applies the type label (feature/bug/idea). Idempotent; never touches the user-owned `model_override`.",
-    inputSchema: {
-      type: "object",
-      properties: {
-        cardId: { type: "string", description: "Card UUID" },
-      },
-      required: ["cardId"],
-    },
-  },
   harmony_share_artifact: {
     description:
       "Create a public, unauthenticated share link for a hosted artifact. Anyone with the link can view the rendered HTML without a Harmony account. Returns the share token and the full public URL. Only needed for a recipient OUTSIDE the workspace — for a teammate, hand over the `app_url` the upload already returned instead of exposing the document publicly.",
@@ -2390,7 +2381,7 @@ export const TOOLS = {
         autoBind: {
           type: "object",
           description:
-            "Auto-bind rule: {priority?: number, mode?: 'all'|'any', when: [{path, op, value}]}. Conditions are evaluated against the card's labels (lowercased), intent, complexity_score and priority with the gate operators eq/neq/gte/gt/lte/lt/contains/exists; 'contains' on labels is membership. Stored even while triggerType is 'manual', so a rule can be armed later without re-authoring it.",
+            "Auto-bind rule: {priority?: number, mode?: 'all'|'any', when: [{path, op, value}]}. Conditions are evaluated against the card's labels (lowercased) and priority with the gate operators eq/neq/gte/gt/lte/lt/contains/exists; 'contains' on labels is membership. `intent` and `complexity_score` are NOT addressable — run sizing is run-scoped and those card columns were dropped, so a rule naming either fails closed and never binds. Stored even while triggerType is 'manual', so a rule can be armed later without re-authoring it.",
         },
         catalogId: {
           type: "string",
@@ -3628,11 +3619,11 @@ async function handleToolCall(
       return { success: true, ...result };
     }
 
-    case "harmony_classify_card": {
-      const cardId = z.string().uuid().parse(args.cardId);
-      const result = await client.classifyCard(cardId);
-      return result;
-    }
+    // Removed — dropped from the advertised TOOLS list; the case remains so an
+    // older `hmy-new` install that still calls it gets a legible notice rather
+    // than an unknown-tool error.
+    case "harmony_classify_card":
+      return deprecatedRemovedToolResult("harmony_classify_card");
 
     // Subtask operations
     case "harmony_create_subtask": {