@polymorphism-tech/morph-spec 8.37.1 → 8.40.0

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

@@ -4,16 +4,6 @@ using Xunit;
 
 namespace Morph.AiKit.Tests.Evals;
 
-/// <summary>
-/// Miss de cache sem credencial REPROVA, nomeado. Jamais <c>skip</c>.
-/// </summary>
-/// <remarks>
-/// A tentacao do <c>skip</c> e real ("nao da para avaliar, entao nao avalio") e recria
-/// exatamente o buraco que este harness fechou duas vezes: gate verde sobre verificacao que
-/// nao aconteceu. E ha razao positiva: em CI, com cache commitada, um miss significa UMA
-/// coisa so — o prompt ou o modelo mudou e ninguem regravou. O miss e o veredito, nao o
-/// obstaculo.
-/// </remarks>
 public sealed class EvalCacheMissTests
 {
     private static async Task<EvalWorkspace> SeededAsync(params string[] scenarioNames)
@@ -41,21 +31,16 @@ public sealed class EvalCacheMissTests
     {
         using var ws = await SeededAsync("a-primeiro");
 
-        // Cenario NOVO, que a cache nunca viu — a situacao literal de "mudou o prompt e
-        // ninguem regravou". O prefixo "b-" e deliberado: os cenarios rodam em ordem ordinal
-        // de caminho, e o teste abaixo depende de o cenario BOM ter rodado antes do miss.
+        // Prefixo "b-": os cenarios rodam em ordem ordinal de caminho, e o miss tem de vir depois do cenario bom.
         ws.WriteScenario("triagem", "acme", "b-novo", "pergunta inedita", """[ { "kind": "contains", "value": "resposta" } ]""");
 
         await using var offline = EvalFixture.Create(ws.Options(EvalMode.Offline));
         var error = await Assert.ThrowsAsync<EvalCacheMissException>(
             () => offline.RunAllAsync(TestContext.Current.CancellationToken));
 
-        // O cenario que faltou, e nao "algum cenario".
         Assert.Equal("triagem.acme.b-novo", error.ScenarioName);
         Assert.Contains("eval-cache-miss", error.Message, StringComparison.Ordinal);
 
-        // P3: a mensagem carrega o comando de regravacao e o arquivo de credencial esperado.
-        // Um "cache miss" seco mandaria o leitor procurar os dois.
         Assert.Contains(EvalCacheMissException.RefreshCommand, error.Message, StringComparison.Ordinal);
         Assert.Contains(".env.evals", error.Message, StringComparison.Ordinal);
         Assert.Contains(ws.CacheRoot, error.Message, StringComparison.Ordinal);
@@ -67,33 +52,22 @@ public sealed class EvalCacheMissTests
         using var ws = await SeededAsync("a-primeiro");
         ws.WriteScenario("triagem", "acme", "b-novo", "pergunta inedita", """[ { "kind": "contains", "value": "resposta" } ]""");
 
-        // O sumario da SEMEADURA, byte a byte. Ele e a linha de base: o que este teste cobra
-        // e que o run que morreu por miss nao escreva NADA por cima dele.
         var summaryBeforeTheMiss = File.ReadAllBytes(ws.SummaryPath);
 
         await using var offline = EvalFixture.Create(ws.Options(EvalMode.Offline));
         await Assert.ThrowsAsync<EvalCacheMissException>(
             () => offline.RunAllAsync(TestContext.Current.CancellationToken));
 
-        // Nao ha caminho pelo qual o cenario ausente apareca como avaliado: a contagem so tem
-        // o que de fato resolveu, e o sumario NAO foi reescrito. Um sumario dizendo
-        // "1 cenario, 1 passou, cache warm" aqui seria exatamente a mentira que a excecao
-        // existe para impedir — e o CLI, que le o ARQUIVO, acreditaria nela.
         Assert.Single(offline.Results);
         Assert.Equal("triagem.acme.a-primeiro", offline.Results[0].Scenario);
         Assert.Equal(summaryBeforeTheMiss, File.ReadAllBytes(ws.SummaryPath));
 
-        // E o contador do cliente interno registra a tentativa: a arvore de cache nao
-        // respondeu, e isso e um FATO medido, nao uma inferencia.
         Assert.Equal(1, offline.Offline!.Invocations);
     }
 
     [Fact]
     public async Task AFailedAssertionMakesTheScenarioREDWithoutAnyCacheMiss()
     {
-        // A outra metade do contrato: reprovar por conteudo e diferente de reprovar por
-        // cache. Uma so mensagem para os dois casos mandaria o leitor procurar no lugar
-        // errado metade das vezes.
         using var ws = new EvalWorkspace();
         ws.WriteScenario("triagem", "acme", "saudacao", "bom dia", """[ { "kind": "contains", "value": "jamais aparece" } ]""");
 
@@ -130,9 +104,6 @@ public sealed class EvalCacheMissTests
         await using var offline = EvalFixture.Create(ws.Options(EvalMode.Offline));
         var summary = await offline.RunAllAsync(TestContext.Current.CancellationToken);
 
-        // Um cenario que nao cobra nada nao pode contar como cobertura. Deixa-lo verde e o
-        // mesmo defeito de um teste vazio — pior aqui, porque ele apareceria na contagem
-        // como se tivesse verificado algo.
         Assert.Equal(1, summary.Failed);
         Assert.Equal(0, summary.Passed);
     }

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

@@ -4,14 +4,6 @@ using Xunit;
 
 namespace Morph.AiKit.Tests.Evals;
 
-/// <summary>
-/// O carimbo: por que a arvore de cache precisa dizer sob quais versoes foi gravada.
-/// </summary>
-/// <remarks>
-/// A funcao de hash que produz a chave nao promete estabilidade entre versoes da biblioteca.
-/// Sem carimbo, subir o pin chega como N misses inexplicaveis num dia em que ninguem mexeu
-/// em nada; com carimbo, chega como uma frase que nomeia as DUAS versoes.
-/// </remarks>
 public sealed class EvalCacheStampTests
 {
     private static readonly FakeClock Today = new(new DateTimeOffset(2026, 9, 8, 12, 0, 0, TimeSpan.Zero));
@@ -51,8 +43,6 @@ public sealed class EvalCacheStampTests
         using var ws = new EvalWorkspace();
         Stamp(ws.CacheRoot);
 
-        // Ler de VOLTA do disco, e nao comparar o objeto com ele mesmo: o carimbo e um
-        // artefato, e um teste que so olhasse a instancia em memoria seria um espelho.
         var read = EvalCacheStamp.Read(ws.CacheRoot);
 
         Assert.NotNull(read);
@@ -72,7 +62,6 @@ public sealed class EvalCacheStampTests
 
         Assert.Equal(EvalStampVerdict.PinMismatch, check.Verdict);
         Assert.Contains("eval-cache-stale-pin", check.Reason, StringComparison.Ordinal);
-        // Dizer so uma das versoes obriga o leitor a ir buscar a outra.
         Assert.Contains("gravada 10.9.0", check.Reason, StringComparison.Ordinal);
         Assert.Contains("pin 10.11.0", check.Reason, StringComparison.Ordinal);
     }
@@ -89,9 +78,6 @@ public sealed class EvalCacheStampTests
         Assert.Equal(EvalStampVerdict.Missing, missing.Verdict);
         Assert.Contains("eval-cache-unstamped", missing.Reason, StringComparison.Ordinal);
 
-        // As duas situacoes pedem acoes diferentes: "regrave" versus "regrave PORQUE o
-        // pacote mudou". Um veredito unico devolveria ao leitor a pergunta que o carimbo
-        // existia para responder.
         Assert.NotEqual(missing.Verdict, divergent.Verdict);
         Assert.DoesNotContain("eval-cache-stale-pin", missing.Reason, StringComparison.Ordinal);
     }
@@ -116,8 +102,6 @@ public sealed class EvalCacheStampTests
 
         var check = EvalCacheStamp.Check(ws.CacheRoot, Pin(pinnedAt: "2027-01-01"), Today);
 
-        // O carimbo CASA com o pin — e mesmo assim reprova, porque a regua esta adulterada.
-        // Uma data no futuro renovaria sozinha qualquer janela medida contra ela.
         Assert.Equal(EvalStampVerdict.FuturePin, check.Verdict);
         Assert.Contains("eval-pin-future", check.Reason, StringComparison.Ordinal);
     }
@@ -135,8 +119,6 @@ public sealed class EvalCacheStampTests
             ["Microsoft.Extensions.AI.Evaluation"] = "10.11.0",
         });
 
-        // A recusa acontece na CONSTRUCAO — antes de qualquer cenario. Nao faz sentido pagar
-        // o relogio de uma suite para descobrir no fim que a cache nao servia.
         var error = Assert.Throws<InvalidOperationException>(
             () => EvalFixture.Create(ws.Options(EvalMode.Offline, pin), timeProvider: Today));
 
@@ -148,7 +130,7 @@ public sealed class EvalCacheStampTests
     {
         using var ws = new EvalWorkspace();
 
-        // Cobrar carimbo em modo de escrita tornaria impossivel criar a PRIMEIRA cache.
+        // Cobrar carimbo na escrita impediria criar a primeira cache.
         var fixture = EvalFixture.Create(ws.Options(EvalMode.Refresh), new FakeChatClient());
 
         Assert.True(fixture.StampCheck.IsOk);

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

@@ -4,37 +4,8 @@ using Xunit;
 
 namespace Morph.AiKit.Tests.Evals;
 
-/// <summary>
-/// A camada que NOMEIA a falha, testada pelo conteudo do que ela produz.
-/// </summary>
-/// <remarks>
-/// <para>
-/// Este arquivo fecha a setima ocorrencia da mesma classe nesta epic, e a mais ironica: o
-/// mecanismo de <c>Failures</c> nasceu para consertar um eval que mordia ANONIMAMENTE, e
-/// nasceu ele proprio sem par. As tres partes podiam ser desligadas sem matar um teste —
-/// <c>CollectFailures</c> devolvendo lista vazia, o ramo da vacuidade nunca disparando, e o
-/// nome da assercao trocado por um literal — e a suite seguia "165 total, 0 falhou".
-/// </para>
-/// <para>
-/// A causa foi uma unica testemunha automatizada: <c>Assert.Empty(summary.Failures)</c> num
-/// run VERDE. Essa asserção é verdadeira exatamente quando o mecanismo esta morto — ela nao
-/// distingue "nao houve falha" de "falha nenhuma e registrada". Uma lista vazia prova a
-/// ausencia de conteudo, jamais a presenca de capacidade.
-/// </para>
-/// <para>
-/// Por isso aqui nao se assere sobre <c>Failures</c> VAZIA: assere-se sobre o CONTEUDO de
-/// <c>Failures</c> num run que falha de verdade — o par, a mesma forma que fechou o motor de
-/// assercao um andar abaixo.
-/// </para>
-/// </remarks>
 public sealed class EvalFailuresTests
 {
-    /// <summary>Grava a cache do cenario com o texto dado, e devolve o fixture offline pronto.</summary>
-    /// <remarks>
-    /// A cache e semeada pelo <see cref="FakeChatClient"/>, NUNCA por chamada paga: o caminho
-    /// de escrita exercitado e o da biblioteca, e a leitura offline le exatamente o que aquela
-    /// escrita produziu.
-    /// </remarks>
     private static async Task<EvalRunSummary> RunOfflineAsync(EvalWorkspace ws, params string[] responses)
     {
         using var fake = new FakeChatClient
@@ -60,9 +31,6 @@ public sealed class EvalFailuresTests
     [Fact]
     public async Task AFailedScenarioIsRegisteredWithScenarioAssertionKindAndReason()
     {
-        // O caso central: uma assercao reprova, e a falha chega ao sumario COM NOME. Sem isso
-        // o veredito viaja como contador — "Expected 0, Actual 1" — e quem le tem de abrir o
-        // dataset para descobrir o que quebrou.
         using var ws = new EvalWorkspace();
         ws.WriteScenario(
             "triagem", "acme", "saudacao", "bom dia",
@@ -79,7 +47,6 @@ public sealed class EvalFailuresTests
         Assert.Contains("FALHOU", failure.Reason, StringComparison.Ordinal);
         Assert.Contains("bom dia", failure.Reason, StringComparison.Ordinal);
 
-        // A linha legivel, no formato que a rubrica eval-de-modelo.md cita como exemplo.
         Assert.Contains(
             "eval-fail: triagem.acme.saudacao",
             summary.DescribeFailures(),
@@ -89,9 +56,6 @@ public sealed class EvalFailuresTests
     [Fact]
     public async Task TwoReprovedAssertionsInOneScenarioProduceTWOLinesEachNamingItsOwnRule()
     {
-        // `Failures` e LISTA, e nao contador, porque as duas linhas dizem coisas diferentes:
-        // uma resposta pode quebrar em mais de uma regua, e isso separa "o prompt mudou" de
-        // "o modelo parou de responder". Se o tipo fosse um booleano, esta distincao morreria.
         using var ws = new EvalWorkspace();
         ws.WriteScenario(
             "triagem", "acme", "saudacao", "bom dia",
@@ -110,18 +74,12 @@ public sealed class EvalFailuresTests
         Assert.Contains(summary.Failures, f => f.Assertion == "cumprimenta" && f.Kind == "Contains");
         Assert.Contains(summary.Failures, f => f.Assertion == "sem-dado-pessoal" && f.Kind == "NoPii");
 
-        // Toda linha nomeia o cenario: uma falha sem cenario obriga a mesma busca que ela
-        // deveria dispensar.
         Assert.All(summary.Failures, f => Assert.Equal("triagem.acme.saudacao", f.Scenario));
     }
 
     [Fact]
     public async Task AScenarioWithNOAssertionsIsCountedANDNamed()
     {
-        // O cenario sem assercao nenhuma reprova por VACUIDADE: ele nao cobra nada. Nesse caso
-        // nao existe outcome reprovado para nomear, e sem um ramo proprio ele seria contado em
-        // `Failed` e invisivel em `Failures` — um numero sem frase, que e metade exata do
-        // defeito que este tipo existe para fechar.
         using var ws = new EvalWorkspace();
         ws.WriteScenario("triagem", "acme", "vazio", "bom dia", "[]");
 
@@ -133,16 +91,12 @@ public sealed class EvalFailuresTests
         Assert.Equal("triagem.acme.vazio", failure.Scenario);
         Assert.Contains("sem assercao deterministica", failure.Reason, StringComparison.Ordinal);
 
-        // Contado E nomeado: o numero e a frase tem de concordar sempre.
         Assert.Equal(summary.Failed, summary.Failures.Count);
     }
 
     [Fact]
     public async Task AGreenRunRegistersNoFailureAndSaysSoInWords()
     {
-        // A reciproca vive AO LADO dos casos que falham de verdade, nunca sozinha: uma suite
-        // cuja unica testemunha e `Assert.Empty` num run verde e verdadeira precisamente
-        // quando o mecanismo esta morto.
         using var ws = new EvalWorkspace();
         ws.WriteScenario(
             "triagem", "acme", "saudacao", "bom dia",
@@ -158,8 +112,6 @@ public sealed class EvalFailuresTests
     [Fact]
     public async Task TheNamedFailuresSurviveTheROUNDTRIPThroughTheSummaryFileTheCLIReads()
     {
-        // O sumario e um ARQUIVO: quem consome as falhas e o CLI, do outro lado do disco. Uma
-        // lista que existisse so em memoria seria invisivel exatamente onde precisa ser lida.
         using var ws = new EvalWorkspace();
         ws.WriteScenario(
             "triagem", "acme", "saudacao", "bom dia",

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

@@ -5,17 +5,6 @@ using Xunit;
 
 namespace Morph.AiKit.Tests.Evals;
 
-/// <summary>
-/// A prova que decide a feature: <b>com a cache quente, o eval roda sem credencial e sem
-/// rede</b>.
-/// </summary>
-/// <remarks>
-/// Essa frase foi derivada de leitura de fonte antes de existir codigo. Aqui ela vira
-/// medicao: uma arvore de cache e gravada por um fake, o processo troca o cliente interno
-/// por um que RECUSA qualquer chamada, e a suite termina verde. Se o cliente interno fosse
-/// tocado uma unica vez, o contador dele deixaria de ser zero — e nenhuma das assercoes
-/// abaixo passaria.
-/// </remarks>
 public sealed class EvalFixtureTests
 {
     [Fact]
@@ -25,18 +14,12 @@ public sealed class EvalFixtureTests
         ws.WriteScenario("triagem", "acme", "saudacao", "bom dia", """[ { "kind": "contains", "value": "Ola" } ]""");
         ws.WriteScenario("triagem", "acme", "handoff", "quero falar com humano", """[ { "kind": "contains", "value": "humano" } ]""");
 
-        // 1) GRAVA a arvore contra o fake — nunca contra uma API paga. O caminho de escrita
-        //    exercitado e o da biblioteca, e a chave e computada pela mesma funcao que a
-        //    leitura vai usar.
         using var fake = new FakeChatClient
         {
             MetadataProviderName = "morph-fake",
             MetadataDefaultModelId = "modelo-exemplar",
         };
-        // A ORDEM importa e e a do dataset (ordinal por caminho): "handoff" vem antes de
-        // "saudacao". Enfileirar na ordem alfabetica errada foi como esta suite descobriu
-        // que LoadAll ordena — e a ordem estavel e o que faz duas maquinas gravarem a mesma
-        // arvore de cache.
+        // A ordem e a do dataset (ordinal por caminho): "handoff" antes de "saudacao".
         fake.EnqueueText("Encaminhando para um humano agora.").EnqueueText("Ola! Bom dia.");
 
         await using (var refresh = EvalFixture.Create(ws.Options(EvalMode.Refresh), fake))
@@ -50,8 +33,6 @@ public sealed class EvalFixtureTests
         Assert.Equal(2, fake.CallCount);
         Assert.True(File.Exists(EvalCacheStamp.PathFor(ws.CacheRoot)), "o refresh tem de deixar o carimbo na raiz da cache");
 
-        // 2) LE offline. O cliente interno agora RECUSA: se a cache nao respondesse, este
-        //    bloco lancaria EvalCacheMissException em vez de terminar verde.
         await using var offline = EvalFixture.Create(ws.Options(EvalMode.Offline));
         var summary = await offline.RunAllAsync(TestContext.Current.CancellationToken);
 
@@ -63,8 +44,6 @@ public sealed class EvalFixtureTests
         Assert.Equal("ok", summary.Stamp);
         Assert.Equal(0, offline.Offline!.Invocations);
 
-        // 3) O fake nao foi tocado de novo: a leitura offline nao passou por ele nem por
-        //    acidente (ele esta noutro processo logico, mas a contagem prova a intencao).
         Assert.Equal(2, fake.CallCount);
     }
 
@@ -84,8 +63,6 @@ public sealed class EvalFixtureTests
         await using var offline = EvalFixture.Create(ws.Options(EvalMode.Offline));
         await offline.RunAllAsync(TestContext.Current.CancellationToken);
 
-        // Ler de volta o ARQUIVO, e nao o objeto em memoria: o contrato com o CLI e o
-        // arquivo, e um teste que so olhasse o objeto seria um espelho do gerador.
         var reread = EvalRunSummary.ReadFrom(ws.SummaryPath);
         Assert.NotNull(reread);
         Assert.Equal(1, reread!.Scenarios);
@@ -103,17 +80,12 @@ public sealed class EvalFixtureTests
         var error = Assert.Throws<ArgumentOutOfRangeException>(() => EvalFixture.Create(options));
 
         Assert.Contains("eval-cache-ttl-too-short", error.Message, StringComparison.Ordinal);
-        // A mensagem tem de dizer POR QUE, e nao so que o numero e pequeno: a expiracao apaga
-        // o diretorio da chave durante a LEITURA.
         Assert.Contains("DESTRUTIVA NA LEITURA", error.Message, StringComparison.Ordinal);
     }
 
     [Fact]
     public void TheLibraryDefaultTimeToLiveWouldExpireACommittedCacheWithinWeeks()
     {
-        // MEDIDO, nao lembrado: e este fato que torna o TTL explicito obrigatorio. Se um dia
-        // a biblioteca subir o default para alem do piso do kit, este teste morre e a
-        // exigencia pode ser rediscutida com um numero na mao.
         Assert.True(
             Defaults.DefaultTimeToLiveForCacheEntries < EvalCacheOptions.MinimumTimeToLive,
             $"o default da biblioteca e {Defaults.DefaultTimeToLiveForCacheEntries.TotalDays:0} dias, "

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

@@ -6,20 +6,9 @@ using Xunit;
 
 namespace Morph.AiKit.Tests.Evals;
 
-/// <summary>
-/// Regravacao, e o invariante barato que protege a premissa de commitabilidade:
-/// <b>uma leitura nao escreve um byte na arvore de cache</b>.
-/// </summary>
 public sealed class EvalRefreshTests
 {
-    /// <summary>
-    /// Impressao digital da arvore: caminho relativo mais SHA-256 de cada arquivo.
-    /// </summary>
-    /// <remarks>
-    /// Comparar CONTEUDO, e nao data de modificacao: o que importa e se o proximo
-    /// <c>git status</c> mostra diff, e o git tambem olha conteudo. Um teste sobre mtime
-    /// falharia por gravacao identica e passaria por gravacao diferente no mesmo tick.
-    /// </remarks>
+    // Compara conteudo, nao mtime: e o conteudo que o git ve como diff.
     private static IReadOnlyList<string> Fingerprint(string root)
     {
         if (!Directory.Exists(root))
@@ -63,9 +52,6 @@ public sealed class EvalRefreshTests
             Assert.Equal(3, summary.Passed);
         }
 
-        // Se algum dia a biblioteca passar a tocar disco no caminho de HIT, este canario morde
-        // no dia em que isso acontecer — e nao meses depois, quando alguem estranhar um diff
-        // que aparece sozinho em toda rodada.
         Assert.Equal(before, Fingerprint(ws.CacheRoot));
     }
 
@@ -79,9 +65,6 @@ public sealed class EvalRefreshTests
         await using var refresh = EvalFixture.Create(ws.Options(EvalMode.Refresh), fake);
         await refresh.RunAllAsync(TestContext.Current.CancellationToken);
 
-        // Os resultados carregam latencia de cronometro, uso de tokens e o flag de cache hit:
-        // commita-los seria diff garantido em toda rodada, sem sinal nenhum. Por CONSTRUCAO
-        // (outra raiz), nunca por um .gitignore dentro de um diretorio commitado.
         Assert.True(Directory.Exists(ws.ResultsRoot), "o store de resultados tem de existir depois de um run");
         Assert.NotEmpty(Directory.EnumerateFiles(ws.ResultsRoot, "*", SearchOption.AllDirectories));
 
@@ -102,8 +85,6 @@ public sealed class EvalRefreshTests
         await using var refresh = EvalFixture.Create(ws.Options(EvalMode.Refresh), fake);
         var summary = await refresh.RunAllAsync(TestContext.Current.CancellationToken);
 
-        // O contador do cliente real e o que torna "a cache esta fria" uma MEDICAO: a arvore
-        // mudou e precisa ser commitada, e isso e afirmacao com consequencia.
         Assert.Equal(3, summary.Misses);
         Assert.Equal(EvalCacheState.Cold, summary.Cache);
         Assert.Equal(3, fake.CallCount);
@@ -121,9 +102,6 @@ public sealed class EvalRefreshTests
             await first.RunAllAsync(TestContext.Current.CancellationToken);
         }
 
-        // O cenario do meio some do dataset. Uma regravacao PARCIAL deixaria a entrada antiga
-        // dele viva na arvore, e a proxima leitura offline nao teria como saber que ela nao
-        // corresponde mais a nenhum cenario declarado.
         File.Delete(Path.Combine(ws.DatasetRoot, "triagem", "acme", "b-segundo.json"));
 
         using (var fake = new FakeChatClient { MetadataProviderName = "morph-fake", MetadataDefaultModelId = "modelo-exemplar" })
@@ -144,24 +122,14 @@ public sealed class EvalRefreshTests
         using var ws = new EvalWorkspace();
         using var fake = new FakeChatClient { MetadataProviderName = "morph-fake", MetadataDefaultModelId = "modelo-exemplar" };
 
-        // A lista vem do KIT, e nao do teste: e ela que da ao pacote .Quality cobertura de
-        // COMPILACAO em src/ (sem um using real la, o pin daquele pacote teria so contrato de
-        // versao, e um rename de API nao reprovaria PR nenhuma).
         var quality = EvalQualityEvaluators.Stable();
 
         await using var fixture = EvalFixture.Create(ws.Options(EvalMode.Refresh), fake, quality);
 
-        // A superficie existe e COMPILA contra as versoes do pin — que e o que um kit de
-        // template tem de garantir. O que ela NAO faz e decidir: o veredito de um cenario sai
-        // das assercoes deterministicas, e uma metrica de juizo entra no sumario como numero.
-        // Sem isso, a variancia de LLM reprovaria PRs e o no de eval nasceria destinado a ser
-        // desligado do CI.
         Assert.Equal(2, quality.Count);
         Assert.All(quality, evaluator => Assert.NotEmpty(evaluator.EvaluationMetricNames));
         Assert.Empty(fixture.Summary.Quality);
 
-        // Uma lista NOVA a cada chamada: um array compartilhado seria estado global
-        // disfarcado de conveniencia, e a ReportingConfiguration guarda o que recebe.
         Assert.NotSame(quality, EvalQualityEvaluators.Stable());
     }
 
@@ -172,11 +140,7 @@ public sealed class EvalRefreshTests
     [InlineData("Microsoft.Extensions.AI.Evaluation.Quality.ToolCallAccuracyEvaluator", true)]
     public void WhichQualityEvaluatorsAreExperimentalIsMEASURED_NotRemembered(string typeName, bool experimental)
     {
-        // O plano descrevia os QUATRO como estaveis. Dois nao sao: eles carregam
-        // [Experimental("AIEVAL001")] na 10.9.0, e usa-los sob TreatWarningsAsErrors exigiria
-        // suprimir um diagnostico — uma decisao consciente, nunca um efeito colateral. Se um
-        // dia a Microsoft estabilizar (ou marcar) algum deles, este teste morre e a escolha e
-        // rediscutida com o fato na mao, em vez de envelhecer em silencio num comentario.
+        // Dois dos quatro avaliadores sao [Experimental("AIEVAL001")] na 10.9.0 e exigiriam supressao sob TreatWarningsAsErrors.
         var type = typeof(RelevanceEvaluator).Assembly.GetType(typeName);
         Assert.NotNull(type);
 

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

@@ -2,15 +2,6 @@ using Morph.AiKit.Evals;
 
 namespace Morph.AiKit.Tests.Evals;
 
-/// <summary>
-/// Um diretorio temporario com dataset, cache e resultados — a montagem que todo teste de
-/// eval precisa, escrita uma vez.
-/// </summary>
-/// <remarks>
-/// Ele existe para que os testes falem de COMPORTAMENTO e nao de caminhos. E cada instancia
-/// tem raiz propria (GUID v7): dois testes rodando lado a lado nao podem disputar a mesma
-/// arvore de cache, senao a falha aparece como flake e nao como defeito.
-/// </remarks>
 internal sealed class EvalWorkspace : IDisposable
 {
     public EvalWorkspace()
@@ -30,7 +21,6 @@ internal sealed class EvalWorkspace : IDisposable
 
     public string SummaryPath => Path.Combine(Root, "summary.json");
 
-    /// <summary>O pin exemplar: a data e as versoes sob as quais a arvore diz ter sido gravada.</summary>
     public static EvalPin Pin { get; } = new(
         "2026-09-08",
         new Dictionary<string, string>(StringComparer.Ordinal)
@@ -39,7 +29,6 @@ internal sealed class EvalWorkspace : IDisposable
             ["Microsoft.Extensions.AI.Evaluation"] = "10.9.0",
         });
 
-    /// <summary>Escreve um cenario no dataset e devolve o caminho do arquivo.</summary>
     public string WriteScenario(string agent, string tenant, string name, string userText, string assertions)
     {
         var dir = Path.Combine(DatasetRoot, agent, tenant);
@@ -54,7 +43,6 @@ internal sealed class EvalWorkspace : IDisposable
         return file;
     }
 
-    /// <summary>Opcoes prontas para um modo, com TTL longo e o pin exemplar.</summary>
     public EvalCacheOptions Options(EvalMode mode, EvalPin? pin = null) => new()
     {
         CacheRoot = CacheRoot,
@@ -79,8 +67,7 @@ internal sealed class EvalWorkspace : IDisposable
         }
         catch (IOException)
         {
-            // Um temporario que resiste a remocao nao pode derrubar um teste que ja provou o
-            // que veio provar.
+            // Falha ao apagar o temporario nao deve derrubar um teste que ja provou o que veio provar.
         }
     }
 }

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

@@ -4,42 +4,14 @@ using Xunit;
 
 namespace Morph.AiKit.Tests.Evals;
 
-/// <summary>
-/// Os evals DO PROPRIO KIT, contra a arvore de cache COMMITADA — sem credencial, sem rede.
-/// </summary>
-/// <remarks>
-/// <para>
-/// Esta classe e o exemplar que um projeto copia. Ela roda com a categoria <c>Eval</c>, le a
-/// cache commitada em <c>eval-cache/</c> e o dataset versionado em <c>eval-datasets/</c>, e
-/// termina verde com TODAS as variaveis de credencial ausentes. Se algum cenario nao estiver
-/// na cache, ela fica VERMELHA nomeando o cenario — nunca <c>skip</c>.
-/// </para>
-/// <para>
-/// <b>A cache foi semeada por FAKE, jamais por chamada paga</b> (ver <see cref="SeedTheCommittedCacheTree"/>).
-/// O caminho de ESCRITA exercitado e o da biblioteca e a chave e computada pela mesma funcao
-/// que a leitura usa — entao a leitura offline le exatamente o que aquela escrita produziu.
-/// A lacuna residual, declarada em voz alta: isto NAO prova que uma cache gravada contra um
-/// provider real e legivel offline. O que a mitiga e um fato medido — a chave depende de
-/// <c>ProviderName</c> e <c>DefaultModelId</c>, e nao do endpoint, e os dois estao sob nosso
-/// controle no cliente offline.
-/// </para>
-/// </remarks>
 public sealed class KitEvals
 {
-    /// <summary>Identidade sob a qual a arvore commitada foi gravada. Ela ENTRA NA CHAVE.</summary>
+    // Provider e Model entram na chave de cache: trocar qualquer um invalida a arvore commitada.
     private const string Provider = "morph-fake";
 
-    /// <summary>Idem. Trocar qualquer um dos dois invalida a arvore inteira.</summary>
     private const string Model = "morph-kit-exemplar";
 
-    /// <summary>
-    /// O diretorio do PROJETO de teste, achado subindo a partir do assembly.
-    /// </summary>
-    /// <remarks>
-    /// De proposito: a alternativa seria copiar a cache para <c>bin/</c> e ler a copia — e ai
-    /// o canario de "um run offline nao suja a arvore commitada" seria verdadeiro por
-    /// vacuidade, porque a arvore commitada nem teria sido tocada.
-    /// </remarks>
+    // Le a arvore commitada no projeto, nao uma copia em bin/: senao o canario de "nao suja a arvore" seria vacuo.
     private static string ProjectDirectory
     {
         get
@@ -59,7 +31,6 @@ public sealed class KitEvals
 
     private static string DatasetRoot => Path.Combine(ProjectDirectory, "eval-datasets");
 
-    /// <summary>O pin sob o qual a arvore commitada foi gravada — o mesmo que o carimbo carrega.</summary>
     private static EvalPin Pin => new(
         "2026-09-08",
         new Dictionary<string, string>(StringComparer.Ordinal)
@@ -74,8 +45,6 @@ public sealed class KitEvals
     {
         CacheRoot = CacheRoot,
         DatasetRoot = DatasetRoot,
-        // FORA da arvore commitada: os resultados carregam latencia, uso de tokens e o flag
-        // de cache hit, e commita-los seria diff garantido em toda rodada.
         ResultsRoot = Path.Combine(Path.GetTempPath(), "morph-kit-eval-results"),
         TimeToLive = TimeSpan.FromDays(3650),
         ProviderName = Provider,
@@ -89,9 +58,6 @@ public sealed class KitEvals
     [Trait("Category", "Eval")]
     public async Task TheKitEvalsRunGreenFromTheCommittedCacheWithNoCredentialAtAll()
     {
-        // Nenhuma variavel de credencial e lida por nada nesta montagem — o cliente interno
-        // e o OfflineChatClient, que nao abre socket e nao le ambiente. Se a cache nao
-        // respondesse, este teste morreria com EvalCacheMissException nomeando o cenario.
         var summaryPath = Path.Combine(Path.GetTempPath(), $"morph-kit-eval-{Guid.CreateVersion7():N}.json");
 
         try
@@ -102,20 +68,13 @@ public sealed class KitEvals
             Assert.Equal(EvalCacheState.Warm, summary.Cache);
             Assert.Equal("ok", summary.Stamp);
 
-            // NOMEANDO. `Assert.Equal(0, summary.Failed)` produzia "Expected 0, Actual 1" —
-            // verdadeiro e inutil: nao dizia qual cenario nem qual regua. A rubrica
-            // eval-de-modelo.md exige, na banda 9-10, derrubar o cenario NOMEANDO-O.
             Assert.True(summary.Failed == 0, summary.DescribeFailures());
             Assert.Empty(summary.Failures);
             Assert.Equal(summary.Scenarios, summary.Passed);
 
-            // A prova central, e ela e um CONTADOR: o cliente interno e o unico caminho para
-            // a rede, e ele nao foi invocado nenhuma vez.
             Assert.Equal(0, summary.Misses);
             Assert.Equal(0, fixture.Offline!.Invocations);
 
-            // O dataset nao pode encolher em silencio: uma suite de eval que passa a avaliar
-            // menos cenarios e a mesma classe de defeito que uma baseline de teste caindo.
             Assert.Equal(4, summary.Scenarios);
         }
         finally
@@ -144,17 +103,10 @@ public sealed class KitEvals
     [Trait("Category", "Eval")]
     public void TheCommittedCacheCarriesNoABSOLUTEPathOfTheMachineThatWroteIt()
     {
-        // A premissa que torna a arvore commitavel: nada dentro dela aponta para o disco de
-        // quem gravou. Medido, e nao suposto — se um dia a biblioteca passar a gravar o
-        // storageRootPath dentro das entradas, este canario morde.
         var root = ProjectDirectory.Replace('\\', '/');
         var drive = Path.GetPathRoot(ProjectDirectory)?.Replace('\\', '/');
 
-        // A raiz so' identifica uma MAQUINA quando e' uma unidade nomeada ("R:/"). Em POSIX
-        // `GetPathRoot` devolve "/", que nao identifica nada — e exigir que a cache nao
-        // contenha "/" reprova qualquer arquivo com um path ou uma URL dentro. O canario
-        // passava no Windows e era ABSURDO no Linux: media "nenhuma barra em lugar nenhum".
-        // Medido no CI (job ai-kit, ubuntu): 1 falha, `Assert.DoesNotContain() Sub-string found`.
+        // Em POSIX GetPathRoot devolve "/", que nao identifica maquina; so unidade nomeada ("R:/") identifica.
         var namedDrive = drive is { Length: > 1 } && drive.Contains(':');
 
         foreach (var file in Directory.EnumerateFiles(CacheRoot, "*", SearchOption.AllDirectories))
@@ -168,21 +120,6 @@ public sealed class KitEvals
         }
     }
 
-    /// <summary>
-    /// Regrava a arvore commitada a partir do fake. NAO roda em CI.
-    /// </summary>
-    /// <remarks>
-    /// <para>
-    /// Ele existe committed, e nao como um script que alguem rodou uma vez e jogou fora,
-    /// porque a proxima pessoa que subir o pin precisa saber COMO regravar — e uma instrucao
-    /// em prosa envelhece; um teste executavel nao.
-    /// </para>
-    /// <para>
-    /// A porta e uma variavel de ambiente, e nao um <c>[Fact(Skip=...)]</c> fixo, porque um
-    /// skip fixo nunca poderia ser exercitado: seria um seeder que ninguem consegue rodar.
-    /// Rode com <c>MORPH_EVAL_SEED=1 dotnet test</c> e commite a arvore que sair.
-    /// </para>
-    /// </remarks>
     [Fact]
     public async Task SeedTheCommittedCacheTree()
     {
@@ -196,9 +133,7 @@ public sealed class KitEvals
             MetadataDefaultModelId = Model,
         };
 
-        // Uma resposta por cenario, na ORDEM do dataset (ordinal por caminho relativo):
-        // orcamento/acme/a-resposta-estruturada, triagem/acme/a-saudacao,
-        // triagem/acme/b-escalonamento, triagem/globex/a-saudacao.
+        // Uma resposta por cenario, na ordem ordinal de caminho do dataset.
         fake.EnqueueText("""{"valor":"R$ 1.200,00","prazo":"5 dias uteis"}""")
             .EnqueueText("Bom dia! Como posso ajudar voce hoje?")
             .EnqueueText("Encaminhando voce para um atendente humano agora.")