Skip to content

Add setting for notification worktree reordering#89

Merged
khoi merged 2 commits into
mainfrom
settings-notification-worktree-jump-toggle
Feb 19, 2026
Merged

Add setting for notification worktree reordering#89
khoi merged 2 commits into
mainfrom
settings-notification-worktree-jump-toggle

Conversation

@khoi
Copy link
Copy Markdown
Contributor

@khoi khoi commented Feb 19, 2026

  • add a global setting moveNotifiedWorktreeToTop (default enabled) and expose it in Notifications settings as "Move notified worktree to top"
  • gate .worktreeNotificationReceived reordering behavior on that setting
  • move the toggle into RepositoriesFeature.State and wire updates from AppFeature settings fan-out so notifications no longer read @Shared(.settingsFile) in the hot path
  • add regression coverage for disabled reordering, reducer state updates, and app-level settings propagation
  • add persistence/settings test updates for the new global field
  • validate with make check, targeted xcodebuild test runs for affected suites, and make build-app

@khoi khoi merged commit da2ae6c into main Feb 19, 2026
1 check passed
@khoi khoi deleted the settings-notification-worktree-jump-toggle branch February 19, 2026 14:06
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