feat(sidebar): shortcut key bindings and hints

Signed-off-by: Adam Setch <[email protected]>

Author: setchy

src/renderer/components/Sidebar.test.tsx

@@ -272,4 +272,187 @@ describe('renderer/components/Sidebar.tsx', () => {
 
     expect(quitAppSpy).toHaveBeenCalledTimes(1);
   });
+
+  describe('keyboard bindings', () => {
+    it('should navigate home when pressing H key', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+      );
+
+      await userEvent.keyboard('h');
+
+      expect(navigateMock).toHaveBeenCalledTimes(1);
+      expect(navigateMock).toHaveBeenCalledWith('/', { replace: true });
+    });
+
+    it('should refresh notifications when pressing R key', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+        {
+          fetchNotifications: fetchNotificationsMock,
+          status: 'success',
+        },
+      );
+
+      await userEvent.keyboard('r');
+
+      expect(navigateMock).toHaveBeenCalledWith('/', { replace: true });
+      expect(fetchNotificationsMock).toHaveBeenCalledTimes(1);
+    });
+
+    it('should not refresh notifications when pressing R key if status is loading', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+        {
+          fetchNotifications: fetchNotificationsMock,
+          status: 'loading',
+        },
+      );
+
+      await userEvent.keyboard('r');
+
+      expect(fetchNotificationsMock).not.toHaveBeenCalled();
+    });
+
+    it('should toggle settings when pressing S key while logged in', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+        {
+          isLoggedIn: true,
+        },
+      );
+
+      await userEvent.keyboard('s');
+
+      expect(navigateMock).toHaveBeenCalledTimes(1);
+      expect(navigateMock).toHaveBeenCalledWith('/settings');
+    });
+
+    it('should not toggle settings when pressing S key while logged out', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+        {
+          isLoggedIn: false,
+        },
+      );
+
+      await userEvent.keyboard('s');
+
+      expect(navigateMock).not.toHaveBeenCalled();
+    });
+
+    it('should toggle filters when pressing F key while logged in', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+        {
+          isLoggedIn: true,
+        },
+      );
+
+      await userEvent.keyboard('f');
+
+      expect(navigateMock).toHaveBeenCalledTimes(1);
+      expect(navigateMock).toHaveBeenCalledWith('/filters');
+    });
+
+    it('should not toggle filters when pressing F key while logged out', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+        {
+          isLoggedIn: false,
+        },
+      );
+
+      await userEvent.keyboard('f');
+
+      expect(navigateMock).not.toHaveBeenCalled();
+    });
+
+    it('should ignore keyboard shortcuts when typing in an input', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+      );
+
+      const input = document.createElement('input');
+      document.body.appendChild(input);
+      input.focus();
+
+      await userEvent.keyboard('h');
+
+      expect(navigateMock).not.toHaveBeenCalled();
+
+      document.body.removeChild(input);
+    });
+
+    it('should ignore keyboard shortcuts when typing in a textarea', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+      );
+
+      const textarea = document.createElement('textarea');
+      document.body.appendChild(textarea);
+      textarea.focus();
+
+      await userEvent.keyboard('r');
+
+      expect(navigateMock).not.toHaveBeenCalled();
+      expect(fetchNotificationsMock).not.toHaveBeenCalled();
+
+      document.body.removeChild(textarea);
+    });
+
+    it('should ignore keyboard shortcuts when modifier keys are pressed', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+        {
+          fetchNotifications: fetchNotificationsMock,
+          status: 'success',
+        },
+      );
+
+      // Note: userEvent.keyboard with modifier syntax like {Shift>h</Shift>} 
+      // would require holding shift, but we test that metaKey/ctrlKey/altKey
+      // prevent action through manual event dispatch
+      const event = new KeyboardEvent('keydown', {
+        key: 'h',
+        metaKey: true,
+      });
+      document.dispatchEvent(event);
+
+      expect(navigateMock).not.toHaveBeenCalled();
+    });
+
+    it('should work with uppercase key press', async () => {
+      renderWithAppContext(
+        <MemoryRouter>
+          <Sidebar />
+        </MemoryRouter>,
+      );
+
+      // userEvent.keyboard converts to lowercase automatically
+      await userEvent.keyboard('H');
+
+      expect(navigateMock).toHaveBeenCalledWith('/', { replace: true });
+    });
+  });
 });