fix(manage_gameobject): compare flags directly for partial static objects

targetGo.isStatic returns true when *any* flag is set, so a partially
static object (e.g. only Navigation) would skip the update when
is_static=true was requested. Compare GetStaticEditorFlags() against
the desired flags instead.

Also adds a "false" string coercion test per review feedback.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Sebastian Muehr

MCPForUnity/Editor/Tools/GameObjects/GameObjectModify.cs

@@ -152,11 +152,16 @@ internal static object Handle(JObject @params, JToken targetToken, string search
             }
 
             bool? isStatic = @params["isStatic"]?.ToObject<bool?>();
-            if (isStatic.HasValue && targetGo.isStatic != isStatic.Value)
+            if (isStatic.HasValue)
             {
-                var flags = isStatic.Value ? (StaticEditorFlags)~0 : 0;
-                GameObjectUtility.SetStaticEditorFlags(targetGo, flags);
-                modified = true;
+                var desiredFlags = isStatic.Value ? (StaticEditorFlags)~0 : 0;
+                var currentFlags = GameObjectUtility.GetStaticEditorFlags(targetGo);
+
+                if (currentFlags != desiredFlags)
+                {
+                    GameObjectUtility.SetStaticEditorFlags(targetGo, desiredFlags);
+                    modified = true;
+                }
             }
 
             Vector3? position = VectorParsing.ParseVector3(@params["position"]);

Server/tests/integration/test_manage_gameobject_is_static.py

@@ -84,6 +84,32 @@ async def fake_send(cmd, params, **kwargs):
     assert captured["params"]["isStatic"] is True
 
 
+@pytest.mark.asyncio
+async def test_manage_gameobject_is_static_string_false_coercion(monkeypatch):
+    """Test that string 'false' is coerced to bool for is_static."""
+    captured = {}
+
+    async def fake_send(cmd, params, **kwargs):
+        captured["params"] = params
+        return {"success": True, "data": {}}
+
+    monkeypatch.setattr(
+        manage_go_mod,
+        "async_send_command_with_retry",
+        fake_send,
+    )
+
+    resp = await manage_go_mod.manage_gameobject(
+        ctx=DummyContext(),
+        action="modify",
+        target="Ground",
+        is_static="false",
+    )
+
+    assert resp.get("success") is True
+    assert captured["params"]["isStatic"] is False
+
+
 @pytest.mark.asyncio
 async def test_manage_gameobject_is_static_omitted(monkeypatch):
     """Test that omitting is_static does not include isStatic in params."""