refactor(renderer): relocate shortcut-registration store into stores

Move the transient global-shortcut registration Zustand store out of the
hooks layer into the stores layer, matching the documented state-ownership
rule for session client state.

- Add stores/useShortcutRegistrationStore.ts and mirror its
  ShortcutRegistrationStore type in stores/types.ts
- Export it from the stores barrel
- Trim hooks/useShortcutRegistration.ts to the effect hook, importing the
  store from the stores layer
- Repoint the seven consumers (GlobalEffects, SystemSettings, and the test
  setup/helpers) to the barrel

No behavior change: registration, revert-on-failure, and the surfaced error
are identical.

Author: setchy

src/renderer/__helpers__/test-utils.tsx

@@ -4,8 +4,13 @@ import { type InitialEntry, MemoryRouter } from 'react-router-dom';
 
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
 
-import { useShortcutRegistrationStore } from '../hooks/useShortcutRegistration';
-import { type FiltersStore, useAccountsStore, useFiltersStore, useSettingsStore } from '../stores';
+import {
+  type FiltersStore,
+  useAccountsStore,
+  useFiltersStore,
+  useSettingsStore,
+  useShortcutRegistrationStore,
+} from '../stores';
 
 import type { Account, SettingsState } from '../types';
 

src/renderer/__helpers__/visual.setup.ts

@@ -162,7 +162,7 @@ beforeEach(async () => {
   const { resetHookMocks } = await import('./hook-mocks');
   resetHookMocks();
 
-  const { useShortcutRegistrationStore } = await import('../hooks/useShortcutRegistration');
+  const { useShortcutRegistrationStore } = await import('../stores');
   useShortcutRegistrationStore.getState().reset();
 });
 

src/renderer/__helpers__/vitest.setup.ts

@@ -153,7 +153,7 @@ beforeEach(async () => {
   const { resetHookMocks } = await import('./hook-mocks');
   resetHookMocks();
 
-  const { useShortcutRegistrationStore } = await import('../hooks/useShortcutRegistration');
+  const { useShortcutRegistrationStore } = await import('../stores');
   useShortcutRegistrationStore.getState().reset();
 });
 

src/renderer/components/GlobalEffects.test.tsx

@@ -3,8 +3,7 @@ import { act } from '@testing-library/react';
 import { renderWithProviders } from '../__helpers__/test-utils';
 import { mockGitHubCloudAccount } from '../__mocks__/account-mocks';
 
-import { useShortcutRegistrationStore } from '../hooks/useShortcutRegistration';
-import { useAccountsStore, useSettingsStore } from '../stores';
+import { useAccountsStore, useSettingsStore, useShortcutRegistrationStore } from '../stores';
 
 import { Theme } from '../types';
 

src/renderer/components/GlobalEffects.tsx

@@ -6,11 +6,13 @@ import { useNotificationsHost } from '../hooks/notifications/useNotificationsHos
 import { useAccounts } from '../hooks/useAccounts';
 import { useAppearance } from '../hooks/useAppearance';
 import { useOnlineStatus } from '../hooks/useOnlineStatus';
+import { useShortcutRegistration } from '../hooks/useShortcutRegistration';
 import {
-  useShortcutRegistration,
+  useAccountsStore,
+  useFiltersStore,
+  useSettingsStore,
   useShortcutRegistrationStore,
-} from '../hooks/useShortcutRegistration';
-import { useAccountsStore, useFiltersStore, useSettingsStore } from '../stores';
+} from '../stores';
 
 import { setTrayIconColorAndTitle } from '../utils/system/tray';
 

src/renderer/components/settings/SystemSettings.tsx

@@ -5,8 +5,11 @@ import { Banner, Button, ButtonGroup, IconButton, Stack, Text } from '@primer/re
 
 import { APPLICATION } from '../../../shared/constants';
 
-import { useShortcutRegistrationStore } from '../../hooks/useShortcutRegistration';
-import { DEFAULT_SETTINGS_STATE, useSettingsStore } from '../../stores';
+import {
+  DEFAULT_SETTINGS_STATE,
+  useSettingsStore,
+  useShortcutRegistrationStore,
+} from '../../stores';
 
 import { Checkbox } from '../fields/Checkbox';
 import { RadioGroup } from '../fields/RadioGroup';

src/renderer/hooks/useShortcutRegistration.test.ts

@@ -1,10 +1,10 @@
 import { renderHook, waitFor } from '@testing-library/react';
 
-import { DEFAULT_SETTINGS_STATE, useSettingsStore } from '../stores';
+import { DEFAULT_SETTINGS_STATE, useSettingsStore, useShortcutRegistrationStore } from '../stores';
 
 import type { KeyboardAcceleratorShortcut } from '../types';
 
-import { useShortcutRegistration, useShortcutRegistrationStore } from './useShortcutRegistration';
+import { useShortcutRegistration } from './useShortcutRegistration';
 
 describe('renderer/hooks/useShortcutRegistration.ts', () => {
   it('registers the configured shortcut and records it as last applied', async () => {

src/renderer/hooks/useShortcutRegistration.ts

@@ -1,49 +1,9 @@
 import { useEffect } from 'react';
 
-import { create } from 'zustand';
-
-import { DEFAULT_SETTINGS_STATE, useSettingsStore } from '../stores';
-
-import type { KeyboardAcceleratorShortcut } from '../types';
+import { useSettingsStore, useShortcutRegistrationStore } from '../stores';
 
 import { applyKeyboardShortcut } from '../utils/system/comms';
 
-interface ShortcutRegistrationStore {
-  /**
-   * User-facing error when the configured global shortcut could not be
-   * registered (e.g. already in use by another application).
-   */
-  shortcutRegistrationError: string | null;
-
-  /**
-   * The last accelerator that registered successfully; used to revert the
-   * setting when a newly chosen accelerator fails to register.
-   */
-  lastAppliedShortcut: KeyboardAcceleratorShortcut;
-
-  setShortcutRegistrationError: (error: string | null) => void;
-  setLastAppliedShortcut: (accelerator: KeyboardAcceleratorShortcut) => void;
-  clearShortcutRegistrationError: () => void;
-  reset: () => void;
-}
-
-/**
- * Transient (non-persisted) state for global keyboard shortcut registration.
- */
-export const useShortcutRegistrationStore = create<ShortcutRegistrationStore>()((set) => ({
-  shortcutRegistrationError: null,
-  lastAppliedShortcut: DEFAULT_SETTINGS_STATE.openGitifyShortcut,
-
-  setShortcutRegistrationError: (error) => set({ shortcutRegistrationError: error }),
-  setLastAppliedShortcut: (accelerator) => set({ lastAppliedShortcut: accelerator }),
-  clearShortcutRegistrationError: () => set({ shortcutRegistrationError: null }),
-  reset: () =>
-    set({
-      shortcutRegistrationError: null,
-      lastAppliedShortcut: DEFAULT_SETTINGS_STATE.openGitifyShortcut,
-    }),
-}));
-
 /**
  * Registers the global open/close shortcut whenever the setting changes,
  * reverting to the last working accelerator when registration fails.

src/renderer/stores/index.ts

@@ -8,3 +8,4 @@ export {
   getNotificationFailureKey,
 } from './useNotificationActionFailuresStore';
 export { default as useSettingsStore } from './useSettingsStore';
+export { default as useShortcutRegistrationStore } from './useShortcutRegistrationStore';

src/renderer/stores/types.ts

@@ -5,6 +5,7 @@ import type {
   Forge,
   GitifyError,
   Hostname,
+  KeyboardAcceleratorShortcut,
   ReasonFilterValue,
   ReviewRequestTypeFilterValue,
   SearchToken,
@@ -273,3 +274,29 @@ export interface NotificationActionFailuresActions {
  */
 export type NotificationActionFailuresStore = NotificationActionFailuresState &
   NotificationActionFailuresActions;
+
+// ============================================================================
+// Shortcut Registration Store Types
+// ============================================================================
+
+/**
+ * Transient (non-persisted) state for global keyboard shortcut registration.
+ */
+export interface ShortcutRegistrationStore {
+  /**
+   * User-facing error when the configured global shortcut could not be
+   * registered (e.g. already in use by another application).
+   */
+  shortcutRegistrationError: string | null;
+
+  /**
+   * The last accelerator that registered successfully; used to revert the
+   * setting when a newly chosen accelerator fails to register.
+   */
+  lastAppliedShortcut: KeyboardAcceleratorShortcut;
+
+  setShortcutRegistrationError: (error: string | null) => void;
+  setLastAppliedShortcut: (accelerator: KeyboardAcceleratorShortcut) => void;
+  clearShortcutRegistrationError: () => void;
+  reset: () => void;
+}