kexsh/dify
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix: show uninstalled plugin nodes in workflow checklist (#29630)

Browse Source
This commit is contained in:
yyh
2025-12-15 10:11:23 +08:00
committed by GitHub
parent b8d54d745e
commit 59137f1d05
24 changed files with 1315 additions and 1270 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
web/app/components/workflow/header/checklist.tsx
View File

@@ -37,9 +37,13 @@ import useNodes from '@/app/components/workflow/store/workflow/use-nodes'
type WorkflowChecklistProps = {
disabled: boolean
showGoTo?: boolean
onItemClick?: (item: ChecklistItem) => void
}
const WorkflowChecklist = ({
disabled,
showGoTo = true,
onItemClick,
}: WorkflowChecklistProps) => {
const { t } = useTranslation()
const [open, setOpen] = useState(false)
@@ -49,9 +53,13 @@ const WorkflowChecklist = ({
const { handleNodeSelect } = useNodesInteractions()
const handleChecklistItemClick = (item: ChecklistItem) => {
if (!item.canNavigate)
const goToEnabled = showGoTo && item.canNavigate && !item.disableGoTo
if (!goToEnabled)
return
handleNodeSelect(item.id)
if (onItemClick)
onItemClick(item)
else
handleNodeSelect(item.id)
setOpen(false)
}
@@ -116,7 +124,7 @@ const WorkflowChecklist = ({
key={node.id}
className={cn(
'group mb-2 rounded-lg border-[0.5px] border-components-panel-border bg-components-panel-bg shadow-xs last-of-type:mb-0',
node.canNavigate ? 'cursor-pointer' : 'cursor-default opacity-80',
showGoTo && node.canNavigate && !node.disableGoTo ? 'cursor-pointer' : 'cursor-default opacity-80',
)}
onClick={() => handleChecklistItemClick(node)}
>
@@ -130,7 +138,7 @@ const WorkflowChecklist = ({
{node.title}
</span>
{
node.canNavigate && (
(showGoTo && node.canNavigate && !node.disableGoTo) && (
<div className='flex h-4 w-[60px] shrink-0 items-center justify-center gap-1 opacity-0 transition-opacity duration-200 group-hover:opacity-100'>
<span className='whitespace-nowrap text-xs font-medium leading-4 text-primary-600'>
{t('workflow.panel.goTo')}

20
web/app/components/workflow/hooks/use-checklist.ts
View File

@@ -66,6 +66,7 @@ export type ChecklistItem = {
unConnected?: boolean
errorMessage?: string
canNavigate: boolean
disableGoTo?: boolean
}
const START_NODE_TYPES: BlockEnum[] = [
@@ -75,6 +76,13 @@ const START_NODE_TYPES: BlockEnum[] = [
BlockEnum.TriggerPlugin,
]
// Node types that depend on plugins
const PLUGIN_DEPENDENT_TYPES: BlockEnum[] = [
BlockEnum.Tool,
BlockEnum.DataSource,
BlockEnum.TriggerPlugin,
]
export const useChecklist = (nodes: Node[], edges: Edge[]) => {
const { t } = useTranslation()
const language = useGetLanguage()
@@ -157,7 +165,14 @@ export const useChecklist = (nodes: Node[], edges: Edge[]) => {
if (node.type === CUSTOM_NODE) {
const checkData = getCheckData(node.data)
const validator = nodesExtraData?.[node.data.type as BlockEnum]?.checkValid
let errorMessage = validator ? validator(checkData, t, moreDataForCheckValid).errorMessage : undefined
const isPluginMissing = PLUGIN_DEPENDENT_TYPES.includes(node.data.type as BlockEnum) && node.data._pluginInstallLocked
// Check if plugin is installed for plugin-dependent nodes first
let errorMessage: string | undefined
if (isPluginMissing)
errorMessage = t('workflow.nodes.common.pluginNotInstalled')
else if (validator)
errorMessage = validator(checkData, t, moreDataForCheckValid).errorMessage
if (!errorMessage) {
const availableVars = map[node.id].availableVars
@@ -194,7 +209,8 @@ export const useChecklist = (nodes: Node[], edges: Edge[]) => {
toolIcon,
unConnected: isUnconnected && !canSkipConnectionCheck,
errorMessage,
canNavigate: true,
canNavigate: !isPluginMissing,
disableGoTo: isPluginMissing,
})
}
}