Remove the leftovers of the old plan (unused packages, Mailpit, template extras) #34

Contexto

O plano anterior a 2026-10-05 deixou no código pacotes e peças sem uso: uma biblioteca de mocks, a caixa de e-mail de teste, versões de pacotes de e-mail e de cache, e extras do modelo do Aspire e do Visual Studio. Eles estão listados em docs/decisoes-de-arquitetura.md, na tabela "O que ainda reflete o plano antigo". Esta issue os remove e corrige a documentação que os descreve como presentes. É a etapa 3 do plano de simplificação e o ensaio dos três comandos.

Escopo

Nenhum comportamento da API muda. Nenhum pacote é adicionado.

• NSubstitute (decisão 27): sai de ControlService.Application.Tests.csproj e de Directory.Packages.props.
• Mailpit (decisão 31): saem de AppHost.cs o recurso, o WithReference, o WaitFor e o comentário; sai o pacote CommunityToolkit.Aspire.Hosting.MailPit do .csproj do AppHost e do Directory.Packages.props; o comentário do .csproj do AppHost é corrigido.
• Versões sem uso (decisão 30): MailKit, Microsoft.Extensions.Caching.Hybrid e FluentValidation.DependencyInjectionExtensions saem de Directory.Packages.props. O comentário "Ready for the next slices" é reescrito, porque o grupo passa a ter só pacotes em uso.
• Service discovery e resiliência de HTTP (decisão 31): saem de ServiceDefaults/Extensions.cs as linhas 22 a 27; saem os pacotes Microsoft.Extensions.ServiceDiscovery e Microsoft.Extensions.Http.Resilience do .csproj e do Directory.Packages.props; o comentário do .csproj é corrigido.
• Pacote preview do Visual Studio (decisão 33): sai Microsoft.VisualStudio.Azure.Containers.Tools.Targets de ControlService.API.csproj e de Directory.Packages.props, com as propriedades DockerDefaultTargetOS e DockerfileContext e o perfil "Container (Dockerfile)" de launchSettings.json.
• Mudança em .github/ autorizada: tirar "NSubstitute" do grupo testing de dependabot.yml. Nada mais nesse arquivo.
• Documentação, no mesmo pull request:
  • README.md: as linhas "Local environment" e "Tests" da tabela, a descrição do ServiceDefaults na árvore e a frase "Start PostgreSQL, Mailpit and the API".
  • backend/ControlService/README.md: a frase do Aspire, a linha do Mailpit nas duas tabelas e "pare os dois" (passa a ser um contêiner).
  • docs/agents/guides/local-environment.md: a frase do Aspire e a linha do Mailpit nas três tabelas.
  • docs/decisoes-de-arquitetura.md:
    • remover as cinco linhas da tabela "O que ainda reflete o plano antigo" (fica a do CI, da Check formatting in CI #35);
    • remover os parágrafos "Sobras do plano antigo" das decisões 31 e 33;
    • na decisão 30, trocar o parágrafo "Sobras do plano antigo" por uma nota: uma versão só é declarada quando um projeto a usa; FluentValidation.DependencyInjectionExtensions foi removido por falta de uso e pode voltar se houver justificativa (por exemplo, registrar os validadores automaticamente);
    • na decisão 6, acrescentar uma nota: um helper de uma linha para registrar handlers validados foi avaliado em 2026-10-07 e adiado, porque só existe um handler validado; reavaliar no levantamento da primeira issue que registrar o segundo (Permission profiles and screen catalog endpoints #10 ou Users endpoints with a temporary password #11).

Fora do escopo

• Comportamento de login, usuários e permissões — etapa 4 (Authorization per screen and per-request account checks #9, Permission profiles and screen catalog endpoints #10, Users endpoints with a temporary password #11).
• Comentários com códigos do plano antigo (T12, D3) em arquivos .cs — issue própria, na etapa 3.
• Helper para registrar handlers validados — decidido no levantamento da Permission profiles and screen catalog endpoints #10.
• Passo de formatação no CI — Check formatting in CI #35. Teste de arquitetura da API — Architecture test: the API uses Infrastructure only in Program.cs #36.
• Comentários "previous plan, see decision 21" — etapa 4.
• Apagar o contêiner controlservice-mailpit da máquina — o dono roda o comando do relatório final.

Dependências

• PRs docs: replace the ADRs with one decisions document #31 e docs(auth): rewrite the access rules for the temporary password #37 na main (feito).
• Docker Desktop ligado, para os testes de integração, T04 e T05.

Riscos

• Um pacote removido trazia outro de carona que o código usa — o build falha com tipo ou método não encontrado. Corrigir a causa, sem devolver o pacote; se não houver saída, parar e avisar.
• A API não subir pelo AppHost sem a descoberta de serviços — T04 falha. A connection string do banco chega por WithReference(database), que não depende dela.
• Outra ferramenta ler DockerDefaultTargetOS ou DockerfileContext — T05 falha.
• O contêiner controlservice-mailpit continua ligado na máquina do dono — aparece no Docker Desktop; não afeta o sistema.

Decisões do levantamento

Tomadas pelo dono em 2026-10-07.

• Helper de registro de handlers validados: não criar agora — há um uso só; fica anotado na decisão 6 e na Permission profiles and screen catalog endpoints #10 para ser reavaliado.
• Comentários T12/D3: issue própria — pedem revisão de leitura em 42 arquivos, e não a prova por build desta issue.
• Remover também os restos do pacote do Visual Studio (duas propriedades e o perfil de execução) — sem o pacote, ficam quebrados.
• Remover FluentValidation.DependencyInjectionExtensions — versão sem uso, como o MailKit; a decisão 30 registra que pode voltar com justificativa.
• Tirar "NSubstitute" do dependabot.yml — única mudança em .github/.
• Manter a instrumentação de HttpClient do OpenTelemetry — pertence à decisão 32 e não está na tabela de sobras.
• O contêiner antigo do Mailpit é apagado pelo dono, com o comando do relatório final.
• Os testes desta issue são verificações por comando — não há comportamento novo para um teste xUnit.

Testes

Comandos rodados da raiz do repositório, salvo indicação.

Sem teste próprio: a API funcionar sem service discovery e resiliência — coberta por T03 (os testes de integração sobem a API inteira) e T04.

Critérios de aceite

• As verificações T01 a T06 passam, com o comando e a saída no relatório final.
• check.ps1 termina sem avisos, com todos os testes do projeto verdes.
• Nenhum arquivo de Auth, Users ou PermissionProfiles aparece no diff.
• Em .github/, o diff tem só a palavra "NSubstitute" removida de dependabot.yml.
• A tabela "O que ainda reflete o plano antigo" fica só com a linha do CI.
• As decisões 6 e 30 têm as notas sobre o helper adiado e o pacote que pode voltar.
• docs/ atualizado no mesmo pull request: README.md, backend/ControlService/README.md, docs/agents/guides/local-environment.md, docs/decisoes-de-arquitetura.md.
• O relatório final traz o comando para o dono apagar o contêiner controlservice-mailpit.

Recomendações futuras

• Avaliar a remoção de AddHttpClientInstrumentation() e do pacote OpenTelemetry.Instrumentation.Http: só agem quando a API chama outro serviço por HTTP. Exige rever a decisão 32.
• Helper de registro de handlers validados: decidir na Permission profiles and screen catalog endpoints #10, com dois ou mais casos reais.
• O modelo de issue não prevê issues só de remoção, em que os testes são verificações por comando. Se isso se repetir, vale uma linha na skill /levantar-issue.