Skip to content

Conversation

@christian-byrne
Copy link
Contributor

@christian-byrne christian-byrne commented May 30, 2025

Filter models in workflow metadata to only include those actually selected in node widgets. Prevents false positives in the missing models dialog when nodes contain outdated metadata.

Fixes #3664

┆Issue is synchronized with this Notion page by Unito

Only include model metadata for models actually selected in node widgets. This prevents false positives in the missing models dialog when nodes contain outdated metadata from previously selected models.

Fixes #3664
@christian-byrne christian-byrne requested review from a team as code owners May 30, 2025 03:23
Copy link
Contributor

@webfiltered webfiltered left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is really just one suggested change.

"Simplify code using idiomatic undefined"

christian-byrne and others added 6 commits May 31, 2025 13:33
Add non-null assertions for test cases expecting arrays and correct test expectations for cases that should return undefined.
@christian-byrne
Copy link
Contributor Author

christian-byrne commented May 31, 2025

Changes:

  • Simplify code using idiomatic undefined
  • Updated unit test expectations to expect undefined rather than empty array to match above change

Copy link
Contributor

@webfiltered webfiltered left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will merge into 1.22

@webfiltered webfiltered merged commit b81c2f7 into main Jun 1, 2025
10 checks passed
@webfiltered webfiltered deleted the fix/model-metadata-persistence branch June 1, 2025 02:43
lordTyrion pushed a commit to playbook3d/ComfyUI_frontend that referenced this pull request Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Model metadata stays after changing workflow models

3 participants