Skip to content

feat(web): add pin-in-progress session layout modes - #1758

Open
techotaku39 wants to merge 1 commit into
tiann:mainfrom
techotaku39:feat/web-pin-session-layout-modes
Open

feat(web): add pin-in-progress session layout modes#1758
techotaku39 wants to merge 1 commit into
tiann:mainfrom
techotaku39:feat/web-pin-session-layout-modes

Conversation

@techotaku39

Copy link
Copy Markdown
Contributor

Summary

  • Add a persisted Detailed/Combined layout preference for pinned in-progress sessions.
  • Preserve the upstream Detailed layout with separate In progress and Active sessions sections.
  • Add Combined layout with project-directory grouping, all active sessions shown, and expandable archived sessions below a divider.
  • Keep the layout selector in the same settings block as pinning and disable it when pinning is turned off.
  • Add localized copy, same-tab preference synchronization, unit tests, and Chromium coverage.

Validation

  • bun run typecheck — passed.
  • Web targeted Vitest tests — passed, 52/52.
  • bun run build — passed.
  • Terminal wrap fidelity Chromium E2E — passed, 2/2.
  • Session list layout Chromium E2E — passed, 2/2.

Related Issues

None

AI Assistance

Implemented and validated with OpenAI Codex (GPT-5.6).

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Findings

  • None.

Summary

  • Review mode: initial
  • No reportable correctness, security, regression, data-loss, performance, or maintainability issues found in the latest diff. Residual risk: the new split active/archived grouping has no focused coverage for project-pinned sessions changing active state or for its interaction with active-only/search filters.

Testing

  • Not run (automation; PR content was not executed). GitHub status at review time: integration passed; test and pr-review pending.

HAPI Bot

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