Skip to content

feat: postgres, config por variavel de ambiente e migrate no servidor - #32

Merged
thnbi merged 19 commits into
devfrom
feat/servidor-postgres
Oct 2, 2026
Merged

thnbi merged 19 commits into
devfrom
feat/servidor-postgres

Conversation

@thnbi

@thnbi thnbi commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

O que mudou

  • Servidor troca SQLite por PostgreSQL com pgx/v5; os testes rodam contra um Postgres real com testcontainers, um banco por teste
  • Configuração vem só de variáveis de ambiente, com porta e tamanho de nota validados por faixa; o config.json sai
  • Migração sai do boot do servidor para o subcomando markupp migrate
  • Compose sobe Postgres, roda a migração e depois o servidor; make test roda no host
  • Postgres fixado em 17.9-alpine3.23 no compose e nos testes
  • DEPLOY.md e README do servidor atualizados

Por quê

Observações

  • A migração 00001 foi editada no lugar, trocando TIMESTAMP por TIMESTAMPTZ, sem migração nova. Só é seguro porque ainda não existe banco PostgreSQL de ninguém
  • O controle de versão já trunca timestamp em milissegundo, então não há 409 falso com o Postgres

Como testar

  • make test (precisa de Go e Docker)
  • make run e os curl da seção Validar do DEPLOY.md

Auto-review (checklist)

  • Descrição clara (o que/por quê/como testar)
  • PR pequeno e focado
  • Casos limite considerados (ex.: vazio, 0, erro)
  • Evidência de teste (manual ou automatizado)

@thnbi
thnbi requested a review from NicolasArthurDev October 2, 2026 22:45
@thnbi thnbi self-assigned this Oct 2, 2026
@thnbi thnbi added the enhancement New feature or request label Oct 2, 2026

@NicolasArthurDev NicolasArthurDev 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.

Muito boa a separação, ficou bem mais fácil de revisar! A troca para pgx está limpa, o storagetest com um banco por teste garante isolamento, e a validação de faixa da porta e do tamanho de nota ficou bem feita, com mensagem que mostra o valor recebido. A observação sobre a 00001 editada no lugar na descrição foi um ótimo cuidado. Aprovado.

@thnbi
thnbi changed the base branch from docs/escala-a-zero-com-keda to dev October 2, 2026 23:28
@thnbi
thnbi merged commit 0010ede into dev Oct 2, 2026
2 checks passed
@thnbi
thnbi deleted the feat/servidor-postgres branch October 2, 2026 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants