taskplane 0.29.2 → 0.30.0

package/extensions/taskplane/settings-tui.ts

@@ -19,7 +19,14 @@
 
 import type { ExtensionContext } from "@mariozechner/pi-coding-agent";
 import { DynamicBorder, getSettingsListTheme } from "@mariozechner/pi-coding-agent";
-import { Container, type SelectItem, SelectList, type SettingItem, SettingsList, Text } from "@mariozechner/pi-tui";
+import {
+	Container,
+	type SelectItem,
+	SelectList,
+	type SettingItem,
+	SettingsList,
+	Text,
+} from "@mariozechner/pi-tui";
 import { readFileSync, writeFileSync, existsSync, mkdirSync, renameSync, unlinkSync } from "fs";
 import { join, dirname } from "path";
 import { parse as yamlParse } from "yaml";
@@ -40,7 +47,6 @@ import {
 } from "./config-loader.ts";
 import { loadPiSettingsPackages } from "./settings-loader.ts";
 
-
 // ── Types ────────────────────────────────────────────────────────────
 
 /** Source of a field's current value */
@@ -84,7 +90,6 @@ export interface SectionDef {
 	readOnly?: boolean;
 }
 
-
 // ── Section & Field Definitions ──────────────────────────────────────
 
 /**
@@ -95,114 +100,401 @@ export const SECTIONS: SectionDef[] = [
 	{
 		name: "Orchestrator",
 		fields: [
-			{ configPath: "orchestrator.orchestrator.maxLanes", label: "Max Lanes", control: "input", layer: "L1", fieldType: "number", description: "Maximum parallel execution lanes" },
-			{ configPath: "orchestrator.orchestrator.worktreeLocation", label: "Worktree Location", control: "toggle", layer: "L1", fieldType: "enum", values: ["sibling", "subdirectory"], description: "Where lane worktree directories are created" },
-			{ configPath: "orchestrator.orchestrator.worktreePrefix", label: "Worktree Prefix", control: "input", layer: "L1", fieldType: "string", description: "Prefix for worktree directory names" },
-			{ configPath: "orchestrator.orchestrator.batchIdFormat", label: "Batch ID Format", control: "toggle", layer: "L1", fieldType: "enum", values: ["timestamp", "sequential"], description: "Batch ID format for logs/branch naming" },
-			{ configPath: "orchestrator.orchestrator.sessionPrefix", label: "Session Prefix", control: "input", layer: "L1+L2", fieldType: "string", prefsKey: "sessionPrefix", description: "Prefix for orchestrator session names" },
-			{ configPath: "orchestrator.orchestrator.operatorId", label: "Operator ID", control: "input", layer: "L1+L2", fieldType: "string", prefsKey: "operatorId", description: "Operator identifier (empty = auto-detect)" },
-			{ configPath: "orchestrator.orchestrator.integration", label: "Integration", control: "picker", layer: "L1", fieldType: "enum", values: ["manual", "supervised", "auto"], description: "How completed batches are integrated. manual = user runs /orch-integrate. supervised = supervisor proposes plan, asks confirmation. auto = supervisor executes without asking." },
+			{
+				configPath: "orchestrator.orchestrator.maxLanes",
+				label: "Max Lanes",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Maximum parallel execution lanes",
+			},
+			{
+				configPath: "orchestrator.orchestrator.worktreeLocation",
+				label: "Worktree Location",
+				control: "toggle",
+				layer: "L1",
+				fieldType: "enum",
+				values: ["sibling", "subdirectory"],
+				description: "Where lane worktree directories are created",
+			},
+			{
+				configPath: "orchestrator.orchestrator.worktreePrefix",
+				label: "Worktree Prefix",
+				control: "input",
+				layer: "L1",
+				fieldType: "string",
+				description: "Prefix for worktree directory names",
+			},
+			{
+				configPath: "orchestrator.orchestrator.batchIdFormat",
+				label: "Batch ID Format",
+				control: "toggle",
+				layer: "L1",
+				fieldType: "enum",
+				values: ["timestamp", "sequential"],
+				description: "Batch ID format for logs/branch naming",
+			},
+			{
+				configPath: "orchestrator.orchestrator.sessionPrefix",
+				label: "Session Prefix",
+				control: "input",
+				layer: "L1+L2",
+				fieldType: "string",
+				prefsKey: "sessionPrefix",
+				description: "Prefix for orchestrator session names",
+			},
+			{
+				configPath: "orchestrator.orchestrator.operatorId",
+				label: "Operator ID",
+				control: "input",
+				layer: "L1+L2",
+				fieldType: "string",
+				prefsKey: "operatorId",
+				description: "Operator identifier (empty = auto-detect)",
+			},
+			{
+				configPath: "orchestrator.orchestrator.integration",
+				label: "Integration",
+				control: "picker",
+				layer: "L1",
+				fieldType: "enum",
+				values: ["manual", "supervised", "auto"],
+				description:
+					"How completed batches are integrated. manual = user runs /orch-integrate. supervised = supervisor proposes plan, asks confirmation. auto = supervisor executes without asking.",
+			},
 		],
 	},
 	{
 		name: "Agent: Supervisor",
 		fields: [
-			{ configPath: "orchestrator.supervisor.model", label: "Supervisor Model", control: "input", layer: "L1+L2", fieldType: "string", prefsKey: "supervisorModel", description: "Supervisor model (inherit = use session model)" },
-			{ configPath: "orchestrator.supervisor.autonomy", label: "Autonomy Level", control: "picker", layer: "L1", fieldType: "enum", values: ["interactive", "supervised", "autonomous"], description: "Recovery action confirmation behavior" },
+			{
+				configPath: "orchestrator.supervisor.model",
+				label: "Supervisor Model",
+				control: "input",
+				layer: "L1+L2",
+				fieldType: "string",
+				prefsKey: "supervisorModel",
+				description: "Supervisor model (inherit = use session model)",
+			},
+			{
+				configPath: "orchestrator.supervisor.autonomy",
+				label: "Autonomy Level",
+				control: "picker",
+				layer: "L1",
+				fieldType: "enum",
+				values: ["interactive", "supervised", "autonomous"],
+				description: "Recovery action confirmation behavior",
+			},
 		],
 	},
 	{
 		name: "Agent: Worker",
 		fields: [
-			{ configPath: "taskRunner.worker.model", label: "Worker Model", control: "input", layer: "L1+L2", fieldType: "string", prefsKey: "workerModel", description: "Worker model (inherit = use session model)" },
-			{ configPath: "taskRunner.worker.tools", label: "Worker Tools", control: "input", layer: "L1", fieldType: "string", description: "Worker tool allowlist" },
-			{ configPath: "taskRunner.worker.thinking", label: "Worker Thinking", control: "picker", layer: "L1", fieldType: "string", description: "Worker thinking mode" },
+			{
+				configPath: "taskRunner.worker.model",
+				label: "Worker Model",
+				control: "input",
+				layer: "L1+L2",
+				fieldType: "string",
+				prefsKey: "workerModel",
+				description: "Worker model (inherit = use session model)",
+			},
+			{
+				configPath: "taskRunner.worker.tools",
+				label: "Worker Tools",
+				control: "input",
+				layer: "L1",
+				fieldType: "string",
+				description: "Worker tool allowlist",
+			},
+			{
+				configPath: "taskRunner.worker.thinking",
+				label: "Worker Thinking",
+				control: "picker",
+				layer: "L1",
+				fieldType: "string",
+				description: "Worker thinking mode",
+			},
 		],
 	},
 	{
 		name: "Agent: Reviewer",
 		fields: [
-			{ configPath: "taskRunner.reviewer.model", label: "Reviewer Model", control: "input", layer: "L1+L2", fieldType: "string", prefsKey: "reviewerModel", description: "Reviewer model (inherit = use session model)" },
-			{ configPath: "taskRunner.reviewer.tools", label: "Reviewer Tools", control: "input", layer: "L1", fieldType: "string", description: "Reviewer tool allowlist" },
-			{ configPath: "taskRunner.reviewer.thinking", label: "Reviewer Thinking", control: "picker", layer: "L1", fieldType: "string", description: "Reviewer thinking mode" },
+			{
+				configPath: "taskRunner.reviewer.model",
+				label: "Reviewer Model",
+				control: "input",
+				layer: "L1+L2",
+				fieldType: "string",
+				prefsKey: "reviewerModel",
+				description: "Reviewer model (inherit = use session model)",
+			},
+			{
+				configPath: "taskRunner.reviewer.tools",
+				label: "Reviewer Tools",
+				control: "input",
+				layer: "L1",
+				fieldType: "string",
+				description: "Reviewer tool allowlist",
+			},
+			{
+				configPath: "taskRunner.reviewer.thinking",
+				label: "Reviewer Thinking",
+				control: "picker",
+				layer: "L1",
+				fieldType: "string",
+				description: "Reviewer thinking mode",
+			},
 		],
 	},
 	{
 		name: "Agent: Merge",
 		fields: [
-			{ configPath: "orchestrator.merge.model", label: "Merge Model", control: "input", layer: "L1+L2", fieldType: "string", prefsKey: "mergeModel", description: "Merge-agent model (inherit = use session model)" },
-			{ configPath: "orchestrator.merge.tools", label: "Merge Tools", control: "input", layer: "L1", fieldType: "string", description: "Merge-agent tool allowlist" },
-			{ configPath: "orchestrator.merge.thinking", label: "Merge Thinking", control: "picker", layer: "L1+L2", fieldType: "string", prefsKey: "mergeThinking", description: "Merge-agent thinking mode" },
-			{ configPath: "orchestrator.merge.order", label: "Merge Order", control: "toggle", layer: "L1", fieldType: "enum", values: ["fewest-files-first", "sequential"], description: "Lane merge ordering policy" },
-			{ configPath: "orchestrator.merge.timeoutMinutes", label: "Merge Timeout (minutes)", control: "input", layer: "L1", fieldType: "number", description: "Max time for merge agent to complete. Increase for large batches (default: 10)" },
+			{
+				configPath: "orchestrator.merge.model",
+				label: "Merge Model",
+				control: "input",
+				layer: "L1+L2",
+				fieldType: "string",
+				prefsKey: "mergeModel",
+				description: "Merge-agent model (inherit = use session model)",
+			},
+			{
+				configPath: "orchestrator.merge.tools",
+				label: "Merge Tools",
+				control: "input",
+				layer: "L1",
+				fieldType: "string",
+				description: "Merge-agent tool allowlist",
+			},
+			{
+				configPath: "orchestrator.merge.thinking",
+				label: "Merge Thinking",
+				control: "picker",
+				layer: "L1+L2",
+				fieldType: "string",
+				prefsKey: "mergeThinking",
+				description: "Merge-agent thinking mode",
+			},
+			{
+				configPath: "orchestrator.merge.order",
+				label: "Merge Order",
+				control: "toggle",
+				layer: "L1",
+				fieldType: "enum",
+				values: ["fewest-files-first", "sequential"],
+				description: "Lane merge ordering policy",
+			},
+			{
+				configPath: "orchestrator.merge.timeoutMinutes",
+				label: "Merge Timeout (minutes)",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Max time for merge agent to complete. Increase for large batches (default: 10)",
+			},
 		],
 	},
 	{
 		name: "Agent Extensions",
-		readOnly: true,  // Dynamically handled — no fixed fields
+		readOnly: true, // Dynamically handled — no fixed fields
 		fields: [],
 	},
 	{
 		name: "Context Limits",
 		fields: [
-			{ configPath: "taskRunner.context.workerContextWindow", label: "Context Window", control: "input", layer: "L1", fieldType: "number", description: "Worker context window size" },
-			{ configPath: "taskRunner.context.warnPercent", label: "Warn %", control: "input", layer: "L1", fieldType: "number", description: "Context utilization warn threshold (%)" },
-			{ configPath: "taskRunner.context.killPercent", label: "Kill %", control: "input", layer: "L1", fieldType: "number", description: "Context utilization hard-stop threshold (%)" },
-			{ configPath: "taskRunner.context.maxWorkerIterations", label: "Max Iterations", control: "input", layer: "L1", fieldType: "number", description: "Max worker iterations per step" },
-			{ configPath: "taskRunner.context.maxReviewCycles", label: "Max Review Cycles", control: "input", layer: "L1", fieldType: "number", description: "Max revise loops per review stage" },
-			{ configPath: "taskRunner.context.noProgressLimit", label: "No Progress Limit", control: "input", layer: "L1", fieldType: "number", description: "Max no-progress iterations before failure" },
-			{ configPath: "taskRunner.context.maxWorkerMinutes", label: "Max Worker Min (ctx)", control: "input", layer: "L1", fieldType: "number", optional: true, description: "Per-worker wall-clock cap (minutes, empty = no cap)" },
+			{
+				configPath: "taskRunner.context.workerContextWindow",
+				label: "Context Window",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Worker context window size",
+			},
+			{
+				configPath: "taskRunner.context.warnPercent",
+				label: "Warn %",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Context utilization warn threshold (%)",
+			},
+			{
+				configPath: "taskRunner.context.killPercent",
+				label: "Kill %",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Context utilization hard-stop threshold (%)",
+			},
+			{
+				configPath: "taskRunner.context.maxWorkerIterations",
+				label: "Max Iterations",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Max worker iterations per step",
+			},
+			{
+				configPath: "taskRunner.context.maxReviewCycles",
+				label: "Max Review Cycles",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Max revise loops per review stage",
+			},
+			{
+				configPath: "taskRunner.context.noProgressLimit",
+				label: "No Progress Limit",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Max no-progress iterations before failure",
+			},
+			{
+				configPath: "taskRunner.context.maxWorkerMinutes",
+				label: "Max Worker Min (ctx)",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				optional: true,
+				description: "Per-worker wall-clock cap (minutes, empty = no cap)",
+			},
 		],
 	},
 	{
 		name: "Failure Policy",
 		fields: [
-			{ configPath: "orchestrator.failure.onTaskFailure", label: "On Task Failure", control: "toggle", layer: "L1", fieldType: "enum", values: ["skip-dependents", "stop-wave", "stop-all"], description: "Batch behavior when a task fails" },
-			{ configPath: "orchestrator.failure.onMergeFailure", label: "On Merge Failure", control: "toggle", layer: "L1", fieldType: "enum", values: ["pause", "abort"], description: "Behavior when a merge step fails" },
-			{ configPath: "orchestrator.failure.stallTimeout", label: "Stall Timeout (min)", control: "input", layer: "L1", fieldType: "number", description: "Stall detection threshold (minutes)" },
-			{ configPath: "orchestrator.failure.maxWorkerMinutes", label: "Max Worker Min", control: "input", layer: "L1", fieldType: "number", description: "Max worker runtime budget per task (minutes)" },
-			{ configPath: "orchestrator.failure.abortGracePeriod", label: "Abort Grace (sec)", control: "input", layer: "L1", fieldType: "number", description: "Graceful abort wait time (seconds)" },
+			{
+				configPath: "orchestrator.failure.onTaskFailure",
+				label: "On Task Failure",
+				control: "toggle",
+				layer: "L1",
+				fieldType: "enum",
+				values: ["skip-dependents", "stop-wave", "stop-all"],
+				description: "Batch behavior when a task fails",
+			},
+			{
+				configPath: "orchestrator.failure.onMergeFailure",
+				label: "On Merge Failure",
+				control: "toggle",
+				layer: "L1",
+				fieldType: "enum",
+				values: ["pause", "abort"],
+				description: "Behavior when a merge step fails",
+			},
+			{
+				configPath: "orchestrator.failure.stallTimeout",
+				label: "Stall Timeout (min)",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Stall detection threshold (minutes)",
+			},
+			{
+				configPath: "orchestrator.failure.maxWorkerMinutes",
+				label: "Max Worker Min",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Max worker runtime budget per task (minutes)",
+			},
+			{
+				configPath: "orchestrator.failure.abortGracePeriod",
+				label: "Abort Grace (sec)",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Graceful abort wait time (seconds)",
+			},
 		],
 	},
 	{
 		name: "Dependencies",
 		fields: [
-			{ configPath: "orchestrator.dependencies.source", label: "Dep Source", control: "toggle", layer: "L1", fieldType: "enum", values: ["prompt", "agent"], description: "Dependency extraction source" },
-			{ configPath: "orchestrator.dependencies.cache", label: "Dep Cache", control: "toggle", layer: "L1", fieldType: "boolean", values: ["true", "false"], description: "Cache dependency analysis results" },
+			{
+				configPath: "orchestrator.dependencies.source",
+				label: "Dep Source",
+				control: "toggle",
+				layer: "L1",
+				fieldType: "enum",
+				values: ["prompt", "agent"],
+				description: "Dependency extraction source",
+			},
+			{
+				configPath: "orchestrator.dependencies.cache",
+				label: "Dep Cache",
+				control: "toggle",
+				layer: "L1",
+				fieldType: "boolean",
+				values: ["true", "false"],
+				description: "Cache dependency analysis results",
+			},
 		],
 	},
 	{
 		name: "Assignment",
 		fields: [
-			{ configPath: "orchestrator.assignment.strategy", label: "Strategy", control: "toggle", layer: "L1", fieldType: "enum", values: ["affinity-first", "round-robin", "load-balanced"], description: "Lane assignment strategy" },
+			{
+				configPath: "orchestrator.assignment.strategy",
+				label: "Strategy",
+				control: "toggle",
+				layer: "L1",
+				fieldType: "enum",
+				values: ["affinity-first", "round-robin", "load-balanced"],
+				description: "Lane assignment strategy",
+			},
 		],
 	},
 	{
 		name: "Pre-Warm",
 		fields: [
-			{ configPath: "orchestrator.preWarm.autoDetect", label: "Auto-Detect", control: "toggle", layer: "L1", fieldType: "boolean", values: ["true", "false"], description: "Enable automatic pre-warm command detection" },
+			{
+				configPath: "orchestrator.preWarm.autoDetect",
+				label: "Auto-Detect",
+				control: "toggle",
+				layer: "L1",
+				fieldType: "boolean",
+				values: ["true", "false"],
+				description: "Enable automatic pre-warm command detection",
+			},
 		],
 	},
 	{
 		name: "Monitoring",
 		fields: [
-			{ configPath: "orchestrator.monitoring.pollInterval", label: "Poll Interval (sec)", control: "input", layer: "L1", fieldType: "number", description: "Poll interval for lane/task monitoring (seconds)" },
+			{
+				configPath: "orchestrator.monitoring.pollInterval",
+				label: "Poll Interval (sec)",
+				control: "input",
+				layer: "L1",
+				fieldType: "number",
+				description: "Poll interval for lane/task monitoring (seconds)",
+			},
 		],
 	},
 	{
 		name: "Global Preferences",
 		fields: [
-			{ configPath: "preferences.dashboardPort", label: "Dashboard Port", control: "input", layer: "L2", fieldType: "number", prefsKey: "dashboardPort", optional: true, description: "Dashboard server port" },
+			{
+				configPath: "preferences.dashboardPort",
+				label: "Dashboard Port",
+				control: "input",
+				layer: "L2",
+				fieldType: "number",
+				prefsKey: "dashboardPort",
+				optional: true,
+				description: "Dashboard server port",
+			},
 		],
 	},
 	{
 		name: "Advanced (JSON Only)",
 		readOnly: true,
-		fields: [],  // Populated dynamically in getAdvancedItems()
+		fields: [], // Populated dynamically in getAdvancedItems()
 	},
 ];
 
-
 // ── Raw Config Readers (Source Detection) ────────────────────────────
 
 /**
@@ -256,7 +548,9 @@ export function readRawYamlConfigs(configRoot: string): Record<string, any> | nu
 			if (parsed && typeof parsed === "object") {
 				result.taskRunner = convertYamlKeys(parsed, "taskRunner");
 			}
-		} catch { /* ignore */ }
+		} catch {
+			/* ignore */
+		}
 	}
 
 	if (hasOrch) {
@@ -266,7 +560,9 @@ export function readRawYamlConfigs(configRoot: string): Record<string, any> | nu
 			if (parsed && typeof parsed === "object") {
 				result.orchestrator = convertYamlKeys(parsed, "orchestrator");
 			}
-		} catch { /* ignore */ }
+		} catch {
+			/* ignore */
+		}
 	}
 
 	return Object.keys(result).length > 0 ? result : null;
@@ -338,7 +634,6 @@ function readRawPreferences(): Record<string, any> | null {
 	}
 }
 
-
 // ── Write-Back ───────────────────────────────────────────────────────
 
 /**
@@ -425,8 +720,8 @@ export function writeProjectConfigField(
 		} catch (e: any) {
 			throw new Error(
 				`Cannot write settings: ${jsonPath} contains malformed JSON. ` +
-				`Please fix or delete the file and try again. ` +
-				`(Parse error: ${e.message ?? "unknown"})`,
+					`Please fix or delete the file and try again. ` +
+					`(Parse error: ${e.message ?? "unknown"})`,
 			);
 		}
 	} else {
@@ -449,7 +744,11 @@ export function writeProjectConfigField(
 		renameSync(tmpPath, jsonPath);
 	} catch {
 		writeFileSync(jsonPath, json, "utf-8");
-		try { if (existsSync(tmpPath)) unlinkSync(tmpPath); } catch { /* cleanup best-effort */ }
+		try {
+			if (existsSync(tmpPath)) unlinkSync(tmpPath);
+		} catch {
+			/* cleanup best-effort */
+		}
 	}
 }
 
@@ -488,7 +787,11 @@ export function writeGlobalPreference(path: string, value: any): void {
 		renameSync(tmpPath, prefsPath);
 	} catch {
 		writeFileSync(prefsPath, json, "utf-8");
-		try { if (existsSync(tmpPath)) unlinkSync(tmpPath); } catch { /* cleanup best-effort */ }
+		try {
+			if (existsSync(tmpPath)) unlinkSync(tmpPath);
+		} catch {
+			/* cleanup best-effort */
+		}
 	}
 }
 
@@ -559,7 +862,6 @@ export function resolveWriteAction(
 	return defaultDest;
 }
 
-
 // ── Source Detection ─────────────────────────────────────────────────
 
 /**
@@ -596,7 +898,6 @@ export function detectFieldSource(
 	return "global";
 }
 
-
 // ── Value Formatting ─────────────────────────────────────────────────
 
 /**
@@ -628,7 +929,6 @@ export function getFieldDisplayValue(
 	return String(val);
 }
 
-
 // ── Validation ───────────────────────────────────────────────────────
 
 export interface ValidationResult {
@@ -683,7 +983,6 @@ export function validateFieldInput(field: FieldDef, input: string): ValidationRe
 	}
 }
 
-
 // ── Advanced Section Items ───────────────────────────────────────────
 
 export interface AdvancedItem {
@@ -772,11 +1071,7 @@ export function getAdvancedItems(config: TaskplaneConfig): AdvancedItem[] {
  * Known subsection objects (like `taskRunner.worker`, `orchestrator.merge`)
  * are recursed into, not reported as leaves themselves.
  */
-function walkConfig(
-	obj: any,
-	prefix: string,
-	visitor: (path: string, value: any) => void,
-): void {
+function walkConfig(obj: any, prefix: string, visitor: (path: string, value: any) => void): void {
 	if (obj === null || obj === undefined) return;
 
 	for (const [key, value] of Object.entries(obj)) {
@@ -861,7 +1156,6 @@ function summarizeArray(arr: any[]): string {
 	return `${arr.length} items`;
 }
 
-
 // ── TUI Rendering ────────────────────────────────────────────────────
 
 /**
@@ -890,7 +1184,10 @@ async function pickModel(ctx: ExtensionContext, currentModel: string): Promise<s
 	if (available.length === 0) {
 		ctx.ui.notify("No available models found in pi model registry", "warning");
 		// Fall back to manual input
-		const manual = await ctx.ui.input("Model (provider/model-id, or empty for inherit)", currentModel || "");
+		const manual = await ctx.ui.input(
+			"Model (provider/model-id, or empty for inherit)",
+			currentModel || "",
+		);
 		if (manual === null || manual === undefined) return undefined;
 		return manual;
 	}
@@ -909,10 +1206,10 @@ async function pickModel(ctx: ExtensionContext, currentModel: string): Promise<s
 		];
 
 		const providerChoice = await selectScrollable(ctx, "Choose model provider", providerOptions);
-		if (!providerChoice) return undefined;  // Cancelled
+		if (!providerChoice) return undefined; // Cancelled
 
 		if (providerChoice.startsWith("inherit")) {
-			return "";  // Empty string = inherit
+			return ""; // Empty string = inherit
 		}
 
 		// Extract provider name (strip " (N models)" suffix)
@@ -941,7 +1238,7 @@ async function pickModel(ctx: ExtensionContext, currentModel: string): Promise<s
 		}
 
 		const modelChoice = await selectScrollable(ctx, `Choose model (${provider})`, modelOptions);
-		if (!modelChoice) continue;  // Cancelled → back to providers
+		if (!modelChoice) continue; // Cancelled → back to providers
 		if (modelChoice === "← Back to providers") continue;
 
 		const resolved = modelOptionMap.get(modelChoice);
@@ -962,7 +1259,9 @@ const THINKING_MODE_OPTIONS: Array<{ value: ThinkingModeValue; label: string }>
 ];
 
 function normalizeThinkingMode(value: unknown): ThinkingModeValue {
-	const cleaned = String(value ?? "").trim().toLowerCase();
+	const cleaned = String(value ?? "")
+		.trim()
+		.toLowerCase();
 	if (!cleaned || cleaned === "inherit") return "";
 	if (cleaned === "on") return "high";
 	if (["off", "minimal", "low", "medium", "high", "xhigh"].includes(cleaned)) {
@@ -1014,15 +1313,17 @@ function resolveModelRecord(ctx: ExtensionContext, modelRef: string): any | unde
 	if (slashIdx > 0) {
 		const provider = trimmed.slice(0, slashIdx).toLowerCase();
 		const id = trimmed.slice(slashIdx + 1).toLowerCase();
-		return available.find((m: any) =>
-			String(m?.provider ?? "").toLowerCase() === provider
-			&& String(m?.id ?? "").toLowerCase() === id,
+		return available.find(
+			(m: any) =>
+				String(m?.provider ?? "").toLowerCase() === provider &&
+				String(m?.id ?? "").toLowerCase() === id,
 		);
 	}
 
-	return available.find((m: any) =>
-		String(m?.id ?? "").toLowerCase() === lower
-		|| `${String(m?.provider ?? "").toLowerCase()}/${String(m?.id ?? "").toLowerCase()}` === lower,
+	return available.find(
+		(m: any) =>
+			String(m?.id ?? "").toLowerCase() === lower ||
+			`${String(m?.provider ?? "").toLowerCase()}/${String(m?.id ?? "").toLowerCase()}` === lower,
 	);
 }
 
@@ -1046,12 +1347,9 @@ export function modelSupportsThinking(model: any): boolean {
 		"reasoning_tokens",
 	];
 
-	const candidateObjects = [
-		model,
-		model.capabilities,
-		model.features,
-		model.metadata,
-	].filter((entry) => entry && typeof entry === "object");
+	const candidateObjects = [model, model.capabilities, model.features, model.metadata].filter(
+		(entry) => entry && typeof entry === "object",
+	);
 
 	for (const candidate of candidateObjects) {
 		for (const key of boolFlags) {
@@ -1145,7 +1443,9 @@ async function selectScrollable(
 		container.addChild(selectList);
 
 		container.addChild(new Text("", 0, 0));
-		container.addChild(new Text(theme.fg("dim", "↑↓ navigate • type to filter • enter select • esc back"), 1, 0));
+		container.addChild(
+			new Text(theme.fg("dim", "↑↓ navigate • type to filter • enter select • esc back"), 1, 0),
+		);
 		container.addChild(new DynamicBorder((s: string) => theme.fg("accent", s)));
 
 		return {
@@ -1160,7 +1460,8 @@ async function selectScrollable(
 
 	if (selectedValue === undefined) return undefined;
 	const selectedIndex = Number(selectedValue);
-	if (!Number.isInteger(selectedIndex) || selectedIndex < 0 || selectedIndex >= options.length) return undefined;
+	if (!Number.isInteger(selectedIndex) || selectedIndex < 0 || selectedIndex >= options.length)
+		return undefined;
 	return options[selectedIndex];
 }
 
@@ -1178,7 +1479,10 @@ export async function openSettingsTui(
  * Reload all config state from disk. Called after write-back to
  * refresh the TUI display.
  */
-function loadConfigState(configRoot: string, pointerConfigRoot?: string): {
+function loadConfigState(
+	configRoot: string,
+	pointerConfigRoot?: string,
+): {
 	mergedConfig: TaskplaneConfig;
 	prefs: GlobalPreferences;
 	rawProject: Record<string, any> | null;
@@ -1211,11 +1515,12 @@ async function showSectionSelectorLoop(
 		const sectionItems: SelectItem[] = SECTIONS.map((section, i) => ({
 			value: String(i),
 			label: section.name,
-			description: section.name === "Agent Extensions"
-				? "Toggle extensions per agent type"
-				: section.readOnly
-					? "Read-only collection/record fields"
-					: `${section.fields.length} setting${section.fields.length === 1 ? "" : "s"}`,
+			description:
+				section.name === "Agent Extensions"
+					? "Toggle extensions per agent type"
+					: section.readOnly
+						? "Read-only collection/record fields"
+						: `${section.fields.length} setting${section.fields.length === 1 ? "" : "s"}`,
 		}));
 
 		const selectedSection = await ctx.ui.custom<string | null>((tui, theme, _kb, done) => {
@@ -1226,7 +1531,9 @@ async function showSectionSelectorLoop(
 
 			// Title
 			container.addChild(new Text(theme.fg("accent", theme.bold("⚙ Settings")), 1, 0));
-			container.addChild(new Text(theme.fg("dim", "Navigate sections to view and edit configuration"), 1, 0));
+			container.addChild(
+				new Text(theme.fg("dim", "Navigate sections to view and edit configuration"), 1, 0),
+			);
 			container.addChild(new Text("", 0, 0));
 
 			// SelectList
@@ -1251,11 +1558,14 @@ async function showSectionSelectorLoop(
 			return {
 				render: (w: number) => container.render(w),
 				invalidate: () => container.invalidate(),
-				handleInput: (data: string) => { selectList.handleInput(data); tui.requestRender(); },
+				handleInput: (data: string) => {
+					selectList.handleInput(data);
+					tui.requestRender();
+				},
 			};
 		});
 
-		if (selectedSection === null) return;  // User pressed Esc
+		if (selectedSection === null) return; // User pressed Esc
 
 		const sectionIndex = parseInt(selectedSection, 10);
 		const section = SECTIONS[sectionIndex];
@@ -1295,15 +1605,17 @@ async function showAdvancedSection(
 
 		// Title
 		container.addChild(new Text(theme.fg("accent", theme.bold("Advanced (JSON Only)")), 1, 0));
-		container.addChild(new Text(theme.fg("dim", "These fields can only be edited directly in the config file"), 1, 0));
+		container.addChild(
+			new Text(theme.fg("dim", "These fields can only be edited directly in the config file"), 1, 0),
+		);
 		container.addChild(new Text("", 0, 0));
 
 		const settingsList = new SettingsList(
 			settingsItems,
 			Math.min(settingsItems.length + 2, 20),
 			getSettingsListTheme(),
-			() => {},  // onChange — no-op (read-only)
-			() => done(undefined),  // onCancel
+			() => {}, // onChange — no-op (read-only)
+			() => done(undefined), // onCancel
 		);
 		container.addChild(settingsList);
 
@@ -1317,7 +1629,10 @@ async function showAdvancedSection(
 		return {
 			render: (w: number) => container.render(w),
 			invalidate: () => container.invalidate(),
-			handleInput: (data: string) => { settingsList.handleInput?.(data); tui.requestRender(); },
+			handleInput: (data: string) => {
+				settingsList.handleInput?.(data);
+				tui.requestRender();
+			},
 		};
 	});
 }
@@ -1351,14 +1666,18 @@ async function showExtensionsSection(
 				container.addChild(new Text(theme.fg("accent", theme.bold("Agent Extensions")), 1, 0));
 				container.addChild(new Text("", 0, 0));
 				container.addChild(new Text(theme.fg("dim", "No third-party extensions found."), 1, 0));
-				container.addChild(new Text(theme.fg("dim", "Install extensions via pi settings to see them here."), 1, 0));
+				container.addChild(
+					new Text(theme.fg("dim", "Install extensions via pi settings to see them here."), 1, 0),
+				);
 				container.addChild(new Text("", 0, 0));
 				container.addChild(new Text(theme.fg("dim", "esc back"), 1, 0));
 				container.addChild(new DynamicBorder((s: string) => theme.fg("accent", s)));
 				return {
 					render: (w: number) => container.render(w),
 					invalidate: () => container.invalidate(),
-					handleInput: (data: string) => { if (data === "\x1b" || data === "\x1b\x1b") done(undefined); },
+					handleInput: (data: string) => {
+						if (data === "\x1b" || data === "\x1b\x1b") done(undefined);
+					},
 				};
 			});
 			return;
@@ -1371,7 +1690,11 @@ async function showExtensionsSection(
 
 		const agentTypes = [
 			{ name: "Worker", exclude: workerExclude, configPath: "taskRunner.worker.excludeExtensions" },
-			{ name: "Reviewer", exclude: reviewerExclude, configPath: "taskRunner.reviewer.excludeExtensions" },
+			{
+				name: "Reviewer",
+				exclude: reviewerExclude,
+				configPath: "taskRunner.reviewer.excludeExtensions",
+			},
 			{ name: "Merger", exclude: mergeExclude, configPath: "orchestrator.merge.excludeExtensions" },
 		];
 
@@ -1391,32 +1714,37 @@ async function showExtensionsSection(
 			}
 		}
 
-		const result = await ctx.ui.custom<{ id: string; value: string } | null>((tui, theme, _kb, done) => {
-			const container = new Container();
-			container.addChild(new DynamicBorder((s: string) => theme.fg("accent", s)));
-			container.addChild(new Text(theme.fg("accent", theme.bold("Agent Extensions")), 1, 0));
-			container.addChild(new Text(theme.fg("dim", "Toggle extensions on/off per agent type"), 1, 0));
-			container.addChild(new Text("", 0, 0));
+		const result = await ctx.ui.custom<{ id: string; value: string } | null>(
+			(tui, theme, _kb, done) => {
+				const container = new Container();
+				container.addChild(new DynamicBorder((s: string) => theme.fg("accent", s)));
+				container.addChild(new Text(theme.fg("accent", theme.bold("Agent Extensions")), 1, 0));
+				container.addChild(new Text(theme.fg("dim", "Toggle extensions on/off per agent type"), 1, 0));
+				container.addChild(new Text("", 0, 0));
 
-			const settingsList = new SettingsList(
-				settingsItems,
-				Math.min(settingsItems.length + 2, 20),
-				getSettingsListTheme(),
-				(id, newValue) => done({ id, value: newValue }),
-				() => done(null),
-			);
-			container.addChild(settingsList);
+				const settingsList = new SettingsList(
+					settingsItems,
+					Math.min(settingsItems.length + 2, 20),
+					getSettingsListTheme(),
+					(id, newValue) => done({ id, value: newValue }),
+					() => done(null),
+				);
+				container.addChild(settingsList);
 
-			container.addChild(new Text("", 0, 0));
-			container.addChild(new Text(theme.fg("dim", "↑↓ navigate • space toggle • esc back"), 1, 0));
-			container.addChild(new DynamicBorder((s: string) => theme.fg("accent", s)));
+				container.addChild(new Text("", 0, 0));
+				container.addChild(new Text(theme.fg("dim", "↑↓ navigate • space toggle • esc back"), 1, 0));
+				container.addChild(new DynamicBorder((s: string) => theme.fg("accent", s)));
 
-			return {
-				render: (w: number) => container.render(w),
-				invalidate: () => container.invalidate(),
-				handleInput: (data: string) => { settingsList.handleInput?.(data); tui.requestRender(); },
-			};
-		});
+				return {
+					render: (w: number) => container.render(w),
+					invalidate: () => container.invalidate(),
+					handleInput: (data: string) => {
+						settingsList.handleInput?.(data);
+						tui.requestRender();
+					},
+				};
+			},
+		);
 
 		if (!result) return; // User pressed Esc
 
@@ -1428,7 +1756,8 @@ async function showExtensionsSection(
 
 		// Read current exclusion array from merged effective config (handles YAML+JSON)
 		const freshConfig = loadProjectConfig(configRoot, pointerConfigRoot);
-		const currentExcludeList: string[] = (getNestedValue(freshConfig, configPath) as string[] | undefined) ?? [];
+		const currentExcludeList: string[] =
+			(getNestedValue(freshConfig, configPath) as string[] | undefined) ?? [];
 
 		let newExcludeList: string[];
 		if (enabling) {
@@ -1444,7 +1773,11 @@ async function showExtensionsSection(
 		try {
 			writeProjectConfigField(configRoot, configPath, newExcludeList, pointerConfigRoot);
 			if (onConfigChanged) {
-				try { onConfigChanged(); } catch { /* non-fatal */ }
+				try {
+					onConfigChanged();
+				} catch {
+					/* non-fatal */
+				}
 			}
 			ctx.ui.notify(
 				`${enabling ? "✅ Enabled" : "❌ Disabled"} ${pkg} for ${configPath.includes("worker") ? "Worker" : configPath.includes("reviewer") ? "Reviewer" : "Merger"}`,
@@ -1463,8 +1796,10 @@ async function showExtensionsSection(
  */
 function formatSourceBadge(source: FieldSource): string {
 	switch (source) {
-		case "project": return "(project)";
-		case "global":  return "(global)";
+		case "project":
+			return "(project)";
+		case "global":
+			return "(global)";
 	}
 }
 
@@ -1487,18 +1822,28 @@ async function showSectionSettingsLoop(
 ): Promise<void> {
 	while (true) {
 		const state = loadConfigState(configRoot, pointerConfigRoot);
-		const result = await showSectionSettingsOnce(ctx, section, state.mergedConfig, state.prefs, state.rawProject, state.rawPrefs);
+		const result = await showSectionSettingsOnce(
+			ctx,
+			section,
+			state.mergedConfig,
+			state.prefs,
+			state.rawProject,
+			state.rawPrefs,
+		);
 
-		if (result === null) return;  // User pressed Esc → back to sections
+		if (result === null) return; // User pressed Esc → back to sections
 
 		// Process the pending change
 		const field = section.fields.find((f) => f.configPath === result.fieldId);
-		if (!field) continue;  // Safety: field not found
+		if (!field) continue; // Safety: field not found
 
 		let previousModelValue = "";
 
 		// Input/picker fields: the submenu returned a sentinel — open the editor picker.
-		if (result.rawValue === "__EDIT_REQUESTED__" && (field.control === "input" || field.control === "picker")) {
+		if (
+			result.rawValue === "__EDIT_REQUESTED__" &&
+			(field.control === "input" || field.control === "picker")
+		) {
 			const state = loadConfigState(configRoot, pointerConfigRoot);
 			const currentDisplay = getFieldDisplayValue(field, state.mergedConfig, state.prefs);
 			const currentClean = String(currentDisplay).replace(/\s+\((?:default|project|global)\)$/, "");
@@ -1508,31 +1853,30 @@ async function showSectionSettingsLoop(
 			if (field.configPath.endsWith(".model")) {
 				previousModelValue = normalizedCurrent;
 				const selected = await pickModel(ctx, normalizedCurrent);
-				if (selected === undefined) continue;  // Cancelled
+				if (selected === undefined) continue; // Cancelled
 				result.rawValue = selected;
 			} else if (field.control === "picker" && field.configPath.endsWith(".thinking")) {
 				const note = buildThinkingUnsupportedNoteForThinkingField(ctx, field, state.mergedConfig);
 				if (note) ctx.ui.notify(note, "info");
 				const selected = await pickThinkingMode(ctx, normalizedCurrent);
-				if (selected === undefined) continue;  // Cancelled
+				if (selected === undefined) continue; // Cancelled
 				result.rawValue = selected;
 			} else if (field.control === "picker" && field.values && field.values.length > 0) {
 				// Enum picker: show scrollable list of allowed values
-				const options = field.values.map((v) =>
-					`${v}${v === normalizedCurrent ? "  ✓ current" : ""}`
-				);
+				const options = field.values.map((v) => `${v}${v === normalizedCurrent ? "  ✓ current" : ""}`);
 				const selected = await selectScrollable(ctx, field.label, options);
-				if (!selected) continue;  // Cancelled
+				if (!selected) continue; // Cancelled
 				result.rawValue = selected.replace(/\s+✓ current$/, "");
 			} else {
-				const placeholder = currentClean === "(not set)" || currentClean === "(inherit)" ? "" : currentClean;
+				const placeholder =
+					currentClean === "(not set)" || currentClean === "(inherit)" ? "" : currentClean;
 
 				const newValue = await ctx.ui.input(
 					`${field.label}${field.description ? ` — ${field.description}` : ""}`,
 					placeholder,
 				);
 
-				if (newValue === null || newValue === undefined) continue;  // Cancelled
+				if (newValue === null || newValue === undefined) continue; // Cancelled
 
 				// Validate
 				const validation = validateFieldInput(field, newValue);
@@ -1585,13 +1929,14 @@ async function showSectionSettingsLoop(
 			}
 			// Notify caller to reload in-memory config from disk
 			if (onConfigChanged) {
-				try { onConfigChanged(); } catch { /* non-fatal */ }
+				try {
+					onConfigChanged();
+				} catch {
+					/* non-fatal */
+				}
 			}
 
-			ctx.ui.notify(
-				`✅ ${field.label} updated.`,
-				"info",
-			);
+			ctx.ui.notify(`✅ ${field.label} updated.`, "info");
 
 			const refreshedState = loadConfigState(configRoot, pointerConfigRoot);
 			const suggestion = buildThinkingSuggestionForModelChange(
@@ -1657,17 +2002,13 @@ async function showSectionSettingsOnce(
 					}));
 					// Return SelectList directly — Container doesn't forward
 					// handleInput to children, which would freeze the TUI.
-					const list = new SelectList(
-						selectItems,
-						Math.min(selectItems.length + 1, 10),
-						{
-							selectedPrefix: (t: string) => `\x1b[36m${t}\x1b[0m`,
-							selectedText: (t: string) => `\x1b[36m${t}\x1b[0m`,
-							description: (t: string) => `\x1b[2m${t}\x1b[0m`,
-							scrollInfo: (t: string) => `\x1b[2m${t}\x1b[0m`,
-							noMatch: (t: string) => `\x1b[33m${t}\x1b[0m`,
-						},
-					);
+					const list = new SelectList(selectItems, Math.min(selectItems.length + 1, 10), {
+						selectedPrefix: (t: string) => `\x1b[36m${t}\x1b[0m`,
+						selectedText: (t: string) => `\x1b[36m${t}\x1b[0m`,
+						description: (t: string) => `\x1b[2m${t}\x1b[0m`,
+						scrollInfo: (t: string) => `\x1b[2m${t}\x1b[0m`,
+						noMatch: (t: string) => `\x1b[33m${t}\x1b[0m`,
+					});
 					const currentIdx = field.values!.indexOf(displayValue);
 					if (currentIdx >= 0) list.setSelectedIndex(currentIdx);
 					list.onSelect = (selected) => done(selected.value);
@@ -1710,7 +2051,7 @@ async function showSectionSettingsOnce(
 				// Exit TUI with the change so the caller can handle write-back
 				done({ fieldId: id, rawValue: newValue });
 			},
-			() => done(null),  // onCancel → back to section selector
+			() => done(null), // onCancel → back to section selector
 			{ enableSearch: settingsItems.length > 5 },
 		);
 		container.addChild(settingsList);
@@ -1723,10 +2064,9 @@ async function showSectionSettingsOnce(
 
 		// Help text
 		container.addChild(new Text("", 0, 0));
-		container.addChild(new Text(
-			theme.fg("dim", "↑↓ navigate • ←→/space cycle • enter edit • esc back"),
-			1, 0,
-		));
+		container.addChild(
+			new Text(theme.fg("dim", "↑↓ navigate • ←→/space cycle • enter edit • esc back"), 1, 0),
+		);
 
 		// Bottom border
 		container.addChild(new DynamicBorder((s: string) => theme.fg("accent", s)));
@@ -1734,12 +2074,14 @@ async function showSectionSettingsOnce(
 		return {
 			render: (w: number) => container.render(w),
 			invalidate: () => container.invalidate(),
-			handleInput: (data: string) => { settingsList.handleInput?.(data); tui.requestRender(); },
+			handleInput: (data: string) => {
+				settingsList.handleInput?.(data);
+				tui.requestRender();
+			},
 		};
 	});
 }
 
-
 // ── Input Submenu ────────────────────────────────────────────────────
 
 /**
@@ -1761,7 +2103,7 @@ function createInputSubmenu(
 		render(width: number): string[] {
 			const lines: string[] = [];
 			const prompt = `  Enter ${field.label}: `;
-			const inputDisplay = inputBuffer + "█";  // Simple cursor
+			const inputDisplay = inputBuffer + "█"; // Simple cursor
 			lines.push(truncateLine(prompt + inputDisplay, width));
 
 			if (field.optional) {
@@ -1818,7 +2160,6 @@ function truncateLine(text: string, width: number): string {
 	return text.substring(0, width - 3) + "...";
 }
 
-
 // ── JSON-Only Footer ─────────────────────────────────────────────────
 
 /**
@@ -1826,17 +2167,17 @@ function truncateLine(text: string, width: number): string {
  * Used to dynamically discover JSON-only sibling fields.
  */
 const SECTION_CONFIG_PREFIXES: Record<string, string[]> = {
-	"Orchestrator": ["orchestrator.orchestrator"],
+	Orchestrator: ["orchestrator.orchestrator"],
 	"Agent: Supervisor": ["orchestrator.supervisor"],
 	"Agent: Worker": ["taskRunner.worker"],
 	"Agent: Reviewer": ["taskRunner.reviewer"],
 	"Agent: Merge": ["orchestrator.merge"],
 	"Context Limits": ["taskRunner.context"],
 	"Failure Policy": ["orchestrator.failure"],
-	"Dependencies": ["orchestrator.dependencies"],
-	"Assignment": ["orchestrator.assignment"],
+	Dependencies: ["orchestrator.dependencies"],
+	Assignment: ["orchestrator.assignment"],
 	"Pre-Warm": ["orchestrator.preWarm"],
-	"Monitoring": ["orchestrator.monitoring"],
+	Monitoring: ["orchestrator.monitoring"],
 };
 
 /**