@hybridlabor-api/aos 4.3.0 → 4.4.0

package/mcps/RhinoMCP/tests/StreamJson.Tests/ConversationStorePruneTests.cs

@@ -1,94 +0,0 @@
-using System.Threading;
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// ConversationStore beyond the round-trip: the 50-conversation cap prunes the oldest on Save, the
-// list comes back newest-first, and a corrupt on-disk slot is skipped rather than thrown. Backed by
-// the in-memory PersistentSettings shim, reset per test.
-[TestFixture]
-public sealed class ConversationStorePruneTests
-{
-    private const int Cap = 50;
-
-    [SetUp]
-    public void Reset() => AISettings.ResetForTest();
-
-    private static void SaveTrivial(string agentName)
-    {
-        Conversation convo = new(Guid.NewGuid(), agentName, "Untitled");
-        convo.BeginTurn("q");
-        convo.CompleteTurn();
-        ConversationStore.Save(convo);
-    }
-
-    [Test]
-    public void Save_beyond_the_cap_prunes_down_to_the_cap()
-    {
-        for (int i = 0; i < Cap + 5; i++)
-            SaveTrivial($"agent-{i}");
-
-        Assert.That(ConversationStore.List(), Has.Count.EqualTo(Cap));
-    }
-
-    [Test]
-    public void Pruning_drops_the_oldest_and_keeps_the_newest_first()
-    {
-        // Distinct StartedAt so the newest-first sort (and thus which slot is pruned) is deterministic.
-        // The constructor stamps StartedAt = UtcNow, so a tiny spacing separates each conversation.
-        Conversation oldest = new(Guid.NewGuid(), "oldest", "Untitled");
-        oldest.BeginTurn("q");
-        oldest.CompleteTurn();
-        ConversationStore.Save(oldest);
-
-        Thread.Sleep(5);
-
-        for (int i = 0; i < Cap; i++)
-        {
-            Thread.Sleep(2);
-            SaveTrivial($"newer-{i}");
-        }
-
-        IReadOnlyList<ConversationDto> all = ConversationStore.List();
-        Assert.That(all, Has.Count.EqualTo(Cap));
-        Assert.That(all.Any(c => c.AgentName == "oldest"), Is.False); // the oldest got pruned
-        // List is newest-first: the head's StartedAt is the max.
-        Assert.That(all[0].StartedAt, Is.GreaterThanOrEqualTo(all[^1].StartedAt));
-    }
-
-    [Test]
-    public void A_corrupt_slot_is_skipped_not_thrown()
-    {
-        SaveTrivial("good");
-        AISettings.Conversations.SetString("garbage-key", "{ this is not valid json ");
-
-        IReadOnlyList<ConversationDto> all = ConversationStore.List();
-
-        Assert.That(all, Has.Count.EqualTo(1));
-        Assert.That(all[0].AgentName, Is.EqualTo("good"));
-    }
-
-    [Test]
-    public void TryLoad_of_a_missing_key_returns_false()
-    {
-        Assert.That(ConversationStore.TryLoad(Guid.NewGuid().ToString(), out _), Is.False);
-    }
-
-    [Test]
-    public void Re_saving_the_same_session_overwrites_in_place_not_duplicates()
-    {
-        Guid id = Guid.NewGuid();
-        Conversation convo = new(id, "claude", "Untitled");
-        convo.BeginTurn("first");
-        convo.CompleteTurn();
-        ConversationStore.Save(convo);
-
-        convo.BeginTurn("second");
-        convo.CompleteTurn();
-        ConversationStore.Save(convo);
-
-        IReadOnlyList<ConversationDto> all = ConversationStore.List();
-        Assert.That(all, Has.Count.EqualTo(1));
-        Assert.That(all[0].Turns, Has.Count.EqualTo(2));
-    }
-}

package/mcps/RhinoMCP/tests/StreamJson.Tests/ConversationStoreRoundTripTests.cs

@@ -1,70 +0,0 @@
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// Guards the headline serialization trap of the SessionId -> AgentSessionId rename: the live
-// Conversation property was renamed, but the persisted ConversationDto.SessionId field/key MUST
-// stay 'sessionId' or every saved transcript orphans. Save then List/TryLoad must round-trip, and
-// an existing on-disk transcript (camelCase 'sessionId') must still deserialize.
-[TestFixture]
-public sealed class ConversationStoreRoundTripTests
-{
-    [SetUp]
-    public void Reset() => AISettings.ResetForTest();
-
-    [Test]
-    public void Save_then_List_and_TryLoad_round_trips_the_transcript()
-    {
-        Guid sessionId = Guid.NewGuid();
-        Conversation conversation = new(sessionId, "claude", "MyDoc.3dm");
-        conversation.NoteSessionStarted();
-        conversation.BeginTurn("hello");
-        conversation.Record(TurnEventKind.AssistantText, "hi");
-        conversation.Record(TurnEventKind.ToolUse, "add_box", args: "{\"size\":1}", id: "toolu_1");
-        conversation.CompleteToolCall("toolu_1", "created");
-        conversation.CompleteTurn();
-
-        ConversationStore.Save(conversation);
-
-        // The persisted key is the live AgentSessionId stringified (the rename's contract).
-        string key = sessionId.ToString();
-
-        IReadOnlyList<ConversationDto> all = ConversationStore.List();
-        Assert.That(all, Has.Count.EqualTo(1));
-        Assert.That(all[0].SessionId, Is.EqualTo(key));
-        Assert.That(all[0].AgentName, Is.EqualTo("claude"));
-        Assert.That(all[0].DocTitle, Is.EqualTo("MyDoc.3dm"));
-
-        Assert.That(ConversationStore.TryLoad(key, out ConversationDto loaded), Is.True);
-        Assert.That(loaded.SessionId, Is.EqualTo(key));
-        Assert.That(loaded.Lifecycle, Has.Count.EqualTo(1));
-        Assert.That(loaded.Turns, Has.Count.EqualTo(1));
-
-        TurnDto turn = loaded.Turns[0];
-        Assert.That(turn.Prompt, Is.EqualTo("hello"));
-        Assert.That(turn.CompletedAt, Is.Not.Null);
-        Assert.That(turn.Events, Has.Count.EqualTo(2));
-        Assert.That(turn.Events[1].Id, Is.EqualTo("toolu_1"));
-        Assert.That(turn.Events[1].Result, Is.EqualTo("created"));
-    }
-
-    [Test]
-    public void Persisted_transcript_keyed_by_legacy_sessionId_field_still_loads()
-    {
-        // A transcript written by the pre-rename build: the serialized field is 'sessionId' (camelCase
-        // via McpSerializer). The rename must not have moved this field, so it must still deserialize.
-        string key = Guid.NewGuid().ToString();
-        string legacyJson =
-            $$"""
-            {"sessionId":"{{key}}","agentName":"codex","docTitle":"Old.3dm","startedAt":"2026-01-01T00:00:00+00:00","lifecycle":[],"turns":[{"prompt":"q","startedAt":"2026-01-01T00:00:00+00:00","completedAt":"2026-01-01T00:00:01+00:00","events":[]}]}
-            """;
-        AISettings.Conversations.SetString(key, legacyJson);
-
-        Assert.That(ConversationStore.TryLoad(key, out ConversationDto loaded), Is.True);
-        Assert.That(loaded.SessionId, Is.EqualTo(key));
-        Assert.That(loaded.AgentName, Is.EqualTo("codex"));
-        Assert.That(loaded.DocTitle, Is.EqualTo("Old.3dm"));
-        Assert.That(loaded.Turns, Has.Count.EqualTo(1));
-        Assert.That(loaded.Turns[0].Prompt, Is.EqualTo("q"));
-    }
-}

package/mcps/RhinoMCP/tests/StreamJson.Tests/ConversationTests.cs

@@ -1,153 +0,0 @@
-using System.Threading;
-using System.Threading.Tasks;
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// The live transcript graph: turn/event recording, CompleteToolCall folding a tool's output back
-// into its originating ToolUse event, the Changed signal firing on every mutation, late/stray-event
-// dropping, and the Record path's thread-safety under concurrent writers. Token accounting is
-// covered separately in ConversationUsageTests.
-[TestFixture]
-public sealed class ConversationTests
-{
-    private static Conversation NewConversation() => new(Guid.NewGuid(), "claude", "Untitled");
-
-    [Test]
-    public void Recorded_events_land_on_the_current_turn_in_order()
-    {
-        Conversation convo = NewConversation();
-        convo.BeginTurn("hi");
-        convo.Record(TurnEventKind.AssistantText, "one");
-        convo.Record(TurnEventKind.AssistantText, "two");
-
-        IReadOnlyList<TurnEvent> events = convo.Turns[0].Events;
-        Assert.That(events.Select(static e => e.Text), Is.EqualTo(new[] { "one", "two" }));
-    }
-
-    [Test]
-    public void CompleteToolCall_folds_the_result_into_its_originating_tool_use_by_id()
-    {
-        Conversation convo = NewConversation();
-        convo.BeginTurn("hi");
-        convo.Record(TurnEventKind.ToolUse, "add_box", args: "{\"size\":1}", id: "toolu_1");
-        convo.Record(TurnEventKind.AssistantText, "between");
-        convo.Record(TurnEventKind.ToolUse, "add_sphere", args: "{}", id: "toolu_2");
-
-        convo.CompleteToolCall("toolu_1", "made a box");
-
-        IReadOnlyList<TurnEvent> events = convo.Turns[0].Events;
-        Assert.That(events, Has.Count.EqualTo(3)); // folding never adds a stray event
-        Assert.That(events[0].Id, Is.EqualTo("toolu_1"));
-        Assert.That(events[0].Result, Is.EqualTo("made a box"));
-        Assert.That(events[2].Result, Is.Empty); // the other tool call is untouched
-    }
-
-    [Test]
-    public void CompleteToolCall_with_unknown_id_is_a_no_op()
-    {
-        Conversation convo = NewConversation();
-        convo.BeginTurn("hi");
-        convo.Record(TurnEventKind.ToolUse, "add_box", id: "toolu_1");
-
-        convo.CompleteToolCall("missing", "orphan result");
-
-        Assert.That(convo.Turns[0].Events[0].Result, Is.Empty);
-    }
-
-    [Test]
-    public void CompleteToolCall_matches_the_most_recent_tool_use_when_ids_repeat()
-    {
-        Conversation convo = NewConversation();
-        convo.BeginTurn("hi");
-        convo.Record(TurnEventKind.ToolUse, "first", id: "dup");
-        convo.Record(TurnEventKind.ToolUse, "second", id: "dup");
-
-        convo.CompleteToolCall("dup", "for the latest");
-
-        IReadOnlyList<TurnEvent> events = convo.Turns[0].Events;
-        Assert.That(events[0].Result, Is.Empty);
-        Assert.That(events[1].Result, Is.EqualTo("for the latest"));
-    }
-
-    [Test]
-    public void Record_after_CompleteTurn_is_dropped_not_misfiled()
-    {
-        Conversation convo = NewConversation();
-        convo.BeginTurn("hi");
-        convo.Record(TurnEventKind.AssistantText, "in turn");
-        convo.CompleteTurn();
-
-        convo.Record(TurnEventKind.AssistantText, "stray"); // no current turn to attach to
-
-        Assert.That(convo.Turns[0].Events, Has.Count.EqualTo(1));
-        Assert.That(convo.Turns[0].Completed, Is.True);
-    }
-
-    [Test]
-    public void NoteSessionStarted_records_a_lifecycle_event_outside_any_turn()
-    {
-        Conversation convo = NewConversation();
-        convo.NoteSessionStarted();
-
-        Assert.That(convo.Turns, Is.Empty);
-        Assert.That(convo.Lifecycle, Has.Count.EqualTo(1));
-        Assert.That(convo.Lifecycle[0].Kind, Is.EqualTo(TurnEventKind.SessionStarted));
-    }
-
-    [Test]
-    public void Changed_fires_on_every_mutation()
-    {
-        Conversation convo = NewConversation();
-        int fired = 0;
-        convo.Changed += () => Interlocked.Increment(ref fired);
-
-        convo.NoteSessionStarted();        // 1
-        convo.BeginTurn("hi");             // 2
-        convo.Record(TurnEventKind.AssistantText, "x"); // 3
-        convo.CompleteToolCall("none", "y");            // 4 (no-op fold still signals a render pass)
-        convo.RecordUsage(new TokenUsage(1, 1));        // 5
-        convo.CompleteTurn();              // 6
-
-        Assert.That(fired, Is.EqualTo(6));
-    }
-
-    [Test]
-    public void Pending_question_round_trips_and_clears_reference_guarded()
-    {
-        Conversation convo = NewConversation();
-        Assert.That(convo.TryGetPendingQuestion(out _), Is.False);
-
-        PendingQuestion first = new("pick one", ["a", "b"], AskUserMode.Single);
-        convo.SetPendingQuestion(first);
-        Assert.That(convo.TryGetPendingQuestion(out PendingQuestion got), Is.True);
-        Assert.That(got, Is.SameAs(first));
-
-        // A stale clear for an already-replaced question must not wipe the newer one.
-        PendingQuestion second = new("again", ["c"], AskUserMode.Multi);
-        convo.SetPendingQuestion(second);
-        convo.ClearPendingQuestion(first);
-        Assert.That(convo.TryGetPendingQuestion(out PendingQuestion still), Is.True);
-        Assert.That(still, Is.SameAs(second));
-
-        convo.ClearPendingQuestion(second);
-        Assert.That(convo.TryGetPendingQuestion(out _), Is.False);
-    }
-
-    [Test]
-    public void Concurrent_Record_writers_lose_no_events()
-    {
-        Conversation convo = NewConversation();
-        convo.BeginTurn("stress");
-
-        const int writers = 8;
-        const int perWriter = 200;
-        Parallel.For(0, writers, w =>
-        {
-            for (int i = 0; i < perWriter; i++)
-                convo.Record(TurnEventKind.AssistantText, $"{w}:{i}");
-        });
-
-        Assert.That(convo.Turns[0].Events, Has.Count.EqualTo(writers * perWriter));
-    }
-}

package/mcps/RhinoMCP/tests/StreamJson.Tests/ConversationUsageTests.cs

@@ -1,56 +0,0 @@
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// RecordUsage lands on the current turn and rolls up into SessionUsage; empty usage and a record
-// arriving after the turn already closed are both ignored rather than mis-filed.
-[TestFixture]
-public sealed class ConversationUsageTests
-{
-    private static Conversation NewConversation() => new(Guid.NewGuid(), "claude", "Untitled");
-
-    [Test]
-    public void Recorded_usage_lands_on_the_current_turn_and_sums_into_the_session()
-    {
-        Conversation convo = NewConversation();
-
-        convo.BeginTurn("one");
-        convo.RecordUsage(new TokenUsage(100, 40, 0.01m));
-        convo.CompleteTurn();
-
-        convo.BeginTurn("two");
-        convo.RecordUsage(new TokenUsage(50, 20));
-        convo.CompleteTurn();
-
-        Assert.That(convo.Turns[0].Usage.TotalTokens, Is.EqualTo(140));
-        Assert.That(convo.Turns[1].Usage.TotalTokens, Is.EqualTo(70));
-
-        Assert.That(convo.SessionUsage.InputTokens, Is.EqualTo(150));
-        Assert.That(convo.SessionUsage.OutputTokens, Is.EqualTo(60));
-        Assert.That(convo.SessionUsage.CostUsd, Is.EqualTo(0.01m)); // one turn reported a cost
-    }
-
-    [Test]
-    public void Empty_usage_is_not_recorded()
-    {
-        Conversation convo = NewConversation();
-        convo.BeginTurn("one");
-        convo.RecordUsage(TokenUsage.Empty);
-
-        Assert.That(convo.Turns[0].Usage.IsEmpty, Is.True);
-        Assert.That(convo.SessionUsage.IsEmpty, Is.True);
-    }
-
-    [Test]
-    public void Usage_after_turn_closed_is_dropped_not_misfiled()
-    {
-        Conversation convo = NewConversation();
-        convo.BeginTurn("one");
-        convo.CompleteTurn(); // Current cleared
-
-        convo.RecordUsage(new TokenUsage(999, 999)); // no current turn to attach to
-
-        Assert.That(convo.Turns[0].Usage.IsEmpty, Is.True);
-        Assert.That(convo.SessionUsage.IsEmpty, Is.True);
-    }
-}

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

@@ -1,12 +0,0 @@
-global using System;
-global using System.Linq;
-global using System.Text.Json;
-global using System.Collections;
-global using System.ComponentModel;
-global using System.Collections.Generic;
-
-global using RhMcp.Server;
-
-global using Rhino;
-
-global using NUnit.Framework;

package/mcps/RhinoMCP/tests/StreamJson.Tests/Shims.cs

@@ -1,93 +0,0 @@
-using System;
-using System.Collections.Generic;
-
-// Headless stand-ins for the few RhinoCommon / plugin-bootstrap surfaces the compiled-in agent
-// sources reach for. They reproduce exactly the API the code under test calls (no more), so the
-// real StreamJsonAgent / ConversationStore / parsers run unchanged in a RhinoCommon-free assembly.
-
-namespace Rhino
-{
-    // StreamJsonAgent's only RhinoCommon dependency is the err/parse-error log line. A no-op sink is
-    // faithful: the design mandates the parsers stay RhinoApp-free and logging is the runner's err
-    // channel only, never an assertion surface.
-    internal static class RhinoApp
-    {
-        public static void WriteLine(string text) { }
-
-        // ConversationStore.Save marshals its PersistentSettings write onto the UI thread. Headless
-        // there is no UI thread, so run the action inline: deterministic and keeps Save synchronous
-        // for the store tests.
-        public static void InvokeOnUiThread(Delegate method, params object[] args) =>
-            method.DynamicInvoke(args);
-    }
-}
-
-namespace Rhino.PlugIns
-{
-    // The slice of PersistentSettings ConversationStore touches: a flat string keystore plus the
-    // child-node accessors AISettings.Conversations relies on. In-memory, deterministic, resettable.
-    internal sealed class PersistentSettings
-    {
-        private Dictionary<string, string> Strings { get; } = new();
-
-        public void SetString(string key, string value) => Strings[key] = value;
-
-        public bool TryGetString(string key, out string value) => Strings.TryGetValue(key, out value!);
-
-        public void DeleteItem(string key) => Strings.Remove(key);
-
-        public IEnumerable<string> Keys => new List<string>(Strings.Keys);
-    }
-}
-
-namespace RhMcp
-{
-    // The AISettings members the compiled-in closure reads. Tests assign these directly to script
-    // settings-driven behaviour (ResolveMcpServers, registry resolution) and to reset the transcript
-    // store. GetAgents/DefaultAgentName feed AgentRegistry; the agent chain defaults to the real
-    // built-ins so registry resolution sees out-of-the-box ordering unless a test overrides it.
-    internal static class AISettings
-    {
-        public static string ExtraMcpServersJson { get; set; } = "{\"mcpServers\":{}}";
-
-        public static Rhino.PlugIns.PersistentSettings Conversations { get; set; } = new();
-
-        public static string DefaultAgentName { get; set; } = "claude";
-
-        // When set (resolution tests), GetAgents returns this chain verbatim. When null, GetAgents
-        // mirrors production: built-ins overlaid with CustomAgents through the SAME pure
-        // AgentRegistry.Overlay, so the overlay/dedup invariant the tests assert can't drift from
-        // the real AISettings.GetAgents.
-        private static IReadOnlyList<AgentDefinition>? ChainOverride { get; set; }
-
-        private static IReadOnlyList<AgentDefinition> CustomAgents { get; set; } = [];
-
-        public static IReadOnlyList<AgentDefinition> GetAgents() =>
-            ChainOverride ?? AgentRegistry.Overlay(AgentRegistry.Builtins(), CustomAgents);
-
-        public static void SetAgentsForTest(IReadOnlyList<AgentDefinition> agents) => ChainOverride = agents;
-
-        public static void SetCustomAgentsForTest(IReadOnlyList<AgentDefinition> custom)
-        {
-            ChainOverride = null;
-            CustomAgents = custom;
-        }
-
-        public static void ResetForTest()
-        {
-            ExtraMcpServersJson = "{\"mcpServers\":{}}";
-            Conversations = new();
-            DefaultAgentName = "claude";
-            ChainOverride = null;
-            CustomAgents = [];
-        }
-    }
-
-    // AgentPrompts.Compose only needs the server-name const; the real RouterMcpConfig also resolves
-    // an on-disk router path through RhMcpPlugin (RhinoCommon), which is out of scope for headless
-    // parser tests. Shimming just the const keeps AgentPrompts pure.
-    internal static class RouterMcpConfig
-    {
-        internal const string ServerName = "rhino";
-    }
-}

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

@@ -1,56 +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.StreamJson.Tests</RootNamespace>
-  </PropertyGroup>
-
-  <ItemGroup>
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.1" />
-    <PackageReference Include="NUnit" Version="4.2.2" />
-    <PackageReference Include="NUnit3TestAdapter" Version="4.6.0" />
-    <PackageReference Include="NUnit.Analyzers" Version="4.3.0" />
-  </ItemGroup>
-
-  <ItemGroup>
-    <ProjectReference Include="../../rhino/acp/Acp.csproj" />
-  </ItemGroup>
-
-  <!--
-    Compile the agent layer's PURE sources straight into the test assembly (mirrors
-    Server.Tests pulling in Protocol.cs). That keeps the assembly RhinoCommon- and
-    AspNetCore-free and side-steps internals-visibility. The two RhinoCommon touch points
-    in this closure (RhinoApp.WriteLine and the AISettings/PersistentSettings persistence
-    surface) plus RouterMcpConfig are satisfied by the tiny headless shims in Shims.cs, so
-    the code under test runs unchanged.
-  -->
-  <ItemGroup>
-    <Compile Include="../../rhino/plugin/Server/Protocol.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/AgentAdapter.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/AgentDefinition.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/AgentRegistry.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/AgentPrompts.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/Attachment.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/UserMessage.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/AcpMessageMapper.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/IStreamJsonParser.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/ParsedLine.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/TokenUsage.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/ClaudeStreamJsonParser.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/CodexStreamJsonParser.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/StreamJsonAgent.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/IAgentRunner.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/AgentRunner.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/RhinoAcpClient.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/Conversation.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/ConversationDto.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Agents/ConversationStore.cs" LinkBase="PluginSources" />
-    <Compile Include="../../rhino/plugin/Tools/AskUserQuestion.cs" LinkBase="PluginSources" />
-  </ItemGroup>
-
-</Project>

package/mcps/RhinoMCP/tests/StreamJson.Tests/StreamJsonAgentLoopbackTests.cs

@@ -1,75 +0,0 @@
-using System.IO;
-using System.Threading.Tasks;
-using Acp;
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// Drives the real StreamJsonAgent over its injected stdin/stdout seam (no process, no CLI). One
-// scripted Claude turn is replayed; we assert the framed prompt reaches stdin, the SessionUpdates
-// flow through the real RhinoAcpClient into the Conversation, and the turn TCS resolves EndTurn.
-[TestFixture]
-public sealed class StreamJsonAgentLoopbackTests
-{
-    private static AgentDefinition ClaudeDef() =>
-        new(
-            Name: "claude",
-            Adapter: AgentAdapter.Claude,
-            Command: "claude",
-            SearchPaths: [],
-            Model: "",
-            ExtraArgs: [],
-            SystemPrompt: "",
-            Enabled: true,
-            IsBuiltin: true);
-
-    [Test]
-    [CancelAfter(10000)]
-    public async Task One_scripted_claude_turn_records_events_and_resolves_end_turn()
-    {
-        Conversation conversation = new(Guid.NewGuid(), "claude", "Untitled");
-        RhinoAcpClient client = new(conversation);
-        StreamJsonAgent agent = new(ClaudeDef(), client, conversation, "/tmp", new ClaudeStreamJsonParser(ClaudeDef()));
-
-        // AgentRunner owns BeginTurn/CompleteTurn in the real path; the loopback seam only drives the
-        // inner IAcpAgent, so open the turn here so RhinoAcpClient.Record lands in a current turn.
-        conversation.BeginTurn("draw a box");
-
-        // The CLI's stdout for one turn: assistant text, a tool_use, its tool_result, then the
-        // terminal result line. Newline-framed exactly like the real `claude` stream-json output.
-        string scriptedStdout = string.Join('\n',
-            """{"type":"assistant","message":{"role":"assistant","content":[{"type":"text","text":"sure"}]}}""",
-            """{"type":"assistant","message":{"role":"assistant","content":[{"type":"tool_use","id":"toolu_1","name":"add_box","input":{"size":10}}]}}""",
-            """{"type":"user","message":{"role":"user","content":[{"type":"tool_result","tool_use_id":"toolu_1","content":"created id 42"}]}}""",
-            """{"type":"result","subtype":"success","is_error":false,"result":"done"}""");
-
-        using StringReader stdout = new(scriptedStdout);
-        StringWriter stdin = new();
-
-        StopReason reason = await agent.RunLoopbackAsync(stdout, stdin, [new TextContentBlock { Text = "draw a box" }]);
-
-        // The turn resolved on the terminal event.
-        Assert.That(reason, Is.EqualTo(StopReason.EndTurn));
-
-        // The framed prompt was written to stdin as Claude's single-line user envelope.
-        string written = stdin.ToString().Trim();
-        Assert.That(written, Does.Contain("\"type\":\"user\""));
-        Assert.That(written, Does.Contain("draw a box"));
-
-        // The Conversation ends with the expected Turn events: assistant text, a tool-use whose
-        // result was folded back into it (matched by id), recorded through the real RhinoAcpClient.
-        Assert.That(conversation.Turns, Has.Count.EqualTo(1));
-        Turn turn = conversation.Turns[0];
-
-        IReadOnlyList<TurnEvent> events = turn.Events;
-        Assert.That(events, Has.Count.EqualTo(2));
-
-        Assert.That(events[0].Kind, Is.EqualTo(TurnEventKind.AssistantText));
-        Assert.That(events[0].Text, Is.EqualTo("sure"));
-
-        Assert.That(events[1].Kind, Is.EqualTo(TurnEventKind.ToolUse));
-        Assert.That(events[1].Text, Is.EqualTo("add_box"));
-        Assert.That(events[1].Id, Is.EqualTo("toolu_1"));
-        Assert.That(events[1].Result, Does.Contain("created id 42"));
-    }
-}