@thehammer/danx-dashboard-mcp 0.1.125 → 0.1.126

package/dist/handlers.js

@@ -432,9 +432,15 @@ const STRUCTURAL_ACTION_FIELDS = ["id", "action", "board"];
  * always allowed) is named in one error, so a caller fixes every offending
  * field in one round trip rather than one at a time.
  */
+// A9 (code-review fix) — takes `object` rather than `Record<string, unknown>`
+// so every caller (each with its own specific `*Args` interface, no index
+// signature) passes `args` directly, with no `as unknown as Record<...>`
+// bridge cast at the call site. `Object.entries` reads the key/value pairs
+// without needing an index signature on the input type either.
 function refuseInapplicableFields(tool, mode, args, allowed) {
-    const offending = Object.keys(args)
-        .filter((key) => args[key] !== undefined && !STRUCTURAL_ACTION_FIELDS.includes(key) && !allowed.has(key))
+    const offending = Object.entries(args)
+        .filter(([key, value]) => value !== undefined && !STRUCTURAL_ACTION_FIELDS.includes(key) && !allowed.has(key))
+        .map(([key]) => key)
         .sort();
     if (offending.length > 0) {
         throw new Error(`${tool} ${mode} refuses fields that do not apply to this action: ${offending.join(", ")}`);

package/dist/index.js

@@ -634,11 +634,11 @@ const STEP_INPUT = z.lazy(() => z
         .positive()
         .optional()
         .describe("present -> this node IS an existing step (matched on this id, 400 if it does not resolve to a live " +
-        "child of this exact parent scope — a supplied id is NEVER claimed by the positional/title fallback " +
-        "below); absent -> matched against unclaimed siblings by TITLE (first unclaimed match, in order), else " +
-        "created new. An id-less MIDDLE insert recreates every later id-less sibling's row unless you send ids " +
-        "for the ones you want preserved — send ids whenever an existing step's identity matters (checked " +
-        "progress, a later single-step edit)."),
+        "child of this exact parent scope — a supplied id is NEVER claimed by the fallback below); absent -> " +
+        "id-less nodes match the first unclaimed sibling with the same title under the same parent, else " +
+        "created new. This title-based match (never positional) is what lets an id-less MIDDLE insert keep " +
+        "every later id-less sibling's id — send ids anyway whenever an existing step's identity matters " +
+        "(checked progress, a later single-step edit), or when two siblings under the same parent share a title."),
     title: z
         .string()
         .min(1)
@@ -714,8 +714,9 @@ strictTool("issue_solution", "One problem's options via /api/issues/:id/problems
     "changed fields, steps?, steps_base_hash?}: `steps` omitted leaves the stored procedure untouched, an explicit " +
     "array (including [], refused if it would leave an action's solution with zero steps) DIFFS against it — send " +
     "back an existing step's `id` (read from the card) to keep it (even while retitling/reordering it) and omit " +
-    "`id` for a genuinely new step; any existing step you don't include gets removed, and an id-less MIDDLE insert " +
-    "recreates every later id-less sibling's row unless you send ids for the ones whose identity matters. " +
+    "`id` for a genuinely new step; any existing step you don't include gets removed. An id-less node matches the " +
+    "first unclaimed sibling with the same title under the same parent (never positional), so an id-less MIDDLE " +
+    "insert keeps every later id-less sibling's id — send ids anyway whenever an existing step's identity matters. " +
     "`steps_base_hash` is REQUIRED whenever `steps` is sent — the solution's current `steps_hash` (read it off the " +
     "card first); a stale value → 409 `stale_steps` carrying the current tree, so a step added/removed since your " +
     "read is never silently dropped. NEVER resend the whole tree just to fix one word — see add_step/edit_step/" +

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thehammer/danx-dashboard-mcp",
-  "version": "0.1.125",
+  "version": "0.1.126",
   "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",