Skip to content

chore: Update display names and categories (CORE-151)#13693

Merged
comfyanonymous merged 3 commits intomasterfrom
alexis/nodes-naming-cleanup
May 5, 2026
Merged

chore: Update display names and categories (CORE-151)#13693
comfyanonymous merged 3 commits intomasterfrom
alexis/nodes-naming-cleanup

Conversation

@alexisrolland
Copy link
Copy Markdown
Member

@alexisrolland alexisrolland commented May 4, 2026

First pass of clean-up on native nodes display names and categories.

Loosely following the dominant pattern

  • Node Id CamelCase → spaces
  • Verb first
  • Standardized "DEPRECATED" label in nodes titles
  • Focus on image category

API Node PR Checklist

Scope

  • Is API Node Change

Pricing & Billing

  • Need pricing update
  • No pricing update

If Need pricing update:

  • Metronome rate cards updated
  • Auto‑billing tests updated and passing

QA

  • QA done
  • QA not required

Comms

  • Informed Kosinkadink

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 4, 2026

📝 Walkthrough

Walkthrough

This pull request updates node UI metadata across multiple files. Changes include renaming node display labels for consistency (e.g., "Image Crop (Deprecated)" to "Crop Image (DEPRECATED)", "Image Blend" to "Blend Images"), reorganizing node categories for better organization (video nodes moved from "image/video" to "video", color nodes from "image/batch" to "image/color"), and adding search aliases for certain nodes. Node execution logic, inputs, outputs, and signatures remain unchanged.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 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.
Title check ✅ Passed The title accurately summarizes the primary change: standardizing display names and categories across native nodes.
Description check ✅ Passed The PR description clearly outlines the intent to clean up node display names and categories following consistent patterns (CamelCase to spaces, verb-first naming, standardized DEPRECATED labels).

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


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.

@alexisrolland alexisrolland self-assigned this May 4, 2026
@alexisrolland alexisrolland changed the title chore: Update display names and categories chore: Update display names and categories (CORE-151) May 4, 2026
@comfyanonymous comfyanonymous merged commit 15a4494 into master May 5, 2026
17 checks passed
@comfyanonymous comfyanonymous deleted the alexis/nodes-naming-cleanup branch May 5, 2026 00:37
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