Corrige diagnósticos do renderizador e do Contract Studio

[devmnsoft]

Motivation

• Eliminar os diagnósticos de análise estática (CA1834, CS8604, CA1305, CA1859) sem alterar a semântica existente de renderização, histórico, comparação, idempotência, autorização ou isolamento por organização.
• Preservar a saída HTML/PDF, histórico e hashes já persistidos, evitando mudanças de contrato HTTP, nulabilidade ou supressões de analisadores.
• Aplicar as menores correções locais necessárias para satisfazer os analisadores e tornar os tipos retornados privados explícitos quando apropriado.

Description

• Corrige CA1834 no renderizador substituindo StringBuilder.Append(string) por Append(char) para a aspa final do atributo de alinhamento em src/Odca.Application/Contracts/ContractDocumentRenderer.cs, preservando exatamente o HTML produzido.
• Resolve CS8604 ao produzir o histórico de preparação em src/Odca.Api/Controllers/ContractStudioController.cs lendo timezone como string?, validando-o com TimeZonePolicy.Resolve(timeZoneId, configuration["TimeZone:DefaultId"]) e retornando um Problem(503) controlado quando a configuração estiver indisponível, além de usar o identificador validado timeZone na consulta SQL (AT TIME ZONE @timeZone) e no SignaturePreparationHistoryPage.
• Corrige CA1859 tornando os helpers privados que materializam resultados do Dapper concretos: LockOperation e AddOperation agora retornam Task<int> e CompareCompositions retorna List<SignatureCompositionChange> para refletir o tipo concretamente usado.
• Corrige CA1305 formatando posições com cultura explícita (old.Position.ToString(CultureInfo.InvariantCulture) / current.Position.ToString(CultureInfo.InvariantCulture)) e adiciona using System.Globalization e using Odca.Application.Tenancy onde necessário; não foram alterados contratos ou formatos que afetem hashes persistidos.

Testing

• git diff --check passou e a árvore de trabalho foi commitada (commit c595437).
• Verificações locais de diffs/estatísticas mostraram as duas modificações em src/Odca.Application/Contracts/ContractDocumentRenderer.cs e src/Odca.Api/Controllers/ContractStudioController.cs.
• Tentar executar dotnet restore Odca.sln --locked-mode, dotnet build Odca.sln --configuration Release --no-restore e dotnet test Odca.sln --configuration Release --no-build não foi possível porque dotnet não está instalado no ambiente; portanto a correção não foi validada por um build/execução de testes neste ambiente.
• Tentativas de instalar o SDK .NET (versão 10.0.400 conforme global.json) falharam por bloqueio do servidor remoto (HTTP 403), portanto a validação de compilação e testes permanece pendente em ambiente com o SDK disponível.

---

Codex Task

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
🔒 Security Review	✅ Completed 2026-09-25T12:25:51.239199Z	e8bdb08	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.