Skip to content

Add unarchive action to session toolbar - #243

Open
tella-butler wants to merge 1 commit into
mainfrom
unarchive-session-toolbar
Open

Add unarchive action to session toolbar#243
tella-butler wants to merge 1 commit into
mainfrom
unarchive-session-toolbar

Conversation

@tella-butler

@tella-butler tella-butler commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • show Unarchive in the archived session toolbar where Add to sidebar appears for non-archived sessions
  • route the action through App's existing optimistic unarchive flow
  • preserve Add to sidebar behavior and avoid a duplicate archived-session overflow action
  • cover archived desktop and non-archived phone action rendering

Checks

  • bun test packages/core/opensession-server/src/frontend/components/SessionSidebarAction.test.tsx
  • bun run lint
  • bun run typecheck
  • light-mode capture harness exercised at 1440×900 and 390×844

Started by Kent de Bruin in this OS session

@open-session-os-tella-dev

open-session-os-tella-dev Bot commented Aug 28, 2026

Copy link
Copy Markdown

🤖 OS review · approve · confidence 5/5

Safe to merge. The PR adds the archived-session Unarchive action in the existing desktop and phone placements, routes it through App's optimistic flow, preserves Add to sidebar, and removes the duplicate overflow action without breaking other callers.

💡 Labels: os-adversarial — deeper two-pass review · os-simplify — quality cleanup pass · os-auto-fix — fix anything outstanding and push until CI passes.
Reviewed 831a9fc · GPT-5.6 Sol · earlier reviews collapse above · open session

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
opensession Ready Ready Preview Aug 28, 2026 3:28pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant