feat: add TRIPO3D_AI

[junmer]

PR Type

Enhancement

---

Description

• Added support for TRIPO3D_AI mode in multiple functions.

• Implemented poll_rodin_job_status_tripo3d_ai to query task status.

• Added create_rodin_job_tripo3d_ai to create 3D model generation tasks.

• Introduced import_generated_asset_tripo3d_ai to fetch and import assets.

---

Changes walkthrough 📝

Relevant files

Enhancement

addon.py Add TRIPO3D_AI mode and API integrations                                  addon.py Added TRIPO3D_AI mode handling in create_rodin_job, poll_rodin_job_status, and import_generated_asset. Implemented poll_rodin_job_status_tripo3d_ai for querying task status from Tripo3D API. Added create_rodin_job_tripo3d_ai for creating 3D model generation tasks via Tripo3D API. Introduced import_generated_asset_tripo3d_ai for fetching and importing assets into Blender. Updated blendermcp_hyper3d_mode EnumProperty to include TRIPO3D_AI. +157/-1

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

Summary by CodeRabbit

• New Features
  • Introduced an AI-powered mode for 3D model generation using text prompts with optional image input.
  • Enabled smooth job submission, status tracking, and asset import for managing 3D assets.
  • Enhanced error handling to deliver a more reliable and consistent user experience in 3D creation.

[coderabbitai]

Walkthrough

This change integrates support for a new mode, "TRIPO3D_AI", into the BlenderMCPServer class. New methods have been added to handle the creation of jobs, polling of job status, and importing of generated assets through the Tripo3D API. The existing methods (create_rodin_job, poll_rodin_job_status, and import_generated_asset) have been updated to include a case for "TRIPO3D_AI". Additionally, a new enum entry for this mode has been introduced.

Changes

File	Change Summary
addon.py	- Added new methods: create_rodin_job_tripo3d_ai, poll_rodin_job_status_tripo3d_ai, and import_generated_asset_tripo3d_ai in BlenderMCPServer. - Updated methods create_rodin_job, poll_rodin_job_status, and import_generated_asset to handle "TRIPO3D_AI". - Added enum item ("TRIPO3D_AI", "tripo3d.ai", "tripo3d.ai") to bpy.types.Scene.blendermcp_hyper3d_mode.

Sequence Diagram(s)

sequenceDiagram
    participant U as User
    participant S as BlenderMCPServer
    participant T as Tripo3D API
    participant F as Filesystem

    U->>S: Request TRIPO3D_AI job creation
    S->>T: Call create_rodin_job_tripo3d_ai(text_prompt, images, bbox_condition)
    T-->>S: Respond with job details
    U->>S: Poll job status request
    S->>T: Call poll_rodin_job_status_tripo3d_ai(request_id)
    T-->>S: Return job status (model URL or error)
    U->>S: Request asset import
    S->>T: Verify task status for model download
    S->>F: Download asset & manage temporary file
    F-->>S: Asset stored and temporary file cleaned up
    S-->>U: Return imported asset info (name, type, location, rotation, scale)

Loading

Poem

I'm a coding rabbit with a twinkle in my eye,
Hop, hop into the code, where new features lie.
TRIPO3D_AI is now in play, clever and spry,
Crafting 3D dreams as the old methods pass by.
Hopping through changes, my joy will never die!

✨ Finishing Touches

• 📝 Generate Docstrings

---

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Generate unit testing code for this file.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
  • @coderabbitai generate unit testing code for this file.
  • @coderabbitai modularize this function.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read src/utils.ts and generate unit testing code.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
  • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

• @coderabbitai pause to pause the reviews on a PR.
• @coderabbitai resume to resume the paused reviews.
• @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai full review to do a full review from scratch and review all the files again.
• @coderabbitai summary to regenerate the summary of the PR.
• @coderabbitai generate docstrings to generate docstrings for this PR.
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai plan to trigger planning for file edits and PR creation.
• @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
• @coderabbitai help to get help.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• Please see the configuration documentation for more information.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[qodo-merge-pro]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪

🧪 No relevant tests

🔒 Security concerns API key exposure: The implementation sends API keys in request headers without validating their presence first. If a request is made with an empty or invalid API key, it could lead to unexpected errors or potentially expose the request structure to unauthorized endpoints. Consider adding validation before making API requests.

⚡ Recommended focus areas for review Error Handling The error handling in the Tripo3D implementation differs from other implementations. Some error cases might not be properly handled or might return inconsistent response structures compared to other API implementations. def poll_rodin_job_status_tripo3d_ai(self, request_id: str): """Query Tripo3D task status""" try: response = requests.get( f"https://api.tripo3d.ai/v2/openapi/task/{request_id}", headers={ "Authorization": f"Bearer {bpy.context.scene.blendermcp_hyper3d_api_key}", } ) if response.status_code != 200: return {"error": f"API request failed with status code {response.status_code}: {response.text}"} data = response.json() # get output data output_data = data.get("data", {}).get("output", {}) # get mode URL model_url = output_data.get("pbr_model") or output_data.get("model") or output_data.get("base_model") if not model_url: return { "status": data.get("code"), "message": data.get("message", "Model URL not available yet"), } return { "status": data.get("code"), "message": data.get("message", "Model URL ready"), "model_url": model_url } except Exception as e: return {"error": str(e)} Resource Cleanup The temporary file handling in import_generated_asset_tripo3d_ai has nested try-except blocks with manual cleanup. Consider using context managers (with statements) for better resource management. temp_file = tempfile.NamedTemporaryFile( delete=False, prefix=request_id, suffix=".glb", ) try: # Download model file response = requests.get(status_response["model_url"], stream=True) response.raise_for_status() # Write to temporary file for chunk in response.iter_content(chunk_size=8192): temp_file.write(chunk) temp_file.close() except Exception as e: # Clean up temporary file temp_file.close() os.unlink(temp_file.name) return {"succeed": False, "error": str(e)} try: # Import model obj = self._clean_imported_glb( filepath=temp_file.name, mesh_name=name ) # Prepare return result result = { "name": obj.name, "type": obj.type, "location": [obj.location.x, obj.location.y, obj.location.z], "rotation": [obj.rotation_euler.x, obj.rotation_euler.y, obj.rotation_euler.z], "scale": [obj.scale.x, obj.scale.y, obj.scale.z], } if obj.type == "MESH": bounding_box = self._get_aabb(obj) result["world_bounding_box"] = bounding_box return { "succeed": True, **result } except Exception as e: return {"succeed": False, "error": str(e)} finally: # Clean up temporary file try: os.unlink(temp_file.name) except: pass Inconsistent Parameters The create_rodin_job_tripo3d_ai function accepts images parameter but doesn't use it, unlike other implementations. This could lead to confusion or unexpected behavior. def create_rodin_job_tripo3d_ai( self, text_prompt: str=None, images: list[tuple[str, str]]=None, bbox_condition=None ): """Call Tripo3D API to create a 3D model generation task""" try: if not text_prompt: return {"error": "Text prompt is required for Tripo3D"} # Prepare request data req_data = { "type": "text_to_model", "prompt": text_prompt }

[qodo-merge-pro]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Improve HTTP error handling The code doesn't handle HTTP errors properly. If the download fails with a non-200 status code, raise_for_status() will raise an exception, but there's no specific error handling for this case. Add a try-except block specifically for HTTP errors to provide a more informative error message. addon.py [1710-1718] # Download model file -response = requests.get(status_response["model_url"], stream=True) -response.raise_for_status() +try: + response = requests.get(status_response["model_url"], stream=True) + response.raise_for_status() -# Write to temporary file -for chunk in response.iter_content(chunk_size=8192): - temp_file.write(chunk) + # Write to temporary file + for chunk in response.iter_content(chunk_size=8192): + temp_file.write(chunk) -temp_file.close() + temp_file.close() +except requests.exceptions.HTTPError as http_err: + temp_file.close() + os.unlink(temp_file.name) + return {"succeed": False, "error": f"HTTP error downloading model: {http_err}"} Apply this suggestion Suggestion importance[1-10]: 7 __ Why: This suggestion adds specific handling for HTTP errors during model download, providing more informative error messages. It's a valid improvement to error handling that would make debugging easier and provide clearer feedback to users when downloads fail.	Medium
More

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

addon.py (1)

1691-1759: Asset importing implementation could use minor improvements.

The code correctly handles the full flow from status checking to file download and import. However, there are a couple of minor issues identified by static analysis.

1. Use a context manager for file opening at line 1703:

-            temp_file = tempfile.NamedTemporaryFile(
-                delete=False,
-                prefix=request_id,
-                suffix=".glb",
-            )
+            with tempfile.NamedTemporaryFile(
+                delete=False,
+                prefix=request_id,
+                suffix=".glb",
+            ) as temp_file:
+                # Download model file
+                response = requests.get(status_response["model_url"], stream=True)
+                response.raise_for_status()
+
+                # Write to temporary file
+                for chunk in response.iter_content(chunk_size=8192):
+                    temp_file.write(chunk)
+
+                temp_file_name = temp_file.name

2. Replace bare except with contextlib.suppress at lines 1753-1756:

-            finally:
-                # Clean up temporary file
-                try:
-                    os.unlink(temp_file.name)
-                except:
-                    pass
+            finally:
+                # Clean up temporary file
+                import contextlib
+                with contextlib.suppress(Exception):
+                    os.unlink(temp_file_name)

🧰 Tools

🪛 Ruff (0.8.2)

1703-1703: Use a context manager for opening files

(SIM115)

---

1753-1756: Use contextlib.suppress(Exception) instead of try-except-pass

Replace with contextlib.suppress(Exception)

(SIM105)

---

1755-1755: Do not use bare except

(E722)

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eec042e and 07448ed.

📒 Files selected for processing (1)

• addon.py (6 hunks)

🧰 Additional context used

🪛 Ruff (0.8.2)

addon.py

1703-1703: Use a context manager for opening files

(SIM115)

---

1753-1756: Use contextlib.suppress(Exception) instead of try-except-pass

Replace with contextlib.suppress(Exception)

(SIM105)

---

1755-1755: Do not use bare except

(E722)

🔇 Additional comments (6)

addon.py (6)

1317-1318: Integration point for TRIPO3D_AI looks good.

The dispatching pattern consistently matches the existing implementation for other platforms.

---

1388-1389: Consistent dispatching implementation for polling.

The pattern follows the established code structure for other supported platforms.

---

1420-1454: Well-implemented status polling function for TRIPO3D_AI.

The implementation appropriately handles API responses and error conditions. I particularly like the way you check for the model URL in multiple possible response locations, making the code robust against potential API response structure changes.

---

1455-1499: Comprehensive job creation implementation for TRIPO3D_AI.

The code properly validates required parameters, constructs the API request, and handles response processing and error cases. The implementation follows the same pattern as the existing platform integrations, which maintains consistency throughout the codebase.

---

1564-1565: Consistent dispatching for asset importing.

The pattern matches the established implementation for other platforms.

---

1870-1870: UI integration for TRIPO3D_AI completed correctly.

The enum addition follows the established pattern and maintains consistency with other platform options.