-
Notifications
You must be signed in to change notification settings - Fork 426
[feat] Add missing nodes warning UI to queue button and breadcrumb #6674
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🎭 Playwright Test Results⏰ Completed at: 11/19/2025, 08:46:42 AM UTC 📈 Summary
📊 Test Reports by Browser
🎉 Click on the links above to view detailed test results for each browser configuration. |
🎨 Storybook Build Status✅ Build completed successfully! ⏰ Completed at: 11/19/2025, 08:36:23 AM UTC 🔗 Links🎉 Your Storybook is ready for review! |
Bundle Size ReportSummary
Category Glance Per-category breakdownApp Entry Points — 3.14 MB (baseline 3.14 MB) • 🔴 +416 BMain entry bundles and manifests
Status: 3 added / 3 removed Graph Workspace — 913 kB (baseline 913 kB) • 🔴 +498 BGraph editor runtime, canvas, workflow orchestration
Status: 1 added / 1 removed Views & Navigation — 8.03 kB (baseline 8.03 kB) • ⚪ 0 BTop-level views, pages, and routed surfaces
Status: 1 added / 1 removed Panels & Settings — 307 kB (baseline 307 kB) • 🔴 +2 BConfiguration panels, inspectors, and settings screens
Status: 6 added / 6 removed UI Components — 130 kB (baseline 130 kB) • 🔴 +337 BReusable component library chunks
Status: 5 added / 5 removed Data & Services — 12.6 kB (baseline 12.6 kB) • ⚪ 0 BStores, services, APIs, and repositories
Status: 3 added / 3 removed Utilities & Hooks — 2.94 kB (baseline 2.94 kB) • ⚪ 0 BHelpers, composables, and utility bundles
Status: 1 added / 1 removed Vendor & Third-Party — 5.32 MB (baseline 5.32 MB) • 🔴 +15 BExternal libraries and shared vendor chunks
Status: 3 added / 3 removed Other — 3.9 MB (baseline 3.9 MB) • ⚪ 0 BBundles that do not match a named category
Status: 19 added / 19 removed |
a6f823e to
b4d2fa3
Compare
src/workbench/extensions/manager/composables/nodePack/useMissingNodes.ts
Show resolved
Hide resolved
b4d2fa3 to
e032417
Compare
e032417 to
6702548
Compare
29ca9ab to
1671d72
Compare
1671d72 to
a2508e2
Compare
Display visual warnings when workflow contains missing nodes. Key changes: - Convert useMissingNodes to singleton with createSharedComposable - Add activeWorkflow watch for automatic updates on workflow changes - Add warning icon + disabled state + tooltip to ComfyQueueButton - Add warning icon + tooltip to root SubgraphBreadcrumbItem - Add i18n keys (menu.runWorkflowDisabled, breadcrumbsMenu.missingNodesWarning) Technical rationale: - createSharedComposable: All components share same instance, prevents duplicate API calls - watch: Ensures persistent components (ComfyQueueButton) update automatically on workflow changes 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
9949458 to
193b24f
Compare
christian-byrne
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@viva-jinyi Commit |
1 similar comment
|
@viva-jinyi Commit |
|
Can't execute, grayed out run button, without any missing nodes, all nodes loaded properly. Reverting back to an older version |
📋 Summary
Add visual warning indicators to notify users when a workflow contains missing nodes.
🎯 Changes
1. Improve Missing Nodes Detection System
Problem:
useMissingNodescomposable was executing independently in each component, causing duplicate API callsSolution:
Why use
createSharedComposable?startFetchWorkflowPacks()executes only onceWhy use
watch?activeWorkflowchanges2. Add UI Warning Indicators
ComfyQueueButton (Run button)
SubgraphBreadcrumbItem (Workflow name)
3. Add i18n Keys
{ "menu": { "runWorkflowDisabled": "Workflow contains unsupported nodes (highlighted red). Remove these to run the workflow." }, "breadcrumbsMenu": { "missingNodesWarning": "Workflow contains unsupported nodes (highlighted red)." } }🧪 Testing
Open workflow with missing nodes
Switch to workflow without missing nodes
Switch back to workflow with missing nodes
📸 Visual Changes
queuebutton.webm
🤖 Generated with Claude Code
┆Issue is synchronized with this Notion page by Unito