@productbrain/mcp 0.0.1-beta.4250 → 0.0.1-beta.4262

package/dist/{chunk-3NAU25R5.js → chunk-WGTVDUBC.js}

@@ -42,7 +42,7 @@ import {
   trackSessionCaptureRate,
   trackWriteBackHintServed,
   trackZeroCaptureAuditFired
-} from "./chunk-WTGHEQHB.js";
+} from "./chunk-SD33RK7E.js";
 
 // src/server.ts
 import { McpServer as McpServer2 } from "@modelcontextprotocol/sdk/server/mcp.js";
@@ -8874,24 +8874,96 @@ async function loadSkillBody(entryId) {
 }
 
 // src/lib/sessionNotices.ts
+var ANSI_AND_CONTROL_RE = /\x1b\[[0-9;]*[A-Za-z]|\x1b\][^\x07]*\x07|\x1b[^[\]0-9;A-Za-z]?|[\x00-\x08\x0b\x0c\x0e-\x1f\x7f]/g;
+function safe(str) {
+  return str ? str.replace(ANSI_AND_CONTROL_RE, "") : "";
+}
 var FEEDBACK_SUBMIT_HINT = "Friction or ideas? Use `feedback action=submit` \u2014 describe how pb behaved, not your workspace content. Once screening completes, it enters the builders' triage queue.";
-function nextActionForVerb(verb) {
+function nextActionForVerb(verb, args) {
   switch (verb) {
     case "feedback.list":
       return { tool: "feedback", description: "Read your workspace's own feedback (all statuses)", parameters: { action: "list" } };
     case "feedback.queue":
       return { tool: "feedback", description: "Read the vendor triage queue (system admins only)", parameters: { action: "queue" } };
+    case "question.create":
+      return { tool: "question", description: "Create a routed question", parameters: { action: "create" } };
+    case "question.adopt":
+      return { tool: "question", description: "Attach routing to a captured tension", parameters: { action: "adopt" } };
+    case "question.assign":
+      return { tool: "question", description: "Confirm or reassign a question", parameters: { action: "assign" } };
+    case "question.snooze":
+      return { tool: "question", description: "Snooze a routed/escalated question", parameters: { action: "snooze" } };
+    case "question.decline":
+      return { tool: "question", description: "Pass on or decline a question", parameters: { action: "decline" } };
+    case "question.answer":
+      return { tool: "question", description: "Answer a question", parameters: { action: "answer" } };
+    case "question.forceClose":
+      return { tool: "question", description: "Team-Lead-only force-close", parameters: { action: "force-close" } };
+    case "question.list":
+      return { tool: "question", description: args.mine === true ? "List every question you asked" : "List open questions", parameters: { action: "list", ...args.mine === true ? { mine: true } : {} } };
     default:
       return null;
   }
 }
 function noticesToNextActions(notices) {
   if (!notices || notices.length === 0) return [];
-  return notices.map((n) => nextActionForVerb(n.action.verb)).filter((a) => a !== null);
+  const actions2 = notices.map((n) => nextActionForVerb(n.action.verb, n.action.args)).filter((a) => a !== null);
+  const seen = /* @__PURE__ */ new Set();
+  return actions2.filter((a) => {
+    const key = `${a.tool}:${JSON.stringify(a.parameters)}`;
+    if (seen.has(key)) return false;
+    seen.add(key);
+    return true;
+  });
+}
+var QUESTION_ACK_GATE_SOURCE_ID = "question-ack-gate";
+function displayVerbFromRoute(route) {
+  switch (route) {
+    case "question.assign":
+      return "take it / reassign";
+    case "question.decline":
+      return "pass / decline";
+    case "question.forceClose":
+      return "force-close";
+    default:
+      return route.replace(/^question\./, "");
+  }
+}
+function formatQuestionBrief(q) {
+  const lines = [];
+  lines.push(`  [${safe(q.sourcePointer)}] ${safe(q.name)} \u2014 ${safe(q.whyYou)}, ${q.ageDays}d`);
+  lines.push(`    "${safe(q.question)}" \u2014 asked by ${safe(q.asker)}${q.domain ? `, ${safe(q.domain)} domain` : ""}`);
+  if (q.dependents.count > 0) {
+    const names = q.dependents.named.map((d) => `${safe(d.name)} (${safe(d.id)})`).join(", ");
+    const countLabel = q.dependents.truncated ? `${q.dependents.count}+` : String(q.dependents.count);
+    lines.push(`    blocks ${countLabel} item(s)${names ? `: ${names}` : ""}`);
+  }
+  if (q.declineContext) lines.push(`    declined: ${safe(q.declineContext)}`);
+  if (q.answerContext) lines.push(`    ${safe(q.answerContext)}`);
+  const verbList = [...new Set(q.actions.map((a) => safe(a.label) || displayVerbFromRoute(a.verb)))].join(" \xB7 ") || "answer";
+  lines.push(`    ${verbList}  ->  question action=answer entryId=${safe(q.sourcePointer)} text="<your answer>"`);
+  return lines;
+}
+function formatQuestionAckGateBlock(notice) {
+  const questions = notice.payload?.questions ?? [];
+  const lines = [];
+  lines.push(`- ${questions.length} open question${questions.length === 1 ? "" : "s"} need you before you continue.`);
+  for (const q of questions) lines.push(...formatQuestionBrief(q));
+  lines.push("  Deal with each question above before proceeding.");
+  if (notice.payload?.truncated) {
+    lines.push("  (This list may be incomplete \u2014 you may have more open questions than shown.)");
+  }
+  return lines;
 }
 function formatNoticeLines(notices) {
   if (!notices || notices.length === 0) return [];
-  return notices.map((n) => `- ${n.subject} (${n.change})`);
+  return notices.flatMap(
+    (n) => n.source === QUESTION_ACK_GATE_SOURCE_ID && n.payload ? formatQuestionAckGateBlock(n) : [`- ${safe(n.subject)} (${safe(n.change)})`]
+  );
+}
+function formatAdoptDiscoveryLine(display) {
+  if (!display) return [];
+  return [`- ${display} open question${display === "1" ? "" : "s"} you raised have no owner yet \u2014 use \`question action=adopt\` to route one.`];
 }
 
 // src/tools/start_pb.ts
@@ -13316,6 +13388,10 @@ async function handleStart2() {
   if (noticeLines.length > 0) {
     lines.push("", "Notices:", ...noticeLines);
   }
+  const adoptLines = formatAdoptDiscoveryLine(result2.adoptDiscoveryHint);
+  if (adoptLines.length > 0) {
+    lines.push("", ...adoptLines);
+  }
   const summary = `Session ${result2.sessionId} active. Workspace ${result2.workspaceName}. Write tools available after orient.`;
   return {
     content: [{ type: "text", text: lines.join("\n") }],
@@ -13326,7 +13402,8 @@ async function handleStart2() {
       toolsScope: result2.toolsScope,
       feedbackQueueNew: result2.feedbackQueueNew,
       feedbackOldestNewAt: result2.feedbackOldestNewAt,
-      notices: result2.notices ?? []
+      notices: result2.notices ?? [],
+      adoptDiscoveryHint: result2.adoptDiscoveryHint
       // Finding #16: bare `parameters: {}` lets orient's schema default kick in
       // (action defaults to "task", which requires a `task` string) — the suggested
       // next-action call failed validation instead of running. `action: "start"` is
@@ -15811,19 +15888,20 @@ function registerShapeTools(server) {
 
 // src/tools/question.ts
 import { z as z34 } from "zod/v3";
-var QUESTION_ACTIONS = ["create", "adopt", "assign", "snooze", "decline", "answer", "force-close"];
+var QUESTION_ACTIONS = ["create", "adopt", "assign", "snooze", "decline", "answer", "force-close", "list"];
 var questionSchema = z34.object({
   action: z34.enum(QUESTION_ACTIONS).describe(
-    "'create': create a new routed question from text (domains come from `pb question --help` / this tool's own refusal \u2014 never hardcode a slug). 'adopt': attach routing to a tension already captured (e.g. via `pb capture \"TEN: ...\"`) \u2014 the pilot's real birth path; without adopt, captured tensions stay invisible to routing forever. 'assign': confirm ('take it', omit toRole/toPerson) or reassign (supply toRole and/or toPerson) ownership. 'snooze': defer a routed/escalated question (max 2, then it escalates). 'decline': pass on a suggested question, or (as owner/lead) decline an owned one \u2014 reason required. 'answer': close the loop \u2014 creates the answering entry itself unless --entry is given. 'force-close': Team-Lead-only escape hatch \u2014 reason required."
+    "'create': create a new routed question from text (domains come from `pb question --help` / this tool's own refusal \u2014 never hardcode a slug). 'adopt': attach routing to a tension already captured (e.g. via `pb capture \"TEN: ...\"`) \u2014 the pilot's real birth path; without adopt, captured tensions stay invisible to routing forever. 'assign': confirm ('take it', omit toRole/toPerson) or reassign (supply toRole and/or toPerson) ownership. 'snooze': defer a routed/escalated question (max 2, then it escalates). 'decline': pass on a suggested question, or (as owner/lead) decline an owned one \u2014 reason required. 'answer': close the loop \u2014 creates the answering entry itself unless --entry is given. 'force-close': Team-Lead-only escape hatch \u2014 reason required. 'list': the ack-gate read model \u2014 what you owe (or, with mine:true, every question you asked, including closed ones)."
   ),
-  entryId: z34.string().max(200).optional().describe("Target question's id (human TEN-nnn form or a raw entry id). Required for every action except 'create'."),
+  entryId: z34.string().max(200).optional().describe("Target question's id (human TEN-nnn form or a raw entry id). Required for every action except 'create'/'list'."),
   text: z34.string().max(5e3).optional().describe("'create': the question text. 'answer': the answer text (mutually exclusive with answerEntryId, max 2000 chars)."),
   domain: z34.string().max(200).optional().describe("'create'/'adopt': the domain slug (see this tool's refusal for the live list)."),
   blocks: z34.array(z34.string().max(200)).max(50).optional().describe("'create'/'adopt': entry ids this question is blocking (repeatable)."),
   toRole: z34.string().max(200).optional().describe("'assign': reassign to this role (human ROL-nnn form or a raw entry id)."),
   toPerson: z34.string().max(200).optional().describe("'assign': reassign to this person (a raw person id)."),
   reason: z34.string().max(2e3).optional().describe("'decline'/'force-close': required reason."),
-  answerEntryId: z34.string().max(200).optional().describe("'answer': point at an already-written entry instead of supplying text (mutually exclusive with text).")
+  answerEntryId: z34.string().max(200).optional().describe("'answer': point at an already-written entry instead of supplying text (mutually exclusive with text)."),
+  mine: z34.boolean().optional().describe("'list': true for arm C (every question YOU asked, every status). Omit/false for the ack-gate view (what you owe).")
 });
 var createVariant = z34.object({ action: z34.literal("create"), text: z34.string().min(1).max(5e3), domain: z34.string().max(200).optional(), blocks: z34.array(z34.string().max(200)).max(50).optional() });
 var adoptVariant = z34.object({ action: z34.literal("adopt"), entryId: z34.string().max(200), domain: z34.string().max(200).optional(), blocks: z34.array(z34.string().max(200)).max(50).optional() });
@@ -15832,6 +15910,7 @@ var snoozeVariant = z34.object({ action: z34.literal("snooze"), entryId: z34.str
 var declineVariant = z34.object({ action: z34.literal("decline"), entryId: z34.string().max(200), reason: z34.string().min(1).max(2e3) });
 var answerVariant = z34.object({ action: z34.literal("answer"), entryId: z34.string().max(200), text: z34.string().max(2e3).optional(), answerEntryId: z34.string().max(200).optional() });
 var forceCloseVariant = z34.object({ action: z34.literal("force-close"), entryId: z34.string().max(200), reason: z34.string().min(1).max(2e3) });
+var listVariant = z34.object({ action: z34.literal("list"), mine: z34.boolean().optional() });
 var questionActionUnion = z34.discriminatedUnion("action", [
   createVariant,
   adoptVariant,
@@ -15839,7 +15918,8 @@ var questionActionUnion = z34.discriminatedUnion("action", [
   snoozeVariant,
   declineVariant,
   answerVariant,
-  forceCloseVariant
+  forceCloseVariant,
+  listVariant
 ]);
 var QUESTION_ACTION_SPECS = {
   create: { params: ["text", "domain", "blocks"], description: "text is required; domain/blocks are optional." },
@@ -15848,7 +15928,8 @@ var QUESTION_ACTION_SPECS = {
   snooze: { params: ["entryId"], description: "entryId is required." },
   decline: { params: ["entryId", "reason"], description: "entryId and reason are both required." },
   answer: { params: ["entryId", "text", "answerEntryId"], description: "entryId is required; supply exactly one of text or answerEntryId." },
-  "force-close": { params: ["entryId", "reason"], description: "entryId and reason are both required." }
+  "force-close": { params: ["entryId", "reason"], description: "entryId and reason are both required." },
+  list: { params: ["mine"], description: "mine is optional \u2014 omit/false for the ack-gate view, true for every question you asked." }
 };
 var HUMAN_ID_PATTERN = /^[A-Za-z]+-\d+$/;
 async function resolveEntryRef(raw, label) {
@@ -15987,7 +16068,7 @@ async function handleQuestionAnswer(entryId, text, answerEntryId) {
     const result2 = await kernelMutation("question.answer", { entryId: resolvedEntryId, text, answerEntryId: resolvedAnswerEntryId });
     const askerName = result2.askerName ?? "The asker";
     const pendingNote = result2.answerPending ? " The answer is a draft awaiting accept." : "";
-    const message = result2.idempotent ? `${entryId} is already answered \u2014 no change.` : `Answered. ${askerName} will see the outcome at their next session.${pendingNote}`;
+    const message = result2.idempotent ? result2.processedVia === "declined" || result2.processedVia === "force_closed" ? `${entryId} is already processed (${result2.processedVia === "declined" ? "declined" : "force-closed"}) \u2014 no change.` : `${entryId} is already answered \u2014 no change.` : `Answered. ${askerName} will see the outcome at their next session (and in pb question list --mine).${pendingNote}`;
     return successResult(message, message, result2);
   } catch (err) {
     return questionErrorResult(err, "answer", { entryId, text, answerEntryId });
@@ -16004,15 +16085,37 @@ async function handleQuestionForceClose(entryId, reason) {
     return questionErrorResult(err, "force-close", { entryId, reason });
   }
 }
+async function handleQuestionList(mine) {
+  try {
+    const result2 = await kernelQuery("question.list", { mine });
+    const rows = result2.questions;
+    if (rows.length === 0) {
+      const message2 = mine ? "You haven't asked any questions yet." : "No open questions.";
+      return successResult(message2, message2, { ...result2, action: "list" });
+    }
+    const lines = rows.map((r) => {
+      const outcomeSuffix = mine && r.outcome ? ` \u2014 ${r.outcome}` : "";
+      return `[${r.entryId}] ${r.question} \u2014 ${r.stateLabel} \u2014 ${r.why} \u2014 ${r.domain} \u2014 ${r.withLabel} \u2014 ${r.ageDays}d${outcomeSuffix}`;
+    });
+    const truncationNote = result2.truncated ? `
+(${mine ? "showing your most recent questions only \u2014 older ones exist but are not shown" : "this list may be incomplete \u2014 some obligations may not be shown"})` : "";
+    const message = `${rows.length} question${rows.length === 1 ? "" : "s"}:
+${lines.join("\n")}${truncationNote}`;
+    return successResult(message, message, { ...result2, action: "list" });
+  } catch (err) {
+    return questionErrorResult(err, "list", { mine });
+  }
+}
 function questionErrorResult(err, action, params = {}) {
   const classified = classifyError(err);
   const definedParams = Object.fromEntries(Object.entries(params).filter(([, v]) => v !== void 0));
+  const fallbackActions = classified.code === "VALIDATION_ERROR" ? void 0 : [{ tool: "question", description: `Retry ${action}`, parameters: { action, ...definedParams } }];
   return failureResult(
     classified.message,
     classified.code,
     classified.message,
     classified.recovery ?? "Use action='create' with no --domain, or `pb question --help` / this tool's description, to find valid domains and next steps.",
-    classified.availableActions ?? [{ tool: "question", description: `Retry ${action}`, parameters: { action, ...definedParams } }],
+    classified.availableActions ?? fallbackActions,
     classified.diagnostics
   );
 }
@@ -16024,13 +16127,14 @@ function registerQuestionTools(server) {
     snooze: (data) => handleQuestionSnooze(data.entryId),
     decline: (data) => handleQuestionDecline(data.entryId, data.reason),
     answer: (data) => handleQuestionAnswer(data.entryId, data.text, data.answerEntryId),
-    "force-close": (data) => handleQuestionForceClose(data.entryId, data.reason)
+    "force-close": (data) => handleQuestionForceClose(data.entryId, data.reason),
+    list: (data) => handleQuestionList(data.mine)
   };
   server.registerTool(
     "question",
     {
       title: "Question",
-      description: "Routed questions \u2014 a question with no owner routes itself to the owning role, the team lead when ambiguous, and up the team-lead chain until someone owns it. Seven write actions:\n\n- **create**: create a new routed question from text. Domains are discovered by using this tool \u2014 an unresolvable `domain` refusal lists the workspace's real slugs; there is no fixed list to memorize.\n- **adopt**: attach routing to a tension already captured (e.g. via `capture` with `collection: tensions`) \u2014 the real birth path for most questions. Without adopt, a captured tension stays invisible to routing forever.\n- **assign**: confirm ('take it', no `toRole`/`toPerson`) or reassign (`toRole`/`toPerson`) ownership.\n- **snooze**: defer a routed/escalated question \u2014 max 2, then it escalates to the team lead.\n- **decline**: pass on a suggested question (never closes it), or \u2014 as the owner/team lead \u2014 decline an owned one (reason required; closes it, or escalates it, depending on dependents).\n- **answer**: close the loop. Creates the answering entry itself from `text`, or points at an existing one via `answerEntryId` \u2014 never both.\n- **force-close**: Team-Lead-only escape hatch (reason required) for a question that cannot otherwise be resolved.\n\n`blocks` (repeatable, on create/adopt) records what a question is holding up \u2014 pass entry ids it blocks.",
+      description: "Routed questions \u2014 a question with no owner routes itself to the owning role, the team lead when ambiguous, and up the team-lead chain until someone owns it. Eight actions:\n\n- **create**: create a new routed question from text. Domains are discovered by using this tool \u2014 an unresolvable `domain` refusal lists the workspace's real slugs; there is no fixed list to memorize.\n- **adopt**: attach routing to a tension already captured (e.g. via `capture` with `collection: tensions`) \u2014 the real birth path for most questions. Without adopt, a captured tension stays invisible to routing forever.\n- **assign**: confirm ('take it', no `toRole`/`toPerson`) or reassign (`toRole`/`toPerson`) ownership.\n- **snooze**: defer a routed/escalated question \u2014 max 2, then it escalates to the team lead.\n- **decline**: pass on a suggested question (never closes it), or \u2014 as the owner/team lead \u2014 decline an owned one (reason required; closes it, or escalates it, depending on dependents).\n- **answer**: close the loop. Creates the answering entry itself from `text`, or points at an existing one via `answerEntryId` \u2014 never both.\n- **force-close**: Team-Lead-only escape hatch (reason required) for a question that cannot otherwise be resolved.\n- **list**: the ack-gate read model \u2014 what you owe (`mine:false`, the default), or (`mine:true`) every question you asked, including closed ones, with its outcome.\n\n`blocks` (repeatable, on create/adopt) records what a question is holding up \u2014 pass entry ids it blocks.",
       inputSchema: questionSchema,
       annotations: { readOnlyHint: false, idempotentHint: false, openWorldHint: false }
     },
@@ -16978,4 +17082,4 @@ export {
   createProductBrainServer,
   initFeatureFlags
 };
-//# sourceMappingURL=chunk-3NAU25R5.js.map
+//# sourceMappingURL=chunk-WGTVDUBC.js.map