Skip to content

chore(qualidade): rota de saude e phpstan nivel 5 (tasks 56, 51) - #37

Merged
andynnnfw merged 10 commits into
feat/csp-relatoriosfrom
chore/operacao-qualidade
Oct 9, 2026
Merged

andynnnfw merged 10 commits into
feat/csp-relatoriosfrom
chore/operacao-qualidade

Conversation

@sousa7tz

@sousa7tz sousa7tz commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

PR empilhado 4 de 6. Base: #36. Faça o merge na ordem; ao mergear o anterior, o GitHub muda a base deste para a main.

Tasks

  • 56 (parte de código): ?url=health (App\Support\Health) responde {"status":"ok"} ou 503 {"status":"erro"}, antes do session_start e sem detalhe. Documentado em docs/operacao.md.
  • 51 (parte de código): phpstan/phpstan ^2.1 como dependência de dev, nível 5 em src, views, bin e public, e o script composer analyse. A baseline tem só os "might not be defined" das views (vêm do extract()).

Fica para a infra (não está neste PR)

  • 56: HEALTHCHECK do Dockerfile apontando para ?url=health, monitor externo e alerta de 500.
  • 51: composer analyse e composer audit no CI, mais o Dependabot.

Como testar

  • composer install (dependência de dev nova), composer analyse, composer test -- --filter HealthTest
  • curl "localhost:8000/index.php?url=health"

🤖 Generated with Claude Code

sousa7tz and others added 8 commits October 6, 2026 17:11
?url=health responde ok/erro em json antes do session_start. O healthcheck
do Docker e o monitor externo ficam para a parte de infraestrutura.

Task 56 (parte de codigo).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A baseline guarda so os "might not be defined" das views (variaveis do
extract). CI, composer audit no workflow e Dependabot ficam pendentes.

Task 51 (parte de codigo).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…padrao

MAIL_DRIVER vazio nao manda nada e avisa no log (sem o conteudo); "log"
grava o e-mail em MAIL_LOG_FILE para dev e testes. Provedor, SPF e DKIM
ficam para a escolha do time.

Task 55 (parte de codigo).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Token aleatorio de uso unico, valido por 1 hora e guardado so como hash
(migration 008). O pedido responde igual exista o e-mail ou nao e tem
limite por ip e por e-mail; trocar a senha derruba as sessoes abertas.

Task 21.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Cadastro manda um link (24 h, uso unico, so o hash no banco); enquanto nao
confirma, o lojista entra mas so ve a tela de confirmar, com reenvio.
Contas existentes contam como confirmadas (migration 009). Os links por
e-mail passam a herdar de MerchantTokenModel.

Task 50.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
handleForgot/handleReset tinham retorno never e o render ainda fazia
return depois: codigo novo nao entra na baseline.

Task 21.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Tabela admins (migration 010), conta criada por bin/create-admin.php,
sessao separada da do lojista. O admin ve as lojas (sem dados dos
clientes) e ativa ou desativa um lojista, que perde o acesso na hora.

Task 30.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
ADR 003 em status "proposta": tabela users com papel dono/atendente,
permissoes por rota, autor de cada movimentacao no points_log e
perguntas em aberto para o time. Sem codigo.

Task 58.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
feat(admin): painel administrativo e adr de atendentes (tasks 30, 58)
feat(conta): e-mail transacional, esqueci a senha e confirmacao do e-mail (tasks 55, 21, 50)

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

LGTM

@andynnnfw
andynnnfw merged commit 8d7283b into feat/csp-relatorios Oct 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants