@antonbabenko/deliberation-mcp 3.6.0 → 3.6.1

package/dist/index.js

@@ -231,18 +231,71 @@ var require_provider = __commonJS({
     var VERDICT_RE = /\bverdict\b[^A-Za-z0-9]*\b(APPROVE|REJECT|REQUEST[_\s]CHANGES)\b/i;
     var BULLET_RE = /^([-*+•]|`?\[)/;
     var BRACKET_CAT_RE = /\[\s*([A-Za-z_]+)\s*\]/g;
+    var SENTINEL_RE = /^[#>*_`\s]*verdict\s*[:=]\s*[*_`\s]*(APPROVE|REJECT|REQUEST[_\s]CHANGES)\b/i;
+    var VERDICT_WORD_RE = /^[#>*_`\s]*verdict[*_`:\s]*$/i;
+    var TOKEN_LINE_RE = /^(APPROVE|REJECT|REQUEST[_\s]CHANGES)$/i;
+    var MD_EMPHASIS = /[*_`~]/g;
+    var FENCE_RE = /^\s*(```|~~~)/;
+    var ARTIFACT_RE = /^[*_`~\s:.\-]*$/;
+    var STRIP_LEAD = /^[*_`~\s:.\-]+/;
+    var STRIP_TRAIL = /[*_`~\s]+$/;
+    var HEADING_RE = /^#{1,6}\s/;
+    function normVerdict(tok) {
+      return tok.replace(/\s+/g, "_").toUpperCase();
+    }
     function parseReview(text) {
       const raw = safeString(text);
-      const lines = raw.split(/\r?\n/);
-      let verdict = null;
-      const criticalIssues = [];
-      for (const line of lines) {
-        if (verdict === null) {
-          const vm = line.match(VERDICT_RE);
-          if (vm) verdict = /** @type {ParsedReview["verdict"]} */
-          vm[1].replace(/\s+/g, "_").toUpperCase();
+      const lines = [];
+      let inFence = false;
+      for (const ln of raw.split(/\r?\n/)) {
+        if (FENCE_RE.test(ln)) {
+          inFence = !inFence;
+          continue;
         }
-        const trimmed = line.trim();
+        if (!inFence) lines.push(ln);
+      }
+      return { verdict: resolveVerdict(lines), criticalIssues: resolveIssues(lines) };
+    }
+    function resolveVerdict(lines) {
+      for (const ln of lines) {
+        const m = ln.match(SENTINEL_RE);
+        if (m) return (
+          /** @type {any} */
+          normVerdict(m[1])
+        );
+      }
+      for (const ln of lines) {
+        const m = ln.match(VERDICT_RE);
+        if (m) return (
+          /** @type {any} */
+          normVerdict(m[1])
+        );
+      }
+      for (let i = 0; i < lines.length; i++) {
+        if (!VERDICT_WORD_RE.test(lines[i].trim())) continue;
+        for (let j = i + 1; j < lines.length && j <= i + 3; j++) {
+          const t = lines[j].replace(MD_EMPHASIS, "").trim();
+          if (!t) continue;
+          if (TOKEN_LINE_RE.test(t)) return (
+            /** @type {any} */
+            normVerdict(t)
+          );
+          break;
+        }
+      }
+      for (const ln of lines) {
+        const t = ln.replace(MD_EMPHASIS, "").trim();
+        if (TOKEN_LINE_RE.test(t)) return (
+          /** @type {any} */
+          normVerdict(t)
+        );
+      }
+      return null;
+    }
+    function resolveIssues(lines) {
+      const out = [];
+      for (let i = 0; i < lines.length; i++) {
+        const trimmed = lines[i].trim();
         if (!BULLET_RE.test(trimmed)) continue;
         let chosen = null;
         for (const mm of trimmed.matchAll(BRACKET_CAT_RE)) {
@@ -258,10 +311,20 @@ var require_provider = __commonJS({
           /** @type {ReviewCriticalIssue["category"]} */
           REVIEW_CATEGORIES.includes(cat) ? cat : REVIEW_FALLBACK_CATEGORY
         );
-        const description = trimmed.slice((chosen.index || 0) + chosen[0].length).replace(/^[`\s:.\-]+/, "").trim();
-        if (description) criticalIssues.push({ category, description });
+        let description = trimmed.slice(chosen.index + chosen[0].length).replace(STRIP_LEAD, "").replace(STRIP_TRAIL, "").trim();
+        if (!description || ARTIFACT_RE.test(description)) {
+          for (let j = i + 1; j < lines.length; j++) {
+            const nt = lines[j].trim();
+            if (!nt) break;
+            if (BULLET_RE.test(nt) || HEADING_RE.test(nt)) break;
+            if (SENTINEL_RE.test(nt) || VERDICT_WORD_RE.test(nt) || VERDICT_RE.test(nt)) break;
+            description = nt.replace(STRIP_LEAD, "").replace(STRIP_TRAIL, "").trim();
+            break;
+          }
+        }
+        if (description && !ARTIFACT_RE.test(description)) out.push({ category, description });
       }
-      return { verdict, criticalIssues };
+      return out;
     }
     module2.exports = {
       toErrorResult,
@@ -324,12 +387,12 @@ ${state.currentPlan}`
       const peerPrompt = [
         body,
         "Review the plan for correctness, security, scope, ambiguity, performance, and ops gaps.",
-        "End with: **Verdict**: APPROVE | REQUEST_CHANGES | REJECT, then categorized critical issues."
+        "End with a line by itself in exactly this form (no markdown, token on the SAME line): VERDICT: APPROVE   (or VERDICT: REQUEST_CHANGES, or VERDICT: REJECT). Then list any critical issues, one per line as: - [category] description   where category is one of security, correctness, scope, ambiguity, performance, ops."
       ].join("\n\n");
       const blindPrompt = [
         body,
         "Give your own independent verdict BEFORE seeing peer opinions.",
-        "End with: **Verdict**: APPROVE | REQUEST_CHANGES | REJECT, then categorized critical issues."
+        "End with a line by itself in exactly this form (no markdown, token on the SAME line): VERDICT: APPROVE   (or VERDICT: REQUEST_CHANGES, or VERDICT: REJECT). Then list any critical issues, one per line as: - [category] description   where category is one of security, correctness, scope, ambiguity, performance, ops."
       ].join("\n\n");
       return { peerPrompt, blindPrompt };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@antonbabenko/deliberation-mcp",
-  "version": "3.6.0",
+  "version": "3.6.1",
   "description": "Deliberation for Claude Code and any MCP host - GPT, Gemini, Grok, and OpenRouter expert subagents.",
   "mcpName": "io.github.antonbabenko/deliberation",
   "repository": { "type": "git", "url": "git+https://github.com/antonbabenko/deliberation.git", "directory": "server/mcp" },