test: cover dashboard settings clone defaults and equality semantics

Drives cloneDashboardSettingsData/dashboardSettingsDataEqual with the
real resolveMenuLayoutMode and a normalizer matching the
dashboard-settings default semantics:

- a sparse legacy settings object clones with every optional field at
  its documented default, statusline fields copied (not aliased), and
  the layout mode + unselected-rows flag derived together with an
  explicit menuLayoutMode winning over the legacy boolean
- equality treats absent optional fields as their defaults, flags any
  single-field difference, compares layout through the resolver (the
  legacy boolean spelling equals the explicit expanded-rows mode), and
  compares statusline fields through the normalizer with order being
  significant

https://claude.ai/code/session_01XNtnkLbBiXZxfQQYLMpucB

Author: claude

test/dashboard-settings-data.test.ts

@@ -0,0 +1,187 @@
+import { describe, expect, it } from "vitest";
+import {
+	cloneDashboardSettingsData,
+	dashboardSettingsDataEqual,
+} from "../lib/codex-manager/dashboard-settings-data.js";
+import {
+	DEFAULT_DASHBOARD_DISPLAY_SETTINGS,
+	type DashboardDisplaySettings,
+	type DashboardStatuslineField,
+} from "../lib/dashboard-settings.js";
+import { resolveMenuLayoutMode } from "../lib/codex-manager/settings-hub.js";
+
+const DEFAULT_FIELDS: DashboardStatuslineField[] = [
+	"last-used",
+	"limits",
+	"status",
+];
+
+// The real production deps: the actual layout-mode resolver, and a
+// normalizer matching dashboard-settings semantics (default on absence).
+const deps = {
+	resolveMenuLayoutMode,
+	normalizeStatuslineFields: (
+		fields: DashboardDisplaySettings["menuStatuslineFields"],
+	) => fields ?? DEFAULT_FIELDS,
+};
+
+function settings(
+	overrides: Partial<DashboardDisplaySettings> = {},
+): DashboardDisplaySettings {
+	return { ...DEFAULT_DASHBOARD_DISPLAY_SETTINGS, ...overrides };
+}
+
+// A sparse settings object as it comes from an older settings.json that
+// predates most optional fields.
+const SPARSE: DashboardDisplaySettings = {
+	showPerAccountRows: true,
+	showQuotaDetails: false,
+	showForecastReasons: true,
+	showRecommendations: false,
+	showLiveProbeNotes: true,
+};
+
+describe("cloneDashboardSettingsData", () => {
+	it("fills every optional field with its documented default", () => {
+		const clone = cloneDashboardSettingsData(SPARSE, deps);
+
+		expect(clone).toMatchObject({
+			actionAutoReturnMs: 2_000,
+			actionPauseOnKey: true,
+			menuAutoFetchLimits: true,
+			menuSortEnabled: true,
+			menuSortMode: "ready-first",
+			menuSortPinCurrent: false,
+			menuSortQuickSwitchVisibleRow: true,
+			uiThemePreset: "green",
+			uiAccentColor: "green",
+			menuShowQuotaSummary: true,
+			menuShowFetchStatus: true,
+			menuQuotaTtlMs: 5 * 60_000,
+			menuFocusStyle: "row-invert",
+			menuHighlightCurrentRow: true,
+			menuLayoutMode: "compact-details",
+			menuShowDetailsForUnselectedRows: false,
+			menuStatuslineFields: DEFAULT_FIELDS,
+		});
+		// The required flags pass through unchanged.
+		expect(clone.showQuotaDetails).toBe(false);
+		expect(clone.showRecommendations).toBe(false);
+	});
+
+	it("derives layout mode and the unselected-rows flag together", () => {
+		const expanded = cloneDashboardSettingsData(
+			{ ...SPARSE, menuShowDetailsForUnselectedRows: true },
+			deps,
+		);
+		expect(expanded.menuLayoutMode).toBe("expanded-rows");
+		expect(expanded.menuShowDetailsForUnselectedRows).toBe(true);
+
+		// An explicit layout mode wins over the legacy boolean.
+		const compact = cloneDashboardSettingsData(
+			{
+				...SPARSE,
+				menuLayoutMode: "compact-details",
+				menuShowDetailsForUnselectedRows: true,
+			},
+			deps,
+		);
+		expect(compact.menuLayoutMode).toBe("compact-details");
+		expect(compact.menuShowDetailsForUnselectedRows).toBe(false);
+	});
+
+	it("copies the statusline fields instead of aliasing the input array", () => {
+		const fields: DashboardStatuslineField[] = ["limits"];
+		const clone = cloneDashboardSettingsData(
+			{ ...SPARSE, menuStatuslineFields: fields },
+			deps,
+		);
+
+		expect(clone.menuStatuslineFields).toEqual(["limits"]);
+		expect(clone.menuStatuslineFields).not.toBe(fields);
+	});
+});
+
+describe("dashboardSettingsDataEqual", () => {
+	it("treats absent optional fields as equal to their defaults", () => {
+		expect(
+			dashboardSettingsDataEqual(
+				SPARSE,
+				cloneDashboardSettingsData(SPARSE, deps),
+				deps,
+			),
+		).toBe(true);
+		// A fully-defaulted object equals the sparse one when the required
+		// flags agree.
+		expect(
+			dashboardSettingsDataEqual(
+				settings({ showQuotaDetails: false, showRecommendations: false }),
+				SPARSE,
+				deps,
+			),
+		).toBe(true);
+	});
+
+	it("detects a difference in any single field", () => {
+		expect(
+			dashboardSettingsDataEqual(
+				settings(),
+				settings({ menuQuotaTtlMs: 60_000 }),
+				deps,
+			),
+		).toBe(false);
+		expect(
+			dashboardSettingsDataEqual(
+				settings(),
+				settings({ uiThemePreset: "mono" }),
+				deps,
+			),
+		).toBe(false);
+		expect(
+			dashboardSettingsDataEqual(
+				settings(),
+				settings({ showQuotaDetails: false }),
+				deps,
+			),
+		).toBe(false);
+	});
+
+	it("compares layout through the resolver, not the raw fields", () => {
+		// expanded-rows expressed via the legacy boolean equals the explicit
+		// layout-mode spelling (no explicit menuLayoutMode on the left, since
+		// an explicit mode would win over the boolean).
+		expect(
+			dashboardSettingsDataEqual(
+				{ ...SPARSE, menuShowDetailsForUnselectedRows: true },
+				{ ...SPARSE, menuLayoutMode: "expanded-rows" },
+				deps,
+			),
+		).toBe(true);
+		expect(
+			dashboardSettingsDataEqual(
+				settings({ menuLayoutMode: "expanded-rows" }),
+				settings({ menuLayoutMode: "compact-details" }),
+				deps,
+			),
+		).toBe(false);
+	});
+
+	it("compares statusline fields through the normalizer", () => {
+		// Absent fields equal the normalized default list...
+		expect(
+			dashboardSettingsDataEqual(
+				settings({ menuStatuslineFields: undefined }),
+				settings({ menuStatuslineFields: DEFAULT_FIELDS }),
+				deps,
+			),
+		).toBe(true);
+		// ...but a different order is a real difference.
+		expect(
+			dashboardSettingsDataEqual(
+				settings({ menuStatuslineFields: ["limits", "status", "last-used"] }),
+				settings({ menuStatuslineFields: DEFAULT_FIELDS }),
+				deps,
+			),
+		).toBe(false);
+	});
+});