agent-dealer 0.1.2 → 0.1.4

package/bundle/server/dist/adapters/agent-deck-deliver.test.js

@@ -0,0 +1,52 @@
+import { test } from "node:test";
+import assert from "node:assert/strict";
+import { assertCallServiceToolSuccess, toCallServiceToolPayload } from "./agent-deck.js";
+test("toCallServiceToolPayload maps serviceName to serviceId verbatim args", () => {
+    const toolCall = {
+        serviceName: "34eb6c24-f151-4da2-8db8-d6996aa296be",
+        toolName: "chat_postMessage",
+        arguments: { channel: "C123", text: "Hello world" },
+    };
+    const payload = toCallServiceToolPayload(toolCall);
+    assert.equal(payload.serviceId, toolCall.serviceName);
+    assert.equal(payload.toolName, toolCall.toolName);
+    assert.deepEqual(payload.arguments, toolCall.arguments);
+});
+test("assertCallServiceToolSuccess throws on deck proxy failure JSON", () => {
+    assert.throws(() => assertCallServiceToolSuccess({
+        content: [{ type: "text", text: JSON.stringify({ success: false, error: "Service not found" }) }],
+    }), /Service not found/);
+});
+test("assertCallServiceToolSuccess throws on nested Slack channel_not_found", () => {
+    const receipt = {
+        content: [
+            {
+                type: "text",
+                text: JSON.stringify({
+                    content: [{ type: "text", text: "channel_not_found\n\nThe channel could not be found." }],
+                    isError: true,
+                }),
+            },
+        ],
+    };
+    assert.throws(() => assertCallServiceToolSuccess(receipt), /channel_not_found/);
+});
+test("assertCallServiceToolSuccess throws on deck-wrapped Slack JSON (live MCP shape)", () => {
+    const receipt = {
+        content: [
+            {
+                type: "text",
+                text: JSON.stringify({
+                    content: [
+                        {
+                            type: "text",
+                            text: "channel_not_found\n\nThe channel could not be found. It may not exist, may be in a different workspace than your app is installed on, or your app may lack permission to access it.",
+                        },
+                    ],
+                    isError: true,
+                }),
+            },
+        ],
+    };
+    assert.throws(() => assertCallServiceToolSuccess(receipt), /channel_not_found/);
+});

package/bundle/server/dist/adapters/agent-deck.js

@@ -1,5 +1,7 @@
 import fs from "node:fs";
 import path from "node:path";
+import { Client } from "@modelcontextprotocol/sdk/client/index.js";
+import { StreamableHTTPClientTransport } from "@modelcontextprotocol/sdk/client/streamableHttp.js";
 import { getAgentDeckConfig } from "../repository/intake-settings.js";
 export function getAgentDeckApiUrl() {
     if (process.env.AGENT_DECK_API_URL) {
@@ -134,3 +136,141 @@ export function isAgentDeckMcpRegistered() {
         return false;
     }
 }
+/** Map PRD toolCall to deck MCP call_service_tool args (serviceName → serviceId). */
+export function toCallServiceToolPayload(toolCall) {
+    return {
+        serviceId: toolCall.serviceName,
+        toolName: toolCall.toolName,
+        arguments: toolCall.arguments,
+    };
+}
+function extractPermalink(toolResult) {
+    if (!toolResult || typeof toolResult !== "object")
+        return undefined;
+    const content = toolResult.content;
+    if (!Array.isArray(content))
+        return undefined;
+    for (const block of content) {
+        if (block.type !== "text" || !block.text)
+            continue;
+        try {
+            const parsed = JSON.parse(block.text);
+            if (parsed.permalink)
+                return parsed.permalink;
+        }
+        catch {
+            const m = block.text.match(/https?:\/\/\S+/);
+            if (m)
+                return m[0];
+        }
+    }
+    return undefined;
+}
+/** Parse Agent Deck call_service_tool MCP result — throws on proxy- or Slack-reported failure. */
+export function assertCallServiceToolSuccess(toolResult) {
+    const err = findCallServiceToolError(toolResult);
+    if (err)
+        throw new Error(err);
+}
+function findCallServiceToolError(toolResult, depth = 0) {
+    if (!toolResult || typeof toolResult !== "object" || depth > 4)
+        return undefined;
+    const tr = toolResult;
+    if (tr.isError) {
+        const text = formatToolResultText(tr) ?? "call_service_tool failed";
+        const nested = parseNestedCallServiceToolError(text, depth);
+        if (nested)
+            return nested;
+        return summarizeToolErrorText(text) ?? text.slice(0, 500);
+    }
+    if (tr.success === false) {
+        return tr.error ?? "call_service_tool failed";
+    }
+    const text = formatToolResultText(tr);
+    if (text) {
+        const nested = parseNestedCallServiceToolError(text, depth);
+        if (nested)
+            return nested;
+        const plain = summarizeToolErrorText(text);
+        if (plain)
+            return plain;
+    }
+    return undefined;
+}
+function parseNestedCallServiceToolError(text, depth) {
+    try {
+        const parsed = JSON.parse(text);
+        return findCallServiceToolError(parsed, depth + 1);
+    }
+    catch {
+        return undefined;
+    }
+}
+const SLACK_ERROR_RE = /channel_not_found|not_in_channel|missing_scope|invalid_auth|user_not_found|cannot_dm|is_archived/i;
+function summarizeToolErrorText(text) {
+    const t = text.trim();
+    if (!t)
+        return undefined;
+    try {
+        const parsed = JSON.parse(t);
+        if (typeof parsed.error === "string" && parsed.error.trim())
+            return parsed.error.trim();
+        if (typeof parsed.message === "string" && parsed.message.trim())
+            return parsed.message.trim();
+    }
+    catch {
+        // not top-level JSON — fall through
+    }
+    if (SLACK_ERROR_RE.test(t)) {
+        const line = t.split("\n").find((l) => SLACK_ERROR_RE.test(l));
+        if (line?.trim())
+            return line.trim().slice(0, 500);
+        const match = t.match(SLACK_ERROR_RE);
+        if (match)
+            return match[0];
+    }
+    if (/"success"\s*:\s*false/i.test(t)) {
+        try {
+            const parsed = JSON.parse(t);
+            if (typeof parsed.error === "string" && parsed.error.trim())
+                return parsed.error.trim();
+        }
+        catch {
+            // ignore
+        }
+    }
+    return undefined;
+}
+function formatToolResultText(tr) {
+    if (!Array.isArray(tr.content))
+        return undefined;
+    return tr.content
+        .filter((b) => b.type === "text" && b.text)
+        .map((b) => b.text)
+        .join("\n")
+        .trim() || undefined;
+}
+export async function deliverOutboundDraft(deckId, toolCall, opts) {
+    const payload = toCallServiceToolPayload(toolCall);
+    const workspaceRoot = opts?.workspaceRoot ?? process.cwd();
+    const callTool = opts?.callTool ??
+        (async (p) => {
+            const mcpBase = getAgentDeckMcpUrl().replace(/\/mcp\/?$/, "");
+            const transport = new StreamableHTTPClientTransport(new URL(`${mcpBase}/mcp`));
+            const client = new Client({ name: "agent-dealer-deliver", version: "0.0.1" });
+            await client.connect(transport);
+            try {
+                await client.callTool({
+                    name: "bind_workspace",
+                    arguments: { deckId, workspaceRoot },
+                });
+                return await client.callTool({ name: "call_service_tool", arguments: p });
+            }
+            finally {
+                await client.close();
+            }
+        });
+    const toolResult = await callTool(payload);
+    assertCallServiceToolSuccess(toolResult);
+    return { toolResult, permalink: extractPermalink(toolResult) };
+}

package/bundle/server/dist/queue/answers.test.js

@@ -0,0 +1,117 @@
+import { test, before } from "node:test";
+import assert from "node:assert/strict";
+import fs from "node:fs";
+import os from "node:os";
+import path from "node:path";
+process.env.AGENT_DEALER_HOME = fs.mkdtempSync(path.join(os.tmpdir(), "dealer-answers-"));
+process.env.MAX_CONCURRENT_RUNS = "0";
+const { migrate } = await import("../db/index.js");
+const { BUILTIN_AGENT_CLAUDE_ID } = await import("@agent-dealer/shared");
+const { updateAgent } = await import("../repository/agents.js");
+const { addArtifact, createRun, getLatestArtifact, getRun, markPlanTriageConsumed, updateRunFields } = await import("../repository/runs.js");
+const { submitPlanAnswers } = await import("./dispatcher.js");
+const QUESTIONS = [
+    { id: "q1", question: "Which approach?", options: [{ label: "A" }, { label: "B" }] },
+    { id: "q2", question: "Ship docs too?", options: [{ label: "Yes" }, { label: "No" }] },
+];
+before(() => {
+    migrate();
+    updateAgent(BUILTIN_AGENT_CLAUDE_ID, { workspaceRoot: process.env.AGENT_DEALER_HOME });
+});
+function seedQuestionRun() {
+    const run = createRun({
+        title: "Answers test",
+        taskCategory: "other",
+        status: "plan_pending",
+        agentId: BUILTIN_AGENT_CLAUDE_ID,
+    });
+    addArtifact(run.id, "draft_plan", { markdown: "# Plan", sessionId: "sess-1" }, "agent");
+    addArtifact(run.id, "plan_triage", { verdict: "needs_review", rationale: "r", questions: QUESTIONS, sessionId: "sess-1", parseFallback: false, consumed: false }, "agent");
+    return getRun(run.id);
+}
+test("all-structured answers approve and mark plan_answers approved", () => {
+    const run = seedQuestionRun();
+    const res = submitPlanAnswers(run.id, {
+        answers: [
+            { questionId: "q1", selectedLabel: "A" },
+            { questionId: "q2", selectedLabel: "No" },
+        ],
+    });
+    assert.equal(res.ok, true);
+    assert.equal(res.ok && res.outcome, "approved");
+    assert.equal(getRun(run.id).status, "plan_approved");
+    const ans = getLatestArtifact(run.id, "plan_answers");
+    assert.match(ans.contentJson, /"outcome":"approved"/);
+    assert.equal(getLatestArtifact(run.id, "approved_plan").author, "human");
+    const triage = getLatestArtifact(run.id, "plan_triage");
+    assert.match(triage.contentJson, /"consumed":true/);
+});
+test("executeModel null does not wipe seeded execute_model", () => {
+    const run = seedQuestionRun();
+    updateRunFields(run.id, { execute_model: "claude-sonnet-4" });
+    const res = submitPlanAnswers(run.id, {
+        answers: [
+            { questionId: "q1", selectedLabel: "A" },
+            { questionId: "q2", selectedLabel: "No" },
+        ],
+        executeModel: null,
+    });
+    assert.equal(res.ok, true);
+    assert.equal(getRun(run.id).executeModel, "claude-sonnet-4");
+});
+test("free-form answer schedules a redraft instead of approving", () => {
+    const run = seedQuestionRun();
+    let redrafted = false;
+    const res = submitPlanAnswers(run.id, {
+        answers: [
+            { questionId: "q1", freeText: "Do it a third way" },
+            { questionId: "q2", selectedLabel: "Yes" },
+        ],
+    }, { onRedraft: () => { redrafted = true; } });
+    assert.equal(res.ok && res.outcome, "redraft");
+    assert.equal(redrafted, true);
+    assert.equal(getRun(run.id).status, "plan_pending");
+});
+test("answers must cover every open question exactly once", () => {
+    const run = seedQuestionRun();
+    const res = submitPlanAnswers(run.id, { answers: [{ questionId: "q1", selectedLabel: "A" }] });
+    assert.equal(res.ok, false);
+    assert.equal(!res.ok && res.code, 400);
+});
+test("double submit returns 409", () => {
+    const run = seedQuestionRun();
+    submitPlanAnswers(run.id, {
+        answers: [
+            { questionId: "q1", selectedLabel: "A" },
+            { questionId: "q2", selectedLabel: "Yes" },
+        ],
+    });
+    const again = submitPlanAnswers(run.id, {
+        answers: [
+            { questionId: "q1", selectedLabel: "B" },
+            { questionId: "q2", selectedLabel: "No" },
+        ],
+    });
+    assert.equal(!again.ok && again.code, 409);
+});
+test("run without open questions returns 409", () => {
+    const run = createRun({
+        title: "No questions",
+        taskCategory: "other",
+        status: "plan_pending",
+        agentId: BUILTIN_AGENT_CLAUDE_ID,
+    });
+    const res = submitPlanAnswers(run.id, { answers: [{ questionId: "q1", selectedLabel: "A" }] });
+    assert.equal(!res.ok && res.code, 409);
+});
+test("consumed triage rejects answers", () => {
+    const run = seedQuestionRun();
+    markPlanTriageConsumed(run.id);
+    const res = submitPlanAnswers(run.id, {
+        answers: [
+            { questionId: "q1", selectedLabel: "A" },
+            { questionId: "q2", selectedLabel: "Yes" },
+        ],
+    });
+    assert.equal(!res.ok && res.code, 409);
+});

package/bundle/server/dist/queue/approve-deliver.js

@@ -0,0 +1,62 @@
+import { deliverOutboundDraft } from "../adapters/agent-deck.js";
+import { syncLinearForRun } from "../adapters/linear-sync.js";
+import { workspaceForRun } from "../runners/prompts.js";
+import { addArtifact, appendEvent, getRun, transitionRun, } from "../repository/runs.js";
+import { deliverInFlight, getPendingOutboundDraft, markOutboundDraftSent, } from "../repository/outbound-drafts.js";
+import { scheduleReflect } from "./dispatcher.js";
+export async function approveRunWithDeliver(runId, deps) {
+    const run = getRun(runId);
+    if (!run)
+        return { ok: false, code: 404, error: "Not found" };
+    if (run.status !== "review") {
+        return { ok: false, code: 400, error: "Run must be in review" };
+    }
+    if (deliverInFlight.has(runId)) {
+        return { ok: false, code: 409, error: "Deliver already in progress" };
+    }
+    const pending = getPendingOutboundDraft(runId);
+    if (!pending) {
+        const updated = transitionRun(runId, "done");
+        syncLinearForRun(updated, "done").catch((e) => console.error("[linear-sync] done:", e));
+        scheduleReflect(updated, { trigger: "approve" });
+        return { ok: true, run: updated, delivered: false };
+    }
+    if (!run.deckId) {
+        return { ok: false, code: 502, error: "No deck bound — cannot deliver outbound draft" };
+    }
+    const deliver = deps?.deliver ?? deliverOutboundDraft;
+    deliverInFlight.add(runId);
+    try {
+        let toolResult;
+        let permalink;
+        try {
+            const result = await deliver(run.deckId, pending.content.draft.toolCall, {
+                workspaceRoot: workspaceForRun(run),
+            });
+            toolResult = result.toolResult;
+            permalink = result.permalink;
+        }
+        catch (err) {
+            const message = err instanceof Error ? err.message : String(err);
+            appendEvent(runId, "deliver_failed", { error: message });
+            return { ok: false, code: 502, error: message };
+        }
+        const sent = markOutboundDraftSent(pending.artifact.id);
+        if (!sent) {
+            return { ok: false, code: 409, error: "Draft already sent or rejected" };
+        }
+        addArtifact(runId, "send_receipt", {
+            draftArtifactId: pending.artifact.id,
+            sentAt: new Date().toISOString(),
+            toolResult: toolResult,
+            permalink,
+        }, "system");
+        const updated = transitionRun(runId, "done");
+        syncLinearForRun(updated, "done").catch((e) => console.error("[linear-sync] done:", e));
+        scheduleReflect(updated, { trigger: "approve" });
+        return { ok: true, run: updated, delivered: true };
+    }
+    finally {
+        deliverInFlight.delete(runId);
+    }
+}

package/bundle/server/dist/queue/dispatcher.js

@@ -1,9 +1,12 @@
-import { addArtifact, countByStatus, getLatestArtifact, getRun, listRuns, transitionRun, } from "../repository/runs.js";
+import { planGateDecision } from "@agent-dealer/shared";
+import { buildPlanRevisePrompt } from "../runners/prompts.js";
+import { addArtifact, countByStatus, getLatestArtifact, getRun, listArtifacts, listRuns, markPlanTriageConsumed, patchRunPhaseBudget, transitionRun, updateRunFields, } from "../repository/runs.js";
 import { runAgent } from "../runners/claude.js";
 import { persistRunOutput, seedDeliverableFromParent } from "../runners/persist.js";
 import { checkAgentDeckHealth } from "../adapters/agent-deck.js";
 import { pollLinearIssues } from "../adapters/external.js";
 import { syncLinearForRun } from "../adapters/linear-sync.js";
+import { pendingSendCount } from "../repository/outbound-drafts.js";
 import { listAgentsWithHealth } from "../adapters/agent-health.js";
 import { listAgents } from "../repository/agents.js";
 import { runReflect } from "../runners/reflect.js";
@@ -42,6 +45,24 @@ function isReadyForPlanReview(run) {
         !!getLatestArtifact(run.id, "draft_plan") &&
         !activePlanDrafts.has(run.id));
 }
+/** Open (unanswered, unconsumed) question count for a plan_pending run. */
+function openQuestionCount(runId) {
+    const tri = getLatestArtifact(runId, "plan_triage");
+    if (!tri?.contentJson)
+        return 0;
+    try {
+        const c = JSON.parse(tri.contentJson);
+        if (c.consumed || !c.questions?.length)
+            return 0;
+        const ans = getLatestArtifact(runId, "plan_answers");
+        if (ans && ans.createdAt > tri.createdAt)
+            return 0;
+        return c.questions.length;
+    }
+    catch {
+        return 0;
+    }
+}
 export function getSnapshot() {
     const all = listRuns();
     const runningRuns = all.filter((r) => r.status === "running");
@@ -57,6 +78,25 @@ export function getSnapshot() {
     const awaitingPlanReview = planningPhase.filter(isReadyForPlanReview);
     const planningActiveRuns = planningPhase.filter((r) => activePlanDrafts.has(r.id));
     const planningQueuedRuns = planningPhase.filter((r) => !activePlanDrafts.has(r.id) && !isReadyForPlanReview(r));
+    const openQuestionCounts = {};
+    for (const run of awaitingPlanReview) {
+        const n = openQuestionCount(run.id);
+        if (n > 0)
+            openQuestionCounts[run.id] = n;
+    }
+    const awaitingAnswerRuns = awaitingPlanReview.filter((r) => openQuestionCounts[r.id]);
+    const pendingSendCounts = {};
+    for (const run of resultReviewRuns) {
+        const n = pendingSendCount(run.id);
+        if (n > 0)
+            pendingSendCounts[run.id] = n;
+    }
+    const autoApprovedRunIds = [...waitingExecution, ...runningRuns]
+        .filter((r) => getLatestArtifact(r.id, "approved_plan")?.author === "system")
+        .map((r) => r.id);
+    const sentRunIds = recentDone
+        .filter((r) => getLatestArtifact(r.id, "send_receipt"))
+        .map((r) => r.id);
     return {
         planReviewCount: awaitingPlanReview.length,
         resultReviewCount: resultReviewRuns.length,
@@ -68,6 +108,11 @@ export function getSnapshot() {
         resultReviewRuns,
         recentDone,
         awaitingPlanReview,
+        awaitingAnswerRuns,
+        openQuestionCounts,
+        pendingSendCounts,
+        sentRunIds,
+        autoApprovedRunIds,
         runs: all,
         agentDeckOnline: false,
         agents: [],
@@ -195,6 +240,49 @@ export async function kickRun(run) {
     }
     await dispatch();
 }
+function priorQuestionRounds(runId, currentTriageArtifactId) {
+    return listArtifacts(runId).filter((a) => {
+        if (a.kind !== "plan_triage" || a.id === currentTriageArtifactId || !a.contentJson)
+            return false;
+        try {
+            const c = JSON.parse(a.contentJson);
+            return (c.questions?.length ?? 0) > 0;
+        }
+        catch {
+            return false;
+        }
+    }).length;
+}
+/** Self-triage gate after a plan draft persists (PRD F2). Exported for tests. */
+export function applyPlanGate(run) {
+    const art = getLatestArtifact(run.id, "plan_triage");
+    if (!art?.contentJson)
+        return "await_review";
+    let triage;
+    try {
+        triage = JSON.parse(art.contentJson);
+    }
+    catch {
+        return "await_review";
+    }
+    const decision = planGateDecision({
+        triage,
+        priorQuestionRounds: priorQuestionRounds(run.id, art.id),
+    });
+    if (decision !== "auto_approve")
+        return decision;
+    const fresh = getRun(run.id);
+    if (!fresh || fresh.status !== "plan_pending")
+        return "await_review";
+    const draft = getLatestArtifact(run.id, "draft_plan");
+    if (!draft?.contentJson)
+        return "await_review";
+    const plan = JSON.parse(draft.contentJson);
+    addArtifact(run.id, "approved_plan", { ...plan, autoApproved: true, rationale: triage.rationale }, "system");
+    transitionRun(run.id, "plan_approved");
+    void forceDispatch();
+    return "auto_approve";
+}
 export async function draftPlan(run, opts) {
     if (run.status === "queued") {
         transitionRun(run.id, "plan_pending");
@@ -208,7 +296,7 @@ export async function draftPlan(run, opts) {
     }
     const rt = runtimeFor(updated);
     try {
-        const result = await runAgent(updated, "plan");
+        const result = await runAgent(updated, "plan", opts?.revise);
         const persisted = persistRunOutput({
             run: updated,
             phase: "plan",
@@ -225,6 +313,18 @@ export async function draftPlan(run, opts) {
         }
         if (persisted.planMarkdown) {
             addArtifact(updated.id, "draft_plan", { markdown: persisted.planMarkdown, sessionId: persisted.sessionId }, "agent", result.logPath);
+            const triage = persisted.planTriage;
+            if (triage) {
+                addArtifact(updated.id, "plan_triage", {
+                    verdict: triage.verdict,
+                    rationale: triage.rationale,
+                    questions: triage.questions,
+                    sessionId: persisted.sessionId,
+                    parseFallback: triage.parseFallback,
+                    consumed: false,
+                }, "agent");
+                applyPlanGate(getRun(run.id));
+            }
         }
         else if (!opts?.replace) {
             addArtifact(updated.id, "feedback", { error: "Agent did not return a plan" }, "system");
@@ -245,6 +345,80 @@ export async function draftPlan(run, opts) {
     await notify();
     return getRun(run.id);
 }
+/** Answer open plan questions (PRD F3). Structured answers approve + dispatch; free-form revises the plan. */
+export function submitPlanAnswers(runId, input, opts) {
+    const run = getRun(runId);
+    if (!run)
+        return { ok: false, code: 404, error: "Not found" };
+    if (run.status !== "plan_pending") {
+        return { ok: false, code: 409, error: `No open questions — run is ${run.status}` };
+    }
+    const triageArt = getLatestArtifact(runId, "plan_triage");
+    if (!triageArt?.contentJson)
+        return { ok: false, code: 409, error: "No open questions" };
+    let triage;
+    try {
+        triage = JSON.parse(triageArt.contentJson);
+    }
+    catch {
+        return { ok: false, code: 409, error: "No open questions" };
+    }
+    if (triage.consumed || triage.questions.length === 0) {
+        return { ok: false, code: 409, error: "No open questions" };
+    }
+    const priorAnswers = getLatestArtifact(runId, "plan_answers");
+    if (priorAnswers && priorAnswers.createdAt > triageArt.createdAt) {
+        return { ok: false, code: 409, error: "Questions already answered" };
+    }
+    const expected = triage.questions.map((q) => q.id).sort().join(",");
+    const got = input.answers.map((a) => a.questionId).sort().join(",");
+    if (expected !== got) {
+        return { ok: false, code: 400, error: "Answers must cover every open question exactly once" };
+    }
+    const freeForm = input.answers.some((a) => a.freeText);
+    const outcome = freeForm ? "redraft" : "approved";
+    addArtifact(runId, "plan_answers", { answers: input.answers, outcome, answeredAt: new Date().toISOString() }, "human");
+    if (!freeForm) {
+        // null/empty = "Default" — do not wipe a seeded execute_model; only set explicit picks
+        if (input.executeModel?.trim()) {
+            updateRunFields(runId, { execute_model: input.executeModel.trim() });
+        }
+        if (input.executeBudget !== undefined) {
+            patchRunPhaseBudget(runId, "execute", input.executeBudget);
+        }
+        markPlanTriageConsumed(runId);
+        try {
+            transitionRun(runId, "plan_approved");
+        }
+        catch (e) {
+            return { ok: false, code: 409, error: String(e) };
+        }
+        const draft = getLatestArtifact(runId, "draft_plan");
+        const plan = draft?.contentJson ? JSON.parse(draft.contentJson) : { markdown: "" };
+        addArtifact(runId, "approved_plan", plan, "human");
+        void forceDispatch();
+        return { ok: true, outcome, run: getRun(runId) };
+    }
+    (opts?.onRedraft ?? scheduleRevisePlan)(getRun(runId), triage, input.answers);
+    return { ok: true, outcome, run: getRun(runId) };
+}
+function scheduleRevisePlan(run, triage, answers) {
+    if (activePlanDrafts.has(run.id))
+        return;
+    activePlanDrafts.add(run.id);
+    void draftPlan(run, {
+        replace: true,
+        revise: {
+            resumeSessionId: triage.sessionId,
+            prompt: buildPlanRevisePrompt(run, triage.questions, answers),
+        },
+    })
+        .catch((e) => console.error("[plan-revise]", run.id, e))
+        .finally(() => {
+        activePlanDrafts.delete(run.id);
+        notify().catch(console.error);
+    });
+}
 /** Fire-and-forget: human requested a new agent plan (replaces draft). */
 export function scheduleRedraft(run) {
     if (activePlanDrafts.has(run.id))

package/bundle/server/dist/queue/plan-gate.test.js

@@ -0,0 +1,55 @@
+import { test, before } from "node:test";
+import assert from "node:assert/strict";
+import fs from "node:fs";
+import os from "node:os";
+import path from "node:path";
+process.env.AGENT_DEALER_HOME = fs.mkdtempSync(path.join(os.tmpdir(), "dealer-gate-"));
+process.env.MAX_CONCURRENT_RUNS = "0";
+const { migrate } = await import("../db/index.js");
+const { BUILTIN_AGENT_CLAUDE_ID } = await import("@agent-dealer/shared");
+const { updateAgent } = await import("../repository/agents.js");
+const { addArtifact, createRun, getLatestArtifact, getRun } = await import("../repository/runs.js");
+const { applyPlanGate, getSnapshot } = await import("./dispatcher.js");
+const QUESTIONS = [
+    { id: "q1", question: "Which approach?", options: [{ label: "A" }, { label: "B" }] },
+];
+before(() => {
+    migrate();
+    updateAgent(BUILTIN_AGENT_CLAUDE_ID, { workspaceRoot: process.env.AGENT_DEALER_HOME });
+});
+function seedRun(triage) {
+    const run = createRun({
+        title: "Gate test",
+        taskCategory: "other",
+        status: "plan_pending",
+        agentId: BUILTIN_AGENT_CLAUDE_ID,
+    });
+    addArtifact(run.id, "draft_plan", { markdown: "# Plan" }, "agent");
+    addArtifact(run.id, "plan_triage", triage, "agent");
+    return getRun(run.id);
+}
+test("trivial triage auto-approves: system approved_plan + plan_approved status", () => {
+    const run = seedRun({ verdict: "trivial", rationale: "tiny", questions: [], parseFallback: false, consumed: false });
+    assert.equal(applyPlanGate(run), "auto_approve");
+    assert.equal(getRun(run.id).status, "plan_approved");
+    const approved = getLatestArtifact(run.id, "approved_plan");
+    assert.equal(approved.author, "system");
+    assert.match(approved.contentJson, /tiny/);
+});
+test("questions await answers and leave status untouched", () => {
+    const run = seedRun({ verdict: "needs_review", rationale: "r", questions: QUESTIONS, parseFallback: false, consumed: false });
+    assert.equal(applyPlanGate(run), "await_answers");
+    assert.equal(getRun(run.id).status, "plan_pending");
+});
+test("third question round goes to manual review", () => {
+    const run = seedRun({ verdict: "needs_review", rationale: "r1", questions: QUESTIONS, parseFallback: false, consumed: false });
+    addArtifact(run.id, "plan_triage", { verdict: "needs_review", rationale: "r2", questions: QUESTIONS, parseFallback: false, consumed: false }, "agent");
+    addArtifact(run.id, "plan_triage", { verdict: "needs_review", rationale: "r3", questions: QUESTIONS, parseFallback: false, consumed: false }, "agent");
+    assert.equal(applyPlanGate(getRun(run.id)), "await_review");
+});
+test("snapshot exposes awaitingAnswerRuns, openQuestionCounts, autoApprovedRunIds", () => {
+    const snap = getSnapshot();
+    assert.equal(Array.isArray(snap.awaitingAnswerRuns), true);
+    assert.equal(typeof snap.openQuestionCounts, "object");
+    assert.equal(Array.isArray(snap.autoApprovedRunIds), true);
+});