@selesai/code 0.5.20 → 0.5.22

package/dist/extensions/question/batch.ts

@@ -0,0 +1,103 @@
+import type {
+	DraftQuestionState,
+	PreparedQuestion,
+	QuestionAnswer,
+	QuestionOption,
+	QuestionResponse,
+	RawQuestion,
+} from "./types.ts";
+
+function clean(value: unknown): string | undefined {
+	if (typeof value !== "string") return undefined;
+	const trimmed = value.trim();
+	return trimmed || undefined;
+}
+
+function prepareOptions(value: unknown, questionNumber: number): { options: QuestionOption[] } | { error: string } {
+	if (!Array.isArray(value) || value.length === 0) {
+		return { error: `Question ${questionNumber}: choice questions require at least one option.` };
+	}
+	const values = new Set<string>();
+	const options: QuestionOption[] = [];
+	for (let index = 0; index < value.length; index++) {
+		const raw = value[index] as { value?: unknown; label?: unknown; description?: unknown };
+		const optionValue = clean(raw?.value);
+		const label = clean(raw?.label);
+		if (!optionValue || !label) {
+			return { error: `Question ${questionNumber}, option ${index + 1}: value and label must be non-empty strings.` };
+		}
+		if (values.has(optionValue)) {
+			return { error: `Question ${questionNumber}: option values must be unique; duplicate value: ${optionValue}` };
+		}
+		values.add(optionValue);
+		options.push({ value: optionValue, label, description: clean(raw.description) });
+	}
+	return { options };
+}
+
+export function prepareQuestions(value: unknown): { questions: PreparedQuestion[] } | { error: string } {
+	if (!Array.isArray(value) || value.length === 0) return { error: "questions must contain at least one question." };
+
+	const ids = new Set<string>();
+	const questions: PreparedQuestion[] = [];
+	for (let index = 0; index < value.length; index++) {
+		const raw = value[index] as RawQuestion;
+		const number = index + 1;
+		const question = clean(raw?.question);
+		if (!question) return { error: `Question ${number} must include a non-empty question.` };
+
+		const id = clean(raw.id) ?? `q${number}`;
+		if (ids.has(id)) return { error: `Question ids must be unique; duplicate id: ${id}` };
+		ids.add(id);
+
+		const common = {
+			id,
+			label: clean(raw.label) ?? `Q${number}`,
+			question,
+			context: clean(raw.context),
+		};
+
+		if (raw.type === "text") {
+			questions.push({ ...common, type: "text", options: [], allowOther: false });
+			continue;
+		}
+		if (raw.type !== "select" && raw.type !== "multiselect") {
+			return { error: `Question ${number}: type must be select, multiselect, or text.` };
+		}
+		const preparedOptions = prepareOptions(raw.options, number);
+		if ("error" in preparedOptions) return preparedOptions;
+		questions.push({
+			...common,
+			type: raw.type,
+			options: preparedOptions.options,
+			allowOther: raw.allowOther ?? false,
+		});
+	}
+	return { questions };
+}
+
+export function saveDraftQuestion(
+	states: Map<string, DraftQuestionState>,
+	id: string,
+	response: QuestionResponse | null,
+	markBlankSkipped: boolean,
+): void {
+	if (response) {
+		states.set(id, { status: "answered", response });
+		return;
+	}
+	if (markBlankSkipped) states.set(id, { status: "skipped" });
+	else states.delete(id);
+}
+
+export function buildQuestionAnswers(
+	questions: PreparedQuestion[],
+	states: ReadonlyMap<string, DraftQuestionState>,
+): QuestionAnswer[] {
+	return questions.map((question) => {
+		const state = states.get(question.id);
+		if (!state) return { id: question.id, status: "unanswered" };
+		if (state.status !== "answered" || !state.response) return { id: question.id, status: "skipped" };
+		return { id: question.id, status: "answered", response: state.response };
+	});
+}

package/dist/extensions/question/constants.ts

@@ -3,7 +3,7 @@
 import { Key } from "@earendil-works/pi-tui";
 import type { ResolvedShortcut } from "./types.ts";
 
-export const QUESTION_VERSION = "1";
+export const QUESTION_VERSION = "2";
 
 export const SPLIT_PANE_MIN_WIDTH = 84;
 export const SPLIT_PANE_LEFT_MIN_WIDTH = 32;

package/dist/extensions/question/dialog-adapter.ts

@@ -8,7 +8,7 @@ import {
 	isCancelled,
 	parseDialogSelections,
 } from "./helpers.ts";
-import type { QuestionOption, QuestionResponse, ResolvedQuestionParams } from "./types.ts";
+import type { BatchAnswer, BatchedQuestion, QuestionOption, QuestionResponse, ResolvedQuestionParams } from "./types.ts";
 import type { FallbackProtocol, UIProtocol } from "./ui-protocol.ts";
 
 const FREEFORM_DIALOG_OPTION = "✏️ Type custom response...";
@@ -58,6 +58,28 @@ async function askViaDialogs(
 	return createSelectionResponse([selected], comment);
 }
 
+export async function askBatchViaDialogs(questions: BatchedQuestion[], protocol: UIProtocol, timeout?: number): Promise<BatchAnswer[] | null> {
+	const deadline = timeout && timeout > 0 ? Date.now() + timeout : undefined;
+	const answers: BatchAnswer[] = [];
+	for (const question of questions) {
+		const remaining = deadline === undefined ? undefined : deadline - Date.now();
+		if (remaining !== undefined && remaining <= 0) return null;
+		const response = await askViaDialogs(
+			protocol,
+			question.question,
+			question.context,
+			question.options,
+			question.allowMultiple,
+			question.allowFreeform,
+			question.allowComment,
+			remaining,
+		);
+		if (response === null) return null;
+		answers.push({ id: question.id, question: question.question, response });
+	}
+	return answers;
+}
+
 export const DialogFallback: FallbackProtocol = {
 	async ask(params, protocol) {
 		return askViaDialogs(

package/dist/extensions/question/helpers.ts

@@ -1,102 +1,58 @@
 // Pure helper functions for the question extension.
 
-import type { MarkdownTheme } from "@selesai/code";
+import type { PreparedQuestion, QuestionOption, QuestionResponse } from "./types.ts";
 
-import type { QuestionOption, QuestionResponse, RawOption } from "./types.ts";
-
-export function getOptionsFormatError(options: RawOption[] | undefined): string | null {
-	for (const option of options ?? []) {
-		if (typeof option !== "string") continue;
-		const text = option.trim();
-		if (!text.startsWith("[") && !text.startsWith("{")) continue;
-		try {
-			const parsed = JSON.parse(text);
-			const jsonOption = (value: unknown) => !!value && typeof value === "object" && typeof (value as { label?: unknown }).label === "string";
-			if (jsonOption(parsed) || (Array.isArray(parsed) && parsed.some(jsonOption))) {
-				return 'Invalid question.options: pass an array of strings or {label, description} objects, not a JSON-encoded string. Example: {"options":[{"label":"Yes","description":"..."}]}, not {"options":["[{\\"label\\":\\"Yes\\"}]"]}.';
-			}
-		} catch {}
-	}
-	return null;
-}
-
-export function normalizeOptions(options: RawOption[] | undefined): QuestionOption[] {
-	return (options ?? [])
-		.map((option) =>
-			typeof option === "string"
-				? { label: option }
-				: { label: option.label, description: option.description },
-		)
-		.filter((option) => option.label.trim().length > 0);
+export function trimText(value: string | null | undefined): string | undefined {
+	const trimmed = value?.trim();
+	return trimmed || undefined;
 }
 
 export function oneLine(value: unknown, max = 100): string {
 	const text = String(value ?? "").replace(/\s+/g, " ").trim();
-	return text.length > max ? `${text.slice(0, max - 1)}…` : text;
+	return text.length > max ? `${text.slice(0, Math.max(0, max - 1))}…` : text;
 }
 
-// ponytail: lazy import avoids hard dependency on pi-coding-agent at module load
-// time (it may not be resolvable in dev/test environments). The function is
-// only called at runtime when the TUI is active.
-let _getMarkdownTheme: (() => MarkdownTheme) | null | undefined;
-export function safeMarkdownTheme(): MarkdownTheme | undefined {
-	try {
-		if (_getMarkdownTheme === undefined) {
-			// Dynamic require avoids breaking module load when package is absent.
-			_getMarkdownTheme = (globalThis as any).require
-				? (globalThis as any).require("@selesai/code").getMarkdownTheme
-				: null;
-		}
-		if (!_getMarkdownTheme) return undefined;
-		const md = _getMarkdownTheme();
-		if (!md) return undefined;
-		md.bold("");
-		return md;
-	} catch {
-		return undefined;
-	}
+export function previewText(value: unknown, max = 160, maxLines = 2): string {
+	const lines = String(value ?? "").trim().split(/\r?\n/);
+	const limited = lines.slice(0, maxLines).join("\n");
+	const wasCut = lines.length > maxLines || limited.length > max;
+	const text = limited.length > max ? limited.slice(0, Math.max(0, max - 1)) : limited;
+	return wasCut ? `${text}…` : text;
 }
 
-export function createFreeformResponse(text: string | null | undefined): QuestionResponse | null {
-	const trimmed = text?.trim();
-	return trimmed ? { kind: "freeform", text: trimmed } : null;
+export function createTextResponse(text: string | null | undefined): QuestionResponse | null {
+	const normalized = trimText(text);
+	return normalized ? { kind: "text", text: normalized } : null;
 }
 
-export function createSelectionResponse(selections: string[], comment?: string | null): QuestionResponse | null {
-	const norm = selections.map((s) => s.trim()).filter(Boolean);
-	if (norm.length === 0) return null;
-	const c = comment?.trim();
-	return c ? { kind: "selection", selections: norm, comment: c } : { kind: "selection", selections: norm };
+export function createSelectionResponse(values: string[], otherText?: string | null): QuestionResponse | null {
+	const normalizedValues = values.map((value) => value.trim()).filter(Boolean);
+	const normalizedOther = trimText(otherText);
+	if (normalizedValues.length === 0 && !normalizedOther) return null;
+	return normalizedOther
+		? { kind: "selection", values: normalizedValues, otherText: normalizedOther }
+		: { kind: "selection", values: normalizedValues };
 }
 
 export function formatResponseSummary(response: QuestionResponse): string {
-	if (response.kind === "freeform") return response.text;
-	const selections = response.selections.join(", ");
-	return response.comment ? `${selections} — ${response.comment}` : selections;
+	if (response.kind === "text") return response.text;
+	return [...response.values, ...(response.otherText ? [response.otherText] : [])].join(", ");
+}
+
+export function formatQuestionResponse(question: PreparedQuestion, response: QuestionResponse): string {
+	if (response.kind === "text") return response.text;
+	const labels = response.values.map(
+		(value) => question.options.find((option) => option.value === value)?.label ?? value,
+	);
+	if (response.otherText) labels.push(`Other: ${response.otherText}`);
+	return labels.join(", ");
 }
 
 export function formatOptionsForMessage(options: QuestionOption[]): string {
 	return options
 		.map((option, index) => {
-			const desc = option.description ? ` — ${option.description}` : "";
-			return `${index + 1}. ${option.label}${desc}`;
+			const description = option.description ? ` — ${option.description}` : "";
+			return `${index + 1}. ${option.label}${description}`;
 		})
 		.join("\n");
 }
-
-export function buildCommentPrompt(prompt: string, selections: string[]): string {
-	const label = selections.length === 1 ? "Selected option" : "Selected options";
-	const lines = selections.map((selection) => `- ${selection}`).join("\n");
-	return `${prompt}\n\n${label}:\n${lines}`;
-}
-
-export function parseDialogSelections(input: string): string[] {
-	return input
-		.split(",")
-		.map((selection) => selection.trim())
-		.filter(Boolean);
-}
-
-export function isCancelled(value: unknown): value is null | undefined {
-	return value === null || value === undefined;
-}