@mrrlin-dev/mcp 0.3.39 → 0.3.41

package/dist/bin.cjs

@@ -38955,7 +38955,7 @@ function startCodexProbe(deps = {}) {
 }
 
 // src/_generated/version.ts
-var PKG_VERSION = "0.3.39";
+var PKG_VERSION = "0.3.41";
 
 // src/api-base-url.ts
 var DEFAULT_API_BASE_URL = "http://127.0.0.1:8787";
@@ -39137,6 +39137,16 @@ function startSchedulerTickLoop(deps) {
 var import_node_child_process5 = require("node:child_process");
 
 // src/agent-pool-snapshot.ts
+function normalizeLastUsedAt(value) {
+  if (value === void 0 || value === null) return null;
+  if (typeof value === "number") {
+    if (!Number.isFinite(value)) return null;
+    const ms = value < 1e12 ? value * 1e3 : value;
+    const date5 = new Date(ms);
+    return Number.isNaN(date5.getTime()) ? null : date5.toISOString();
+  }
+  return value;
+}
 function buildAgentPoolSnapshot(statusRows, stats, opts) {
   if (!opts.available) {
     return {
@@ -39168,7 +39178,7 @@ function buildAgentPoolSnapshot(statusRows, stats, opts) {
       tokens_in: tokensIn,
       tokens_out: tokensOut,
       success_rate: successRate,
-      last_used_at: row.last_used_at ?? null
+      last_used_at: normalizeLastUsedAt(row.last_used_at)
     };
   });
   let totalCalls = 0;
@@ -53387,31 +53397,77 @@ async function resolveActiveRunHead(input, deps) {
 
 // src/async-tools/consensus.ts
 var import_node_crypto8 = require("node:crypto");
-function buildGateQuestionOptions(category) {
-  if (category === "scope") {
-    return [
-      {
-        answer: "Use the narrower scope only. Update the spec to keep the original limited scope and do not add the broader interpretation unless explicitly required.",
-        description: "Do only the originally requested narrower scope and leave the broader bucket out unless explicitly required.",
-        label: "Narrower"
-      },
-      {
-        answer: "Use the broader scope. Update the spec to explicitly include the broader interpretation and any required acceptance criteria.",
-        description: "Treat the broader interpretation as in scope and spell out the extra acceptance criteria it adds.",
-        label: "Broader"
-      }
-    ];
+function normalizeClarificationQuestion(question) {
+  return question.replace(/\s+/g, " ").trim().replace(/[?.!]+$/, "").trim();
+}
+function stripLeadingPromptPhrase(part) {
+  return part.replace(
+    /^(?:should|could|would|can|do|does|is|are)\s+(?:this|we|it|the spec|the plan|the task)\s+/i,
+    ""
+  ).trim();
+}
+function stripLeadVerb(part) {
+  return part.replace(/^(?:cover|include|support|use|keep|allow|propose|add|handle)\s+/i, "").trim();
+}
+function isAbstractOnlyChoice(part) {
+  const normalized = stripLeadVerb(stripLeadingPromptPhrase(part)).replace(/^(?:the|a|an)\s+/i, "").trim().toLowerCase();
+  return /^(?:broader|narrower|safer(?: default)?|stronger(?: interpretation)?)$/.test(normalized);
+}
+function sentenceCase(text) {
+  return text ? text.charAt(0).toUpperCase() + text.slice(1) : text;
+}
+function deriveGateChoice(question) {
+  const normalized = normalizeClarificationQuestion(question);
+  if (!normalized || normalized.length > 180 || /[;:]/.test(normalized)) return null;
+  const matches = [...normalized.matchAll(/\sor\s/gi)];
+  if (matches.length !== 1) return null;
+  const match = matches[0];
+  if (!match) return null;
+  const splitAt = match.index ?? -1;
+  if (splitAt <= 0) return null;
+  let left = stripLeadingPromptPhrase(normalized.slice(0, splitAt).trim());
+  let right = stripLeadingPromptPhrase(normalized.slice(splitAt + match[0].length).trim());
+  if (!left || !right || /\sor\s/i.test(left) || /\sor\s/i.test(right)) return null;
+  if (isAbstractOnlyChoice(left) || isAbstractOnlyChoice(right)) return null;
+  if (/^only\s+/i.test(left) && /^also\s+/i.test(right)) {
+    const leftScope = stripLeadVerb(left.replace(/^only\s+/i, "").trim());
+    const rightScope = stripLeadVerb(right.replace(/^also\s+/i, "").trim());
+    if (!leftScope || !rightScope) return null;
+    return {
+      leftLabel: `Only ${leftScope}`,
+      leftDescription: `Keep the scope limited to ${leftScope} only.`,
+      leftAnswer: `Keep the scope limited to ${leftScope} only. Update the spec so that limit is explicit.`,
+      rightLabel: `${sentenceCase(leftScope)} + ${rightScope}`,
+      rightDescription: `Expand the scope to cover ${leftScope} and ${rightScope}.`,
+      rightAnswer: `Expand the scope to cover ${leftScope} and ${rightScope}. Update the spec so both are explicitly in scope.`
+    };
   }
+  left = sentenceCase(left);
+  right = sentenceCase(right);
+  if (!left || !right) return null;
+  return {
+    leftLabel: left,
+    leftDescription: `Choose the first direction: ${left}.`,
+    leftAnswer: `Choose ${left}. Update the spec so that direction is explicit.`,
+    rightLabel: right,
+    rightDescription: `Choose the second direction: ${right}.`,
+    rightAnswer: `Choose ${right}. Update the spec so that direction is explicit.`
+  };
+}
+function buildGateQuestionOptions(question, category) {
+  if (category !== "scope" && category !== "ambiguity") return null;
+  const derived = deriveGateChoice(question);
+  if (!derived) return null;
   return [
     {
-      answer: "Use the safer/default interpretation. Update the spec to document that assumption explicitly.",
-      description: "Choose the lower-risk default interpretation and record it explicitly so implementation is unblocked.",
-      label: "Safer default"
+      answer: derived.leftAnswer,
+      description: derived.leftDescription,
+      label: derived.leftLabel
     },
     {
-      answer: "Use the broader/stronger interpretation. Update the spec to make that interpretation explicit.",
-      description: "Commit to the stronger interpretation instead of a conservative default, and update the spec to match.",
-      label: "Stronger interpretation"
+      answer: derived.rightAnswer,
+      description: derived.rightDescription,
+      label: derived.rightLabel
     }
   ];
 }
@@ -53499,13 +53555,14 @@ async function run(ctx, input) {
           const scopeId = taskId;
           const id = `II-${scopeId}-gate-${(0, import_node_crypto8.createHash)("sha256").update(clarification.question).digest("hex").slice(0, 10)}`;
           try {
+            const questionOptions = buildGateQuestionOptions(clarification.question, clarification.category);
             await ctx.client.createInboxItem(input.projectSlug, {
               id,
               kind: "question",
               taskId: scopeId,
               body: clarification.question,
               origin: "gate",
-              questionOptions: buildGateQuestionOptions(clarification.category)
+              ...questionOptions ? { questionOptions } : {}
             });
           } catch {
           }
@@ -53516,13 +53573,14 @@ async function run(ctx, input) {
           const scopeId = planId;
           const id = `II-${scopeId}-gate-${(0, import_node_crypto8.createHash)("sha256").update(clarification.question).digest("hex").slice(0, 10)}`;
           try {
+            const questionOptions = buildGateQuestionOptions(clarification.question, clarification.category);
             await ctx.client.createInboxItem(input.projectSlug, {
               id,
               kind: "question",
               planId: scopeId,
               body: clarification.question,
               origin: "gate",
-              questionOptions: buildGateQuestionOptions(clarification.category)
+              ...questionOptions ? { questionOptions } : {}
             });
           } catch {
           }
@@ -53595,7 +53653,7 @@ function onComplete2(outcome, input) {
   return {
     kind: "handoff",
     ...taskId ? { taskId } : {},
-    body: `Consensus could not complete for Specs/${input.pageId}${taskRef}: ${outcome.error} Needs operator attention.`
+    body: `The spec review stopped for Specs/${input.pageId}${taskRef}. ${outcome.error} Review the item, consult the Director if needed, then continue in Codex or handle it manually.`
   };
 }
 var consensusDescriptor = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrrlin-dev/mcp",
-  "version": "0.3.39",
+  "version": "0.3.41",
   "type": "module",
   "bin": {
     "mrrlin-mcp": "dist/bin.cjs"
@@ -22,11 +22,11 @@
     "esbuild": "^0.24.0",
     "tsx": "^4.22.3",
     "@mrrlin/director-e2e": "0.0.0",
+    "@mrrlin/wiki": "0.0.0",
     "@mrrlin/schemas": "0.0.0",
-    "@mrrlin/tsconfig": "0.0.0",
     "@mrrlin/codex-client": "0.0.0",
-    "@mrrlin/client": "0.0.0",
-    "@mrrlin/wiki": "0.0.0"
+    "@mrrlin/tsconfig": "0.0.0",
+    "@mrrlin/client": "0.0.0"
   },
   "dependencies": {
     "@iarna/toml": "^2.2.5",