@hybridlabor-api/aos 4.3.0 → 4.4.0

package/mcps/RhinoMCP/tests/Server.Tests/ResourceRegistryTests.cs

@@ -1,218 +0,0 @@
-using System.Reflection;
-using Microsoft.Extensions.DependencyInjection;
-using NUnit.Framework;
-using RhMcp.Server;
-
-namespace RhMcp.Server.Tests;
-
-// Test-only resource types. Scan() discovers these via their attributes.
-[McpServerResourceType]
-internal sealed class FixtureResources
-{
-    [McpServerResource(UriTemplate = "rhino://config/version", Name = "version")]
-    public static string GetVersion() => "1.0";
-
-    [McpServerResource(UriTemplate = "rhino://config/{key}", Name = "byKey")]
-    public static string GetByKey(string key) => key;
-
-    [McpServerResource(UriTemplate = "rhino://docs/{section}/{page}", Name = "doc")]
-    public static string GetDoc(string section, string page) => $"{section}/{page}";
-
-    [McpServerResource(UriTemplate = "rhino://about", Name = "about", MimeType = "text/plain")]
-    public static string GetAbout() => "about";
-}
-
-internal sealed class NotADecoratedType
-{
-    [McpServerResource(UriTemplate = "rhino://ignored")]
-    public static string Ignored() => "ignored";
-}
-
-[TestFixture]
-public class ResourceRegistryTests
-{
-    private static IServiceProvider EmptyServices()
-        => new ServiceCollection().BuildServiceProvider();
-
-    private static ResourceRegistry BuildRegistry()
-        => ResourceRegistry.Scan(typeof(FixtureResources).Assembly, EmptyServices());
-
-    [Test]
-    public void Scan_discovers_resources_on_decorated_types_only()
-    {
-        ResourceRegistry registry = BuildRegistry();
-        string[] templates = registry.All.Select(h => h.UriTemplate).ToArray();
-        Assert.That(templates, Does.Contain("rhino://config/version"));
-        Assert.That(templates, Does.Contain("rhino://config/{key}"));
-        Assert.That(templates, Does.Contain("rhino://docs/{section}/{page}"));
-        Assert.That(templates, Does.Contain("rhino://about"));
-        Assert.That(templates, Does.Not.Contain("rhino://ignored"),
-            "methods on undecorated types must not be registered");
-    }
-
-    [Test]
-    public void StaticResources_excludes_templated_handlers()
-    {
-        ResourceRegistry registry = BuildRegistry();
-        string[] templates = registry.StaticResources.Select(h => h.UriTemplate).ToArray();
-        Assert.That(templates, Is.EquivalentTo(new[]
-        {
-            "rhino://config/version",
-            "rhino://about",
-        }));
-    }
-
-    [Test]
-    public void Templated_excludes_static_handlers()
-    {
-        ResourceRegistry registry = BuildRegistry();
-        string[] templates = registry.Templated.Select(h => h.UriTemplate).ToArray();
-        Assert.That(templates, Is.EquivalentTo(new[]
-        {
-            "rhino://config/{key}",
-            "rhino://docs/{section}/{page}",
-        }));
-    }
-
-    [Test]
-    public void TryMatch_static_exact_match_returns_empty_variables()
-    {
-        ResourceHandler h = new(
-            method: typeof(FixtureResources).GetMethod(nameof(FixtureResources.GetAbout))!,
-            uriTemplate: "rhino://about",
-            name: "about",
-            description: null,
-            mimeType: null,
-            services: EmptyServices());
-
-        bool matched = h.TryMatch("rhino://about", out IReadOnlyDictionary<string, string> vars);
-        Assert.That(matched, Is.True);
-        Assert.That(vars, Is.Empty);
-    }
-
-    [Test]
-    public void TryMatch_static_rejects_other_uris()
-    {
-        ResourceHandler h = new(
-            method: typeof(FixtureResources).GetMethod(nameof(FixtureResources.GetAbout))!,
-            uriTemplate: "rhino://about",
-            name: "about",
-            description: null,
-            mimeType: null,
-            services: EmptyServices());
-
-        Assert.That(h.TryMatch("rhino://about/extra", out _), Is.False);
-        Assert.That(h.TryMatch("rhino://other", out _), Is.False);
-    }
-
-    [Test]
-    public void TryMatch_template_captures_named_variable()
-    {
-        ResourceHandler h = new(
-            method: typeof(FixtureResources).GetMethod(nameof(FixtureResources.GetByKey))!,
-            uriTemplate: "rhino://config/{key}",
-            name: "byKey",
-            description: null,
-            mimeType: null,
-            services: EmptyServices());
-
-        bool matched = h.TryMatch("rhino://config/theme", out IReadOnlyDictionary<string, string> vars);
-        Assert.That(matched, Is.True);
-        Assert.That(vars["key"], Is.EqualTo("theme"));
-    }
-
-    [Test]
-    public void TryMatch_template_captures_multiple_variables()
-    {
-        ResourceHandler h = new(
-            method: typeof(FixtureResources).GetMethod(nameof(FixtureResources.GetDoc))!,
-            uriTemplate: "rhino://docs/{section}/{page}",
-            name: "doc",
-            description: null,
-            mimeType: null,
-            services: EmptyServices());
-
-        bool matched = h.TryMatch("rhino://docs/intro/install", out IReadOnlyDictionary<string, string> vars);
-        Assert.That(matched, Is.True);
-        Assert.That(vars["section"], Is.EqualTo("intro"));
-        Assert.That(vars["page"], Is.EqualTo("install"));
-    }
-
-    [Test]
-    public void TryMatch_template_does_not_cross_slashes()
-    {
-        ResourceHandler h = new(
-            method: typeof(FixtureResources).GetMethod(nameof(FixtureResources.GetByKey))!,
-            uriTemplate: "rhino://config/{key}",
-            name: "byKey",
-            description: null,
-            mimeType: null,
-            services: EmptyServices());
-
-        Assert.That(h.TryMatch("rhino://config/a/b", out _), Is.False,
-            "template variables match anything except '/'");
-    }
-
-    [Test]
-    public void TryMatch_template_rejects_partial_uri()
-    {
-        ResourceHandler h = new(
-            method: typeof(FixtureResources).GetMethod(nameof(FixtureResources.GetByKey))!,
-            uriTemplate: "rhino://config/{key}",
-            name: "byKey",
-            description: null,
-            mimeType: null,
-            services: EmptyServices());
-
-        Assert.That(h.TryMatch("rhino://config/", out _), Is.False,
-            "the variable segment is at least one char");
-        Assert.That(h.TryMatch("rhino://other/value", out _), Is.False);
-    }
-
-    [Test]
-    public void Constructor_rejects_empty_variable_name()
-    {
-        MethodInfo method = typeof(FixtureResources).GetMethod(nameof(FixtureResources.GetByKey))!;
-        Assert.Throws<InvalidOperationException>(() => new ResourceHandler(
-            method,
-            uriTemplate: "rhino://config/{}",
-            name: "bad",
-            description: null,
-            mimeType: null,
-            services: EmptyServices()));
-    }
-
-    [Test]
-    public void Constructor_rejects_unterminated_brace()
-    {
-        MethodInfo method = typeof(FixtureResources).GetMethod(nameof(FixtureResources.GetByKey))!;
-        Assert.Throws<InvalidOperationException>(() => new ResourceHandler(
-            method,
-            uriTemplate: "rhino://config/{key",
-            name: "bad",
-            description: null,
-            mimeType: null,
-            services: EmptyServices()));
-    }
-
-    // Register the templated handler first to defeat any incidental ordering
-    // from reflection — the static handler must still win.
-    [Test]
-    public void Match_prefers_static_over_templated_when_both_apply()
-    {
-        ResourceRegistry registry = new();
-        IServiceProvider sp = EmptyServices();
-        registry.Add(new ResourceHandler(
-            typeof(FixtureResources).GetMethod(nameof(FixtureResources.GetByKey))!,
-            "rhino://config/{key}", "byKey", null, null, sp));
-        registry.Add(new ResourceHandler(
-            typeof(FixtureResources).GetMethod(nameof(FixtureResources.GetVersion))!,
-            "rhino://config/version", "version", null, null, sp));
-
-        ResourceHandler? matched = registry.Match("rhino://config/version", out IReadOnlyDictionary<string, string> vars);
-        Assert.That(matched, Is.Not.Null);
-        Assert.That(matched!.UriTemplate, Is.EqualTo("rhino://config/version"),
-            "exact-URI handler should win over a templated handler that also matches");
-        Assert.That(vars, Is.Empty);
-    }
-}

package/mcps/RhinoMCP/tests/Server.Tests/ResultUnwrapperTests.cs

@@ -1,53 +0,0 @@
-using NUnit.Framework;
-using RhMcp.Server;
-
-namespace RhMcp.Server.Tests;
-
-[TestFixture]
-public class ResultUnwrapperTests
-{
-    [Test]
-    public async Task Null_unwraps_to_null()
-    {
-        object? result = await ResultUnwrapper.UnwrapAsync(null);
-        Assert.That(result, Is.Null);
-    }
-
-    // Task.CompletedTask is actually a Task<VoidTaskResult> singleton at
-    // runtime — the unwrapper must surface that as null, not leak the
-    // VoidTaskResult sentinel struct.
-    [Test]
-    public async Task NonGeneric_Task_unwraps_to_null()
-    {
-        object? result = await ResultUnwrapper.UnwrapAsync(Task.CompletedTask);
-        Assert.That(result, Is.Null);
-    }
-
-    [Test]
-    public async Task Generic_Task_unwraps_to_inner_value()
-    {
-        object? result = await ResultUnwrapper.UnwrapAsync(Task.FromResult(42));
-        Assert.That(result, Is.EqualTo(42));
-    }
-
-    [Test]
-    public async Task NonGeneric_ValueTask_unwraps_to_null()
-    {
-        object? result = await ResultUnwrapper.UnwrapAsync(new ValueTask());
-        Assert.That(result, Is.Null);
-    }
-
-    [Test]
-    public async Task Generic_ValueTask_unwraps_to_inner_value()
-    {
-        object? result = await ResultUnwrapper.UnwrapAsync(new ValueTask<string>("done"));
-        Assert.That(result, Is.EqualTo("done"));
-    }
-
-    [Test]
-    public async Task Synchronous_value_passes_through()
-    {
-        object? result = await ResultUnwrapper.UnwrapAsync("plain");
-        Assert.That(result, Is.EqualTo("plain"));
-    }
-}

package/mcps/RhinoMCP/tests/Server.Tests/SchemaBuilderTests.cs

@@ -1,166 +0,0 @@
-using System.Reflection;
-using System.Text.Json;
-using Microsoft.Extensions.DependencyInjection;
-using NUnit.Framework;
-using RhMcp.Server;
-
-namespace RhMcp.Server.Tests;
-
-[TestFixture]
-public class SchemaBuilderTests
-{
-    private enum Color { Red, Green, Blue }
-
-    private class SampleMethods
-    {
-        public void Required(
-            string name,
-            int count,
-            int? maybeCount,
-            string label = "x",
-            bool flag = false) { }
-
-        public void Types(
-            string s, bool b, int i, long l, double d, decimal m,
-            System.Guid g, System.DateTime dt, System.TimeSpan ts,
-            Color color,
-            int[] ints,
-            List<string> strs,
-            IEnumerable<int> nums,
-            Dictionary<string, int> map,
-            SampleMethods complex) { }
-    }
-
-    private static ParameterDescriptor Arg(string method, string param)
-    {
-        ParameterInfo pi = typeof(SampleMethods).GetMethod(method)!
-            .GetParameters().Single(p => p.Name == param);
-        return new ParameterDescriptor(pi, ParameterBindingKind.Argument);
-    }
-
-    private static JsonElement Build(params ParameterDescriptor[] descriptors)
-        => SchemaBuilder.BuildInputSchema(descriptors);
-
-    [Test]
-    public void Schema_root_is_object_with_properties()
-    {
-        JsonElement schema = Build(Arg(nameof(SampleMethods.Required), "name"));
-        Assert.That(schema.GetProperty("type").GetString(), Is.EqualTo("object"));
-        Assert.That(schema.TryGetProperty("properties", out _), Is.True);
-    }
-
-    [Test]
-    public void Required_array_includes_value_type_with_no_default()
-    {
-        JsonElement schema = Build(Arg(nameof(SampleMethods.Required), "count"));
-        Assert.That(schema.TryGetProperty("required", out JsonElement required), Is.True);
-        string[] names = required.EnumerateArray().Select(e => e.GetString()!).ToArray();
-        Assert.That(names, Does.Contain("count"));
-    }
-
-    [Test]
-    public void Required_array_omits_value_type_with_default()
-    {
-        JsonElement schema = Build(Arg(nameof(SampleMethods.Required), "flag"));
-        Assert.That(schema.TryGetProperty("required", out _), Is.False,
-            "params with default values must not appear in `required`");
-    }
-
-    [Test]
-    public void Required_array_omits_nullable_value_type()
-    {
-        JsonElement schema = Build(Arg(nameof(SampleMethods.Required), "maybeCount"));
-        Assert.That(schema.TryGetProperty("required", out _), Is.False,
-            "Nullable<T> params are implicitly optional");
-    }
-
-    [Test]
-    public void Required_array_includes_non_nullable_reference_type_with_no_default()
-    {
-        JsonElement schema = Build(Arg(nameof(SampleMethods.Required), "name"));
-        Assert.That(schema.TryGetProperty("required", out JsonElement required), Is.True,
-            "string parameter without a default should be required");
-        string[] names = required.EnumerateArray().Select(e => e.GetString()!).ToArray();
-        Assert.That(names, Does.Contain("name"));
-    }
-
-    [Test]
-    public void Required_array_omitted_when_no_params_are_required()
-    {
-        JsonElement schema = Build(Arg(nameof(SampleMethods.Required), "label"));
-        Assert.That(schema.TryGetProperty("required", out _), Is.False);
-    }
-
-    [TestCase("s", "string")]
-    [TestCase("b", "boolean")]
-    [TestCase("i", "integer")]
-    [TestCase("l", "integer")]
-    [TestCase("d", "number")]
-    [TestCase("m", "number")]
-    [TestCase("g", "string")]
-    [TestCase("dt", "string")]
-    [TestCase("ts", "string")]
-    [TestCase("ints", "array")]
-    [TestCase("strs", "array")]
-    [TestCase("nums", "array")]
-    [TestCase("complex", "object")]
-    public void MapType_emits_expected_json_type(string paramName, string expected)
-    {
-        JsonElement schema = Build(Arg(nameof(SampleMethods.Types), paramName));
-        string actual = schema.GetProperty("properties").GetProperty(paramName)
-            .GetProperty("type").GetString()!;
-        Assert.That(actual, Is.EqualTo(expected));
-    }
-
-    // Round-trip: whatever representation the schema advertises for an enum,
-    // the binder must accept that same representation. Catches drift between
-    // SchemaBuilder.MapType and McpSerializer.Options enum-converter setup.
-    [Test]
-    public void Enum_schema_and_binder_agree_on_representation()
-    {
-        ParameterDescriptor desc = Arg(nameof(SampleMethods.Types), "color");
-        JsonElement schema = Build(desc);
-        string schemaType = schema.GetProperty("properties").GetProperty("color")
-            .GetProperty("type").GetString()!;
-
-        string argJson = schemaType switch
-        {
-            "integer" => """{ "color": 1 }""",
-            "string" => """{ "color": "Green" }""",
-            _ => throw new AssertionException($"Unsupported enum schema type '{schemaType}'"),
-        };
-        JsonDocument doc = JsonDocument.Parse(argJson);
-        Dictionary<string, JsonElement> args = new();
-        foreach (JsonProperty prop in doc.RootElement.EnumerateObject())
-            args[prop.Name] = prop.Value.Clone();
-
-        object? value = ParameterBinder.Resolve(
-            desc, args, new ServiceCollection().BuildServiceProvider(), default);
-        Assert.That(value, Is.EqualTo(Color.Green));
-    }
-
-    // Dictionary<,> falls through to "object" with no inner shape. It must
-    // not be advertised as "array" — that would mislead clients.
-    [Test]
-    public void Dictionary_param_is_object_not_array()
-    {
-        JsonElement schema = Build(Arg(nameof(SampleMethods.Types), "map"));
-        string actual = schema.GetProperty("properties").GetProperty("map")
-            .GetProperty("type").GetString()!;
-        Assert.That(actual, Is.EqualTo("object"));
-    }
-
-    [Test]
-    public void Service_and_cancellation_params_are_excluded_from_schema()
-    {
-        ParameterInfo nameParam = typeof(SampleMethods).GetMethod(nameof(SampleMethods.Required))!
-            .GetParameters().Single(p => p.Name == "name");
-        ParameterDescriptor service = new(nameParam, ParameterBindingKind.Service);
-        ParameterDescriptor ct = new(nameParam, ParameterBindingKind.CancellationToken);
-        ParameterDescriptor templ = new(nameParam, ParameterBindingKind.UriTemplate);
-
-        JsonElement schema = Build(service, ct, templ);
-        JsonElement props = schema.GetProperty("properties");
-        Assert.That(props.EnumerateObject().Count(), Is.EqualTo(0));
-    }
-}

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

@@ -1,40 +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.Server.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" />
-    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="8.0.0" />
-  </ItemGroup>
-
-  <!--
-    Pull only the pure (non-Rhino, non-AspNetCore) Server sources into the test
-    assembly. McpEndpoint.cs depends on Microsoft.AspNetCore and ToolRegistry.cs
-    depends on RhinoApp; those need a different host to test and live elsewhere.
-    Compiling the sources in (rather than ProjectReference'ing RhMcp.csproj)
-    keeps the test assembly RhinoCommon-free and side-steps internals-visibility.
-  -->
-  <ItemGroup>
-    <Compile Include="../../rhino/plugin/Server/Attributes.cs" LinkBase="ServerSources" />
-    <Compile Include="../../rhino/plugin/Server/Protocol.cs" LinkBase="ServerSources" />
-    <Compile Include="../../rhino/plugin/Server/SchemaBuilder.cs" LinkBase="ServerSources" />
-    <Compile Include="../../rhino/plugin/Server/ParameterBinder.cs" LinkBase="ServerSources" />
-    <Compile Include="../../rhino/plugin/Server/ResourceRegistry.cs" LinkBase="ServerSources" />
-  </ItemGroup>
-
-  <ItemGroup>
-    <Compile Include="../../rhino/plugin/McpConfigEdit.cs" LinkBase="PluginSources" />
-  </ItemGroup>
-
-</Project>

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

@@ -1,82 +0,0 @@
-using System.Text;
-using Acp;
-using RhMcp;
-using ContentBlock = Acp.ContentBlock;
-
-namespace RhMcp.StreamJson.Tests;
-
-// AcpMessageMapper is the UserMessage/attachment <-> ACP ContentBlock seam. The agent has no
-// filesystem access, so every attachment must arrive inline: images base64, text files fenced.
-// These pin that mapping plus the empty/edge shapes and the TextOf reverse direction.
-[TestFixture]
-public sealed class AcpMessageMapperTests
-{
-    private static Attachment Image(byte[] data, string media = "image/png") =>
-        new(AttachmentKind.Image, "shot.png", media, data);
-
-    private static Attachment TextFile(string body, string name = "notes.txt") =>
-        new(AttachmentKind.TextFile, name, "text/plain", Encoding.UTF8.GetBytes(body));
-
-    [Test]
-    public void Text_only_message_maps_to_a_single_text_block()
-    {
-        ContentBlock[] blocks = AcpMessageMapper.Prompt(UserMessage.FromText("draw a box"));
-
-        Assert.That(blocks, Has.Length.EqualTo(1));
-        Assert.That(((TextContentBlock)blocks[0]).Text, Is.EqualTo("draw a box"));
-    }
-
-    [Test]
-    public void Empty_text_with_no_attachments_yields_no_blocks()
-    {
-        ContentBlock[] blocks = AcpMessageMapper.Prompt(UserMessage.FromText(string.Empty));
-
-        Assert.That(blocks, Is.Empty);
-    }
-
-    [Test]
-    public void Image_attachment_is_inlined_as_base64_with_its_media_type()
-    {
-        byte[] payload = [1, 2, 3, 4];
-        UserMessage message = new(string.Empty, [Image(payload, "image/jpeg")]);
-
-        ContentBlock[] blocks = AcpMessageMapper.Prompt(message);
-
-        Assert.That(blocks, Has.Length.EqualTo(1));
-        ImageContentBlock image = (ImageContentBlock)blocks[0];
-        Assert.That(image.Data, Is.EqualTo(Convert.ToBase64String(payload)));
-        Assert.That(image.MimeType, Is.EqualTo("image/jpeg"));
-    }
-
-    [Test]
-    public void Text_file_attachment_is_fenced_with_its_name_and_decoded_body()
-    {
-        UserMessage message = new(string.Empty, [TextFile("hello world", "spec.md")]);
-
-        ContentBlock[] blocks = AcpMessageMapper.Prompt(message);
-
-        Assert.That(blocks, Has.Length.EqualTo(1));
-        Assert.That(((TextContentBlock)blocks[0]).Text, Is.EqualTo("```spec.md\nhello world\n```"));
-    }
-
-    [Test]
-    public void Text_then_attachments_preserve_order_text_first()
-    {
-        UserMessage message = new("look at this", [Image([9]), TextFile("body")]);
-
-        ContentBlock[] blocks = AcpMessageMapper.Prompt(message);
-
-        Assert.That(blocks, Has.Length.EqualTo(3));
-        Assert.That(blocks[0], Is.InstanceOf<TextContentBlock>());
-        Assert.That(((TextContentBlock)blocks[0]).Text, Is.EqualTo("look at this"));
-        Assert.That(blocks[1], Is.InstanceOf<ImageContentBlock>());
-        Assert.That(blocks[2], Is.InstanceOf<TextContentBlock>());
-    }
-
-    [Test]
-    public void TextOf_returns_the_text_of_a_text_block_and_empty_for_non_text()
-    {
-        Assert.That(AcpMessageMapper.TextOf(new TextContentBlock { Text = "hi" }), Is.EqualTo("hi"));
-        Assert.That(AcpMessageMapper.TextOf(new ImageContentBlock { Data = "x", MimeType = "image/png" }), Is.Empty);
-    }
-}

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

@@ -1,56 +0,0 @@
-using RhMcp;
-
-namespace RhMcp.StreamJson.Tests;
-
-// Pins the pull-only grounding contract (PLAN locked decision: no auto-injection, mitigated by an
-// always-on steer that is "never dropped"). Compose is the one place that invariant lives in code,
-// so these tests guard that a future edit can't silently drop a steer or let a custom system prompt
-// replace (rather than follow) them, for both the empty- and non-empty-prompt branches.
-[TestFixture]
-public sealed class AgentPromptsTests
-{
-    [Test]
-    public void Compose_with_empty_prompt_carries_all_three_always_on_steers()
-    {
-        string composed = AgentPrompts.Compose(string.Empty);
-
-        Assert.That(composed, Does.Contain(AgentPrompts.AskUserSteer));
-        Assert.That(composed, Does.Contain(AgentPrompts.GroundingSteer));
-        Assert.That(composed, Does.Contain(AgentPrompts.GrasshopperSteer));
-    }
-
-    [Test]
-    public void Compose_with_a_custom_prompt_keeps_every_steer()
-    {
-        string composed = AgentPrompts.Compose("custom");
-
-        Assert.That(composed, Does.Contain(AgentPrompts.AskUserSteer));
-        Assert.That(composed, Does.Contain(AgentPrompts.GroundingSteer));
-        Assert.That(composed, Does.Contain(AgentPrompts.GrasshopperSteer));
-    }
-
-    [Test]
-    public void Compose_appends_the_system_prompt_after_the_steers_never_replacing_them()
-    {
-        const string systemPrompt = "agent-specific instructions";
-        string composed = AgentPrompts.Compose(systemPrompt);
-
-        Assert.That(composed, Does.Contain(systemPrompt));
-        // The prompt follows the steers, never replaces them: every steer index precedes it.
-        Assert.That(composed.IndexOf(systemPrompt, StringComparison.Ordinal),
-            Is.GreaterThan(composed.IndexOf(AgentPrompts.AskUserSteer, StringComparison.Ordinal)));
-        Assert.That(composed.IndexOf(systemPrompt, StringComparison.Ordinal),
-            Is.GreaterThan(composed.IndexOf(AgentPrompts.GroundingSteer, StringComparison.Ordinal)));
-        Assert.That(composed.IndexOf(systemPrompt, StringComparison.Ordinal),
-            Is.GreaterThan(composed.IndexOf(AgentPrompts.GrasshopperSteer, StringComparison.Ordinal)));
-    }
-
-    [Test]
-    public void Compose_with_empty_prompt_is_the_steers_alone_with_no_trailing_prompt()
-    {
-        // The empty branch must be exactly the three steers (no stray appended segment), so a future
-        // edit that always appends can't smuggle an empty trailing block.
-        string steersOnly = AgentPrompts.AskUserSteer + "\n\n" + AgentPrompts.GroundingSteer + "\n\n" + AgentPrompts.GrasshopperSteer;
-        Assert.That(AgentPrompts.Compose(string.Empty), Is.EqualTo(steersOnly));
-    }
-}