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

2 participants