@selesai/code 0.13.40 → 0.14.3

package/dist/extensions/pi-zentui/test/config.test.ts

@@ -26,6 +26,7 @@ import {
 	getExtensionStatusPlacement,
 	hasUnsupportedComponentStyle,
 	mergeConfig,
+	migrateComponentSelections,
 	saveAccentRailEditorStylePatch,
 	saveColorSourcesPatch,
 	saveContextStylePatch,
@@ -43,6 +44,7 @@ import {
 	saveGitBranchPatch,
 	saveGitCommitPatch,
 	saveGitMetricsPatch,
+	saveIconsModePatch,
 	saveMinimalistEditorStylePatch,
 	saveMinimalistPatch,
 	savePathDisplayPatch,
@@ -64,6 +66,7 @@ import {
 	renderStyleForSource,
 	renderTerminalStyle,
 } from "../extensions/zentui/style";
+import { DEFAULT_TURN_SUMMARY_FORMAT } from "../extensions/zentui/turn-summary-format";
 
 function configTempFiles(dir: string, filename = "zentui.json"): string[] {
 	return readdirSync(dir).filter(
@@ -92,9 +95,11 @@ function readRaw(path: string): Record<string, any> {
 
 describe("canonical config resolution", () => {
 	it("provides complete canonical defaults and the established palette defaults", () => {
-		const config = mergeConfig({});
+		const config = mergeConfig({}, {});
 		expect(config.components).toEqual({
+			extensionStatuses: { defaultVisibility: "show", visibility: {} },
 			editor: {
+				codexQuota: false,
 				enabled: false,
 				style: "opencode",
 				colorSource: "theme",
@@ -122,8 +127,10 @@ describe("canonical config resolution", () => {
 						showSessionName: true,
 						showTimer: true,
 						showCost: true,
+						showCacheHit: false,
 						showGit: true,
 						contextThresholds: { warning: 70, error: 90 },
+						separator: "dash",
 					},
 				},
 			},
@@ -137,6 +144,7 @@ describe("canonical config resolution", () => {
 			workingLine: {
 				enabled: true,
 				turnSummary: true,
+				turnSummaryFormat: DEFAULT_TURN_SUMMARY_FORMAT,
 				spinner: "star-bloom",
 				spinnerIntervalMs: 100,
 				animateSpinnerColor: false,
@@ -147,10 +155,12 @@ describe("canonical config resolution", () => {
 					custom: true,
 					values: [...defaultConfig.components.workingLine.messages.values],
 				},
-				segments: { tool: true, elapsed: true, thought: true, tokens: true },
+				segments: { tool: true, elapsed: true, thought: true, tokens: true, tokenRate: false },
+				placement: "above",
 			},
 			selectorBorders: { enabled: true, style: "zentui", colorSource: "theme" },
 			footer: {
+				codexQuota: false,
 				style: "native",
 				colorSource: "theme",
 				modelLabel: "name",
@@ -178,7 +188,7 @@ describe("canonical config resolution", () => {
 			},
 		});
 		expect(config.projectRefreshIntervalMs).toBe(30_000);
-		expect(config.icons.cacheHit).toBe("c");
+		expect(config.icons).toMatchObject({ mode: "auto", effectiveMode: "ascii", cacheHit: "c" });
 		expect(config.colors).toEqual(defaultConfig.colors);
 		expect(defaultConfig.components).toEqual(config.components);
 	});
@@ -387,6 +397,21 @@ describe("canonical config resolution", () => {
 			).toBe("percent");
 		}
 
+		for (const separator of ["dash", "dot"]) {
+			expect(
+				mergeConfig({
+					components: { editor: { styles: { minimalist: { separator } } } },
+				}).components.editor.styles.minimalist.separator,
+			).toBe(separator);
+		}
+		for (const separator of ["pipe", "", 1, null, true]) {
+			expect(
+				mergeConfig({
+					editorStyles: { minimalist: { separator: "dot" } },
+					components: { editor: { styles: { minimalist: { separator } } } },
+				}).components.editor.styles.minimalist.separator,
+			).toBe("dash");
+		}
 		const valid = mergeConfig({
 			components: {
 				editor: {
@@ -396,7 +421,9 @@ describe("canonical config resolution", () => {
 							showSessionName: false,
 							showTimer: false,
 							showCost: false,
+							showCacheHit: true,
 							showGit: false,
+							separator: "dot",
 						},
 					},
 				},
@@ -407,7 +434,9 @@ describe("canonical config resolution", () => {
 			showSessionName: false,
 			showTimer: false,
 			showCost: false,
+			showCacheHit: true,
 			showGit: false,
+			separator: "dot",
 		});
 
 		const invalid = mergeConfig({
@@ -417,6 +446,7 @@ describe("canonical config resolution", () => {
 					showSessionName: false,
 					showTimer: false,
 					showCost: false,
+					showCacheHit: true,
 					showGit: false,
 				},
 			},
@@ -428,6 +458,7 @@ describe("canonical config resolution", () => {
 							showSessionName: "yes",
 							showTimer: null,
 							showCost: 0,
+							showCacheHit: "yes",
 							showGit: "no",
 						},
 					},
@@ -752,7 +783,20 @@ describe("working-line config", () => {
 			}).components.workingLine.spinnerIntervalMs,
 		).toBe(100);
 	});
-
+	it("normalizes canonical working-line placement and falls back invalid values to above", () => {
+		for (const placement of ["above", "border"] as const) {
+			expect(
+				mergeConfig({ components: { workingLine: { placement } } }).components.workingLine
+					.placement,
+			).toBe(placement);
+		}
+		for (const placement of ["input", "below", "", 1, null, true]) {
+			expect(
+				mergeConfig({ components: { workingLine: { placement } } }).components.workingLine
+					.placement,
+			).toBe("above");
+		}
+	});
 	it("defaults malformed or missing Turn summary to true and preserves explicit false", () => {
 		expect(mergeConfig({}).components.workingLine.turnSummary).toBe(true);
 		expect(
@@ -803,7 +847,7 @@ describe("working-line config", () => {
 						mode: "replace",
 						values: [" One ", "One", "\x1b[31mTwo\x1b[0m", "\n"],
 					},
-					segments: { tool: false, elapsed: true, thought: true, tokens: false },
+					segments: { tool: false, elapsed: true, thought: true, tokens: false, tokenRate: false },
 				},
 			},
 			colors: {
@@ -816,6 +860,7 @@ describe("working-line config", () => {
 		expect(config.components.workingLine).toEqual({
 			enabled: true,
 			turnSummary: true,
+			turnSummaryFormat: DEFAULT_TURN_SUMMARY_FORMAT,
 			spinner: "pinwheel",
 			spinnerIntervalMs: 160,
 			animateSpinnerColor: true,
@@ -823,7 +868,8 @@ describe("working-line config", () => {
 			textAnimation: "kitt",
 			colorSource: "terminal",
 			messages: { custom: true, values: ["One", "Two"] },
-			segments: { tool: false, elapsed: true, thought: true, tokens: false },
+			segments: { tool: false, elapsed: true, thought: true, tokens: false, tokenRate: false },
+			placement: "above",
 		});
 		expect(config.colors).toMatchObject({
 			workingLineLow: "fg:240",
@@ -858,7 +904,7 @@ describe("working-line config", () => {
 			textAnimation: "classic",
 			colorSource: "theme",
 			messages: { custom: true },
-			segments: { tool: true, elapsed: true, thought: true, tokens: true },
+			segments: { tool: true, elapsed: true, thought: true, tokens: true, tokenRate: false },
 		});
 		expect(component.messages.values).toHaveLength(40);
 		expect(component.messages.values.every((value) => value.length > 0)).toBe(true);
@@ -1246,7 +1292,8 @@ describe("canonical snapshot persistence", () => {
 				},
 			},
 			(path) => {
-				const config = saveEditorComponentPatch({ enabled: true }, path);
+				saveEditorComponentPatch({ enabled: true }, path);
+				const config = migrateComponentSelections(path);
 				const raw = readRaw(path);
 				expect(Object.keys(raw.components)).toEqual(
 					expect.arrayContaining([
@@ -1279,9 +1326,10 @@ describe("canonical snapshot persistence", () => {
 		);
 	});
 
-	it("prevents legacy edits from recoupling surfaces after the first save", () => {
+	it("prevents legacy edits from recoupling surfaces after explicit migration", () => {
 		withConfig({ features: { editor: false, copyFriendly: true } }, (path) => {
 			saveEditorComponentPatch({ enabled: true }, path);
+			migrateComponentSelections(path);
 			const raw = readRaw(path);
 			expect(raw.components.editor.enabled).toBe(true);
 			expect(raw.components.userMessages.enabled).toBe(false);
@@ -1308,7 +1356,10 @@ describe("canonical snapshot persistence", () => {
 				path,
 			);
 			saveAccentRailEditorStylePatch({ transparent: true }, path);
-			saveMinimalistEditorStylePatch({ showGit: false, contextGauge: true }, path);
+			saveMinimalistEditorStylePatch(
+				{ showGit: false, contextGauge: true, showCacheHit: true },
+				path,
+			);
 			saveUserMessagesComponentPatch({ enabled: false, colorSource: "terminal" }, path);
 			saveSelectorBordersComponentPatch({ enabled: false, colorSource: "terminal" }, path);
 			saveFooterComponentPatch({ style: "native", modelLabel: "name" }, path);
@@ -1331,6 +1382,7 @@ describe("canonical snapshot persistence", () => {
 			expect(config.components.editor.styles.minimalist).toMatchObject({
 				showGit: false,
 				contextGauge: true,
+				showCacheHit: true,
 			});
 			expect(config.components.userMessages).toMatchObject({
 				enabled: false,
@@ -1458,13 +1510,15 @@ describe("compatibility saver recipes", () => {
 		);
 	});
 
-	it("materializes a complete snapshot when a compatibility saver creates the file", () => {
+	it("materializes a complete snapshot on explicit migration after a compatibility save", () => {
 		withConfig(undefined, (path) => {
 			saveUiFeaturesPatch({ editor: false }, path);
+			migrateComponentSelections(path);
 			const raw = readRaw(path);
 			expect(Object.keys(raw)).toEqual(["components"]);
 			expect(Object.keys(raw.components).sort()).toEqual([
 				"editor",
+				"extensionStatuses",
 				"footer",
 				"selectorBorders",
 				"thinkingSteps",
@@ -1481,9 +1535,9 @@ describe("compatibility saver recipes", () => {
 
 describe("mergeConfig", () => {
 	it("defaults project refresh polling to 30 seconds and Starship styles", () => {
-		const config = mergeConfig({});
+		const config = mergeConfig({}, {});
 		expect(config.projectRefreshIntervalMs).toBe(30_000);
-		expect(config.icons.cacheHit).toBe("c");
+		expect(config.icons).toMatchObject({ mode: "auto", effectiveMode: "ascii", cacheHit: "c" });
 		expect(config.icons.editorPrompt).toBe("");
 		expect(config.colors.gitBranch).toBe("bold purple");
 		expect(config.colors.packageVersion).toBe("208");
@@ -1534,6 +1588,10 @@ describe("mergeConfig", () => {
 		});
 	});
 
+	it("registers the canonical thinking level variable", () => {
+		expect(FOOTER_FORMAT_VARIABLES).toContain("thinkingLevel");
+	});
+
 	it("registers the canonical telemetry variables without aliases", () => {
 		expect(FOOTER_FORMAT_VARIABLES).toEqual(
 			expect.arrayContaining(["cache_read", "cache_write", "subscription", "auto_compaction"]),
@@ -1554,7 +1612,7 @@ describe("mergeConfig", () => {
 
 	it("defaults and normalizes responsive footer settings", () => {
 		expect(DEFAULT_COMPACT_FOOTER_FORMAT).toBe(
-			"$cwd$wrap(in $session_name)$wrap(on $git_branch) $git_status$wrap$context$wrap_sep$tokens",
+			"$cwd$wrap(in $session_name)$wrap(on $git_branch) $git_status$wrap$context$wrap_sep$tokens$wrap_sep($codex_quota)",
 		);
 		expect(mergeConfig({})).toMatchObject({
 			responsiveFooter: true,
@@ -2241,7 +2299,9 @@ describe("mergeConfig", () => {
 			expect(raw.colors.cost).toBe("success");
 			expect(raw.colorSources).toEqual({ editor: "terminal" });
 			expect(raw.components.footer.colorSource).toBe("terminal");
-			expect(raw.components.editor.colorSource).toBe("terminal");
+			expect(raw.components.editor).toBeUndefined();
+			migrateComponentSelections(path);
+			expect(readRaw(path).components.editor.colorSource).toBe("terminal");
 		} finally {
 			rmSync(dir, { recursive: true, force: true });
 		}
@@ -2301,8 +2361,10 @@ describe("mergeConfig", () => {
 			});
 			expect(Object.keys(raw)).toEqual(["components"]);
 			expect(raw.components.footer.colorSource).toBe("terminal");
-			expect(raw.components.editor.colorSource).toBe("theme");
-			expect(raw.components.userMessages.colorSource).toBe("theme");
+			expect(Object.keys(raw.components)).toEqual(["footer"]);
+			migrateComponentSelections(path);
+			expect(readRaw(path).components.editor.colorSource).toBe("theme");
+			expect(readRaw(path).components.userMessages.colorSource).toBe("theme");
 		} finally {
 			rmSync(dir, { recursive: true, force: true });
 		}
@@ -2638,6 +2700,7 @@ describe("mergeConfig", () => {
 			expect(raw.components.footer.styles.starship.extensionStatuses.colorModes).toEqual({
 				alpha: "zentui",
 				beta: "original",
+				invalid: "muted",
 			});
 		} finally {
 			rmSync(dir, { recursive: true, force: true });