Skip to content

[Partner Nodes] chore(Google): drop retiring Veo 2 and Veo 3.0 models - #15883

Merged
bigcat88 merged 2 commits into
masterfrom
chore/partner-nodes/veo-drop-retiring-models
Aug 26, 2026
Merged

[Partner Nodes] chore(Google): drop retiring Veo 2 and Veo 3.0 models#15883
bigcat88 merged 2 commits into
masterfrom
chore/partner-nodes/veo-drop-retiring-models

Conversation

@bigcat88

@bigcat88 bigcat88 commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Google will remove those models in a few days, so we are removing them as well.

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 25, 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: 050c5141-b880-4ccb-b0f2-f372adb99aaa

📥 Commits

Reviewing files that changed from the base of the PR and between a1079ba and deaaf0a.

📒 Files selected for processing (1)
  • comfy_api_nodes/nodes_veo2.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. (8)
  • GitHub Check: test (windows-2022)
  • GitHub Check: test (ubuntu-latest)
  • GitHub Check: test (macos-latest)
  • GitHub Check: test
  • GitHub Check: test (macos-latest)
  • GitHub Check: test (ubuntu-latest)
  • GitHub Check: Run Pylint
  • GitHub Check: test (windows-latest)
🧰 Additional context used
📓 Path-based instructions (6)
Third-party API integration nodes. Focus on:

⚙️ CodeRabbit configuration file

Files:

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

⚙️ CodeRabbit configuration file

Files:

  • comfy_api_nodes/nodes_veo2.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_veo2.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_veo2.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_veo2.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_veo2.py
🔇 Additional comments (2)
comfy_api_nodes/nodes_veo2.py (2)

26-29: LGTM!

Also applies to: 59-59, 112-112, 143-145, 164-168, 184-184


26-29: 📐 Maintainability & Code Quality

No residual textual references to VeoVideoGenerationNode or the retired Veo 2/3.0 model aliases remain in the repository.


📝 Walkthrough

Walkthrough

The Veo integration now supports only Veo 3.1 models. It removes the Veo 2 node and Veo 3.0 model mappings. The Veo 3 node defaults to veo-3.1-generate, limits 4K rejection to veo-3.1-lite, updates pricing groups, and always sends resolution in the request.

🚥 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 3 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.
Title check ✅ Passed The title clearly states that the PR removes the retiring Google Veo 2 and Veo 3.0 models, which is the main change.
Description check ✅ Passed The description explains that Google will remove the Veo 2 and Veo 3.0 models and that this PR removes them from Partner Nodes.
  • Fix all pre-merge checks with AI

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.

@bigcat88
bigcat88 merged commit b133e48 into master Aug 26, 2026
22 of 23 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 26, 2026
@bigcat88
bigcat88 deleted the chore/partner-nodes/veo-drop-retiring-models branch August 26, 2026 12:12
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