mirror of
https://github.com/labring/FastGPT.git
synced 2026-05-07 01:02:55 +08:00
76d6234de6
* Agent features (#6345) * Test agent (#6220) * squash: compress all commits into one * feat: plan response in ui * response ui * perf: agent config * merge * tool select ux * perf: chat ui * perf: agent editform * tmp code * feat: save chat * Complete agent parent (#6049) * add role and tools filling * add: file-upload --------- Co-authored-by: xxyyh <2289112474@qq> * perf: top agent code * top agent (#6062) Co-authored-by: xxyyh <2289112474@qq> * fix: ts * skill editor ui * ui * perf: rewrite type with zod * skill edit ui * skill agent (#6089) * cp skill chat * rebasefdf933dand add skill chat * 1. skill 的 CRUD 2. skill 的信息渲染到前端界面 * solve comment * remove chatid and chatItemId * skill match * perf: skill manage * fix: ts --------- Co-authored-by: xxyyh <2289112474@qq> Co-authored-by: archer <545436317@qq.com> * fix: ts * fix: loop import * skill tool config (#6114) Co-authored-by: xxyyh <2289112474@qq> * feat: load tool in agent * skill memory (#6126) Co-authored-by: xxyyh <2289112474@qq> * perf: agent skill editor * perf: helperbot ui * agent code * perf: context * fix: request context * agent usage * perf: agent context and pause * perf: plan response * Test agent sigle skill (#6184) * feat:top box fill * prompt fix --------- Co-authored-by: xxyyh <2289112474@qq> * perf: agent chat ui * Test agent new (#6219) * have-replan * agent --------- Co-authored-by: xxyyh <2289112474@qq> * fix: ts --------- Co-authored-by: YeYuheng <57035043+YYH211@users.noreply.github.com> Co-authored-by: xxyyh <2289112474@qq> * feat: consolidate agent and MCP improvements This commit consolidates 17 commits including: - MCP tools enhancements and fixes - Agent system improvements and optimizations - Auth limit and prompt updates - Tool response compression and error tracking - Simple app adaptation - Code quality improvements (TypeScript, ESLint, Zod) - Version type migration to schema - Remove deprecated useRequest2 - Add LLM error tracking - Toolset ID validation fixes --------- Co-authored-by: YeYuheng <57035043+YYH211@users.noreply.github.com> Co-authored-by: xxyyh <2289112474@qq> * fix: transform avatar copy;perf: filter invalid tool * update llm response storage time * fix: openapi schema * update skill desc * feat: cache hit data * i18n * lock * chat logs support error filter & user search (#6373) * chat log support searching by user name * support error filter * fix * fix overflow * optimize * fix init script * fix * perf: get log users * updat ecomment * fix: ts * fix: test --------- Co-authored-by: archer <545436317@qq.com> * Fix: agent (#6376) * Agent features (#6345) * Test agent (#6220) * squash: compress all commits into one * feat: plan response in ui * response ui * perf: agent config * merge * tool select ux * perf: chat ui * perf: agent editform * tmp code * feat: save chat * Complete agent parent (#6049) * add role and tools filling * add: file-upload --------- Co-authored-by: xxyyh <2289112474@qq> * perf: top agent code * top agent (#6062) Co-authored-by: xxyyh <2289112474@qq> * fix: ts * skill editor ui * ui * perf: rewrite type with zod * skill edit ui * skill agent (#6089) * cp skill chat * rebasefdf933dand add skill chat * 1. skill 的 CRUD 2. skill 的信息渲染到前端界面 * solve comment * remove chatid and chatItemId * skill match * perf: skill manage * fix: ts --------- Co-authored-by: xxyyh <2289112474@qq> Co-authored-by: archer <545436317@qq.com> * fix: ts * fix: loop import * skill tool config (#6114) Co-authored-by: xxyyh <2289112474@qq> * feat: load tool in agent * skill memory (#6126) Co-authored-by: xxyyh <2289112474@qq> * perf: agent skill editor * perf: helperbot ui * agent code * perf: context * fix: request context * agent usage * perf: agent context and pause * perf: plan response * Test agent sigle skill (#6184) * feat:top box fill * prompt fix --------- Co-authored-by: xxyyh <2289112474@qq> * perf: agent chat ui * Test agent new (#6219) * have-replan * agent --------- Co-authored-by: xxyyh <2289112474@qq> * fix: ts --------- Co-authored-by: YeYuheng <57035043+YYH211@users.noreply.github.com> Co-authored-by: xxyyh <2289112474@qq> * feat: consolidate agent and MCP improvements This commit consolidates 17 commits including: - MCP tools enhancements and fixes - Agent system improvements and optimizations - Auth limit and prompt updates - Tool response compression and error tracking - Simple app adaptation - Code quality improvements (TypeScript, ESLint, Zod) - Version type migration to schema - Remove deprecated useRequest2 - Add LLM error tracking - Toolset ID validation fixes --------- Co-authored-by: YeYuheng <57035043+YYH211@users.noreply.github.com> Co-authored-by: xxyyh <2289112474@qq> * 1. 把辅助生成前端上的 system prompt 加入到上下文中 2. mcp工具的前端渲染(图标) 3. 文件读取工具和文件上传进行关联 4. 添加了辅助生成返回格式出错的重试方案 5. ask 不出现在 plan 步骤中 6. 添加了辅助生成的头像和交互 UI * fix:read_file * helperbot ui * ts error * helper ui * delete Unused import * perf: helper bot * lock --------- Co-authored-by: Archer <545436317@qq.com> Co-authored-by: xxyyh <2289112474@qq> * fix date variable required & model auth (#6386) * fix date variable required & model auth * doc * feat: add chat id to finish callback * fix: iphone safari shareId (#6387) * fix: iphone safari shareId * fix: mcp file list can't setting * fix: reason output field * fix: skip JSON validation for HTTP tool body with variable (#6392) * fix: skip JSON validation for HTTP tool body with variable * doc * workflow fitview * perf: selecting memory * perf: cp api * ui * perf: toolcall auto adapt * fix: catch workflow error * fix: ts * perf: pagination type * remove * ignore * update doc * fix: simple app tool select * add default avatar to logs user * perf: loading user * select dataset ui * rename version * feat: add global/common test * perf: packages/global/common test * feat: package/global/ai,app test * add global/chat test * global/core test * global/core test * feat: packages/global all test * perf: test * add server api test * perf: init shell * perf: init4150 shell * remove invalid code * update doc * remove log * fix: chat effect * fix: plan fake tool (#6398) * 1. 提示词防注入功能 2. 无工具不进入 plan,防止虚拟工具生成 * Agent-dataset * dataset * dataset presetInfo * prefix * perf: prompt --------- Co-authored-by: xxyyh <2289112474@qq> Co-authored-by: archer <545436317@qq.com> * fix: review * adapt kimi2.5 think toolcall * feat: invoke fastgpt user info (#6403) feat: invoke fastgpt user info * fix: invoke fastgpt user info return orgs (#6404) * skill and version * retry helperbot (#6405) Co-authored-by: xxyyh <2289112474@qq> * update template * remove log * doc * update doc * doc * perf: internal ip check * adapt get paginationRecords * tool call adapt * fix: test * doc * fix: agent initial version * adapt completions v1 * feat: instrumentation check * rename skill * add workflow demo mode tracks (#6407) * chore: 统一 skills 目录命名为小写 将 .claude/Skills/ 重命名为 .claude/skills/ 以保持命名一致性。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * add workflow demo mode tracks * code * optimize * fix: improve workflowDemoTrack based on PR review - Add comment to empty catch block for maintainability - Add @param docs to onDemoChange clarifying nodeCount usage - Replace silent .catch with console.debug for dev debugging - Handle appId changes by reporting old data before re-init Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: archer <545436317@qq.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * remove repeat skill * fix(workflow): filter out orphan edges to prevent runtime errors (#6399) * fix(workflow): filter out orphan edges to prevent runtime errors Runtime edges that reference non-existent nodes (orphan edges) can cause unexpected behavior or crashes during workflow dispatch. This change adds a pre-check to filter out such edges before execution begins, ensuring system stability even with inconsistent graph data. * fix(workflow): enhance orphan edge filtering with logging and tests - Refactor: Extract logic to 'filterOrphanEdges' in utils.ts for better reusability - Feat: Add performance monitoring (warn if >100ms) and comprehensive logging - Feat: Support detailed edge inspection in debug mode - Docs: Add JSDoc explaining causes of orphan edges (migration, manual edits) - Test: Add unit tests covering edge cases and performance (1000 edges) Addresses PR review feedback regarding logging, variable naming, and testing." * move code * move code * add more unit test --------- Co-authored-by: archer <545436317@qq.com> * test * perf: test * add server/common/string test * fix: resolve $ref references in MCP tool input schemas (#6395) (#6409) * fix: resolve $ref references in MCP tool input schemas (#6395) * add test code --------- Co-authored-by: archer <545436317@qq.com> * chore(docs): add fastgpt, fastgpt-plugin version choice guide (#6411) * chore(doc): add fastgpt version description * doc * doc --------- Co-authored-by: archer <545436317@qq.com> * fix:dataset cite and description info (#6410) * 1. 添加知识库引用(plan 步骤和直接知识库调用) 2. 提示词框中的@知识库工具 3. plan 中 step 的 description dataset_search 改为中文 * fix: i18n * prompt * prompt --------- Co-authored-by: xxyyh <2289112474@qq> * fix: tool call * perf: workflow props * fix: merge ECharts toolbox options instead of overwriting (#6269) (#6412) * feat: integrate logtape and otel (#6400) * fix: deps * feat(logger): integrate logtape and otel * wip(log): add basic infras logs * wip(log): add request id and inject it into context * wip(log): add basic tx logs * wip(log): migrate * wip(log): category * wip(log): more sub category * fix: type * fix: sessionRun * fix: export getLogger from client.ts * chore: improve logs * docs: update signoz and changelog * change type * fix: ts * remove skill.md * fix: lockfile specifier * fix: test --------- Co-authored-by: archer <545436317@qq.com> * init log * doc * remove invalid log * fix: review * template * replace new log * fix: ts * remove log * chore: migrate all addLog to logtape * move skill * chore: migrate all addLog to logtape (#6417) * update skill * remove log * fix: tool check --------- Co-authored-by: YeYuheng <57035043+YYH211@users.noreply.github.com> Co-authored-by: xxyyh <2289112474@qq> Co-authored-by: heheer <heheer@sealos.io> Co-authored-by: Finley Ge <32237950+FinleyGe@users.noreply.github.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: xuyafei1996 <54217479+xuyafei1996@users.noreply.github.com> Co-authored-by: ToukoYui <2331631097@qq.com> Co-authored-by: roy <whoeverimf5@gmail.com>
992 lines
27 KiB
TypeScript
992 lines
27 KiB
TypeScript
import { describe, expect, it, vi, beforeEach } from 'vitest';
|
|
import { authChatCrud, authCollectionInChat } from '@/service/support/permission/auth/chat';
|
|
import { MongoChat } from '@fastgpt/service/core/chat/chatSchema';
|
|
import { MongoChatItem } from '@fastgpt/service/core/chat/chatItemSchema';
|
|
import { MongoChatItemResponse } from '@fastgpt/service/core/chat/chatItemResponseSchema';
|
|
import { AuthUserTypeEnum } from '@fastgpt/global/support/permission/constant';
|
|
import { ChatErrEnum } from '@fastgpt/global/common/error/code/chat';
|
|
import { DatasetErrEnum } from '@fastgpt/global/common/error/code/dataset';
|
|
import { authApp } from '@fastgpt/service/support/permission/app/auth';
|
|
import { authOutLink } from '@/service/support/permission/auth/outLink';
|
|
import { authTeamSpaceToken } from '@/service/support/permission/auth/team';
|
|
import type { ChatHistoryItemResType } from '@fastgpt/global/core/chat/type';
|
|
import { getFlatAppResponses } from '@fastgpt/global/core/chat/utils';
|
|
import { FlowNodeTypeEnum } from '@fastgpt/global/core/workflow/node/constant';
|
|
import { AppPermission } from '@fastgpt/global/support/permission/app/controller';
|
|
import { PublishChannelEnum } from '@fastgpt/global/support/outLink/constant';
|
|
import type { OutLinkSchema } from '@fastgpt/global/support/outLink/type';
|
|
|
|
vi.mock('@fastgpt/service/core/chat/chatSchema', () => ({
|
|
MongoChat: {
|
|
findOne: vi.fn()
|
|
}
|
|
}));
|
|
|
|
vi.mock('@fastgpt/service/core/chat/chatItemSchema', () => ({
|
|
MongoChatItem: {
|
|
findOne: vi.fn(),
|
|
find: vi.fn()
|
|
}
|
|
}));
|
|
|
|
vi.mock('@fastgpt/service/core/chat/chatItemResponseSchema', () => ({
|
|
MongoChatItemResponse: {
|
|
find: vi.fn()
|
|
}
|
|
}));
|
|
|
|
vi.mock('@fastgpt/service/support/permission/app/auth');
|
|
vi.mock('@/service/support/permission/auth/outLink');
|
|
vi.mock('@/service/support/permission/auth/team');
|
|
vi.mock('@fastgpt/global/core/chat/utils', () => ({
|
|
getFlatAppResponses: vi.fn()
|
|
}));
|
|
|
|
const buildOutLinkConfig = (
|
|
overrides: Partial<OutLinkSchema> = {},
|
|
omitKeys: (keyof OutLinkSchema)[] = []
|
|
): OutLinkSchema => {
|
|
const config: OutLinkSchema = {
|
|
_id: 'outLink1',
|
|
shareId: 'share1',
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
appId: 'app1',
|
|
name: 'out-link',
|
|
usagePoints: 0,
|
|
lastTime: new Date(),
|
|
type: PublishChannelEnum.share,
|
|
showCite: true,
|
|
showRunningStatus: true,
|
|
showFullText: false,
|
|
canDownloadSource: false,
|
|
showWholeResponse: false,
|
|
app: undefined,
|
|
...overrides
|
|
};
|
|
|
|
omitKeys.forEach((key) => {
|
|
delete (config as Partial<OutLinkSchema>)[key];
|
|
});
|
|
|
|
return config;
|
|
};
|
|
|
|
const buildQuoteList = (...ids: (string | undefined)[]): any[] =>
|
|
ids.map((id) => (id ? { collectionId: id } : {}));
|
|
|
|
const buildResponse = (
|
|
overrides: Partial<ChatHistoryItemResType> = {}
|
|
): ChatHistoryItemResType => ({
|
|
nodeId: 'node1',
|
|
id: 'response1',
|
|
moduleType: FlowNodeTypeEnum.appModule,
|
|
moduleName: 'module',
|
|
...overrides
|
|
});
|
|
|
|
const buildQuoteResponse = (...ids: (string | undefined)[]): ChatHistoryItemResType =>
|
|
buildResponse({ quoteList: buildQuoteList(...ids) });
|
|
|
|
describe('authChatCrud', () => {
|
|
beforeEach(() => {
|
|
vi.clearAllMocks();
|
|
});
|
|
|
|
describe('validation', () => {
|
|
it('should reject if appId is empty string', async () => {
|
|
await expect(authChatCrud({ appId: '', req: {} as any, authToken: true })).rejects.toBe(
|
|
ChatErrEnum.unAuthChat
|
|
);
|
|
});
|
|
|
|
it('should reject if appId is undefined', async () => {
|
|
await expect(
|
|
authChatCrud({ appId: undefined as any, req: {} as any, authToken: true })
|
|
).rejects.toBe(ChatErrEnum.unAuthChat);
|
|
});
|
|
|
|
it('should reject if appId is null', async () => {
|
|
await expect(
|
|
authChatCrud({ appId: null as any, req: {} as any, authToken: true })
|
|
).rejects.toBe(ChatErrEnum.unAuthChat);
|
|
});
|
|
});
|
|
|
|
describe('teamDomain authentication', () => {
|
|
it('should auth with teamId and teamToken without chatId', async () => {
|
|
vi.mocked(authTeamSpaceToken).mockResolvedValue({
|
|
uid: 'user1',
|
|
tmbId: 'tmb1'
|
|
});
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
teamId: 'team1',
|
|
teamToken: 'token1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toEqual({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'user1',
|
|
showCite: true,
|
|
showRunningStatus: true,
|
|
showFullText: true,
|
|
canDownloadSource: true,
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
});
|
|
});
|
|
|
|
it('should auth with teamId and teamToken with valid chatId', async () => {
|
|
const mockChat = {
|
|
appId: 'app1',
|
|
outLinkUid: 'user1',
|
|
teamId: 'team1'
|
|
};
|
|
|
|
vi.mocked(authTeamSpaceToken).mockResolvedValue({
|
|
uid: 'user1',
|
|
tmbId: 'tmb1'
|
|
});
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChat)
|
|
} as any);
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
teamId: 'team1',
|
|
teamToken: 'token1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toEqual({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'user1',
|
|
chat: mockChat,
|
|
showCite: true,
|
|
showRunningStatus: true,
|
|
showFullText: true,
|
|
canDownloadSource: true,
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
});
|
|
});
|
|
|
|
it('should handle missing chat for teamDomain auth', async () => {
|
|
vi.mocked(authTeamSpaceToken).mockResolvedValue({
|
|
uid: 'user1',
|
|
tmbId: 'tmb1'
|
|
});
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(null)
|
|
} as any);
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
teamId: 'team1',
|
|
teamToken: 'token1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toEqual({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'user1',
|
|
showCite: true,
|
|
showRunningStatus: true,
|
|
showFullText: true,
|
|
canDownloadSource: true,
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
});
|
|
});
|
|
|
|
it('should reject if chat outLinkUid does not match user for teamDomain', async () => {
|
|
const mockChat = {
|
|
appId: 'app1',
|
|
outLinkUid: 'different-user',
|
|
teamId: 'team1'
|
|
};
|
|
|
|
vi.mocked(authTeamSpaceToken).mockResolvedValue({
|
|
uid: 'user1',
|
|
tmbId: 'tmb1'
|
|
});
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChat)
|
|
} as any);
|
|
|
|
await expect(
|
|
authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
teamId: 'team1',
|
|
teamToken: 'token1',
|
|
req: {} as any,
|
|
authToken: true
|
|
})
|
|
).rejects.toBe(ChatErrEnum.unAuthChat);
|
|
});
|
|
});
|
|
|
|
describe('outLink authentication', () => {
|
|
it('should auth outLink without chatId', async () => {
|
|
vi.mocked(authOutLink).mockResolvedValue({
|
|
outLinkConfig: buildOutLinkConfig({
|
|
canDownloadSource: true
|
|
}),
|
|
uid: 'user1',
|
|
appId: 'app1'
|
|
});
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
shareId: 'share1',
|
|
outLinkUid: 'user1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toMatchObject({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'user1',
|
|
showCite: true,
|
|
showRunningStatus: true,
|
|
showFullText: false,
|
|
canDownloadSource: true,
|
|
authType: AuthUserTypeEnum.outLink
|
|
});
|
|
});
|
|
|
|
it('should auth outLink with default showRunningStatus and canDownloadSource', async () => {
|
|
vi.mocked(authOutLink).mockResolvedValue({
|
|
outLinkConfig: buildOutLinkConfig(
|
|
{
|
|
showCite: false
|
|
},
|
|
['showRunningStatus', 'canDownloadSource']
|
|
),
|
|
uid: 'user1',
|
|
appId: 'app1'
|
|
});
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
shareId: 'share1',
|
|
outLinkUid: 'user1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toMatchObject({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'user1',
|
|
showCite: false,
|
|
showRunningStatus: true, // default
|
|
canDownloadSource: false, // default
|
|
authType: AuthUserTypeEnum.outLink
|
|
});
|
|
});
|
|
|
|
it('should auth outLink with chatId', async () => {
|
|
const mockChat = {
|
|
appId: 'app1',
|
|
outLinkUid: 'user1'
|
|
};
|
|
|
|
vi.mocked(authOutLink).mockResolvedValue({
|
|
outLinkConfig: buildOutLinkConfig({
|
|
canDownloadSource: true
|
|
}),
|
|
uid: 'user1',
|
|
appId: 'app1'
|
|
});
|
|
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChat)
|
|
} as any);
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
shareId: 'share1',
|
|
outLinkUid: 'user1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toMatchObject({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'user1',
|
|
chat: mockChat,
|
|
showCite: true,
|
|
showRunningStatus: true,
|
|
showFullText: false,
|
|
canDownloadSource: true,
|
|
authType: AuthUserTypeEnum.outLink
|
|
});
|
|
});
|
|
|
|
it('should handle missing chat for outLink auth', async () => {
|
|
vi.mocked(authOutLink).mockResolvedValue({
|
|
outLinkConfig: buildOutLinkConfig({
|
|
showRunningStatus: false,
|
|
canDownloadSource: true
|
|
}),
|
|
uid: 'user1',
|
|
appId: 'app1'
|
|
});
|
|
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(null)
|
|
} as any);
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
shareId: 'share1',
|
|
outLinkUid: 'user1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toEqual({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'user1',
|
|
showCite: true,
|
|
showRunningStatus: false,
|
|
showFullText: false,
|
|
canDownloadSource: true,
|
|
authType: AuthUserTypeEnum.outLink
|
|
});
|
|
});
|
|
|
|
it('should reject if chat outLinkUid does not match for outLink auth', async () => {
|
|
const mockChat = {
|
|
appId: 'app1',
|
|
outLinkUid: 'different-user'
|
|
};
|
|
|
|
vi.mocked(authOutLink).mockResolvedValue({
|
|
outLinkConfig: buildOutLinkConfig({
|
|
showFullText: true,
|
|
canDownloadSource: true
|
|
}),
|
|
uid: 'user1',
|
|
appId: 'app1'
|
|
});
|
|
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChat)
|
|
} as any);
|
|
|
|
await expect(
|
|
authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
shareId: 'share1',
|
|
outLinkUid: 'user1',
|
|
req: {} as any,
|
|
authToken: true
|
|
})
|
|
).rejects.toBe(ChatErrEnum.unAuthChat);
|
|
});
|
|
|
|
it('should reject if outLink appId does not match', async () => {
|
|
vi.mocked(authOutLink).mockResolvedValue({
|
|
outLinkConfig: buildOutLinkConfig(),
|
|
uid: 'user1',
|
|
appId: 'different-app'
|
|
});
|
|
|
|
await expect(
|
|
authChatCrud({
|
|
appId: 'app1',
|
|
shareId: 'share1',
|
|
outLinkUid: 'user1',
|
|
req: {} as any,
|
|
authToken: true
|
|
})
|
|
).rejects.toBe(ChatErrEnum.unAuthChat);
|
|
});
|
|
|
|
it('should reject if shareId provided without outLinkUid', async () => {
|
|
// Mock authApp to simulate what happens when req is provided but shareId/outLinkUid combo is incomplete
|
|
vi.mocked(authApp).mockRejectedValue(new Error('Auth failed'));
|
|
|
|
await expect(
|
|
authChatCrud({
|
|
appId: 'app1',
|
|
shareId: 'share1',
|
|
req: {} as any,
|
|
authToken: true
|
|
})
|
|
).rejects.toThrow();
|
|
});
|
|
|
|
it('should reject if outLinkUid provided without shareId', async () => {
|
|
// Mock authApp to simulate what happens when req is provided but shareId/outLinkUid combo is incomplete
|
|
vi.mocked(authApp).mockRejectedValue(new Error('Auth failed'));
|
|
|
|
await expect(
|
|
authChatCrud({
|
|
appId: 'app1',
|
|
outLinkUid: 'user1',
|
|
req: {} as any,
|
|
authToken: true
|
|
})
|
|
).rejects.toThrow();
|
|
});
|
|
});
|
|
|
|
describe('cookie authentication', () => {
|
|
it('should auth with cookie without chatId', async () => {
|
|
vi.mocked(authApp).mockResolvedValue({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
permission: new AppPermission({
|
|
isOwner: true
|
|
}),
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
} as any);
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toEqual({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'tmb1',
|
|
showCite: true,
|
|
showRunningStatus: true,
|
|
showFullText: true,
|
|
canDownloadSource: true,
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
});
|
|
});
|
|
|
|
it('should auth with cookie and valid chatId for same team', async () => {
|
|
const mockChat = {
|
|
appId: 'app1',
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1'
|
|
};
|
|
|
|
vi.mocked(authApp).mockResolvedValue({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
permission: new AppPermission({
|
|
isOwner: true
|
|
}),
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
} as any);
|
|
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChat)
|
|
} as any);
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toEqual({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'tmb1',
|
|
chat: mockChat,
|
|
showCite: true,
|
|
showRunningStatus: true,
|
|
showFullText: true,
|
|
canDownloadSource: true,
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
});
|
|
});
|
|
|
|
it('should auth with readChatLogPer permission for different user chat', async () => {
|
|
const mockChat = {
|
|
appId: 'app1',
|
|
teamId: 'team1',
|
|
tmbId: 'different-tmb'
|
|
};
|
|
|
|
vi.mocked(authApp).mockResolvedValue({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
permission: new AppPermission({
|
|
isOwner: false,
|
|
role: 8 // ReadChatLogRole value 0b1000
|
|
}),
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
} as any);
|
|
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChat)
|
|
} as any);
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toEqual({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'tmb1',
|
|
chat: mockChat,
|
|
showCite: true,
|
|
showRunningStatus: true,
|
|
showFullText: true,
|
|
canDownloadSource: true,
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
});
|
|
});
|
|
|
|
it('should handle missing chat for cookie auth', async () => {
|
|
vi.mocked(authApp).mockResolvedValue({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
permission: new AppPermission({
|
|
isOwner: true
|
|
}),
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
} as any);
|
|
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(null)
|
|
} as any);
|
|
|
|
const result = await authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
req: {} as any,
|
|
authToken: true
|
|
});
|
|
|
|
expect(result).toEqual({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
uid: 'tmb1',
|
|
showCite: true,
|
|
showRunningStatus: true,
|
|
showFullText: true,
|
|
canDownloadSource: true,
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
});
|
|
});
|
|
|
|
it('should reject if chat belongs to different team', async () => {
|
|
const mockChat = {
|
|
appId: 'app1',
|
|
teamId: 'different-team',
|
|
tmbId: 'tmb1'
|
|
};
|
|
|
|
vi.mocked(authApp).mockResolvedValue({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
permission: new AppPermission({
|
|
isOwner: true
|
|
}),
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
} as any);
|
|
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChat)
|
|
} as any);
|
|
|
|
await expect(
|
|
authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
req: {} as any,
|
|
authToken: true
|
|
})
|
|
).rejects.toBe(ChatErrEnum.unAuthChat);
|
|
});
|
|
|
|
it('should reject if user has no permission for different user chat', async () => {
|
|
const mockChat = {
|
|
appId: 'app1',
|
|
teamId: 'team1',
|
|
tmbId: 'different-tmb'
|
|
};
|
|
|
|
vi.mocked(authApp).mockResolvedValue({
|
|
teamId: 'team1',
|
|
tmbId: 'tmb1',
|
|
permission: new AppPermission({
|
|
isOwner: false,
|
|
role: 0 // no role/permissions
|
|
}),
|
|
authType: AuthUserTypeEnum.teamDomain
|
|
} as any);
|
|
|
|
vi.mocked(MongoChat.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChat)
|
|
} as any);
|
|
|
|
await expect(
|
|
authChatCrud({
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
req: {} as any,
|
|
authToken: true
|
|
})
|
|
).rejects.toBe(ChatErrEnum.unAuthChat);
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('authCollectionInChat', () => {
|
|
beforeEach(() => {
|
|
vi.clearAllMocks();
|
|
vi.mocked(MongoChatItem.find).mockReturnValue({
|
|
sort: () => ({
|
|
limit: () => ({
|
|
lean: () => Promise.resolve([])
|
|
})
|
|
})
|
|
} as any);
|
|
});
|
|
|
|
describe('validation', () => {
|
|
it('should reject if chat item not found', async () => {
|
|
// Mock the find method to return empty array (no cite collection ids found)
|
|
vi.mocked(MongoChatItem.find).mockReturnValue({
|
|
sort: () => ({
|
|
limit: () => ({
|
|
lean: () => Promise.resolve([])
|
|
})
|
|
})
|
|
} as any);
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(null)
|
|
} as any);
|
|
|
|
await expect(
|
|
authCollectionInChat({
|
|
collectionIds: ['col1'],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
})
|
|
).rejects.toBe(DatasetErrEnum.unAuthDatasetFile);
|
|
});
|
|
|
|
it('should reject with empty collectionIds array', async () => {
|
|
// Mock the find method to return empty array (no cite collection ids found)
|
|
vi.mocked(MongoChatItem.find).mockReturnValue({
|
|
sort: () => ({
|
|
limit: () => ({
|
|
lean: () => Promise.resolve([])
|
|
})
|
|
})
|
|
} as any);
|
|
|
|
const mockChatItem = {
|
|
time: new Date(),
|
|
responseData: []
|
|
};
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItem)
|
|
} as any);
|
|
vi.mocked(MongoChatItemResponse.find).mockReturnValue({
|
|
lean: () => Promise.resolve([])
|
|
} as any);
|
|
vi.mocked(getFlatAppResponses).mockReturnValue([]);
|
|
|
|
const result = await authCollectionInChat({
|
|
collectionIds: [],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
});
|
|
|
|
expect(result).toEqual(undefined);
|
|
});
|
|
|
|
it('should handle missing appId, chatId, or chatItemDataId', async () => {
|
|
await expect(
|
|
authCollectionInChat({
|
|
collectionIds: ['col1'],
|
|
appId: '',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
})
|
|
).rejects.toBe(DatasetErrEnum.unAuthDatasetFile);
|
|
});
|
|
});
|
|
|
|
describe('response data handling', () => {
|
|
it('should auth collection ids in chat item with existing responseData', async () => {
|
|
// Mock the find method to return empty array (no cite collection ids found)
|
|
vi.mocked(MongoChatItem.find).mockReturnValue({
|
|
sort: () => ({
|
|
limit: () => ({
|
|
lean: () => Promise.resolve([])
|
|
})
|
|
})
|
|
} as any);
|
|
|
|
const mockChatItem = {
|
|
time: new Date(),
|
|
citeCollectionIds: ['col1', 'col2']
|
|
};
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItem)
|
|
} as any);
|
|
vi.mocked(getFlatAppResponses).mockReturnValue([buildQuoteResponse('col1', 'col2')]);
|
|
|
|
const result = await authCollectionInChat({
|
|
collectionIds: ['col1', 'col2'],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
});
|
|
|
|
expect(result).toEqual(undefined);
|
|
});
|
|
|
|
it('should fetch responseData from MongoChatItemResponse when missing', async () => {
|
|
const mockChatItem: { time: Date; citeCollectionIds: string[]; responseData?: any[] } = {
|
|
time: new Date(),
|
|
citeCollectionIds: ['col1', 'col2']
|
|
};
|
|
const mockChatItemResponses = [
|
|
{ data: buildQuoteResponse('col1') },
|
|
{ data: buildQuoteResponse('col2') }
|
|
];
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItem)
|
|
} as any);
|
|
vi.mocked(MongoChatItemResponse.find).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItemResponses)
|
|
} as any);
|
|
vi.mocked(getFlatAppResponses).mockReturnValue([
|
|
buildQuoteResponse('col1'),
|
|
buildQuoteResponse('col2')
|
|
]);
|
|
|
|
const result = await authCollectionInChat({
|
|
collectionIds: ['col1', 'col2'],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
});
|
|
|
|
expect(mockChatItem.responseData).toEqual([
|
|
buildQuoteResponse('col1'),
|
|
buildQuoteResponse('col2')
|
|
]);
|
|
expect(result).toEqual(undefined);
|
|
});
|
|
|
|
it('should handle empty responseData array', async () => {
|
|
const mockChatItem = {
|
|
time: new Date(),
|
|
responseData: []
|
|
};
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItem)
|
|
} as any);
|
|
vi.mocked(MongoChatItemResponse.find).mockReturnValue({
|
|
lean: () => Promise.resolve([])
|
|
} as any);
|
|
vi.mocked(getFlatAppResponses).mockReturnValue([]);
|
|
|
|
await expect(
|
|
authCollectionInChat({
|
|
collectionIds: ['col1'],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
})
|
|
).rejects.toBe(DatasetErrEnum.unAuthDatasetFile);
|
|
});
|
|
|
|
it('should handle plugin, tool and loop details in response data', async () => {
|
|
const mockChatItem = {
|
|
time: new Date(),
|
|
responseData: [
|
|
buildResponse({
|
|
quoteList: buildQuoteList('col1'),
|
|
pluginDetail: [buildQuoteResponse('col2')],
|
|
toolDetail: [buildQuoteResponse('col3')],
|
|
loopDetail: [buildQuoteResponse('col4')]
|
|
})
|
|
]
|
|
};
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItem)
|
|
} as any);
|
|
vi.mocked(getFlatAppResponses).mockReturnValue([
|
|
buildQuoteResponse('col1'),
|
|
buildQuoteResponse('col2'),
|
|
buildQuoteResponse('col3'),
|
|
buildQuoteResponse('col4')
|
|
]);
|
|
|
|
const result = await authCollectionInChat({
|
|
collectionIds: ['col1', 'col2', 'col3', 'col4'],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
});
|
|
|
|
expect(result).toEqual(undefined);
|
|
});
|
|
|
|
it('should reject if collection ids not found in quotes', async () => {
|
|
const mockChatItem = {
|
|
time: new Date(),
|
|
responseData: [buildQuoteResponse('col1')]
|
|
};
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItem)
|
|
} as any);
|
|
vi.mocked(getFlatAppResponses).mockReturnValue([buildQuoteResponse('col1')]);
|
|
|
|
await expect(
|
|
authCollectionInChat({
|
|
collectionIds: ['col2'],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
})
|
|
).rejects.toBe(DatasetErrEnum.unAuthDatasetFile);
|
|
});
|
|
|
|
it('should reject if only some collection ids are found', async () => {
|
|
const mockChatItem = {
|
|
time: new Date(),
|
|
responseData: [buildQuoteResponse('col1', 'col2')]
|
|
};
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItem)
|
|
} as any);
|
|
vi.mocked(getFlatAppResponses).mockReturnValue([buildQuoteResponse('col1', 'col2')]);
|
|
|
|
await expect(
|
|
authCollectionInChat({
|
|
collectionIds: ['col1', 'col2', 'col3'], // col3 not found
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
})
|
|
).rejects.toBe(DatasetErrEnum.unAuthDatasetFile);
|
|
});
|
|
|
|
it('should handle quotes with missing collectionId', async () => {
|
|
const mockChatItem = {
|
|
time: new Date(),
|
|
responseData: [buildQuoteResponse('col1', undefined)]
|
|
};
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItem)
|
|
} as any);
|
|
vi.mocked(getFlatAppResponses).mockReturnValue([buildQuoteResponse('col1', undefined)]);
|
|
|
|
const result = await authCollectionInChat({
|
|
collectionIds: ['col1'],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
});
|
|
|
|
expect(result).toEqual(undefined);
|
|
});
|
|
|
|
it('should handle missing quoteList in response data', async () => {
|
|
const mockChatItem = {
|
|
time: new Date(),
|
|
responseData: [
|
|
{
|
|
// no quoteList
|
|
}
|
|
]
|
|
};
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItem)
|
|
} as any);
|
|
vi.mocked(getFlatAppResponses).mockReturnValue([]);
|
|
|
|
await expect(
|
|
authCollectionInChat({
|
|
collectionIds: ['col1'],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
})
|
|
).rejects.toBe(DatasetErrEnum.unAuthDatasetFile);
|
|
});
|
|
});
|
|
|
|
describe('error handling', () => {
|
|
it('should reject if database query throws error', async () => {
|
|
vi.mocked(MongoChatItem.findOne).mockImplementation(() => {
|
|
throw new Error('Database error');
|
|
});
|
|
|
|
await expect(
|
|
authCollectionInChat({
|
|
collectionIds: ['col1'],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
})
|
|
).rejects.toBe(DatasetErrEnum.unAuthDatasetFile);
|
|
});
|
|
|
|
it('should reject if getFlatAppResponses throws error', async () => {
|
|
const mockChatItem = {
|
|
time: new Date(),
|
|
responseData: [{}]
|
|
};
|
|
|
|
vi.mocked(MongoChatItem.findOne).mockReturnValue({
|
|
lean: () => Promise.resolve(mockChatItem)
|
|
} as any);
|
|
vi.mocked(getFlatAppResponses).mockImplementation(() => {
|
|
throw new Error('Processing error');
|
|
});
|
|
|
|
await expect(
|
|
authCollectionInChat({
|
|
collectionIds: ['col1'],
|
|
appId: 'app1',
|
|
chatId: 'chat1',
|
|
chatItemDataId: 'item1'
|
|
})
|
|
).rejects.toBe(DatasetErrEnum.unAuthDatasetFile);
|
|
});
|
|
});
|
|
});
|