@hybridlabor-api/aos 4.3.0 → 4.4.0

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

@@ -1,63 +0,0 @@
-using RhMcp.Integration.Tests.Harness;
-
-namespace RhMcp.Integration.Tests;
-
-// Pins the autoSpawnedSlot side-channel contract: when a tool is called without
-// a `slot` argument and the router has to launch a Rhino to serve it, the
-// response must carry an `autoSpawnedSlot` payload telling the agent what just
-// happened. Subsequent slotless calls reuse the same Rhino and must NOT
-// re-announce — the agent learns the slot id once.
-//
-// Uses RouterFixture (fresh router per test) so each test starts with no slots.
-[TestFixture]
-public sealed class AutoSpawnNotificationTests : RouterFixture
-{
-    [Test]
-    public async Task slotless_tool_call_on_fresh_router_announces_auto_spawn()
-    {
-        ReturnResult result = await _router.CallToolAsync("run_python", Args.Of(("script", "pass")));
-
-        Assert.That(result.Error, Is.Null,
-            $"run_python should succeed. Error: {result.Error?.Code}: {result.Error?.Message}");
-        Assert.That(result.AutoSpawnedSlot, Is.Not.Null,
-            "First slotless call must populate autoSpawnedSlot so the agent learns the slot id.");
-        Assert.That(result.AutoSpawnedSlot!.SlotId, Is.Not.Empty);
-        Assert.That(result.AutoSpawnedSlot.Version, Is.EqualTo("8"));
-        Assert.That(result.AutoSpawnedSlot.Reason, Does.Contain("run_python"));
-    }
-
-    [Test]
-    public async Task subsequent_slotless_call_reuses_slot_and_omits_auto_spawn()
-    {
-        ReturnResult first = await _router.CallToolAsync("run_python", Args.Of(("script", "pass")));
-        Assert.That(first.AutoSpawnedSlot, Is.Not.Null, "Precondition: first call should auto-spawn.");
-        string firstSlot = first.AutoSpawnedSlot!.SlotId;
-
-        ReturnResult second = await _router.CallToolAsync("run_python", Args.Of(("script", "pass")));
-        Assert.That(second.Error, Is.Null);
-        Assert.That(second.AutoSpawnedSlot, Is.Null,
-            "Reuse of an existing slot must not re-announce — the agent already knows about it.");
-
-        // And list_slots confirms there's still only one Rhino.
-        ReturnResult list = await _router.CallToolAsync("list_slots");
-        Assert.That(list.Payload?.GetArrayLength(), Is.EqualTo(1));
-        Assert.That(list.Payload!.Value[0].GetProperty("slotId").GetString(), Is.EqualTo(firstSlot));
-    }
-
-    [Test]
-    public async Task explicit_slot_argument_does_not_trigger_auto_spawn_notification()
-    {
-        // Spawn first via the explicit tool — agent knows the slot id from the
-        // spawn_slot response, no side-channel needed.
-        ReturnResult spawn = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-        string slot = spawn.Payload!.Value.GetProperty("slotId").GetString()!;
-
-        ReturnResult result = await _router.CallToolAsync(
-            "run_python",
-            Args.Of(("slot", (object?)slot), ("script", "pass")));
-
-        Assert.That(result.Error, Is.Null);
-        Assert.That(result.AutoSpawnedSlot, Is.Null,
-            "Explicit `slot` arg means the agent already knows the slot — no announcement.");
-    }
-}

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

@@ -1,56 +0,0 @@
-using RhMcp.Integration.Tests.Harness;
-
-namespace RhMcp.Integration.Tests;
-
-// Drives Claude via the local CLI against an isolated rhino-mcp-router. No
-// Rhino install required — close_slot on a bogus slot only touches the router.
-[TestFixture]
-[McpDependency("rhino")]
-public sealed class CloseSlotAgentTests : AgenticTestBase
-{
-    private string _isolatedTempDir = null!;
-
-    protected override void ConfigureHarness()
-    {
-        _isolatedTempDir = RhinoRouterPaths.CreateIsolatedTempDir();
-        UseMcp(
-            name: "rhino",
-            command: RhinoRouterPaths.ResolveBinary(),
-            env: RhinoRouterPaths.IsolatedEnv(_isolatedTempDir));
-        UseDefaults(maxBudgetUsd: 0.25);
-    }
-
-    [OneTimeTearDown]
-    public void CleanupTempDir()
-    {
-        RhinoRouterPaths.TryDeleteDirectory(_isolatedTempDir);
-    }
-
-    [Test]
-    public async Task agent_receives_slot_not_found_when_closing_unknown_slot()
-    {
-        AgentRun run = await Agent
-            .WithAllowedTools("mcp__rhino__close_slot")
-            .WithBuiltinTools("")
-            .WithWorkingDirectory(_isolatedTempDir)
-            .WithSystemPrompt(
-                "You are an integration test harness. Call exactly the tools requested. " +
-                "When reporting results, include the raw JSON the tool returned.")
-            .RunAsync(
-                "Close the rhino slot named 'made-up-slot-xyz'. " +
-                "Do not call list_slots first — just attempt the close once and report what the tool returned.");
-
-        Assert.That(run.Metadata?["is_error"], Is.EqualTo(false),
-            $"Claude reported an error. Payload: {run.Metadata?["error_payload"]}\n" +
-            $"Stderr:\n{run.Metadata?["stderr"]}");
-
-        Assert.That(run, Did.CallTool("mcp__rhino__close_slot"));
-
-        ToolCall close = run.ToolCalls.First(c => c.Name == "mcp__rhino__close_slot");
-
-        // The JSON property assertions auto-unwrap the MCP content envelope,
-        // so we don't have to care whether the harness flattened it for us.
-        // Failure envelopes carry a structured `error: { code, message }`.
-        Assert.That(close.Result, Json.HasProperty("error", Json.HasProperty("code", Is.EqualTo("slot_not_found"))));
-    }
-}

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

@@ -1,37 +0,0 @@
-using RhMcp.Integration.Tests.Harness;
-
-namespace RhMcp.Integration.Tests;
-
-// Exercises the router's close_slot tool directly. No Rhino install required —
-// these tests run against a freshly-spawned router with an isolated state dir.
-[TestFixture]
-internal sealed class CloseSlotTests : SharedRouterFixture
-{
-    // Regression: a status-agnostic existence check is required so launching
-    // slots are not mistaken for missing slots. The envelope's `error` field
-    // is a structured ErrorInfo { code, message } that agents key off of when
-    // deciding whether to retry, list slots, etc.
-    [Test]
-    public async Task close_slot_returns_slot_not_found_for_unknown_slot()
-    {
-        ReturnResult result = await _router.CallToolAsync("close_slot", Args.Of(("slot", "does-not-exist")));
-
-        Assert.That(result.Error?.Code, Is.EqualTo("slot_not_found"));
-        Assert.That(result.Error?.Message, Does.Contain("does-not-exist"));
-        Assert.That(result.Error?.Message, Does.Contain("list_slots"));
-        Assert.That(result.Payload, Is.Null);
-    }
-
-    // The advertised envelope promises null fields are omitted. The source-gen
-    // policy is pinned by Router.Tests/ReturnResultTests; this end-to-end version
-    // catches the orthogonal failure mode of something downstream (MCP SDK
-    // transport, content-block wrap) re-introducing the null fields.
-    [Test]
-    public async Task close_slot_failure_envelope_omits_payload_and_autoSpawnedSlot_on_the_wire()
-    {
-        string json = await _router.CallToolTextAsync("close_slot", Args.Of(("slot", "another-bogus-slot")));
-
-        Assert.That(json, Does.Not.Contain("\"payload\":null"));
-        Assert.That(json, Does.Not.Contain("\"autoSpawnedSlot\":null"));
-    }
-}

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

@@ -1,42 +0,0 @@
-using RhMcp.Integration.Tests.Harness;
-
-namespace RhMcp.Integration.Tests;
-
-// Exercises open_doc / close_doc — the plugin-side document lifecycle tools.
-[TestFixture]
-public sealed class DocLifecycleTests : SharedRouterFixture
-{
-
-    [TestCase("8")]
-    [TestCase("WIP")]
-    public async Task close_doc_in_spawned_slot_returns_closed_message(string version)
-    {
-        _ = await _router.CallToolTextAsync("spawn_slot", Args.Of(("version", version)));
-
-        string response = await _router.CallToolTextAsync("close_doc");
-
-        Assert.That(response, Does.Contain("closed").IgnoreCase);
-    }
-
-    // TODO : close_doc with no active slot — semantics aren't pinned down yet
-    // (does the router auto-spawn, no-op, or return a structured error?). The
-    // earlier version asserted an empty JSON array, which never matched any
-    // observed behaviour. Re-enable once the contract is decided.
-    [Test]
-    [Ignore("Pending: define router behaviour for close_doc with no active slot.")]
-    public Task close_doc_with_no_active_slot_is_a_no_op()
-    {
-        return Task.CompletedTask;
-    }
-
-    // TODO : open_doc requires a `path` argument (see OpenDocTool.cs). The
-    // earlier version of this test passed none, so it could only ever hit the
-    // ArgumentException path. Add a real .3dm fixture under tests/Fixtures and
-    // re-enable.
-    [Test]
-    [Ignore("Pending: add a .3dm fixture file and assert against the import response.")]
-    public Task open_doc_imports_fixture_and_appears_in_list_slots()
-    {
-        return Task.CompletedTask;
-    }
-}

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

@@ -1,3 +0,0 @@
-global using NUnit.Framework;
-global using Ngentic;
-global using Ngentic.NUnit;

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

@@ -1,56 +0,0 @@
-using System.Text.Json;
-using RhMcp.Integration.Tests.Harness;
-
-namespace RhMcp.Integration.Tests;
-
-// Exercises g1_start / g2_start: starting Grasshopper inside a spawned Rhino
-// should produce its own slot entry alongside the parent Rhino slot.
-[TestFixture]
-public sealed class GrasshopperStartTests : RouterFixture
-{
-
-    [Test]
-    public async Task g2_start_in_rhino_8_produces_distinct_slot()
-    {
-        _ = await _router.CallToolAsync("spawn_slot", Args.Of(("version", "8")));
-
-        ReturnResult gh2 = await _router.CallToolAsync("g2_start");
-        Assert.That(gh2.Payload?.GetString(), Does.Contain("Opened G2").IgnoreCase);
-
-        ReturnResult list = await _router.CallToolAsync("list_slots");
-        Assert.That(list.Payload?.GetArrayLength(), Is.EqualTo(2));
-
-        List<JsonElement> slots = list.Payload!.Value.EnumerateArray().ToList();
-        foreach (string property in new[] { "slotId", "port", "endpoint" })
-        {
-            HashSet<string> set = slots.Select(j => j.GetProperty(property).ToString().ToLowerInvariant()).ToHashSet();
-            Assert.That(set, Has.Count.EqualTo(2));
-        }
-    }
-
-    // g2_start with no Rhino already spawned should auto-spawn a host. Today
-    // that path picks Rhino WIP — the assertion mirrors that contract.
-    [Test]
-    public async Task g2_start_with_no_host_spawns_one_slot()
-    {
-        ReturnResult gh2 = await _router.CallToolAsync("g2_start");
-        Assert.That(gh2.Payload?.GetString(), Does.Contain("Opened G2").IgnoreCase);
-
-        ReturnResult list = await _router.CallToolAsync("list_slots");
-        Assert.That(list.Payload?.GetArrayLength(), Is.EqualTo(1));
-
-        JsonElement slot = list.Payload!.Value.EnumerateArray().Single();
-        Assert.That(slot.GetProperty("version").GetString(), Is.EqualTo("WIP"),
-            "g2_start with no host auto-spawns Rhino WIP");
-    }
-
-    [TestCase("8")]
-    [TestCase("WIP")]
-    public async Task g1_start_inside_rhino_opens_grasshopper(string version)
-    {
-        _ = await _router.CallToolAsync("spawn_slot", Args.Of(("version", version)));
-
-        ReturnResult gh1 = await _router.CallToolAsync("g1_start");
-        Assert.That(gh1.Payload?.GetString(), Does.Contain("Opened Grasshopper").IgnoreCase);
-    }
-}

package/mcps/RhinoMCP/tests/Integration.Tests/Harness/ReturnResult.cs

@@ -1,28 +0,0 @@
-using System.Text.Json;
-using System.Text.Json.Serialization;
-
-namespace RhMcp.Integration.Tests.Harness;
-
-// Test-side mirror of the router's ReturnResult envelope. Deliberately a copy,
-// not a reference — Integration.Tests stays decoupled from the router's
-// assembly, and the structural-match-at-deserialization is itself a contract
-// test: if the router changes a field name without updating this copy, tests
-// fail loud.
-//
-// `Payload` is JsonElement? (vs the router's JsonNode?) because tests only
-// read; JsonElement is the natural shape for inspection and plugs into the
-// Ngentic Json.* constraints without conversion.
-public sealed record ReturnResult(
-    [property: JsonPropertyName("payload")] JsonElement? Payload,
-    [property: JsonPropertyName("error")] ErrorInfo? Error = null,
-    [property: JsonPropertyName("autoSpawnedSlot")] SlotInfo? AutoSpawnedSlot = null);
-
-public sealed record ErrorInfo(
-    [property: JsonPropertyName("code")] string Code,
-    [property: JsonPropertyName("message")] string Message,
-    [property: JsonPropertyName("crashReportPath")] string? CrashReportPath = null);
-
-public sealed record SlotInfo(
-    [property: JsonPropertyName("slotId")] string SlotId,
-    [property: JsonPropertyName("version")] string Version,
-    [property: JsonPropertyName("reason")] string Reason);

package/mcps/RhinoMCP/tests/Integration.Tests/Harness/RhinoMcpFixture.cs

@@ -1,90 +0,0 @@
-using System.Text.Json;
-using Microsoft.Extensions.AI;
-using ModelContextProtocol.Client;
-using ModelContextProtocol.Protocol;
-
-namespace RhMcp.Integration.Tests.Harness;
-
-// One-stop harness for ngentic-style fixtures that drive the rhino MCP:
-// spawns the freshly-built router in an isolated TMPDIR, hands back the tool
-// list as AITool instances (ready to drop into ChatOptions.Tools), and exposes
-// the handful of slot-management calls a fixture wants to make outside the
-// agent's trajectory (spawn/close/list_objects).
-//
-// The agent invokes tools via the FunctionInvokingChatClient → the AITool list
-// here; the fixture invokes the same tools out-of-band via direct MCP calls.
-// Both share the same router process, so they see the same slot store.
-internal sealed class RhinoMcpFixture : IAsyncDisposable
-{
-    private readonly RhinoMcpRouter _router;
-    private readonly IReadOnlyList<AITool> _tools;
-
-    private RhinoMcpFixture(RhinoMcpRouter router, IReadOnlyList<AITool> tools)
-    {
-        _router = router;
-        _tools = tools;
-    }
-
-    public IReadOnlyList<AITool> Tools => _tools;
-
-    public static async Task<RhinoMcpFixture> CreateAsync(CancellationToken ct = default)
-    {
-        RhinoMcpRouter router = await RhinoMcpRouter.LaunchIsolatedAsync(ct).ConfigureAwait(false);
-        try
-        {
-            // ListToolsAsync paginates internally; we get the full set in one call.
-            IList<McpClientTool> tools = await router.Client.ListToolsAsync(cancellationToken: ct)
-                .ConfigureAwait(false);
-            // McpClientTool : AIFunction : AITool, so the upcast is trivial.
-            return new RhinoMcpFixture(router, tools.Cast<AITool>().ToList());
-        }
-        catch
-        {
-            await router.DisposeAsync().ConfigureAwait(false);
-            throw;
-        }
-    }
-
-    public async Task<string> SpawnSlotAsync(string? version = null, CancellationToken ct = default)
-    {
-        Dictionary<string, object?>? args = version is null
-            ? null
-            : new Dictionary<string, object?> { ["version"] = version };
-        ReturnResult result = await _router.CallToolAsync("spawn_slot", args, ct).ConfigureAwait(false);
-        if (result.Error is { } error)
-        {
-            throw new InvalidOperationException(
-                $"spawn_slot returned error '{error.Code}': {error.Message}\nFull payload: {result.Payload}");
-        }
-        if (result.Payload is not { } payload
-            || !payload.TryGetProperty("slotId", out JsonElement slotIdEl)
-            || slotIdEl.GetString() is not string slotId)
-        {
-            throw new InvalidOperationException($"spawn_slot returned no slotId: {result.Payload}");
-        }
-        return slotId;
-    }
-
-    public async Task CloseSlotAsync(string slotId, CancellationToken ct = default)
-    {
-        string text = await _router.CallToolTextAsync(
-            "close_slot",
-            new Dictionary<string, object?> { ["slot"] = slotId },
-            ct).ConfigureAwait(false);
-        using JsonDocument doc = JsonDocument.Parse(text);
-        // close_slot returns { closed: bool, error?: string, message?: string }.
-        // We don't throw on closed=false because teardown is best-effort — a
-        // slot the agent already closed shows up here as slot_not_found.
-    }
-
-    public async Task<JsonDocument> ListObjectsAsync(string slotId, CancellationToken ct = default)
-    {
-        string text = await _router.CallToolTextAsync(
-            "list_objects",
-            new Dictionary<string, object?> { ["slot"] = slotId },
-            ct).ConfigureAwait(false);
-        return JsonDocument.Parse(text);
-    }
-
-    public ValueTask DisposeAsync() => _router.DisposeAsync();
-}

package/mcps/RhinoMCP/tests/Integration.Tests/Harness/RhinoMcpRouter.cs

@@ -1,111 +0,0 @@
-using System.Text.Json;
-using ModelContextProtocol.Client;
-using ModelContextProtocol.Protocol;
-
-namespace RhMcp.Integration.Tests.Harness;
-
-// Spawns the rhino-mcp-router binary as a child process and connects an MCP
-// stdio client to it. Each instance gets its own TMPDIR so the router's
-// state.db / listener-announcement dir are isolated from the user's regularly
-// running router. Disposal kills the child process via the transport's
-// shutdown path.
-public sealed class RhinoMcpRouter : IAsyncDisposable
-{
-    private readonly string _isolatedTempDir;
-
-    public McpClient Client { get; }
-
-    public HashSet<string> OpenedSlots { get; } = new();
-
-    // Mirrors RouterPaths.ListenersDir on the router side. Tests can drop
-    // *.json announcement files here to simulate a user-started Rhino.
-    public string ListenersDir => Path.Combine(_isolatedTempDir, "rhino-mcp", "listeners");
-
-    private RhinoMcpRouter(string isolatedTempDir, McpClient client)
-    {
-        _isolatedTempDir = isolatedTempDir;
-        Client = client;
-    }
-
-    public static async Task<RhinoMcpRouter> LaunchIsolatedAsync(CancellationToken ct = default)
-    {
-        string isolatedTempDir = RhinoRouterPaths.CreateIsolatedTempDir();
-        StdioClientTransportOptions options = new()
-        {
-            Name = "rhino-mcp-router (test)",
-            Command = RhinoRouterPaths.ResolveBinary(),
-            EnvironmentVariables = RhinoRouterPaths.IsolatedEnv(isolatedTempDir),
-            ShutdownTimeout = TimeSpan.FromMilliseconds(250),
-        };
-        StdioClientTransport transport = new(options);
-
-        try
-        {
-            McpClient client = await McpClient.CreateAsync(transport, cancellationToken: ct);
-            return new RhinoMcpRouter(isolatedTempDir, client);
-        }
-        catch
-        {
-            RhinoRouterPaths.TryDeleteDirectory(isolatedTempDir);
-            throw;
-        }
-    }
-
-    public async Task<string> CallToolTextAsync(
-        string toolName,
-        Dictionary<string, object?>? arguments = null,
-        CancellationToken ct = default)
-    {
-        CallToolResult result = await Client.CallToolAsync(toolName, arguments, cancellationToken: ct);
-        return string.Concat(result.Content.OfType<TextContentBlock>().Select(t => t.Text));
-    }
-
-    // Typed view of the router's ReturnResult envelope. Bypass: get_viewport_image
-    // returns binary content blocks, not the envelope — use CallToolTextAsync for it.
-    public async Task<ReturnResult> CallToolAsync(
-        string toolName,
-        Dictionary<string, object?>? arguments = null,
-        CancellationToken ct = default)
-    {
-        string json = await CallToolTextAsync(toolName, arguments, ct);
-        return JsonSerializer.Deserialize<ReturnResult>(json)
-            ?? throw new InvalidOperationException(
-                $"Tool '{toolName}' returned a null ReturnResult envelope: {json}");
-    }
-
-    public async ValueTask DisposeAsync()
-    {
-        // Close every slot the router is tracking before tearing down the
-        // transport. On macOS the app is single-instance: leaving a Rhino
-        // alive between tests means the next `open -a` just foregrounds it,
-        // the new port never binds, and the spawn waits the full
-        // startup timeout before failing. Cooperative close keeps tests
-        // independent and fast.
-        try
-        {
-            string listJson = await CallToolTextAsync("list_slots");
-            using JsonDocument doc = JsonDocument.Parse(listJson);
-            if (doc.RootElement.TryGetProperty("payload", out JsonElement payload)
-                && payload.ValueKind == JsonValueKind.Array)
-            {
-                foreach (JsonElement slot in payload.EnumerateArray())
-                {
-                    if (slot.TryGetProperty("slotId", out JsonElement slotIdEl)
-                        && slotIdEl.GetString() is string slotId)
-                    {
-                        _ = await CallToolTextAsync("close_slot", new() { { "slot", slotId } });
-                    }
-                }
-            }
-        }
-        catch { /* best effort */ }
-
-        try
-        {
-            await Client.DisposeAsync();
-        }
-        catch { /* best effort */ }
-        RhinoRouterPaths.TryDeleteDirectory(_isolatedTempDir);
-    }
-}
-

package/mcps/RhinoMCP/tests/Integration.Tests/Harness/RhinoRouterPaths.cs

@@ -1,83 +0,0 @@
-using System.Runtime.InteropServices;
-
-namespace RhMcp.Integration.Tests.Harness;
-
-internal static class RhinoRouterPaths
-{
-    public static string ResolveBinary()
-    {
-        string repoRoot = FindRepoRoot();
-        string rid = CurrentRid();
-        string exe = RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? "rhino-mcp-router.exe" : "rhino-mcp-router";
-        string rhinoTarget = Environment.GetEnvironmentVariable("RhinoTarget") is { Length: > 0 } target ? target : "R9";
-        string pluginOS = RuntimeInformation.IsOSPlatform(OSPlatform.Windows) ? "win" : "osx";
-        string binRoot = Path.Combine(repoRoot, "rhino", "plugin", "bin", $"{rhinoTarget}-{pluginOS}");
-
-        // Mirrors RhMcp.csproj: bin/$(RhinoTarget)-$(PluginOS)/$(Configuration)/router/$(rid)/.
-        // The test runner doesn't know which configuration the plugin was built in, so probe both.
-        string[] configurations = ["Release", "Debug"];
-        List<string> probed = [];
-        foreach (string configuration in configurations)
-        {
-            string path = Path.Combine(binRoot, configuration, "router", rid, exe);
-            if (File.Exists(path))
-            {
-                return path;
-            }
-            probed.Add(path);
-        }
-
-        throw new FileNotFoundException(
-            $"Router binary not found. Probed:{Environment.NewLine}  {string.Join($"{Environment.NewLine}  ", probed)}{Environment.NewLine}" +
-            $"Build the plugin first: `dotnet build rhino/plugin/RhMcp.csproj -c Release -p:RhinoTarget={rhinoTarget}`.",
-            probed[0]);
-    }
-
-    // RHINO_MCP_HOME redirects the router+plugin shared dir to a unique location
-    // (see RouterPaths.BaseDir), isolating us from the user's live slot store.
-    public static Dictionary<string, string?> IsolatedEnv(string tempDir)
-    {
-        return new Dictionary<string, string?> { ["RHINO_MCP_HOME"] = tempDir };
-    }
-
-    public static string CreateIsolatedTempDir()
-    {
-        string path = Path.Combine(Path.GetTempPath(), $"rhmcp-test-{Guid.NewGuid():N}");
-        Directory.CreateDirectory(path);
-        return path;
-    }
-
-    public static void TryDeleteDirectory(string path)
-    {
-        try { Directory.Delete(path, recursive: true); } catch { /* best effort */ }
-    }
-
-    private static string CurrentRid()
-    {
-        if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
-        {
-            return RuntimeInformation.OSArchitecture == Architecture.Arm64 ? "win-arm64" : "win-x64";
-        }
-        if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX))
-        {
-            return RuntimeInformation.OSArchitecture == Architecture.Arm64 ? "osx-arm64" : "osx-x64";
-        }
-        throw new PlatformNotSupportedException($"Unsupported OS: {RuntimeInformation.OSDescription}");
-    }
-
-    private static string FindRepoRoot()
-    {
-        string? dir = AppContext.BaseDirectory;
-        while (dir is not null)
-        {
-            if (File.Exists(Path.Combine(dir, "rhino.slnx")))
-            {
-                return dir;
-            }
-            dir = Path.GetDirectoryName(dir);
-        }
-        throw new InvalidOperationException(
-            "Could not locate repo root from the test binary's directory. " +
-            "Expected to find a parent directory containing rhino.slnx.");
-    }
-}

package/mcps/RhinoMCP/tests/Integration.Tests/Harness/RouterFixture.cs

@@ -1,50 +0,0 @@
-using NUnit.Framework;
-
-namespace RhMcp.Integration.Tests.Harness;
-
-// Shared boilerplate for fixtures that need an isolated rh-mcp-router child
-// process. Two flavours:
-//   - RouterFixture: fresh router per test, for fixtures that mutate slot state.
-//   - SharedRouterFixture: one router for the whole fixture, for read-only or
-//     idempotent suites where the spawn cost matters.
-// NUnit runs base [SetUp]/[OneTimeSetUp] before derived ones, so derived classes
-// can add their own setup methods without overriding anything here.
-public abstract class RouterFixture
-{
-    protected RhinoMcpRouter _router = null!;
-
-    [SetUp]
-    public async Task RouterFixtureSetUp()
-    {
-        _router = await RhinoMcpRouter.LaunchIsolatedAsync();
-    }
-
-    [TearDown]
-    public async Task RouterFixtureTearDown()
-    {
-        if (_router is not null)
-        {
-            await _router.DisposeAsync();
-        }
-    }
-}
-
-public abstract class SharedRouterFixture
-{
-    protected RhinoMcpRouter _router = null!;
-
-    [OneTimeSetUp]
-    public async Task SharedRouterFixtureSetUp()
-    {
-        _router = await RhinoMcpRouter.LaunchIsolatedAsync();
-    }
-
-    [OneTimeTearDown]
-    public async Task SharedRouterFixtureTearDown()
-    {
-        if (_router is not null)
-        {
-            await _router.DisposeAsync();
-        }
-    }
-}

package/mcps/RhinoMCP/tests/Integration.Tests/Integration.Tests.csproj

@@ -1,31 +0,0 @@
-<Project Sdk="Microsoft.NET.Sdk">
-
-  <PropertyGroup>
-    <TargetFramework>net8.0</TargetFramework>
-    <LangVersion>12</LangVersion>
-    <Nullable>enable</Nullable>
-    <ImplicitUsings>enable</ImplicitUsings>
-    <IsPackable>false</IsPackable>
-    <IsTestProject>true</IsTestProject>
-    <RootNamespace>RhMcp.Integration.Tests</RootNamespace>
-    <!-- NUnit1032 fires on IAsyncDisposable fields disposed in [OneTimeTearDown];
-         the analyzer only recognises per-test [TearDown], which is wrong for
-         fixture-scoped resources like the rhino router subprocess. -->
-    <NoWarn>$(NoWarn);NUnit1032</NoWarn>
-  </PropertyGroup>
-
-  <ItemGroup>
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.1" />
-    <PackageReference Include="NUnit" Version="4.6.0" />
-    <PackageReference Include="NUnit3TestAdapter" Version="4.6.0" />
-    <PackageReference Include="NUnit.Analyzers" Version="4.4.0" />
-    <PackageReference Include="coverlet.collector" Version="6.0.0" />
-    <PackageReference Include="ModelContextProtocol" Version="1.3.0" />
-  </ItemGroup>
-
-  <ItemGroup>
-    <ProjectReference Include="../ngentic/src/Ngentic.Core/Ngentic.Core.csproj" />
-    <ProjectReference Include="../ngentic/src/Ngentic.NUnit/Ngentic.NUnit.csproj" />
-  </ItemGroup>
-
-</Project>