@longtable/mcp 0.1.23 → 0.1.25

package/README.md

@@ -14,7 +14,7 @@ longtable-state
 Run:
 
 ```bash
-npx -y @longtable/mcp@0.1.22
+npx -y @longtable/mcp@0.1.25
 ```
 
 Self-test:
@@ -31,3 +31,6 @@ longtable mcp install --provider codex --checkpoint-ui strong --write
 
 If MCP elicitation is unavailable or not approved, the server returns the same
 pending `QuestionRecord` as a numbered fallback.
+
+Provider guidance should use `elicit_question` first when the MCP tool is
+available. `longtable question --print` is only the CLI fallback transport.

package/dist/server.js

@@ -1,4 +1,4 @@
-import { readFile } from "node:fs/promises";
+import { readFile, writeFile } from "node:fs/promises";
 import { existsSync } from "node:fs";
 import { resolve } from "node:path";
 import { cwd, exit } from "node:process";
@@ -11,7 +11,7 @@ import { renderQuestionRecordInput } from "@longtable/provider-claude";
 import { renderQuestionRecordPrompt } from "@longtable/provider-codex";
 import { answerWorkspaceQuestion, createWorkspaceQuestion, inspectProjectWorkspace, loadProjectContextFromDirectory, loadWorkspaceState, syncCurrentWorkspaceView } from "@longtable/cli";
 const SERVER_NAME = "longtable-state";
-const SERVER_VERSION = "0.1.22";
+const SERVER_VERSION = "0.1.25";
 const TOOL_NAMES = [
     "read_project",
     "read_session",
@@ -27,6 +27,12 @@ const TOOL_NAMES = [
 const cwdSchema = z.object({
     cwd: z.string().optional().describe("LongTable project directory or child path. Defaults to server cwd.")
 });
+const questionOptionSchema = z.object({
+    value: z.string().min(1),
+    label: z.string().min(1),
+    description: z.string().optional(),
+    recommended: z.boolean().optional()
+});
 function textResult(structuredContent) {
     return {
         content: [
@@ -70,6 +76,29 @@ function renderQuestionFallback(record, provider = "codex") {
         ? renderQuestionRecordInput(record)
         : renderQuestionRecordPrompt(record);
 }
+async function markQuestionTransport(context, questionId, status, message) {
+    const state = await loadWorkspaceState(context);
+    let updatedQuestion = null;
+    state.questionLog = (state.questionLog ?? []).map((record) => {
+        if (record.id !== questionId) {
+            return record;
+        }
+        updatedQuestion = {
+            ...record,
+            updatedAt: new Date().toISOString(),
+            transportStatus: {
+                surface: "mcp_elicitation",
+                status,
+                updatedAt: new Date().toISOString(),
+                ...(message ? { message } : {})
+            }
+        };
+        return updatedQuestion;
+    });
+    await writeFile(context.stateFilePath, JSON.stringify(state, null, 2), "utf8");
+    await syncCurrentWorkspaceView(context);
+    return updatedQuestion;
+}
 function buildElicitationParams(record) {
     const choices = [
         ...record.prompt.options.map((option) => ({
@@ -91,8 +120,8 @@ function buildElicitationParams(record) {
         message: [
             record.prompt.title,
             record.prompt.question,
-            ...record.prompt.rationale.slice(0, 2).map((entry) => `Why now: ${entry}`)
-        ].join("\n"),
+            record.prompt.displayReason ? `Decision context: ${record.prompt.displayReason}` : undefined
+        ].filter(Boolean).join("\n"),
         requestedSchema: {
             type: "object",
             properties: {
@@ -101,11 +130,6 @@ function buildElicitationParams(record) {
                     title: "Decision",
                     oneOf: choices,
                     default: choices[0]?.const
-                },
-                rationale: {
-                    type: "string",
-                    title: "Rationale",
-                    description: "Optional short note for the LongTable decision log."
                 }
             },
             required: ["answer"]
@@ -120,12 +144,20 @@ function acceptedAnswer(result) {
     if (typeof answer !== "string" || answer.length === 0) {
         return null;
     }
-    const rationale = result.content?.rationale;
     return {
-        answer,
-        ...(typeof rationale === "string" && rationale.length > 0 ? { rationale } : {})
+        answer
     };
 }
+function statusForElicitationError(error) {
+    const message = error instanceof Error ? error.message : String(error);
+    if (/timed?\s*out|timeout/i.test(message)) {
+        return "timeout";
+    }
+    if (/unsupported|not supported|unavailable|does not support/i.test(message)) {
+        return "unsupported";
+    }
+    return "error";
+}
 async function readAllowedProjectFiles(context) {
     const current = existsSync(context.currentFilePath)
         ? await readFile(context.currentFilePath, "utf8")
@@ -255,10 +287,13 @@ export function createLongTableMcpServer() {
             prompt: z.string().min(1),
             title: z.string().optional(),
             question: z.string().optional(),
+            checkpointKey: z.string().optional(),
+            options: z.array(questionOptionSchema).optional(),
+            displayReason: z.string().optional(),
             provider: z.enum(["codex", "claude"]).optional(),
             required: z.boolean().optional()
         })
-    }, async ({ cwd: inputCwd, prompt, title, question, provider, required }) => {
+    }, async ({ cwd: inputCwd, prompt, title, question, checkpointKey, options, displayReason, provider, required }) => {
         try {
             const context = await requireContext(inputCwd);
             const result = await createWorkspaceQuestion({
@@ -266,6 +301,9 @@ export function createLongTableMcpServer() {
                 prompt,
                 title,
                 question,
+                checkpointKey,
+                questionOptions: options,
+                displayReason,
                 provider,
                 required
             });
@@ -285,11 +323,14 @@ export function createLongTableMcpServer() {
             prompt: z.string().min(1),
             title: z.string().optional(),
             question: z.string().optional(),
+            checkpointKey: z.string().optional(),
+            options: z.array(questionOptionSchema).optional(),
+            displayReason: z.string().optional(),
             provider: z.enum(["codex", "claude"]).default("codex"),
             required: z.boolean().optional(),
             fallbackOnly: z.boolean().default(false).describe("Create and render the checkpoint without calling MCP elicitation.")
         })
-    }, async ({ cwd: inputCwd, prompt, title, question, provider, required, fallbackOnly }) => {
+    }, async ({ cwd: inputCwd, prompt, title, question, checkpointKey, options, displayReason, provider, required, fallbackOnly }) => {
         try {
             const context = await requireContext(inputCwd);
             const created = await createWorkspaceQuestion({
@@ -297,24 +338,33 @@ export function createLongTableMcpServer() {
                 prompt,
                 title,
                 question,
+                checkpointKey,
+                questionOptions: options,
+                displayReason,
                 provider,
                 required
             });
             const fallback = renderQuestionFallback(created.question, provider);
             if (fallbackOnly) {
+                const marked = await markQuestionTransport(context, created.question.id, "fallback_rendered", "MCP elicitation skipped by fallbackOnly.");
                 return textResult({
-                    question: created.question,
+                    question: marked ?? created.question,
                     elicitation: { attempted: false, reason: "fallbackOnly" },
                     fallback,
                     nextAction: `longtable decide --question ${created.question.id} --answer <value>`
                 });
             }
             try {
+                await markQuestionTransport(context, created.question.id, "attempted");
                 const elicited = await server.server.elicitInput(buildElicitationParams(created.question));
                 const accepted = acceptedAnswer(elicited);
                 if (!accepted) {
+                    const status = elicited.action === "decline" || elicited.action === "cancel"
+                        ? "declined"
+                        : "fallback_rendered";
+                    const marked = await markQuestionTransport(context, created.question.id, status, `MCP elicitation returned action: ${elicited.action}.`);
                     return textResult({
-                        question: created.question,
+                        question: marked ?? created.question,
                         elicitation: { attempted: true, action: elicited.action },
                         fallback,
                         nextAction: `longtable decide --question ${created.question.id} --answer <value>`
@@ -324,22 +374,26 @@ export function createLongTableMcpServer() {
                     context,
                     questionId: created.question.id,
                     answer: accepted.answer,
-                    rationale: accepted.rationale,
-                    provider: provider
+                    provider: provider,
+                    surface: "mcp_elicitation"
                 });
+                const marked = await markQuestionTransport(context, created.question.id, "accepted");
                 return textResult({
-                    question: decided.question,
+                    question: marked ? { ...decided.question, transportStatus: marked.transportStatus } : decided.question,
                     decision: decided.decision,
                     elicitation: { attempted: true, action: elicited.action }
                 });
             }
             catch (elicitationError) {
+                const status = statusForElicitationError(elicitationError);
+                const message = elicitationError instanceof Error ? elicitationError.message : String(elicitationError);
+                const marked = await markQuestionTransport(context, created.question.id, status, message);
                 return textResult({
-                    question: created.question,
+                    question: marked ?? created.question,
                     elicitation: {
                         attempted: true,
-                        supported: false,
-                        error: elicitationError instanceof Error ? elicitationError.message : String(elicitationError)
+                        supported: status !== "unsupported" ? undefined : false,
+                        error: message
                     },
                     fallback,
                     nextAction: `longtable decide --question ${created.question.id} --answer <value>`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@longtable/mcp",
-  "version": "0.1.23",
+  "version": "0.1.25",
   "private": false,
   "description": "LongTable MCP transport for workspace state and Researcher Checkpoints",
   "type": "module",
@@ -26,11 +26,11 @@
     "self-test": "node ./dist/server.js --self-test"
   },
   "dependencies": {
-    "@longtable/checkpoints": "0.1.23",
-    "@longtable/cli": "0.1.23",
-    "@longtable/core": "0.1.23",
-    "@longtable/provider-claude": "0.1.23",
-    "@longtable/provider-codex": "0.1.23",
+    "@longtable/checkpoints": "0.1.25",
+    "@longtable/cli": "0.1.25",
+    "@longtable/core": "0.1.25",
+    "@longtable/provider-claude": "0.1.25",
+    "@longtable/provider-codex": "0.1.25",
     "@modelcontextprotocol/sdk": "^1.29.0",
     "zod": "^4.0.0"
   },