The latest updates on your projects. Learn more about Vercel for Git ↗︎

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Olá @louieaveline! Obrigado pela contribuição!

Preciso fazer algumas observações importantes antes do merge:

1. Handler não usa o padrão do projeto
O handler não usa export default app().get(handler). O wrapper app() aplica automaticamente cors, cache, logger, firewall e errorHandler. Sem ele, essas funcionalidades não estarão disponíveis.

2. console.log em código de produção
O handler tem 2 console.log que vão poluir os logs em produção. Use console.error apenas para erros, e só quando necessário.

3. Tratamento de erros incorreto
Erros são capturados e retornados com res.status(404).json({...}) diretamente. O padrão é lançar instâncias das classes de erro (throw new NotFoundError(...)) e deixar o middleware errorHandler formatar a resposta.

4. Ausência de testes
Novos endpoints precisam de testes E2E em tests/estados-v1.test.js (ou similar).

5. Ausência de documentação OpenAPI
É necessário criar pages/docs/doc/estados.json com a especificação OpenAPI.

6. Funcionalidade já parcialmente existente
O endpoint /ibge/uf/v1/{code} já retorna informações de estados. Qual informação adicional este endpoint oferece em relação ao existente?

Sugiro revisar o Guia de Criação de Endpoints antes de ajustar.