Skip to content

Add comfyui-ZoeyTool to custom node list#2984

Open
liangzoey wants to merge 1 commit into
Comfy-Org:mainfrom
liangzoey:main
Open

Add comfyui-ZoeyTool to custom node list#2984
liangzoey wants to merge 1 commit into
Comfy-Org:mainfrom
liangzoey:main

Conversation

@liangzoey

Copy link
Copy Markdown

Add @liangzoey/comfyui-ZoeyTool custom node to ComfyUI Manager.

  • GitHub: https://github.com/liangzoey/comfyui-ZoeyTool
  • 19 custom nodes for image editing, lighting control, video processing, prompt generation/translation, and more.
  • Properly structured with NODE_CLASS_MAPPINGS, WEB_DIRECTORY, and init.py exports.

@coderabbitai

coderabbitai Bot commented Jun 12, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: fe754163-e03c-40ba-abb2-bf89ce58739a

📥 Commits

Reviewing files that changed from the base of the PR and between 194bced and 53aa48d.

📒 Files selected for processing (1)
  • custom-node-list.json

📝 Walkthrough

Walkthrough

A new custom node entry for comfyui-ZoeyTool has been added to the registry with metadata including author, repository link, git-clone installation method, and description of image processing, video, masking, and prompt translation features.

Changes

Node Registry Update

Layer / File(s) Summary
ZoeyTool node entry
custom-node-list.json
Node registry expanded with comfyui-ZoeyTool entry, specifying author (liangzoey), repository reference, git-clone installation type, and feature summary of image editing, lighting control, video processing, prompt/translation, mask drawing, and batch capabilities—you might say this list just found its perfect tool!
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified 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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ltdrdata

Copy link
Copy Markdown
Member

Hi @liangzoey — thanks! Holding comfyui-ZoeyTool for a few fixes first:

  1. Path containment: several nodes take a free-form path (save_path, output_directory, input_directory, input/output_folder) and feed it to makedirs/save/open('w')/shutil.copy2/os.rename with no confinement — absolute or ../ paths write/rename/read anywhere on the host (the batch renamer's os.rename is especially destructive). These are reachable cross-origin via ComfyUI's /prompt API, so please add real containment (realpath + commonpath against the input/output dir) to every save/rename/crop/load node.
  2. requirements.txt: comfyui-extra-nodes>=1.4.0 doesn't exist on PyPI — installs will fail. Please drop or fix it.
  3. Please branch off the current custom-node-list.json and change only your entry — no full-file reformat (it reverts unrelated entries and can't be merged).

Happy to re-review once these are in. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants