Skip to content

Add Subgraphs #3905

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 48 commits into
base: main
Choose a base branch
from
Draft

Add Subgraphs #3905

wants to merge 48 commits into from

Conversation

webfiltered
Copy link
Contributor

@webfiltered webfiltered commented May 16, 2025 •
edited by sync-by-unito bot
Loading

┆Issue is synchronized with this Notion page by Unito

@webfiltered webfiltered mentioned this pull request May 16, 2025
Draft
@socket-security Socket Security
Copy link

socket-security bot commented May 16, 2025 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Updated@​comfyorg/​litegraph@​0.15.11 ⏵ 0.16.0-sub.577 -2310010098 -2100

View full report

webfiltered added 27 commits May 21, 2025 12:55
@webfiltered
[Nope] Add Zod recursive schema requirements
14de9cf
@webfiltered
[Nope] Zod schema recursive II
681d6d8
@webfiltered
[Refactor] Generic TS type dedupe
369bd71
@webfiltered
Add i18n for nothing selected notification
4aef838
@webfiltered
Add subgraph workflow fields to change tracker
17deca3
@webfiltered
[TS] Remove unintended type assertion
33f4649
@webfiltered
Fix corruption when app.graph is changed
1b69216
@webfiltered
[PARTIAL] Add Subgraph
58cc1f3
@webfiltered
Fix DOM widgets after rebase
e467c40
@webfiltered
nit - prevent duplicate subgraph registrations
2818916
@webfiltered
Fix nested subgraph breadcrumbs
60b7661
@webfiltered
Remove unnecessary copy of litegraph objects
c96327e
@webfiltered
Prune DOM widgets outside of current subgraph
669dbdb
@webfiltered
[TS] Remove unnecessary assertion
2f35039
@webfiltered
Fix desync of litegraph type and nodedefs
65cec46
@webfiltered
nit
bcfc944
@webfiltered
[TS] Replace ts-ignore w/error
5f9388c
@webfiltered
Track subgraph open history for breadcrumbs
e06efbd
@webfiltered
nit
6ba5116
@webfiltered
Fix DOM widgets disappear
7448c57
@webfiltered
Add convert to subgraph command
b2722ac
@webfiltered
[Debug] Include more items in graph diff output
e059fe2
@webfiltered
Clear DOM widgets instead of trying to manage refs
8c8f320
@webfiltered
Keep subgraph nav state when swapping workflows
46d964f
@webfiltered
[Test] Update expectations
cf8a914
@webfiltered
Add simpler interface for active DOM widgets
60cf8ff
@webfiltered
Use subgraph npm
51c0173
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Yorha4D Yorha4D Awaiting requested review from Yorha4D Yorha4D will be requested when the pull request is marked ready for review Yorha4D is a code owner

@KarryCharon KarryCharon Awaiting requested review from KarryCharon KarryCharon will be requested when the pull request is marked ready for review KarryCharon is a code owner

@shinshin86 shinshin86 Awaiting requested review from shinshin86 shinshin86 will be requested when the pull request is marked ready for review shinshin86 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@webfiltered