com.valectric.mooserunner 2.1.21 → 2.1.23

package/Samples~/Demos/MooseRunner.Demo.Support/MooseRunnerSampleVersionCheck.cs

@@ -1,57 +1,57 @@
-using System;
-using System.IO;
-using System.Linq;
-using UnityEngine;
-
-namespace MooseRunner.Internal.Tests
-{
-    /// <summary>
-    /// Detects duplicate MooseRunner sample-version folders in Assets/Samples/.
-    /// Unity's UPM Sample import creates a new <c>Assets/Samples/MooseRunner/&lt;version&gt;/...</c>
-    /// folder on every upgrade and never deletes the previous one. The duplicate .cs files
-    /// share class FQNs (e.g. <c>MooseRunner.Internal.Tests.Enemy</c>) and stable .meta GUIDs,
-    /// which produces a "is not derived from MonoBehaviour or ScriptableObject!" runtime
-    /// error when prefabs instantiate. Called from each sample test's OneTimeSetUp so the
-    /// check fires only when the customer actually uses the samples.
-    /// </summary>
-    public static class MooseRunnerSampleVersionCheck
-    {
-        private static bool _alreadyChecked;
-        private const string SamplesRoot = "Assets/Samples/MooseRunner";
-
-        /// <summary>
-        /// Throws <see cref="InvalidOperationException"/> if multiple MooseRunner sample-version
-        /// folders are present. First call performs the scan; subsequent calls within the same
-        /// domain are no-ops (the flag resets on domain reload, which is fine — the check is
-        /// idempotent and cheap).
-        /// </summary>
-        public static void EnsureSingleVersion()
-        {
-            if (_alreadyChecked) return;
-            _alreadyChecked = true;
-
-            if (!Directory.Exists(SamplesRoot)) return;
-
-            var versionDirs = Directory.GetDirectories(SamplesRoot);
-            if (versionDirs.Length <= 1) return;
-
-            var sortedFolders = versionDirs
-                .Select(Path.GetFileName)
-                .OrderBy(n => n)
-                .Select(n => "  " + SamplesRoot + "/" + n);
-
-            string msg =
-                "Multiple MooseRunner sample versions detected. Unity's UPM 'Reimport Sample' " +
-                "creates a new versioned folder on every package upgrade and never deletes the " +
-                "previous one. Duplicate .cs files with the same class FQN and stable .meta GUIDs " +
-                "produce the runtime error \"... is not derived from MonoBehaviour or ScriptableObject!\" " +
-                "when prefabs instantiate.\n" +
-                "\n" +
-                "Found:\n" + string.Join("\n", sortedFolders) + "\n" +
-                "\n" +
-                "Fix: delete all but the newest folder under Assets/Samples/MooseRunner/, then re-run.";
-            Debug.LogError(msg);
-            throw new InvalidOperationException(msg);
-        }
-    }
-}
+using System;
+using System.IO;
+using System.Linq;
+using UnityEngine;
+
+namespace MooseRunner.Internal.Tests
+{
+    /// <summary>
+    /// Detects duplicate MooseRunner sample-version folders in Assets/Samples/.
+    /// Unity's UPM Sample import creates a new <c>Assets/Samples/MooseRunner/&lt;version&gt;/...</c>
+    /// folder on every upgrade and never deletes the previous one. The duplicate .cs files
+    /// share class FQNs (e.g. <c>MooseRunner.Internal.Tests.Enemy</c>) and stable .meta GUIDs,
+    /// which produces a "is not derived from MonoBehaviour or ScriptableObject!" runtime
+    /// error when prefabs instantiate. Called from each sample test's OneTimeSetUp so the
+    /// check fires only when the customer actually uses the samples.
+    /// </summary>
+    public static class MooseRunnerSampleVersionCheck
+    {
+        private static bool _alreadyChecked;
+        private const string SamplesRoot = "Assets/Samples/MooseRunner";
+
+        /// <summary>
+        /// Throws <see cref="InvalidOperationException"/> if multiple MooseRunner sample-version
+        /// folders are present. First call performs the scan; subsequent calls within the same
+        /// domain are no-ops (the flag resets on domain reload, which is fine — the check is
+        /// idempotent and cheap).
+        /// </summary>
+        public static void EnsureSingleVersion()
+        {
+            if (_alreadyChecked) return;
+            _alreadyChecked = true;
+
+            if (!Directory.Exists(SamplesRoot)) return;
+
+            var versionDirs = Directory.GetDirectories(SamplesRoot);
+            if (versionDirs.Length <= 1) return;
+
+            var sortedFolders = versionDirs
+                .Select(Path.GetFileName)
+                .OrderBy(n => n)
+                .Select(n => "  " + SamplesRoot + "/" + n);
+
+            string msg =
+                "Multiple MooseRunner sample versions detected. Unity's UPM 'Reimport Sample' " +
+                "creates a new versioned folder on every package upgrade and never deletes the " +
+                "previous one. Duplicate .cs files with the same class FQN and stable .meta GUIDs " +
+                "produce the runtime error \"... is not derived from MonoBehaviour or ScriptableObject!\" " +
+                "when prefabs instantiate.\n" +
+                "\n" +
+                "Found:\n" + string.Join("\n", sortedFolders) + "\n" +
+                "\n" +
+                "Fix: delete all but the newest folder under Assets/Samples/MooseRunner/, then re-run.";
+            Debug.LogError(msg);
+            throw new InvalidOperationException(msg);
+        }
+    }
+}

package/Samples~/Demos/MooseRunner.Demo.Support/MooseRunnerSampleVersionCheck.cs.meta

@@ -1,2 +1,2 @@
-fileFormatVersion: 2
+fileFormatVersion: 2
 guid: fb03d23dca7f786428fa1f8e5887bf4b

package/Samples~/Demos/MooseRunner.Demo.Tests/ExplicitAttributeTests.cs

@@ -1,36 +1,36 @@
-using NUnit.Framework;
-using UnityEngine;
-
-namespace MooseRunner.Internal.Tests
-{
-    /// <summary>
-    /// Tests to verify [Explicit] attribute handling in MooseRunner.
-    /// - Normal tests should run in batch mode (class/assembly/root selection)
-    /// - [Explicit] tests should only run when directly selected
-    /// </summary>
-    [TestFixture]
-    public class ExplicitAttributeTests
-    {
-        [Test]
-        public void NormalTest_ShouldAlwaysRun()
-        {
-            Debug.Log("NormalTest_ShouldAlwaysRun: This test runs in all scenarios.");
-            Assert.Pass("Normal test passed");
-        }
-
-        [Test]
-        [Explicit("This test should only run when explicitly selected")]
-        public void ExplicitTest_ShouldOnlyRunWhenDirectlySelected()
-        {
-            Debug.Log("ExplicitTest_ShouldOnlyRunWhenDirectlySelected: This test should only run when directly selected.");
-            Assert.Pass("Explicit test passed - this means it was directly selected");
-        }
-
-        [Test]
-        public void AnotherNormalTest_ShouldAlwaysRun()
-        {
-            Debug.Log("AnotherNormalTest_ShouldAlwaysRun: This is another normal test.");
-            Assert.Pass("Another normal test passed");
-        }
-    }
-}
+using NUnit.Framework;
+using UnityEngine;
+
+namespace MooseRunner.Internal.Tests
+{
+    /// <summary>
+    /// Tests to verify [Explicit] attribute handling in MooseRunner.
+    /// - Normal tests should run in batch mode (class/assembly/root selection)
+    /// - [Explicit] tests should only run when directly selected
+    /// </summary>
+    [TestFixture]
+    public class ExplicitAttributeTests
+    {
+        [Test]
+        public void NormalTest_ShouldAlwaysRun()
+        {
+            Debug.Log("NormalTest_ShouldAlwaysRun: This test runs in all scenarios.");
+            Assert.Pass("Normal test passed");
+        }
+
+        [Test]
+        [Explicit("This test should only run when explicitly selected")]
+        public void ExplicitTest_ShouldOnlyRunWhenDirectlySelected()
+        {
+            Debug.Log("ExplicitTest_ShouldOnlyRunWhenDirectlySelected: This test should only run when directly selected.");
+            Assert.Pass("Explicit test passed - this means it was directly selected");
+        }
+
+        [Test]
+        public void AnotherNormalTest_ShouldAlwaysRun()
+        {
+            Debug.Log("AnotherNormalTest_ShouldAlwaysRun: This is another normal test.");
+            Assert.Pass("Another normal test passed");
+        }
+    }
+}

package/Samples~/Demos/MooseRunner.Demo.Tests/InstanceHandlingVerificationTest.cs

@@ -1,100 +1,100 @@
-using System;
-using System.Collections;
-using System.Collections.Generic;
-using NUnit.Framework;
-using UnityEngine;
-using UnityEngine.TestTools;
-
-namespace MooseRunner.Internal.Tests
-{
-    /// <summary>
-    /// Test class to verify that OneTimeSetUp and OneTimeTearDown methods get fresh instances
-    /// while regular test methods (SetUp/Test/TearDown) share the same instance within a test cycle.
-    /// Uses _instanceId with default value 99 to track instance persistence.
-    /// </summary>
-    public class InstanceHandlingVerificationTest
-    {
-
-        // Instance identifier - defaults to 99, each method should verify the expected previous value
-        private int _instanceId = 99;
-
-        // Constructor called when creating a new instance
-        public InstanceHandlingVerificationTest()
-        {
-            Debug.Log($"Constructor called, _instanceId set to {_instanceId}");
-        }
-
-        [OneTimeSetUp]
-        public void OneTimeSetUpMethod()
-        {
-            MooseRunnerSampleVersionCheck.EnsureSingleVersion();
-            // OneTimeSetUp should get a fresh instance, so _instanceId should be 99 (default)
-            Assert.AreEqual(99, _instanceId, "OneTimeSetUp should start with default _instanceId = 99 (fresh instance)");
-
-            _instanceId = 100; // Set to new value
-            Debug.Log($"OneTimeSetUp: verified _instanceId was 99, changed to {_instanceId}");
-        }
-
-        [SetUp]
-        public void SetUpMethod()
-        {
-            // SetUp should get a fresh instance for each test, so _instanceId should be 99 (default)
-            Assert.AreEqual(99, _instanceId, "SetUp should start with default _instanceId = 99 (fresh instance for each test)");
-
-            _instanceId = 200; // Set to new value for test methods to verify
-            Debug.Log($"SetUp: verified _instanceId was 99, changed to {_instanceId}");
-        }
-
-        [Test]
-        public void FirstTest()
-        {
-            // Test should use same instance as SetUp, so _instanceId should be 200
-            Assert.AreEqual(200, _instanceId, "FirstTest should see _instanceId = 200 from SetUp (same instance)");
-
-            _instanceId = 300; // Change for TearDown to verify
-            Debug.Log($"FirstTest: verified _instanceId was 200, changed to {_instanceId}");
-        }
-
-        [Test]
-        public void SecondTest()
-        {
-            // Test should use same instance as SetUp, so _instanceId should be 200
-            Assert.AreEqual(200, _instanceId, "SecondTest should see _instanceId = 200 from SetUp (same instance)");
-
-            _instanceId = 400; // Change for TearDown to verify
-            Debug.Log($"SecondTest: verified _instanceId was 200, changed to {_instanceId}");
-        }
-
-        [UnityTest]
-        public IEnumerator CoroutineTest()
-        {
-            // Test should use same instance as SetUp, so _instanceId should be 200
-            Assert.AreEqual(200, _instanceId, "CoroutineTest should see _instanceId = 200 from SetUp (same instance)");
-
-            _instanceId = 500; // Change for TearDown to verify
-            Debug.Log($"CoroutineTest: verified _instanceId was 200, changed to {_instanceId}");
-            yield return new WaitForSeconds(0.1f);
-        }
-
-        [TearDown]
-        public void TearDownMethod()
-        {
-            // TearDown should use same instance as the test method, so _instanceId should be what the test set
-            // (300 for FirstTest, 400 for SecondTest, 500 for CoroutineTest)
-            Assert.IsTrue(_instanceId == 300 || _instanceId == 400 || _instanceId == 500,
-                $"TearDown should see _instanceId from test method (300, 400, or 500), but got {_instanceId}");
-            
-            Debug.Log($"TearDown: verified _instanceId was {_instanceId} (from test method - same instance)");
-        }
-
-        [OneTimeTearDown]
-        public void OneTimeTearDownMethod()
-        {
-            // OneTimeTearDown should get a fresh instance, so _instanceId should be 99 (default)
-            Assert.AreEqual(99, _instanceId, "OneTimeTearDown should start with default _instanceId = 99 (fresh instance)");
-            
-            Debug.Log($"OneTimeTearDown: verified _instanceId was 99 (fresh instance)");
-            
-        }
-    }
+using System;
+using System.Collections;
+using System.Collections.Generic;
+using NUnit.Framework;
+using UnityEngine;
+using UnityEngine.TestTools;
+
+namespace MooseRunner.Internal.Tests
+{
+    /// <summary>
+    /// Test class to verify that OneTimeSetUp and OneTimeTearDown methods get fresh instances
+    /// while regular test methods (SetUp/Test/TearDown) share the same instance within a test cycle.
+    /// Uses _instanceId with default value 99 to track instance persistence.
+    /// </summary>
+    public class InstanceHandlingVerificationTest
+    {
+
+        // Instance identifier - defaults to 99, each method should verify the expected previous value
+        private int _instanceId = 99;
+
+        // Constructor called when creating a new instance
+        public InstanceHandlingVerificationTest()
+        {
+            Debug.Log($"Constructor called, _instanceId set to {_instanceId}");
+        }
+
+        [OneTimeSetUp]
+        public void OneTimeSetUpMethod()
+        {
+            MooseRunnerSampleVersionCheck.EnsureSingleVersion();
+            // OneTimeSetUp should get a fresh instance, so _instanceId should be 99 (default)
+            Assert.AreEqual(99, _instanceId, "OneTimeSetUp should start with default _instanceId = 99 (fresh instance)");
+
+            _instanceId = 100; // Set to new value
+            Debug.Log($"OneTimeSetUp: verified _instanceId was 99, changed to {_instanceId}");
+        }
+
+        [SetUp]
+        public void SetUpMethod()
+        {
+            // SetUp should get a fresh instance for each test, so _instanceId should be 99 (default)
+            Assert.AreEqual(99, _instanceId, "SetUp should start with default _instanceId = 99 (fresh instance for each test)");
+
+            _instanceId = 200; // Set to new value for test methods to verify
+            Debug.Log($"SetUp: verified _instanceId was 99, changed to {_instanceId}");
+        }
+
+        [Test]
+        public void FirstTest()
+        {
+            // Test should use same instance as SetUp, so _instanceId should be 200
+            Assert.AreEqual(200, _instanceId, "FirstTest should see _instanceId = 200 from SetUp (same instance)");
+
+            _instanceId = 300; // Change for TearDown to verify
+            Debug.Log($"FirstTest: verified _instanceId was 200, changed to {_instanceId}");
+        }
+
+        [Test]
+        public void SecondTest()
+        {
+            // Test should use same instance as SetUp, so _instanceId should be 200
+            Assert.AreEqual(200, _instanceId, "SecondTest should see _instanceId = 200 from SetUp (same instance)");
+
+            _instanceId = 400; // Change for TearDown to verify
+            Debug.Log($"SecondTest: verified _instanceId was 200, changed to {_instanceId}");
+        }
+
+        [UnityTest]
+        public IEnumerator CoroutineTest()
+        {
+            // Test should use same instance as SetUp, so _instanceId should be 200
+            Assert.AreEqual(200, _instanceId, "CoroutineTest should see _instanceId = 200 from SetUp (same instance)");
+
+            _instanceId = 500; // Change for TearDown to verify
+            Debug.Log($"CoroutineTest: verified _instanceId was 200, changed to {_instanceId}");
+            yield return new WaitForSeconds(0.1f);
+        }
+
+        [TearDown]
+        public void TearDownMethod()
+        {
+            // TearDown should use same instance as the test method, so _instanceId should be what the test set
+            // (300 for FirstTest, 400 for SecondTest, 500 for CoroutineTest)
+            Assert.IsTrue(_instanceId == 300 || _instanceId == 400 || _instanceId == 500,
+                $"TearDown should see _instanceId from test method (300, 400, or 500), but got {_instanceId}");
+            
+            Debug.Log($"TearDown: verified _instanceId was {_instanceId} (from test method - same instance)");
+        }
+
+        [OneTimeTearDown]
+        public void OneTimeTearDownMethod()
+        {
+            // OneTimeTearDown should get a fresh instance, so _instanceId should be 99 (default)
+            Assert.AreEqual(99, _instanceId, "OneTimeTearDown should start with default _instanceId = 99 (fresh instance)");
+            
+            Debug.Log($"OneTimeTearDown: verified _instanceId was 99 (fresh instance)");
+            
+        }
+    }
 }

package/Third Party Notices.md

@@ -1,37 +1,37 @@
-# Third Party Notices
-
-MooseRunner depends on the following third-party components. Each is
-distributed under its own license, summarized below. The full text of each
-license is available at the linked source.
-
-## Customer-installed dependencies
-
-These are declared as UPM `dependencies` in `package.json` and installed by
-the Package Manager into the customer's project. We do not redistribute
-their binaries.
-
-- **Newtonsoft.Json** (`com.unity.nuget.newtonsoft-json`) — MIT License.
-  https://github.com/JamesNK/Newtonsoft.Json
-- **UniTask** (`com.cysharp.unitask`) — MIT License.
-  https://github.com/Cysharp/UniTask
-
-## Optional integrations
-
-If the customer has any of the following installed, MooseRunner integrates
-with them. We do not redistribute their binaries.
-
-- **Hot Reload for Unity** (`com.singularitygroup.hotreload`) — commercial,
-  Singularity Group. https://hotreload.net
-- **Unity Recorder** (`com.unity.recorder`) — Unity Companion License.
-  https://docs.unity3d.com/Packages/com.unity.recorder@latest
-- **ParrelSync** (`com.veriorpies.parrelsync`) — MIT License.
-  https://github.com/VeriorPies/ParrelSync
-- **MCP-for-Unity** (`com.coplaydev.unity-mcp`) — used only as a defensive
-  hook (CloneBootstrapper prevents MCP auto-start in ParrelSync clones).
-  https://github.com/CoplayDev/unity-mcp
-
-## NUnit
-
-Tests use NUnit 3 via Unity's Test Framework
-(`com.unity.test-framework`) — MIT License.
-https://github.com/nunit/nunit
+# Third Party Notices
+
+MooseRunner depends on the following third-party components. Each is
+distributed under its own license, summarized below. The full text of each
+license is available at the linked source.
+
+## Customer-installed dependencies
+
+These are declared as UPM `dependencies` in `package.json` and installed by
+the Package Manager into the customer's project. We do not redistribute
+their binaries.
+
+- **Newtonsoft.Json** (`com.unity.nuget.newtonsoft-json`) — MIT License.
+  https://github.com/JamesNK/Newtonsoft.Json
+- **UniTask** (`com.cysharp.unitask`) — MIT License.
+  https://github.com/Cysharp/UniTask
+
+## Optional integrations
+
+If the customer has any of the following installed, MooseRunner integrates
+with them. We do not redistribute their binaries.
+
+- **Hot Reload for Unity** (`com.singularitygroup.hotreload`) — commercial,
+  Singularity Group. https://hotreload.net
+- **Unity Recorder** (`com.unity.recorder`) — Unity Companion License.
+  https://docs.unity3d.com/Packages/com.unity.recorder@latest
+- **ParrelSync** (`com.veriorpies.parrelsync`) — MIT License.
+  https://github.com/VeriorPies/ParrelSync
+- **MCP-for-Unity** (`com.coplaydev.unity-mcp`) — used only as a defensive
+  hook (CloneBootstrapper prevents MCP auto-start in ParrelSync clones).
+  https://github.com/CoplayDev/unity-mcp
+
+## NUnit
+
+Tests use NUnit 3 via Unity's Test Framework
+(`com.unity.test-framework`) — MIT License.
+https://github.com/nunit/nunit

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "com.valectric.mooserunner",
-  "version": "2.1.21",
+  "version": "2.1.23",
   "displayName": "MooseRunner",
   "description": "MooseRunner boosts PlayMode testing with Hot Reload, MCP for AIs, timescale control, and Task support. It enables rapid iteration, auto test reruns, full documentation and dedicated support.",
   "unity": "6000.2",