Skip to content

fix(ui): expose archived conversations in the sidebar - #2573

Merged
Astro-Han merged 1 commit into
apache:mainfrom
me2seeks:fix/2317-restore-archived-sessions
Aug 9, 2026
Merged

fix(ui): expose archived conversations in the sidebar#2573
Astro-Han merged 1 commit into
apache:mainfrom
me2seeks:fix/2317-restore-archived-sessions

Conversation

@me2seeks

@me2seeks me2seeks commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Archived conversations already remain in the Desktop catalog and already have a Host-backed Unarchive row action, but the sidebar exposes no way to enter its existing archived filter. This adds explicit Conversations and Archived destinations to the session navigation, so an archived conversation stays discoverable and can be restored through the existing row menu.

The change does not add another catalog, storage, or IPC path. It only makes the existing navigation state reachable and adds localized, selected-state coverage plus an archived-list Storybook state.

Fixes #2317

中文对照

已归档会话本来就保留在 Desktop catalog 中,行菜单也已经具备由 Runtime Host 支持的“取消归档”动作;但侧栏没有任何入口能进入已有的 archived 筛选状态。本 PR 在会话导航中显式加入“会话”和“已归档”两个入口,使归档记录仍可发现,并能通过现有行菜单恢复。

本次修改不新增 catalog、存储或 IPC 路径,只让已有导航状态真正可达,并补充中英文选中态测试与归档列表 Storybook 场景。

Verification

  • npm --workspace @maka/ui test — 264 tests passed
  • npm --workspace @maka/desktop run typecheck:stories
  • npm --workspace @maka/desktop run build-storybook
  • npm --workspace @maka/desktop run smoke:storybook
  • npx biome check on all changed source, test, and story files
  • Rendered the archived-list story in Chromium and checked the localized destinations, selected state, and archived row

Checklist

  • Tests cover the change and fail without it
  • Lint, format, typecheck and the affected suites pass locally

Does this PR entail a change in behavior?

  • Yes — archived conversations are reachable and restorable from the Desktop sidebar
  • No

@Astro-Han Astro-Han left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. This keeps the change nicely scoped to the existing navigation and unarchive paths without introducing another state or storage boundary. Thanks for the focused fix!

@Astro-Han
Astro-Han merged commit c872ac0 into apache:main Aug 9, 2026
12 checks passed
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.

Desktop: archived conversations have no way to restore/unarchive in the UI

2 participants