Fix subgraph conversion of primitives

[AustinMroz]

┆Issue is synchronized with this Notion page by Unito

[github-actions]

🎭 Playwright Test Results

⚠️ Tests passed with flaky tests

⏰ Completed at: 11/08/2025, 08:37:34 PM UTC

📈 Summary

• Total Tests: 500
• Passed: 468 ✅
• Failed: 0
• Flaky: 2 ⚠️
• Skipped: 30 ⏭️

📊 Test Reports by Browser

• ✅ chromium: View Report • ✅ 459 / ❌ 0 / ⚠️ 2 / ⏭️ 30
• ✅ chromium-2x: View Report • ✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0
• ✅ chromium-0.5x: View Report • ✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0
• ✅ mobile-chrome: View Report • ✅ 6 / ❌ 0 / ⚠️ 0 / ⏭️ 0

---

🎉 Click on the links above to view detailed test results for each browser configuration.

[github-actions]

🎨 Storybook Build Status

✅ Build completed successfully!

⏰ Completed at: 11/08/2025, 08:23:50 PM UTC

🔗 Links

• 📊 View Workflow Run
• 🎨 View Storybook

---

🎉 Your Storybook is ready for review!

[github-actions]

Bundle Size Report

Summary

• Raw size: 12.2 MB baseline 12.2 MB — 🔴 +158 B
• Gzip: 2.49 MB baseline 2.49 MB — 🔴 +45 B
• Brotli: 1.96 MB baseline 1.96 MB — 🔴 +67 B
• Bundles: 58 current • 58 baseline • 13 added / 13 removed

Category Glance
App Entry Points 🔴 +158 B (3.25 MB) · Vendor & Third-Party ⚪ 0 B (5.32 MB) · Other ⚪ 0 B (2.55 MB) · Graph Workspace ⚪ 0 B (796 kB) · Panels & Settings ⚪ 0 B (293 kB) · UI Components ⚪ 0 B (12.6 kB) · + 3 more

Per-category breakdown

App Entry Points — 3.25 MB (baseline 3.25 MB) • 🔴 +158 B

Main entry bundles and manifests

File	Before	After	Δ Raw	Δ Gzip	Δ Brotli
assets/index-D8KGDlDL.js (new)	—	2.87 MB	🔴 +2.87 MB	🔴 +594 kB	🔴 +450 kB
assets/index-rRRe0ARS.js (removed)	2.87 MB	—	🟢 -2.87 MB	🟢 -594 kB	🟢 -450 kB
assets/index-CTsEB1_L.js (new)	—	382 kB	🔴 +382 kB	🔴 +76.6 kB	🔴 +62.1 kB
assets/index-B1bk73P0.js (removed)	382 kB	—	🟢 -382 kB	🟢 -76.6 kB	🟢 -62.1 kB
assets/index-BVl_zYuX.js (removed)	1.75 kB	—	🟢 -1.75 kB	🟢 -574 B	🟢 -485 B
assets/index-P5Ym2FmJ.js (new)	—	1.75 kB	🔴 +1.75 kB	🔴 +577 B	🔴 +487 B

Status: 3 added / 3 removed

Graph Workspace — 796 kB (baseline 796 kB) • ⚪ 0 B

Graph editor runtime, canvas, workflow orchestration

File	Before	After	Δ Raw	Δ Gzip	Δ Brotli
assets/GraphView-BB9ADjl_.js (new)	—	796 kB	🔴 +796 kB	🔴 +156 kB	🔴 +120 kB
assets/GraphView-BhaGVmV5.js (removed)	796 kB	—	🟢 -796 kB	🟢 -156 kB	🟢 -120 kB

Status: 1 added / 1 removed

Views & Navigation — 8.18 kB (baseline 8.18 kB) • ⚪ 0 B

Top-level views, pages, and routed surfaces

File	Before	After	Δ Raw	Δ Gzip	Δ Brotli
assets/UserSelectView-DEX8yW6g.js (new)	—	8.18 kB	🔴 +8.18 kB	🔴 +2.48 kB	🔴 +2.17 kB
assets/UserSelectView-DHJ2BZoC.js (removed)	8.18 kB	—	🟢 -8.18 kB	🟢 -2.48 kB	🟢 -2.19 kB

Status: 1 added / 1 removed

Panels & Settings — 293 kB (baseline 293 kB) • ⚪ 0 B

Configuration panels, inspectors, and settings screens

File	Before	After	Δ Raw	Δ Gzip	Δ Brotli
assets/CreditsPanel-CBHISViC.js (removed)	23 kB	—	🟢 -23 kB	🟢 -5.48 kB	🟢 -4.79 kB
assets/CreditsPanel-DVgeCRL3.js (new)	—	23 kB	🔴 +23 kB	🔴 +5.48 kB	🔴 +4.79 kB
assets/KeybindingPanel-C-1z-DHQ.js (removed)	15.3 kB	—	🟢 -15.3 kB	🟢 -3.78 kB	🟢 -3.33 kB
assets/KeybindingPanel-CMiKOn2p.js (new)	—	15.3 kB	🔴 +15.3 kB	🔴 +3.78 kB	🔴 +3.32 kB
assets/ExtensionPanel-C_7yDEoI.js (new)	—	12.1 kB	🔴 +12.1 kB	🔴 +2.84 kB	🔴 +2.5 kB
assets/ExtensionPanel-DMeWn89G.js (removed)	12.1 kB	—	🟢 -12.1 kB	🟢 -2.84 kB	🟢 -2.5 kB
assets/AboutPanel-Cg6f2kro.js (new)	—	10.3 kB	🔴 +10.3 kB	🔴 +2.67 kB	🔴 +2.34 kB
assets/AboutPanel-CWOc3ek4.js (removed)	10.3 kB	—	🟢 -10.3 kB	🟢 -2.67 kB	🟢 -2.34 kB
assets/ServerConfigPanel-CODDFqPq.js (removed)	8.23 kB	—	🟢 -8.23 kB	🟢 -2.17 kB	🟢 -1.91 kB
assets/ServerConfigPanel-CTsjNmpe.js (new)	—	8.23 kB	🔴 +8.23 kB	🔴 +2.17 kB	🔴 +1.92 kB
assets/UserPanel-itsyr3SZ.js (removed)	7.94 kB	—	🟢 -7.94 kB	🟢 -2.07 kB	🟢 -1.81 kB
assets/UserPanel-XB6WwdT8.js (new)	—	7.94 kB	🔴 +7.94 kB	🔴 +2.07 kB	🔴 +1.81 kB
assets/settings-0O6mq5to.js	24.3 kB	24.3 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/settings-BYaBy7dC.js	20.4 kB	20.4 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/settings-C3vygQN4.js	25.7 kB	25.7 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/settings-CbKYXyH0.js	22.7 kB	22.7 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/settings-CCholIsI.js	25 kB	25 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/settings-DFX7vRkK.js	19.8 kB	19.8 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/settings-INJLrcmT.js	31.3 kB	31.3 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/settings-iR6BKRXe.js	23.7 kB	23.7 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/settings-YjQmudNE.js	23.5 kB	23.5 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B

Status: 6 added / 6 removed

UI Components — 12.6 kB (baseline 12.6 kB) • ⚪ 0 B

Reusable component library chunks

File	Before	After	Δ Raw	Δ Gzip	Δ Brotli
assets/ComfyQueueButton-B6shR7E-.js (removed)	11.3 kB	—	🟢 -11.3 kB	🟢 -2.83 kB	🟢 -2.5 kB
assets/ComfyQueueButton-DJ1jNo3b.js (new)	—	11.3 kB	🔴 +11.3 kB	🔴 +2.83 kB	🔴 +2.49 kB
assets/UserAvatar.vue_vue_type_script_setup_true_lang-D2s8tnS2.js	1.26 kB	1.26 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B

Status: 1 added / 1 removed

Data & Services — 10.4 kB (baseline 10.4 kB) • ⚪ 0 B

Stores, services, APIs, and repositories

File	Before	After	Δ Raw	Δ Gzip	Δ Brotli
assets/keybindingService-UlLk7r1W.js (removed)	7.6 kB	—	🟢 -7.6 kB	🟢 -1.84 kB	🟢 -1.59 kB
assets/keybindingService-VQmyg-cw.js (new)	—	7.6 kB	🔴 +7.6 kB	🔴 +1.84 kB	🔴 +1.59 kB
assets/serverConfigStore-BOo8M-7s.js	2.79 kB	2.79 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B

Status: 1 added / 1 removed

Utilities & Hooks — 1.07 kB (baseline 1.07 kB) • ⚪ 0 B

Helpers, composables, and utility bundles

File	Before	After	Δ Raw	Δ Gzip	Δ Brotli
assets/mathUtil-CTARWQ-l.js	1.07 kB	1.07 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B

Vendor & Third-Party — 5.32 MB (baseline 5.32 MB) • ⚪ 0 B

External libraries and shared vendor chunks

File	Before	After	Δ Raw	Δ Gzip	Δ Brotli
assets/vendor-other-Bfb5Ofrh.js	3.22 MB	3.22 MB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/vendor-primevue-PESgPnbc.js	517 B	517 B	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/vendor-three-JDoAqkQm.js	1.37 MB	1.37 MB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/vendor-tiptap-Dvg9y4X0.js	232 kB	232 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/vendor-vue-SdQKVoRx.js	92.6 kB	92.6 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/vendor-xterm-BZLod3g9.js	407 kB	407 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B

Other — 2.55 MB (baseline 2.55 MB) • ⚪ 0 B

Bundles that do not match a named category

File	Before	After	Δ Raw	Δ Gzip	Δ Brotli
assets/commands-B2KZRBmX.js	15.1 kB	15.1 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/commands-Bw-ckyga.js	13.9 kB	13.9 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/commands-C_NmM85I.js	13.8 kB	13.8 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/commands-CuozCW4W.js	14 kB	14 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/commands-DGfVUJCR.js	16.2 kB	16.2 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/commands-dOJNDogK.js	14.5 kB	14.5 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/commands-DwiE551e.js	14.7 kB	14.7 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/commands-Fw7mvqSy.js	13.1 kB	13.1 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/commands-FXnO1W4Q.js	13.2 kB	13.2 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/main-B28vuueQ.js	58.7 kB	58.7 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/main-B5DMAz5p.js	76.3 kB	76.3 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/main-B9oeRB3u.js	66.2 kB	66.2 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/main-BfxLm2q9.js	68.4 kB	68.4 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/main-Bqw7zi8p.js	92.9 kB	92.9 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/main-CxXcIMmb.js	80.3 kB	80.3 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/main-DbrkzQIr.js	70.6 kB	70.6 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/main-DwpdJL5U.js	67.5 kB	67.5 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/main-mC3TQk_1.js	59.4 kB	59.4 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/nodeDefs-3I1vPgv4.js	181 kB	181 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/nodeDefs-B2huPGKQ.js	190 kB	190 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/nodeDefs-BWugyUzd.js	215 kB	215 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/nodeDefs-bXqu6Stq.js	194 kB	194 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/nodeDefs-CtB2M3sY.js	229 kB	229 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/nodeDefs-D-rCrn-T.js	200 kB	200 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/nodeDefs-D38DSnl1.js	179 kB	179 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/nodeDefs-DAsU52ON.js	192 kB	192 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B
assets/nodeDefs-DnGONaA_.js	196 kB	196 kB	⚪ 0 B	⚪ 0 B	⚪ 0 B

[AustinMroz]

Connections after conversion were failing because the process of configuring a node would result in the config reference of input.widget being dropped.

I'm quite conflicted on the location of this change. The current fix for this is functional, but not great

[AustinMroz]

Some further digging:

• Widgets are normally setup through a graph.onConfigure. Adding a graph.onConfigure?.apply(subgraph, data) does result in proper setup of widget config, and seems to better handle some cases (conversion of a primitive along with it's dependent), but this isn't valid from a types perspective.

EDIT: Resolved by directly applying the node callbacks, which don't take args.