78:T11d2,Follow-up to #168. Moving a session between Active and Settled still stuttered. The animation timing was fine, but four full React renders ran inside the 240ms row animation and blocked the main thread. Each render also reached well beyond the sidebar. ## What changed **One render on the click, instead of four** - The client store moves the session at once and puts it back if the environment refuses (`packages/environment-client/src/settle.ts`, used by the websocket and mock clients). This replaces the view's own stand-in (`useOptimisticSettle`) and the extra render when that stand-in was released. - While a request is in flight, its `session.updated` echo is held back and only recorded as what the environment confirmed. This stops earlier clicks from pulling the row back and forth. - `session.updated` and re-listing now keep the state as it was when they restate a session, so the redundant echo render is gone. - One small render remains when the answer lands: it swaps the local timestamp for the server's. **No layout projection on the rows** - `layout: 'position'` is dropped from the Active and Settled rows. The height animation already moves the neighbouring rows, and layout made Motion measure every row on each render. - Trade-off: a row that changes places, for example one moving to the top on new activity, now jumps instead of sliding. **The sidebar's session churn stays in the sidebar** - Sessions are now served on their own context, `SidebarSessionsContext` / `useSidebarSessions`, split out of `SidebarDataValue`. The web and desktop providers both supply it. - The web provider keeps rows, groups and the whole map as the same objects while nothing the sidebar shows has changed. Composer picks and thread ids no longer reach the sidebar. - The catalog and commands in `SidebarData` no longer change with every session. So the new-session landing, the ⌘K palette and the Add project dialog stop re-rendering with the list. - The topbar breadcrumb reads only the open session's title. - SessionState read the whole active session object through `openDraft`'s dependencies. It now reads only the session's id and workspace. ActiveThread builds its details from the fields it shows. A settle, title change or composer pick on the session you're viewing no longer re-renders the composer, the chat pane or the thread. ## Measurements The probe used real Chrome driven by playwright. It clicked Settle and Move back to active three times each on a copy of real data, with fix and baseline built from the same data and ports. Frame times come from rAF sampling plus the Long Animation Frames API. | Build / CPU | Baseline worst frame | Fix worst frame | Baseline blocking per move | Fix blocking per move | |---|---|---|---|---| | Production, 4x slowed CPU | 100–217ms | 50–83ms | 24–372ms | 5–61ms | | Production, 4x, settling the open session | 100–200ms | 50–83ms | 42–173ms | 5–43ms | | Dev server, normal CPU | mostly 50ms (33–67) | mostly 17–34ms (17–50) | – | – | Painted frames per move went from 2–4 to 3–6 at prod 4x, and from 5–9 to 9–12 in dev. Production at normal CPU was already smooth and still is. A React commit recorder confirms that a settle now renders only sidebar components, even with the settled session open in the chat pane. ### Next candidate (not in this PR) With the session open, the largest remaining app-level cost in the profile is Fluid's `SidebarMenu` hover tracking (`useMenuScope` / `use-fluid-hover`) on the Settled list. Motion's own row rendering comes next. ## Tests - `environment-client`: added tracker tests (instant move, refusal, the latest click winning, held-back echo), reducer no-op tests, and a websocket test showing that neither the echo nor the answer renders again. - `app-core` `sidebar-settle.test.tsx`: now runs through the mock environment client and providers. - It checks that the row moves before the environment answers. - It checks that settling the open session renders none of SessionState, SidebarData, ActiveThread or Composer. This was mutation-checked: re-adding the whole-session dependency makes it fail with each reader rendering twice. - Local runs: - environment-client: 172 passed. - app-core: all passed except `boundary.test.ts`, which timed out under load and passes when run alone. - web: 118 passed. - Typecheck is clean for environment-client, app-core, web and desktop (web). Lint and prettier are clean on the changed files. 🤖 Generated with [Claude Code](https://claude.com/claude-code) 79:T95a,

**[Medium risk]** Refactors session state updates to optimize sidebar rendering\. The PR appears safe to merge; no actionable issue was identified.

Summary

The PR moves optimistic settling into the environment clients, separates sidebar sessions from catalog context, preserves unchanged session identities, and removes row layout projection to reduce renders during moves. - The websocket and mock clients now show a settle immediately, hold its event echo, and restore the confirmed value on refusal. - Web sidebar data and active\-session readers subscribe more narrowly so unrelated session changes do not reach the main pane.

Diagram

```mermaid %%{init: {'theme': 'neutral'}}%% flowchart LR Click[Settle click] --> Tracker[Settle tracker] Tracker --> Store[Optimistic session state] Tracker --> Server[Environment request] Server --> Echo[Session update echo] Echo --> Tracker Server --> Answer[Answer or refusal] Answer --> Tracker Store --> Sidebar[Sidebar sessions context] ``` _{Reviews (1) · Last reviewed commit: ["fix(sidebar): settle in one render and k..."](https://github.com/calledforth/openmanager/commit/385cdb88bfe6f3e113cd9225aad2480b51d20808)} 7a:T49c,fix(sidebar): settle in one render and keep session churn out of the main pane Settling or unsettling a session ran four full renders inside the 240ms row animation, and each reached far past the sidebar: - The client store now moves the session at once and reverts on refusal (`createSettleTracker`), replacing the view's own stand-in. The echo while a request is in flight is held back, and `session.updated` and re-listings that restate a session keep the state as it was. - Rows no longer use `layout: 'position'`; the height animation already moves their neighbours, and layout made Motion measure every row on each render. - Sessions are served on their own context (`useSidebarSessions`), kept as the same objects while nothing the sidebar shows changed. The catalog and commands no longer change with every session, so the landing, palette and topbar stop re-rendering with the list. - Session state and the active thread read the session on screen by the fields they use, so a settle, title or composer pick on it no longer re-renders the composer, the chat pane and the thread. Co-Authored-By: Claude Opus 5.5 (1M context) 62:["$","$L75",null,{"entries":[{"key":["platform",{"userId":"anon"},"pulls","page",{"owner":"calledforth","repo":"openmanager","pullNumber":178}],"data":{"repositoryId":"2045","detail":{"author":{"avatarUrl":"https://cdn.trylle.com/media/images/df85a2a863690ececdfb4ceab2ebc19a?key=AYAb%2FP%2FITdOrIocJPPXAonOcVf4PkC6tWA%2FOIgQlcYyqnQQIaNnaIVD85WBzhMjz7crp9yDFQCczdUtjOlw7OfZ8AuzqFPX%2FcqsTEhdldE%2F6mDiLciX5tf9MbvCL9%2BLdJ9BZgwrkD3bRDnCxtIXBNT4aRcfWVjEqG3spdGCdDBtfTAXg8Qp%2FqZl%2FkQ%3D%3D","externalProvider":"github","login":"calledforth","type":"User","url":"https://github.com/calledforth"},"authorId":"65070429","baseRefName":"main","closedAt":null,"comments":1,"createdAt":"2026-09-28T10:41:00Z","headRefName":"fix/sidebar-render-isolation","id":"4906670","isDraft":false,"labels":[],"number":178,"repository":{"name":"openmanager","owner":"calledforth","provider":"forgejo"},"state":"open","title":"fix(sidebar): move settled rows at once and keep session churn out of the main pane","updatedAt":"2026-10-07T18:02:11Z","url":"https://trylle.com/calledforth/openmanager/pulls/178","body":"$78","additions":596,"deletions":210,"changedFiles":15,"commits":1,"reviewComments":0,"headSha":"385cdb88bfe6f3e113cd9225aad2480b51d20808","baseSha":"af2253b48332e161dcb0241660d41410c0832da2","headRepoOwner":"calledforth","headRepoName":"openmanager","isMerged":false,"mergeCommitSha":null,"mergedBy":null,"mergeable":false,"mergeableState":"dirty","conflictingFiles":["packages/app-core/src/components/sidebar/WorkspaceSidebar.tsx","packages/app-core/src/components/sidebar/WorkspaceSidebarView.tsx","packages/app-core/src/providers/environment-application.tsx","packages/app-core/src/providers/sidebar-provider.ts","packages/app-core/tests/sidebar-settle.test.tsx","packages/environment-client/src/mock.ts","packages/environment-client/src/state.ts","packages/environment-client/src/websocket.ts","packages/environment-client/tests/state.test.ts","packages/environment-client/tests/websocket.test.ts"],"requestedTeams":[],"requestedReviewers":[],"graphqlId":"issue:178"},"comments":[{"id":26689981,"graphqlId":"26689981","body":"$79","createdAt":"2026-09-28T10:46:49Z","author":{"avatarUrl":"https://cdn.trylle.com/media/images/d5fdefbe765e9f429d38a31328621291?key=Acl0%2BlhblDOxCrSlqfonNDhgrt1LwjH88FRgEld3W8TJMWD14w5hVMk945%2FUo6%2F4SXM8sWkkR1IS3%2FPkJicqQw3Nzkwzis1r900F6Ii81AW%2B6%2FgoEHT7u6ClQhGHk5zI5yHvly2ZTjc6HDW38z1iK%2FBzByCTQdN9niTDbBlKfNdTv6bYqay%2Boc9at4A%3D","externalProvider":"github","login":"greptile-apps[bot]","type":"Bot","url":"https://github.com/apps/greptile-apps"}}],"commits":[{"sha":"385cdb88bfe6f3e113cd9225aad2480b51d20808","message":"$7a","createdAt":"2026-09-28T16:10:34+05:30","author":{"id":128,"avatarUrl":"https://cdn.trylle.com/media/images/df85a2a863690ececdfb4ceab2ebc19a?key=ATVW84LPIaR4kyz6cU5x7tlwLlIdlIesqGFdvCQgv65nYLIFL5Vfl5wvIrU370FqK6f1eFsWpJN1PITP6Be4SMtzUKGZCD9JqUNSBeFv0QtIx1OtF1g2k7JxyI9C0gJqWbhK12vaE1OfuVrfcKUIKv5GrkTtzpj5svfyNTx9vNMy0dJTLI1cQMASIQ%3D%3D","login":"calledforth","name":"calledforth","type":"User","url":"https://trylle.com/calledforth"},"verification":{"verified":false,"reason":"gpg.error.not_signed_commit"}}],"events":[{"id":26689288,"event":"pull_push","createdAt":"2026-09-28T10:41:00Z","actor":{"id":128,"avatarUrl":"https://cdn.trylle.com/media/images/df85a2a863690ececdfb4ceab2ebc19a?key=ATVW84LPIaR4kyz6cU5x7tlwLlIdlIesqGFdvCQgv65nYLIFL5Vfl5wvIrU370FqK6f1eFsWpJN1PITP6Be4SMtzUKGZCD9JqUNSBeFv0QtIx1OtF1g2k7JxyI9C0gJqWbhK12vaE1OfuVrfcKUIKv5GrkTtzpj5svfyNTx9vNMy0dJTLI1cQMASIQ%3D%3D","login":"calledforth","name":"calledforth","type":"User","url":"https://trylle.com/calledforth","authorAssociation":"OWNER"},"body":"{\"is_force_push\":false,\"commit_ids\":[\"385cdb88bfe6f3e113cd9225aad2480b51d20808\"]}","commitIds":["385cdb88bfe6f3e113cd9225aad2480b51d20808"]}],"reviews":[],"stackEvents":[],"commentPagination":{"totalCount":1,"perPage":30,"loadedPages":[1]},"eventPagination":{"loadedPages":[1],"hasMore":false},"headRefDeleted":false},"fetchedAt":1793408510146}],"children":"$L7b"}] 64:null 6b:[["$","title","0",{"children":"Commits · fix(sidebar): move settled rows at once and keep session churn out of the main pane · calledforth/openmanager | Trylle"}],["$","meta","1",{"name":"description","content":"Commits on fix(sidebar): move settled rows at once and keep session churn out of the main pane (#178) in calledforth/openmanager."}],["$","meta","2",{"name":"application-name","content":"Trylle"}],["$","link","3",{"rel":"manifest","href":"/manifest.json","crossOrigin":"$undefined"}],["$","meta","4",{"name":"keywords","content":"Commits · fix(sidebar): move settled rows at once and keep session churn out of the main pane · calledforth/openmanager, calledforth/openmanager, calledforth/openmanager repository, openmanager, TypeScript repository, source code, trylle, agent-native git platform, git platform for agents and humans, git platform for ai agents, coding agent pull requests, ai code review, repository automation, stacked pull requests, two-way github sync, github-compatible actions, try cli"}],["$","meta","5",{"name":"robots","content":"index, follow, max-image-preview:large, max-snippet:-1, max-video-preview:-1"}],["$","meta","6",{"name":"mobile-web-app-capable","content":"yes"}],["$","meta","7",{"name":"googlebot","content":"index, follow, max-image-preview:large, max-snippet:-1, max-video-preview:-1"}],["$","meta","8",{"name":"twitter:domain","content":"trylle.com"}],["$","meta","9",{"name":"twitter:url","content":"https://trylle.com/calledforth/openmanager/pulls/178/commits"}],["$","link","10",{"rel":"canonical","href":"https://trylle.com/calledforth/openmanager/pulls/178/commits"}],["$","meta","11",{"name":"format-detection","content":"telephone=no"}],["$","meta","12",{"name":"mobile-web-app-capable","content":"yes"}],["$","meta","13",{"name":"apple-mobile-web-app-title","content":"Trylle"}],["$","meta","14",{"name":"apple-mobile-web-app-status-bar-style","content":"black-translucent"}],["$","meta","15",{"property":"og:title","content":"Commits · fix(sidebar): move settled rows at once and keep session churn out of the main pane · calledforth/openmanager | Trylle"}],["$","meta","16",{"property":"og:description","content":"Commits on fix(sidebar): move settled rows at once and keep session churn out of the main pane (#178) in calledforth/openmanager."}],["$","meta","17",{"property":"og:url","content":"https://trylle.com/calledforth/openmanager/pulls/178/commits"}],["$","meta","18",{"property":"og:site_name","content":"Trylle"}],["$","meta","19",{"property":"og:locale","content":"en_US"}],["$","meta","20",{"property":"og:image","content":"https://trylle.com/api/og.png?kind=repo&owner=calledforth&repo=openmanager"}],["$","meta","21",{"property":"og:image:secure_url","content":"https://trylle.com/api/og.png?kind=repo&owner=calledforth&repo=openmanager"}],["$","meta","22",{"property":"og:image:type","content":"image/png"}],["$","meta","23",{"property":"og:image:width","content":"1200"}],["$","meta","24",{"property":"og:image:height","content":"630"}],["$","meta","25",{"property":"og:image:alt","content":"calledforth/openmanager repository preview on Trylle"}],["$","meta","26",{"property":"og:type","content":"website"}],["$","meta","27",{"name":"twitter:card","content":"summary_large_image"}],["$","meta","28",{"name":"twitter:site","content":"@trylle"}],["$","meta","29",{"name":"twitter:title","content":"Commits · fix(sidebar): move settled rows at once and keep session churn out of the main pane · calledforth/openmanager | Trylle"}],["$","meta","30",{"name":"twitter:description","content":"Commits on fix(sidebar): move settled rows at once and keep session churn out of the main pane (#178) in calledforth/openmanager."}],["$","meta","31",{"name":"twitter:image","content":"https://trylle.com/api/og.png?kind=repo&owner=calledforth&repo=openmanager"}],["$","meta","32",{"name":"twitter:image:alt","content":"calledforth/openmanager repository preview on Trylle"}],"$L7c","$L7d","$L7e","$L7f"]

Commits on Mon, Sep 28, 2026