Add Model Fetcher (comfyui-modelfetcher) - #3189
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe custom-node registry adds the “Model Fetcher” entry. The entry identifies its repository, uses ChangesModel Fetcher registration
Suggested reviewers: Merge Risk: ⚪ Minimal · up to This PR adds a localized registry entry, and no actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
This pack exposes network-reachable routes with no access control: one writes files from a caller-supplied job, and a token route writes/clears a credentials file. A
Please make these routes local-only, restrict the fetch to a host allow-list, and confine the write path. I'll re-evaluate once constrained. |
Adds Model Fetcher, already published on the Comfy Registry: https://registry.comfy.org/nodes/comfyui-modelfetcher
models/subfolder (extra_model_paths aware, resume, HuggingFace token kept to HF hosts only), flags likely duplicates, and relinks loader nodes to copies already on disk.requests,aiohttp); no build step;comfy node validatepasses clean.🤖 Generated with Claude Code