blenderwright 2.2.0__tar.gz → 2.2.1__tar.gz

{blenderwright-2.2.0 → blenderwright-2.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: blenderwright
-Version: 2.2.0
+Version: 2.2.1
 Summary: The most intuitive and efficient MCP Server for Blender
 License-Expression: MIT
 License-File: LICENSE.md

{blenderwright-2.2.0 → blenderwright-2.2.1}/addon/__init__.py

@@ -8,7 +8,7 @@ Blender's Python API.
 bl_info = {
     "name": "blenderwright",
     "author": "blenderwright",
-    "version": (2, 2, 0),
+    "version": (2, 2, 1),
     "blender": (4, 2, 0),
     "location": "View3D > Sidebar > blenderwright",
     "description": "MCP Server integration for AI-assisted 3D workflows",

{blenderwright-2.2.0 → blenderwright-2.2.1}/addon/blender_manifest.toml

@@ -1,7 +1,7 @@
 schema_version = "1.0.0"
 
 id = "blenderwright"
-version = "2.2.0"
+version = "2.2.1"
 name = "blenderwright"
 tagline = "MCP server integration for AI-assisted 3D workflows"
 maintainer = "jabberwock <88205+jabberwock@users.noreply.github.com>"

{blenderwright-2.2.0 → blenderwright-2.2.1}/addon/handlers/materials.py

@@ -55,9 +55,19 @@ def handle_assign_material(params: dict) -> dict:
                 obj.active_material_index = i
                 return {"object": obj.name, "material": mat.name, "action": "already_assigned", "slot": i}
 
+        # Fill the first empty slot before adding one. A boolean modifier
+        # leaves an empty slot 0 that every face points at; appending puts
+        # the material in slot 1 and the object still renders grey.
+        for i, slot_mat in enumerate(obj.data.materials):
+            if slot_mat is None:
+                obj.data.materials[i] = mat
+                obj.active_material_index = i
+                return {"object": obj.name, "material": mat.name, "action": "assigned", "slot": i}
+
         obj.data.materials.append(mat)
-        obj.active_material_index = len(obj.data.materials) - 1
-        return {"object": obj.name, "material": mat.name, "action": "assigned", "slot": len(obj.data.materials) - 1}
+        slot = len(obj.data.materials) - 1
+        obj.active_material_index = slot
+        return {"object": obj.name, "material": mat.name, "action": "assigned", "slot": slot}
     except Exception as e:
         raise RuntimeError(f"Failed to assign material: {e}")
 

{blenderwright-2.2.0 → blenderwright-2.2.1}/addon/handlers/objects.py

@@ -39,6 +39,21 @@ def _get_primitive_op(ptype: str):
     return ops_map.get(ptype)
 
 
+def _bake_scale(obj, scale) -> None:
+    """Apply a creation scale the way primitive_cube_add does: into the mesh.
+
+    Mesh data is transformed so the object keeps unit scale and modifiers
+    see the real size. An empty has no mesh, so it gets object scale.
+    """
+    sx, sy, sz = (float(v) for v in scale)
+    if (sx, sy, sz) == (1.0, 1.0, 1.0):
+        return
+    if obj.data is not None and hasattr(obj.data, "transform"):
+        obj.data.transform(mathutils.Matrix.Diagonal((sx, sy, sz, 1)))
+    else:
+        obj.scale = (sx, sy, sz)
+
+
 def _extent(obj) -> dict:
     """The object's size and where it ends, in world space.
 
@@ -77,9 +92,14 @@ def handle_create_object(params: dict) -> dict:
             op_func = _get_primitive_op(obj_type)
             if op_func is None:
                 raise ValueError(f"Unknown object type: {obj_type}")
-            op_func(location=location, rotation=rotation, scale=scale)
+            # Created at unit scale on purpose: the scale is baked below.
+            # Blender 5.1's primitive_plane_add and primitive_circle_add
+            # silently ignore their scale argument while primitive_cube_add
+            # bakes it into the mesh, so the op cannot be trusted with it.
+            op_func(location=location, rotation=rotation)
 
         obj = bpy.context.active_object
+        _bake_scale(obj, scale)
         if name:
             obj.name = name
             if obj.data:

{blenderwright-2.2.0 → blenderwright-2.2.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "blenderwright"
-version = "2.2.0"
+version = "2.2.1"
 description = "The most intuitive and efficient MCP Server for Blender"
 readme = "README.md"
 requires-python = ">=3.10"

{blenderwright-2.2.0 → blenderwright-2.2.1}/tests/test_addon/test_handlers/test_materials_handler.py

@@ -481,3 +481,67 @@ class TestRegistration:
             "set_color_ramp_interpolation",
             "get_color_ramp",
         } <= registered
+
+
+# ---------------------------------------------------------------------------
+# assign_material fills an empty slot before adding one
+# ---------------------------------------------------------------------------
+
+
+class TestAssignMaterialSlots:
+    """An object that has been through a boolean modifier carries an empty
+    material slot 0. Appending put the material in slot 1, every face still
+    pointed at slot 0, and the wall rendered grey. The first empty slot is
+    filled instead; a new slot is only added when none is empty."""
+
+    def _obj(self, slots):
+        obj = MagicMock()
+        obj.name = "Wall"
+        obj.data.materials = list(slots)
+        return obj
+
+    def _mat(self, name):
+        mat = MagicMock()
+        mat.name = name
+        return mat
+
+    def _patch(self, monkeypatch, obj, mat):
+        import bpy
+        monkeypatch.setattr(bpy.data.objects, "get", lambda n: obj)
+        monkeypatch.setattr(bpy.data.materials, "get", lambda n: mat)
+
+    def test_fills_the_empty_slot(self, mh, monkeypatch):
+        sage = self._mat("Wall_Sage")
+        obj = self._obj([None])
+        self._patch(monkeypatch, obj, sage)
+        result = mh.handle_assign_material({"object_name": "Wall", "material_name": "Wall_Sage"})
+        assert obj.data.materials == [sage]
+        assert result["slot"] == 0
+        assert result["action"] == "assigned"
+
+    def test_fills_the_first_empty_slot_only(self, mh, monkeypatch):
+        sage = self._mat("Wall_Sage")
+        other = self._mat("Other")
+        obj = self._obj([other, None, None])
+        self._patch(monkeypatch, obj, sage)
+        result = mh.handle_assign_material({"object_name": "Wall", "material_name": "Wall_Sage"})
+        assert obj.data.materials == [other, sage, None]
+        assert result["slot"] == 1
+
+    def test_appends_when_no_slot_is_empty(self, mh, monkeypatch):
+        sage = self._mat("Wall_Sage")
+        other = self._mat("Other")
+        obj = self._obj([other])
+        self._patch(monkeypatch, obj, sage)
+        result = mh.handle_assign_material({"object_name": "Wall", "material_name": "Wall_Sage"})
+        assert obj.data.materials == [other, sage]
+        assert result["slot"] == 1
+
+    def test_already_assigned_is_reported(self, mh, monkeypatch):
+        sage = self._mat("Wall_Sage")
+        obj = self._obj([None, sage])
+        self._patch(monkeypatch, obj, sage)
+        result = mh.handle_assign_material({"object_name": "Wall", "material_name": "Wall_Sage"})
+        assert result["action"] == "already_assigned"
+        assert result["slot"] == 1
+        assert obj.data.materials == [None, sage]

{blenderwright-2.2.0 → blenderwright-2.2.1}/tests/test_addon/test_handlers/test_objects_handler.py

@@ -544,3 +544,65 @@ class TestCreateReturnsExtent:
         result = objects_handler.handle_create_object({"type": "CUBE"})
         assert result["name"] == "Fuselage"
         assert "bounds" not in result
+
+
+# ---------------------------------------------------------------------------
+# create_object bakes the scale itself
+# ---------------------------------------------------------------------------
+
+
+class TestCreateBakesScale:
+    """Blender 5.1's primitive_plane_add and primitive_circle_add silently
+    ignore their scale argument, while primitive_cube_add bakes it into the
+    mesh. A hallway floor asked for at scale (3, 4.5, 1) came back 2m square.
+    The handler now creates every primitive at unit scale and bakes the scale
+    into the mesh itself, so every type behaves the way the cube always did."""
+
+    def _obj(self, bpy):
+        obj = MagicMock()
+        obj.name = "Floor"
+        obj.type = "MESH"
+        obj.location = [0, 0, 0]
+        obj.dimensions = [6.0, 9.0, 0.0]
+        obj.bound_box = None
+        for op in ("primitive_plane_add", "primitive_cube_add", "primitive_circle_add"):
+            getattr(bpy.ops.mesh, op).side_effect = None
+        bpy.context.active_object = obj
+        return obj
+
+    def test_plane_scale_is_baked_into_the_mesh(self, objects_handler):
+        import bpy
+        import mathutils
+        bpy.reset_mock()
+        mathutils.Matrix.reset_mock()
+        obj = self._obj(bpy)
+        objects_handler.handle_create_object({
+            "type": "PLANE", "location": (0, 4.5, 0), "scale": (3, 4.5, 1),
+        })
+        mathutils.Matrix.Diagonal.assert_called_once_with((3, 4.5, 1, 1))
+        obj.data.transform.assert_called_once_with(mathutils.Matrix.Diagonal.return_value)
+
+    def test_the_op_is_not_asked_to_scale(self, objects_handler):
+        """Passing scale to the op as well would double it on the cube."""
+        import bpy
+        bpy.reset_mock()
+        self._obj(bpy)
+        objects_handler.handle_create_object({"type": "CUBE", "scale": (2, 2, 2)})
+        assert "scale" not in bpy.ops.mesh.primitive_cube_add.call_args.kwargs
+
+    def test_unit_scale_does_not_touch_the_mesh(self, objects_handler):
+        import bpy
+        bpy.reset_mock()
+        obj = self._obj(bpy)
+        objects_handler.handle_create_object({"type": "CIRCLE", "scale": (1, 1, 1)})
+        obj.data.transform.assert_not_called()
+
+    def test_empty_gets_object_scale(self, objects_handler):
+        """An empty has no mesh to bake into, so its object scale is set."""
+        import bpy
+        bpy.reset_mock()
+        obj = self._obj(bpy)
+        obj.type = "EMPTY"
+        obj.data = None
+        objects_handler.handle_create_object({"type": "EMPTY", "scale": (2, 3, 4)})
+        assert tuple(obj.scale) == (2, 3, 4)

{blenderwright-2.2.0 → blenderwright-2.2.1}/uv.lock

@@ -51,7 +51,7 @@ wheels = [
 
 [[package]]
 name = "blenderwright"
-version = "2.2.0"
+version = "2.2.1"
 source = { editable = "." }
 dependencies = [
     { name = "mcp" },