Skip to content

Fix local API origin resolution - #42

Merged
pedrobragabes merged 1 commit into
mainfrom
codex/issue-41-local-api
Jul 16, 2026
Merged

Fix local API origin resolution#42
pedrobragabes merged 1 commit into
mainfrom
codex/issue-41-local-api

Conversation

@pedrobragabes

Copy link
Copy Markdown
Collaborator

O que mudou

  • usa window.location.origin + /api em qualquer origem HTTP(S) local ou de preview;
  • mantém comerciobes.com.br e www.comerciobes.com.br apontando para a API oficial;
  • reconhece loopback IPv4 e IPv6 para o fallback de dados;
  • preserva o fallback explícito de file:// para localhost:3000;
  • sobe o cache PWA para v11 para distribuir a configuração corrigida;
  • documenta portas locais arbitrárias e adiciona testes comportamentais.

Impacto

Abrir a vitrine em uma porta diferente de 3000 não consulta mais outro processo por engano. Frontend, API e painel permanecem same-origin no desenvolvimento.

Validação

  • 14 testes direcionados de configuração e service worker
  • node --check js/config.js
  • node --check sw.js
  • npm run build
  • npm run frontend:check
  • npm audit --omit=dev --audit-level=high — 0 vulnerabilidades
  • QA em http://127.0.0.1:4317: API conectada, 16 comércios carregados e 8 cards renderizados

Closes #41

@pedrobragabes pedrobragabes added P1 Alta prioridade frontend Frontend e UX devops CI, deploy e operacao labels Jul 16, 2026
@pedrobragabes
pedrobragabes marked this pull request as ready for review July 16, 2026 14:49
Copilot AI review requested due to automatic review settings July 16, 2026 14:49
@pedrobragabes
pedrobragabes merged commit 4752e23 into main Jul 16, 2026
1 check passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops CI, deploy e operacao frontend Frontend e UX P1 Alta prioridade

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dev: usar API same-origin em qualquer porta local

2 participants