{{ message }}
Repository navigation
feat: endereço da sede dos bancos (rebase #763) - #876
Merged
Merged
Conversation
… de todos os bancos da API Implementação de endereco da sede de todos os 352 bancos cadastrados na API. feat #746
Co-authored-by: Hermes Agent <noreply@nousresearch.com> Ai-tool: hermes Ai-model: deepseek-v4-flash Ai-os: wsl
O teste do fork validava um esquema (headquarters_address, address_source) que não corresponde à feature localizacao_sede deste PR, e os snapshots são dados temporários. Co-authored-by: Hermes Agent <noreply@nousresearch.com> Ai-tool: hermes Ai-model: deepseek-v4-flash Ai-os: wsl
O teste do fork valida o enriquecimento headquarters_address (BCB Olinda + CNPJ) que faz parte do PR; o snapshot latest.json é o fallback usado pelo serviço quando o BCB está indisponível. Co-authored-by: Hermes Agent <noreply@nousresearch.com> Ai-tool: hermes Ai-model: deepseek-v4-flash Ai-os: wsl
This was referenced Aug 10, 2026
lucianopf
added a commit
that referenced
this pull request
Aug 10, 2026
#885) * refactor: lint limpo no CI, teste NCM honesto e move codes.js para services - CI: step de ESLint no workflow E2E (lint nunca rodava — 43 erros passaram despercebidos) - lint: mapUfToUfCode (42x prettier), weather.js, cpf (console.log redundante), cep-error-handler (disable scoped) - teste NCM: expect.assertions + comportamento real (200 [] pós-984c956) — antes passava sem validar nada - move pages/api/indices/codes.js → services/indices/codes.js (módulo de constantes virava rota 500 em produção) Co-authored-by: Hermes Agent <noreply@nousresearch.com> * docs: paridade total rotas ↔ OpenAPI (0 rotas sem doc, 0 paths fantasma) Auditoria achou 26 rotas sem doc + ~10 params divergentes (doc PT-BR vs código EN, dos rebases #861-#876). Correções: - params renomeados: fipe (tipoVeiculo→vehicleType...), ibge (siglaUF→uf...), cptec (icaoCode→airport), ibpt (siglaUF→uf), solar (path→query) - paths novos: /ibge/uf/v1/regiao, /fipe/marcas/v1, /feriados/v1, /oms/cid10/v1, /status/v1 (cid10.json + status.json novos) - eleicoes.json: prefixo /api removido dos paths - holydays/statewide: path fantasma {ano}/{uf} removido (UF é query) - ibpt: ?ex=01 removido do path key (query param) - teste novo: docs-parity.test.js (paridade rotas reais ↔ doc) Co-authored-by: Hermes Agent <noreply@nousresearch.com> --------- Co-authored-by: Hermes Agent <noreply@nousresearch.com>
lucianopf
added a commit
that referenced
this pull request
Aug 10, 2026
* refactor: lint limpo no CI, teste NCM honesto e move codes.js para services - CI: step de ESLint no workflow E2E (lint nunca rodava — 43 erros passaram despercebidos) - lint: mapUfToUfCode (42x prettier), weather.js, cpf (console.log redundante), cep-error-handler (disable scoped) - teste NCM: expect.assertions + comportamento real (200 [] pós-984c956) — antes passava sem validar nada - move pages/api/indices/codes.js → services/indices/codes.js (módulo de constantes virava rota 500 em produção) Co-authored-by: Hermes Agent <noreply@nousresearch.com> * docs: paridade total rotas ↔ OpenAPI (0 rotas sem doc, 0 paths fantasma) Auditoria achou 26 rotas sem doc + ~10 params divergentes (doc PT-BR vs código EN, dos rebases #861-#876). Correções: - params renomeados: fipe (tipoVeiculo→vehicleType...), ibge (siglaUF→uf...), cptec (icaoCode→airport), ibpt (siglaUF→uf), solar (path→query) - paths novos: /ibge/uf/v1/regiao, /fipe/marcas/v1, /feriados/v1, /oms/cid10/v1, /status/v1 (cid10.json + status.json novos) - eleicoes.json: prefixo /api removido dos paths - holydays/statewide: path fantasma {ano}/{uf} removido (UF é query) - ibpt: ?ex=01 removido do path key (query param) - teste novo: docs-parity.test.js (paridade rotas reais ↔ doc) Co-authored-by: Hermes Agent <noreply@nousresearch.com> * feat: campo logo_url no /banks/v1 (issue #834) Adiciona logo_url (CDN jsDelivr do projeto logos-bancos-br, do autor da issue) para bancos com logo mapeado por ISPB; null nos demais (154/469 hoje, 33%). - snapshot services/banco-central/logo-urls.json (476 ISPB→URL, gerado do pacote npm logos-bancos-br@0.6.2) - enrich em getBanksData cobre todos os caminhos (CSV/snapshot/fallback) - doc: schema Bank com logo_url opcional (string|null) - testes: logo presente, null para sem-logo, compatibilidade retroativa Co-authored-by: Hermes Agent <noreply@nousresearch.com> --------- Co-authored-by: Hermes Agent <noreply@nousresearch.com>
lucianopf
added a commit
that referenced
this pull request
Aug 10, 2026
* refactor: lint limpo no CI, teste NCM honesto e move codes.js para services - CI: step de ESLint no workflow E2E (lint nunca rodava — 43 erros passaram despercebidos) - lint: mapUfToUfCode (42x prettier), weather.js, cpf (console.log redundante), cep-error-handler (disable scoped) - teste NCM: expect.assertions + comportamento real (200 [] pós-984c956) — antes passava sem validar nada - move pages/api/indices/codes.js → services/indices/codes.js (módulo de constantes virava rota 500 em produção) Co-authored-by: Hermes Agent <noreply@nousresearch.com> * docs: paridade total rotas ↔ OpenAPI (0 rotas sem doc, 0 paths fantasma) Auditoria achou 26 rotas sem doc + ~10 params divergentes (doc PT-BR vs código EN, dos rebases #861-#876). Correções: - params renomeados: fipe (tipoVeiculo→vehicleType...), ibge (siglaUF→uf...), cptec (icaoCode→airport), ibpt (siglaUF→uf), solar (path→query) - paths novos: /ibge/uf/v1/regiao, /fipe/marcas/v1, /feriados/v1, /oms/cid10/v1, /status/v1 (cid10.json + status.json novos) - eleicoes.json: prefixo /api removido dos paths - holydays/statewide: path fantasma {ano}/{uf} removido (UF é query) - ibpt: ?ex=01 removido do path key (query param) - teste novo: docs-parity.test.js (paridade rotas reais ↔ doc) Co-authored-by: Hermes Agent <noreply@nousresearch.com> * feat: filtro por UF em /cvm/corretoras/v1 (issue #736) GET /api/cvm/corretoras/v1?uf=SP retorna apenas corretoras do estado (166 de 375 hoje). UF inválida → 400 validation_error; UF sem corretoras → 200 [] (padrão do repo). Sem uf → comportamento atual (lista completa). - doc: param opcional uf no OpenAPI - testes: filtro SP (todos uf=SP), UF sem resultado, UF inválida 400 Co-authored-by: Hermes Agent <noreply@nousresearch.com> * test: move import do helper CORS para o topo (lint import/first) --------- Co-authored-by: Hermes Agent <noreply@nousresearch.com>
lucianopf
added a commit
that referenced
this pull request
Aug 10, 2026
* refactor: lint limpo no CI, teste NCM honesto e move codes.js para services - CI: step de ESLint no workflow E2E (lint nunca rodava — 43 erros passaram despercebidos) - lint: mapUfToUfCode (42x prettier), weather.js, cpf (console.log redundante), cep-error-handler (disable scoped) - teste NCM: expect.assertions + comportamento real (200 [] pós-984c956) — antes passava sem validar nada - move pages/api/indices/codes.js → services/indices/codes.js (módulo de constantes virava rota 500 em produção) Co-authored-by: Hermes Agent <noreply@nousresearch.com> * docs: paridade total rotas ↔ OpenAPI (0 rotas sem doc, 0 paths fantasma) Auditoria achou 26 rotas sem doc + ~10 params divergentes (doc PT-BR vs código EN, dos rebases #861-#876). Correções: - params renomeados: fipe (tipoVeiculo→vehicleType...), ibge (siglaUF→uf...), cptec (icaoCode→airport), ibpt (siglaUF→uf), solar (path→query) - paths novos: /ibge/uf/v1/regiao, /fipe/marcas/v1, /feriados/v1, /oms/cid10/v1, /status/v1 (cid10.json + status.json novos) - eleicoes.json: prefixo /api removido dos paths - holydays/statewide: path fantasma {ano}/{uf} removido (UF é query) - ibpt: ?ex=01 removido do path key (query param) - teste novo: docs-parity.test.js (paridade rotas reais ↔ doc) Co-authored-by: Hermes Agent <noreply@nousresearch.com> * test: skip B3 quando o provider bloqueia o runner (403 datacenter) O b3-v1.test.js não tinha o padrão health-check+skip dos outros providers — quando a B3 bloqueia o runner (403 por IP de datacenter), 22 testes E2E + CORS falhavam e derrubavam o CI de todos os PRs. Padrão: describe.skipIf(shouldSkipTests) (como ibge-populacao-fix). Co-authored-by: Hermes Agent <noreply@nousresearch.com> * test: skip CEP v3 quando OpenCEP falha no runner (coordenadas null) O runner pegou OpenCEP fora do ar: coordinates vieram null e o teste toEqua... quebrava (1 falha flaky derrubando o CI). Mesmo padrão describe.skipIf(shouldSkipTests) dos outros providers. Co-authored-by: Hermes Agent <noreply@nousresearch.com> --------- Co-authored-by: Hermes Agent <noreply@nousresearch.com>
This branch had an error being deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rebase do PR original (#763, fork lucsantosdev) na main atual.
Feature:
GET /banks/v1e/banks/v1/:codeagora incluem:localizacao_sede(estático, do banksList com 327/469 bancos)headquarters_address+address_source+address_last_sync+address_confidence+cnpj(enriquecimento dinâmico via BCB Olinda + CNPJ, com cache 6h e snapshot fallback)Ajustes no rebase:
latest.jsonmantido (fallback do serviço)snapshot:bankspara regenerar o snapshotDestaque: o fork do autor continha ~28k linhas de mudanças não relacionadas (cptec, holidays, ibge, isbn, vite.config) — tudo descartado, só o bank ficou.