@mflrevan/ucp 0.6.2 → 0.6.3

package/bridge/com.ucp.bridge/{Runtime.meta → Editor/Ui.meta}

@@ -1,5 +1,5 @@
 fileFormatVersion: 2
-guid: ed1329cd64b53ba45b1a53c86007094a
+guid: 72ee32817b4f45abac53f471c69e899e
 folderAsset: yes
 DefaultImporter:
   externalObjects: {}

package/bridge/com.ucp.bridge/Tests/Editor/ControllerSmokeTests.cs

@@ -7,6 +7,7 @@ using NUnit.Framework;
 using UnityEditor;
 using UnityEditor.SceneManagement;
 using UnityEngine;
+using UnityEngine.SceneManagement;
 using UnityEngine.Profiling;
 using UnityEngine.TestTools;
 
@@ -189,7 +190,7 @@ namespace UCP.Bridge.Tests
             try
             {
                 go.transform.localPosition = new Vector3(1.5f, 2.5f, 3.5f);
-                var id = go.GetInstanceID();
+                var id = go.GetId();
 
                 var serialized = _router.Dispatch("object/get-property", 1, "{\"instanceId\":ID,\"component\":\"Transform\",\"property\":\"m_LocalPosition\"}".Replace("ID", id.ToString()));
                 Assert.That(serialized.error, Is.Null);
@@ -657,6 +658,80 @@ namespace UCP.Bridge.Tests
             Assert.That(getResult["stackTrace"], Is.EqualTo("stack line 1\nstack line 2"));
         }
 
+        [Test]
+        public void EditorStateSummary_ReportsModeSceneAndConsoleAndCountsNewEntries()
+        {
+            var cursor = LogsController.GetLatestId();
+            var summary = EditorStateSummary.Capture(cursor);
+
+            Assert.That(summary["mode"], Is.EqualTo("edit"));
+            var scene = (Dictionary<string, object>)summary["scene"];
+            Assert.That(scene["name"], Is.EqualTo(SceneManager.GetActiveScene().name));
+            Assert.That(scene["dirty"], Is.EqualTo(SceneManager.GetActiveScene().isDirty));
+            var console = (Dictionary<string, object>)summary["console"];
+            Assert.That(Convert.ToInt32(console["errors"]), Is.GreaterThanOrEqualTo(0));
+            Assert.That(Convert.ToInt32(console["warnings"]), Is.GreaterThanOrEqualTo(0));
+            Assert.That(console.ContainsKey("newWarnings"), Is.False, "nothing was logged after the cursor");
+            Assert.That(summary.ContainsKey("recording"), Is.False);
+
+            LogAssert.Expect(LogType.Warning, "ucp summary probe warning");
+            Debug.LogWarning("ucp summary probe warning");
+
+            var after = (Dictionary<string, object>)EditorStateSummary.Capture(cursor)["console"];
+            Assert.That(Convert.ToInt32(after["newWarnings"]), Is.EqualTo(1));
+            Assert.That(after.ContainsKey("newErrors"), Is.False);
+            LogsController.CountLevels(cursor, out var errors, out var warnings);
+            Assert.That(errors, Is.EqualTo(0));
+            Assert.That(warnings, Is.EqualTo(1));
+        }
+
+        [Test]
+        public void EditorStateSummary_ReadsTheConsoleBadgeCounts()
+        {
+            Assert.That(EditorStateSummary.TryReadConsoleCounts(out var errors, out var warnings), Is.True,
+                "UnityEditor.LogEntries.GetCountsByType must be reachable on this editor");
+            Assert.That(errors, Is.GreaterThanOrEqualTo(0));
+            Assert.That(warnings, Is.GreaterThanOrEqualTo(0));
+        }
+
+        [Test]
+        public void Responses_CarryTheEditorSummaryOnlyWhenDispatchedOnTheMainThread()
+        {
+            var response = JsonRpcResponse.Success(7, new Dictionary<string, object> { ["ok"] = true });
+
+            var offThread = BridgeServer.ResponseToDict(response);
+            Assert.That(offThread.ContainsKey("editor"), Is.False);
+
+            var onThread = BridgeServer.ResponseToDict(response, LogsController.GetLatestId());
+            var editor = (Dictionary<string, object>)onThread["editor"];
+            Assert.That(editor["mode"], Is.EqualTo("edit"));
+            Assert.That(onThread["result"], Is.SameAs(response.result));
+
+            var json = MiniJson.Serialize(onThread);
+            var parsed = (Dictionary<string, object>)MiniJson.Deserialize(json);
+            Assert.That(((Dictionary<string, object>)parsed["editor"]).ContainsKey("console"), Is.True);
+        }
+
+        [Test]
+        public void TestGuard_IgnoresLogsInsideIgnoreFailingMessagesWindows()
+        {
+            var entries = new List<string>
+            {
+                "real error before",
+                "\n" + LogsController.IgnoreFailingMessagesOnMarker,
+                "expected importer error",
+                "expected warning",
+                LogsController.IgnoreFailingMessagesOffMarker + "\n",
+                "real error after",
+                LogsController.IgnoreFailingMessagesOnMarker,
+                "still ignored when a test never resets the flag"
+            };
+
+            var kept = LogsController.ExcludeExpectedNoise(entries, entry => entry);
+
+            Assert.That(kept, Is.EqualTo(new[] { "real error before", "real error after" }));
+        }
+
         [Test]
         public void ObjectLifecycle_CreateMutateAndDelete_WorksEndToEnd()
         {
@@ -664,7 +739,7 @@ namespace UCP.Bridge.Tests
             Assert.That(create.error, Is.Null);
 
             var createResult = (Dictionary<string, object>)create.result;
-            var instanceId = Convert.ToInt32(createResult["instanceId"]);
+            var instanceId = Convert.ToInt64(createResult["instanceId"]);
 
             var rename = _router.Dispatch("object/set-name", 1, "{\"instanceId\":" + instanceId + ",\"name\":\"RenamedSmoke\"}");
             Assert.That(rename.error, Is.Null);
@@ -1376,7 +1451,7 @@ namespace UCP.Bridge.Tests
         {
             var create = _router.Dispatch("object/create", 1, "{\"name\":\"PrefabSource\"}");
             Assert.That(create.error, Is.Null);
-            var sourceId = Convert.ToInt32(((Dictionary<string, object>)create.result)["instanceId"]);
+            var sourceId = Convert.ToInt64(((Dictionary<string, object>)create.result)["instanceId"]);
 
             var createPrefab = _router.Dispatch(
                 "prefab/create",
@@ -1391,7 +1466,7 @@ namespace UCP.Bridge.Tests
                 "{\"prefab\":\"Assets/UcpControllerSmoke.prefab\",\"name\":\"PrefabInstance\"}"
             );
             Assert.That(instantiate.error, Is.Null);
-            var instanceId = Convert.ToInt32(((Dictionary<string, object>)instantiate.result)["instanceId"]);
+            var instanceId = Convert.ToInt64(((Dictionary<string, object>)instantiate.result)["instanceId"]);
 
             var status = _router.Dispatch("prefab/status", 1, "{\"instanceId\":" + instanceId + "}");
             Assert.That(status.error, Is.Null);

package/bridge/com.ucp.bridge/Tests/Editor/SpatialVisualControllerTests.cs

@@ -62,7 +62,7 @@ namespace UCP.Bridge.Tests
         {
             var res = Result(_router.Dispatch("object/create", 1,
                 "{\"name\":\"Greybox\",\"primitive\":\"Cube\"}"));
-            var id = Convert.ToInt32(res["instanceId"]);
+            var id = Convert.ToInt64(res["instanceId"]);
             var go = ObjectLocator.FindByInstanceId(id);
             _spawned.Add(go);
 
@@ -179,7 +179,7 @@ namespace UCP.Bridge.Tests
             var res = Result(_router.Dispatch("physics/raycast", 1,
                 "{\"origin\":[0,5,0],\"direction\":[0,-1,0]}"));
             Assert.That(Convert.ToBoolean(res["hit"]), Is.True);
-            Assert.That(Convert.ToInt32(res["instanceId"]), Is.EqualTo(ground.GetId()));
+            Assert.That(Convert.ToInt64(res["instanceId"]), Is.EqualTo(ground.GetId()));
         }
 
         [Test]