Skip to content

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
Copy Markdown
Member

@ltdrdata ltdrdata commented Apr 9, 2025

Manual setup required, yet.

Comfy-Org/ComfyUI-Manager#1737

@christian-byrne
Copy link
Copy Markdown
Contributor

christian-byrne commented Apr 10, 2025

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

And add to requirements.txt?

It's still in the development stage, so the version is not pinned yet.
…hen `--disable-all-custom-nodes` is used.

feat: Disable specific custom nodes according to the policy of `comfyui_manager`.
@christian-byrne
Copy link
Copy Markdown
Contributor

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

@ltdrdata
Copy link
Copy Markdown
Member Author

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
Copy Markdown
Member Author

ltdrdata commented Apr 28, 2025

--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?

@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
hanzo-dev pushed a commit to hanzoai/studio-frontend that referenced this pull request Mar 31, 2026
…ger (#5635)

## Summary
- Updated frontend to align with backend changes in ComfyUI core PR
#7555
- Changed manager startup argument from `--disable-manager` (opt-out) to
`--enable-manager` (opt-in)
- Manager is now disabled by default unless explicitly enabled

## Changes
- Modified `useManagerState.ts` to check for `--enable-manager` flag
presence
- Inverted logic: manager is disabled when the flag is NOT present
- Updated all related tests to reflect the new opt-in behavior
- Fixed edge case where `systemStats` is null

## Related
- Backend PR: Comfy-Org/ComfyUI#7555

## Test Plan
- [x] All unit tests pass
- [x] Verified manager state logic with different flag combinations
- [x] TypeScript type checking passes
- [x] Linting passes

🤖 Generated with [Claude Code](https://claude.ai/code)

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-5635-refactor-Change-manager-flag-from-disable-manager-to-enable-manager-2726d73d36508153a88bd9f152132b2a)
by [Unito](https://www.unito.io)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Core Core team dependency Core-Important

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants