[Partner Nodes] revert last 3 PRs#14602
Conversation
…r SeeDance 2.0 (#14588)"
📝 WalkthroughWalkthroughThree sets of video node configuration changes are applied across separate provider files. In 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 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 current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@comfy_api_nodes/nodes_wan.py`:
- Line 2162: The validation error message in the ValueError contains a
duplicated word "reference reference" which should be just "reference". Locate
the error message that reads "At least one reference reference image must be
provided." and remove the duplicate "reference" word so it reads "At least one
reference image must be provided."
🪄 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 Plus
Run ID: 750f17b6-e37e-4324-bbbd-2cdc03177d6d
⛔ Files ignored due to path filters (1)
comfy_api_nodes/apis/bytedance.pyis excluded by!comfy_api_nodes/apis/**
📒 Files selected for processing (3)
comfy_api_nodes/nodes_bytedance.pycomfy_api_nodes/nodes_grok.pycomfy_api_nodes/nodes_wan.py
| ) | ||
| if not media: | ||
| raise ValueError("At least one reference image must be provided.") | ||
| raise ValueError("At least one reference reference image must be provided.") |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Fix duplicated word in the validation error text.
Line 2162 says "reference reference image", which is a user-facing typo.
Suggested patch
- raise ValueError("At least one reference reference image must be provided.")
+ raise ValueError("At least one reference image must be provided.")📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| raise ValueError("At least one reference reference image must be provided.") | |
| raise ValueError("At least one reference image must be provided.") |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@comfy_api_nodes/nodes_wan.py` at line 2162, The validation error message in
the ValueError contains a duplicated word "reference reference" which should be
just "reference". Locate the error message that reads "At least one reference
reference image must be provided." and remove the duplicate "reference" word so
it reads "At least one reference image must be provided."
Reverting next PRs:
API Node PR Checklist
Scope
Pricing & Billing
If Need pricing update:
QA
Comms