@antonbabenko/deliberation-mcp 3.5.4 → 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 };
     }
@@ -520,7 +583,14 @@ var require_orchestrate = __commonJS({
       } catch {
       }
     }
-    async function callProvider(provider, req, logger, tool, cache) {
+    function withOrientation(provider, req, orientationFiles) {
+      if (Array.isArray(orientationFiles) && orientationFiles.length && !(Array.isArray(req.files) && req.files.length) && provider.capabilities && provider.capabilities.walksFilesystem === false) {
+        return { ...req, files: orientationFiles };
+      }
+      return req;
+    }
+    async function callProvider(provider, req, logger, tool, cache, orientationFiles) {
+      req = withOrientation(provider, req, orientationFiles);
       const useCache = cache && !(Array.isArray(req.files) && req.files.length);
       if (useCache) {
         const hit = cache.get(provider.name, req);
@@ -553,7 +623,7 @@ var require_orchestrate = __commonJS({
       const logger = opts.logger || NULL_LOGGER;
       const tool = opts.tool || "ask-all";
       const settled = await Promise.allSettled(
-        providers.map((p) => callProvider(p, req, logger, tool, opts.cache))
+        providers.map((p) => callProvider(p, req, logger, tool, opts.cache, opts.orientationFiles))
       );
       return settled.map(
         (s, i) => s.status === "fulfilled" ? s.value : {
@@ -568,7 +638,7 @@ var require_orchestrate = __commonJS({
       );
     }
     async function askOne2(provider, req, opts = {}) {
-      return callProvider(provider, req, opts.logger || NULL_LOGGER, opts.tool || "ask-one", opts.cache);
+      return callProvider(provider, req, opts.logger || NULL_LOGGER, opts.tool || "ask-one", opts.cache, opts.orientationFiles);
     }
     function buildArbiterPrompt(question, opinions) {
       const blocks = opinions.map((o, i) => `### Opinion ${i + 1}
@@ -593,17 +663,17 @@ ${blocks}`,
         // Promise.resolve().then(...) so even a SYNCHRONOUS throw in ask() is caught
         // by the rejection handler (a bare arbiter.ask() could throw before awaiting).
         Promise.resolve().then(
-          () => arbiter.ask({
+          () => arbiter.ask(withOrientation(arbiter, {
             ...req,
             files: req.files ? req.files.map((f) => ({ ...f })) : void 0,
             developerInstructions: opts.arbiterInstructions || req.developerInstructions
-          })
+          }, opts.orientationFiles))
         ).then((v) => v, () => null)
       ) : Promise.resolve(
         /** @type {DelegationResult|null} */
         null
       );
-      const [opinions, blindVerdict] = await Promise.all([askAll2(providers, req, { logger: opts.logger, tool: "consensus" }), blindPromise]);
+      const [opinions, blindVerdict] = await Promise.all([askAll2(providers, req, { logger: opts.logger, tool: "consensus", orientationFiles: opts.orientationFiles }), blindPromise]);
       const ok = (
         /** @type {DelegationSuccess[]} */
         opinions.filter((o) => !o.isError)
@@ -666,8 +736,8 @@ ${feedback || "(reviewers gave no specific issues; tighten the weakest part)"}`
           const { peerPrompt, blindPrompt } = loop.prepareRound(state);
           const roundNo = state.round;
           const [blindRes, peerResults] = await Promise.all([
-            Promise.resolve().then(() => arbiter.ask({ ...req, prompt: blindPrompt })).then((r) => r, () => null),
-            askAll2(providers, { ...req, prompt: peerPrompt }, { logger, tool: "consensus" })
+            Promise.resolve().then(() => arbiter.ask(withOrientation(arbiter, { ...req, prompt: blindPrompt }, opts.orientationFiles))).then((r) => r, () => null),
+            askAll2(providers, { ...req, prompt: peerPrompt }, { logger, tool: "consensus", orientationFiles: opts.orientationFiles })
           ]);
           state = loop.recordBlindVerdict(state, okText(blindRes) || "(blind pass unavailable)");
           lastResults = peerResults.map(
@@ -732,6 +802,52 @@ ${feedback || "(reviewers gave no specific issues; tighten the weakest part)"}`
   }
 });
 
+// ../../core/orientation.js
+var require_orientation = __commonJS({
+  "../../core/orientation.js"(exports2, module2) {
+    "use strict";
+    var fs = require("node:fs");
+    var path = require("node:path");
+    var ORIENTATION_CANDIDATES = [
+      "CLAUDE.md",
+      "AGENTS.md",
+      "README.md",
+      "package.json",
+      "pyproject.toml",
+      "Cargo.toml",
+      "go.mod",
+      "tsconfig.json",
+      "main.tf"
+    ];
+    var DEFAULT_MAX_FILES = 6;
+    function resolveOrientationFiles(cwd, opts = {}) {
+      const base = cwd || process.cwd();
+      const max = Number.isInteger(opts.maxFiles) && /** @type {number} */
+      opts.maxFiles > 0 ? (
+        /** @type {number} */
+        opts.maxFiles
+      ) : DEFAULT_MAX_FILES;
+      const candidates = opts.candidates || ORIENTATION_CANDIDATES;
+      const out = [];
+      for (const name of candidates) {
+        if (out.length >= max) break;
+        const abs = path.join(base, name);
+        try {
+          if (fs.statSync(abs).isFile()) out.push({ path: abs });
+        } catch {
+        }
+      }
+      return out;
+    }
+    function orientationFilesFor2(config, cwd) {
+      const o = config && config.orientation;
+      if (!o || o.enabled !== true) return void 0;
+      return resolveOrientationFiles(cwd, { maxFiles: o.maxFiles });
+    }
+    module2.exports = { resolveOrientationFiles, orientationFilesFor: orientationFilesFor2, ORIENTATION_CANDIDATES, DEFAULT_MAX_FILES };
+  }
+});
+
 // ../../core/prompts/index.js
 var require_prompts = __commonJS({
   "../../core/prompts/index.js"(exports2, module2) {
@@ -1443,7 +1559,7 @@ var require_openai_compatible = __commonJS({
         /** @type {any} */
         {
           name,
-          capabilities: { canImplement: false, fileUpload: false, multiTurn: true },
+          capabilities: { canImplement: false, fileUpload: false, multiTurn: true, walksFilesystem: false },
           /** Test-only: current number of cached sessions. */
           get __sessionCount() {
             return sessions.size;
@@ -1525,7 +1641,7 @@ var require_grok = __commonJS({
         name: "grok",
         // multiTurn is not wired through Core (runGrok/runWithFiles return no threadId),
         // so report false to match reality.
-        capabilities: { canImplement: false, fileUpload: true, multiTurn: false },
+        capabilities: { canImplement: false, fileUpload: true, multiTurn: false, walksFilesystem: false },
         async health() {
           return process.env.XAI_API_KEY ? { ok: true } : { ok: false, reason: "XAI_API_KEY unset" };
         },
@@ -1585,7 +1701,7 @@ var require_antigravity = __commonJS({
       const model = opts.model || process.env.GEMINI_DEFAULT_MODEL || "auto-gemini-3";
       return {
         name: "gemini",
-        capabilities: { canImplement: true, fileUpload: false, multiTurn: true },
+        capabilities: { canImplement: true, fileUpload: false, multiTurn: true, walksFilesystem: true },
         async health() {
           return typeof bridge.runGemini === "function" ? { ok: true } : { ok: false, reason: "agy bridge unavailable" };
         },
@@ -1666,7 +1782,7 @@ var require_codex = __commonJS({
       const model = opts.model || "default";
       return {
         name: "codex",
-        capabilities: { canImplement: true, fileUpload: false, multiTurn: false },
+        capabilities: { canImplement: true, fileUpload: false, multiTurn: false, walksFilesystem: true },
         // Option A: no threadId continuity
         async health() {
           return { ok: true };
@@ -4525,6 +4641,7 @@ var require_openrouter = __commonJS({
 // index.js
 var { makeRegistry, pinAlias } = require_registry();
 var { askAll, askOne, consensus, runToConvergence } = require_orchestrate();
+var { orientationFilesFor } = require_orientation();
 var { PROMPTS } = require_prompts();
 var analyzeCore = require_analyze();
 var ADVISORY = { readOnlyHint: true };
@@ -4801,6 +4918,9 @@ function buildServer({ providers, getConfig, getConfigError, sessionsDir, notify
     if (!persona) return request;
     return { ...request, developerInstructions: persona };
   }
+  function orient(req) {
+    return orientationFilesFor(getConfig(), req && req.cwd);
+  }
   function sessionsCfg() {
     const c = getConfig() || {};
     const s = c.sessions || {};
@@ -4898,7 +5018,7 @@ function buildServer({ providers, getConfig, getConfigError, sessionsDir, notify
       lg.logEvent({ event: "dispatch_start", at: Date.now(), tool: "ask-all", voices: selected.length });
     } catch {
     }
-    const results = await askAll(selected, withPersona(req, expert), { logger: lg, tool: "ask-all", cache: opts.noCache ? void 0 : resultCache });
+    const results = await askAll(selected, withPersona(req, expert), { logger: lg, tool: "ask-all", cache: opts.noCache ? void 0 : resultCache, orientationFiles: orient(req) });
     return {
       payload: { results, omitted },
       parts: { opinions: results, blindVerdict: null, verdict: null, arbiter: null, warnings: [] }
@@ -4916,13 +5036,13 @@ function buildServer({ providers, getConfig, getConfigError, sessionsDir, notify
     const resolved = await resolveArbiter(arbiterSpec, selected, registry, getConfig);
     if (resolved.warning) warnings.push(resolved.warning);
     if (resolved.mode === "host") {
-      const opinions = await askAll(selected, withPersona(req, expert), { logger: currentLogger(), tool: "consensus" });
+      const opinions = await askAll(selected, withPersona(req, expert), { logger: currentLogger(), tool: "consensus", orientationFiles: orient(req) });
       const arbiter2 = { mode: "host" };
       const body = { opinions, blindVerdict: null, verdict: null, arbiter: arbiter2, warnings };
       return { payload: body, parts: body };
     }
     if (!resolved.provider) {
-      const out2 = await consensus(selected, withPersona(req, expert), { arbiterInstructions: PROMPTS.arbiter, logger: currentLogger() });
+      const out2 = await consensus(selected, withPersona(req, expert), { arbiterInstructions: PROMPTS.arbiter, logger: currentLogger(), orientationFiles: orient(req) });
       const arbiter2 = { mode: "server", provider: null };
       return {
         payload: { opinions: out2.opinions, blindVerdict: out2.blindVerdict, verdict: out2.verdict, error: out2.error, arbiter: arbiter2, warnings },
@@ -4935,7 +5055,7 @@ function buildServer({ providers, getConfig, getConfigError, sessionsDir, notify
       peers = selected;
       warnings.push(`panel too small to exclude arbiter '${arbiterP.name}'; kept it in the peer panel (floor of 2)`);
     }
-    const out = await consensus(peers, withPersona(req, expert), { arbiter: arbiterP, arbiterInstructions: PROMPTS.arbiter, blindVote, logger: currentLogger() });
+    const out = await consensus(peers, withPersona(req, expert), { arbiter: arbiterP, arbiterInstructions: PROMPTS.arbiter, blindVote, logger: currentLogger(), orientationFiles: orient(req) });
     const arbiter = { mode: "server", provider: arbiterP.name };
     return {
       payload: { opinions: out.opinions, blindVerdict: out.blindVerdict, verdict: out.verdict, error: out.error, arbiter, warnings },
@@ -4965,7 +5085,7 @@ function buildServer({ providers, getConfig, getConfigError, sessionsDir, notify
       }
       const maxRounds = Number.isInteger(maxRoundsOverride) && /** @type {number} */
       maxRoundsOverride > 0 ? maxRoundsOverride : Number.isInteger(cc.maxRounds) && cc.maxRounds > 0 ? cc.maxRounds : void 0;
-      const out = await runToConvergence(peers, withPersona(req, expert), { arbiter: arbiterP, maxRounds, logger: currentLogger() });
+      const out = await runToConvergence(peers, withPersona(req, expert), { arbiter: arbiterP, maxRounds, logger: currentLogger(), orientationFiles: orient(req) });
       const allWarnings = out.error ? warnings.concat([`loop: ${out.error}`]) : warnings;
       const rounds = Array.isArray(out.rounds) ? out.rounds.length : 0;
       const arbiter = { mode: "server", provider: arbiterP.name };
@@ -5106,7 +5226,7 @@ function buildServer({ providers, getConfig, getConfigError, sessionsDir, notify
           lg.logEvent({ event: "dispatch_start", at: Date.now(), tool: "consensus", round: cur.round, voices: selected.length });
         } catch {
         }
-        const peerResults = await askAll(selected, withPersona(peerReq, ex), { logger: lg, tool: "consensus" });
+        const peerResults = await askAll(selected, withPersona(peerReq, ex), { logger: lg, tool: "consensus", orientationFiles: orient(peerReq) });
         const results = peerResults.map(
           (r) => r.isError ? { source: r.provider, isError: true, errorKind: r.errorKind, verdict: null, criticalIssues: [], model: r.model, reasoningEffort: r.reasoningEffort ?? null, ms: r.ms } : { ...parseReview(typeof r.text === "string" ? r.text : ""), source: r.provider, isError: false, model: r.model, reasoningEffort: r.reasoningEffort ?? null, ms: r.ms }
         );
@@ -5230,7 +5350,7 @@ function buildServer({ providers, getConfig, getConfigError, sessionsDir, notify
       if (!p) {
         return jsonResult({ error: `provider "${want}" is not in the active panel`, panel: selected.map((x) => x.name) });
       }
-      const result = await askOne(p, withPersona(req, expert), { logger: currentLogger(), tool: "ask-one", cache: resultCache });
+      const result = await askOne(p, withPersona(req, expert), { logger: currentLogger(), tool: "ask-one", cache: resultCache, orientationFiles: orient(req) });
       return jsonResult({ result });
     }
     if (name === "analyze") {
@@ -5299,12 +5419,12 @@ function buildServer({ providers, getConfig, getConfigError, sessionsDir, notify
     if (Object.prototype.hasOwnProperty.call(ASK_PROVIDER, name)) {
       const p = registry.get(ASK_PROVIDER[name]);
       if (!p) return { content: [{ type: "text", text: JSON.stringify({ error: `provider ${ASK_PROVIDER[name]} not registered` }) }] };
-      const result = await askOne(p, withPersona(req, expert), { logger: currentLogger(), tool: "ask-one", cache: resultCache });
+      const result = await askOne(p, withPersona(req, expert), { logger: currentLogger(), tool: "ask-one", cache: resultCache, orientationFiles: orient(req) });
       return { content: [{ type: "text", text: JSON.stringify({ result }) }] };
     }
     if (EXPERTS.includes(name)) {
       const { providers: selected } = registry.selectForAskAll({ config: getConfig(), expert: name });
-      const results = await askAll(selected, withPersona({ ...req, expert: name }, expert), { logger: currentLogger(), tool: name, cache: resultCache });
+      const results = await askAll(selected, withPersona({ ...req, expert: name }, expert), { logger: currentLogger(), tool: name, cache: resultCache, orientationFiles: orient(req) });
       return { content: [{ type: "text", text: JSON.stringify({ results }) }] };
     }
     throw new Error(`unknown tool: ${name}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@antonbabenko/deliberation-mcp",
-  "version": "3.5.4",
+  "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" },