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

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

Quality Gate passed

Issues
0 New issues
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á @manuseiro, a proposta de incidência solar é interessante! Mas o PR precisa de uma refatoração completa antes do merge.

Problemas estruturais (bloqueadores):

1. O arquivo handler não está no lugar certo e está incompleto

O arquivo chamado Incidence não tem extensão .js, não está em pages/api/, e não segue o padrão de handlers do projeto:

// pages/api/solar-incidence/v1/[location].js
import app from '@/app';
import NotFoundError from '@/errors/NotFoundError';
import { getSolarIncidence } from '@/services/solarIncidence';

async function handler(request, response) {
  const { location } = request.query;
  const data = await getSolarIncidence(location);
  return response.status(200).json(data);
}

export default app().get(handler);

• Use import app from '@/app' (wrapper com CORS, cache, logger, etc.)
• O alias @/ aponta para a raiz do projeto
• Não use try/catch genérico no handler — deixe o errorHandler middleware tratar

2. Testes com Jest em vez de Vitest

O projeto usa Vitest, não Jest. Troque:

// ❌
import { getSolarIncidence } from '../src/services/solarIncidenceService';
jest.mock('axios');

// ✅
import { describe, expect, test } from 'vitest';
import axios from 'axios';
// testes E2E com global.SERVER_URL (sem mocks de axios)

Os testes devem ser E2E que fazem chamadas HTTP reais ao servidor local. Veja tests/cep-v1.test.js ou tests/fipe-v1.test.js como referência.

3. Fonte de dados: verificar termos de uso

• api.sunrise-sunset.org é uma API de terceiros — confirme que não há limitações de uso que possam afetar a BrasilAPI em produção.
• Nominatim (OpenStreetMap) tem limitações de uso: máximo 1 req/s e obrigatoriedade de User-Agent identificador. Em produção, sem cache adequado, isso pode violar os termos.

4. Path do serviço fora do padrão

Mova para: services/solar-incidence/index.js (ou services/solarIncidence/index.js), seguindo o padrão services/{domain}/.

5. ErrorMessage redefinido na documentação

O schema ErrorMessage já existe globalmente no projeto. Não redefina — remova o bloco e use apenas $ref: "#/components/schemas/ErrorMessage" nas respostas de erro.

Recomendo ler o Guia de Criação de Endpoints do projeto antes de refatorar — ele tem templates prontos para handler, service, teste e documentação.