@mrrlin-dev/mcp 0.3.40 → 0.3.41

package/dist/bin.cjs

@@ -38955,7 +38955,7 @@ function startCodexProbe(deps = {}) {
 }
 
 // src/_generated/version.ts
-var PKG_VERSION = "0.3.40";
+var PKG_VERSION = "0.3.41";
 
 // src/api-base-url.ts
 var DEFAULT_API_BASE_URL = "http://127.0.0.1:8787";
@@ -53397,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
     }
   ];
 }
@@ -53509,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 {
           }
@@ -53526,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 {
           }
@@ -53605,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.40",
+  "version": "0.3.41",
   "type": "module",
   "bin": {
     "mrrlin-mcp": "dist/bin.cjs"
@@ -21,12 +21,12 @@
     "@types/ws": "^8.18.1",
     "esbuild": "^0.24.0",
     "tsx": "^4.22.3",
-    "@mrrlin/client": "0.0.0",
     "@mrrlin/director-e2e": "0.0.0",
     "@mrrlin/wiki": "0.0.0",
     "@mrrlin/schemas": "0.0.0",
     "@mrrlin/codex-client": "0.0.0",
-    "@mrrlin/tsconfig": "0.0.0"
+    "@mrrlin/tsconfig": "0.0.0",
+    "@mrrlin/client": "0.0.0"
   },
   "dependencies": {
     "@iarna/toml": "^2.2.5",