@mtsmgn0 is attempting to deploy a commit to the BrasilAPI Team on Vercel.

A member of the Team first needs to authorize it.

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á @mtsmgn0! A adição de feriados estaduais é uma funcionalidade bem solicitada pela comunidade.

Algumas observações:

1. Sem endpoint exposto
O PR adiciona a função getStateHolidays no serviço, mas não cria nem modifica nenhum endpoint de API para expô-la. O endpoint atual /feriados/v1/{ano} não aceita parâmetro de estado. Seria necessário criar um novo endpoint (ex: /feriados/v1/{ano}/{estado}) ou adicionar um query param.

2. Dados estaticamente codificados
Os feriados estaduais estão hardcoded no código. Isso é um problema de manutenibilidade — eles mudam com legislações estaduais. Qual é a fonte dos dados para garantir precisão?

3. Arquivos não relacionados
O PR inclui package-lock.json e vite.config.js com alterações que não fazem parte do escopo.

Sugestão: Se quiser avançar, abra primeiro uma issue para discutir o design do novo endpoint, a fonte dos dados e como manter os feriados estaduais atualizados. Assim podemos planejar a funcionalidade completa antes da implementação.

Olá @mtsmgn0! A adição de feriados estaduais é uma boa ideia, mas a PR tem um problema crítico.

Bloqueador: remoção do campo weekday

O PR #755 adicionou o campo weekday aos feriados — esse campo está em produção e usuários já podem depender dele. Sua PR remove esse campo de todos os feriados:

-    weekday: getWeekdayName(easterDate),

Isso quebraria sistemas em produção que usam response[i].weekday. Nunca remova campos existentes das respostas — o projeto tem zero tolerância para quebra de compatibilidade retroativa.

Como corrigir

1. Faça rebase no main atual (que já tem o campo weekday do PR feat: Adicionar campo weekday no endpoint de feriados #755)
2. Não remova o campo weekday dos feriados existentes
3. Adicione weekday nos novos feriados estaduais também (ou não adicione — mas não remova dos existentes)

Outras observações

• Os testes no arquivo tests/feriados/getHolidays.test.js testam a função de serviço diretamente, o que está ok para lógica interna. Mas lembre-se de também adicionar um teste E2E para o endpoint HTTP quando criar o endpoint para feriados estaduais.
• Seria necessário criar um novo endpoint (/api/feriados/v1/{ano}/{uf}) para expor os feriados estaduais via HTTP.
• O arquivo vite.config.js foi alterado desnecessariamente — por favor, reverta.

Corrija esses pontos e aí posso fazer uma revisão completa!