test(gateway): verify models endpoint contract

Author: alichherawalla

docs/FUNCTIONAL_TEST_STRATEGY.md

@@ -46,7 +46,7 @@ really a test of a third-party framework or engine we merely depend on:
   dependency (binary/model) is absent, so it runs on a dev Mac + release builds and SKIPS in a
   plain `npm ci` CI rather than failing. This is where "the feature works" is proven.
 - **swift unit (XCTest / swift-testing via SwiftPM)** - pure logic inside a Swift package.
-- **e2e (Playwright Electron)** - the rendered app tour (e2e/, 26 tests today).
+- **e2e (Playwright Electron)** - the rendered app tour (e2e/, 27 tests today).
 - **gateway smoke (`npm run smoke`)** - the OpenAI `/v1` surface against a running app.
 
 ## Surface matrix (status - keep current)
@@ -65,7 +65,7 @@ really a test of a third-party framework or engine we merely depend on:
 | Capture -> OCR -> memory ingest seam                       | integration     | vitest: feed a fixture frame through the ingest path into a temp DB                                                                                                                                                             | TODO                                           |
 | RAG end-to-end (retrieve -> prompt -> answer)              | integration     | vitest: seed a temp SQLite, run retrieval, assert grounded context                                                                                                                                                              | TODO                                           |
 | Vault (kdbx + Argon2)                                      | integration     | vitest vs temp dir                                                                                                                                                                                                              | DONE (`vault-service.test.ts`)                 |
-| Renderer surfaces render                                   | e2e             | Playwright tour                                                                                                                                                                                                                 | DONE (26)                                      |
+| Renderer surfaces render                                   | e2e             | Playwright tour                                                                                                                                                                                                                 | DONE (27)                                      |
 | Pro dictation / clipboard / replay                         | e2e + unit      | Playwright (`pro.spec.ts`) + pro unit                                                                                                                                                                                           | PARTIAL                                        |
 
 ## Rules

docs/P0_P2_INTEGRATION_COVERAGE.md

@@ -7,19 +7,21 @@ manual claim does not count as complete integration coverage.
 
 ## Current status - 2026-07-17
 
-- Scope: 155 journeys - 74 P0, 71 P1, and 10 P2.
-- Covered by direct integration or E2E evidence: 29 - 22 P0, 6 P1, and 1 P2.
-- Left: 126 - including partially covered journeys whose exact release behavior is not yet proven.
+- Status snapshot:
+  - P0: 74 total, 23 covered, 51 left.
+  - P1: 71 total, 6 covered, 65 left.
+  - P2: 10 total, 1 covered, 9 left.
+  - Overall: 155 total, 30 covered, 125 left.
 - Green gates today:
   - `npm test`: 202 files passed, 1 skipped; 2,190 tests passed, 1 skipped.
   - `npm run test:coverage`: 96.75% statements, 91.54% branches, 96.14% functions,
     97.52% lines.
   - `npm run test:db`: 13 files and 105 real SQLite integration tests passed; Electron ABI
     restored afterward.
-  - `npm run test:e2e`: 26 Playwright Electron tests passed against fresh synthetic temp profiles.
+  - `npm run test:e2e`: 27 Playwright Electron tests passed against fresh synthetic temp profiles.
   - Both TypeScript projects pass.
-- Not yet a clean handoff: the exact P0-P2 journey count is 29/155, and strict ESLint currently
-  exposes a legacy backlog. Neither is hidden by the coverage percentage.
+- Not yet a clean handoff: release-journey coverage remains incomplete, and strict ESLint exposes
+  a legacy backlog. Neither is hidden by the coverage percentage.
 
 ## Covered P0 journeys
 
@@ -37,6 +39,9 @@ manual claim does not count as complete integration coverage.
 - #63 - Image cancellation keeps text. `MemoryChat.tool-image-cancel.test.tsx` drives the real
   rendered MemoryChat path with the image engine as the external boundary and verifies the text
   turn persists.
+- #79 - Gateway models endpoint. `e2e/smoke.spec.ts` starts the real Electron gateway, seeds an
+  active model only inside the disposable profile, calls `/v1/models` over HTTP, and verifies
+  modality metadata in both supported response shapes.
 - #87 - Replay timeline renders. `e2e/pro.spec.ts` launches the real Pro build path with synthetic
   data and verifies Replay renders instead of the upgrade screen.
 - #94 - Delete all removes capture corpus. `pro/main/__tests__/personal-data.integration.test.ts`
@@ -188,7 +193,6 @@ manual claim does not count as complete integration coverage.
 - #76 - Expired connector becomes error.
 - #77 - Dead connector does not hang all tools.
 - #78 - Connector delete removes secrets.
-- #79 - Gateway models endpoint.
 - #80 - Gateway chat streaming.
 - #81 - Gateway image route.
 - #82 - Gateway failure envelope.

e2e/smoke.spec.ts

@@ -109,3 +109,50 @@ test('system:health IPC returns the component list', async () => {
   expect(ids).toContain('chat')
   expect(ids).toContain('gateway')
 })
+
+test('gateway /v1/models serves active local models with modality metadata', async () => {
+  const modelsDir = path.join(userDataDir, 'models')
+  fs.mkdirSync(modelsDir, { recursive: true })
+  fs.writeFileSync(path.join(modelsDir, 'e2e-active.gguf'), 'synthetic gateway model fixture')
+  fs.writeFileSync(
+    path.join(modelsDir, 'active-model.json'),
+    JSON.stringify({ id: 'e2e-active-model', primary: 'e2e-active.gguf', mmproj: null })
+  )
+
+  let catalog: {
+    object?: string
+    data?: Array<{ id?: string; object?: string; kind?: string }>
+    models?: Array<{ name?: string; model?: string; kind?: string }>
+  } = {}
+  await expect
+    .poll(
+      async () => {
+        try {
+          const response = await fetch('http://127.0.0.1:7878/v1/models')
+          if (!response.ok) return false
+          catalog = await response.json()
+          return catalog.data?.some((model) => model.id === 'e2e-active-model') ?? false
+        } catch {
+          return false
+        }
+      },
+      { timeout: 10000 }
+    )
+    .toBe(true)
+
+  expect(catalog.object).toBe('list')
+  expect(catalog.data).toContainEqual(
+    expect.objectContaining({
+      id: 'e2e-active-model',
+      object: 'model',
+      kind: 'chat'
+    })
+  )
+  expect(catalog.models).toContainEqual(
+    expect.objectContaining({
+      name: 'e2e-active-model',
+      model: 'e2e-active-model',
+      kind: 'chat'
+    })
+  )
+})