@hybridlabor-api/aos 4.2.0 → 4.4.0

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

@@ -1,88 +0,0 @@
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// The overlay/dedup invariant AISettings.GetAgents relies on (PLAN W1: custom entries that alias a
-// built-in name override the built-in in place, never duplicated; the built-ins are always present).
-// Production now routes through this pure AgentRegistry.Overlay, and the test shim's GetAgents reuses
-// the SAME method, so this fixture pins the contract once for both paths. Exercised directly against
-// Overlay (no plugin), then through Refresh/Chain to prove the shim doesn't diverge from production.
-[TestFixture]
-public sealed class AgentRegistryOverlayTests
-{
-    private static AgentDefinition Custom(string name, AgentAdapter adapter, string command, bool enabled = true) =>
-        new(name, adapter, command, [command], string.Empty, [], string.Empty, enabled, IsBuiltin: false);
-
-    [Test]
-    public void Overlay_with_no_custom_entries_is_the_builtins_unchanged()
-    {
-        IReadOnlyList<AgentDefinition> builtins = AgentRegistry.Builtins();
-
-        IReadOnlyList<AgentDefinition> chain = AgentRegistry.Overlay(builtins, []);
-
-        Assert.That(chain.Select(static a => a.Name), Is.EqualTo(builtins.Select(static a => a.Name)));
-        Assert.That(chain, Has.All.Matches<AgentDefinition>(static a => a.IsBuiltin));
-    }
-
-    [Test]
-    public void Overlay_replaces_an_aliasing_custom_entry_in_place_never_duplicating()
-    {
-        IReadOnlyList<AgentDefinition> builtins = AgentRegistry.Builtins();
-        AgentDefinition customClaude = Custom("claude", AgentAdapter.Claude, "/opt/my/claude");
-
-        IReadOnlyList<AgentDefinition> chain = AgentRegistry.Overlay(builtins, [customClaude]);
-
-        // Same count as built-ins: the alias overrode, it did not append a second "claude".
-        Assert.That(chain, Has.Count.EqualTo(builtins.Count));
-        Assert.That(chain.Count(static a => a.Name == "claude"), Is.EqualTo(1));
-
-        AgentDefinition claude = chain.Single(static a => a.Name == "claude");
-        Assert.That(claude.Command, Is.EqualTo("/opt/my/claude")); // the custom command won
-        Assert.That(claude.IsBuiltin, Is.True);                    // but the built-in flag is preserved
-    }
-
-    [Test]
-    public void Overlay_keeps_the_builtin_slot_position_when_overriding()
-    {
-        IReadOnlyList<AgentDefinition> builtins = AgentRegistry.Builtins();
-        AgentDefinition customCodex = Custom("codex", AgentAdapter.Codex, "/opt/my/codex");
-
-        IReadOnlyList<AgentDefinition> chain = AgentRegistry.Overlay(builtins, [customCodex]);
-
-        // Order is untouched: claude, codex, gemini. The override lands in codex's existing slot.
-        Assert.That(chain.Select(static a => a.Name), Is.EqualTo(new[] { "claude", "codex", "gemini" }));
-        Assert.That(chain[1].Command, Is.EqualTo("/opt/my/codex"));
-    }
-
-    [Test]
-    public void Overlay_appends_a_new_named_custom_entry_after_the_builtins()
-    {
-        IReadOnlyList<AgentDefinition> builtins = AgentRegistry.Builtins();
-        AgentDefinition mine = Custom("my-agent", AgentAdapter.Claude, "/opt/my/agent");
-
-        IReadOnlyList<AgentDefinition> chain = AgentRegistry.Overlay(builtins, [mine]);
-
-        Assert.That(chain, Has.Count.EqualTo(builtins.Count + 1));
-        Assert.That(chain[^1].Name, Is.EqualTo("my-agent"));
-        Assert.That(chain[^1].IsBuiltin, Is.False); // a genuinely new entry stays non-built-in
-    }
-
-    [Test]
-    public void GetAgents_shim_flows_custom_entries_through_the_same_overlay()
-    {
-        AISettings.ResetForTest();
-        try
-        {
-            AISettings.SetCustomAgentsForTest([Custom("claude", AgentAdapter.Claude, "/opt/my/claude")]);
-
-            IReadOnlyList<AgentDefinition> agents = AISettings.GetAgents();
-            Assert.That(agents.Count(static a => a.Name == "claude"), Is.EqualTo(1)); // de-duped
-            Assert.That(agents.Single(static a => a.Name == "claude").Command, Is.EqualTo("/opt/my/claude"));
-            Assert.That(agents.Single(static a => a.Name == "claude").IsBuiltin, Is.True);
-        }
-        finally
-        {
-            AISettings.ResetForTest();
-        }
-    }
-}

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

@@ -1,152 +0,0 @@
-using System.IO;
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// AgentRegistry resolution logic exercised without a loaded plugin: the chain comes from the
-// AISettings shim, availability is a real File.Exists probe over each definition's SearchPaths (so
-// we point them at real temp files, not a mock), and TryResolveActive picks the first Enabled &&
-// Available with the configured default name preferred. Refresh() rewrites the static chain, so each
-// test resets the shim and re-Refreshes; no [Parallelizable] because that state is shared.
-[TestFixture]
-public sealed class AgentRegistryResolutionTests
-{
-    private string ProbeDir { get; set; } = string.Empty;
-
-    [SetUp]
-    public void SetUp()
-    {
-        AISettings.ResetForTest();
-        ProbeDir = Path.Combine(Path.GetTempPath(), "rhmcp-registry-" + Guid.NewGuid().ToString("N"));
-        Directory.CreateDirectory(ProbeDir);
-    }
-
-    [TearDown]
-    public void TearDown()
-    {
-        if (Directory.Exists(ProbeDir))
-            Directory.Delete(ProbeDir, recursive: true);
-        AISettings.ResetForTest();
-    }
-
-    // A real, existing path the File.Exists probe will accept -> the agent is Available.
-    private string PresentPath(string name)
-    {
-        string path = Path.Combine(ProbeDir, name);
-        File.WriteAllText(path, string.Empty);
-        return path;
-    }
-
-    // A path guaranteed not to exist -> the agent is Unavailable.
-    private string MissingPath(string name) => Path.Combine(ProbeDir, "missing-" + name);
-
-    private AgentDefinition Def(string name, AgentAdapter adapter, string searchPath, bool enabled) =>
-        new(name, adapter, name, [searchPath], string.Empty, [], string.Empty, enabled, IsBuiltin: false);
-
-    [Test]
-    public void Refresh_marks_available_by_probing_search_paths()
-    {
-        AISettings.SetAgentsForTest(
-        [
-            Def("claude", AgentAdapter.Claude, PresentPath("claude"), enabled: true),
-            Def("codex", AgentAdapter.Codex, MissingPath("codex"), enabled: true),
-        ]);
-        AgentRegistry.Refresh();
-
-        ResolvedAgent claude = AgentRegistry.Chain.Single(r => r.Definition.Name == "claude");
-        ResolvedAgent codex = AgentRegistry.Chain.Single(r => r.Definition.Name == "codex");
-        Assert.That(claude.Available, Is.True);
-        Assert.That(codex.Available, Is.False);
-    }
-
-    [Test]
-    public void TryResolveActive_prefers_the_configured_default_when_available()
-    {
-        AISettings.DefaultAgentName = "codex";
-        AISettings.SetAgentsForTest(
-        [
-            Def("claude", AgentAdapter.Claude, PresentPath("claude"), enabled: true),
-            Def("codex", AgentAdapter.Codex, PresentPath("codex"), enabled: true),
-        ]);
-        AgentRegistry.Refresh();
-
-        Assert.That(AgentRegistry.TryResolveActive(out AgentDefinition active), Is.True);
-        Assert.That(active.Name, Is.EqualTo("codex"));
-    }
-
-    [Test]
-    public void TryResolveActive_falls_to_chain_order_when_the_default_is_unavailable()
-    {
-        // Default prefers claude, but claude isn't installed; the first available in chain order wins.
-        AISettings.DefaultAgentName = "claude";
-        AISettings.SetAgentsForTest(
-        [
-            Def("claude", AgentAdapter.Claude, MissingPath("claude"), enabled: true),
-            Def("codex", AgentAdapter.Codex, PresentPath("codex"), enabled: true),
-            Def("gemini", AgentAdapter.Gemini, PresentPath("gemini"), enabled: true),
-        ]);
-        AgentRegistry.Refresh();
-
-        Assert.That(AgentRegistry.TryResolveActive(out AgentDefinition active), Is.True);
-        Assert.That(active.Name, Is.EqualTo("codex")); // first available, chain order
-    }
-
-    [Test]
-    public void TryResolveActive_skips_disabled_even_when_available()
-    {
-        AISettings.DefaultAgentName = "claude";
-        AISettings.SetAgentsForTest(
-        [
-            Def("claude", AgentAdapter.Claude, PresentPath("claude"), enabled: false), // installed but off
-            Def("codex", AgentAdapter.Codex, PresentPath("codex"), enabled: true),
-        ]);
-        AgentRegistry.Refresh();
-
-        Assert.That(AgentRegistry.TryResolveActive(out AgentDefinition active), Is.True);
-        Assert.That(active.Name, Is.EqualTo("codex"));
-    }
-
-    [Test]
-    public void TryResolveActive_is_false_when_nothing_is_enabled_and_available()
-    {
-        AISettings.SetAgentsForTest(
-        [
-            Def("claude", AgentAdapter.Claude, MissingPath("claude"), enabled: true),  // unavailable
-            Def("codex", AgentAdapter.Codex, PresentPath("codex"), enabled: false),    // disabled
-        ]);
-        AgentRegistry.Refresh();
-
-        Assert.That(AgentRegistry.TryResolveActive(out _), Is.False);
-    }
-
-    [Test]
-    public void TryGet_finds_by_name_regardless_of_enabled_or_available()
-    {
-        AISettings.SetAgentsForTest(
-        [
-            Def("codex", AgentAdapter.Codex, MissingPath("codex"), enabled: false),
-        ]);
-        AgentRegistry.Refresh();
-
-        Assert.That(AgentRegistry.TryGet("codex", out AgentDefinition def), Is.True);
-        Assert.That(def.Adapter, Is.EqualTo(AgentAdapter.Codex));
-        Assert.That(AgentRegistry.TryGet("nope", out _), Is.False);
-    }
-
-    [Test]
-    public void Builtins_are_claude_first_then_codex_then_gemini()
-    {
-        IReadOnlyList<AgentDefinition> builtins = AgentRegistry.Builtins();
-        Assert.That(builtins.Select(static b => b.Name), Is.EqualTo(new[] { "claude", "codex", "gemini" }));
-        Assert.That(builtins, Has.All.Matches<AgentDefinition>(static b => b.IsBuiltin && b.Enabled));
-    }
-
-    [Test]
-    public void Default_search_paths_carry_the_command_name_and_are_deduped()
-    {
-        IReadOnlyList<string> paths = AgentRegistry.DefaultSearchPaths("claude");
-        Assert.That(paths, Is.Not.Empty);
-        Assert.That(paths.Distinct().Count(), Is.EqualTo(paths.Count));
-        Assert.That(paths, Has.Some.Matches<string>(static p => Path.GetFileName(p).StartsWith("claude")));
-    }
-}

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

@@ -1,149 +0,0 @@
-using System.Text.Json;
-using System.Threading;
-using System.Threading.Tasks;
-using Acp;
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// End-to-end turn orchestration of the real AgentRunner over a classicist FakeAcpAgent (a genuine
-// second IAcpAgent implementation, not a mock) wired to the runner's own RhinoAcpClient. This closes
-// the only end-to-end gap in the single-seam path: the loopback test opens the turn by hand, so the
-// contract that AgentRunner itself opens exactly one turn, notes the session-started lifecycle once,
-// records updates INSIDE that open turn, and persists on completion, was asserted nowhere. Here the
-// runner drives the whole begin -> initialize/new-session -> prompt -> complete -> save sequence.
-[TestFixture]
-public sealed class AgentRunnerTurnTests
-{
-    [SetUp]
-    public void Reset() => AISettings.ResetForTest();
-
-    [TearDown]
-    public void Cleanup() => AISettings.ResetForTest();
-
-    private static AgentDefinition ClaudeDef() =>
-        new("claude", AgentAdapter.Claude, "claude", [], string.Empty, [], string.Empty, Enabled: true, IsBuiltin: true);
-
-    [Test]
-    [CancelAfter(10000)]
-    public async Task PromptAsync_opens_one_turn_records_inside_it_notes_the_session_once_and_saves()
-    {
-        FakeAcpAgent fake = new();
-        AgentRunner runner = new(ClaudeDef(), "Untitled", (client, _, _) => fake.AttachTo(client));
-
-        await runner.PromptAsync(UserMessage.FromText("draw a box"), "http://localhost:10500/agent", "/tmp");
-
-        Conversation convo = runner.Conversation;
-
-        // Exactly one turn opened (BeginTurn), and it is closed (CompleteTurn ran in the finally).
-        Assert.That(convo.Turns, Has.Count.EqualTo(1));
-        Turn turn = convo.Turns[0];
-        Assert.That(turn.Prompt, Is.EqualTo("draw a box"));
-        Assert.That(turn.Completed, Is.True);
-
-        // The session-started lifecycle was noted exactly once (the first prompt starts the session).
-        Assert.That(convo.Lifecycle, Has.Count.EqualTo(1));
-        Assert.That(convo.Lifecycle[0].Kind, Is.EqualTo(TurnEventKind.SessionStarted));
-
-        // The agent's update streamed through the runner's RhinoAcpClient and landed INSIDE the turn,
-        // not dropped for want of a current turn (which is what would happen if BeginTurn hadn't run).
-        Assert.That(turn.Events, Has.Count.EqualTo(1));
-        Assert.That(turn.Events[0].Kind, Is.EqualTo(TurnEventKind.AssistantText));
-        Assert.That(turn.Events[0].Text, Is.EqualTo("on it"));
-
-        // The runner initialized then opened a session against the doc's MCP url (once).
-        Assert.That(fake.InitializeCount, Is.EqualTo(1));
-        Assert.That(fake.NewSessionCount, Is.EqualTo(1));
-        Assert.That(fake.PromptCount, Is.EqualTo(1));
-
-        // CompleteTurn triggered the persistence: the transcript is in the store keyed by session id.
-        Assert.That(ConversationStore.TryLoad(convo.AgentSessionId.ToString(), out ConversationDto saved), Is.True);
-        Assert.That(saved.Turns, Has.Count.EqualTo(1));
-        Assert.That(saved.Turns[0].Prompt, Is.EqualTo("draw a box"));
-        Assert.That(saved.Lifecycle, Has.Count.EqualTo(1));
-    }
-
-    [Test]
-    [CancelAfter(10000)]
-    public async Task Second_PromptAsync_reuses_the_session_opening_a_second_turn_without_re_initializing()
-    {
-        FakeAcpAgent fake = new();
-        AgentRunner runner = new(ClaudeDef(), "Untitled", (client, _, _) => fake.AttachTo(client));
-
-        await runner.PromptAsync(UserMessage.FromText("first"), "http://localhost:10500/agent", "/tmp");
-        await runner.PromptAsync(UserMessage.FromText("second"), "http://localhost:10500/agent", "/tmp");
-
-        Conversation convo = runner.Conversation;
-
-        Assert.That(convo.Turns, Has.Count.EqualTo(2));
-        Assert.That(convo.Turns.Select(static t => t.Prompt), Is.EqualTo(new[] { "first", "second" }));
-
-        // Initialize + new-session happen once for the life of the runner; only PromptAsync repeats.
-        Assert.That(fake.InitializeCount, Is.EqualTo(1));
-        Assert.That(fake.NewSessionCount, Is.EqualTo(1));
-        Assert.That(fake.PromptCount, Is.EqualTo(2));
-
-        // Still a single session-started lifecycle event across both turns.
-        Assert.That(convo.Lifecycle, Has.Count.EqualTo(1));
-    }
-
-    // A real (classicist) IAcpAgent the runner drives: it captures the runner's own RhinoAcpClient and,
-    // on each prompt, streams one assistant-message update back through it exactly as a live agent's
-    // read loop would, so recording flows through the real seam. Counters pin the lifecycle calls.
-    private sealed class FakeAcpAgent : IAcpAgent
-    {
-        private IAcpClient? Client { get; set; }
-        private string SessionId { get; set; } = string.Empty;
-
-        public int InitializeCount { get; private set; }
-        public int NewSessionCount { get; private set; }
-        public int PromptCount { get; private set; }
-
-        public IAcpAgent AttachTo(IAcpClient client)
-        {
-            Client = client;
-            return this;
-        }
-
-        public ValueTask<InitializeResponse> InitializeAsync(InitializeRequest request, CancellationToken cancellationToken = default)
-        {
-            InitializeCount++;
-            return new ValueTask<InitializeResponse>(new InitializeResponse { ProtocolVersion = ProtocolConstants.Version });
-        }
-
-        public ValueTask<NewSessionResponse> SessionNewAsync(NewSessionRequest request, CancellationToken cancellationToken = default)
-        {
-            NewSessionCount++;
-            SessionId = "session-" + NewSessionCount;
-            return new ValueTask<NewSessionResponse>(new NewSessionResponse { SessionId = SessionId });
-        }
-
-        public async ValueTask<PromptResponse> SessionPromptAsync(PromptRequest request, CancellationToken cancellationToken = default)
-        {
-            PromptCount++;
-            IAcpClient client = Client ?? throw new InvalidOperationException("FakeAcpAgent was not attached to a client.");
-            await client.SessionUpdateAsync(new SessionNotification
-            {
-                SessionId = SessionId,
-                Update = new AgentMessageChunkSessionUpdate { Content = new TextContentBlock { Text = "on it" } },
-            }, cancellationToken).ConfigureAwait(false);
-            return new PromptResponse { StopReason = StopReason.EndTurn };
-        }
-
-        public ValueTask SessionCancelAsync(CancelNotification notification, CancellationToken cancellationToken = default) => default;
-
-        public ValueTask<AuthenticateResponse> AuthenticateAsync(AuthenticateRequest request, CancellationToken cancellationToken = default) =>
-            new(new AuthenticateResponse());
-
-        public ValueTask<LoadSessionResponse> SessionLoadAsync(LoadSessionRequest request, CancellationToken cancellationToken = default) =>
-            new(new LoadSessionResponse());
-
-        public ValueTask<SetSessionModeResponse> SessionSetModeAsync(SetSessionModeRequest request, CancellationToken cancellationToken = default) =>
-            new(new SetSessionModeResponse());
-
-        public ValueTask<JsonElement> ExtMethodAsync(string method, JsonElement @params, CancellationToken cancellationToken = default) =>
-            new(default(JsonElement));
-
-        public ValueTask ExtNotificationAsync(string method, JsonElement @params, CancellationToken cancellationToken = default) => default;
-    }
-}

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

@@ -1,118 +0,0 @@
-using Acp;
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// Canned-line tests for the Claude Code stream-json translation. Each test feeds one real-shaped
-// stdout line and asserts the emitted SessionUpdate kinds/fields, plus the terminal/junk contract.
-[TestFixture]
-public sealed class ClaudeStreamJsonParserTests
-{
-    private static ClaudeStreamJsonParser NewParser() =>
-        new(new AgentDefinition(
-            Name: "claude",
-            Adapter: AgentAdapter.Claude,
-            Command: "claude",
-            SearchPaths: [],
-            Model: "",
-            ExtraArgs: [],
-            SystemPrompt: "",
-            Enabled: true,
-            IsBuiltin: true));
-
-    [Test]
-    public void Assistant_text_emits_one_agent_message_chunk()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"type":"assistant","message":{"role":"assistant","content":[{"type":"text","text":"hi there"}]}}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.False);
-        Assert.That(parsed.Updates, Has.Count.EqualTo(1));
-        AgentMessageChunkSessionUpdate chunk = (AgentMessageChunkSessionUpdate)parsed.Updates[0];
-        Assert.That(((TextContentBlock)chunk.Content).Text, Is.EqualTo("hi there"));
-    }
-
-    [Test]
-    public void Assistant_tool_use_emits_tool_call_with_id_title_and_raw_input()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"type":"assistant","message":{"role":"assistant","content":[{"type":"tool_use","id":"toolu_1","name":"mcp__rhino__add_box","input":{"x":1}}]}}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.False);
-        Assert.That(parsed.Updates, Has.Count.EqualTo(1));
-        ToolCallSessionUpdate call = (ToolCallSessionUpdate)parsed.Updates[0];
-        Assert.That(call.ToolCallId, Is.EqualTo("toolu_1"));
-        Assert.That(call.Title, Is.EqualTo("mcp__rhino__add_box"));
-        Assert.That(call.RawInput, Is.Not.Null);
-        Assert.That(call.RawInput!.Value.GetProperty("x").GetInt32(), Is.EqualTo(1));
-    }
-
-    [Test]
-    public void User_tool_result_emits_completed_tool_call_update_with_raw_output()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"type":"user","message":{"role":"user","content":[{"type":"tool_result","tool_use_id":"toolu_1","content":"done"}]}}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.False);
-        Assert.That(parsed.Updates, Has.Count.EqualTo(1));
-        ToolCallUpdateSessionUpdate update = (ToolCallUpdateSessionUpdate)parsed.Updates[0];
-        Assert.That(update.ToolCallId, Is.EqualTo("toolu_1"));
-        Assert.That(update.Status, Is.EqualTo(ToolCallStatus.Completed));
-        Assert.That(update.RawOutput, Is.Not.Null);
-        Assert.That(update.RawOutput!.Value.GetString(), Is.EqualTo("done"));
-    }
-
-    [Test]
-    public void Result_line_is_the_terminal_event_and_ends_the_turn()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"type":"result","subtype":"success","is_error":false,"result":"all done"}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.True);
-        Assert.That(parsed.Reason, Is.EqualTo(StopReason.EndTurn));
-        Assert.That(parsed.Updates, Is.Empty);
-        Assert.That(parsed.Usage.IsEmpty, Is.True); // no usage object -> Empty, not faulted
-    }
-
-    [Test]
-    public void Result_line_surfaces_token_usage_and_cost()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"type":"result","subtype":"success","is_error":false,"result":"done","total_cost_usd":0.0123,"usage":{"input_tokens":1500,"output_tokens":640}}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.True);
-        Assert.That(parsed.Usage.InputTokens, Is.EqualTo(1500));
-        Assert.That(parsed.Usage.OutputTokens, Is.EqualTo(640));
-        Assert.That(parsed.Usage.TotalTokens, Is.EqualTo(2140));
-        Assert.That(parsed.Usage.CostUsd, Is.EqualTo(0.0123m));
-    }
-
-    [Test]
-    public void Result_line_without_cost_is_tokens_only()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"type":"result","subtype":"success","is_error":false,"result":"done","usage":{"input_tokens":10,"output_tokens":5}}""");
-
-        Assert.That(parsed.Usage.TotalTokens, Is.EqualTo(15));
-        Assert.That(parsed.Usage.CostUsd, Is.Null);
-        Assert.That(parsed.Usage.IsEmpty, Is.False);
-    }
-
-    [Test]
-    public void Unknown_event_type_is_fail_soft_none()
-    {
-        ParsedLine parsed = NewParser().Parse("""{"type":"system","subtype":"init","cwd":"/tmp"}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.False);
-        Assert.That(parsed.Updates, Is.Empty);
-    }
-
-    [Test]
-    public void Junk_line_without_type_is_none()
-    {
-        ParsedLine parsed = NewParser().Parse("""{"unrelated":"shape"}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.False);
-        Assert.That(parsed.Updates, Is.Empty);
-    }
-}

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

@@ -1,100 +0,0 @@
-using Acp;
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// Canned-line tests for the Codex (`codex exec --experimental-json`) translation. These pin our
-// internal parser contract; they do NOT prove the contract matches the shipped binary (that is a
-// separate manual live check, per the design's codexApproach caveat).
-[TestFixture]
-public sealed class CodexStreamJsonParserTests
-{
-    private static CodexStreamJsonParser NewParser() =>
-        new(new AgentDefinition(
-            Name: "codex",
-            Adapter: AgentAdapter.Codex,
-            Command: "codex",
-            SearchPaths: [],
-            Model: "",
-            ExtraArgs: [],
-            SystemPrompt: "",
-            Enabled: true,
-            IsBuiltin: true));
-
-    [Test]
-    public void Session_configured_is_none_session_start_is_the_runners_concern()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"msg":{"type":"session_configured","session_id":"abc"}}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.False);
-        Assert.That(parsed.Updates, Is.Empty);
-    }
-
-    [Test]
-    public void Agent_message_emits_one_agent_message_chunk()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"msg":{"type":"agent_message","message":"working on it"}}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.False);
-        Assert.That(parsed.Updates, Has.Count.EqualTo(1));
-        AgentMessageChunkSessionUpdate chunk = (AgentMessageChunkSessionUpdate)parsed.Updates[0];
-        Assert.That(((TextContentBlock)chunk.Content).Text, Is.EqualTo("working on it"));
-    }
-
-    [Test]
-    public void Mcp_tool_call_emits_tool_call_titled_by_tool_name()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"msg":{"type":"mcp_tool_call","tool":"add_box","server":"rhino"}}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.False);
-        Assert.That(parsed.Updates, Has.Count.EqualTo(1));
-        ToolCallSessionUpdate call = (ToolCallSessionUpdate)parsed.Updates[0];
-        Assert.That(call.ToolCallId, Is.EqualTo("add_box"));
-        Assert.That(call.Title, Is.EqualTo("add_box"));
-    }
-
-    [Test]
-    public void Task_complete_is_the_terminal_event_and_ends_the_turn()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"msg":{"type":"task_complete","last_agent_message":"all done"}}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.True);
-        Assert.That(parsed.Reason, Is.EqualTo(StopReason.EndTurn));
-        Assert.That(parsed.Updates, Is.Empty);
-        Assert.That(parsed.Usage.IsEmpty, Is.True); // no usage object -> Empty, never faulted
-    }
-
-    [Test]
-    public void Task_complete_surfaces_token_usage_without_cost()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"msg":{"type":"task_complete","last_agent_message":"done","usage":{"input_tokens":200,"output_tokens":90}}}""");
-
-        Assert.That(parsed.Usage.InputTokens, Is.EqualTo(200));
-        Assert.That(parsed.Usage.OutputTokens, Is.EqualTo(90));
-        Assert.That(parsed.Usage.CostUsd, Is.Null); // Codex reports tokens only
-    }
-
-    [Test]
-    public void Unknown_event_type_is_fail_soft_none()
-    {
-        ParsedLine parsed = NewParser().Parse(
-            """{"msg":{"type":"token_count","input_tokens":42}}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.False);
-        Assert.That(parsed.Updates, Is.Empty);
-    }
-
-    [Test]
-    public void Junk_line_without_type_is_none()
-    {
-        ParsedLine parsed = NewParser().Parse("""{"unrelated":"shape"}""");
-
-        Assert.That(parsed.IsTurnComplete, Is.False);
-        Assert.That(parsed.Updates, Is.Empty);
-    }
-}