Merged
Conversation
dosubot
bot
added
the
size:M
🎭 Playwright Tests: ✅ PassedResults: 507 passed, 0 failed, 0 flaky, 8 skipped (Total: 515) 📊 Browser Reports
|
🎨 Storybook Build Status✅ Build completed successfully! ⏰ Completed at: 01/29/2026, 02:45:34 AM UTC 🔗 Links🎉 Your Storybook is ready for review! |
Contributor
📝 WalkthroughWalkthroughThis pull request introduces support for marking nodes as "dev-only" and toggling their visibility based on development mode. Changes include schema updates to recognize the dev_only flag, UI badge rendering, filtering logic, and integration with existing dev mode settings. Changes
Sequence DiagramsequenceDiagram
participant NodeDef as Node Definition
participant Store as Node Def Store
participant Settings as Settings Store
participant Filter as Node Filter
participant UI as Search UI
NodeDef->>Store: Load node with dev_only flag
activate Store
Store->>Settings: Read Comfy.DevMode setting
activate Settings
Settings-->>Store: Return dev mode status
deactivate Settings
Store->>Filter: Register core.dev_only filter<br/>(showDevOnly || !nodeDef.dev_only)
activate Filter
Filter-->>Store: Filter applied
deactivate Filter
alt DevMode Enabled
Filter->>Store: Include dev-only node
Store->>UI: Render node with [DEV] badge
else DevMode Disabled
Filter->>Store: Hide dev-only node
Store->>UI: Node filtered out
end
deactivate Store
UI->>UI: Display filtered nodes
Suggested reviewers
✨ Finishing touches
Comment |
Support `dev_only` property to node definitions that hides nodes from search and menus unless dev mode is enabled. Dev-only nodes display a "DEV" badge when visible. This functionality is primarily intended to support unit-testing nodes on Comfy Cloud, but also has other uses.
coderabbitai
bot
requested review from
DrJKL,
KarryCharon,
Myestery and
shinshin86
Bundle Size ReportSummary
Category Glance Per-category breakdownApp Entry Points — 26 kB (baseline 26 kB) • ⚪ 0 BMain entry bundles and manifests
Status: 1 added / 1 removed Graph Workspace — 973 kB (baseline 973 kB) • 🔴 +207 BGraph editor runtime, canvas, workflow orchestration
Status: 1 added / 1 removed Views & Navigation — 80.7 kB (baseline 80.7 kB) • ⚪ 0 BTop-level views, pages, and routed surfaces
Status: 9 added / 9 removed Panels & Settings — 471 kB (baseline 471 kB) • 🟢 -8 BConfiguration panels, inspectors, and settings screens
Status: 12 added / 12 removed User & Accounts — 3.94 kB (baseline 3.94 kB) • ⚪ 0 BAuthentication, profile, and account management bundles
Status: 3 added / 3 removed Editors & Dialogs — 2.89 kB (baseline 2.89 kB) • ⚪ 0 BModals, dialogs, drawers, and in-app editors
Status: 2 added / 2 removed UI Components — 33.7 kB (baseline 33.7 kB) • ⚪ 0 BReusable component library chunks
Status: 5 added / 5 removed Data & Services — 2.7 MB (baseline 2.7 MB) • 🔴 +894 BStores, services, APIs, and repositories
Status: 8 added / 8 removed Utilities & Hooks — 25.3 kB (baseline 25.3 kB) • ⚪ 0 BHelpers, composables, and utility bundles
Status: 7 added / 7 removed Vendor & Third-Party — 10.7 MB (baseline 10.7 MB) • ⚪ 0 BExternal libraries and shared vendor chunks
Other — 7.05 MB (baseline 7.05 MB) • 🟢 -178 BBundles that do not match a named category
Status: 36 added / 36 removed |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@src/stores/nodeDefStore.ts`:
- Around line 176-180: The nodeLifeCycleBadgeText getter currently returns a
hardcoded '[DEV]' string; replace that raw string with a localized badge by
importing/using the vue-i18n key g.devOnly and composing the badge (e.g.,
`[\${t('g.devOnly')}]`) inside the nodeLifeCycleBadgeText getter so all
user-facing text uses the i18n entry instead of a literal '[DEV]'.
- Around line 429-435: Replace the hard-coded user-facing strings in the
registerNodeDefFilter call with vue-i18n keys: update the name and description
fields in the filter with i18n lookups (e.g., t('nodes.filters.dev_only.name')
and t('nodes.filters.dev_only.description')) in the same block that defines id:
'core.dev_only' and keep the predicate using showDevOnly.value ||
!nodeDef.dev_only unchanged; then add matching entries under the same keys in
src/locales/en/main.json (nodes.filters.dev_only.name and
nodes.filters.dev_only.description) with the English text previously used.
DrJKL
reviewed
Jan 28, 2026
DrJKL
reviewed
Jan 28, 2026
DrJKL
reviewed
Jan 28, 2026
DrJKL
approved these changes
Jan 29, 2026
guill
added
the
preview-cpu
christian-byrne
pushed a commit
that referenced
this pull request
Jan 30, 2026
## Summary Support `dev_only` property to node definitions that hides nodes from search and menus unless dev mode is enabled. Dev-only nodes display a "DEV" badge when visible. This functionality is primarily intended to support unit-testing nodes on Comfy Cloud, but also has other uses. ## Changes - **What**: Nodes flagged as dev_only in the node schema will only appear in search and menus if Dev Mode is on. ## Screenshots (if applicable) With Dev Mode off: <img width="2189" height="1003" alt="image" src="https://github.com/user-attachments/assets/a08e1fd7-dca9-4ce1-9964-5f4f3b7b95ac" /> With Dev Mode on: <img width="2201" height="1066" alt="image" src="https://github.com/user-attachments/assets/7fe6cd1f-f774-4f48-b604-a528e286b584" /> ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-8359-feat-support-dev-only-nodes-2f66d73d36508102839ee7cd66a26129) by [Unito](https://www.unito.io)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Support
dev_onlyproperty to node definitions that hides nodes from search and menus unless dev mode is enabled. Dev-only nodes display a "DEV" badge when visible.This functionality is primarily intended to support unit-testing nodes on Comfy Cloud, but also has other uses.
Changes
Screenshots (if applicable)
With Dev Mode off:
With Dev Mode on:
┆Issue is synchronized with this Notion page by Unito