@thehammer/danx-dashboard-mcp 0.1.206 → 0.1.208

package/dist/restart-notice.js

@@ -18,8 +18,8 @@
  *     project connected a plan, or the plan is archived/deleted, or the record is
  *     stale. The plugin stays silent, as PLN-11 R-10 requires of a project that
  *     never connected a plan.
- *   - exit 1, reason on stderr: the lookup itself failed (credential, network,
- *     non-2xx, bad shape). Never silent: the hook prints a could-not-load notice.
+ *   - exit 1, reasons on stderr: NO candidate answered (credential, network,
+ *     non-2xx, bad shape, for every one). Never silent: the hook prints a could-not-load notice.
  *
  * The credential is the PREDECESSOR's own record's source, resolved through the
  * same resolver the bridge uses, so the lookup reaches the dashboard that
@@ -112,6 +112,10 @@ resolveFrom = {}) {
         return 1;
     }
     const candidates = findPredecessorCandidates(projectKeyOf(cwd), sessionId, resolveFrom).slice(0, RESTART_NOTICE_MAX_CANDIDATES);
+    // One candidate failing (its credential moved, its dashboard is down) must not hide a LATER candidate that
+    // would answer: keep going, and report failure only when no candidate answered at all.
+    const failures = [];
+    let answered = 0;
     for (const predecessor of candidates) {
         let options;
         try {
@@ -119,8 +123,8 @@ resolveFrom = {}) {
         }
         catch (err) {
             const start = err;
-            process.stderr.write(`${start.reason}: ${start.message}. Fix: ${start.fix}.\n`);
-            return 1;
+            failures.push(`${start.reason}: ${start.message}. Fix: ${start.fix}.`);
+            continue;
         }
         const output = await fetchRestartNotice({
             dashboardUrl: options.dashboardUrl,
@@ -129,13 +133,18 @@ resolveFrom = {}) {
             askingSessionId: sessionId,
         }, { fetch: (input, init) => fetch(input, init), requestTimeoutMs: RESTART_NOTICE_REQUEST_TIMEOUT_MS });
         if (!output.ok) {
-            process.stderr.write(`${output.reason}: could not ask the dashboard what the previous session of this project was working\n`);
-            return 1;
+            failures.push(`${output.reason}: could not ask the dashboard what session ${predecessor.sessionId} was working`);
+            continue;
         }
+        answered += 1;
         if (output.text !== null) {
             process.stdout.write(output.text);
             return 0;
         }
     }
+    if (answered === 0 && failures.length > 0) {
+        process.stderr.write(`no candidate answered (${failures.length} of ${candidates.length} failed):\n${failures.join("\n")}\n`);
+        return 1;
+    }
     return 0;
 }

package/dist/terms-halt.js

@@ -1,12 +1,13 @@
 const TERMS_ACCEPTANCE_PATH = "/api/account/terms-acceptance";
 // Mirrors `termsAcceptUrl("/")` in src/lib/terms-accept-path.ts (a published package cannot import it);
 // terms-gate.integration.test.ts pins the server's link to the same literal. Only meaningful for an
-// operator session: a dispatch's DANXBOT_DASHBOARD_URL is internal, and agent credentials are never terms-gated.
+// operator session: a dispatch's DANXBOT_DASHBOARD_URL is internal, and a dispatch's own agent credential is never terms-gated.
 const ACCEPT_SCREEN_PATH = "/terms/accept?return=%2F";
 /** Keep in step with `termsRefusalInstruction` in src/dashboard/route-policy.ts. */
-export function haltText(acceptUrl) {
-    return ("STOP ALL WORK NOW. The dashboard refuses your access until the person you act for accepts the " +
-        "current Terms of Use.\n\n" +
+export function haltText(acceptUrl, owner) {
+    return ("STOP ALL WORK NOW. The dashboard refuses your access until " +
+        (owner === undefined ? "the person you act for" : `${owner}, the user you act for,`) +
+        " accepts the current Terms of Use.\n\n" +
         `Give the person this link to accept them: ${acceptUrl}\n\n` +
         "Do not retry, do not look for another route, and never accept the Terms yourself. " +
         "A dispatched agent ends with danxbot_complete({status: \"failed\", summary}) naming this link, " +
@@ -22,7 +23,10 @@ export function termsHaltFor(result) {
     if (typeof body.acceptUrl !== "string") {
         throw new Error("[danx-dashboard-mcp] a terms_not_accepted refusal carried no acceptUrl");
     }
-    return haltText(body.acceptUrl);
+    if (typeof body.owner !== "string") {
+        throw new Error("[danx-dashboard-mcp] a terms_not_accepted refusal carried no owner");
+    }
+    return haltText(body.acceptUrl, body.owner);
 }
 /** The refusal for a call that would accept the Terms (never an agent's act), or null for any other call. */
 export function termsAcceptanceCallHalt(args, dashboardUrl) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@thehammer/danx-dashboard-mcp",
-  "version": "0.1.206",
+  "version": "0.1.208",
   "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, danxbot_complete, and other per-dispatch lifecycle callbacks) (DX-3465, PLN-11 R-25).",
   "license": "MIT",
   "type": "module",