From a8ec1a598a82a59e88cd97496534210c64eb028f Mon Sep 17 00:00:00 2001 From: Robin Newhouse Date: Thu, 14 May 2026 14:33:42 -0700 Subject: [PATCH] fix(cli): move advanced settings tab to end --- sdk/apps/cli/src/tui/views/config-view-helpers.ts | 2 +- sdk/apps/cli/src/tui/views/config-view.test.ts | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/sdk/apps/cli/src/tui/views/config-view-helpers.ts b/sdk/apps/cli/src/tui/views/config-view-helpers.ts index 9f45f6ab11..82b71ede0d 100644 --- a/sdk/apps/cli/src/tui/views/config-view-helpers.ts +++ b/sdk/apps/cli/src/tui/views/config-view-helpers.ts @@ -17,7 +17,6 @@ export type ConfigAction = const CONFIG_TABS: InteractiveConfigTab[] = [ "general", - "advanced", "mcp", "skills", "rules", @@ -25,6 +24,7 @@ const CONFIG_TABS: InteractiveConfigTab[] = [ "plugins", "agents", "hooks", + "advanced", ]; export function getConfigTabs(): InteractiveConfigTab[] { diff --git a/sdk/apps/cli/src/tui/views/config-view.test.ts b/sdk/apps/cli/src/tui/views/config-view.test.ts index 8562f86c53..f7ebae0245 100644 --- a/sdk/apps/cli/src/tui/views/config-view.test.ts +++ b/sdk/apps/cli/src/tui/views/config-view.test.ts @@ -86,8 +86,9 @@ describe("config view helpers", () => { expect(toTabLabel("advanced")).toBe("Advanced"); expect(resolveInitialConfigTab("skills")).toBe("skills"); expect(resolveInitialConfigTab(undefined)).toBe("general"); - expect(getAdjacentConfigTab("general", "right")).toBe("advanced"); - expect(getAdjacentConfigTab("advanced", "left")).toBe("general"); + expect(getAdjacentConfigTab("hooks", "right")).toBe("advanced"); + expect(getAdjacentConfigTab("advanced", "left")).toBe("hooks"); + expect(getAdjacentConfigTab("advanced", "right")).toBe("general"); expect(getAdjacentConfigTab("mcp", "right")).toBe("skills"); expect(getAdjacentConfigTab("skills", "left")).toBe("mcp"); });