Skip to content

Add ComfyUI-NukeLink#3063

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

Add ComfyUI-NukeLink#3063
drberkowitz wants to merge 1 commit into
Comfy-Org:mainfrom
drberkowitz:main

Conversation

@drberkowitz

@drberkowitz drberkowitz commented Jul 7, 2026

Copy link
Copy Markdown

Summary

Adds ComfyUI-NukeLink to the ComfyUI Manager custom node list. ComfyUI-NukeLink bridges Foundry's Nuke and ComfyUI on the same machine.

Extension Information

Features

  • Send one or more Nuke Read nodes to ComfyUI with knob values intact: file path, colorspace, frame range, and missing frames mode
  • Automatic Path Builder node placement, pre-populated with output location, and shot name
  • Shot name and output location derived from pipeline environment variables or script filename
  • Send ComfyUI outputs back to Nuke as Read nodes via right-click menu on any NukeLink - Write node
  • Sequence and still image preview with right-click menu playback controls on Read and Write nodes
  • Missing frames handling: black, hold, nearest, and error modes
  • Colorspace support: raw, sRGB, linear, and ACEScg
  • Multiple simultaneous Nuke instances
  • Standard, portable, and desktop ComfyUI installs supported

@coderabbitai

coderabbitai Bot commented Jul 7, 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 Plus

Run ID: 170daa59-210b-4b65-bc6e-6a32cbeeedf8

📥 Commits

Reviewing files that changed from the base of the PR and between e11e903 and be053dd.

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

📝 Walkthrough

Walkthrough

A single new JSON entry for the ComfyUI-NukeLink custom node was added to custom-node-list.json, including metadata fields and a git-clone install source, describing a bridge between Nuke and ComfyUI.

Changes

Node Registry Update

Layer / File(s) Summary
Add ComfyUI-NukeLink entry
custom-node-list.json
A new node object is appended with author, title, id, reference, files, install_type, and description fields describing a Nuke-to-ComfyUI bridge node.

Possibly related PRs

No links, just Nuke — a node to unhook, plugging ComfyUI in with barely a look. 🔧✨

🚥 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.
✨ 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.

@drberkowitz

Copy link
Copy Markdown
Author

Hi @ltdrdata, first time contributor here, just checking in on this PR. Wanted to make sure everything looks right on my end. Seems to have passed CodeRabbit with no issues flagged. More than happy to make any changes if something's needed. Thank you!

@ltdrdata

Copy link
Copy Markdown
Member

Hi @drberkowitz — thanks for the PR! Holding this for a couple of security fixes first:

  1. NukeLinkWrite: file_path isn't confined to the output dir — absolute/.. paths write anywhere. Please add realpath+commonpath containment.
  2. /nukelink/openinfolder: this unauthenticated route passes the caller's path straight to open/explorer. On macOS open will launch a .app — an app-launch capability a "reveal in folder" endpoint shouldn't have. Best fix: only ever pass a folder to open (and reject .app bundles, which are technically directories), so the endpoint can only open a directory and never launch anything. Same containment applies to the other path-taking routes.

(Note: the CodeRabbit review above only covers this PR's custom-node-list.json diff, not your node repo — the above is from a separate review of ComfyUI-NukeLink.)

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