@polymorphism-tech/morph-spec 8.38.0 → 8.40.0

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.")

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

@@ -4,9 +4,6 @@ using Xunit;
 
 namespace Morph.AiKit.Tests.Evals;
 
-/// <summary>
-/// O cliente que RECUSA: a peca de que depende "o eval roda no CI sem credencial".
-/// </summary>
 public sealed class OfflineChatClientTests
 {
     private static OfflineChatClient Build(Func<string>? scenario = null)
@@ -19,13 +16,9 @@ public sealed class OfflineChatClientTests
 
         var metadata = Assert.IsType<ChatClientMetadata>(client.GetService(typeof(ChatClientMetadata)));
 
-        // Estes DOIS campos entram na chave de cache. Se divergissem do run que gravou, todo
-        // cenario daria miss e a cache quente seria indistinguivel de uma cache vazia.
         Assert.Equal("morph-fake", metadata.ProviderName);
         Assert.Equal("modelo-exemplar", metadata.DefaultModelId);
 
-        // E o endpoint e NULO de proposito: ele nao entra na chave (medido na v10.9.0), e um
-        // cliente que nunca liga para lugar nenhum nao deve anunciar um destino.
         Assert.Null(metadata.ProviderUri);
     }
 
@@ -59,9 +52,7 @@ public sealed class OfflineChatClientTests
     [Fact]
     public void TheScenarioNameIsReadAtCALLTIME_NotCaptured()
     {
-        // Um fixture roda N cenarios com o MESMO cliente. Se o nome fosse capturado na
-        // construcao, toda mensagem de miss nomearia o primeiro cenario e mandaria o leitor
-        // investigar o arquivo errado.
+        // O fixture roda N cenarios com o mesmo cliente: o nome do cenario nao pode ser capturado na construcao.
         var current = "primeiro";
         using var client = Build(() => current);
 

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

@@ -3,30 +3,6 @@ using Microsoft.Extensions.AI;
 
 namespace Morph.AiKit.Tests.Fakes;
 
-/// <summary>
-/// <see cref="IChatClient"/> roteirizado: a costura que dispensa rede e chave de API na
-/// suite inteira do kit.
-/// </summary>
-/// <remarks>
-/// <para>
-/// <b>Por que ele e o coracao da historia de teste:</b> o unico ponto em que o kit toca um
-/// provider e um <see cref="IChatClient"/>. Trocando esse ponto por uma fila de respostas
-/// escritas a mao, TODO o resto — <c>ModelRegistry</c>, <c>AgentFactory</c>,
-/// <c>UsageCostMiddleware</c>, <c>ConversationStore</c> — passa a ser testavel sem socket,
-/// sem segredo e sem flake. Nenhum tipo desta pasta abre cliente HTTP nem le variavel de
-/// ambiente — o canario estrutural disso e um grep sobre <c>tests/</c> (chave de provider,
-/// leitura de ambiente, cliente HTTP) que precisa voltar VAZIO, e o canario de
-/// comportamento e
-/// <c>FakeChatClientTests.TheWholeSeamAnswersFromMemoryAloneWithNoConfigurationAndNoConnection</c>.
-/// </para>
-/// <para>
-/// <b>Chamada nao roteirizada e defeito de teste, nao "comportamento padrao".</b> Esgotada
-/// a fila, o fake lanca com a contagem exata de chamadas. Devolver uma resposta vazia
-/// silenciosamente faria um teste passar por engano — que e a unica coisa pior do que um
-/// teste falhando.
-/// </para>
-/// <para>Seguro para uso concorrente: fila, contadores e capturas vivem sob um lock unico.</para>
-/// </remarks>
 public sealed class FakeChatClient : IChatClient
 {
     private readonly Lock _gate = new();
@@ -34,12 +10,10 @@ public sealed class FakeChatClient : IChatClient
     private readonly List<ChatOptions?> _receivedOptions = [];
     private readonly List<IReadOnlyList<ChatMessage>> _receivedMessages = [];
 
-    /// <summary>Cria um fake com a fila vazia. Enfileire antes de usar.</summary>
     public FakeChatClient()
     {
     }
 
-    /// <summary>Cria um fake ja roteirizado com as respostas dadas, na ordem.</summary>
     public FakeChatClient(params ChatResponse[] scripted)
     {
         ArgumentNullException.ThrowIfNull(scripted);
@@ -50,7 +24,6 @@ public sealed class FakeChatClient : IChatClient
         }
     }
 
-    /// <summary>Atalho: uma resposta de assistente por texto, na ordem dada.</summary>
     public static FakeChatClient Scripted(params string[] texts)
     {
         ArgumentNullException.ThrowIfNull(texts);
@@ -64,51 +37,35 @@ public sealed class FakeChatClient : IChatClient
         return client;
     }
 
-    /// <summary>Quantas vezes o cliente foi invocado (streaming incluso).</summary>
     public int CallCount { get; private set; }
 
-    /// <summary>
-    /// O <c>ProviderName</c> anunciado no <see cref="ChatClientMetadata"/>. Configuravel
-    /// porque ele ENTRA NA CHAVE da cache de eval: a arvore que este fake grava so e legivel
-    /// depois pelo <c>OfflineChatClient</c> se os dois se apresentarem com a mesma
-    /// identidade. Default preservado ("fake"): quem nao usa eval nao muda de comportamento.
-    /// </summary>
+    // ProviderName e DefaultModelId entram na chave da cache de eval: o OfflineChatClient precisa anunciar os mesmos.
     public string MetadataProviderName { get; init; } = "fake";
 
-    /// <summary>
-    /// O <c>DefaultModelId</c> anunciado no <see cref="ChatClientMetadata"/>. Tambem entra na
-    /// chave de cache. Nulo por default, como antes.
-    /// </summary>
     public string? MetadataDefaultModelId { get; init; }
 
-    /// <summary>Se <see cref="Dispose"/> ja foi chamado — usado para provar ownership de cache.</summary>
     public bool IsDisposed { get; private set; }
 
-    /// <summary>As <see cref="ChatOptions"/> recebidas, uma entrada por chamada, na ordem.</summary>
     public IReadOnlyList<ChatOptions?> ReceivedOptions
     {
         get { lock (_gate) { return [.. _receivedOptions]; } }
     }
 
-    /// <summary>As mensagens recebidas, uma entrada por chamada, na ordem.</summary>
     public IReadOnlyList<IReadOnlyList<ChatMessage>> ReceivedMessages
     {
         get { lock (_gate) { return [.. _receivedMessages]; } }
     }
 
-    /// <summary>As opcoes da ultima chamada, ou nulo se nunca foi chamado.</summary>
     public ChatOptions? LastOptions
     {
         get { lock (_gate) { return _receivedOptions.Count == 0 ? null : _receivedOptions[^1]; } }
     }
 
-    /// <summary>As mensagens da ultima chamada, ou nulo se nunca foi chamado.</summary>
     public IReadOnlyList<ChatMessage>? LastMessages
     {
         get { lock (_gate) { return _receivedMessages.Count == 0 ? null : _receivedMessages[^1]; } }
     }
 
-    /// <summary>Enfileira uma resposta ja montada. Devolve o proprio fake, para encadear.</summary>
     public FakeChatClient Enqueue(ChatResponse response)
     {
         ArgumentNullException.ThrowIfNull(response);
@@ -121,18 +78,9 @@ public sealed class FakeChatClient : IChatClient
         return this;
     }
 
-    /// <summary>
-    /// Enfileira uma resposta de assistente com o texto dado e, opcionalmente, um
-    /// <see cref="UsageDetails"/> controlado — e assim que o teste do
-    /// <c>UsageCostMiddleware</c> (T11) verifica tarifa sem rede.
-    /// </summary>
     public FakeChatClient EnqueueText(string text, UsageDetails? usage = null)
         => Enqueue(new ChatResponse(new ChatMessage(ChatRole.Assistant, text)) { Usage = usage });
 
-    /// <summary>
-    /// Enfileira uma resposta com contagem de tokens conhecida. Atalho para o caso mais
-    /// comum de teste de custo.
-    /// </summary>
     public FakeChatClient EnqueueTextWithUsage(
         string text,
         long inputTokens,
@@ -148,7 +96,6 @@ public sealed class FakeChatClient : IChatClient
                 CachedInputTokenCount = cachedInputTokens,
             });
 
-    /// <inheritdoc />
     public Task<ChatResponse> GetResponseAsync(
         IEnumerable<ChatMessage> messages,
         ChatOptions? options = null,
@@ -160,7 +107,6 @@ public sealed class FakeChatClient : IChatClient
         return Task.FromResult(Record(messages, options));
     }
 
-    /// <inheritdoc />
     public async IAsyncEnumerable<ChatResponseUpdate> GetStreamingResponseAsync(
         IEnumerable<ChatMessage> messages,
         ChatOptions? options = null,
@@ -180,7 +126,6 @@ public sealed class FakeChatClient : IChatClient
         await Task.CompletedTask.ConfigureAwait(false);
     }
 
-    /// <inheritdoc />
     public object? GetService(Type serviceType, object? serviceKey = null)
     {
         ArgumentNullException.ThrowIfNull(serviceType);
@@ -198,7 +143,6 @@ public sealed class FakeChatClient : IChatClient
         return serviceType.IsInstanceOfType(this) ? this : null;
     }
 
-    /// <inheritdoc />
     public void Dispose() => IsDisposed = true;
 
     private ChatResponse Record(IEnumerable<ChatMessage> messages, ChatOptions? options)

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

@@ -4,17 +4,6 @@ using Xunit;
 
 namespace Morph.AiKit.Tests.Fakes;
 
-/// <summary>
-/// Canarios da costura de teste (T9). Um fake so vale se ele proprio for testado: um fake
-/// que devolve na ordem errada, ou que engole as <see cref="ChatOptions"/> recebidas,
-/// transforma todo teste que depende dele num teste que nao mede nada.
-/// </summary>
-/// <remarks>
-/// Todo <c>await</c> passa <c>TestContext.Current.CancellationToken</c> porque o analisador
-/// xUnit1051 cobra isso e <c>TreatWarningsAsErrors</c> transforma a cobranca em build
-/// vermelho. Nao ha <c>#pragma</c> aqui: a regra e boa (teste cancelavel) e o custo e um
-/// argumento.
-/// </remarks>
 public sealed class FakeChatClientTests
 {
     [Fact]
@@ -114,8 +103,6 @@ public sealed class FakeChatClientTests
     [Fact]
     public async Task UsageIsControlledByTheTestAndSurvivesTheRoundTrip()
     {
-        // E assim que a T11 vai verificar tarifa sem rede: Usage conhecido entra,
-        // Usage conhecido sai.
         using var client = new FakeChatClient();
         client.EnqueueTextWithUsage("ok", inputTokens: 1_000, outputTokens: 250, cachedInputTokens: 400);
 
@@ -155,17 +142,6 @@ public sealed class FakeChatClientTests
     [Fact]
     public async Task TheWholeSeamAnswersFromMemoryAloneWithNoConfigurationAndNoConnection()
     {
-        // O caminho inteiro que os consumidores usam — documento JSON, registry, fabrica,
-        // cliente — exercitado com nada alem de objetos em memoria: nenhuma chave, nenhum
-        // arquivo, nenhum socket. E este o teste a apontar quando alguem perguntar "como o
-        // kit testa IA sem provider".
-        //
-        // A propriedade "passa sem chave de provider no ambiente" NAO e afirmada aqui por
-        // codigo: afirma-la exigiria ler variavel de ambiente, e um teste que faz isso
-        // derruba o canario estrutural da T9 — o grep sobre tests/ (chave de provider,
-        // leitura de ambiente, cliente HTTP) tem de voltar VAZIO. A propriedade e
-        // DEMONSTRADA rodando a suite com o ambiente esvaziado, o que cobre os 50 testes
-        // em vez de um.
         var document = """
             {
               "version": "2.0",
@@ -195,8 +171,6 @@ public sealed class FakeChatClientTests
     [Fact]
     public async Task ConcurrentCallersNeverLoseACallOrACapture()
     {
-        // ConversationStore (T13) exercita duas sessoes sobre a MESMA instancia. Se o fake
-        // perdesse chamada sob concorrencia, aquele teste mediria o fake, nao o store.
         using var client = new FakeChatClient();
         for (var i = 0; i < 50; i++)
         {

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

@@ -1,30 +1,12 @@
 namespace Morph.AiKit.Tests.Fakes;
 
-/// <summary>
-/// <see cref="TimeProvider"/> controlado pelo teste: o relogio so anda quando o teste manda.
-/// </summary>
-/// <remarks>
-/// <para>
-/// Existe <b>escrito a mao, em 20 linhas</b>, e nao via
-/// <c>Microsoft.Extensions.TimeProvider.Testing</c>, por uma razao de contrato desta
-/// feature: toda versao de pacote vem de <c>framework/ai-pin.json</c>, e aquele pacote nao
-/// esta no pin. Acrescenta-lo seria uma versao decidida fora do pin — exatamente a segunda
-/// verdade que o kit existe para impedir. O achado esta registrado no relatorio da T11.
-/// </para>
-/// <para>
-/// So o necessario esta implementado: <see cref="GetUtcNow"/> e o avanco manual. Timers nao
-/// sao usados por nenhum tipo do kit; herdar o comportamento base e mais honesto que
-/// simular um agendador que ninguem exercita.
-/// </para>
-/// </remarks>
+// Escrito a mao: Microsoft.Extensions.TimeProvider.Testing nao esta no pin (framework/ai-pin.json).
 public sealed class FakeClock(DateTimeOffset start) : TimeProvider
 {
     private DateTimeOffset _now = start;
 
-    /// <inheritdoc />
     public override DateTimeOffset GetUtcNow() => _now;
 
-    /// <summary>Avanca o relogio. Rejeita andar para tras: o tempo do teste e monotono.</summary>
     public void Advance(TimeSpan delta)
     {
         ArgumentOutOfRangeException.ThrowIfLessThan(delta, TimeSpan.Zero);