luulam/AFFiNE
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

chore(core): hide embedding status in chat (#13605)

Browse Source 
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## 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.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
This commit is contained in:
Cats Juice
2025-09-19 19:40:39 +08:00
committed by GitHub
parent 360c9545f4
commit b6a3241451
2 changed files with 16 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
packages/frontend/core/src/blocksuite/ai/components/ai-chat-composer/ai-chat-composer.ts
View File

@@ -198,12 +198,6 @@ export class AIChatComposer extends SignalWatcher(
<ai-chat-composer-tip <ai-chat-composer-tip
.tips=${[ .tips=${[
html`<span>AI outputs can be misleading or wrong</span>`, html`<span>AI outputs can be misleading or wrong</span>`,
this.embeddingCompleted
? null
: html`<ai-chat-embedding-status-tooltip
.affineWorkspaceDialogService=${this
.affineWorkspaceDialogService}
/>`,
].filter(Boolean)} ].filter(Boolean)}
.loop=${false} .loop=${false}
></ai-chat-composer-tip> ></ai-chat-composer-tip>

32
tests/affine-cloud-copilot/e2e/basic/chat.spec.ts
View File

@@ -17,24 +17,24 @@ test.describe('AIBasic/Chat', () => {
await expect(page.getByTestId('ai-onboarding')).toBeVisible(); await expect(page.getByTestId('ai-onboarding')).toBeVisible();
}); });
test('should open embedding settings when clicking check status button', async ({ // test('should open embedding settings when clicking check status button', async ({
loggedInPage: page, // loggedInPage: page,
utils, // utils,
}) => { // }) => {
await utils.editor.createDoc(page, 'Doc 1', 'doc1'); // await utils.editor.createDoc(page, 'Doc 1', 'doc1');
await utils.editor.createDoc(page, 'Doc 2', 'doc2'); // await utils.editor.createDoc(page, 'Doc 2', 'doc2');
await utils.editor.createDoc(page, 'Doc 3', 'doc3'); // await utils.editor.createDoc(page, 'Doc 3', 'doc3');
await utils.editor.createDoc(page, 'Doc 4', 'doc4'); // await utils.editor.createDoc(page, 'Doc 4', 'doc4');
await utils.editor.createDoc(page, 'Doc 5', 'doc5'); // await utils.editor.createDoc(page, 'Doc 5', 'doc5');
const check = await page.getByTestId( // const check = await page.getByTestId(
'ai-chat-embedding-status-tooltip-check' // 'ai-chat-embedding-status-tooltip-check'
); // );
await expect(check).toBeVisible({ timeout: 50 * 1000 }); // await expect(check).toBeVisible({ timeout: 50 * 1000 });
await check.click(); // await check.click();
await expect(page.getByTestId('workspace-setting:embedding')).toBeVisible(); // await expect(page.getByTestId('workspace-setting:embedding')).toBeVisible();
}); // });
test(`should send message and receive AI response: test(`should send message and receive AI response:
- send message - send message