merge from source by elasticdotventures · Pull Request #2 · elasticdotventures/blender-mcp

elasticdotventures · 2025-11-23T03:08:52Z

test

Updated Windows installation instructions for UV. The previous instructions used set Path=..., which only works in CMD and only temporarily modifies the PATH for that CMD session (and any new processes launched from it). This was inconsistent with the preceding PowerShell-based install command (irm ... | iex), and would fail if run in PowerShell. Updated the example to use a proper PowerShell approach that dynamically resolves the current user's home directory via $env:USERPROFILE and appends .local\bin to the user-level PATH persistently (so the user does not have to set it each time before launching Claude Code).

Fixed possible issue in instructions for adding uv to user path in Windows, so as not to inadvertently overwrite existing user path with system path.

Revised UV installation steps for Windows

Co-authored-by: 家靖 <zhangjiajing.zjj@alibaba-inc.com>

Copilot

Pull request overview

This pull request merges updates from source, primarily adding Hunyuan3D support and telemetry infrastructure to the Blender MCP project.

Key Changes

Telemetry system: Added anonymous telemetry tracking for tool usage via Supabase, with environment variable opt-out support
Hunyuan3D integration: Implemented 3D model generation using Tencent's Hunyuan3D API (both official and local API modes)
Infrastructure improvements: Increased socket timeout from 15s to 180s to support longer-running 3D generation tasks

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 26 comments.

Show a summary per file

File	Description
src/blender_mcp/telemetry.py	New telemetry collection module for tracking tool usage and performance metrics
src/blender_mcp/telemetry_decorator.py	Decorator for instrumenting MCP tools with telemetry tracking
src/blender_mcp/server.py	Added telemetry decorators to existing tools, implemented Hunyuan3D tool functions, increased socket timeouts
addon.py	Implemented Hunyuan3D API integration including job creation, polling, and asset import; added UI properties for configuration
pyproject.toml	Updated version to 1.4.0 and added dependencies for supabase and tomli
README.md	Updated release notes and installation instructions for Windows
.gitignore	Added config.py to gitignore for storing telemetry secrets

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-11-23T03:14:59Z

 import shutil
 import zipfile
-from bpy.props import StringProperty, IntProperty, BoolProperty, EnumProperty
+from bpy.props import IntProperty


The import statement for EnumProperty has been removed, but it's still needed by the blendermcp_hunyuan3d_mode property defined at line 2259. This will cause a NameError when the addon is registered.

Suggested change

from bpy.props import IntProperty

from bpy.props import IntProperty, EnumProperty

Copilot · 2025-11-23T03:14:59Z

+
+    def poll_hunyuan_job_status_ai(self, job_id: str):
+        """Call the job status API to get the job status"""
+        print(job_id)


Debug print statement left in production code. This should be removed or converted to proper logging.

Copilot · 2025-11-23T03:14:59Z

 import shutil
 import zipfile
-from bpy.props import StringProperty, IntProperty, BoolProperty, EnumProperty
+from bpy.props import IntProperty


The import statement for FloatProperty is missing but it's needed by the blendermcp_hunyuan3d_guidance_scale property defined at line 2304. This will cause a NameError when the addon is registered.

Suggested change

from bpy.props import IntProperty

from bpy.props import IntProperty, FloatProperty

Copilot · 2025-11-23T03:14:59Z

+        )
+        self._worker.start()
+
+        logger.warning(f"Telemetry initialized (enabled={self.config.enabled}, has_supabase={HAS_SUPABASE}, customer_uuid={self._customer_uuid})")


Using logger.warning() for informational telemetry messages (lines 96, 113, 178, 181, 184) is incorrect. These should use logger.info() or logger.debug() instead, as they are not actual warnings that need user attention.

Suggested change

logger.warning(f"Telemetry initialized (enabled={self.config.enabled}, has_supabase={HAS_SUPABASE}, customer_uuid={self._customer_uuid})")

logger.info(f"Telemetry initialized (enabled={self.config.enabled}, has_supabase={HAS_SUPABASE}, customer_uuid={self._customer_uuid})")

Copilot · 2025-11-23T03:15:00Z

+            logger.warning(f"Supabase not available, skipping event: {event_type}")
+            return
+
+        logger.warning(f"Recording telemetry event: {event_type}, tool={tool_name}")


Using logger.warning() for informational telemetry messages is incorrect. These should use logger.info() or logger.debug() instead, as they are not actual warnings that need user attention.

Suggested change

logger.warning(f"Recording telemetry event: {event_type}, tool={tool_name}")

logger.info(f"Recording telemetry event: {event_type}, tool={tool_name}")

Copilot · 2025-11-23T03:15:04Z

+            }
+
+            response = supabase.table("telemetry_events").insert(data, returning="minimal").execute()
+            logger.debug(f"Telemetry sent: {event.event_type}")


Variable response is not used.

Suggested change

logger.debug(f"Telemetry sent: {event.event_type}")

logger.debug(f"Telemetry sent: {event.event_type}, response: {response}")

Copilot · 2025-11-23T03:15:04Z

+        temp_dir = tempfile.mkdtemp(prefix="tencent_obj_")
+        zip_file_path = osp.join(temp_dir, "model.zip")
+        obj_file_path = osp.join(temp_dir, "model.obj")
+        mtl_file_path = osp.join(temp_dir, "model.mtl")


Variable mtl_file_path is not used.

Copilot · 2025-11-23T03:15:04Z

 from urllib.parse import urlparse

+# Import telemetry
+from .telemetry import record_startup, get_telemetry


Import of 'get_telemetry' is not used.

Suggested change

from .telemetry import record_startup, get_telemetry

from .telemetry import record_startup

Copilot · 2025-11-23T03:15:04Z

+"""
+
+import contextlib
+import json


Import of 'json' is not used.

Suggested change

import json

Copilot · 2025-11-23T03:15:05Z

+                    data = tomli.load(f)
+                    return data["project"]["version"]
+    except Exception:
+        pass


'except' clause does nothing but pass and there is no explanatory comment.

Suggested change

pass

# Log the exception but return "unknown" as a fallback

logger.exception("Failed to read version from pyproject.toml")

* feat: Add Sketchfab model preview functionality - Add get_sketchfab_model_preview handler in addon.py - Fetches thumbnail from Sketchfab API - Returns base64-encoded image with model info - Add get_sketchfab_model_preview MCP tool in server.py - Returns Image type for visual confirmation - Allows users to preview models before downloading * feat: Add size normalization for Sketchfab model imports - Add normalize_size and target_size parameters to download_sketchfab_model - Calculate combined bounding box for all imported mesh objects - Apply uniform scale to normalize largest dimension to target_size - Return detailed import info: dimensions, bounding box, scale factor - Default: normalize_size=True, target_size=1.0 (1 meter) This prevents extremely large or small models from being imported, making it easier for LLMs to work with consistent object sizes. * feat: Make target_size required parameter for Sketchfab downloads - Remove normalize_size parameter (always normalize now) - Make target_size a required parameter with no default value - Add helpful examples in docstring for common object sizes - Forces LLM to explicitly consider and specify model size * chore: Remove temporary test files from development * fix: Correct bounding box calculation and scale application for hierarchical models - Add recursive get_all_mesh_children() to collect meshes from entire hierarchy - Fix bounding box calculation to include all nested mesh objects - Use transform_apply() to properly apply scale to entire object hierarchy - Prevents parts from becoming scattered when scaling complex models This fixes issues with Sketchfab models that have deep hierarchies like: Sketchfab_model (EMPTY) └─ RootNode (EMPTY) └─ Chassis (MESH) ← now correctly included * fix: Apply scale only to root objects to prevent cumulative scaling - Find root objects (objects without parents) instead of all selected objects - Apply scale factor ONLY to root objects, not children - Remove transform_apply (parent scale automatically affects children via matrix_world) - Child meshes inherit scaling through Blender's hierarchy system Fixes issue where scale was applied to every object in hierarchy, causing cumulative scaling (0.007^5 ≈ 0) that made models invisible.

Added instructions for using the Claude Code CLI to add the blender MCP server.

mvanchura-ux and others added 6 commits October 18, 2025 02:34

Revise Windows user path setup for uv

3ee4b2d

Fixed possible issue in instructions for adding uv to user path in Windows, so as not to inadvertently overwrite existing user path with system path.

Merge pull request #157 from mvanchura-ux/patch-1

a14db74

Revised UV installation steps for Windows

anon telemetry

6a58e20

hunyuan support added

5f81abe

feat: add Hunyuan3D official or local api (#124)

668be7f

Co-authored-by: 家靖 <zhangjiajing.zjj@alibaba-inc.com>

Copilot AI review requested due to automatic review settings November 23, 2025 03:08

Copilot started reviewing on behalf of elasticdotventures November 23, 2025 03:09 View session

Copilot finished reviewing on behalf of elasticdotventures November 23, 2025 03:12

Copilot AI reviewed Nov 23, 2025

View reviewed changes

ahujasid and others added 8 commits December 8, 2025 20:03

Update README.md

9cefe8e

add tnc

209af47

Update README with Claude Code CLI instructions (#170)

21007bc

Added instructions for using the Claude Code CLI to add the blender MCP server.

edit tnc

6e60c3d

remove sponsor

61fb53e

Fix Cursor Install MCP Server link (#173)

f0554aa

fixed telemetry not being read

7636d13

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

merge from source#2

merge from source#2
elasticdotventures wants to merge 14 commits into
elasticdotventures:mainfrom
ahujasid:main

elasticdotventures commented Nov 23, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Nov 23, 2025

Uh oh!

Copilot AI Nov 23, 2025

Uh oh!

Copilot AI Nov 23, 2025

Uh oh!

Copilot AI Nov 23, 2025

Uh oh!

Copilot AI Nov 23, 2025

Uh oh!

Copilot AI Nov 23, 2025

Uh oh!

Copilot AI Nov 23, 2025

Uh oh!

Copilot AI Nov 23, 2025

Uh oh!

Copilot AI Nov 23, 2025

Uh oh!

Copilot AI Nov 23, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

	from bpy.props import IntProperty
	from bpy.props import IntProperty, EnumProperty

	from bpy.props import IntProperty
	from bpy.props import IntProperty, FloatProperty

	logger.warning(f"Telemetry initialized (enabled={self.config.enabled}, has_supabase={HAS_SUPABASE}, customer_uuid={self._customer_uuid})")
	logger.info(f"Telemetry initialized (enabled={self.config.enabled}, has_supabase={HAS_SUPABASE}, customer_uuid={self._customer_uuid})")

	logger.warning(f"Recording telemetry event: {event_type}, tool={tool_name}")
	logger.info(f"Recording telemetry event: {event_type}, tool={tool_name}")

	logger.debug(f"Telemetry sent: {event.event_type}")
	logger.debug(f"Telemetry sent: {event.event_type}, response: {response}")

	from .telemetry import record_startup, get_telemetry
	from .telemetry import record_startup

	pass
	# Log the exception but return "unknown" as a fallback
	logger.exception("Failed to read version from pyproject.toml")

Conversation

elasticdotventures commented Nov 23, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Key Changes

Reviewed changes

Uh oh!

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants