youngkingdom/dify
1
0
Fork 0
mirror of https://github.com/langgenius/dify.git synced 2026-05-05 01:48:04 +08:00
Code Issues Packages Projects Releases Wiki Activity

refactor(skill): use node.parent chain for ancestor traversal

Browse Source 
Replace getAncestorIds(treeData) with node.parent chain traversal
for more efficient ancestor lookup. This avoids re-traversing the
tree data structure and uses react-arborist's built-in parent refs.

Also rename hook to useSyncTreeWithActiveTab for clarity.
This commit is contained in:
yyh
2026-01-16 14:27:21 +08:00
parent 76da178cc1
commit 0f5d3f38da
3 changed files with 56 additions and 50 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
web/app/components/workflow/skill/file-tree/index.tsx
View File

@ -16,8 +16,8 @@ import Toast from '@/app/components/base/toast'
import { useStore, useWorkflowStore } from '@/app/components/workflow/store'
import { useRenameAppAssetNode } from '@/service/use-app-asset'
import { cn } from '@/utils/classnames'
import { useRevealActiveTab } from '../hooks/use-reveal-active-tab'
import { useSkillAssetTreeData } from '../hooks/use-skill-asset-tree'
import { useSyncTreeWithActiveTab } from '../hooks/use-sync-tree-with-active-tab'
import TreeContextMenu from './tree-context-menu'
import TreeNode from './tree-node'
@ -85,10 +85,9 @@ const FileTree: React.FC<FileTreeProps> = ({ className }) => {
})
}, [appId, renameNode, t])
useRevealActiveTab({
useSyncTreeWithActiveTab({
treeRef,
activeTabId,
treeChildren: treeData?.children,
})
if (isLoading) {