Skip to content

v5.0.1 — Housekeeping: IPC modules, store cleanup, a11y, reduced-motion#26

Merged
Wintersta7e merged 3 commits intomainfrom
chore/housekeeping
Apr 8, 2026
Merged

v5.0.1 — Housekeeping: IPC modules, store cleanup, a11y, reduced-motion#26
Wintersta7e merged 3 commits intomainfrom
chore/housekeeping

Conversation

@Wintersta7e
Copy link
Copy Markdown
Owner

Summary

  • Extract cost:bind/cost:unbind IPC handlers from index.ts to ipc-cost.ts (follows module pattern)
  • Move worktreePaths state from UiSlice to SessionsSlice (session lifecycle)
  • Extract executeWithRetry from workflow-engine's processNode (reduce god-function)
  • 4 interactive click targets enlarged to 32x32px minimum (tab-close, search buttons, status bar, toggle)
  • 8 animations get prefers-reduced-motion: reduce overrides

Blockers / Notes

  • 608 tests passing, lint clean
  • Zero behavioral changes — all refactoring is internal

LOW-4: Add min-width/min-height 32px to TerminalSearchBar buttons,
Titlebar tab-close, StatusBar status-cmd, and Toggle wrapper/button.

LOW-5: Add @media (prefers-reduced-motion: reduce) blocks to all 8
CSS files with animations that lacked overrides.

Co-Authored-By: Rooty
Pulls the retry loop, attempt tracking, and run-history recording out
of processNode into a dedicated executeWithRetry helper that returns
'success' | 'failed' | 'stopped'. processNode now delegates execution
and maps the result to scheduler calls, reducing its responsibilities.

Co-Authored-By: Rooty
- Extract cost:bind/cost:unbind from inline handlers in index.ts to
  src/main/ipc/ipc-cost.ts (follows existing IPC module pattern)
- Remove unused ipcMain and SAFE_ID_RE imports from index.ts
- Move worktreePaths/setWorktreePath/clearWorktreePath from UiSlice
  to SessionsSlice (logically belongs with session lifecycle)

Co-Authored-By: Rooty
@Wintersta7e Wintersta7e merged commit 3a55963 into main Apr 8, 2026
1 check passed
@Wintersta7e Wintersta7e deleted the chore/housekeeping branch April 8, 2026 13:19
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