@polymorphism-tech/morph-spec 8.37.1 → 8.40.0

package/framework/templates/dotnet/ai-kit/tests/Morph.AiKit.Tests/MediaReviewTests.cs

@@ -3,15 +3,6 @@ using Xunit;
 
 namespace Morph.AiKit.Tests;
 
-/// <summary>
-/// Canarios de <see cref="MediaReview"/>.
-/// </summary>
-/// <remarks>
-/// A afirmacao que este arquivo protege nao e "aprovar funciona" — e que
-/// <see cref="MediaReview"/> <b>delega</b> ao <see cref="MediaJob"/> em vez de reimplementar
-/// a regra. Uma segunda copia de "so Approved publica" e exatamente como as duas divergem, e
-/// e o padrao de falha que esta epic catalogou sete vezes: o teste mede uma COPIA do objeto.
-/// </remarks>
 public sealed class MediaReviewTests
 {
     private static readonly DateTimeOffset T0 = new(2026, 9, 8, 12, 0, 0, TimeSpan.Zero);
@@ -31,7 +22,6 @@ public sealed class MediaReviewTests
             new MediaProvenance("openai", "gpt-image-2", "p", null, [], "-", null, 0m, T0, true)).Value;
     }
 
-    /// <summary>O atalho aprova e carimba.</summary>
     [Fact]
     public void ApproveDelegatesAndStamps()
     {
@@ -42,7 +32,6 @@ public sealed class MediaReviewTests
         Assert.Equal("lucas", result.Value.Review!.Reviewer);
     }
 
-    /// <summary>Rejeitar e terminal e tambem carimba.</summary>
     [Fact]
     public void RejectDelegatesAndStamps()
     {
@@ -53,12 +42,6 @@ public sealed class MediaReviewTests
         Assert.Equal(MediaReviewDecision.Rejected, result.Value.Review!.Decision);
     }
 
-    /// <summary>
-    /// O atalho de publicar NAO tem uma regra propria: ele recusa exatamente onde
-    /// <see cref="MediaJob.Publish"/> recusa. Confere-se comparando as DUAS chamadas sobre o
-    /// MESMO job — se <c>MediaReview</c> tivesse copiado a regra, aqui e onde a copia
-    /// divergiria.
-    /// </summary>
     [Fact]
     public void PublishThroughTheReviewFacadeAgreesWithTheJobItself()
     {
@@ -72,12 +55,10 @@ public sealed class MediaReviewTests
         Assert.True(MediaReview.Publish(approved, Label, T0).IsSuccess);
     }
 
-    /// <summary>Job nulo e falha de composicao, nao Result: quem chama errou o codigo.</summary>
     [Fact]
     public void ANullJobIsAProgrammingError()
         => Assert.Throws<ArgumentNullException>(() => MediaReview.Approve(null!, "lucas", T0));
 
-    /// <summary>Um juiz LLM cabe na nota, nunca no campo do revisor de uma foto de catalogo.</summary>
     [Fact]
     public void TheNoteCarriesThePreFilterVerdict()
     {

package/framework/templates/dotnet/ai-kit/tests/Morph.AiKit.Tests/MediaStoreTests.cs

@@ -5,22 +5,12 @@ using Xunit;
 
 namespace Morph.AiKit.Tests;
 
-/// <summary>
-/// Canarios do <see cref="MediaStore"/>: endereco por conteudo, e gravacao unica.
-/// </summary>
-/// <remarks>
-/// A armadilha deste arquivo, e a razao de <see cref="InMemoryMediaBlobs.Writes"/> existir:
-/// um teste que so conferisse "o hash e o mesmo" <b>nao distingue dedupe de regravacao</b>,
-/// porque os dois produzem o mesmo hash. A afirmacao "grava uma vez" so e verificavel
-/// contando gravacoes no blob.
-/// </remarks>
 public sealed class MediaStoreTests
 {
     private static CancellationToken Ct => TestContext.Current.CancellationToken;
 
     private static ReadOnlyMemory<byte> Bytes(string s) => Encoding.UTF8.GetBytes(s);
 
-    /// <summary>Mesmo conteudo, mesmo hash, mesmo caminho — e UMA gravacao.</summary>
     [Fact]
     public async Task TheSameContentIsStoredOnce()
     {
@@ -37,7 +27,6 @@ public sealed class MediaStoreTests
         Assert.Equal(1, blobs.Writes);
     }
 
-    /// <summary>Conteudo diferente, endereco diferente, duas gravacoes.</summary>
     [Fact]
     public async Task DifferentContentGetsADifferentAddress()
     {
@@ -52,18 +41,10 @@ public sealed class MediaStoreTests
         Assert.Equal(2, blobs.Writes);
     }
 
-    /// <summary>
-    /// O hash e o do CONTEUDO, e nao do conteudo mais o MIME: o mesmo PNG servido como
-    /// <c>image/png</c> e como <c>application/octet-stream</c> e o mesmo arquivo.
-    /// </summary>
     [Fact]
     public void TheHashIsOfTheContentAlone()
         => Assert.Equal(MediaHash.Of(Bytes("x").Span), MediaHash.Of(Bytes("x").Span));
 
-    /// <summary>
-    /// O caminho tem dois niveis de prefixo. Um diretorio unico com centenas de milhares de
-    /// arquivos degrada listagem em praticamente todo sistema de arquivos.
-    /// </summary>
     [Fact]
     public void ThePathIsShardedByTheFirstFourHexDigits()
     {
@@ -73,10 +54,6 @@ public sealed class MediaStoreTests
         Assert.Equal($"{hash.Value[..2]}/{hash.Value[2..4]}/{hash.Value}.png", path);
     }
 
-    /// <summary>
-    /// MIME desconhecido vira <c>.bin</c> e NAO recusa: perder a extensao e cosmetico,
-    /// perder os bytes pelos quais se pagou nao e.
-    /// </summary>
     [Theory]
     [InlineData("image/png", ".png")]
     [InlineData("image/jpeg", ".jpg")]
@@ -86,7 +63,6 @@ public sealed class MediaStoreTests
     public void UnknownMediaTypesStillGetAPath(string mediaType, string expected)
         => Assert.Equal(expected, MediaStore.ExtensionFor(mediaType));
 
-    /// <summary>MIME em branco e defeito de chamada, e grita no lugar em que aconteceu.</summary>
     [Fact]
     public async Task ABlankMediaTypeIsRefused()
     {

package/framework/templates/dotnet/ai-kit/tests/Morph.AiKit.Tests/ModelRegistryTests.cs

@@ -6,14 +6,8 @@ using Xunit;
 
 namespace Morph.AiKit.Tests;
 
-/// <summary>
-/// Canarios do <see cref="ModelRegistry"/> (T8). Tres coisas sao cobradas aqui, e as tres
-/// sao as que o registry v1 nao tinha: <b>protocolo por alias</b>, <b>tarifa por alias</b>
-/// e <b>falha explicita</b> onde antes havia nulo silencioso.
-/// </summary>
 public sealed class ModelRegistryTests
 {
-    /// <summary>O exemplar de verdade, embutido do proprio arquivo que o kit publica.</summary>
     private static string ExampleDocument()
     {
         using var stream = typeof(ModelRegistryTests).Assembly
@@ -33,10 +27,6 @@ public sealed class ModelRegistryTests
         }
         """;
 
-    // ---------------------------------------------------------------------------------
-    // O exemplar publicado carrega — e expoe api e pricing por alias
-    // ---------------------------------------------------------------------------------
-
     [Fact]
     public void ThePublishedExampleLoadsAndExposesApiAndPricingPerAlias()
     {
@@ -48,11 +38,9 @@ public sealed class ModelRegistryTests
 
         Assert.Equal("text-default", registry.DefaultAlias);
 
-        // 'api' omitido cai em Responses — a decisao da casa, verificada e nao suposta.
         Assert.Equal(AgentApi.Responses, registry.Resolve("text-default").Api);
         Assert.Equal(AgentApi.Responses, registry.Resolve("text-heavy").Api);
 
-        // ... e o MESMO modelo, sob outro alias, pode falar Chat Completions.
         var chat = registry.Resolve("text-heavy-chat");
         Assert.Equal(AgentApi.Chat, chat.Api);
         Assert.Equal(registry.Resolve("text-heavy").Model, chat.Model);
@@ -62,7 +50,6 @@ public sealed class ModelRegistryTests
         Assert.Equal(10.0m, pricing.OutputPerMTok);
         Assert.Equal(0.125m, pricing.CachedInputPerMTok);
 
-        // cachedInputPerMTok explicitamente nulo = sem desconto declarado (nao "de graca").
         Assert.Null(registry.Resolve("vision").RequirePricing().CachedInputPerMTok);
 
         Assert.Equal("http://localhost:11434/v1", registry.Resolve("local-dev").Endpoint);
@@ -80,10 +67,6 @@ public sealed class ModelRegistryTests
         Assert.Contains("reasoning_effort", constraints.Provenance!, StringComparison.Ordinal);
     }
 
-    // ---------------------------------------------------------------------------------
-    // Tarifa ausente e ALTO, nunca nulo silencioso
-    // ---------------------------------------------------------------------------------
-
     [Fact]
     public void AnAliasWithNeitherPricingNorWaiverIsRefusedAtLoadNamingTheAlias()
     {
@@ -151,7 +134,7 @@ public sealed class ModelRegistryTests
     [Fact]
     public void ATypoInAFieldNameIsRefusedInsteadOfSilentlyIgnored()
     {
-        // "priceing" seria lido como "sem pricing" — e a ausencia silenciosa e o defeito.
+        // "priceing" e erro de grafia proposital: nao pode ser lido como "sem pricing".
         var document = Document("""
             "digitado-errado": {
               "provider": "openai", "model": "gpt-5.4",
@@ -165,10 +148,6 @@ public sealed class ModelRegistryTests
         Assert.Contains("priceing", error.Message, StringComparison.Ordinal);
     }
 
-    // ---------------------------------------------------------------------------------
-    // api: por alias
-    // ---------------------------------------------------------------------------------
-
     [Fact]
     public void AnUnknownApiValueIsRefusedListingTheValidOnes()
     {
@@ -219,10 +198,6 @@ public sealed class ModelRegistryTests
         Assert.Contains("'1.0'", error.Message, StringComparison.Ordinal);
     }
 
-    // ---------------------------------------------------------------------------------
-    // Resolucao e memoizacao
-    // ---------------------------------------------------------------------------------
-
     [Fact]
     public void ResolvingAnUnknownAliasFailsExplicitlyAndListsWhatExists()
     {
@@ -281,7 +256,7 @@ public sealed class ModelRegistryTests
         Assert.True(client.IsDisposed);
         Assert.Throws<ObjectDisposedException>(() => registry.GetChatClient("text-default"));
 
-        registry.Dispose(); // idempotente
+        registry.Dispose();
     }
 
     [Fact]
@@ -312,16 +287,12 @@ public sealed class ModelRegistryTests
         Assert.IsAssignableFrom<System.Text.Json.JsonException>(error.InnerException);
     }
 
-    // ---------------------------------------------------------------------------------
-    // Tarifa: a conta
-    // ---------------------------------------------------------------------------------
-
     [Fact]
     public void CostIsTheTariffTimesTheTokensDividedByAMillion()
     {
         var pricing = new ModelPricing(InputPerMTok: 1.25m, OutputPerMTok: 10m, CachedInputPerMTok: null);
 
-        // 1.000.000 de entrada = 1.25 USD; 100.000 de saida = 1.00 USD
+        // 1M de entrada * 1.25 + 100k de saida * 10.0 = 1.25 + 1.00 USD.
         Assert.Equal(2.25m, pricing.ComputeCostUsd(inputTokens: 1_000_000, outputTokens: 100_000));
         Assert.Equal(0m, pricing.ComputeCostUsd(0, 0));
     }
@@ -332,13 +303,12 @@ public sealed class ModelRegistryTests
         var comCache = new ModelPricing(1.25m, 10m, CachedInputPerMTok: 0.125m);
         var semCache = comCache with { CachedInputPerMTok = null };
 
-        // 1M de entrada, dos quais 800k vieram do cache:
-        //   com tarifa de cache: 200k * 1.25 + 800k * 0.125 = 0.25 + 0.10 = 0.35
+        // 800k dos 1M vieram do cache: 200k * 1.25 + 800k * 0.125 = 0.25 + 0.10 = 0.35.
         Assert.Equal(
             0.35m,
             comCache.ComputeCostUsd(inputTokens: 1_000_000, outputTokens: 0, cachedInputTokens: 800_000));
 
-        // sem tarifa de cache, a entrada inteira e cobrada como entrada — nunca de graca.
+        // Sem tarifa de cache, a entrada inteira e cobrada cheia.
         Assert.Equal(
             1.25m,
             semCache.ComputeCostUsd(inputTokens: 1_000_000, outputTokens: 0, cachedInputTokens: 800_000));
@@ -355,36 +325,28 @@ public sealed class ModelRegistryTests
             () => pricing.ComputeCostUsd(inputTokens: -1, outputTokens: 0));
     }
 
-    // ---------------------------------------------------------------------------------
-    // Limites por alias — o limite REAL, nao o folclore
-    // ---------------------------------------------------------------------------------
-
     [Fact]
     public void TheChatToolLimitFiresOnlyWithAllFourFactsTogether()
     {
         using var registry = ModelRegistry.FromJson(ExampleDocument(), new FakeProviderClientFactory());
         var alias = registry.Resolve("text-heavy-chat");
 
-        // os quatro fatos juntos: Chat + tools + effort enviado + effort != none
+        // Os quatro fatos juntos: Chat + tools + effort enviado + effort != none.
         var violado = alias.CheckRequest(
             AgentApi.Chat, hasTools: true, reasoningEffort: ReasoningEffort.High, temperatureIsSet: false);
 
         Assert.False(violado.IsSatisfied);
         Assert.Contains("reasoning effort", Assert.Single(violado.Violations), StringComparison.Ordinal);
 
-        // mesmo pedido, na Responses API: sem violacao
         Assert.True(alias.CheckRequest(
             AgentApi.Responses, hasTools: true, ReasoningEffort.High, temperatureIsSet: false).IsSatisfied);
 
-        // sem tools: sem violacao
         Assert.True(alias.CheckRequest(
             AgentApi.Chat, hasTools: false, ReasoningEffort.High, temperatureIsSet: false).IsSatisfied);
 
-        // effort 'none': e exatamente o que a documentacao permite
         Assert.True(alias.CheckRequest(
             AgentApi.Chat, hasTools: true, ReasoningEffort.None, temperatureIsSet: false).IsSatisfied);
 
-        // effort nao enviado: o default do provider vale, e o kit NAO afirma saber qual e
         Assert.True(alias.CheckRequest(
             AgentApi.Chat, hasTools: true, reasoningEffort: null, temperatureIsSet: false).IsSatisfied);
     }
@@ -404,10 +366,6 @@ public sealed class ModelRegistryTests
     [Fact]
     public void TheTemperatureLimitIsPerAliasAndOnlyBitesWhenTemperatureIsSent()
     {
-        // A afirmacao "temperatura e obsoleta para modelo de raciocinio" NAO foi confirmada
-        // em fonte primaria (issue #93). Por isso ela e uma flag por alias com procedencia
-        // escrita — nunca uma regra universal no codigo. Aqui a procedencia diz, por extenso,
-        // que o exemplo e sintetico.
         var document = Document("""
             "sem-temperatura": {
               "provider": "openai", "model": "modelo-hipotetico",

package/framework/templates/dotnet/ai-kit/tests/Morph.AiKit.Tests/SkeletonSmokeTests.cs

@@ -4,19 +4,7 @@ using Xunit;
 
 namespace Morph.AiKit.Tests;
 
-/// <summary>
-/// Canario do esqueleto: prova que o harness de teste EXECUTA de verdade e que o
-/// assembly do kit foi compilado no TFM do pin. Nao testa comportamento — os testes
-/// de AgentFactory, ModelRegistry, middlewares, ConversationStore e DeltaIngest sao
-/// das tasks T7-T14.
-///
-/// Por que ele existe em vez de um projeto de teste literalmente vazio: um projeto
-/// sem nenhum teste NAO sai 0. Microsoft.Testing.Platform devolve exit code 8
-/// ("zero tests ran") e o VSTest antigo devolvia 1 com "no test is available".
-/// Um esqueleto que so "compila" e um CI verde que nunca executou nada — e um verde
-/// desses e indistinguivel de um verde real, que e exatamente o silencio que esta
-/// feature existe para quebrar.
-/// </summary>
+// Um projeto sem teste nenhum nao sai 0 (o Microsoft.Testing.Platform sai 8): este caso prova que o harness executa.
 public sealed class SkeletonSmokeTests
 {
     [Fact]

package/framework/templates/dotnet/ai-kit/tests/Morph.AiKit.Tests/TelemetryTests.cs

@@ -9,14 +9,7 @@ using Xunit;
 
 namespace Morph.AiKit.Tests;
 
-/// <summary>
-/// Canarios da telemetria (T12). O arquivo inteiro mede <b>spans</b>, com
-/// <see cref="ActivityListener"/> — nao propriedades de configuracao. Um guard contra span
-/// duplicado so pode ser provado contando spans.
-/// </summary>
-/// <remarks>
-/// Cada caso usa um <c>SourceName</c> unico, para que um teste nunca conte o span do outro.
-/// </remarks>
+// Cada caso usa um SourceName proprio, para nunca contar o span de outro teste.
 public sealed class TelemetryTests
 {
     private const string Document = """
@@ -42,15 +35,6 @@ public sealed class TelemetryTests
             Options = options ?? new AgentSpecOptions(),
         };
 
-    // ---------------------------------------------------------------------------------
-    // O caso normal: um span de agente, um span de chat
-    // ---------------------------------------------------------------------------------
-
-    /// <summary>
-    /// A forma que as GenAI conventions descrevem: <c>invoke_agent</c> como raiz e
-    /// <c>chat</c> como filho. Este teste tambem e a prova de que a escolha da T10 de montar
-    /// o proprio pipeline nao custou o span de chat — ele so existe porque a T12 o coloca.
-    /// </summary>
     [Fact]
     public async Task AnInstrumentedAgentEmitsOneInvokeAgentSpanAndOneChatSpan()
     {
@@ -66,14 +50,6 @@ public sealed class TelemetryTests
         Assert.NotNull(Single(spans, "chat").Parent);
     }
 
-    // ---------------------------------------------------------------------------------
-    // O guard — as duas metades da mesma afirmacao
-    // ---------------------------------------------------------------------------------
-
-    /// <summary>
-    /// <b>O guard, no default.</b> Um pipeline que ja carrega instrumentacao (o caso real:
-    /// um <c>IChatClient</c> que veio do DI ja instrumentado) continua com UM span de chat.
-    /// </summary>
     [Fact]
     public async Task TheDefaultDoesNotAddASecondChatSpanOverAnAlreadyInstrumentedPipeline()
     {
@@ -87,11 +63,6 @@ public sealed class TelemetryTests
         Assert.Equal(1, CountOf(spans, "chat"));
     }
 
-    /// <summary>
-    /// <b>O guard, desligado.</b> Forcar <c>InstrumentChatClient = true</c> sobre o MESMO
-    /// pipeline ja instrumentado produz o par duplicado. E o canario que documenta o risco:
-    /// sem o ramo automatico, este e o resultado.
-    /// </summary>
     [Fact]
     public async Task ForcingInstrumentChatClientProducesTheDuplicateChatSpan()
     {
@@ -105,10 +76,6 @@ public sealed class TelemetryTests
         Assert.Equal(2, CountOf(spans, "chat"));
     }
 
-    /// <summary>
-    /// O mesmo guard no nivel do agente: aplicar a telemetria duas vezes no
-    /// <see cref="AIAgentBuilder"/> nao produz dois <c>invoke_agent</c>.
-    /// </summary>
     [Fact]
     public async Task TheAgentGuardKeepsASingleInvokeAgentSpanWhenTelemetryIsAppliedTwice()
     {
@@ -124,12 +91,6 @@ public sealed class TelemetryTests
         Assert.Equal(1, CountOf(spans, "invoke_agent"));
     }
 
-    /// <summary>
-    /// <c>false</c> nao e o mesmo que o default. Com a <c>AgentFactory</c> — que monta o
-    /// proprio pipeline — desligar a instrumentacao de chat deixa a invocacao com o span do
-    /// agente e <b>nenhum</b> span de chat. O teste existe para que essa consequencia seja
-    /// um fato escrito, e nao uma surpresa em producao.
-    /// </summary>
     [Fact]
     public async Task TurningTheChatInstrumentationOffLeavesTheAgentSpanAndNoChatSpan()
     {
@@ -143,14 +104,6 @@ public sealed class TelemetryTests
         Assert.Equal(0, CountOf(spans, "chat"));
     }
 
-    // ---------------------------------------------------------------------------------
-    // Dado sensivel
-    // ---------------------------------------------------------------------------------
-
-    /// <summary>
-    /// Conteudo sensivel nasce desligado nos dois niveis — o standard exige <c>false</c>
-    /// fora de dev/staging, porque prompt e resposta sao PII do cliente.
-    /// </summary>
     [Fact]
     public void SensitiveDataIsOffByDefaultAtBothLevels()
     {
@@ -163,11 +116,6 @@ public sealed class TelemetryTests
             agent.GetService(typeof(OpenTelemetryChatClient))).EnableSensitiveData);
     }
 
-    /// <summary>
-    /// A spec pode ligar o dado sensivel para UM agente sem mexer na configuracao global —
-    /// e o destino que a T7 declarou para <c>EnableSensitiveTelemetry</c>. Nulo na spec
-    /// ("nao opinei") preserva o valor global; preenchido, vence.
-    /// </summary>
     [Fact]
     public void TheSpecCanOverrideSensitiveDataForASingleAgent()
     {
@@ -183,25 +131,12 @@ public sealed class TelemetryTests
             loud.GetService(typeof(OpenTelemetryChatClient))).EnableSensitiveData);
     }
 
-    // ---------------------------------------------------------------------------------
-    // Configuracao invalida
-    // ---------------------------------------------------------------------------------
-
-    /// <summary>
-    /// Source em branco e recusado na construcao: um <c>ActivitySource</c> sem nome nao pode
-    /// ser registrado em <c>AddSource(...)</c>, e a telemetria sumiria em silencio.
-    /// </summary>
     [Fact]
     public void ABlankSourceNameIsRefusedAtConstruction()
         => Assert.Throws<ArgumentException>(() => new MorphTelemetryOptions { SourceName = "   " });
 
-    // ---------------------------------------------------------------------------------
-    // Apoio dos testes
-    // ---------------------------------------------------------------------------------
-
     private static string NewSourceName() => "morph.aikit.test." + Guid.CreateVersion7().ToString("N");
 
-    /// <summary>Um pipeline que ja chega instrumentado — o caso real do cliente vindo do DI.</summary>
     private static IChatClient AlreadyInstrumented(string sourceName)
         => new FakeChatClient()
             .EnqueueText("pronto")
@@ -219,11 +154,7 @@ public sealed class TelemetryTests
         return factory.Create(spec ?? Spec());
     }
 
-    /// <summary>
-    /// Coleta os spans de UMA invocacao. O listener e registrado antes de o agente existir e
-    /// removido no fim — telemetria e estado global do processo, e um listener vazado
-    /// contaminaria os outros casos.
-    /// </summary>
+    // Telemetria e estado global do processo: o listener entra antes do agente e sai no fim.
     private static async Task<IReadOnlyList<Activity>> CollectAsync(string sourceName, Func<AIAgent> build)
     {
         var spans = new List<Activity>();