Skip to content

Commit

Permalink
4.8.5 perf (#1854)
Browse files Browse the repository at this point in the history
* update yml

* perf: chat slider

* perf: workflow name

* i18n

* fix: ts

* fix: ts
  • Loading branch information
c121914yu authored Jun 26, 2024
1 parent 93b44fc commit e247545
Show file tree
Hide file tree
Showing 15 changed files with 136 additions and 33 deletions.
9 changes: 5 additions & 4 deletions docSite/content/zh-cn/docs/development/upgrading/485.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,8 @@ curl --location --request POST 'https://{{host}}/api/admin/init/485' \
7. 优化 - 问答拆分/手动录入,当有`a`字段时,自动将`q`作为补充索引。
8. 优化 - 对话框页面代码
9. 修复 - SSR渲染
10. 修复 - 定时任务无法实际关闭
11. 修复 - 输入引导特殊字符导致正则报错
12. 修复 - 文件包含特殊字符`%`,且为转义时会导致页面崩溃
13. 修复 - 自定义输入选择知识库引用时页面崩溃
10. 优化 - 工作流新节点自动增加序号名
11. 修复 - 定时任务无法实际关闭
12. 修复 - 输入引导特殊字符导致正则报错
13. 修复 - 文件包含特殊字符`%`,且为转义时会导致页面崩溃
14. 修复 - 自定义输入选择知识库引用时页面崩溃
1 change: 1 addition & 0 deletions files/docker/docker-compose-milvus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -179,6 +179,7 @@ services:
oneapi:
container_name: oneapi
image: ghcr.io/songquanpeng/one-api:latest
# image: registry.cn-hangzhou.aliyuncs.com/fastgpt/one-api:v0.6.6 # 阿里云
ports:
- 3001:3000
depends_on:
Expand Down
1 change: 1 addition & 0 deletions files/docker/docker-compose-pgvector.yml
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,7 @@ services:
oneapi:
container_name: oneapi
image: ghcr.io/songquanpeng/one-api:latest
# image: registry.cn-hangzhou.aliyuncs.com/fastgpt/one-api:v0.6.6 # 阿里云
ports:
- 3001:3000
depends_on:
Expand Down
1 change: 1 addition & 0 deletions files/docker/docker-compose-zilliz.yml
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,7 @@ services:
oneapi:
container_name: oneapi
image: ghcr.io/songquanpeng/one-api:latest
# image: registry.cn-hangzhou.aliyuncs.com/fastgpt/one-api:v0.6.6 # 阿里云
ports:
- 3001:3000
depends_on:
Expand Down
5 changes: 2 additions & 3 deletions packages/web/i18n/zh/common.json
Original file line number Diff line number Diff line change
Expand Up @@ -596,8 +596,7 @@
"success": "开始同步"
}
},
"training": {
}
"training": {}
},
"data": {
"Auxiliary Data": "辅助数据",
Expand Down Expand Up @@ -1461,7 +1460,7 @@
"Upgrade plan": "升级套餐",
"function": {
"History store": "{{amount}} 天对话记录保留",
"Max app": "{{amount}} 个应用",
"Max app": "{{amount}} 个应用&插件",
"Max dataset": "{{amount}} 个知识库",
"Max dataset size": "{{amount}} 组知识库索引",
"Max members": "{{amount}} 个团队成员",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ const SelectOneResource = ({
</Flex>
)}
<Avatar ml={index !== 0 ? '0.5rem' : 0} src={item.avatar} w={'1.25rem'} />
<Box fontSize={['md', 'sm']} ml={2}>
<Box fontSize={['md', 'sm']} ml={2} className="textEllipsis">
{item.name}
</Box>
</Flex>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ const AppCard = () => {
>
{appDetail.intro || t('core.app.tip.Add a intro to app')}
</Box>
<HStack>
<HStack alignItems={'flex-end'}>
<Button
size={['sm', 'md']}
variant={'whitePrimary'}
Expand Down Expand Up @@ -155,7 +155,7 @@ const AppCard = () => {
colorSchema="gray"
onClick={() => (appDetail.permission.hasManagePer ? onOpenInfoEdit() : undefined)}
>
<PermissionIconText defaultPermission={appDetail.defaultPermission} fontSize={'md'} />
<PermissionIconText defaultPermission={appDetail.defaultPermission} />
</MyTag>
</HStack>
</Box>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ import { ParentIdType } from '@fastgpt/global/common/parentFolder/type';
import MyBox from '@fastgpt/web/components/common/MyBox';
import FolderPath from '@/components/common/folder/Path';
import { getAppFolderPath } from '@/web/core/app/api/app';
import { useWorkflowUtils } from './hooks/useUtils';

type ModuleTemplateListProps = {
isOpen: boolean;
Expand Down Expand Up @@ -59,6 +60,13 @@ const NodeTemplatesModal = ({ isOpen, onClose }: ModuleTemplateListProps) => {
WorkflowContext,
(v) => v
);
const [pluginBuffer, setPluginBuffer] = useState<{
systemPlugin: FlowNodeTemplateType[];
teamPlugin: FlowNodeTemplateType[];
}>({
[TemplateTypeEnum.systemPlugin]: [],
[TemplateTypeEnum.teamPlugin]: []
});

const [templateType, setTemplateType] = useState(TemplateTypeEnum.basic);

Expand All @@ -85,14 +93,34 @@ const NodeTemplatesModal = ({ isOpen, onClose }: ModuleTemplateListProps) => {
});
}
if (templateType === TemplateTypeEnum.systemPlugin) {
return getSystemPlugTemplates();
if (pluginBuffer.systemPlugin.length === 0) {
return getSystemPlugTemplates().then((res) => {
setPluginBuffer((state) => ({
...state,
systemPlugin: res
}));
return res;
});
} else {
return pluginBuffer.systemPlugin;
}
}
if (templateType === TemplateTypeEnum.teamPlugin) {
return getTeamPlugTemplates({
parentId,
searchKey,
type: [AppTypeEnum.folder, AppTypeEnum.httpPlugin, AppTypeEnum.plugin]
});
if (pluginBuffer.teamPlugin.length === 0) {
return getTeamPlugTemplates({
parentId,
searchKey,
type: [AppTypeEnum.folder, AppTypeEnum.httpPlugin, AppTypeEnum.plugin]
}).then((res) => {
setPluginBuffer((state) => ({
...state,
teamPlugin: res
}));
return res;
});
} else {
return pluginBuffer.teamPlugin;
}
}
return [];
},
Expand Down Expand Up @@ -240,6 +268,7 @@ const RenderList = React.memo(function RenderList({
const { toast } = useToast();
const reactFlowWrapper = useContextSelector(WorkflowContext, (v) => v.reactFlowWrapper);
const setNodes = useContextSelector(WorkflowContext, (v) => v.setNodes);
const { computedNewNodeName } = useWorkflowUtils();

const formatTemplates = useMemo<nodeTemplateListType>(() => {
const copy: nodeTemplateListType = JSON.parse(JSON.stringify(workflowNodeTemplateList(t)));
Expand All @@ -266,6 +295,8 @@ const RenderList = React.memo(function RenderList({
setLoading(false);
return res;
}

// base node
return { ...template };
} catch (e) {
toast({
Expand All @@ -284,7 +315,11 @@ const RenderList = React.memo(function RenderList({
const node = nodeTemplate2FlowNode({
template: {
...templateNode,
name: t(templateNode.name),
name: computedNewNodeName({
templateName: t(templateNode.name),
flowNodeType: templateNode.flowNodeType,
pluginId: templateNode.pluginId
}),
intro: t(templateNode.intro || '')
},
position: { x: mouseX, y: mouseY - 20 },
Expand All @@ -301,7 +336,7 @@ const RenderList = React.memo(function RenderList({
.concat(node)
);
},
[reactFlowWrapper, setLoading, setNodes, t, toast, x, y, zoom]
[computedNewNodeName, reactFlowWrapper, setLoading, setNodes, t, toast, x, y, zoom]
);

const Render = useMemo(() => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,13 @@ import { Node } from 'reactflow';
import { FlowNodeItemType } from '@fastgpt/global/core/workflow/type';
import { useContextSelector } from 'use-context-selector';
import { WorkflowContext, getWorkflowStore } from '../../context';
import { useWorkflowUtils } from './useUtils';

export const useKeyboard = () => {
const { t } = useTranslation();
const { setNodes, onSaveWorkflow } = useContextSelector(WorkflowContext, (v) => v);
const { copyData } = useCopyData();
const { computedNewNodeName } = useWorkflowUtils();

const [isDowningCtrl, setIsDowningCtrl] = useState(false);

Expand Down Expand Up @@ -56,6 +58,11 @@ export const useKeyboard = () => {
id: nodeId,
data: {
...item.data,
name: computedNewNodeName({
templateName: item.data?.name || '',
flowNodeType: item.data?.flowNodeType || '',
pluginId: item.data?.pluginId
}),
nodeId
},
position: {
Expand All @@ -75,7 +82,7 @@ export const useKeyboard = () => {
.concat(newNodes)
);
} catch (error) {}
}, [hasInputtingElement, setNodes]);
}, [computedNewNodeName, hasInputtingElement, setNodes]);

const handleKeyDown = useCallback(
(event: KeyboardEvent) => {
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
import { useContextSelector } from 'use-context-selector';
import { WorkflowContext } from '../../context';
import { useTranslation } from 'next-i18next';
import { useCallback } from 'react';
import { FlowNodeTypeEnum } from '@fastgpt/global/core/workflow/node/constant';

export const useWorkflowUtils = () => {
const { t } = useTranslation();
const nodeList = useContextSelector(WorkflowContext, (v) => v.nodeList);

const computedNewNodeName = useCallback(
({
templateName,
flowNodeType,
pluginId
}: {
templateName: string;
flowNodeType: FlowNodeTypeEnum;
pluginId?: string;
}) => {
const nodeLength = nodeList.filter((node) => {
if (node.flowNodeType === flowNodeType) {
if (node.flowNodeType === FlowNodeTypeEnum.pluginModule) {
return node.pluginId === pluginId;
} else {
return true;
}
}
}).length;
return nodeLength > 0 ? `${templateName}#${nodeLength + 1}` : templateName;
},
[nodeList]
);

return {
computedNewNodeName
};
};

export default function Dom() {
return <></>;
}
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ import MyTooltip from '@fastgpt/web/components/common/MyTooltip';
import { useSystemStore } from '@/web/common/system/useSystemStore';
import { useMount } from 'ahooks';
import { useRequest2 } from '@fastgpt/web/hooks/useRequest';
import { useWorkflowUtils } from '../../hooks/useUtils';

type Props = FlowNodeItemType & {
children?: React.ReactNode | React.ReactNode[] | string;
Expand Down Expand Up @@ -292,6 +293,7 @@ const MenuRender = React.memo(function MenuRender({

const setNodes = useContextSelector(WorkflowContext, (v) => v.setNodes);
const setEdges = useContextSelector(WorkflowContext, (v) => v.setEdges);
const { computedNewNodeName } = useWorkflowUtils();

const onCopyNode = useCallback(
(nodeId: string) => {
Expand All @@ -300,7 +302,11 @@ const MenuRender = React.memo(function MenuRender({
if (!node) return state;
const template = {
avatar: node.data.avatar,
name: node.data.name,
name: computedNewNodeName({
templateName: node.data.name,
flowNodeType: node.data.flowNodeType,
pluginId: node.data.pluginId
}),
intro: node.data.intro,
flowNodeType: node.data.flowNodeType,
inputs: node.data.inputs,
Expand All @@ -323,12 +329,13 @@ const MenuRender = React.memo(function MenuRender({
inputs: template.inputs,
outputs: template.outputs,
version: template.version
}
},
selected: true
})
);
});
},
[setNodes]
[computedNewNodeName, setNodes]
);
const onDelNode = useCallback(
(nodeId: string) => {
Expand Down
19 changes: 11 additions & 8 deletions projects/app/src/pages/app/list/components/List.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -36,13 +36,17 @@ const ConfigPerModal = dynamic(() => import('@/components/support/permission/Con
import type { EditHttpPluginProps } from './HttpPluginEditModal';
import { postCopyApp } from '@/web/core/app/api/app';
import { getTeamMembers } from '@/web/support/user/team/api';
import { formatTimeToChatTime } from '@fastgpt/global/common/string/time';
import { useSystem } from '@fastgpt/web/hooks/useSystem';
import { useSystemStore } from '@/web/common/system/useSystemStore';
const HttpEditModal = dynamic(() => import('./HttpPluginEditModal'));

const ListItem = () => {
const { t } = useTranslation();
const { appT } = useI18n();
const router = useRouter();
const { isPc } = useSystem();

const { myApps, loadMyApps, onUpdateApp, setMoveAppId, folderDetail, appType } =
useContextSelector(AppListContext, (v) => v);
const [loadingAppId, setLoadingAppId] = useState<string>();
Expand Down Expand Up @@ -175,9 +179,6 @@ const ListItem = () => {
isFolder: app.type === AppTypeEnum.folder
})}
>
{/* <Box position={'absolute'} top={3.5} right={0}>
<AppTypeTag type={app.type} />
</Box> */}
<HStack>
<Avatar src={app.avatar} borderRadius={'md'} w={'1.5rem'} />
<Box flex={'1 0 0'} fontSize={'1.125rem'}>
Expand All @@ -188,7 +189,7 @@ const ListItem = () => {
</Box>
</HStack>
<Box
flex={'1 0 80px'}
flex={['1 0 60px', '1 0 80px']}
mt={3}
pr={8}
textAlign={'justify'}
Expand Down Expand Up @@ -219,10 +220,12 @@ const ListItem = () => {
</HStack>

<HStack>
{/* <HStack spacing={0.5} className="time">
<MyIcon name={'history'} w={'0.85rem'} />
<Box>{formatTimeToChatTime(app.updateTime)}</Box>
</HStack> */}
{isPc && (
<HStack spacing={0.5} className="time">
<MyIcon name={'history'} w={'0.85rem'} />
<Box>{formatTimeToChatTime(app.updateTime)}</Box>
</HStack>
)}
{app.permission.hasManagePer && (
<Box className="more" display={['', 'none']}>
<MyMenu
Expand Down
2 changes: 1 addition & 1 deletion projects/app/src/pages/app/list/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ const MyApps = () => {
gap={5}
display={'flex'}
alignItems={'center'}
fontSize={'md'}
fontSize={['sm', 'md']}
onChange={(e) => {
router.push({
query: {
Expand Down
5 changes: 4 additions & 1 deletion projects/app/src/pages/chat/components/ChatHistorySlider.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,10 @@ const ChatHistorySlider = ({
{!isPc && appId && (
<LightRowTabs<TabEnum>
flex={'1 0 0'}
mr={2}
mr={1}
inlineStyles={{
px: 1
}}
list={[
{ label: t('core.chat.Recent use'), value: TabEnum.recently },
{ label: t('App'), value: TabEnum.app },
Expand Down
Loading

0 comments on commit e247545

Please sign in to comment.