@thehammer/danx-dashboard-mcp 0.1.18 → 0.1.20

package/dist/handlers.js

@@ -205,6 +205,111 @@ export async function issueDependency(client, args) {
         board,
     });
 }
+/**
+ * DX-1362 — targeted checklist CUD. Action-dispatched onto the
+ * `/api/issues/:id/checklists[/:cid[/items[/:iid]]]` route family so an agent
+ * mutates ONE checklist or item without the wholesale `issue_edit({checklists})`
+ * replace (which silently drops any omitted checklist + churns every item id).
+ * A missing required arg for the chosen action throws at this boundary (no
+ * round-trip). The dashboard's structured error envelope passes through verbatim.
+ */
+export async function issueChecklist(client, args) {
+    const idEnc = encodeURIComponent(args.id);
+    const board = args.board;
+    switch (args.action) {
+        case "add_list": {
+            if (typeof args.name !== "string") {
+                throw new Error("issue_checklist action=add_list requires name");
+            }
+            const body = { name: args.name };
+            if (args.items !== undefined)
+                body.items = args.items;
+            return client.request({
+                method: "POST",
+                path: `/${idEnc}/checklists`,
+                body,
+                board,
+            });
+        }
+        case "update_list": {
+            if (args.checklist_id === undefined) {
+                throw new Error("issue_checklist action=update_list requires checklist_id");
+            }
+            if (typeof args.name !== "string") {
+                throw new Error("issue_checklist action=update_list requires name");
+            }
+            return client.request({
+                method: "PATCH",
+                path: `/${idEnc}/checklists/${args.checklist_id}`,
+                body: { name: args.name },
+                board,
+            });
+        }
+        case "remove_list": {
+            if (args.checklist_id === undefined) {
+                throw new Error("issue_checklist action=remove_list requires checklist_id");
+            }
+            return client.request({
+                method: "DELETE",
+                path: `/${idEnc}/checklists/${args.checklist_id}`,
+                board,
+            });
+        }
+        case "add_item": {
+            if (args.checklist_id === undefined) {
+                throw new Error("issue_checklist action=add_item requires checklist_id");
+            }
+            if (typeof args.label !== "string") {
+                throw new Error("issue_checklist action=add_item requires label");
+            }
+            const body = { label: args.label };
+            if (args.detail !== undefined)
+                body.detail = args.detail;
+            if (args.status !== undefined)
+                body.status = args.status;
+            return client.request({
+                method: "POST",
+                path: `/${idEnc}/checklists/${args.checklist_id}/items`,
+                body,
+                board,
+            });
+        }
+        case "update_item": {
+            if (args.checklist_id === undefined) {
+                throw new Error("issue_checklist action=update_item requires checklist_id");
+            }
+            if (args.item_id === undefined) {
+                throw new Error("issue_checklist action=update_item requires item_id");
+            }
+            const body = {};
+            if (args.label !== undefined)
+                body.label = args.label;
+            if (args.detail !== undefined)
+                body.detail = args.detail;
+            if (args.status !== undefined)
+                body.status = args.status;
+            return client.request({
+                method: "PATCH",
+                path: `/${idEnc}/checklists/${args.checklist_id}/items/${args.item_id}`,
+                body,
+                board,
+            });
+        }
+        case "remove_item": {
+            if (args.checklist_id === undefined) {
+                throw new Error("issue_checklist action=remove_item requires checklist_id");
+            }
+            if (args.item_id === undefined) {
+                throw new Error("issue_checklist action=remove_item requires item_id");
+            }
+            return client.request({
+                method: "DELETE",
+                path: `/${idEnc}/checklists/${args.checklist_id}/items/${args.item_id}`,
+                board,
+            });
+        }
+    }
+}
 export async function issueRequiresHuman(client, args) {
     const idEnc = encodeURIComponent(args.id);
     const board = args.board;

package/dist/http-client.js

@@ -11,6 +11,13 @@ export class DashboardHttpClient {
             Authorization: `Bearer ${this.config.token}`,
             Accept: "application/json",
         };
+        // DX-1398 Hop 3 (outbound) — stamp the cross-process trace at the SAME
+        // single choke point that stamps Authorization, so every call (GET + every
+        // write) chains under the originating launch. Omitted when the dispatch
+        // carries no trace context (the dashboard then mints a fresh root).
+        if (this.config.traceparent) {
+            headers["traceparent"] = this.config.traceparent;
+        }
         let bodyString;
         if (args.body !== undefined) {
             headers["Content-Type"] = "application/json";

package/dist/index.js

@@ -18,6 +18,7 @@
  *   - issue_transition       POST   /api/issues/:id/transition
  *   - issue_triage           POST   /api/issues/:id/triage
  *   - issue_comment          POST/PATCH/DELETE /api/issues/:id/comments[/:cid]
+ *   - issue_checklist        POST/PATCH/DELETE /api/issues/:id/checklists[/:cid[/items[/:iid]]]
  *   - issue_dependency       POST/DELETE /api/issues/:id/dependencies[/:did]
  *   - issue_requires_human   POST/DELETE /api/issues/:id/requires-human
  *   - issue_retro            PUT    /api/issues/:id/retro
@@ -50,7 +51,7 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { z } from "zod";
 import { DashboardHttpClient } from "./http-client.js";
-import { issueAttach, issueComment, issueCreate, issueDependency, issueEdit, issueGet, issueList, issueRequiresHuman, issueRetro, issueTransition, issueTriage, } from "./handlers.js";
+import { issueAttach, issueChecklist, issueComment, issueCreate, issueDependency, issueEdit, issueGet, issueList, issueRequiresHuman, issueRetro, issueTransition, issueTriage, } from "./handlers.js";
 function readEnvOrDie(name) {
     const v = process.env[name];
     if (typeof v !== "string" || v === "") {
@@ -59,6 +60,17 @@ function readEnvOrDie(name) {
     }
     return v;
 }
+/**
+ * Read an OPTIONAL env var, treating empty string as absent. DX-1398 — the
+ * worker injects `DANXBOT_TRACEPARENT` for a traced dispatch and substitutes it
+ * to `""` for an untraced one (the optional-placeholder default), so empty
+ * MUST read as "no trace" rather than a literal value. Never exits — an
+ * untraced dispatch is valid.
+ */
+function readEnvOptional(name) {
+    const v = process.env[name];
+    return typeof v === "string" && v !== "" ? v : undefined;
+}
 // Compose the dispatch's qualified board id (`<repo>:<slug>`) from the
 // two env halves the worker injects. DANXBOT_BOARD_NAME already carries
 // the board SLUG at the spawn site (src/dispatch/core.ts sets it from
@@ -68,6 +80,10 @@ const config = {
     baseUrl: readEnvOrDie("DANXBOT_DASHBOARD_URL"),
     token: readEnvOrDie("DANXBOT_DISPATCH_TOKEN"),
     board: `${readEnvOrDie("DANX_REPO_NAME")}:${readEnvOrDie("DANXBOT_BOARD_NAME")}`,
+    // DX-1398 — OPTIONAL cross-process trace context. Present → stamped on every
+    // outbound call so the agent's card writes chain under the launch; absent
+    // (untraced dispatch) → omitted, and the dashboard mints a fresh root.
+    traceparent: readEnvOptional("DANXBOT_TRACEPARENT"),
 };
 const client = new DashboardHttpClient(config);
 const server = new McpServer({
@@ -229,6 +245,31 @@ server.tool("issue_comment", "Comment CRUD via /api/issues/:id/comments[/:cid].
     text: z.string().min(1).optional(),
     ...boardField,
 }, async (args) => jsonResult(await issueComment(client, args)));
+// ---------------- issue_checklist ----------------
+const CHECKLIST_ITEM_INPUT = z.object({
+    label: z.string().min(1),
+    detail: z.string().optional(),
+    status: z.enum(CHECKLIST_ITEM_STATUSES).optional(),
+});
+server.tool("issue_checklist", "Targeted checklist CUD via /api/issues/:id/checklists[/:cid[/items[/:iid]]] (DX-1362). Mutates ONE checklist or item WITHOUT the wholesale `issue_edit({checklists})` replace — use this for the common case (flip an item's status, add/rename a checklist, add/edit/remove an item); the wholesale path silently DROPS any checklist you omit and churns every item id (orphaning its Trello mirror), so prefer this for single-item changes. Action-dispatched: add_list (POST :id/checklists {name, items?}) — create a named checklist, optionally with initial items; update_list (PATCH :id/checklists/:cid {name}) — rename; remove_list (DELETE :id/checklists/:cid) — soft-delete the checklist (audit trail preserved); add_item (POST :id/checklists/:cid/items {label, detail?, status?}) — append an item (status defaults `incomplete`); update_item (PATCH :id/checklists/:cid/items/:iid {label?, detail?, status?}) — change ONLY the fields you pass, in place (the item keeps its id + Trello linkage; at least one field required); remove_item (DELETE :id/checklists/:cid/items/:iid) — soft-delete one item. 4-state status: incomplete|failing|passing|cancelled (terminal = passing|cancelled). checklist_id is required for every action except add_list; item_id for update_item/remove_item. Each returns the {ok,status,body} envelope; unknown card/checklist/item → 404, invalid status → 400. ADDITIVE — the wholesale `issue_edit({checklists})` path stays for bulk authoring.", {
+    id: z.string().min(1),
+    action: z.enum([
+        "add_list",
+        "update_list",
+        "remove_list",
+        "add_item",
+        "update_item",
+        "remove_item",
+    ]),
+    checklist_id: z.number().int().positive().optional(),
+    item_id: z.number().int().positive().optional(),
+    name: z.string().min(1).optional(),
+    items: z.array(CHECKLIST_ITEM_INPUT).optional(),
+    label: z.string().min(1).optional(),
+    detail: z.string().optional(),
+    status: z.enum(CHECKLIST_ITEM_STATUSES).optional(),
+    ...boardField,
+}, async (args) => jsonResult(await issueChecklist(client, args)));
 // ---------------- issue_dependency ----------------
 server.tool("issue_dependency", "Dependency CRUD via /api/issues/:id/dependencies[/:did]. action=add → POST {kind, target_id, reason} where kind ∈ {depends_on, conflict_on}. depends_on adds are CYCLE-CHECKED (BFS from target back to source — 409 if loop). Idempotent: re-adding a live triple returns the existing id. Self-loops refuse 409. action=remove → DELETE /:did. The server REQUIRES the literal reason=\"recorded_in_error\" on removal (encodes \"removal means NOT related, never satisfied\") — this MCP boundary hardcodes it, so callers do not pass reason on remove.", {
     id: z.string().min(1),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thehammer/danx-dashboard-mcp",
-  "version": "0.1.18",
+  "version": "0.1.20",
   "description": "Stdio MCP server wrapping danxbot's dashboard /api/issues/* normalized DB-backed HTTP routes for dispatched agents (DX-704 Phase 2).",
   "license": "MIT",
   "type": "module",