Running "update all" bug; cannot find Manager due to directory case change

[kaptainkory]

Describe the bug
Running "update all" does not recognize Manager installation and produces the following error:
ComfyUI-Manager not found: /ComfyUI/custom_nodes/ComfyUI-Manager/cm-cli.py

To Reproduce
Install the latest ComfyUI and Manager WITHOUT the command line client, then run the update command with the client to update all.

Expected behavior
Running the following command should update ComfyUI and Manager:
comfy update all

Additional context
I believe the issue is simply that Manager now installs to "comfyui-manager" by default, rather than "ComfyUI -Manager". Notice the case change!

[dosubot]

Hi @kaptainkory! I'm Dosu and I’m helping the comfy-cli team.

The issue happens because comfy-cli looks for the Manager in a hardcoded, case-sensitive directory: custom_nodes/ComfyUI-Manager. If the Manager is installed as custom_nodes/comfyui-manager (all lowercase), comfy-cli won't find it and will throw the error you saw. This is confirmed by the code, which checks for the exact path and fails if it doesn't exist (source).

As a workaround, you can rename the custom_nodes/comfyui-manager directory to custom_nodes/ComfyUI-Manager (matching the expected case exactly). After renaming, the comfy update all command should work as intended.

There’s currently no logic in comfy-cli to handle both directory names or to match case-insensitively (source). This is a known pain point for other nodepacks as well.

To reply, just mention @dosu.

---

^{How did I do? Good | Irrelevant | Incorrect | Verbose | Hallucination | Report 🐛 | Other}