@hybridlabor-api/aos 4.3.0 → 4.4.0

package/mcps/RhinoMCP/tests/Integration.Tests/SpawnSlotTests.cs

@@ -1,66 +0,0 @@
-using RhMcp.Integration.Tests.Harness;
-
-namespace RhMcp.Integration.Tests;
-
-// Exercises spawn_slot end-to-end: the router must launch a real Rhino,
-// receive its listener announcement, and return the slot metadata.
-[TestFixture]
-internal sealed class SpawnSlotTests : RouterFixture
-{
-
-    [TestCase("8")]
-    [TestCase("9")]
-    [TestCase("WIP")]
-    public async Task spawn_slot_returns_slot_metadata(string version)
-    {
-        ReturnResult result = await _router.CallToolAsync("spawn_slot", Args.Of(("version", version)));
-
-        Assert.That(result.Payload?.GetProperty("slotId").GetString(), Is.Not.Empty);
-        Assert.That(result.Payload?.GetProperty("version").GetString(), Is.EqualTo(version));
-        Assert.That(result.Payload?.GetProperty("adopted").GetBoolean(), Is.False);
-    }
-
-    // Regression: a version sent as a JSON number must coerce and spawn, not fail
-    // string binding with a bare invocation error.
-    [Test]
-    public async Task spawn_slot_accepts_numeric_version()
-    {
-        // 8 is passed as a JSON number, not the string "8".
-        ReturnResult result = await _router.CallToolAsync("spawn_slot", Args.Of(("version", 8)));
-
-        Assert.That(result.Error, Is.Null);
-        Assert.That(result.Payload?.GetProperty("slotId").GetString(), Is.Not.Empty);
-        Assert.That(result.Payload?.GetProperty("version").GetString(), Is.EqualTo("8"));
-    }
-
-    [Test]
-    public async Task spawn_three_slots_returns_distinct_metadata()
-    {
-        ReturnResult r1 = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-        ReturnResult r2 = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-        ReturnResult r3 = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-
-        foreach (ReturnResult result in new[] { r1, r2, r3 })
-        {
-            Assert.That(result.Payload?.GetProperty("slotId").GetString(), Is.Not.Empty);
-            Assert.That(result.Payload?.GetProperty("version").GetString(), Is.EqualTo("8"));
-            Assert.That(result.Payload?.GetProperty("adopted").GetBoolean(), Is.False);
-        }
-    }
-
-    // Round-trip: spawn produces a slotId that close_slot will accept.
-    [Test]
-    public async Task spawn_then_close_slot_round_trip()
-    {
-        ReturnResult open = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-
-        Assert.That(open.Payload?.GetProperty("slotId").GetString(), Is.Not.Empty);
-        Assert.That(open.Payload?.GetProperty("version").GetString(), Is.EqualTo("8"));
-        Assert.That(open.Payload?.GetProperty("adopted").GetBoolean(), Is.False);
-
-        string slotId = open.Payload!.Value.GetProperty("slotId").GetString()!;
-
-        ReturnResult close = await _router.CallToolAsync("close_slot", Args.Of(("slot", slotId)));
-        Assert.That(close.Payload?.GetProperty("closed").GetBoolean(), Is.True);
-    }
-}

package/mcps/RhinoMCP/tests/Integration.Tests/ToolDispatchBySlotTests.cs

@@ -1,53 +0,0 @@
-using RhMcp.Integration.Tests.Harness;
-
-namespace RhMcp.Integration.Tests;
-
-// Pins down slot-routing semantics for plugin-side tool calls:
-//   - An explicit `slot` arg must route to that exact Rhino.
-//   - An unknown `slot` arg must produce a structured slot_not_found payload
-//     (not a hang, and not a generic MCP error).
-[TestFixture]
-public sealed class ToolDispatchBySlotTests : RouterFixture
-{
-    [Test]
-    public async Task explicit_slot_routes_to_correct_rhino()
-    {
-        ReturnResult spawnA = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-        ReturnResult spawnB = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-        string slotA = spawnA.Payload!.Value.GetProperty("slotId").GetString()!;
-        string slotB = spawnB.Payload!.Value.GetProperty("slotId").GetString()!;
-
-        // Drop three lines into slot A; leave slot B untouched.
-        _ = await _router.CallToolAsync("run_python", Args.Of(
-            ("slot", (object?)slotA),
-            ("script", """
-                from Rhino.Geometry import Point3d, Line
-                doc = __rhino_doc__
-                for i in range(3):
-                    doc.Objects.AddLine(Line(Point3d(i, 0, 0), Point3d(i, 1, 0)))
-                """)));
-
-        ReturnResult listA = await _router.CallToolAsync("list_objects", Args.Of(("slot", slotA)));
-        ReturnResult listB = await _router.CallToolAsync("list_objects", Args.Of(("slot", slotB)));
-
-        Assert.Multiple((Action)(() =>
-        {
-            Assert.That(listA.Payload?.GetProperty("count").GetInt32(), Is.EqualTo(3));
-            Assert.That(listB.Payload?.GetProperty("count").GetInt32(), Is.EqualTo(0));
-        }));
-    }
-
-    [Test]
-    public async Task tool_call_with_unknown_slot_returns_slot_not_found_payload()
-    {
-        // No spawn — just call a plugin tool with a bogus slot id. The router
-        // must short-circuit in the dispatcher with a structured error, not
-        // attempt to auto-spawn and not hang.
-        ReturnResult response = await _router.CallToolAsync(
-            "list_objects",
-            Args.Of(("slot", "made-up-slot-xyz")));
-
-        Assert.That(response.Error?.Code, Is.EqualTo("slot_not_found"));
-        Assert.That(response.Error?.Message, Does.Contain("made-up-slot-xyz"));
-    }
-}

package/mcps/RhinoMCP/tests/Integration.Tests/WindowsProcessIsolationTests.cs

@@ -1,62 +0,0 @@
-using System.Text.Json;
-using RhMcp.Integration.Tests.Harness;
-
-namespace RhMcp.Integration.Tests;
-
-// Pins down the Windows-specific contract: every slot gets its own OS process,
-// regardless of version. This is the inverse of MacSharedProcessTests and is
-// the simpler invariant to verify — if either test ever fails, the shared-vs-
-// isolated branch in RhinoManager has drifted. [Platform(Win)] excludes the
-// case from macOS where the contract is the opposite.
-[TestFixture]
-[Platform("Win")]
-public sealed class WindowsProcessIsolationTests : RouterFixture
-{
-    [Test]
-    public async Task two_slots_same_version_have_distinct_pids_and_ports()
-    {
-        _ = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-        _ = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-
-        ReturnResult list = await _router.CallToolAsync("list_slots");
-        Assert.That(list.Payload?.GetArrayLength(), Is.EqualTo(2));
-
-        List<JsonElement> slots = list.Payload!.Value.EnumerateArray().ToList();
-        HashSet<int> pids = slots.Select(s => s.GetProperty("pid").GetInt32()).ToHashSet();
-        HashSet<int> ports = slots.Select(s => s.GetProperty("port").GetInt32()).ToHashSet();
-
-        Assert.That(pids, Has.Count.EqualTo(2), "Windows slots must be backed by distinct Rhino processes.");
-        Assert.That(ports, Has.Count.EqualTo(2));
-    }
-
-    [Test]
-    public async Task closing_one_slot_does_not_kill_the_other()
-    {
-        ReturnResult spawnA = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-        ReturnResult spawnB = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-        string slotA = spawnA.Payload!.Value.GetProperty("slotId").GetString()!;
-        string slotB = spawnB.Payload!.Value.GetProperty("slotId").GetString()!;
-        int pidB = spawnB.Payload.Value.GetProperty("pid").GetInt32();
-
-        ReturnResult close = await _router.CallToolAsync("close_slot", Args.Of(("slot", slotA)));
-        Assert.That(close.Payload?.GetProperty("closed").GetBoolean(), Is.True);
-
-        ReturnResult list = await _router.CallToolAsync("list_slots");
-        Assert.That(list.Payload?.GetArrayLength(), Is.EqualTo(1));
-
-        JsonElement remaining = list.Payload!.Value[0];
-        Assert.That(remaining.GetProperty("slotId").GetString(), Is.EqualTo(slotB));
-        Assert.That(IsProcessAlive(pidB), Is.True, "Sibling slot's Rhino must outlive close_slot on a peer.");
-    }
-
-    private static bool IsProcessAlive(int pid)
-    {
-        try
-        {
-            using System.Diagnostics.Process p = System.Diagnostics.Process.GetProcessById(pid);
-            return !p.HasExited;
-        }
-        catch (ArgumentException) { return false; }
-        catch (InvalidOperationException) { return false; }
-    }
-}

package/mcps/RhinoMCP/tests/Router.Tests/CodegenTests.cs

@@ -1,125 +0,0 @@
-using System.Linq;
-using System.Threading;
-using Microsoft.CodeAnalysis;
-using Microsoft.CodeAnalysis.CSharp;
-using Microsoft.CodeAnalysis.Text;
-using NUnit.Framework;
-using RhMcp.Router.Codegen;
-
-namespace RhMcp.Router.Tests;
-
-// Regression guard for the router source generator. Plugin tool sites write
-// [McpServerTool(...)] positionally (name, title, readOnly, destructive). An
-// earlier version only read named attribute args, so every tool's name came
-// back null and zero proxies were emitted. These tests run the generator over
-// synthetic tool files and assert a proxy IS produced.
-[TestFixture]
-public class CodegenTests
-{
-    [Test]
-    public void Emits_proxy_for_positional_attribute()
-    {
-        string output = RunGenerator(
-            """
-            namespace RhMcp.Tools;
-
-            [McpServerToolType]
-            public class XTool
-            {
-                [McpServerTool("x", "X", true, false)]
-                public static string Run(RhinoDoc doc) => "ok";
-            }
-            """);
-
-        Assert.That(output, Does.Contain("public class XToolProxy"));
-        Assert.That(output, Does.Contain("Name = \"x\""));
-        Assert.That(output, Does.Contain("Title = \"X\""));
-        Assert.That(output, Does.Contain("ReadOnly = true"));
-        Assert.That(output, Does.Contain("Destructive = false"));
-        // Registrar must wire the proxy up so Program.cs picks it up.
-        Assert.That(output, Does.Contain("WithTools<global::RhMcp.Router.Tools.Generated.XToolProxy>"));
-    }
-
-    [Test]
-    public void Named_argument_overrides_positional_slot()
-    {
-        string output = RunGenerator(
-            """
-            namespace RhMcp.Tools;
-
-            [McpServerToolType]
-            public class YTool
-            {
-                [McpServerTool("ignored", Name = "real_name", Destructive = true)]
-                public static string Run(RhinoDoc doc) => "ok";
-            }
-            """);
-
-        Assert.That(output, Does.Contain("Name = \"real_name\""));
-        Assert.That(output, Does.Not.Contain("Name = \"ignored\""));
-        Assert.That(output, Does.Contain("Destructive = true"));
-    }
-
-    [Test]
-    public void Folds_concatenated_description_literals()
-    {
-        // AskUserTool builds its [Description] with + concatenation across lines.
-        // The generator must fold the constant string so the proxy carries the full
-        // text, not an empty Description("").
-        string output = RunGenerator(
-            """
-            namespace RhMcp.Tools;
-
-            [McpServerToolType]
-            public class ZTool
-            {
-                [McpServerTool("z", "Z", true, false)]
-                [Description("first part " + "second part " + "third part")]
-                public static string Run(
-                    RhinoDoc doc,
-                    [Description("flag on " + "two lines")] bool flag = false) => "ok";
-            }
-            """);
-
-        Assert.That(output, Does.Contain("Description(\"first part second part third part\")"));
-        Assert.That(output, Does.Contain("Description(\"flag on two lines\")"));
-        Assert.That(output, Does.Not.Contain("Description(\"\")"));
-    }
-
-    private static string RunGenerator(string toolSource)
-    {
-        SyntaxTree compilationTree = CSharpSyntaxTree.ParseText("// placeholder compilation unit");
-        CSharpCompilation compilation = CSharpCompilation.Create(
-            "RouterCodegenTest",
-            [compilationTree],
-            references: [],
-            new CSharpCompilationOptions(OutputKind.DynamicallyLinkedLibrary));
-
-        // The generator only scans AdditionalFiles whose path contains /plugin/Tools/.
-        AdditionalText toolFile = new InMemoryAdditionalText(
-            "/repo/rhino/plugin/Tools/SyntheticTool.cs",
-            toolSource);
-
-        CSharpGeneratorDriver driver = CSharpGeneratorDriver.Create(
-            generators: [new RouterToolGenerator().AsSourceGenerator()],
-            additionalTexts: [toolFile]);
-
-        GeneratorDriverRunResult result = driver.RunGenerators(compilation).GetRunResult();
-        Assert.That(result.Diagnostics, Is.Empty);
-
-        GeneratedSourceResult generated = result.Results
-            .Single()
-            .GeneratedSources
-            .Single();
-
-        return generated.SourceText.ToString();
-    }
-
-    private sealed class InMemoryAdditionalText(string path, string content) : AdditionalText
-    {
-        public override string Path { get; } = path;
-
-        public override SourceText GetText(CancellationToken cancellationToken = default)
-            => SourceText.From(content);
-    }
-}

package/mcps/RhinoMCP/tests/Router.Tests/ReturnResultTests.cs

@@ -1,90 +0,0 @@
-using System.Text.Json.Nodes;
-using NUnit.Framework;
-using RhMcp.Router;
-
-namespace RhMcp.Router.Tests;
-
-// Pins the wire shape of the ReturnResult envelope so the agent contract can't
-// regress silently. The serializer is RouterJsonContext (AOT source-gen) — the
-// shape these tests assert is exactly what hits the MCP transport, no separate
-// JsonSerializerOptions layer in between.
-[TestFixture]
-public class ReturnResultTests
-{
-    [Test]
-    public void Success_with_only_payload_omits_error_and_autoSpawnedSlot_fields()
-    {
-        // The JsonIgnoreCondition.WhenWritingNull policy declared on
-        // RouterJsonContext is what keeps the envelope tidy. If someone changes
-        // the policy or forgets to apply it to ReturnResult, the failure path
-        // would still work but every success result would carry "error":null
-        // and "autoSpawnedSlot":null — noise the agent has to ignore.
-        ReturnResult result = new(
-            Payload: JsonValue.Create("ok"),
-            Error: null,
-            AutoSpawnedSlot: null);
-
-        string json = result.AsJson;
-
-        Assert.That(json, Does.Contain("\"payload\""));
-        Assert.That(json, Does.Not.Contain("\"error\""));
-        Assert.That(json, Does.Not.Contain("\"autoSpawnedSlot\""));
-        // Belt and braces: the literal "null" must not appear in the envelope
-        // for the absent fields either, regardless of property-name policy.
-        Assert.That(json, Does.Not.Contain(":null"));
-    }
-
-    [Test]
-    public void Failure_with_only_error_omits_payload_and_autoSpawnedSlot_fields()
-    {
-        ReturnResult result = new(
-            Payload: null,
-            Error: new ErrorInfo(Code: "slot_not_found", Message: "..."),
-            AutoSpawnedSlot: null);
-
-        string json = result.AsJson;
-
-        Assert.That(json, Does.Contain("\"error\""));
-        Assert.That(json, Does.Not.Contain("\"payload\""));
-        Assert.That(json, Does.Not.Contain("\"autoSpawnedSlot\""));
-    }
-
-    [Test]
-    public void ErrorInfo_with_null_crashReportPath_omits_that_field()
-    {
-        // CrashReportPath is the optional sub-field on ErrorInfo. Same policy
-        // must reach the nested record; otherwise non-crash errors carry a
-        // misleading "crashReportPath":null.
-        ReturnResult result = new(
-            Payload: null,
-            Error: new ErrorInfo(Code: "tool_call_failed", Message: "something went wrong"),
-            AutoSpawnedSlot: null);
-
-        string json = result.AsJson;
-
-        Assert.That(json, Does.Not.Contain("\"crashReportPath\""));
-    }
-
-    [Test]
-    public void Auto_spawn_side_channel_serializes_alongside_payload()
-    {
-        // Auto-spawn populates the side channel on success — the dispatcher uses
-        // this on the very tool call that triggered the spawn. Both fields must
-        // appear; payload is the tool's normal output, autoSpawnedSlot is the
-        // one-shot notification.
-        ReturnResult result = new(
-            Payload: JsonValue.Create("done"),
-            Error: null,
-            AutoSpawnedSlot: new SlotInfo(
-                SlotId: "armadillo",
-                Version: "WIP",
-                Reason: "Auto-spawned Rhino WIP to serve 'g2_search_components'."));
-
-        string json = result.AsJson;
-
-        Assert.That(json, Does.Contain("\"payload\""));
-        Assert.That(json, Does.Contain("\"autoSpawnedSlot\""));
-        Assert.That(json, Does.Contain("\"slotId\":\"armadillo\""));
-        Assert.That(json, Does.Not.Contain("\"error\""));
-    }
-}

package/mcps/RhinoMCP/tests/Router.Tests/RhinoCrashReportFinderTests.cs

@@ -1,207 +0,0 @@
-using System.Runtime.InteropServices;
-using Microsoft.Extensions.Logging.Abstractions;
-using NUnit.Framework;
-using RhMcp.Router;
-
-namespace RhMcp.Router.Tests;
-
-[TestFixture]
-public class RhinoCrashReportFinderTests
-{
-    // Hand-crafted minidump exercises the Windows parser without needing a real
-    // Rhino crash on disk. Verifies pid extraction (from the MiscInfo stream)
-    // and exception-code → Signal/Termination mapping. Drops the file into the
-    // WER LocalDumps path that TryFindWindows scans, so the test also covers
-    // the directory-discovery half of the code path.
-    [Test]
-    public void TryFind_on_windows_parses_synthetic_minidump()
-    {
-        if (!RuntimeInformation.IsOSPlatform(OSPlatform.Windows)) return;
-
-        const int FakePid = 4242;
-        const uint AccessViolation = 0xC0000005;
-
-        string werDir = Path.Combine(
-            Environment.GetFolderPath(Environment.SpecialFolder.LocalApplicationData),
-            "CrashDumps");
-        Directory.CreateDirectory(werDir);
-        string dumpPath = Path.Combine(werDir, $"Rhino.exe.{FakePid}.synthetic-{Guid.NewGuid():N}.dmp");
-
-        try
-        {
-            File.WriteAllBytes(dumpPath, BuildMinidump(FakePid, AccessViolation));
-
-            var finder = new RhinoCrashReportFinder(NullLogger<RhinoCrashReportFinder>.Instance);
-            var report = finder.TryFind(FakePid);
-
-            Assert.That(report, Is.Not.Null);
-            Assert.That(report!.Path, Is.EqualTo(dumpPath));
-            Assert.That(report.Signal, Is.EqualTo("0xC0000005"));
-            Assert.That(report.Termination, Is.EqualTo("EXCEPTION_ACCESS_VIOLATION"));
-            Assert.That(report.CaptureTime, Is.Not.Null);
-            Assert.That(report.ManagedFrames, Is.Empty);
-            Assert.That(report.TopFrames, Is.Empty);
-        }
-        finally
-        {
-            try { File.Delete(dumpPath); } catch { }
-        }
-    }
-
-    // Verbatim sample of what Rhino's UnhandledException handler drops on the
-    // desktop when an SEHException escapes the WPF message loop. Tests the
-    // full path: file IO, FATAL UNHANDLED EXCEPTION extraction, frame capture,
-    // and Windows-style build-path stripping.
-    [Test]
-    public void TryParseDotNetCrash_extracts_exception_and_frames()
-    {
-        const string sample =
-            "[ERROR] FATAL UNHANDLED EXCEPTION: ExampleApp.SomeException (0x80004005): External component has thrown an exception.\n" +
-            "   at ExampleApp.NativeMethods.DoNativeCall(String token, Boolean flag)\n" +
-            "   at ExampleApp.UI.CommandPanel.PostToken(String token, Boolean flag) in C:\\BuildAgent\\work\\src\\CommandPanel.cs:line 572\n" +
-            "   at ExampleApp.UI.CommandPanel.Execute() in C:\\BuildAgent\\work\\src\\CommandPanel.cs:line 641\n" +
-            "   at ExampleApp.UI.CommandPanel.KeyDown(Object sender, KeyEventArgs e) in C:\\BuildAgent\\work\\src\\CommandPanel.cs:line 663\n" +
-            "   at ExampleApp.Forms.Control.TriggerEvent[T](Object key, Object sender, T args) in C:\\BuildAgent\\work\\src\\Control.cs:line 231\n" +
-            "   at ExampleApp.Forms.Control.OnKeyDown(KeyEventArgs e) in C:\\BuildAgent\\work\\src\\Control.cs:line 149\n" +
-            "   at ExampleApp.Forms.Control.Callback.OnKeyDown(Control widget, KeyEventArgs e) in C:\\BuildAgent\\work\\src\\Control.cs:line 1590\n" +
-            "   at ExampleApp.Wpf.FrameworkElement`3.HandleKeyDown(Object sender, KeyEventArgs e) in C:\\BuildAgent\\work\\src\\FrameworkElement.cs:line 794\n" +
-            "   at System.Windows.RoutedEventArgs.InvokeHandler(Delegate handler, Object target)\n" +
-            "   at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)\n" +
-            "[END ERROR]\n";
-
-        string path = Path.Combine(Path.GetTempPath(), $"RhinoDotNetCrash-test-{Guid.NewGuid():N}.txt");
-        File.WriteAllText(path, sample);
-
-        try
-        {
-            var finder = new RhinoCrashReportFinder(NullLogger<RhinoCrashReportFinder>.Instance);
-            var report = finder.TryParseDotNetCrash(path);
-
-            Assert.That(report, Is.Not.Null);
-            Assert.That(report!.Path, Is.EqualTo(path));
-            Assert.That(report.CaptureTime, Is.Not.Null);
-            Assert.That(
-                report.ManagedException,
-                Is.EqualTo("ExampleApp.SomeException (0x80004005): External component has thrown an exception."));
-
-            // Frames are capped at 10. Native frames keep no path; frames with
-            // " in C:\\..." get rewritten to "(filename:line)".
-            Assert.That(report.ManagedFrames.Length, Is.EqualTo(10));
-            Assert.That(report.ManagedFrames, Is.All.StartWith("at "));
-            Assert.That(
-                report.ManagedFrames[0],
-                Is.EqualTo("at ExampleApp.NativeMethods.DoNativeCall(String token, Boolean flag)"));
-            Assert.That(report.ManagedFrames[1], Does.Contain("(CommandPanel.cs:572)"));
-            Assert.That(report.ManagedFrames[7], Does.Contain("(FrameworkElement.cs:794)"));
-            // Build-machine path must be stripped from every frame.
-            Assert.That(report.ManagedFrames, Has.None.Matches<string>(f => f.Contains("C:\\BuildAgent")));
-
-            // .txt has no native stack or signal — those fields stay null/empty.
-            Assert.That(report.Signal, Is.Null);
-            Assert.That(report.Termination, Is.Null);
-            Assert.That(report.TopFrames, Is.Empty);
-        }
-        finally
-        {
-            try { File.Delete(path); } catch { }
-        }
-    }
-
-    // Minimal minidump: header + 2-entry directory (Exception @ stream type 6,
-    // MiscInfo @ stream type 15). Parser only reads the leading bytes of each
-    // stream, so the streams don't need to be fully realistic.
-    private static byte[] BuildMinidump(int pid, uint exceptionCode)
-    {
-        const int HeaderSize = 32;
-        const int DirEntrySize = 12;
-        const int ExceptionStreamSize = 32;
-        const int MiscInfoStreamSize = 12;
-
-        int dirOffset = HeaderSize;
-        int exceptionOffset = dirOffset + DirEntrySize * 2;
-        int miscOffset = exceptionOffset + ExceptionStreamSize;
-        int totalSize = miscOffset + MiscInfoStreamSize;
-
-        byte[] buf = new byte[totalSize];
-        using var ms = new MemoryStream(buf);
-        using var bw = new BinaryWriter(ms);
-
-        bw.Write(0x504D444Du); // Signature 'MDMP'
-        bw.Write(0x0000A793u); // Version (low 16 bits)
-        bw.Write(2u);          // NumberOfStreams
-        bw.Write((uint)dirOffset);
-        bw.Write(0u);          // CheckSum
-        bw.Write((uint)DateTimeOffset.UtcNow.ToUnixTimeSeconds());
-        bw.Write(0UL);         // Flags
-
-        bw.Write(6u);                            // StreamType: ExceptionStream
-        bw.Write((uint)ExceptionStreamSize);
-        bw.Write((uint)exceptionOffset);
-        bw.Write(15u);                           // StreamType: MiscInfoStream
-        bw.Write((uint)MiscInfoStreamSize);
-        bw.Write((uint)miscOffset);
-
-        bw.Write(100u);          // ThreadId
-        bw.Write(0u);            // __alignment
-        bw.Write(exceptionCode);
-        bw.Write(0u);            // ExceptionFlags
-        bw.Write(0UL);           // ExceptionRecord
-        bw.Write(0xDEADBEEFUL);  // ExceptionAddress
-
-        bw.Write((uint)MiscInfoStreamSize);
-        bw.Write(1u);            // Flags1: MINIDUMP_MISC1_PROCESS_ID
-        bw.Write((uint)pid);
-
-        return buf;
-    }
-
-
-    // Smoke test against the user's actual macOS crash reports directory if it
-    // exists and has at least one Rhino .ips. Skipped on Windows / when there
-    // are no reports — the parser's correctness is what we're checking, not
-    // that crashes have happened.
-    //
-    // Pid-match path has no time window, so we can verify parsing against any
-    // historical .ips by reading its pid from the file directly and asking the
-    // finder to look it up.
-    [Test]
-    public void TryFind_by_pid_parses_real_ips_when_available()
-    {
-        var dir = Path.Combine(
-            Environment.GetFolderPath(Environment.SpecialFolder.UserProfile),
-            "Library", "Logs", "DiagnosticReports");
-        if (!Directory.Exists(dir)) return;
-        var ips = Directory.GetFiles(dir, "Rhinoceros-*.ips")
-            .OrderByDescending(p => new FileInfo(p).LastWriteTimeUtc)
-            .FirstOrDefault();
-        if (ips is null) return;
-
-        // Sniff pid out of the body JSON ourselves so we can hand it to the finder.
-        var text = File.ReadAllText(ips);
-        var nl = text.IndexOf('\n');
-        if (nl < 0) return;
-        using var doc = System.Text.Json.JsonDocument.Parse(text[(nl + 1)..]);
-        if (!doc.RootElement.TryGetProperty("pid", out var pidEl) || !pidEl.TryGetInt32(out var pid)) return;
-
-        var finder = new RhinoCrashReportFinder(NullLogger<RhinoCrashReportFinder>.Instance);
-        var report = finder.TryFind(pid);
-
-        Assert.That(report, Is.Not.Null);
-        Assert.That(report!.Path, Is.EqualTo(ips));
-        Assert.That(report.TopFrames, Is.Not.Empty);
-        Assert.That(report.Signal, Is.Not.Null); // every macOS crash report has one
-
-        // ManagedException is optional (older .ips, non-managed crashes), but
-        // when present it must come with at least one managed frame — otherwise
-        // we've extracted a header without the stack it belongs to.
-        if (report.ManagedException is not null)
-        {
-            Assert.That(report.ManagedFrames, Is.Not.Empty);
-            // Every managed frame starts with "at " — sanity check the parse.
-            Assert.That(report.ManagedFrames, Is.All.StartWith("at "));
-            // Build-server paths must be stripped — the whole point of the
-            // post-process step.
-            Assert.That(report.ManagedFrames, Has.None.Matches<string>(f => f.Contains("/Users/bozo/TeamCity")));
-        }
-    }
-}

package/mcps/RhinoMCP/tests/Router.Tests/RhinoLocatorOverrideTests.cs

@@ -1,61 +0,0 @@
-using NUnit.Framework;
-using RhMcp.Router;
-
-namespace RhMcp.Router.Tests;
-
-// The per-version path override (debug-build support). Uses a real temp file as
-// the override target so File.Exists passes regardless of what Rhino is installed.
-[TestFixture]
-public sealed class RhinoLocatorOverrideTests
-{
-    private string _fakeExe = null!;
-
-    [SetUp]
-    public void SetUp()
-    {
-        _fakeExe = Path.Combine(Path.GetTempPath(), "rhmcp-fake-rhino-" + Guid.NewGuid().ToString("N"));
-        File.WriteAllText(_fakeExe, "");
-    }
-
-    [TearDown]
-    public void TearDown()
-    {
-        try { File.Delete(_fakeExe); } catch { /* best effort */ }
-    }
-
-    private Dictionary<string, string> Override(string version) => new() { [version] = _fakeExe };
-
-    [Test]
-    public void Override_wins_for_its_version()
-    {
-        Assert.That(RhinoLocator.ResolveRhinoExe("9", Override("9")), Is.EqualTo(_fakeExe));
-        Assert.That(RhinoLocator.IsOverride("9", Override("9")), Is.True);
-    }
-
-    [Test]
-    public void Override_for_9_also_serves_wip()
-    {
-        // GH2 tools pin "WIP"; a Rhino 9 build is the WIP, so the compat fallback applies.
-        Assert.That(RhinoLocator.ResolveRhinoExe("WIP", Override("9")), Is.EqualTo(_fakeExe));
-        Assert.That(RhinoLocator.IsOverride("WIP", Override("9")), Is.True);
-    }
-
-    [Test]
-    public void Override_does_not_leak_to_unrelated_version()
-    {
-        Assert.That(RhinoLocator.IsOverride("8", Override("9")), Is.False);
-    }
-
-    [Test]
-    public void Missing_override_target_is_ignored()
-    {
-        var overrides = new Dictionary<string, string> { ["9"] = "/does/not/exist" };
-        Assert.That(RhinoLocator.IsOverride("9", overrides), Is.False);
-    }
-
-    [Test]
-    public void Null_overrides_is_not_an_override()
-    {
-        Assert.That(RhinoLocator.IsOverride("9", null), Is.False);
-    }
-}