Skip to content

fix(desktop): hide sessions during removal - #44913

Merged
Brendonovich merged 2 commits into
v2from
v2-archive-optimistic-ui
Aug 25, 2026
Merged

fix(desktop): hide sessions during removal#44913
Brendonovich merged 2 commits into
v2from
v2-archive-optimistic-ui

Conversation

@opencode-agent

@opencode-agentopencode-agentBot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a Desktop-only session mutation overlay around the shared Solid data source
  • apply pending removals when Desktop reads local or Home-query session lists
  • route Desktop Home and session-timeline deletion through the overlay, restoring rows on failure and retaining successful removals until the server event lands

Desktop V2 counterpart to #44905. The shared client data layer is unchanged.

Checks

  • bun typecheck (packages/app)
  • focused Desktop data, Home session, and session-domain unit tests (packages/app)
  • bun typecheck (packages/client)
  • production benchmark attempted before and after; the existing home navigation fixture did not load its target row, so neither run emitted comparable metrics

Requested by: @Brendonovich (Brendan via Slack)

@opencode-agent
opencode-agentBotforce-pushed the v2-archive-optimistic-ui branch from 76763ca to 7f28044CompareAugust 25, 2026 07:44
@opencode-agentopencode-agentBot changed the title fix(tui): remove deleted sessions from local datafix(desktop): hide sessions during removalAug 25, 2026
@opencode-agent
opencode-agentBotforce-pushed the v2-archive-optimistic-ui branch from 7f28044 to 39af454CompareAugust 25, 2026 08:03
@opencode-agent
opencode-agentBotforce-pushed the v2-archive-optimistic-ui branch from 39af454 to ec81b8bCompareAugust 25, 2026 08:06
# Conflicts:
#	packages/app/src/home/sessions/controller.tsx
@Brendonovich
Brendonovich merged commit 7601ab9 into v2Aug 25, 2026
6 of 7 checks passed
@Brendonovich
Brendonovich deleted the v2-archive-optimistic-ui branch August 25, 2026 13:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@Brendonovich