@longtable/mcp 0.1.33 → 0.1.34

package/README.md

@@ -15,7 +15,7 @@ longtable-state
 Run:
 
 ```bash
-npx -y @longtable/mcp@0.1.33
+npx -y @longtable/mcp@0.1.34
 ```
 
 Self-test:

package/dist/first-research-shape.js

@@ -9,7 +9,7 @@ function questionTitle(shape) {
 }
 function questionText(shape) {
     if (shape.protectedDecision) {
-        return `Before LongTable moves forward, what should stay explicitly open about ${shape.protectedDecision}?`;
+        return "Before LongTable moves forward, what protected research decision should stay explicit?";
     }
     if (shape.currentBlocker) {
         return "What should LongTable do with the main unresolved issue in this emerging study?";
@@ -49,8 +49,8 @@ function baseOptions(shape) {
     if (shape.protectedDecision) {
         options.push({
             value: "protect_decision",
-            label: `Keep ${shape.protectedDecision} open`,
-            description: "Treat this as the guarded judgment while the broader direction stays provisional.",
+            label: "Keep the protected decision open",
+            description: `Treat this as the guarded judgment while the broader direction stays provisional: ${shape.protectedDecision}`,
             recommended: recommended === "protect_decision"
         });
     }

package/dist/server.js

@@ -10,10 +10,10 @@ import { classifyCheckpointTrigger } from "@longtable/checkpoints";
 import { renderQuestionRecordInput } from "@longtable/provider-claude";
 import { renderQuestionRecordPrompt } from "@longtable/provider-codex";
 import { loadSetupOutput } from "@longtable/setup";
-import { answerWorkspaceQuestion, createOrUpdateProjectWorkspace, createWorkspaceQuestion, inspectProjectWorkspace, loadProjectContextFromDirectory, loadWorkspaceState, syncCurrentWorkspaceView } from "@longtable/cli";
+import { answerWorkspaceQuestion, clearWorkspaceQuestion, createOrUpdateProjectWorkspace, createWorkspaceQuestion, inspectProjectWorkspace, loadProjectContextFromDirectory, loadWorkspaceState, syncCurrentWorkspaceView } from "@longtable/cli";
 import { buildFirstResearchShapeQuestion, firstResearchShapeAnswerConfirms, firstResearchShapeAnswerStatus } from "./first-research-shape.js";
 const SERVER_NAME = "longtable-state";
-const SERVER_VERSION = "0.1.33";
+const SERVER_VERSION = "0.1.34";
 const TOOL_NAMES = [
     "read_project",
     "read_session",
@@ -198,6 +198,19 @@ async function beginInterviewHook(context, options) {
     if (existing) {
         return { hook: existing, state };
     }
+    const confirmedShape = state.firstResearchShape?.confirmedAt ? state.firstResearchShape : undefined;
+    if (confirmedShape) {
+        const confirmedHook = [...(state.hooks ?? [])].reverse().find((hook) => isInterviewHookRun(hook) &&
+            hook.status === "confirmed" &&
+            hook.firstResearchShape?.handle === confirmedShape.handle);
+        return {
+            hook: confirmedHook,
+            state,
+            alreadyConfirmed: true,
+            shape: confirmedShape,
+            nextQuestion: options.openingQuestion ?? confirmedShape.openQuestions[0] ?? confirmedShape.nextAction
+        };
+    }
     const timestamp = new Date().toISOString();
     const hook = {
         id: createId("hook_interview"),
@@ -329,15 +342,10 @@ async function summarizeInterviewHook(context, options) {
         visibility: "explicit",
         importance: shape.confidence
     });
-    const questionSpec = buildFirstResearchShapeQuestion(shape);
-    const withObligation = ensureFirstResearchShapeObligation(updated, shape, {
-        prompt: questionSpec.question,
-        reason: questionSpec.displayReason
-    });
     await writeFile(context.sessionFilePath, JSON.stringify(session, null, 2), "utf8");
-    await writeFile(context.stateFilePath, JSON.stringify(withObligation, null, 2), "utf8");
+    await writeFile(context.stateFilePath, JSON.stringify(updated, null, 2), "utf8");
     await syncCurrentWorkspaceView(context);
-    return { hook, shape, state: withObligation, session };
+    return { hook, shape, state: updated, session };
 }
 function findQuestion(records, questionId) {
     if (questionId) {
@@ -350,6 +358,22 @@ function renderQuestionFallback(record, provider = "codex") {
         ? renderQuestionRecordInput(record)
         : renderQuestionRecordPrompt(record);
 }
+function compactInterviewHook(hook) {
+    if (!hook) {
+        return undefined;
+    }
+    return {
+        id: hook.id,
+        kind: hook.kind,
+        status: hook.status,
+        depth: hook.depth,
+        provider: hook.provider,
+        turnCount: hook.turns?.length ?? 0,
+        firstResearchShape: hook.firstResearchShape?.handle,
+        createdAt: hook.createdAt,
+        updatedAt: hook.updatedAt
+    };
+}
 async function markQuestionTransport(context, questionId, status, message) {
     const state = asInterviewState(await loadWorkspaceState(context));
     let updatedQuestion = null;
@@ -467,6 +491,46 @@ async function markFirstResearchShapeConfirmation(context, shape, answer, questi
     await syncCurrentWorkspaceView(context);
     return { state: nextState, session, shape: confirmedShape };
 }
+async function markAlreadyConfirmedFirstResearchShape(context, shape) {
+    const state = asInterviewState(await loadWorkspaceState(context));
+    const timestamp = new Date().toISOString();
+    const confirmedShape = shape.confirmedAt ? shape : { ...shape, confirmedAt: timestamp };
+    state.firstResearchShape = confirmedShape;
+    state.workingState = {
+        ...state.workingState,
+        firstResearchShape: confirmedShape
+    };
+    state.hooks = (state.hooks ?? []).map((hook) => {
+        if (!isInterviewHookRun(hook)) {
+            return hook;
+        }
+        const matchesSource = shape.sourceHookId && hook.id === shape.sourceHookId;
+        const matchesHandle = !shape.sourceHookId && hook.firstResearchShape?.handle === shape.handle;
+        if (!matchesSource && !matchesHandle) {
+            return hook;
+        }
+        return {
+            ...hook,
+            status: "confirmed",
+            updatedAt: timestamp,
+            firstResearchShape: confirmedShape
+        };
+    });
+    const nextState = resolveFirstResearchShapeObligation(state, {
+        sourceHookId: confirmedShape.sourceHookId,
+        status: "satisfied"
+    });
+    const session = {
+        ...context.session,
+        firstResearchShape: confirmedShape,
+        lastUpdatedAt: timestamp
+    };
+    context.session = session;
+    await writeFile(context.sessionFilePath, JSON.stringify(session, null, 2), "utf8");
+    await writeFile(context.stateFilePath, JSON.stringify(nextState, null, 2), "utf8");
+    await syncCurrentWorkspaceView(context);
+    return { state: nextState, session, shape: confirmedShape };
+}
 function statusForElicitationError(error) {
     const message = error instanceof Error ? error.message : String(error);
     if (/timed?\s*out|timeout/i.test(message)) {
@@ -617,7 +681,18 @@ export function createLongTableMcpServer() {
                 openingQuestion,
                 seedAnswer
             });
-            return textResult(result);
+            return textResult({
+                hook: compactInterviewHook(result.hook),
+                alreadyConfirmed: "alreadyConfirmed" in result ? Boolean(result.alreadyConfirmed) : false,
+                shape: "shape" in result ? result.shape : result.hook?.firstResearchShape,
+                nextQuestion: "nextQuestion" in result ? result.nextQuestion : openingQuestion,
+                workspace: {
+                    projectName: context.project.projectName,
+                    currentGoal: context.session.currentGoal,
+                    currentBlocker: context.session.currentBlocker,
+                    nextAction: context.session.nextAction
+                }
+            });
         }
         catch (error) {
             return errorResult(error instanceof Error ? error.message : String(error));
@@ -669,7 +744,16 @@ export function createLongTableMcpServer() {
                 hookId,
                 shape: shape
             });
-            return textResult(result);
+            return textResult({
+                hook: compactInterviewHook(result.hook),
+                shape: result.shape,
+                session: {
+                    currentGoal: result.session.currentGoal,
+                    currentBlocker: result.session.currentBlocker,
+                    nextAction: result.session.nextAction,
+                    firstResearchShape: result.session.firstResearchShape
+                }
+            });
         }
         catch (error) {
             return errorResult(error instanceof Error ? error.message : String(error));
@@ -691,6 +775,13 @@ export function createLongTableMcpServer() {
             if (!shape) {
                 return errorResult("No First Research Shape was found to confirm. Run summarize_interview first.");
             }
+            if (shape.confirmedAt) {
+                const confirmation = await markAlreadyConfirmedFirstResearchShape(context, shape);
+                return textResult({
+                    shape: confirmation.shape,
+                    elicitation: { attempted: false, reason: "already_confirmed" }
+                });
+            }
             const spec = buildFirstResearchShapeQuestion(shape);
             const created = await createWorkspaceQuestion({
                 context,
@@ -729,8 +820,15 @@ export function createLongTableMcpServer() {
                         ? "declined"
                         : "fallback_rendered";
                     const marked = await markQuestionTransport(context, created.question.id, status, `MCP elicitation returned action: ${elicited.action}.`);
+                    const cleared = status === "declined"
+                        ? await clearWorkspaceQuestion({
+                            context,
+                            questionId: created.question.id,
+                            reason: `MCP elicitation returned action: ${elicited.action}; confirmation was deferred without a research-direction decision.`
+                        })
+                        : undefined;
                     return textResult({
-                        question: marked ?? created.question,
+                        question: cleared?.question ?? marked ?? created.question,
                         shape,
                         elicitation: { attempted: true, action: elicited.action },
                         fallback

package/package.json

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