Add ComfyUI-TJ_NODE_STUDIO_ONE to custom-node-list#3022
Conversation
📝 WalkthroughWalkthroughAdds one ChangesCustom node registry update
🚥 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 5-9: The registry entry for the TJ node package is using a name
that does not match the upstream install instructions, which can mislead users
and break discovery. Verify the canonical package identifier with the upstream
repository owner, then update the entry’s title/id/reference and any related
docs together so the Manager name, clone target, and repo URL all use the same
identifier. Use the existing custom-node-list.json entry for
ComfyUI-TJ_NODE_STUDIO_ONE as the place to align these symbols.
🪄 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: e6bd45a6-9645-43d9-a086-7efac189ccae
📒 Files selected for processing (1)
custom-node-list.json
| "title": "ComfyUI-TJ_NODE_STUDIO_ONE", | ||
| "id": "ComfyUI-TJ_NODE_STUDIO_ONE", | ||
| "reference": "https://github.com/designloves2/ComfyUI-TJ_NODE_STUDIO_ONE", | ||
| "files": [ | ||
| "https://github.com/designloves2/ComfyUI-TJ_NODE_STUDIO_ONE" |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
Align the registry name with the upstream install docs.
The upstream README still tells users to search for ComfyUI-TJ_NODE_ONE in Manager and to clone designloves2/ComfyUI-TJ_NODE_ONE.git, but that repo currently 404s. Registering this as ComfyUI-TJ_NODE_STUDIO_ONE means the name game is out of tune and can send users on a dead-link chase. Please confirm the canonical package identifier upstream before merging, or update the registry entry/docs together. (github.com)
🤖 Prompt for 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.
In `@custom-node-list.json` around lines 5 - 9, The registry entry for the TJ node
package is using a name that does not match the upstream install instructions,
which can mislead users and break discovery. Verify the canonical package
identifier with the upstream repository owner, then update the entry’s
title/id/reference and any related docs together so the Manager name, clone
target, and repo URL all use the same identifier. Use the existing
custom-node-list.json entry for ComfyUI-TJ_NODE_STUDIO_ONE as the place to align
these symbols.
There was a problem hiding this comment.
@designloves2 Thanks for the update — sounds aligned and tidy now. Name of the game: fixed, no drift.
|
Thanks for the contribution! Before this can be merged, the following need to be addressed:
I'll re-evaluate once these are addressed. |
Hi, I would like to add my new custom node package ComfyUI-TJ_NODE_STUDIO_ONE to the ComfyUI Manager custom node list.
Repository:
https://github.com/designloves2/ComfyUI-TJ_NODE_STUDIO_ONE
This package provides one-node studio UIs for Z-Image Turbo, Flux.2 Klein, Qwen Image Edit 2511, and Krea 2 workflows, including T2I, I2I, inpaint, outpaint, edit, faceswap, angle, and upscale modes.
Only
custom-node-list.jsonis changed in this PR.Thank you!