Skip to content

[backport cloud/1.41] fix: load API format workflows with missing node types#10037

Merged
christian-byrne merged 1 commit intocloud/1.41from
backport-9694-to-cloud-1.41
Mar 16, 2026
Merged

[backport cloud/1.41] fix: load API format workflows with missing node types#10037
christian-byrne merged 1 commit intocloud/1.41from
backport-9694-to-cloud-1.41

Conversation

@comfy-pr-bot
Copy link
Member

@comfy-pr-bot comfy-pr-bot commented Mar 16, 2026

Backport of #9694 to cloud/1.41

Automatically created by backport workflow.

┆Issue is synchronized with this Notion page by Unito

## Summary

`loadApiJson` early-returns when missing node types are detected,
preventing the entire API-format workflow from loading onto the canvas.

## Changes

- **What**: Remove early `return` in `loadApiJson` so missing nodes are
skipped while the rest of the workflow loads normally, consistent with
how `loadGraphData` handles missing nodes in standard workflow format.

## Review Focus

The existing code already handles missing nodes gracefully:
- `LiteGraph.createNode()` returns `null` for unregistered types
- `if (!node) continue` skips missing nodes during graph construction
- `if (!fromNode) continue` skips connections to missing nodes
- `if (!node) return` skips input processing for missing nodes

The early `return` was unnecessarily preventing the entire load. The
warning modal is still shown via `showMissingNodesError`.

## Test workflow & screen recording
[04wan2.2smoothmix图生视频
(3).json](https://github.com/user-attachments/files/25858354/04wan2.2smoothmix.3.json)

[screen-capture.webm](https://github.com/user-attachments/assets/9c396f80-fff1-4d17-882c-35ada86542c1)
@comfy-pr-bot comfy-pr-bot requested a review from a team as a code owner March 16, 2026 12:49
@comfy-pr-bot comfy-pr-bot added the backport Backporting a PR onto a release candidate label Mar 16, 2026
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Mar 16, 2026
@github-actions
Copy link

github-actions bot commented Mar 16, 2026

🎨 Storybook: ✅ Built — View Storybook

Details

⏰ Completed at: 03/16/2026, 12:51:05 PM UTC

Links

@github-actions
Copy link

github-actions bot commented Mar 16, 2026

🎭 Playwright: ✅ 561 passed, 0 failed · 2 flaky

📊 Browser Reports
  • chromium: View Report (✅ 548 / ❌ 0 / ⚠️ 2 / ⏭️ 10)
  • chromium-2x: View Report (✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0)
  • chromium-0.5x: View Report (✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0)
  • mobile-chrome: View Report (✅ 10 / ❌ 0 / ⚠️ 0 / ⏭️ 0)

@christian-byrne christian-byrne merged commit 2d93372 into cloud/1.41 Mar 16, 2026
37 checks passed
@christian-byrne christian-byrne deleted the backport-9694-to-cloud-1.41 branch March 16, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Backporting a PR onto a release candidate size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants