Skip to content

feat(sessions): add collapsible pinned section - #709

Closed
Zhangs-11 wants to merge 1 commit into
agegr:mainfrom
Zhangs-11:feat/collapsible-pinned-sessions
Closed

feat(sessions): add collapsible pinned section#709
Zhangs-11 wants to merge 1 commit into
agegr:mainfrom
Zhangs-11:feat/collapsible-pinned-sessions

Conversation

@Zhangs-11

@Zhangs-11 Zhangs-11 commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a hover action to pin or unpin a session within the current project
  • move pinned session families into a collapsible section without duplicating them in the regular list
  • add a card-level setting for the global default collapsed state
  • persist pins and the default preference in browser localStorage, and prune stale pins after sessions disappear

Test plan

  • npm test (891 passing)
  • node_modules/.bin/tsc --noEmit
  • npm run lint
  • git diff --check
  • verified pinning, unpinning, project isolation, collapse/expand, reload persistence, and the global default setting in a real browser

Demo

Pin sessions, collapse the pinned section, and configure its global default

Demo provenance:

  • demonstrated commit: 0da9c81de8d2cdc5d5fe2d782988d2fb82d18c7e
  • served from the clean feat/collapsible-pinned-sessions worktree at http://127.0.0.1:30156
  • mode: Next.js development server (the repository instructions prohibit next build during development)
  • browser: fresh isolated headless Chrome profile; the repository-declared Playwright CDP fallback was used after agent-browser control became unavailable
  • transport: normal Pi Web API/RPC flow; no fixtures, mocks, DOM-event injection, or test-only hooks
  • real model rounds: yes — two GPT-6-Astra rounds created the demonstrated sessions

Related to #406.

@Zhangs-11
Zhangs-11 force-pushed the feat/collapsible-pinned-sessions branch from f0c31e5 to 0da9c81 Compare September 5, 2026 10:28
Zhangs-11 added a commit to Zhangs-11/pi-web that referenced this pull request Sep 5, 2026
@agegr

agegr commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Thanks for the contribution. I plan to redesign session organization more broadly, so I am closing this PR for now.

@agegr agegr closed this Sep 5, 2026
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.

2 participants