Skip to content

feat(seguranca): acessibilidade, csp completa e relatorios por periodo com csv (tasks 18, 48, 59) - #36

Merged
andynnnfw merged 14 commits into
feat/balcao-cadastrofrom
feat/csp-relatorios
Oct 9, 2026
Merged

andynnnfw merged 14 commits into
feat/balcao-cadastrofrom
feat/csp-relatorios

Conversation

@sousa7tz

@sousa7tz sousa7tz commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

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

Tasks

  • 18: nenhum style="" nas views. src/ts/forms.ts:

    • o botão mostra "carregando" e trava o clique duplo;
    • o campo apontado pelo erro fica com aria-invalid, ligado à mensagem e com foco (mapa $flashFields no flash.php).

    O foco do teclado fica sempre visível.

  • 48: CSP default-src 'self'; script-src 'self' (Google Fonts liberado) e HSTS só em HTTPS.

    • Os scripts inline foram para src/ts/nav.ts e src/ts/customer.ts.
    • onsubmit=confirm virou data-confirm.
    • Um teste varre as views atrás de <script> inline e on*=.
  • 59: relatórios por período (App\Support\Period: hoje, 7 dias, mês, intervalo); o saldo em circulação continua o atual. merchant/export baixa CSV (; + BOM, protegido contra injeção de fórmula, só desta loja).

Pontos de atenção

  • O JS todo mudou e só tem teste em node com DOM falso. Vale um passeio no navegador com o console aberto:
    • menu no celular;
    • Desfazer com contagem;
    • prévia do valor e +1/+5/+10;
    • confirmação ao excluir prêmio, ao remover a regra e ao estornar;
    • botão de imprimir do cartaz.
  • 59: "Este mês" vai do dia 1 até hoje. O estorno conta na data dele. "Clientes" virou "Clientes novos": cartões criados no período, sem os anonimizados.
  • O CI só roda masks.test.js. Os testes novos forms.test.js e customer.test.js ficaram de fora porque isso exige mexer no .github/.

Como testar

  • composer test -- --filter "AccessibilityTest|SecurityHeadersTest|PeriodTest|CsvTest|ReportsPeriodTest"
  • node tests/js/forms.test.js && node tests/js/customer.test.js

🤖 Generated with Claude Code

sousa7tz and others added 11 commits October 6, 2026 16:51
… erro marcado

- style="" das views vira classe (a barra de progresso usa passos de 5%)
- src/ts/forms.ts: botao mostra carregando e o formulario nao e enviado
  duas vezes; o campo apontado pelo erro fica marcado, com aria-invalid e foco
- foco do teclado sempre visivel

Task 18.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- scripts inline viram src/ts/nav.ts e src/ts/customer.ts (compilados em public/js)
- onsubmit/onclick viram data-confirm (forms.js) e data-print (nav.js)
- RequestGuard manda CSP com default-src 'self' e, em https, Strict-Transport-Security

Task 48.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- indicadores e historico por periodo (hoje, 7 dias, mes, intervalo); o saldo
  em circulacao continua o atual
- merchant/export baixa o historico do periodo ou a lista de clientes em csv
  (; + BOM, protegido contra injecao de formula), so desta loja

Task 59.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
?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)
chore(qualidade): rota de saude e phpstan nivel 5 (tasks 56, 51)
@andynnnfw
andynnnfw merged commit 5468e97 into feat/balcao-cadastro 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