test(ci): convert logic-heavy Playwright specs to Vitest and optimize CI pipeline (TASK-322, TASK-323, TASK-324)

Move pure-logic Playwright tests to Vitest unit tests, reduce visual/CSS
specs, and restructure CI for faster PR checks.

TASK-322: Convert sorting-ignore-words and keyboard-shortcuts logic to
Vitest (new sorting-ignore-words.test.js, extended shortcuts.test.js).
Reduce playback.spec.js from 1310 to 361 lines (hardware-dependent only),
visual-regression.spec.js from 540 to 135 lines (8 critical snapshots),
startup-fouc.spec.js from 441 to 151 lines (4 critical tests).
Net: ~2,639 Playwright lines removed, 42 new Vitest tests added.

TASK-323: Replace cargo tarpaulin with cargo nextest for PR checks
(saves ~1.5 min). Move tarpaulin coverage to main-push-only job. Add
dorny/paths-filter to gate Playwright on frontend changes.

TASK-324: Tracking task complete — all three phases done.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: pythoninthegrass

.github/workflows/test.yml

@@ -57,9 +57,27 @@ jobs:
       - name: Cargo clippy
         run: cargo clippy --workspace --all-features -- -D warnings
 
-      # cargo-binstall and cargo-tarpaulin are installed separately from the
-      # composite action (which runs in check mode and skips cargo tools).
-      # These are needed only for coverage — not for fmt, clippy, or check.
+      - name: Run Rust tests
+        run: cargo nextest run --workspace
+
+  # Coverage runs only on main push (not PRs) to keep the critical path fast
+  rust-coverage:
+    name: Rust Coverage
+    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
+    runs-on: [macOS, ARM64]
+    timeout-minutes: 25
+    env:
+      CARGO_INCREMENTAL: 1
+      CARGO_TERM_COLOR: always
+
+    steps:
+      - uses: actions/checkout@v6
+
+      - name: Setup Tauri build environment
+        uses: ./.github/actions/setup-tauri-build
+        with:
+          mode: check
+
       - uses: cargo-bins/cargo-binstall@main
 
       - name: Install cargo-tarpaulin
@@ -213,8 +231,26 @@ jobs:
           path: app/frontend/coverage/
           retention-days: 30
 
+  # Detect frontend changes for conditional Playwright execution
+  changes:
+    name: Detect Changes
+    runs-on: blacksmith-4vcpu-ubuntu-2404
+    timeout-minutes: 2
+    outputs:
+      frontend: ${{ steps.filter.outputs.frontend }}
+    steps:
+      - uses: actions/checkout@v6
+      - uses: dorny/paths-filter@v3
+        id: filter
+        with:
+          filters: |
+            frontend:
+              - 'app/frontend/**'
+
   playwright-tests:
     name: Playwright E2E Tests
+    needs: [changes]
+    if: needs.changes.outputs.frontend == 'true' || github.event_name == 'push'
     runs-on: [macOS, ARM64]
     timeout-minutes: 20
 

app/frontend/__tests__/shortcuts.test.js

@@ -90,6 +90,8 @@ describe('keyboard shortcuts', () => {
     mockUi.modal = null;
     mockQueue.stopAfterCurrent = false;
     mockLibrary.searchQuery = '';
+    mockPlayer.volume = 50;
+    mockPlayer.currentTime = 30000;
     initKeyboardShortcuts();
   });
 
@@ -180,6 +182,115 @@ describe('keyboard shortcuts', () => {
     });
   });
 
+  describe('Escape context-aware behavior', () => {
+    it('should close settings when view is settings', () => {
+      mockUi.view = 'settings';
+      const event = createKeyEvent('Escape', { key: 'Escape' });
+      keydownHandler(event);
+      expect(mockUi.toggleSettings).toHaveBeenCalled();
+      expect(event.preventDefault).toHaveBeenCalled();
+    });
+
+    it('should close modal when modal is open', () => {
+      mockUi.modal = 'someModal';
+      const event = createKeyEvent('Escape', { key: 'Escape' });
+      keydownHandler(event);
+      expect(mockUi.closeModal).toHaveBeenCalled();
+      expect(event.preventDefault).toHaveBeenCalled();
+    });
+
+    it('should clear search query when no modal or settings open', () => {
+      mockLibrary.searchQuery = 'test';
+      const event = createKeyEvent('Escape', { key: 'Escape' });
+      keydownHandler(event);
+      expect(mockLibrary.searchQuery).toBe('');
+      expect(mockLibrary.search).toHaveBeenCalledWith('');
+    });
+
+    it('should do nothing when no modal, settings, or search active', () => {
+      mockLibrary.searchQuery = '';
+      const event = createKeyEvent('Escape', { key: 'Escape' });
+      keydownHandler(event);
+      expect(mockUi.toggleSettings).not.toHaveBeenCalled();
+      expect(mockUi.closeModal).not.toHaveBeenCalled();
+    });
+  });
+
+  describe('volume and seek shortcuts', () => {
+    it('should increase volume on ArrowUp', () => {
+      const event = createKeyEvent('ArrowUp', { key: 'ArrowUp' });
+      keydownHandler(event);
+      expect(mockPlayer.setVolume).toHaveBeenCalledWith(55);
+    });
+
+    it('should decrease volume on ArrowDown', () => {
+      const event = createKeyEvent('ArrowDown', { key: 'ArrowDown' });
+      keydownHandler(event);
+      expect(mockPlayer.setVolume).toHaveBeenCalledWith(45);
+    });
+
+    it('should go next on ArrowRight', () => {
+      const event = createKeyEvent('ArrowRight', { key: 'ArrowRight' });
+      keydownHandler(event);
+      expect(mockPlayer.next).toHaveBeenCalled();
+    });
+
+    it('should seek forward on Cmd+ArrowRight', () => {
+      const event = createKeyEvent('ArrowRight', { key: 'ArrowRight', metaKey: true });
+      keydownHandler(event);
+      expect(mockPlayer.seek).toHaveBeenCalledWith(35000);
+    });
+
+    it('should go previous on ArrowLeft', () => {
+      const event = createKeyEvent('ArrowLeft', { key: 'ArrowLeft' });
+      keydownHandler(event);
+      expect(mockPlayer.previous).toHaveBeenCalled();
+    });
+
+    it('should seek back on Cmd+ArrowLeft', () => {
+      const event = createKeyEvent('ArrowLeft', { key: 'ArrowLeft', metaKey: true });
+      keydownHandler(event);
+      expect(mockPlayer.seek).toHaveBeenCalledWith(25000);
+    });
+
+    it('should clamp seek back to 0', () => {
+      mockPlayer.currentTime = 2000;
+      const event = createKeyEvent('ArrowLeft', { key: 'ArrowLeft', metaKey: true });
+      keydownHandler(event);
+      expect(mockPlayer.seek).toHaveBeenCalledWith(0);
+    });
+  });
+
+  describe('input suppression', () => {
+    it('should NOT trigger playback shortcuts when typing in INPUT', () => {
+      const event = createKeyEvent('Space', {
+        key: ' ',
+        target: { tagName: 'INPUT', isContentEditable: false },
+      });
+      keydownHandler(event);
+      expect(mockPlayer.togglePlay).not.toHaveBeenCalled();
+    });
+
+    it('should NOT trigger playback shortcuts when typing in TEXTAREA', () => {
+      const event = createKeyEvent('ArrowUp', {
+        key: 'ArrowUp',
+        target: { tagName: 'TEXTAREA', isContentEditable: false },
+      });
+      keydownHandler(event);
+      expect(mockPlayer.setVolume).not.toHaveBeenCalled();
+    });
+
+    it('should still trigger modifier shortcuts in INPUT fields', () => {
+      const event = createKeyEvent('KeyM', {
+        metaKey: true,
+        shiftKey: true,
+        target: { tagName: 'INPUT', isContentEditable: false },
+      });
+      keydownHandler(event);
+      expect(mockPlayer.toggleMute).toHaveBeenCalled();
+    });
+  });
+
   describe('SHORTCUT_DEFINITIONS', () => {
     it('should list mute as requiring modifier+shift', () => {
       const muteDef = SHORTCUT_DEFINITIONS.find((s) => s.action === 'Mute / Unmute');