Develop

.github/workflows/ci.yml

@@ -16,6 +16,7 @@ jobs:
   test:
     name: Typecheck & Unit Tests
     runs-on: self-hosted
+    timeout-minutes: 10
     steps:
       - uses: actions/checkout@v4
 
@@ -29,6 +30,22 @@ jobs:
           cache: "pnpm"
           cache-dependency-path: pnpm-lock.yaml
 
+      - name: Cache node_modules
+        uses: actions/cache@v4
+        with:
+          path: node_modules
+          key: ${{ runner.os }}-node_modules-${{ hashFiles('pnpm-lock.yaml') }}
+          restore-keys: |
+            ${{ runner.os }}-node_modules-
+
+      - name: Cache Vite transform cache
+        uses: actions/cache@v4
+        with:
+          path: node_modules/.vite
+          key: ${{ runner.os }}-vite-${{ hashFiles('pnpm-lock.yaml', 'vite.config.ts') }}
+          restore-keys: |
+            ${{ runner.os }}-vite-
+
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
 

package.json

@@ -57,6 +57,7 @@
     "@vue/test-utils": "^2.4.6",
     "@vue/tsconfig": "^0.8.1",
     "autoprefixer": "^10.4.24",
+    "happy-dom": "^20.8.8",
     "jsdom": "^28.0.0",
     "msw": "^2.12.10",
     "playwright": "^1.58.2",

src/__tests__/usePermissions.spec.ts

@@ -1,67 +1,56 @@
+// @vitest-environment node
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 import { setActivePinia, createPinia } from 'pinia'
 
+// vi.mock is hoisted — this mock is in place before any import below
+vi.mock('../stores/auth', () => ({
+  useAuthStore: vi.fn(),
+}))
+
+import { usePermissions } from '../composables/usePermissions'
+import { useAuthStore } from '../stores/auth'
+
+const mockStore = (tier: string) => {
+  vi.mocked(useAuthStore).mockReturnValue({ user: { subscription_tier: tier } } as any)
+}
+
 describe('usePermissions', () => {
   beforeEach(() => {
     setActivePinia(createPinia())
   })
 
-  it('starter tier gets correct chat limit (300)', async () => {
-    vi.doMock('../stores/auth', () => ({
-      useAuthStore: () => ({ user: { subscription_tier: 'starter' } })
-    }))
-    vi.resetModules()
-    const { usePermissions } = await import('../composables/usePermissions')
+  it('starter tier gets correct chat limit (300)', () => {
+    mockStore('starter')
     const { chatLimit } = usePermissions()
     expect(chatLimit.value).toBe(300)
   })
 
-  it('pro tier has unlimited chat', async () => {
-    vi.doMock('../stores/auth', () => ({
-      useAuthStore: () => ({ user: { subscription_tier: 'pro' } })
-    }))
-    vi.resetModules()
-    const { usePermissions } = await import('../composables/usePermissions')
+  it('pro tier has unlimited chat', () => {
+    mockStore('pro')
     const { isUnlimitedChat } = usePermissions()
     expect(isUnlimitedChat.value).toBe(true)
   })
 
-  it('free tier cannot use auto tasks', async () => {
-    vi.doMock('../stores/auth', () => ({
-      useAuthStore: () => ({ user: { subscription_tier: 'free' } })
-    }))
-    vi.resetModules()
-    const { usePermissions } = await import('../composables/usePermissions')
+  it('free tier cannot use auto tasks', () => {
+    mockStore('free')
     const { canUseAutoTasks } = usePermissions()
     expect(canUseAutoTasks.value).toBe(false)
   })
 
-  it('starter tier can use auto tasks', async () => {
-    vi.doMock('../stores/auth', () => ({
-      useAuthStore: () => ({ user: { subscription_tier: 'starter' } })
-    }))
-    vi.resetModules()
-    const { usePermissions } = await import('../composables/usePermissions')
+  it('starter tier can use auto tasks', () => {
+    mockStore('starter')
     const { canUseAutoTasks } = usePermissions()
     expect(canUseAutoTasks.value).toBe(true)
   })
 
-  it('unknown tier falls back to free limits', async () => {
-    vi.doMock('../stores/auth', () => ({
-      useAuthStore: () => ({ user: { subscription_tier: 'bogus' } })
-    }))
-    vi.resetModules()
-    const { usePermissions } = await import('../composables/usePermissions')
+  it('unknown tier falls back to free limits', () => {
+    mockStore('bogus')
     const { chatLimit } = usePermissions()
     expect(chatLimit.value).toBe(30)
   })
 
-  it('unlimited tier has unlimited projects', async () => {
-    vi.doMock('../stores/auth', () => ({
-      useAuthStore: () => ({ user: { subscription_tier: 'unlimited' } })
-    }))
-    vi.resetModules()
-    const { usePermissions } = await import('../composables/usePermissions')
+  it('unlimited tier has unlimited projects', () => {
+    mockStore('unlimited')
     const { isUnlimitedProjects } = usePermissions()
     expect(isUnlimitedProjects.value).toBe(true)
   })

src/composables/__tests__/useVoiceInput.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
 
 // Mock SpeechRecognition
 const mockRecognition = {
@@ -48,20 +48,34 @@ vi.stubGlobal('navigator', {
   },
 })
 
+// Track the stop function for each test so we can clean up the RAF animation
+// loop after tests that call start(). Without this, the RAF loop runs forever
+// as an infinite microtask chain (each frame enqueues the next), hanging the
+// process.
+let currentStop: (() => void) | null = null
+
 describe('useVoiceInput', () => {
   beforeEach(() => {
     vi.clearAllMocks()
+    currentStop = null
+  })
+
+  afterEach(() => {
+    currentStop?.()
+    currentStop = null
   })
 
   it('starts with isListening = false', async () => {
     const { useVoiceInput } = await import('@/composables/useVoiceInput')
-    const { isListening } = useVoiceInput()
+    const { isListening, stop } = useVoiceInput()
+    currentStop = stop
     expect(isListening.value).toBe(false)
   })
 
   it('start() sets isListening = true and calls recognition.start', async () => {
     const { useVoiceInput } = await import('@/composables/useVoiceInput')
-    const { isListening, start } = useVoiceInput()
+    const { isListening, start, stop } = useVoiceInput()
+    currentStop = stop
     await start()
     expect(isListening.value).toBe(true)
     expect(mockRecognition.start).toHaveBeenCalled()
@@ -70,6 +84,7 @@ describe('useVoiceInput', () => {
   it('stop() sets isListening = false and calls recognition.stop', async () => {
     const { useVoiceInput } = await import('@/composables/useVoiceInput')
     const { isListening, start, stop } = useVoiceInput()
+    currentStop = stop
     await start()
     stop()
     expect(isListening.value).toBe(false)
@@ -78,13 +93,15 @@ describe('useVoiceInput', () => {
 
   it('amplitudeBars has BAR_COUNT entries', async () => {
     const { useVoiceInput, BAR_COUNT } = await import('@/composables/useVoiceInput')
-    const { amplitudeBars } = useVoiceInput()
+    const { amplitudeBars, stop } = useVoiceInput()
+    currentStop = stop
     expect(amplitudeBars.value).toHaveLength(BAR_COUNT)
   })
 
   it('transcript updates on recognition result', async () => {
     const { useVoiceInput } = await import('@/composables/useVoiceInput')
-    const { start, transcript } = useVoiceInput()
+    const { start, stop, transcript } = useVoiceInput()
+    currentStop = stop
     await start()
     // Simulate a speech result
     mockRecognition.onresult({
@@ -97,7 +114,8 @@ describe('useVoiceInput', () => {
   it('isFinal result fires onFinal callback', async () => {
     const { useVoiceInput } = await import('@/composables/useVoiceInput')
     const onFinal = vi.fn()
-    const { start } = useVoiceInput({ onFinal })
+    const { start, stop } = useVoiceInput({ onFinal })
+    currentStop = stop
     await start()
     mockRecognition.onresult({
       results: [[{ transcript: 'confirmed text', isFinal: true }]],

src/test/globalSetup.ts

@@ -0,0 +1,10 @@
+// globalSetup runs in the main Vitest process (not in workers).
+// teardown() is called after all test files and reporters complete.
+// We force-exit here because some open handle in the main process
+// (likely Vitest's internal file watcher or a jsdom resource) prevents
+// a natural exit even after all tests pass.
+export default function () {
+    return function teardown() {
+        process.exit(0);
+    };
+}