From 8658c6458e15711f336ebeeca900052e469de344 Mon Sep 17 00:00:00 2001 From: Ariane Emory Date: Wed, 24 Dec 2025 02:49:05 -0500 Subject: [PATCH 1/3] feat: Add persistent sidebar overlay toggle in command palette Instead of a config file option, allow toggling sidebar overlay behavior directly from the command palette. The state persists to kv.json. Changes: - Add sidebarOverlayEnabled signal persisted to kv.json (defaults to true) - Add sidebarOverlay toggle command in System category - Add optional sidebar_overlay_toggle keybind config (default: none) This provides instant toggling without requiring restart/reload. --- .../src/cli/cmd/tui/routes/session/index.tsx | 20 ++++++++++++++++++- packages/opencode/src/config/config.ts | 1 + 2 files changed, 20 insertions(+), 1 deletion(-) diff --git a/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx b/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx index c685d8c66cc..cb5394286ee 100644 --- a/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx +++ b/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx @@ -128,6 +128,7 @@ export function Session() { const [showDetails, setShowDetails] = createSignal(kv.get("tool_details_visibility", true)) const [showScrollbar, setShowScrollbar] = createSignal(kv.get("scrollbar_visible", false)) const [userMessageMarkdown, setUserMessageMarkdown] = createSignal(kv.get("user_message_markdown", true)) + const [sidebarOverlayEnabled, setSidebarOverlayEnabled] = createSignal(kv.get("sidebar_overlay", true)) const [diffWrapMode, setDiffWrapMode] = createSignal<"word" | "none">("word") const wide = createMemo(() => dimensions().width > 120) @@ -138,7 +139,10 @@ export function Session() { if (sidebar() === "auto" && wide()) return true return false }) - const sidebarOverlay = createMemo(() => sidebarVisible() && !wide()) + const sidebarOverlay = createMemo(() => { + if (!sidebarOverlayEnabled()) return false + return sidebarVisible() && !wide() + }) const contentWidth = createMemo(() => dimensions().width - (sidebarVisible() && !sidebarOverlay() ? 42 : 0) - 4) const scrollAcceleration = createMemo(() => { @@ -489,6 +493,20 @@ export function Session() { dialog.clear() }, }, + { + title: sidebarOverlayEnabled() ? "Disable sidebar overlay" : "Enable sidebar overlay", + value: "sidebar_overlay", + keybind: "sidebar_overlay_toggle", + category: "System", + onSelect: (dialog) => { + setSidebarOverlayEnabled((prev) => { + const next = !prev + kv.set("sidebar_overlay", next) + return next + }) + dialog.clear() + }, + }, { title: usernameVisible() ? "Hide username" : "Show username", value: "session.username_visible.toggle", diff --git a/packages/opencode/src/config/config.ts b/packages/opencode/src/config/config.ts index ba9d1973025..c9fd55821ed 100644 --- a/packages/opencode/src/config/config.ts +++ b/packages/opencode/src/config/config.ts @@ -469,6 +469,7 @@ export namespace Config { .default("h") .describe("Toggle code block concealment in messages"), tool_details: z.string().optional().default("none").describe("Toggle tool details visibility"), + sidebar_overlay_toggle: z.string().optional().default("none").describe("Toggle sidebar overlay mode"), model_list: z.string().optional().default("m").describe("List available models"), model_cycle_recent: z.string().optional().default("f2").describe("Next recently used model"), model_cycle_recent_reverse: z.string().optional().default("shift+f2").describe("Previous recently used model"), From fcefdddee38f704613ea78b2212eb45573b9ac14 Mon Sep 17 00:00:00 2001 From: Ariane Emory Date: Fri, 16 Jan 2026 02:03:31 -0500 Subject: [PATCH 2/3] fix: make sidebar overlay toggle functional The sidebarOverlay() memo was computed but never used in rendering. Changed the Switch/Match conditions from wide() to sidebarOverlay() so the toggle command actually controls overlay behavior. --- packages/opencode/src/cli/cmd/tui/routes/session/index.tsx | 4 ++-- packages/opencode/src/cli/cmd/tui/routes/session/sidebar.tsx | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx b/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx index 109c58c5adb..0cb42182553 100644 --- a/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx +++ b/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx @@ -1086,10 +1086,10 @@ export function Session() { - + - + sync.session.get(props.sessionID)!) @@ -77,7 +77,7 @@ export function Sidebar(props: { sessionID: string; overlay?: boolean }) { paddingBottom={1} paddingLeft={2} paddingRight={2} - position={props.overlay ? "absolute" : "relative"} + position="relative" > From 8c4daef7b058f2600e930f511be692d1033ab885 Mon Sep 17 00:00:00 2001 From: Ariane Emory Date: Fri, 16 Jan 2026 02:26:55 -0500 Subject: [PATCH 3/3] fix: remove redundant kv.set call from sidebar overlay toggle The explicit kv.set() inside the setter callback was causing a nested setStore that interfered with SolidJS reactive tracking. This made the toggle appear to work (value persisted) but the UI wouldn't update. --- packages/opencode/src/cli/cmd/tui/routes/session/index.tsx | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx b/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx index 0cb42182553..44f6098e2dc 100644 --- a/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx +++ b/packages/opencode/src/cli/cmd/tui/routes/session/index.tsx @@ -494,11 +494,7 @@ export function Session() { keybind: "sidebar_overlay_toggle", category: "System", onSelect: (dialog) => { - setSidebarOverlayEnabled((prev) => { - const next = !prev - kv.set("sidebar_overlay", next) - return next - }) + setSidebarOverlayEnabled((prev) => !prev) dialog.clear() }, },