[Partner Nodes] add optional auto-downscaling of videos for ByteDance 2 nodes

[bigcat88]

• Restructured SEEDANCE2_REF_VIDEO_PIXEL_LIMITS to be per-resolution so 1920×1080 reference videos are accepted when output resolution is 1080p (previously blocked by the 927,408-pixel cap)
• Added a shared _compute_downscale_dims helper and a new resize_video_to_pixel_budget utility
• Added an opt-in advanced auto_downscale_videos toggle on ByteDance2ReferenceNode that transparently re-encodes oversized reference videos to fit the provider's per-resolution pixel budget before validation

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

[coderabbitai]

📝 Walkthrough

Walkthrough

This change adds resolution-aware pixel budget validation and automatic video downscaling to Seedance 2.0 reference video handling. The ByteDance2ReferenceNode gains an optional auto_downscale input that, when enabled, automatically downscales reference videos to fit within a specified pixel budget. Supporting utility functions compute aspect-preserving downscaled dimensions, re-encode videos with h264 encoding and audio preservation, and validate videos against resolution-specific pixel limits. The validation function signature is updated to accept resolution as a parameter for looking up appropriate pixel limits.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding optional auto-downscaling of videos for ByteDance 2 nodes, which matches the primary objectives of the pull request.
Docstring Coverage	✅ Passed	Docstring coverage is 81.82% which is sufficient. The required threshold is 80.00%.
Description check	✅ Passed	The pull request description clearly relates to the changeset, describing the restructuring of pixel limits, new utility functions, and the auto-downscale feature for ByteDance reference videos.

_{✏️ 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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

🤖 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_bytedance.py`:
- Around line 1496-1506: The synchronous PyAV re-encode in the block that checks
model.get("auto_downscale") and iterates reference_videos (calling
resize_video_to_pixel_budget) must be moved off the async event loop; replace
the direct calls to resize_video_to_pixel_budget with calls scheduled on a
thread executor (e.g., asyncio.to_thread or loop.run_in_executor), create async
tasks for each reference_videos entry, await them and then assign the returned
resized blobs back into reference_videos; ensure you keep the same checks for
max_px and model_id resolution and preserve the mapping keys when reassigning
results.

In `@comfy_api_nodes/util/conversions.py`:
- Around line 443-444: The code currently uses src_video_stream.average_rate
which can be None; replace that with a call that uses the Input.Video helper to
compute a safe frame rate (e.g. use src_video_stream.get_frame_rate() or an
equivalent function on the Input.Video object) when creating the output stream
via output_container.add_stream("h264", rate=...). Update the stream creation in
the block where src_video_stream is assigned (input_container.streams.video[0])
to pass the result of get_frame_rate() (or a small, documented fallback like 1)
instead of average_rate so variable-frame-rate or metadata-light inputs don't
cause a failure.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 78e75790-7bf3-4578-929a-b1e007b4e485

📥 Commits

Reviewing files that changed from the base of the PR and between 3086026 and 4510bac.

⛔ Files ignored due to path filters (1)

• comfy_api_nodes/apis/bytedance.py is excluded by !comfy_api_nodes/apis/**

📒 Files selected for processing (3)

• comfy_api_nodes/nodes_bytedance.py
• comfy_api_nodes/util/__init__.py
• comfy_api_nodes/util/conversions.py

[coderabbitai]

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/util/conversions.py`:
- Around line 141-146: After computing scale and applying max(2, ...) and the
evenness clamps to new_w and new_h, re-check the resulting pixel area (new_w *
new_h) against total_pixels; if the area exceeds total_pixels compute an
adjustment_scale = sqrt(total_pixels / (new_w * new_h)) and rescale new_w and
new_h by that factor (apply max(2, int(...)), then re-apply the evenness clamp).
Repeat or loop once more if necessary until new_w * new_h <= total_pixels. Use
the existing symbols scale, new_w, new_h, total_pixels, pixels, src_w, src_h to
locate and update the logic.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a2928ddf-e79f-49f6-b661-ba0347750b92

📥 Commits

Reviewing files that changed from the base of the PR and between 4510bac and 7b0a0e2.

⛔ Files ignored due to path filters (1)

• comfy_api_nodes/apis/bytedance.py is excluded by !comfy_api_nodes/apis/**

📒 Files selected for processing (3)

• comfy_api_nodes/nodes_bytedance.py
• comfy_api_nodes/util/__init__.py
• comfy_api_nodes/util/conversions.py

✅ Files skipped from review due to trivial changes (1)

• comfy_api_nodes/util/init.py

🚧 Files skipped from review as they are similar to previous changes (1)

• comfy_api_nodes/nodes_bytedance.py