Skip to content

feat: Support ComfyUI-Manager for pip version #7555

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
Merged
comfyanonymous merged 196 commits into from
Dec 2, 2025
Merged

feat: Support ComfyUI-Manager for pip version #7555

comfyanonymous merged 196 commits into from
Dec 2, 2025

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