Skip to content

feat: unify shared mail actions - #339

Closed
salmonumbrella wants to merge 3 commits into
maathimself:mainfrom
salmonumbrella:feat/shared-mail-actions
Closed

feat: unify shared mail actions#339
salmonumbrella wants to merge 3 commits into
maathimself:mainfrom
salmonumbrella:feat/shared-mail-actions

Conversation

@salmonumbrella

Copy link
Copy Markdown
Contributor

Summary

Consolidates supported mail mutations behind the registry-owned command controller so Command-K, shortcuts, list and pane controls, context menus, bulk actions, hover actions, and mobile swipes share target resolution and execution behavior.

This is the third stacked command-system PR. It includes prerequisite commits from #335 and #337; review the last commit only (6a604730).

Closes #338.

Changes

  • adds dependency-injected executors for archive, move, snooze, read/unread, star, trash, spam/ham, reply/forward, and GTD classification
  • preserves account-scoped frozen targets, thread expansion, optimistic rollback, read/removal guards, the 4.5-second undo window, unload flushing, and exact partial-result counts
  • routes the message list, message pane, context menu, bulk/hover/swipe surfaces, and compatibility shortcut events through the shared controller
  • adds typed Move/Snooze continuations, native palette icons, localized aliases/outcomes across all seven locales, and regression coverage for IME-safe palette behavior retained from feat: add contextual command palette #337
  • adds no backend route, migration, runtime dependency, or cmdk dependency

Testing

  • cd frontend && npm test — 1,648 passed
  • cd frontend && npm run lint — zero warnings
  • cd frontend && npm run build — passed (existing chunk-size advisory only)
  • git diff --check 20589608..HEAD — clean
  • verified exactly one PR-specific commit over feat: add contextual command palette #337

Contributor License Agreement

By submitting this pull request I confirm that:

  • I have read and agree to the Contributor License Agreement.
  • My contribution is my own original work (or I have identified any third-party material and confirmed it is compatible with the CLA).
  • I have the right to submit this contribution under the terms of the CLA.

@salmonumbrella
salmonumbrella force-pushed the feat/shared-mail-actions branch from 6a60473 to f2cb008 Compare August 1, 2026 02:51
@maathimself maathimself closed this Aug 3, 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.

Consolidate shared mail actions behind the command controller

2 participants