From 962ab3bc8cf0b051e9858db0c6c3e4b96a42bf1b Mon Sep 17 00:00:00 2001 From: adamelmore <2363879+adamdottv@users.noreply.github.com> Date: Sat, 24 Jan 2026 00:35:02 -0600 Subject: fix(app): reactive loops --- packages/app/src/context/layout.tsx | 31 +++++++++++++++++++++++++------ 1 file changed, 25 insertions(+), 6 deletions(-) (limited to 'packages/app/src/context') diff --git a/packages/app/src/context/layout.tsx b/packages/app/src/context/layout.tsx index ddd66aa2f..c24e433d1 100644 --- a/packages/app/src/context/layout.tsx +++ b/packages/app/src/context/layout.tsx @@ -267,17 +267,36 @@ export const { use: useLayout, provider: LayoutProvider } = createSimpleContext( return map }) - createEffect(() => { + const rootFor = (directory: string) => { const map = roots() - if (map.size === 0) return + if (map.size === 0) return directory + + const visited = new Set() + const chain = [directory] + + while (chain.length) { + const current = chain[chain.length - 1] + if (!current) return directory + const next = map.get(current) + if (!next) return current + + if (visited.has(next)) return directory + visited.add(next) + chain.push(next) + } + + return directory + } + + createEffect(() => { const projects = server.projects.list() const seen = new Set(projects.map((project) => project.worktree)) batch(() => { for (const project of projects) { - const root = map.get(project.worktree) - if (!root) continue + const root = rootFor(project.worktree) + if (root === project.worktree) continue server.projects.close(project.worktree) @@ -350,7 +369,7 @@ export const { use: useLayout, provider: LayoutProvider } = createSimpleContext( projects: { list, open(directory: string) { - const root = roots().get(directory) ?? directory + const root = rootFor(directory) if (server.projects.list().find((x) => x.worktree === root)) return globalSync.project.loadSessions(root) server.projects.open(root) @@ -384,7 +403,7 @@ export const { use: useLayout, provider: LayoutProvider } = createSimpleContext( setStore("sidebar", "width", width) }, workspaces(directory: string) { - return createMemo(() => store.sidebar.workspaces[directory] ?? store.sidebar.workspacesDefault ?? false) + return () => store.sidebar.workspaces[directory] ?? store.sidebar.workspacesDefault ?? false }, setWorkspaces(directory: string, value: boolean) { setStore("sidebar", "workspaces", directory, value) -- cgit v1.2.3