@longtable/cli 0.1.56 → 0.1.57

package/dist/cli.js

@@ -159,6 +159,7 @@ function usage() {
         "  longtable clarify --prompt <task-context> [--provider codex|claude] [--required|--advisory] [--print] [--cwd <path>] [--json] [--force]",
         "  longtable question --prompt <decision-context> [--title <text>] [--text <question>] [--provider codex|claude] [--required|--advisory] [--print] [--cwd <path>] [--json]",
         "  longtable clear-question --question <id> --reason <text> [--cwd <path>] [--json]",
+        "  longtable repair-state [--cwd <path>] [--dry-run] [--json]",
         "  longtable panel [--prompt <text>] [--role <role[,role]>] [--mode review|critique|draft|commit] [--visibility synthesis_only|show_on_conflict|always_visible] [--provider codex|claude] [--native-workers|--native-subagents] [--wait [ms]] [--print] [--json] [--setup <path>] [--cwd <path>]",
         "  longtable panel status --run <panel_run_id> [--wait [ms]] [--cwd <path>] [--json]",
         "  longtable panel stop --run <panel_run_id> [--cwd <path>] [--json]",
@@ -200,7 +201,7 @@ function parseArgs(argv) {
     const values = {};
     let subcommand = maybeSubcommand;
     const modeCommand = command && VALID_MODES.has(command);
-    const directCommand = command && ["init", "setup", "start", "resume", "doctor", "status", "audit", "roles", "show", "install", "mcp", "codex", "claude", "ask", "clarify", "question", "clear-question", "prune-questions", "panel", "handoff", "decide", "sentinel", "access", "search", "spec"].includes(command);
+    const directCommand = command && ["init", "setup", "start", "resume", "doctor", "status", "audit", "roles", "show", "install", "mcp", "codex", "claude", "ask", "clarify", "question", "clear-question", "repair-state", "prune-questions", "panel", "handoff", "decide", "sentinel", "access", "search", "spec"].includes(command);
     let startIndex = 1;
     if (modeCommand) {
         subcommand = undefined;
@@ -1783,6 +1784,10 @@ function renderDoctorStatus(status) {
     if (!status.workspace.found) {
         nextActions.push("longtable start");
     }
+    if ((status.workspace.answerWarnings ?? []).some((warning) => warning.issue.includes("legacy answer shape"))) {
+        const root = status.workspace.rootPath ? ` --cwd "${status.workspace.rootPath}"` : "";
+        nextActions.push(`longtable repair-state${root}`);
+    }
     nextActions.push(...status.hardStop.nextActions);
     const firstQuestion = status.workspace.pendingQuestions?.[0];
     if (firstQuestion && status.hardStop.nextActions.length === 0) {
@@ -3431,6 +3436,39 @@ async function runClearQuestion(args) {
     console.log(`- state: ${context.stateFilePath}`);
     console.log(`- current: ${context.currentFilePath}`);
 }
+async function runRepairState(args) {
+    const workingDirectory = typeof args.cwd === "string" ? args.cwd : cwd();
+    const context = await loadProjectContextFromDirectory(workingDirectory);
+    if (!context) {
+        throw new Error("No LongTable project workspace was found here. Run this inside a project or pass --cwd.");
+    }
+    const result = await repairWorkspaceStateConsistency({
+        context,
+        dryRun: args["dry-run"] === true
+    });
+    if (args.json === true) {
+        console.log(JSON.stringify({
+            dryRun: args["dry-run"] === true,
+            repaired: result.repaired,
+            files: {
+                state: context.stateFilePath,
+                current: context.currentFilePath
+            }
+        }, null, 2));
+        return;
+    }
+    console.log(args["dry-run"] === true ? "LongTable state repair preview" : "LongTable state repaired");
+    if (result.repaired.length === 0) {
+        console.log("- no repairs needed");
+    }
+    else {
+        for (const item of result.repaired) {
+            console.log(`- ${item}`);
+        }
+    }
+    console.log(`- state: ${context.stateFilePath}`);
+    console.log(`- current: ${context.currentFilePath}`);
+}
 async function runPruneQuestions(args) {
     const workingDirectory = typeof args.cwd === "string" ? args.cwd : cwd();
     const context = await loadProjectContextFromDirectory(workingDirectory);
@@ -4278,6 +4316,10 @@ async function main() {
         await runClearQuestion(values);
         return;
     }
+    if (command === "repair-state") {
+        await runRepairState(values);
+        return;
+    }
     if (command === "prune-questions") {
         await runPruneQuestions(values);
         return;

package/dist/project-session.d.ts

@@ -490,6 +490,7 @@ export declare function pruneWorkspaceQuestions(options: {
 }>;
 export declare function repairWorkspaceStateConsistency(options: {
     context: LongTableProjectContext;
+    dryRun?: boolean;
 }): Promise<{
     state: ResearchState;
     repaired: string[];

package/dist/project-session.js

@@ -497,6 +497,98 @@ function formatQuestionMetadata(record) {
     ].filter(Boolean);
     return parts.length > 0 ? ` [${parts.join("; ")}]` : "";
 }
+const QUESTION_SURFACES = new Set([
+    "native_structured",
+    "mcp_elicitation",
+    "numbered",
+    "terminal_selector",
+    "web_form"
+]);
+function asStringArray(value) {
+    if (Array.isArray(value)) {
+        return value.filter((entry) => typeof entry === "string");
+    }
+    return typeof value === "string" ? [value] : [];
+}
+function legacyQuestionAnswerRecord(record) {
+    return asRecord(record.answer);
+}
+function labelForQuestionAnswerValue(record, value) {
+    const option = record.prompt.options.find((candidate) => candidate.value === value || candidate.label === value);
+    if (option) {
+        return option.label;
+    }
+    if (value === "other") {
+        return record.prompt.otherLabel ?? "Other";
+    }
+    return value;
+}
+function selectedValuesForQuestion(record) {
+    const answer = legacyQuestionAnswerRecord(record);
+    if (!answer) {
+        return [];
+    }
+    const directValues = asStringArray(answer.selectedValues);
+    if (directValues.length > 0) {
+        return directValues;
+    }
+    const legacyValues = [
+        ...asStringArray(answer.selectedValue),
+        ...asStringArray(answer.selected),
+        ...asStringArray(answer.selectedOption),
+        ...asStringArray(answer.answer),
+        ...asStringArray(answer.value)
+    ];
+    return uniqueStrings(legacyValues);
+}
+function selectedLabelsForQuestion(record, selectedValues) {
+    const answer = legacyQuestionAnswerRecord(record);
+    const directLabels = asStringArray(answer?.selectedLabels);
+    if (directLabels.length > 0) {
+        return directLabels;
+    }
+    return selectedValues.map((value) => labelForQuestionAnswerValue(record, value));
+}
+function legacyAnswerShapeWarnings(questions) {
+    return questions.flatMap((record) => {
+        if (record.status !== "answered" || !legacyQuestionAnswerRecord(record)) {
+            return [];
+        }
+        const selectedValues = asStringArray(legacyQuestionAnswerRecord(record)?.selectedValues);
+        const selectedLabels = asStringArray(legacyQuestionAnswerRecord(record)?.selectedLabels);
+        if (selectedValues.length > 0 && selectedLabels.length > 0) {
+            return [];
+        }
+        return [{
+                questionId: record.id,
+                ...(record.decisionRecordId ? { decisionRecordId: record.decisionRecordId } : {}),
+                issue: "Answered question uses a legacy answer shape that is missing selectedValues or selectedLabels.",
+                suggestion: "Run `longtable repair-state --cwd <project-path>` to normalize the answer without changing the recorded selection."
+            }];
+    });
+}
+function numericOtherAnswerWarnings(questions) {
+    return questions.flatMap((record) => {
+        if (record.status !== "answered" || !selectedValuesForQuestion(record).includes("other")) {
+            return [];
+        }
+        const answer = legacyQuestionAnswerRecord(record);
+        const raw = typeof answer?.otherText === "string"
+            ? answer.otherText
+            : selectedLabelsForQuestion(record, selectedValuesForQuestion(record))[0] ?? "";
+        if (!/^\d+$/.test(raw.trim())) {
+            return [];
+        }
+        const index = Number(raw.trim()) - 1;
+        const option = record.prompt.options[index];
+        return [{
+                questionId: record.id,
+                ...(record.decisionRecordId ? { decisionRecordId: record.decisionRecordId } : {}),
+                issue: `Numeric answer "${raw.trim()}" was stored as other text.`,
+                ...(option ? { suggestion: `Use "${option.value}" (${option.label}) for this checkpoint option.` } : {})
+            }];
+    });
+}
 function compactLine(value, limit = 160) {
     const compacted = value.replace(/\s+/g, " ").trim();
     return compacted.length > limit ? `${compacted.slice(0, limit - 1)}…` : compacted;
@@ -854,22 +946,10 @@ function summarizeWorkspaceInspection(context, state) {
             ...(record.selectedOption ? { selectedOption: record.selectedOption } : {}),
             timestamp: record.timestamp
         })),
-        answerWarnings: questions
-            .filter((record) => record.status === "answered" && record.answer?.selectedValues.includes("other"))
-            .flatMap((record) => {
-            const raw = record.answer?.otherText ?? record.answer?.selectedLabels[0] ?? "";
-            if (!/^\d+$/.test(raw.trim())) {
-                return [];
-            }
-            const index = Number(raw.trim()) - 1;
-            const option = record.prompt.options[index];
-            return [{
-                    questionId: record.id,
-                    ...(record.decisionRecordId ? { decisionRecordId: record.decisionRecordId } : {}),
-                    issue: `Numeric answer "${raw.trim()}" was stored as other text.`,
-                    ...(option ? { suggestion: `Use "${option.value}" (${option.label}) for this checkpoint option.` } : {})
-                }];
-        })
+        answerWarnings: [
+            ...(legacyAnswerShapeWarnings(questions) ?? []),
+            ...(numericOtherAnswerWarnings(questions) ?? [])
+        ]
     };
 }
 function buildProjectAgentsMd(project, session) {
@@ -3242,7 +3322,51 @@ export async function repairWorkspaceStateConsistency(options) {
             })
         };
     }
-    if (repaired.length > 0) {
+    const timestamp = nowIso();
+    const repairedQuestionLog = (updated.questionLog ?? []).map((record) => {
+        if (record.status !== "answered") {
+            return record;
+        }
+        const answer = legacyQuestionAnswerRecord(record);
+        if (!answer) {
+            return record;
+        }
+        const selectedValues = selectedValuesForQuestion(record);
+        if (selectedValues.length === 0) {
+            return record;
+        }
+        const selectedLabels = selectedLabelsForQuestion(record, selectedValues);
+        const needsRepair = asStringArray(answer.selectedValues).length === 0 ||
+            asStringArray(answer.selectedLabels).length === 0 ||
+            typeof answer.promptId !== "string" ||
+            !QUESTION_SURFACES.has(answer.surface);
+        if (!needsRepair) {
+            return record;
+        }
+        repaired.push(`normalized legacy answer shape for question ${record.id}`);
+        const normalizedAnswer = {
+            ...answer,
+            promptId: typeof answer.promptId === "string" ? answer.promptId : record.prompt.id,
+            selectedValues,
+            selectedLabels,
+            ...(typeof answer.otherText === "string" && answer.otherText.trim() ? { otherText: answer.otherText } : {}),
+            ...(typeof answer.rationale === "string" && answer.rationale.trim() ? { rationale: answer.rationale } : {}),
+            ...(answer.provider === "codex" || answer.provider === "claude" ? { provider: answer.provider } : {}),
+            surface: QUESTION_SURFACES.has(answer.surface) ? answer.surface : "numbered"
+        };
+        return {
+            ...record,
+            updatedAt: timestamp,
+            answer: normalizedAnswer
+        };
+    });
+    if (repairedQuestionLog.some((record, index) => record !== (updated.questionLog ?? [])[index])) {
+        updated = {
+            ...updated,
+            questionLog: repairedQuestionLog
+        };
+    }
+    if (repaired.length > 0 && !options.dryRun) {
         await writeFile(options.context.stateFilePath, JSON.stringify(updated, null, 2), "utf8");
         await syncCurrentWorkspaceView(options.context);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@longtable/cli",
-  "version": "0.1.56",
+  "version": "0.1.57",
   "private": false,
   "description": "Researcher-facing LongTable CLI",
   "type": "module",
@@ -29,12 +29,12 @@
   },
   "dependencies": {
     "@clack/prompts": "^1.2.0",
-    "@longtable/checkpoints": "0.1.56",
-    "@longtable/core": "0.1.56",
-    "@longtable/memory": "0.1.56",
-    "@longtable/provider-claude": "0.1.56",
-    "@longtable/provider-codex": "0.1.56",
-    "@longtable/setup": "0.1.56"
+    "@longtable/checkpoints": "0.1.57",
+    "@longtable/core": "0.1.57",
+    "@longtable/memory": "0.1.57",
+    "@longtable/provider-claude": "0.1.57",
+    "@longtable/provider-codex": "0.1.57",
+    "@longtable/setup": "0.1.57"
   },
   "devDependencies": {
     "@types/node": "^22.10.1",