Skip to content

Deprecate stale sessions#93

Merged
mattmoran56 merged 3 commits into
mainfrom
session/deprecate/stale-sessions
May 5, 2026
Merged

Deprecate stale sessions#93
mattmoran56 merged 3 commits into
mainfrom
session/deprecate/stale-sessions

Conversation

@mattmoran56
Copy link
Copy Markdown
Owner

@mattmoran56 mattmoran56 commented May 5, 2026

Summary

  • Removes the Stale Sessions sidebar section and all supporting state, IPC, app actions, types, mocks, and tests. The feature is superseded by:
    • PR-state grouping in the Sessions section (No PR / Draft PR / Open PR / Merged PR with inline "Clean up").
    • Auto-cleanup on merge (mergedCleanupAction / mergedCleanupDelay).
    • PR sorting (PRSortFilterMenu — number / updated / created).
  • Sidebar layout drops from 3/4 panels to 2/3 (Sessions ↔ PRs, or Sessions ↔ Claude Web ↔ PRs when enabled).
  • Persisted sessions with a stray staleAt field are loaded back into the unified sessions list automatically — no migration needed.

Screenshots

Sidebar (default — Sessions ↔ Pull Requests, no Stale Sessions panel):

Sidebar

Sidebar with Claude Web enabled (Sessions ↔ Claude Web ↔ Pull Requests):

Sidebar with Claude Web

Full layout:

Full layout

Test plan

  • Screenshots — sidebar in default mode and with Claude Web enabled, confirming no Stale Sessions panel renders.
  • npx tsc --noEmit — clean.
  • npm run test for tests/unit/stores/sessionStore.test.ts and tests/unit/components/sessions/SessionCard.test.tsx — both pass.
  • Manual smoke in npm run dev:
    • Sessions ↔ Pull Requests resize handle works.
    • Toggle Claude Web on; 3-panel layout (Sessions ↔ Claude Web ↔ PRs) with two working resize handles.
    • SessionCard ⋮ menu no longer shows "Mark as stale".
    • In Settings → Custom Buttons, "Mark as Stale" / "Reactivate Session" actions are gone.
    • Merge a PR for an open session — confirm the "Merged PR" group + auto-cleanup still work as the replacement.

🤖 Generated with Claude Code

mattmoran56 and others added 2 commits May 5, 2026 14:13
Removes the Stale Sessions sidebar section and all supporting
machinery. The feature is superseded by PR-state grouping in the
Sessions section, the merged-cleanup setting, and PR sorting.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Update e2e sidebar test to no longer assert the removed Stale Sessions section.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mattmoran56 mattmoran56 marked this pull request as ready for review May 5, 2026 13:36
Regenerate sessions, hero, and claude-web-sidebar PNGs so they reflect
the 2-3 panel layout (no Stale Sessions section) shipped in this branch.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mattmoran56 mattmoran56 merged commit c8e5f4f into main May 5, 2026
3 checks passed
@mattmoran56 mattmoran56 deleted the session/deprecate/stale-sessions branch May 5, 2026 13:50
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