refactor: centraliza geração de URL pré-assinada no serviço de docume…

[JoaoVictor-Lucena]

…ntos (resolves #947)

O que mudou?

O método generatePresignedUrl estava duplicado palavra por palavra em dois controllers (PatientDocumentsControllerImpl e HealthProfessionalControllerImpl). Este PR centraliza a lógica na camada de serviço (MinioDocumentApplicationService), eliminando a duplicação e substituindo o System.err.println por log.error com logger adequado.

Tarefas Relacionadas

• Issue: [REFACTOR] Juntar o método de gerar URL de documento que está duplicado #947
• Outras dependências: N/A

Mudanças Realizadas

• Move generatePresignedUrl para MinioDocumentApplicationService e o declara na interface DocumentApplicationService
• Remove todas as chamadas diretas a getPresignedDocumentUrl nos controllers, substituindo por chamadas ao novo método centralizado
• Substitui System.err.println por log.error com logger SLF4J declarado como campo da classe
• Atualiza os testes de PatientDocumentsControllerTest e HealthProfessionalControllerTest para mockar generatePresignedUrl corretamente

Evidências

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 29459156-4e09-4046-ad7e-680442068502

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[dariomatias-dev]

Dois ajustes:

1. O parâmetro args em DocumentApplicationService.java:39 é redundante, tudo nele vem do dto. Por isso o builder continua repetido em 4 lugares:

• PatientDocumentsControllerImpl.java:124
• PatientDocumentsControllerImpl.java:150
• PatientDocumentsControllerImpl.java:190
• HealthProfessionalControllerImpl.java:105

Trocar a assinatura para generatePresignedUrl(DocumentDTO dto, Duration expiry) e montar o args dentro do serviço.

2. O getProfessionalDocuments (HealthProfessionalControllerImpl.java:93-123) ficou com 4 espaços a mais que o resto da classe, o que joga ~30 linhas sem alteração real no diff. Recuar o método todo em 4 espaços resolve.

[dariomatias-dev]

Remova o apps/api/build_out.txt, é um log de build que entrou no último commit.