Skip to content

[Partner Nodes] feat(Grok): add 1080p resolution to Grok Image node#14597

Merged
alexisrolland merged 3 commits into
masterfrom
feat/api-nodes/Grok-Imagine
Jun 23, 2026
Merged

[Partner Nodes] feat(Grok): add 1080p resolution to Grok Image node#14597
alexisrolland merged 3 commits into
masterfrom
feat/api-nodes/Grok-Imagine

Conversation

@bigcat88

@bigcat88 bigcat88 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Also we will switch from grok-imagine-video-1.5-preview to the grok-imagine-video-1.5 model ID.

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

@bigcat88 bigcat88 marked this pull request as ready for review June 23, 2026 15:02
@coderabbitai

coderabbitai Bot commented Jun 23, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The PR updates comfy_api_nodes/nodes_grok.py to add 1080p resolution support to GrokVideoNode. The internal API model mapping for "grok-imagine-video-1.5" is changed from the -preview identifier to the stable one. The resolution schema input gains a 1080p option with an updated tooltip. The price_badge USD calculation is reworked with new per-resolution rates, a revised image-connected cost add-on, and a * 1.43 multiplier applied to the final total. A runtime ValueError is added in execute to reject resolution == "1080p" when the selected model is not grok-imagine-video-1.5.

🚥 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
Title check ✅ Passed The title mentions adding 1080p resolution to the Grok Image node, which aligns with the main change of exposing 1080p resolution support and updating model identifiers.
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 pull request description directly addresses the changeset by mentioning the model ID switch and includes a completed API Node PR Checklist relevant to the changes.

✏️ 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.

@alexisrolland alexisrolland merged commit 0f949d0 into master Jun 23, 2026
22 checks passed
@bigcat88 bigcat88 deleted the feat/api-nodes/Grok-Imagine branch June 23, 2026 15:46
@bigcat88 bigcat88 mentioned this pull request Jun 23, 2026
8 tasks
@bigcat88 bigcat88 restored the feat/api-nodes/Grok-Imagine branch June 24, 2026 06:02
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