Skip to content

Comments

feat: Support ComfyUI-Manager for pip version#7555

Merged
comfyanonymous merged 196 commits intoComfy-Org:masterfrom
ltdrdata:dr-support-pip-cm
Dec 2, 2025
Merged

feat: Support ComfyUI-Manager for pip version#7555
comfyanonymous merged 196 commits intoComfy-Org:masterfrom
ltdrdata:dr-support-pip-cm

Conversation

@ltdrdata
Copy link
Member

@ltdrdata ltdrdata commented Apr 9, 2025 •
edited
Loading

Manual setup required, yet.

Comfy-Org/ComfyUI-Manager#1737

@christian-byrne
Copy link
Contributor

christian-byrne commented Apr 10, 2025 •
edited
Loading

Can we also ensure the comfyui manager custom node is not imported, except if --disable-manager?

And add to requirements.txt?

@christian-byrne christian-byrne mentioned this pull request Apr 10, 2025
Closed
9 tasks
@ltdrdata ltdrdata mentioned this pull request Apr 10, 2025
Closed
ltdrdata added 2 commits April 12, 2025 19:08
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
311f64a
@ltdrdata
add comfyui_manager to requirements.txt
cc975e5 
It's still in the development stage, so the version is not pinned yet.
@ltdrdata ltdrdata mentioned this pull request Apr 12, 2025
Draft
19 tasks
ltdrdata added 2 commits April 12, 2025 21:53
@ltdrdata
fixed: Ensure that comfyui_manager's prestartup always runs, even w…
418eaed 
…hen `--disable-all-custom-nodes` is used.

feat: Disable specific custom nodes according to the policy of `comfyui_manager`.
@ltdrdata
add --enable-manager-legacy-ui
94f61c6
@christian-byrne
Copy link
Contributor

christian-byrne commented Apr 15, 2025

I think if --disable-manager is set, we should not import the package at all.

@ltdrdata
Copy link
Member Author

ltdrdata commented Apr 15, 2025

My intention was that --disable-manager should only disable the Manager web service, while keeping other auxiliary features intact.
This includes components like the security scanner, and also allows the Manager to be used as a library.

@ltdrdata
Copy link
Member Author

ltdrdata commented Apr 28, 2025 •
edited
Loading

--disable-manager-ui has been added.
And --disable-manager will prevent importing ComfyUI-Manager.

Now, it is possible to distinguish between disabling ComfyUI-Manager entirely and disabling only the UI of ComfyUI-Manager.

@christian-byrne If the endpoint is blocked using --disable-manager-ui, is it also possible to block ui from the frontend?

ltdrdata and others added 24 commits October 24, 2025 12:17
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
0432bcc
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
3c4b429
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
fe26f30
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
c07908a
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
de357a0
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
b88c66b
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
ad4b959
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
8f492d8
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
16359ab
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
d8b821e
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
671a769
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
32bd557
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
5cb77fb
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
2d4dd39
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
b15ef99
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
a58c4fb
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
7b9ad52
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
6206a6d
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
1abf69e
@Kosinkadink
Merge branch 'master' into dr-support-pip-cm
aa878cc
@Kosinkadink
Move manager requirement into its own file
79fb964
@ltdrdata
Merge branch 'master' into dr-support-pip-cm
a5e0674
@ltdrdata
updated: manager_requirements.txt
d69c8b3
@ltdrdata
updated: manager_requirements.txt
4061eaa
@benceruleanlu benceruleanlu mentioned this pull request Nov 26, 2025
Merged
@Kosinkadink Kosinkadink mentioned this pull request Nov 27, 2025
Open
@comfyanonymous comfyanonymous merged commit b4a20ac into Comfy-Org:master Dec 2, 2025
10 checks passed
@ltdrdata ltdrdata deleted the dr-support-pip-cm branch December 26, 2025 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@christian-byrne christian-byrne christian-byrne left review comments

@yoland68 yoland68 Awaiting requested review from yoland68

@robinjhuang robinjhuang Awaiting requested review from robinjhuang

@webfiltered webfiltered Awaiting requested review from webfiltered

@pythongosssss pythongosssss Awaiting requested review from pythongosssss

@Kosinkadink Kosinkadink Awaiting requested review from Kosinkadink Kosinkadink is a code owner

@guill guill Awaiting requested review from guill guill is a code owner

@comfyanonymous comfyanonymous Awaiting requested review from comfyanonymous comfyanonymous is a code owner

Assignees

No one assigned

Labels

Core Core team dependency Core-Important

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@ltdrdata @christian-byrne @Kosinkadink @yoland68 @comfyanonymous @viva-jinyi