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

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

Olá @vinimrs, a ideia de autocomplete de CEPs é interessante! Mas o PR precisa de uma refatoração significativa antes do merge.

Bloqueadores:

1. package.json e package-lock.json com versões muito antigas

O PR foi criado a partir de um fork antigo. O package.json no diff mostra "next": "10.2.0" e "node": ">=16 <17", enquanto o projeto atual usa next: 12.x e node: >=20 <22. O lock file tem 16k+ linhas de diffs. Essas mudanças rebaixariam as dependências do projeto. Atualize o fork com o main atual e remova todas as mudanças em package.json e package-lock.json que não sejam os novos pacotes que você está adicionando.

2. Handler não usa o padrão do projeto

// ❌ atual
export default cors(Cep);

// ✅ correto
export default app().get(action);

O app() já inclui CORS, cache, logger, firewall e errorHandler. Use-o.

3. Sem classes de erro do projeto

// ❌ atual
response.status(400); return response.json(error);

// ✅ correto
throw new BadRequestError({ message: '...' });

Deixe o errorHandler middleware formatar erros. Não use response.status() manual nos handlers.

4. CSV lido do disco em cada requisição

const json = await csvToJson().fromFile(csvFilePath); // leitura em toda request!

Em serverless, isso é lento e não compartilha cache entre invocações. Considere importar o CSV como módulo no topo do arquivo (ou usar import dinâmico com cache), para que seja lido apenas uma vez por instância.

5. Lógica de bloqueio de IP no handler

const tempBlockedIps = [];
const tempBlockedUserAgents = ['Go-http-client/2.0'];

Regras de bloqueio não pertencem a um handler específico. O firewall middleware (middlewares/firewall.js) já existe para isso.

6. Testes com sintaxe CommonJS e sem imports do Vitest

// ❌
const axios = require('axios');

// ✅
import axios from 'axios';
import { describe, test, expect } from 'vitest';

7. Testes com ordem de resultados hardcoded

expect(response.data[0]).toEqual({ state: 'SP', city: 'São Paulo', ... });
expect(response.data[1]).toEqual({ state: 'PR', city: 'Alto Paraná', ... });

A ordem dos resultados de autocomplete não é garantida. Prefira expect.arrayContaining([...]) ou ordene os resultados explicitamente no serviço.

8. ESLint no-else-return: 'off'

Não desabilite regras ESLint project-wide para um endpoint específico. Ajuste o código para respeitar a regra.

O conceito da feature é válido. Com as correções estruturais acima, pode ser um endpoint útil. Recomendo rebase do fork com main antes de refatorar.