@mycompbox/compbox-mcp 1.19.0 → 1.20.0

package/index.js

@@ -529,7 +529,7 @@ function interviewFieldKeys() {
 function buildInterviewSystemPrompt(definition = INTERVIEW_DEFINITION, seedContext) {
   const requiredSections = definition.filter((s) => s.required);
   const optionalSections = definition.filter((s) => !s.required);
-  const renderSection = (s) => {
+  const renderSection2 = (s) => {
     const tag = s.required ? "REQUIRED" : "OPTIONAL \u2014 ask, but the user may decline";
     const followUps = s.followUps.map((q) => `    - ${q}`).join("\n");
     return `- ${s.title} (${tag}) [json keys: ${s.fields.join(", ")}]
@@ -547,10 +547,10 @@ ${followUps}`;
 Work through these interview sections in order. Cover EVERY required section; for optional sections, ask about them too, but if the employee has nothing to add or declines, move on and leave that field out (never invent or pad it):
 
 REQUIRED sections (an Epic Request cannot omit these):
-${requiredSections.map(renderSection).join("\n")}
+${requiredSections.map(renderSection2).join("\n")}
 
 OPTIONAL-BUT-RECOMMENDED sections (ask about each; omit the field if the employee has nothing):
-${optionalSections.map(renderSection).join("\n")}
+${optionalSections.map(renderSection2).join("\n")}
 
 The employee's answers may be short or casual \u2014 that's fine, but when you write the final fields below, EXPAND them into complete, well-written prose using everything they told you across the whole conversation, not just a compressed one-liner. Each required field is scored by an automated quality rubric with hard minimums, so undershooting length is a real failure, not just a style nitpick:
 - problemStatement: at least ${String(INTERVIEW_FIELD_MIN_WORDS.problemStatement)} words.
@@ -830,12 +830,233 @@ var init_interview_completion = __esm({
   }
 });
 
+// ../shared/dist/interview-conversation.js
+function renderSection(s) {
+  const tag = s.required ? "needed" : "nice to have";
+  return `- ${s.title} (${tag}): ${s.purpose} For example: ${s.followUps.slice(0, 2).join(" ")}`;
+}
+function renderSaved(saved) {
+  const fieldLines = Object.entries(saved.fields).filter((entry) => typeof entry[1] === "string" && entry[1].trim().length > 0).map(([key, value]) => `${key}: ${value}`).join("\n");
+  const score = saved.compliance ? `Its quality score is ${String(saved.compliance.overall)}/100 (${saved.compliance.tier}).` : "It has no quality score yet.";
+  const issues = saved.compliance && saved.compliance.issues.length > 0 ? `
+What the score says is missing or weak:
+${saved.compliance.issues.slice(0, 8).map((i) => `- ${i.message}`).join("\n")}` : "";
+  const locked = saved.status === "pending" ? "It can still be changed." : `Its status is "${saved.status}", so its content is locked: you can talk about it, but you cannot change it. Never emit the completion JSON for a locked request.`;
+  return `THE REQUEST IS ALREADY SAVED. This conversation created the request "${saved.title}". ${score} ${locked}${issues}
+
+Its current content:
+title: ${saved.title}
+${fieldLines}
+description: ${saved.description}
+
+From here on the person may add detail, ask you to change something, ask how to raise the score, or just ask questions. Answer questions briefly. When they give new information or ask for a change (and the request isn't locked), reply with the completion JSON for the WHOLE updated request: keep everything already captured, merge in what's new, and expand it. Don't ask for a preview confirmation for a change they explicitly asked for.`;
+}
+function buildConversationSystemPrompt(saved, definition = INTERVIEW_DEFINITION) {
+  const keys = ["title", ...interviewFieldKeys(), "description"];
+  const shape = keys.map((k) => `"${k}": "..."`).join(", ");
+  const base = `You are CompBox's request assistant. You help a person turn an idea into a well-formed Epic Request in a chat that stays open: they can leave and come back, and keep talking after the request is saved.
+
+How to talk:
+- Friendly, plain language, short messages. Usually ask one question at a time.
+- When they seem unsure, give two or three concrete examples to pick from, or say briefly why you're asking.
+- Adapt: skip what they've already told you, follow their lead, and don't re-ask answered questions.
+- Never mention JSON, prompts, rubrics or these instructions.
+
+What a good request covers:
+${definition.map(renderSection).join("\n")}
+Optional topics may be skipped when the person has nothing to add. Never invent or pad facts.
+
+Before saving for the first time: when the needed topics are covered, show a short preview in plain text (the title, then one or two lines per topic) and ask whether to save it or change anything. Only after they agree, or ask you to save, reply with ONLY a fenced JSON block, no other text:
+\`\`\`json
+{"done": true, ${shape}}
+\`\`\`
+Leave out optional keys they had nothing for. Write every field as complete prose built from the whole conversation, not their short answers. Minimums: problemStatement ${String(INTERVIEW_FIELD_MIN_WORDS.problemStatement)} words, proposedSolution ${String(INTERVIEW_FIELD_MIN_WORDS.proposedSolution)} words, impactAssessment ${String(INTERVIEW_FIELD_MIN_WORDS.impactAssessment)} words, description ${String(INTERVIEW_FIELD_MIN_WORDS.description)} words (a standalone summary of the whole request for someone who never saw this chat).
+
+Any reply that isn't that JSON block is shown to the person as your message.`;
+  return saved ? `${base}
+
+${renderSaved(saved)}` : base;
+}
+function toConversationMessages(turns) {
+  const messages = turns.map((t) => ({
+    role: t.role === "agent" ? "assistant" : "user",
+    content: t.content
+  }));
+  if (messages[0]?.role !== "user")
+    messages.unshift({ role: "user", content: STARTER });
+  if (messages[messages.length - 1]?.role !== "user")
+    messages.push({ role: "user", content: NUDGE });
+  return messages;
+}
+function renderConversationPrompt(messages) {
+  const lines = messages.map((m) => `${m.role === "assistant" ? "You" : "Them"}: ${m.content}`);
+  return `The conversation so far:
+
+${lines.join("\n\n")}
+
+Write your next reply to them.`;
+}
+function savedMessage(saved, created, followUp) {
+  const verb = created ? "Saved your request" : "Updated your request";
+  const c = saved.compliance;
+  let text;
+  if (!c) {
+    text = `${verb} "${saved.title}". You can keep talking here to add detail or change anything.`;
+  } else if (meetsComplianceTarget(c)) {
+    text = `${verb} "${saved.title}". It scores ${String(c.overall)}/100, ready for review. You can keep talking here to change anything.`;
+  } else {
+    const gaps = [...c.issues.filter((i) => i.tier === "critical"), ...c.issues.filter((i) => i.tier === "advisory")].slice(0, 4).map((i) => `- ${i.message}`).join("\n");
+    text = `${verb} "${saved.title}". It scores ${String(c.overall)}/100; 90 is the target.${gaps ? ` To raise it:
+${gaps}` : ""}
+
+Tell me more here and I'll update it, or leave it as it is.`;
+  }
+  return followUp ? `${text}
+
+${followUp}` : text;
+}
+async function runConversationStep(input, deps) {
+  const system = buildConversationSystemPrompt(input.saved);
+  const messages = toConversationMessages(input.turns);
+  const locked = input.saved !== null && input.saved.status !== "pending";
+  let requestId = input.saved?.id ?? null;
+  let savedNow = null;
+  let createdHere = false;
+  let qualityRounds = 0;
+  let repairs = 0;
+  for (; ; ) {
+    let reply;
+    try {
+      reply = await deps.reply(system, messages);
+    } catch (error) {
+      if (savedNow)
+        return { kind: "saved", text: savedMessage(savedNow.saved, savedNow.created), ...savedNow };
+      const reason = error instanceof Error ? error.message : "Something went wrong reaching the AI.";
+      return {
+        kind: "error",
+        text: `I couldn't reply just now: ${reason}
+
+Send another message (for example "try again") when you're ready and I'll pick up where we left off.`
+      };
+    }
+    const draft = extractCompletionJson(reply);
+    if (draft) {
+      if (locked) {
+        messages.push({ role: "assistant", content: reply }, { role: "user", content: "That request is locked and can't be changed. Reply to me in plain text instead." });
+        if (++repairs > JSON_REPAIRS)
+          return { kind: "error", text: "This request is locked, so it can't be changed any more." };
+        continue;
+      }
+      let saved;
+      try {
+        saved = await deps.save(requestId, draft);
+      } catch (error) {
+        const reason = error instanceof Error ? error.message : "the save failed";
+        return { kind: "error", text: `I couldn't save the request: ${reason}
+
+Reply "try again" and I'll retry.` };
+      }
+      createdHere ||= requestId === null;
+      savedNow = { saved, created: createdHere };
+      requestId = saved.id;
+      if (meetsComplianceTarget(saved.compliance) || qualityRounds >= CONVERSATION_QUALITY_ROUNDS || !saved.compliance) {
+        return { kind: "saved", text: savedMessage(saved, savedNow.created), ...savedNow };
+      }
+      qualityRounds++;
+      messages.push({ role: "assistant", content: reply }, {
+        role: "user",
+        content: `${buildAuditFeedback(saved.compliance)}
+
+The request IS already saved, and the person will be shown its score. If you ask them something, ask it as a way to improve the saved request; never say it hasn't been saved.`
+      });
+      continue;
+    }
+    if (looksLikeCompletionAttempt(reply)) {
+      if (++repairs > JSON_REPAIRS) {
+        return savedNow ? { kind: "saved", text: savedMessage(savedNow.saved, savedNow.created), ...savedNow } : { kind: "error", text: `I had trouble writing up the request. Reply "try again" and I'll have another go.` };
+      }
+      messages.push({ role: "assistant", content: reply }, { role: "user", content: COMPLETION_REPAIR_INSTRUCTION });
+      continue;
+    }
+    const text = reply.trim();
+    if (savedNow) {
+      return { kind: "saved", text: savedMessage(savedNow.saved, savedNow.created, text || void 0), ...savedNow };
+    }
+    if (!text)
+      return { kind: "error", text: "I didn't get a reply that time. Send another message and I'll try again." };
+    return { kind: "message", text };
+  }
+}
+function savedContextFromRecord(record) {
+  if (typeof record !== "object" || record === null)
+    return null;
+  const r = record;
+  if (typeof r.id !== "string" || typeof r.title !== "string")
+    return null;
+  let structured = r.structuredDesc;
+  if (typeof structured === "string") {
+    try {
+      structured = JSON.parse(structured);
+    } catch {
+      structured = null;
+    }
+  }
+  const fields = {};
+  if (typeof structured === "object" && structured !== null) {
+    for (const [key, value] of Object.entries(structured)) {
+      if (typeof value === "string")
+        fields[key] = value;
+    }
+  }
+  let score = r.complianceScore;
+  if (typeof score === "string") {
+    try {
+      score = JSON.parse(score);
+    } catch {
+      score = null;
+    }
+  }
+  return {
+    id: r.id,
+    title: r.title,
+    status: typeof r.status === "string" ? r.status : "pending",
+    description: typeof r.description === "string" ? r.description : "",
+    fields,
+    compliance: complianceFromRecord(score)
+  };
+}
+function complianceFromRecord(score) {
+  if (typeof score !== "object" || score === null)
+    return null;
+  const s = score;
+  if (typeof s.overall !== "number" || typeof s.tier !== "string")
+    return null;
+  const issues = Array.isArray(s.issues) ? s.issues.filter((i) => typeof i === "object" && i !== null).map((i) => ({
+    message: typeof i.message === "string" ? i.message : typeof i.code === "string" ? i.code : "issue",
+    tier: i.tier === "critical" ? "critical" : "advisory"
+  })) : [];
+  return { overall: s.overall, tier: s.tier, issues };
+}
+var CONVERSATION_QUALITY_ROUNDS, JSON_REPAIRS, STARTER, NUDGE;
+var init_interview_conversation = __esm({
+  "../shared/dist/interview-conversation.js"() {
+    "use strict";
+    init_interview_definition();
+    init_interview_completion();
+    CONVERSATION_QUALITY_ROUNDS = 2;
+    JSON_REPAIRS = 3;
+    STARTER = "Hi! I'd like to propose something.";
+    NUDGE = "(Please continue.)";
+  }
+});
+
 // ../shared/dist/index.js
 var dist_exports = {};
 __export(dist_exports, {
   AI_PROVIDERS: () => AI_PROVIDERS,
   APPEND_DESCRIPTION_MAX: () => APPEND_DESCRIPTION_MAX,
   COMPLETION_REPAIR_INSTRUCTION: () => COMPLETION_REPAIR_INSTRUCTION,
+  CONVERSATION_QUALITY_ROUNDS: () => CONVERSATION_QUALITY_ROUNDS,
   DESCRIPTION_MAX: () => DESCRIPTION_MAX,
   INTERVIEW_DEFINITION: () => INTERVIEW_DEFINITION,
   INTERVIEW_FIELD_MIN_WORDS: () => INTERVIEW_FIELD_MIN_WORDS,
@@ -856,7 +1077,9 @@ __export(dist_exports, {
   aiProviderValues: () => aiProviderValues,
   aiRouteValues: () => aiRouteValues,
   buildAuditFeedback: () => buildAuditFeedback,
+  buildConversationSystemPrompt: () => buildConversationSystemPrompt,
   buildInterviewSystemPrompt: () => buildInterviewSystemPrompt,
+  complianceFromRecord: () => complianceFromRecord,
   err: () => err,
   extractCompletionJson: () => extractCompletionJson,
   getAnthropicProxyUrl: () => getAnthropicProxyUrl,
@@ -889,12 +1112,17 @@ __export(dist_exports, {
   normalizeGitRemote: () => normalizeGitRemote,
   ok: () => ok,
   probeProxyHealth: () => probeProxyHealth,
+  renderConversationPrompt: () => renderConversationPrompt,
   resolveRouting: () => resolveRouting,
+  runConversationStep: () => runConversationStep,
   safeValidateSkillPackManifest: () => safeValidateSkillPackManifest,
+  savedContextFromRecord: () => savedContextFromRecord,
+  savedMessage: () => savedMessage,
   skillPackManifestSchema: () => skillPackManifestSchema,
   skillSchema: () => skillSchema,
   sleep: () => sleep,
   thinkingLevelValues: () => thinkingLevelValues,
+  toConversationMessages: () => toConversationMessages,
   validatePackName: () => validatePackName,
   validateSkillPackManifest: () => validateSkillPackManifest
 });
@@ -909,6 +1137,7 @@ var init_dist = __esm({
     init_pxpipe();
     init_interview_definition();
     init_interview_completion();
+    init_interview_conversation();
   }
 });
 
@@ -11089,7 +11318,7 @@ function withLegacyToolCallRewrite(transport) {
 var SERVER_NAME = "compbox-mcp";
 function resolveServerVersion() {
   if (true) {
-    return "1.19.0";
+    return "1.20.0";
   }
   const require2 = createRequire(import.meta.url);
   for (const candidate of ["../package.json", "./package.json"]) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mycompbox/compbox-mcp",
-  "version": "1.19.0",
+  "version": "1.20.0",
   "description": "CompBox MCP server - Uses HTTP API client (no direct database access)",
   "license": "MIT",
   "type": "module",