+
@@ -89,7 +58,7 @@ async function restoreFocus() {
-
+
diff --git a/src/theme/chat/history.ts b/src/theme/chat/history.ts
index 52dd55041..0c57d7689 100644
--- a/src/theme/chat/history.ts
+++ b/src/theme/chat/history.ts
@@ -23,7 +23,6 @@ export const chatHistoryTheme = tv({
label: 'block truncate text-xs',
empty: 'px-2 py-4 text-center text-xs text-muted',
form: 'space-y-2 border-t border-border p-2',
- actions: 'flex justify-end gap-1',
- confirmation: 'text-xs text-surface'
+ actions: 'flex justify-end gap-1'
}
})
diff --git a/tests/e2e/chat/file-identity.spec.ts b/tests/e2e/chat/file-identity.spec.ts
index 737637aad..f8885b7db 100644
--- a/tests/e2e/chat/file-identity.spec.ts
+++ b/tests/e2e/chat/file-identity.spec.ts
@@ -1,5 +1,14 @@
import { test, expect } from '@playwright/test'
+// Playwright's bundled Chromium on macOS closes the page when a FileSystemFileHandle is read
+// back from IndexedDB, before any app code runs; installed Google Chrome on the same machine
+// reads and compares it correctly.
+test.skip(
+ ({ browserName, channel }) =>
+ browserName === 'chromium' && !channel && process.platform === 'darwin',
+ "Playwright's bundled Chromium closes the page when reading a file handle back from IndexedDB on macOS"
+)
+
test('browser file identity survives reopening and distinguishes same-named files', async ({
page
}) => {
diff --git a/tests/e2e/chat/menu.spec.ts b/tests/e2e/chat/menu.spec.ts
index b0427aacb..945f9c744 100644
--- a/tests/e2e/chat/menu.spec.ts
+++ b/tests/e2e/chat/menu.spec.ts
@@ -29,3 +29,37 @@ test('mobile conversation menu stays attached to its trigger and inside the view
})
.toBe(true)
})
+
+test('deleting a conversation asks in a confirmation dialog', async ({
+ configuredChat: chat,
+ page
+}) => {
+ await chat.submit('Conversation to keep')
+ await expect(chat.assistantMessage()).toBeVisible()
+ await page.getByRole('button', { name: 'New chat', exact: true }).first().click()
+ await chat.submit('Conversation to delete')
+ await expect(chat.assistantMessage()).toBeVisible()
+ const actions = page.getByRole('button', { name: 'Conversation actions' })
+
+ await actions.click()
+ await page.getByRole('menuitem', { name: 'Delete' }).click()
+ const dialog = page.getByRole('alertdialog', { name: 'Delete this conversation permanently?' })
+ await expect(dialog).toContainText('“Conversation to delete” and its messages are removed.')
+ await dialog.getByRole('button', { name: 'Cancel' }).click()
+ await expect(dialog).toBeHidden()
+ await expect(chat.userMessage()).toContainText('Conversation to delete')
+
+ await actions.click()
+ await page.getByRole('menuitem', { name: 'Delete' }).click()
+ await dialog.getByRole('button', { name: 'Delete' }).click()
+ await expect(dialog).toBeHidden()
+ await expect(page.getByTestId('chat-message-user')).toHaveCount(0)
+
+ // The conversation is gone from storage, not only from the panel.
+ await page.reload()
+ await chat.chatTab.click()
+ await page.getByRole('button', { name: 'Conversation history', exact: true }).click()
+ await page.getByRole('button', { name: 'All documents', exact: true }).click()
+ await expect(page.getByRole('button', { name: /Conversation to keep/ })).toBeVisible()
+ await expect(page.getByRole('button', { name: /Conversation to delete/ })).toHaveCount(0)
+})
diff --git a/tests/e2e/chat/profiles.spec.ts b/tests/e2e/chat/profiles.spec.ts
index 4bb61979e..d18388cf3 100644
--- a/tests/e2e/chat/profiles.spec.ts
+++ b/tests/e2e/chat/profiles.spec.ts
@@ -1,3 +1,4 @@
+import { CATALOG_ONLY_MODEL } from '#tests/helpers/chat/catalog'
import { expect, test } from '#tests/helpers/chat/fixture'
test('Design profile selector exposes provider and capabilities', async ({
@@ -16,6 +17,8 @@ test('OpenRouter accepts a custom model ID from Settings', async ({ configuredCh
await chat.page.getByTestId('provider-settings-trigger').click()
await chat.page.locator('[data-model-id]').first().click()
await chat.page.getByLabel('Model ID').click()
+ // The catalog replaces the fallback list once it loads; pick only after it has settled.
+ await expect(chat.page.getByRole('option', { name: CATALOG_ONLY_MODEL })).toBeVisible()
await chat.page.getByRole('option', { name: 'Custom model…' }).click()
const input = chat.page.getByTestId('provider-settings-custom-model')
await input.fill(customModel)
diff --git a/tests/e2e/chat/reasoning.spec.ts b/tests/e2e/chat/reasoning.spec.ts
index 23e1618e2..8693043ae 100644
--- a/tests/e2e/chat/reasoning.spec.ts
+++ b/tests/e2e/chat/reasoning.spec.ts
@@ -19,9 +19,11 @@ test('expand while thinking auto-collapses but preserves manual opening', async
await expect(trigger).toHaveAttribute('aria-expanded', 'true')
await trigger.click()
await trigger.click()
+ // Time the auto-close instead of sleeping past it.
+ await page.clock.install()
await finishReasoning(page)
await expect(chat.assistantMessage()).toContainText('Finished inspecting')
- await page.waitForTimeout(1200)
+ await page.clock.runFor(1500)
await expect(trigger).toHaveAttribute('aria-expanded', 'true')
})
diff --git a/tests/e2e/chat/turns.spec.ts b/tests/e2e/chat/turns.spec.ts
index cef9f87f2..c4a53249f 100644
--- a/tests/e2e/chat/turns.spec.ts
+++ b/tests/e2e/chat/turns.spec.ts
@@ -21,12 +21,17 @@ test('reverting a reply undoes its edits, marks it, saves the chat, and tells th
await expect(chat.assistantMessage()).toContainText('Done')
expect(await cardCount(page)).toBe(1)
+ const content = chat.assistantMessage().locator('[data-slot="chat-reply-content"]')
await chat.assistantMessage().getByTestId('chat-revert-turn').click()
expect(await cardCount(page)).toBe(0)
+ await expect(content).toHaveAttribute('data-reverted', 'true')
+ await expect(content).toHaveCSS('opacity', '0.5')
// Restore redoes the edits, then reverting again leaves the reply marked.
await chat.assistantMessage().getByTestId('chat-restore-turn').click()
expect(await cardCount(page)).toBe(1)
+ await expect(content).toHaveAttribute('data-reverted', 'false')
+ await expect(content).toHaveCSS('opacity', '1')
await chat.assistantMessage().getByTestId('chat-revert-turn').click()
expect(await cardCount(page)).toBe(0)
diff --git a/tests/e2e/settings/credentials.spec.ts b/tests/e2e/settings/credentials.spec.ts
index 25f3e2e54..20bb3f23f 100644
--- a/tests/e2e/settings/credentials.spec.ts
+++ b/tests/e2e/settings/credentials.spec.ts
@@ -238,7 +238,8 @@ test('remembered browser credentials survive reload and clear centrally', async
})
test('browser credential preferences live in General, not the footer', async ({ page }) => {
- await page.goto('/')
+ await page.goto('/?test')
+ await new CanvasHelper(page).waitForInit()
await page.keyboard.press('ControlOrMeta+,')
const panel = page.getByTestId('settings-general-panel')
const remember = panel.getByRole('switch', { name: 'Remember API keys on this device' })
diff --git a/tests/helpers/chat/catalog.ts b/tests/helpers/chat/catalog.ts
new file mode 100644
index 000000000..d76cf3257
--- /dev/null
+++ b/tests/helpers/chat/catalog.ts
@@ -0,0 +1,26 @@
+import type { Page } from '@playwright/test'
+
+/** A model only the fixed catalog lists, so a test can tell when the catalog has loaded. */
+export const CATALOG_ONLY_MODEL = 'Catalog Test Model'
+
+const CATALOG = {
+ openrouter: {
+ models: {
+ 'test/catalog-model': {
+ name: CATALOG_ONLY_MODEL,
+ tool_call: true,
+ attachment: true,
+ release_date: '2026-01-01',
+ modalities: { output: ['text'] },
+ limit: { output: 16384 }
+ }
+ }
+ }
+}
+
+/** Serves a small fixed models.dev catalog, so model pickers never depend on the live one. */
+export async function routeModelCatalog(page: Page): Promise {
+ await page.route('https://models.dev/api.json', (route) =>
+ route.fulfill({ contentType: 'application/json', body: JSON.stringify(CATALOG) })
+ )
+}
diff --git a/tests/helpers/chat/fixture.ts b/tests/helpers/chat/fixture.ts
index 3fb965647..51a279955 100644
--- a/tests/helpers/chat/fixture.ts
+++ b/tests/helpers/chat/fixture.ts
@@ -1,8 +1,10 @@
import { test as base, expect } from '@playwright/test'
import { CanvasHelper } from '#tests/helpers/canvas'
+import { routeModelCatalog } from '#tests/helpers/chat/catalog'
import { ChatHarness } from '#tests/helpers/chat/harness'
import { injectMockChatTransport } from '#tests/helpers/chat/transport'
+import { routeVisionModel } from '#tests/helpers/chat/vision'
interface ChatFixtures {
chat: ChatHarness
@@ -11,6 +13,8 @@ interface ChatFixtures {
export const test = base.extend({
chat: async ({ page }, use) => {
+ await routeModelCatalog(page)
+ await routeVisionModel(page)
const harness = new ChatHarness(page)
await harness.open()
const canvas = new CanvasHelper(page)
diff --git a/tests/helpers/chat/vision.ts b/tests/helpers/chat/vision.ts
new file mode 100644
index 000000000..76f5b3ad8
--- /dev/null
+++ b/tests/helpers/chat/vision.ts
@@ -0,0 +1,30 @@
+import type { Page } from '@playwright/test'
+
+/** What the stand-in Vision model reports for every attached image. */
+const VISION_FINDINGS = 'A centered portrait on a light background.'
+
+/**
+ * Answers the Vision model's request for attached images, which goes to OpenRouter directly rather
+ * than through the chat transport, so image messages never reach the network.
+ */
+export async function routeVisionModel(page: Page): Promise {
+ await page.route('https://openrouter.ai/api/v1/chat/completions', (route) =>
+ route.fulfill({
+ contentType: 'application/json',
+ body: JSON.stringify({
+ id: 'vision-test',
+ object: 'chat.completion',
+ created: 0,
+ model: 'test/vision',
+ choices: [
+ {
+ index: 0,
+ message: { role: 'assistant', content: VISION_FINDINGS },
+ finish_reason: 'stop'
+ }
+ ],
+ usage: { prompt_tokens: 1, completion_tokens: 1, total_tokens: 2 }
+ })
+ })
+ )
+}