Register ComfyUI Dynamic Text-to-Combo Generator custom node#3029
Register ComfyUI Dynamic Text-to-Combo Generator custom node#3029gh055 wants to merge 2 commits into
Conversation
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughA single new entry is added to the ChangesCustom Node Registration
🚥 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@custom-node-list.json`:
- Around line 3-9: The custom node registration entry is missing required
custom_nodes contract fields, so it won’t be accepted or discovered. Update the
object in custom-node-list.json to include both files and install_type, and make
sure the values match the intended install flow so json-checker.py passes and
scanner.py can detect the node via the expected files[0] path when install_type
is git-clone.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 9ca3beba-8e67-4e6e-9a26-105bc32cf0db
📒 Files selected for processing (1)
custom-node-list.json
|
Thanks — I've applied the updated Closing this PR — the nodepack is now registered with the updated metadata. |
…mic-Combo — adopt PR title/desc - title: Dynamic Text-to-Combo Generator → ComfyUI Dynamic Text-to-Combo Generator - description: PR wording (Set/Get node setups) - author/reference/files/install_type unchanged - root + node_db/new sync
No description provided.