@thehammer/danx-dashboard-mcp 0.1.278 → 0.1.279

package/dist/listen.js

@@ -79,6 +79,11 @@ function isRecord(value) {
 function isCappedText(value) {
     return isRecord(value) && typeof value.text === "string" && typeof value.truncated === "boolean";
 }
+/** DX-4078 - hand-copied from `DESIGN_DECISION_OUTCOMES` (`src/issues/view-types.ts`): this published package cannot import danxbot source. */
+const DESIGN_OUTCOMES = ["approved", "rejected", "changes_requested"];
+function isDesignOutcome(value) {
+    return typeof value === "string" && DESIGN_OUTCOMES.includes(value);
+}
 /**
  * DX-3880 — the part `solution_answered` and `problem_answer_changed` share: the
  * decision's `outcome` and the either/or body it implies. A rejection (the
@@ -188,6 +193,20 @@ export function invalidEventReason(value) {
             if (typeof d.review_url !== "string")
                 return "detail.review_url is not a string";
             return isCappedText(d.excerpt) ? null : "detail.excerpt is not capped text";
+        // DX-4078: the operator's verdict on a design revision. An outcome this listener does not know is refused, never rendered as the bare
+        // kind line, which would name no outcome and no note and leave the agent to re-read the card. `rejected` and `changes_requested` always
+        // carry their note (the table CHECK); `approved` may carry none.
+        case "design_decided": {
+            if (typeof d.decision_id !== "number")
+                return "detail.decision_id is not a number";
+            if (typeof d.revision_no !== "number")
+                return "detail.revision_no is not a number";
+            if (!isDesignOutcome(d.outcome))
+                return 'detail.outcome is not "approved", "rejected" or "changes_requested"';
+            if (d.note === null)
+                return d.outcome === "approved" ? null : `detail.note (the ${d.outcome} reason) is null`;
+            return isCappedText(d.note) ? null : "detail.note is not null or capped text";
+        }
         default:
             return null;
     }
@@ -266,6 +285,16 @@ function describe(event) {
         // DX-4622: the agent must act on the review, so the line names who, where, and what to do next.
         case "pull_request_changes_requested":
             return `${who}: ${d.reviewer} requested changes on the pull request (${d.review_url})${d.excerpt.text === "" ? "" : `: ${quoted(d.excerpt)}`}. Address the review, push, and reply on the pull request`;
+        // DX-4078: names the outcome, the revision and the note, and what the agent does next, so it never has to re-read the card to learn the verdict.
+        case "design_decided": {
+            const note = d.note;
+            const revision = `revision ${d.revision_no} of the design`;
+            if (d.outcome === "approved")
+                return `${who} APPROVED ${revision}${note === null ? "" : ` — note: ${quoted(note)}`}. The design is closed; build from it`;
+            if (d.outcome === "rejected")
+                return `${who} REJECTED ${revision} — reason: ${quoted(note)}. The design is closed; do not continue it`;
+            return `${who} requested changes on ${revision} — ${quoted(note)}. Publish a new revision that addresses it`;
+        }
         default:
             // A kind newer than this listener still produces a line: an event the
             // agent is never told about is exactly the failure this reader prevents.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thehammer/danx-dashboard-mcp",
-  "version": "0.1.278",
+  "version": "0.1.279",
   "description": "Stdio MCP server fronting danxbot's dashboard API for agents (operator sessions and dispatched workers alike): the generic danxbot_api/danxbot_api_list/danxbot_api_spec tools reach any route the caller's credential may call, plus dedicated agent-specific tools (plan_connect, request_permission, attach_file, and danxbot_slack_reply) (DX-3465, PLN-11 R-25).",
   "license": "MIT",
   "type": "module",