Skip to content

[Partner Nodes] feat(Recraft): add V4 Styles - #15903

Merged
bigcat88 merged 1 commit into
masterfrom
feat/partner-nodes/recraft-v4-styles
Aug 26, 2026
Merged

[Partner Nodes] feat(Recraft): add V4 Styles#15903
bigcat88 merged 1 commit into
masterfrom
feat/partner-nodes/recraft-v4-styles

Conversation

@bigcat88

@bigcat88 bigcat88 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

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

Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 944a6cc6-8035-46d1-9dba-9bad81983c74

📥 Commits

Reviewing files that changed from the base of the PR and between b133e48 and 53d8fa4.

⛔ Files ignored due to path filters (1)
  • comfy_api_nodes/apis/recraft.py is excluded by !comfy_api_nodes/apis/**
📒 Files selected for processing (1)
  • comfy_api_nodes/nodes_recraft.py

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

📜 Recent review details
⏰ Context from checks skipped due to timeout. (9)
  • GitHub Check: test (macos-latest)
  • GitHub Check: test (windows-2022)
  • GitHub Check: test (ubuntu-latest)
  • GitHub Check: test
  • GitHub Check: Run Pylint
  • GitHub Check: test (ubuntu-latest)
  • GitHub Check: test (windows-latest)
  • GitHub Check: test (macos-latest)
  • GitHub Check: Run Pylint
🧰 Additional context used
📓 Path-based instructions (6)
Third-party API integration nodes. Focus on:

⚙️ CodeRabbit configuration file

Files:

  • comfy_api_nodes/nodes_recraft.py
IMPORTANT: Only comment on issues directly introduced by this PR's code changes.

⚙️ CodeRabbit configuration file

Files:

  • comfy_api_nodes/nodes_recraft.py
Treat legacy combo, `io.Combo`, and `io.DynamicCombo` values affecting filesystem access as untrusted; revalidate them at load/save boundaries with `folder_paths`, containment checks, or fixed allowlists.

📄 CodeRabbit inference engine (AGENTS.md)

Files:

  • comfy_api_nodes/nodes_recraft.py
Keep state and capability flags on the object that owns the behavior. Prefer explicit parent-owned attributes over probing child objects with `getattr`; use child checks only when the child owns the delegated behavior.

📄 CodeRabbit inference engine (AGENTS.md)

Files:

  • comfy_api_nodes/nodes_recraft.py
Keep changes small, direct, and limited to the narrowest necessary code path and smallest number of files.

📄 CodeRabbit inference engine (AGENTS.md)

Files:

  • comfy_api_nodes/nodes_recraft.py
Keep warning and info messages short and actionable, remove noisy or misleading logging, and make documentation edits concise, factual, and tied to changed behavior.

📄 CodeRabbit inference engine (AGENTS.md)

Files:

  • comfy_api_nodes/nodes_recraft.py
🔇 Additional comments (8)
comfy_api_nodes/nodes_recraft.py (8)

183-209: LGTM!


1387-1402: LGTM!

Also applies to: 1623-1638


1417-1446: LGTM!


336-338: LGTM!

Also applies to: 357-357, 1302-1326, 1538-1562, 1709-1709


1657-1676: 🎯 Functional Correctness

No change needed: the mapping covers both models. RECRAFT_V4_VECTOR_MODEL_FOR_STYLE maps recraftv4 to recraftv4_vector and recraftv4_pro to recraftv4_pro_vector.


1349-1374: 🎯 Functional Correctness

No change is required. RECRAFT_STYLE_MATCH_OPTIONS starts with "precise", so the widget default matches the execute default.


508-524: 🗄️ Data Integrity & Integration

No change is needed. RecraftCreateStyleRequest declares the optional model field, so RecraftV4CreateStyleNode can pass it to the styles request.


212-234: 🎯 Functional Correctness

RECRAFT_V4_STYLES_MODELS includes all four IDs: recraftv4_styles, recraftv4_styles_vector, recraftv4_styles_pro, and recraftv4_styles_pro_vector.


📝 Walkthrough

Walkthrough

The PR adds shared validation and encoding for V4 style references. It adds a node that creates reusable styles. V4 image and vector nodes now accept style IDs, style matching, and reference images. They return generated style IDs and use style-compatible models when required. Pricing includes style models and reference-image charges. Existing V3 style descriptions are corrected. The new style creation node is registered.

🚥 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%. Docstring coverage is scoped to functions touched by this diff. Analyzed 16 functions across 1 files. 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.
Description check ✅ Passed The description identifies this as an API node change, records the pricing decision, and states that QA is complete. These points relate to the Recraft V4 style changes.
Title check ✅ Passed The title clearly and concisely identifies the Recraft V4 Styles feature added by the pull request.
  • Fix all pre-merge checks with AI

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

@bigcat88
bigcat88 merged commit 30d03fe into master Aug 26, 2026
18 checks passed
@bigcat88
bigcat88 deleted the feat/partner-nodes/recraft-v4-styles branch August 26, 2026 18:22
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants