@mrrlin-dev/mcp 0.4.1 → 0.4.2

package/dist/bin.cjs

@@ -38187,6 +38187,9 @@ var mrrlinInboxQuestionOptionsSchema = external_exports.array(mrrlinInboxQuestio
     });
   }
 });
+var REVIEW_AGAIN_ANSWER = "Review the current spec again without changing it.";
+var EDIT_SPEC_ANSWER = "Hold the task until the spec is edited by hand.";
+var PROCEED_ANYWAY_ANSWER = "Proceed without agreement; record the unsettled points on the spec.";
 var mrrlinInboxItemSchema = external_exports.object({
   actionPayload: mrrlinInboxItemActionPayloadSchema.nullable(),
   answer: external_exports.string().nullable(),
@@ -40187,7 +40190,7 @@ function startCodexProbe(deps = {}) {
 }
 
 // src/_generated/version.ts
-var PKG_VERSION = "0.4.1";
+var PKG_VERSION = "0.4.2";
 
 // src/api-base-url-migrate.ts
 var import_node_fs2 = __toESM(require("node:fs"), 1);
@@ -41312,7 +41315,7 @@ async function runConsensus(input) {
   let decision;
   for (let round = 1; round <= effectiveRounds; round++) {
     if (calls + input.heads.length > input.maxHeadCalls) {
-      return { outcome: "CONSENSUS_BLOCKED", rounds: round - 1, finalArtifact: artifact, history, ...decision ? { escalation: decision } : {} };
+      return { outcome: "CONSENSUS_BLOCKED", rounds: round - 1, finalArtifact: artifact, history, unresolved: history[history.length - 1]?.accepted ?? [], ...decision ? { escalation: decision } : {} };
     }
     const dismissedBlock = dismissedFwd.length ? `PREVIOUSLY DISMISSED (do not re-raise unless new information):
 ${dismissedFwd.slice(-20).join("\n")}` : "";
@@ -41334,7 +41337,7 @@ ${dismissedFwd.slice(-20).join("\n")}` : "";
       (r) => r.res.ok
     );
     if (responding.length === 0) {
-      return { outcome: "CONSENSUS_UNAVAILABLE", rounds: round, finalArtifact: artifact, history };
+      return { outcome: "CONSENSUS_UNAVAILABLE", rounds: round, finalArtifact: artifact, history, unresolved: [] };
     }
     const headOutcomes = input.heads.map((persona) => ({
       persona,
@@ -41360,7 +41363,7 @@ ${dismissedFwd.slice(-20).join("\n")}` : "";
     let adj;
     if (issues.length > 0) {
       if (calls + 1 > input.maxHeadCalls) {
-        return { outcome: "CONSENSUS_BLOCKED", rounds: round, finalArtifact: artifact, history, ...decision ? { escalation: decision } : {} };
+        return { outcome: "CONSENSUS_BLOCKED", rounds: round, finalArtifact: artifact, history, unresolved: history[history.length - 1]?.accepted ?? [], ...decision ? { escalation: decision } : {} };
       }
       calls++;
       try {
@@ -41374,7 +41377,7 @@ ${dismissedFwd.slice(-20).join("\n")}` : "";
     history.push({ round, accepted: adj.accepted, dismissed: adj.dismissed, headOutcomes });
     adj.dismissed.forEach((d) => dismissedFwd.push(`- ${d.id}: ${d.reason}`));
     if (adj.accepted.length === 0) {
-      return { outcome: "CONVERGED", rounds: round, finalArtifact: artifact, history, ...decision ? { escalation: decision } : {} };
+      return { outcome: "CONVERGED", rounds: round, finalArtifact: artifact, history, unresolved: [], ...decision ? { escalation: decision } : {} };
     }
     if (round === effectiveRounds) {
       if (!decision) {
@@ -41384,12 +41387,12 @@ ${dismissedFwd.slice(-20).join("\n")}` : "";
         escalated = true;
         effectiveRounds = Math.min(effectiveRounds + decision.grantedRounds, input.rounds * 2);
       } else {
-        return { outcome: "CONSENSUS_BLOCKED", rounds: round, finalArtifact: artifact, history, ...decision ? { escalation: decision } : {} };
+        return { outcome: "CONSENSUS_BLOCKED", rounds: round, finalArtifact: artifact, history, unresolved: history[history.length - 1]?.accepted ?? [], ...decision ? { escalation: decision } : {} };
       }
     }
     artifact = await input.revise(artifact, adj.accepted);
   }
-  return { outcome: "CONSENSUS_BLOCKED", rounds: effectiveRounds, finalArtifact: artifact, history };
+  return { outcome: "CONSENSUS_BLOCKED", rounds: effectiveRounds, finalArtifact: artifact, history, unresolved: history[history.length - 1]?.accepted ?? [] };
 }
 
 // src/consensus/spec-identity.ts
@@ -41688,7 +41691,8 @@ async function runSpecGate(_target, deps, opts) {
       outcome: "CONSENSUS_BLOCKED",
       ...keptClarifications.length > 0 ? { operatorClarifications: keptClarifications } : {},
       ...droppedByCap.length > 0 ? { droppedByCap } : {},
-      ...result.escalation ? { escalation: result.escalation } : {}
+      ...result.escalation ? { escalation: result.escalation } : {},
+      unresolved: result.unresolved
     };
   }
   return { outcome: result.outcome };
@@ -57470,6 +57474,44 @@ function humanizeClarificationBody(clarification) {
   const context = clarification.rawQuestion.replace(/\b(?:parseable|SPEC_IDENTITY_MISMATCH|machine-usable)\b/gi, "").replace(/\s+/g, " ").trim();
   return context ? `${prefix} ${context}` : prefix;
 }
+var BLOCK_DECISION_OPENER = "The reviewers could not agree on this spec";
+var BLOCK_BODY_TRAILER = " How should I proceed?";
+function renderUnresolvedPoints(unresolved) {
+  return unresolved.map(
+    (issue2) => issue2.text.replace(/\b(?:parseable|SPEC_IDENTITY_MISMATCH|machine-usable)\b/gi, "").replace(/\s+/g, " ").trim()
+  ).join("; ");
+}
+function renderBlockDecisionBody(taskRef, unresolved) {
+  if (unresolved.length === 0) {
+    return `${BLOCK_DECISION_OPENER} (task ${taskRef}), and their objections could not be read back.${BLOCK_BODY_TRAILER}`;
+  }
+  return `${BLOCK_DECISION_OPENER} (task ${taskRef}). Still unsettled: ${renderUnresolvedPoints(unresolved)}.${BLOCK_BODY_TRAILER}`;
+}
+function buildBlockDecisionOptions(diagnosis, reviewAgainAvailable) {
+  const recommendReviewAgain = reviewAgainAvailable && (diagnosis === "INFRA" || diagnosis === "SUBSTANCE");
+  const recommendEditSpec = !reviewAgainAvailable || diagnosis === "AMBIGUITY";
+  const options = [];
+  if (reviewAgainAvailable) {
+    options.push({
+      label: "Review again",
+      description: "Send the current spec back for another review.",
+      answer: REVIEW_AGAIN_ANSWER,
+      ...recommendReviewAgain ? { recommended: true } : {}
+    });
+  }
+  options.push({
+    label: "Edit spec",
+    description: "Open the spec and change it by hand; the review restarts on save.",
+    answer: EDIT_SPEC_ANSWER,
+    ...recommendEditSpec ? { recommended: true } : {}
+  });
+  options.push({
+    label: "Proceed anyway",
+    description: "Start the work with these points unsettled and recorded.",
+    answer: PROCEED_ANYWAY_ANSWER
+  });
+  return options;
+}
 var defaultNativeSpecGateBuilders = { buildSpecGateDeps, runSpecGate };
 async function runNativeSpecGate(ctx, input, builders = defaultNativeSpecGateBuilders) {
   const deps = builders.buildSpecGateDeps({
@@ -57516,6 +57558,7 @@ function stripHumanizeOpener(body) {
   }
   return body;
 }
+var DID_NOT_CONVERGE_ERROR = "Consensus blocked: reviewers did not converge within budget.";
 async function handleConsensusBlocked(result, ctx, input, resolvedTaskId) {
   const operatorClarifications = result.operatorClarifications ?? [];
   const diagnostics = result.diagnostics ?? [];
@@ -57523,9 +57566,10 @@ async function handleConsensusBlocked(result, ctx, input, resolvedTaskId) {
   const taskId = page?.taskId ?? null;
   if (!resolvedTaskId && taskId) resolvedTaskId = taskId;
   const adjudicated = [];
+  let taskInboxItems = [];
   if (taskId) {
     try {
-      const taskInboxItems = await ctx.client.listInboxItems(input.projectSlug, { taskId });
+      taskInboxItems = await ctx.client.listInboxItems(input.projectSlug, { taskId });
       for (const item of taskInboxItems) {
         if (item.origin === "gate" && (item.status === "acknowledged" || item.status === "rejected")) {
           adjudicated.push({ body: stripHumanizeOpener(item.body), answer: item.answer, status: item.status });
@@ -57594,6 +57638,35 @@ async function handleConsensusBlocked(result, ctx, input, resolvedTaskId) {
       filedQuestionCount = toFile.length;
     }
   }
+  if (filedQuestionCount === 0 && diagnostics.length === 0 && taskId && page) {
+    const domain2 = await resolveSpecDomain(ctx.client, input.projectSlug, {
+      id: page.id,
+      taskId: page.taskId ?? null
+    });
+    const reviewHash = domainScopedReviewHash2(page.contentHash, domain2);
+    const baseBlockItemId = `II-${taskId}-gateblock-${(0, import_node_crypto9.createHash)("sha256").update(reviewHash).digest("hex").slice(0, 10)}`;
+    let taskRef = taskId;
+    try {
+      const t = await ctx.client.getTask(input.projectSlug, taskId);
+      if (t?.title) taskRef = `${taskId} \u2014 ${t.title}`;
+    } catch {
+    }
+    const reviewAgainAvailable = !taskInboxItems.some(
+      (item) => item.id === baseBlockItemId && item.status !== "pending" && item.answer === REVIEW_AGAIN_ANSWER
+    );
+    const blockItemId = reviewAgainAvailable ? baseBlockItemId : `${baseBlockItemId}-2`;
+    try {
+      await ctx.client.createInboxItem(input.projectSlug, {
+        id: blockItemId,
+        kind: "question",
+        taskId,
+        body: renderBlockDecisionBody(taskRef, result.unresolved ?? []),
+        origin: "gate_block",
+        questionOptions: buildBlockDecisionOptions(result.escalation?.diagnosis, reviewAgainAvailable)
+      });
+    } catch {
+    }
+  }
   if (operatorClarifications.length > 0) {
     try {
       if (page) {
@@ -57634,7 +57707,7 @@ async function handleConsensusBlocked(result, ctx, input, resolvedTaskId) {
   } else if (operatorClarifications.length > 0) {
     error51 = `Consensus blocked on ${operatorClarifications.length} clarification(s), but the spec page is not linked to a task or plan, so none could be filed as questions.`;
   } else {
-    error51 = "Consensus blocked: reviewers did not converge within budget.";
+    error51 = DID_NOT_CONVERGE_ERROR;
   }
   if (result.escalation) {
     error51 += ` (diagnosis: ${result.escalation.diagnosis})`;
@@ -57776,6 +57849,14 @@ function onComplete2(outcome, input) {
       body: taskRef ? `Consensus blocked on ${taskRef} (Specs/${input.pageId}); filed ${n} clarifying question${n === 1 ? "" : "s"} \u2014 answer to unblock the run.` : `Consensus blocked on Specs/${input.pageId}; filed ${n} clarifying question${n === 1 ? "" : "s"} \u2014 answer to unblock the run.`
     };
   }
+  if (outcome.kind === "failed" && outcome.error.startsWith(DID_NOT_CONVERGE_ERROR) && taskId) {
+    return {
+      kind: "info",
+      suppressInbox: true,
+      ...taskId ? { taskId } : {},
+      body: taskRef ? `Consensus blocked on ${taskRef} (Specs/${input.pageId}); the reviewers did not agree \u2014 answer the decision question to unblock the run.` : `Consensus blocked on Specs/${input.pageId}; the reviewers did not agree \u2014 answer the decision question to unblock the run.`
+    };
+  }
   const handoffTaskRef = taskRef ? ` (${taskRef})` : "";
   return {
     kind: "handoff",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrrlin-dev/mcp",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "type": "module",
   "bin": {
     "mrrlin-mcp": "dist/bin.cjs"
@@ -23,10 +23,10 @@
     "tsx": "^4.22.3",
     "@mrrlin/client": "0.0.0",
     "@mrrlin/codex-client": "0.0.0",
-    "@mrrlin/director-e2e": "0.0.0",
-    "@mrrlin/schemas": "0.0.0",
     "@mrrlin/tsconfig": "0.0.0",
-    "@mrrlin/wiki": "0.0.0"
+    "@mrrlin/wiki": "0.0.0",
+    "@mrrlin/schemas": "0.0.0",
+    "@mrrlin/director-e2e": "0.0.0"
   },
   "dependencies": {
     "@iarna/toml": "^2.2.5",