@longtable/cli 0.1.57 → 0.1.59

package/dist/project-session.js

@@ -499,96 +499,12 @@ function formatQuestionMetadata(record) {
 }
 const QUESTION_SURFACES = new Set([
     "native_structured",
+    "tmux_popup",
     "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;
@@ -598,6 +514,15 @@ function asRecord(value) {
         ? value
         : null;
 }
+function asStringArray(value) {
+    if (Array.isArray(value)) {
+        return value.filter((entry) => typeof entry === "string" && entry.trim().length > 0);
+    }
+    return typeof value === "string" && value.trim().length > 0 ? [value] : [];
+}
+function isQuestionSurfaceValue(value) {
+    return typeof value === "string" && QUESTION_SURFACES.has(value);
+}
 const SPEC_DIFF_IGNORED_PATHS = new Set([
     "createdAt",
     "updatedAt",
@@ -694,10 +619,10 @@ function buildResearchSpecificationGapQuestion(gaps, timestamp, sourceEvidenceId
                 "Research Specification is the required durable interview artifact.",
                 "Missing required sections can make later resume, screening, coding, or evidence decisions stale."
             ],
-            preferredSurfaces: ["mcp_elicitation", "numbered"]
+            preferredSurfaces: ["tmux_popup", "mcp_elicitation", "numbered"]
         },
         transportStatus: {
-            surface: "mcp_elicitation",
+            surface: "tmux_popup",
             status: "not_attempted",
             updatedAt: timestamp,
             ...(sourceEvidenceIds.length > 0 ? { message: `Source evidence: ${sourceEvidenceIds.join(", ")}` } : {})
@@ -946,10 +871,22 @@ function summarizeWorkspaceInspection(context, state) {
             ...(record.selectedOption ? { selectedOption: record.selectedOption } : {}),
             timestamp: record.timestamp
         })),
-        answerWarnings: [
-            ...(legacyAnswerShapeWarnings(questions) ?? []),
-            ...(numericOtherAnswerWarnings(questions) ?? [])
-        ]
+        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.` } : {})
+                }];
+        })
     };
 }
 function buildProjectAgentsMd(project, session) {
@@ -2762,7 +2699,7 @@ export async function createWorkspaceFollowUpQuestions(options) {
     const createdAt = nowIso();
     const preferredSurfaces = options.provider === "claude"
         ? ["native_structured", "terminal_selector", "numbered"]
-        : ["mcp_elicitation", "terminal_selector", "numbered"];
+        : ["tmux_popup", "mcp_elicitation", "terminal_selector", "numbered"];
     const specs = buildQuestionOpportunitySpecs(options.prompt, {
         includeFallback: options.force === true ? true : options.auto !== true,
         autoOnly: options.auto === true,
@@ -2875,7 +2812,7 @@ export async function createWorkspaceQuestion(options) {
             rationale,
             preferredSurfaces: options.provider === "claude"
                 ? ["native_structured", "numbered"]
-                : ["mcp_elicitation", "numbered"]
+                : ["tmux_popup", "mcp_elicitation", "numbered"]
         }
     };
     const updated = appendQuestionRecords(state, [question]);
@@ -3078,6 +3015,15 @@ function normalizeQuestionAnswerSelection(question, rawAnswer) {
         ...(inlineRationale ? { inlineRationale } : {})
     };
 }
+function selectedLabelsForValues(question, selectedValues) {
+    return selectedValues.map((value) => {
+        if (value === "other") {
+            return question.prompt.otherLabel ?? "Other";
+        }
+        const option = question.prompt.options.find((candidate) => candidate.value === value);
+        return option?.label ?? value;
+    });
+}
 export async function answerWorkspaceQuestion(options) {
     const state = await loadResearchState(options.context.stateFilePath);
     const question = findQuestionForDecision(state, options.questionId);
@@ -3116,7 +3062,12 @@ export async function answerWorkspaceQuestion(options) {
         updatedAt: timestamp,
         status: "answered",
         answer,
-        decisionRecordId: decision.id
+        decisionRecordId: decision.id,
+        transportStatus: {
+            surface: answer.surface,
+            status: "accepted",
+            updatedAt: timestamp
+        }
     };
     const withQuestion = {
         ...state,
@@ -3322,41 +3273,49 @@ export async function repairWorkspaceStateConsistency(options) {
             })
         };
     }
-    const timestamp = nowIso();
+    const repairTimestamp = nowIso();
     const repairedQuestionLog = (updated.questionLog ?? []).map((record) => {
-        if (record.status !== "answered") {
-            return record;
-        }
-        const answer = legacyQuestionAnswerRecord(record);
-        if (!answer) {
+        const answerRecord = asRecord(record.answer);
+        if (!answerRecord) {
             return record;
         }
-        const selectedValues = selectedValuesForQuestion(record);
+        const selectedValues = [
+            ...asStringArray(answerRecord.selectedValues),
+            ...asStringArray(answerRecord.selectedValue),
+            ...asStringArray(answerRecord.selectedOptions),
+            ...asStringArray(answerRecord.value)
+        ];
         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);
+        const selectedLabels = asStringArray(answerRecord.selectedLabels);
+        const needsRepair = typeof answerRecord.promptId !== "string" ||
+            selectedLabels.length === 0 ||
+            !isQuestionSurfaceValue(answerRecord.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"
+            promptId: typeof answerRecord.promptId === "string" ? answerRecord.promptId : record.prompt.id,
+            selectedValues: uniqueStrings(selectedValues),
+            selectedLabels: selectedLabels.length > 0
+                ? selectedLabels
+                : selectedLabelsForValues(record, uniqueStrings(selectedValues)),
+            ...(typeof answerRecord.otherText === "string" && answerRecord.otherText.trim()
+                ? { otherText: answerRecord.otherText }
+                : {}),
+            ...(typeof answerRecord.rationale === "string" && answerRecord.rationale.trim()
+                ? { rationale: answerRecord.rationale }
+                : {}),
+            ...(answerRecord.provider === "codex" || answerRecord.provider === "claude"
+                ? { provider: answerRecord.provider }
+                : {}),
+            surface: isQuestionSurfaceValue(answerRecord.surface) ? answerRecord.surface : "numbered"
         };
         return {
             ...record,
-            updatedAt: timestamp,
+            updatedAt: repairTimestamp,
             answer: normalizedAnswer
         };
     });
@@ -3366,7 +3325,7 @@ export async function repairWorkspaceStateConsistency(options) {
             questionLog: repairedQuestionLog
         };
     }
-    if (repaired.length > 0 && !options.dryRun) {
+    if (repaired.length > 0) {
         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.57",
+  "version": "0.1.59",
   "private": false,
   "description": "Researcher-facing LongTable CLI",
   "type": "module",
@@ -29,12 +29,12 @@
   },
   "dependencies": {
     "@clack/prompts": "^1.2.0",
-    "@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"
+    "@longtable/checkpoints": "0.1.59",
+    "@longtable/core": "0.1.59",
+    "@longtable/memory": "0.1.59",
+    "@longtable/provider-claude": "0.1.59",
+    "@longtable/provider-codex": "0.1.59",
+    "@longtable/setup": "0.1.59"
   },
   "devDependencies": {
     "@types/node": "^22.10.1",