Merge PR #261: refactor: extract statusline order helper

Author: ndycode

lib/codex-manager/settings-hub.ts

@@ -91,6 +91,10 @@ import {
 	normalizeStatuslineFields,
 } from "./settings-preview.js";
 import { withQueuedRetry } from "./settings-write-queue.js";
+import { reorderStatuslineField } from "./statusline-order.js";
+
+const reorderField = reorderStatuslineField;
+
 import { promptStatuslineSettingsPanel } from "./statusline-settings-panel.js";
 import { promptThemeSettingsPanel } from "./theme-settings-panel.js";
 import {
@@ -481,24 +485,6 @@ async function configureDashboardDisplaySettings(
 	});
 }
 
-function reorderField(
-	fields: DashboardStatuslineField[],
-	key: DashboardStatuslineField,
-	direction: -1 | 1,
-): DashboardStatuslineField[] {
-	const index = fields.indexOf(key);
-	if (index < 0) return fields;
-	const target = index + direction;
-	if (target < 0 || target >= fields.length) return fields;
-	const next = [...fields];
-	const current = next[index];
-	const swap = next[target];
-	if (!current || !swap) return fields;
-	next[index] = swap;
-	next[target] = current;
-	return next;
-}
-
 async function promptStatuslineSettings(
 	initial: DashboardDisplaySettings,
 ): Promise<DashboardDisplaySettings | null> {

lib/codex-manager/statusline-order.ts

@@ -0,0 +1,19 @@
+import type { DashboardStatuslineField } from "../dashboard-settings.js";
+
+export function reorderStatuslineField(
+	fields: DashboardStatuslineField[],
+	key: DashboardStatuslineField,
+	direction: -1 | 1,
+): DashboardStatuslineField[] {
+	const index = fields.indexOf(key);
+	if (index < 0) return fields;
+	const target = index + direction;
+	if (target < 0 || target >= fields.length) return fields;
+	const next = [...fields];
+	const current = next[index];
+	const swap = next[target];
+	if (!current || !swap) return fields;
+	next[index] = swap;
+	next[target] = current;
+	return next;
+}