Skip to content

Add ComfyUI_PNG2JPEG node to custom node list - #3396

Closed
yourknife wants to merge 1 commit into
Comfy-Org:mainfrom
yourknife:add-png2jpeg-node
Closed

yourknife wants to merge 1 commit into
Comfy-Org:mainfrom
yourknife:add-png2jpeg-node

Conversation

@yourknife

@yourknife yourknife commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Add a minimal custom node that saves workflow images as JPEG.

  • Node: PNG2JPEG (category image/convert)
  • Features: selectable jpg/jpeg extension, quality 1-100, background fill (white/black) for transparent pixels, progressive JPEG option
  • No extra pip dependencies (uses ComfyUI-bundled torch/numpy/Pillow)
  • Tested on ComfyUI 0.42; includes unit tests (tests/test_core.py)

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: bb3b1d73-d95c-4ee7-be3d-8ac95baf1042
📥 Commits

Reviewing files that changed from the base of the PR and between a74929c and 6e85a3e.

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

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

📝 Walkthrough

The custom node catalog adds ComfyUI_PNG2JPEG, with its repository, installation method, and documented PNG-to-JPEG conversion options.

Changes: PNG2JPEG catalog entry

Layer / File(s) Summary
Add the custom node catalog entry
custom-node-list.json
The catalog now lists ComfyUI_PNG2JPEG, its repository and git-clone installation method, and its documented JPEG conversion options.

Suggested reviewers: ltdrdata

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 6e85a

The catalog addition is ready to merge with minimal risk.

Architecture risk: 🔵 Low · up to 6e85a

The change affects 1 system.

Changed systems: custom-node-list.json

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — custom-node-list.json (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in custom-node-list.json: Added a custom-node catalog entry for ComfyUI_PNG2JPEG, including its author, repository reference, git-clone installation method, and supported JPEG conversion options.
Pre-merge checks: Passed 2
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 a new PR
✨ Simplify code
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@coderabbitai
coderabbitai Bot requested a review from ltdrdata October 20, 2026 18:06
@yourknife

Copy link
Copy Markdown
Contributor Author

Superseded by the new ComfyUI_SaveJPEG registration.

@yourknife yourknife closed this Oct 21, 2026
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.

1 participant