Skip to content

[Partner Nodes] chore(Tripo): remove retired Refine node and v2.0 model version - #15851

Open
bigcat88 wants to merge 1 commit into
masterfrom
feat/partner-nodes/tripo-remove-refine
Open

[Partner Nodes] chore(Tripo): remove retired Refine node and v2.0 model version#15851
bigcat88 wants to merge 1 commit into
masterfrom
feat/partner-nodes/tripo-remove-refine

Conversation

@bigcat88

@bigcat88 bigcat88 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Both are retired upstream and can no longer create a task. Verified live against on 2026-08-24

  • TripoRefineNode - rejected at submit for both of its possible inputs: a v1.4 draft (its documented input) with 2015 "The version has been deprecated", a v2.5 model+ (the suggested "higher version") with 2006 "The original task type is not supported"
  • v2.0-20240919 - rejected with 2015 on text_to_model, image_to_model and multiview_to_model alike, so it was a dead option in three model_version combos

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

…el version

Signed-off-by: bigcat88 <bigcat88@icloud.com>
@coderabbitai

coderabbitai Bot commented Aug 24, 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: 92074ab1-3c31-4f79-baa3-08333098f178

📥 Commits

Reviewing files that changed from the base of the PR and between b78cec8 and 6776065.

⛔ Files ignored due to path filters (1)
  • comfy_api_nodes/apis/tripo.py is excluded by !comfy_api_nodes/apis/**
📒 Files selected for processing (1)
  • comfy_api_nodes/nodes_tripo.py
💤 Files with no reviewable changes (1)
  • comfy_api_nodes/nodes_tripo.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 (macos-latest)
  • GitHub Check: test (ubuntu-latest)
  • GitHub Check: test (windows-2022)
  • GitHub Check: test (macos-latest)
  • GitHub Check: test (windows-latest)
  • GitHub Check: test (ubuntu-latest)
  • GitHub Check: Run Pylint
  • GitHub Check: test

📝 Walkthrough

Walkthrough

The PR removes the TripoRefineModelRequest import and deletes TripoRefineNode, including its schema, pricing metadata, request submission, and polling behavior. TripoExtension.get_node_list() no longer registers the refine node.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.
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 identifies the removal of the retired Tripo Refine node and v2.0 model version.
Description check ✅ Passed The description explains why both retired Tripo components are removed and records live upstream verification results.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant