From b6a3241451abef66b3090d1e6ac625e28d4aadbe Mon Sep 17 00:00:00 2001 From: Cats Juice Date: Fri, 19 Sep 2025 19:40:39 +0800 Subject: [PATCH] chore(core): hide embedding status in chat (#13605) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary by CodeRabbit * **Style** * Simplified the AI chat composer tip: removed the dynamic embedding-status tooltip so only a single static caution remains — “AI outputs can be misleading or wrong.” * **Tests** * One end-to-end test related to embedding status was commented out and is no longer executed. --- .../ai-chat-composer/ai-chat-composer.ts | 6 ---- .../e2e/basic/chat.spec.ts | 32 +++++++++---------- 2 files changed, 16 insertions(+), 22 deletions(-) diff --git a/packages/frontend/core/src/blocksuite/ai/components/ai-chat-composer/ai-chat-composer.ts b/packages/frontend/core/src/blocksuite/ai/components/ai-chat-composer/ai-chat-composer.ts index 4fa2b156e9..2ec606390a 100644 --- a/packages/frontend/core/src/blocksuite/ai/components/ai-chat-composer/ai-chat-composer.ts +++ b/packages/frontend/core/src/blocksuite/ai/components/ai-chat-composer/ai-chat-composer.ts @@ -198,12 +198,6 @@ export class AIChatComposer extends SignalWatcher( AI outputs can be misleading or wrong`, - this.embeddingCompleted - ? null - : html``, ].filter(Boolean)} .loop=${false} > diff --git a/tests/affine-cloud-copilot/e2e/basic/chat.spec.ts b/tests/affine-cloud-copilot/e2e/basic/chat.spec.ts index afb0dae44a..f4337073ab 100644 --- a/tests/affine-cloud-copilot/e2e/basic/chat.spec.ts +++ b/tests/affine-cloud-copilot/e2e/basic/chat.spec.ts @@ -17,24 +17,24 @@ test.describe('AIBasic/Chat', () => { await expect(page.getByTestId('ai-onboarding')).toBeVisible(); }); - test('should open embedding settings when clicking check status button', async ({ - loggedInPage: page, - utils, - }) => { - await utils.editor.createDoc(page, 'Doc 1', 'doc1'); - await utils.editor.createDoc(page, 'Doc 2', 'doc2'); - await utils.editor.createDoc(page, 'Doc 3', 'doc3'); - await utils.editor.createDoc(page, 'Doc 4', 'doc4'); - await utils.editor.createDoc(page, 'Doc 5', 'doc5'); + // test('should open embedding settings when clicking check status button', async ({ + // loggedInPage: page, + // utils, + // }) => { + // await utils.editor.createDoc(page, 'Doc 1', 'doc1'); + // await utils.editor.createDoc(page, 'Doc 2', 'doc2'); + // await utils.editor.createDoc(page, 'Doc 3', 'doc3'); + // await utils.editor.createDoc(page, 'Doc 4', 'doc4'); + // await utils.editor.createDoc(page, 'Doc 5', 'doc5'); - const check = await page.getByTestId( - 'ai-chat-embedding-status-tooltip-check' - ); - await expect(check).toBeVisible({ timeout: 50 * 1000 }); + // const check = await page.getByTestId( + // 'ai-chat-embedding-status-tooltip-check' + // ); + // await expect(check).toBeVisible({ timeout: 50 * 1000 }); - await check.click(); - await expect(page.getByTestId('workspace-setting:embedding')).toBeVisible(); - }); + // await check.click(); + // await expect(page.getByTestId('workspace-setting:embedding')).toBeVisible(); + // }); test(`should send message and receive AI response: - send message