feat(api-nodes): update xAI Grok nodes#13140
Conversation
📝 WalkthroughWalkthroughThis pull request introduces two new Grok video API nodes to the node catalog: 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ 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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@comfy_api_nodes/nodes_grok.py`:
- Around line 694-702: The VideoExtensionRequest currently omits the chosen
model value from model["model"], causing inconsistency with other video nodes;
update the call that constructs VideoExtensionRequest (in the sync_op call using
upload_video_to_comfyapi and VideoExtensionRequest) to include
model=model["model"] when building the request object (or if the extension
endpoint truly doesn't accept a model, remove the DynamicCombo that supplies
model["model"]), ensuring the request usage matches
GrokVideoEditNode/GrokVideoReferenceNode patterns.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 656a4f90-fb50-4248-8735-756870e9662e
⛔ Files ignored due to path filters (1)
comfy_api_nodes/apis/grok.pyis excluded by!comfy_api_nodes/apis/**
📒 Files selected for processing (1)
comfy_api_nodes/nodes_grok.py
API Node PR Checklist
Scope
Pricing & Billing
If Need pricing update:
QA
Comms