mirror of
https://github.com/Comfy-Org/desktop.git
synced 2026-11-04 23:47:43 +00:00
[GH-ISSUE #1659] [Bug]: Model auto-download creates nested Users folder inside models directory (macOS) #492
Labels
No labels
area: desktop-app
Bounty
bug
bug-cop:answered
bug-cop:response-received
component: installer
component: preload
custom node
dependencies
Design
documentation
duplicate
enhancement
expected
frontend
good first issue
installation
Low-Priority
performance
Planned
platform: cross-platform
platform: windows
Potential Bug
Potential Bug
pull-request
question
refactor
Release
stale
User Support
V1
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
Comfy-Org/desktop#492
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @mshkhusn on GitHub (Mar 15, 2026).
Original GitHub issue: https://github.com/Comfy-Org/desktop/issues/1659
App Version
v0.8.21
Expected Behavior
When using template workflows that trigger automatic model downloads, models should be saved to the correct paths:
~/Documents/ComfyUI/models/unet/flux-2-klein-4b.safetensors
~/Documents/ComfyUI/models/vae/flux2-vae.safetensors
~/Documents/ComfyUI/models/text_encoders/qwen_3_4b.safetensors
Actual Behavior
Models are downloaded to a deeply nested path that duplicates the full absolute path inside the models directory:
~/Documents/ComfyUI/models/Users/[username]/Documents/ComfyUI/models/unet/flux-2-klein-4b.safetensors
~/Documents/ComfyUI/models/Users/[username]/Documents/ComfyUI/models/vae/flux2-vae.safetensors
~/Documents/ComfyUI/models/Users/[username]/Documents/ComfyUI/models/text_encoders/qwen_3_4b.safetensors
A "Users/" folder is created inside the models/ directory with the full user path nested within it. This happens consistently with every template model download.
Steps to Reproduce
extra_models_config.yaml content:
ComfyUI extra_model_paths.yaml for darwin
comfyui_desktop:
is_default: "true"
custom_nodes: custom_nodes/
download_model_base: models
base_path: /Users/[username]/Documents/ComfyUI
desktop_extensions:
custom_nodes: /Applications/ComfyUI.app/Contents/Resources/ComfyUI/custom_nodes
It appears that download_model_base (relative path "models") is being combined with base_path in a way that creates the full absolute path as a subdirectory, rather than resolving it correctly.
Debug Logs
Browser Logs
No response
Settings JSON
No response
Other
Workaround: manually move files after each download:
cd ~/Documents/ComfyUI/models
mv Users/[username]/Documents/ComfyUI/models/unet/* unet/
mv Users/[username]/Documents/ComfyUI/models/vae/* vae/
rm -rf Users
Tested with multiple templates (Flux.2 Klein 4B, Z-Image-Turbo) — same behavior every time.
┆Issue is synchronized with this Notion page by Unito
@dosubot[bot] commented on GitHub (Mar 15, 2026):
Hi @mshkhusn! I'm Dosu and I’m helping the desktop team.
This is a known bug with active fixes in progress.
The root cause is in
DownloadManager.ts— thegetLocalSavePath()method unconditionally joins paths without checking if they're already absolute:This was introduced by a regression from the missing-model dialog rewrite in ComfyUI_frontend, which now sends absolute paths instead of relative ones. The DownloadManager still assumes relative paths, causing the nested structure you're seeing.
Two PRs from this week address this:
Your workaround of manually moving files is correct for now. This should be fixed once one of these PRs is merged.
To reply, just mention @dosu.
Docs are dead. Just use Dosu.
@mshkhusn commented on GitHub (Mar 15, 2026):
@dosu Thank you for the quick response. Good to know this is already being tracked and fixed. Looking forward to PR #1657 or #1655 being merged. Will continue using the manual move workaround until then.
@dosubot[bot] commented on GitHub (Mar 15, 2026):
You're welcome! Your detailed bug report with the exact paths and config file was very helpful for confirming this matches the known issue. Feel free to follow those PRs for updates on when the fix lands. 👍
To reply, just mention @dosu.
Share context across your team and agents. Try Dosu.
@benceruleanlu commented on GitHub (Mar 16, 2026):
@mshkhusn #1657 has been released to users, are you still seeing this problem?
@mshkhusn commented on GitHub (Mar 17, 2026):
Thanks — I tested it on the latest Desktop build and it looks fixed on my side. I’m no longer seeing the nested
models/Users/...path issue.