# F6.I2.T1 — Backup do banco **antes** do `migrate`

| Campo | Valor |
|---|---|
| **Fase** | F6 — Deploy e operação |
| **Issue** | F6.I2 — Base nova: backup, restore e PITR |
| **kd task** | `task_01e0cyc2` |
| **Âncora** | `wiki/implementation/06-deploy-migracao.md` |
| **Status da revisão** | 🔧 |

## O que foi implementado

- Fase **5.9** do `up_prod.sh`: `pg_dump -Fc --no-owner --no-acl` → `$STORAGE_PATH/backups/db-<UTC>.dump`,
  validado com `pg_restore --list`; dump vazio/ilegível **aborta** o deploy.
- Retenção local determinística por nome (`BACKUP_KEEP`, default 7) e cópia fora do host
  (`BACKUP_OFFSITE=1` → `core:backup:push`, que confere o tamanho no destino).
- Flag `--backup-only` para operação avulsa. Provado ao vivo: 327 825 bytes, poda 5→2, upload conferido.

## Arquivos para revisar

- `up_prod.sh` (funções `backup_db`/`prune_backups`)
- `src/core/Internal/System/Storage/Console/PushDatabaseBackupCommand.php`
- `.env.example` (`BACKUP_KEEP`/`BACKUP_OFFSITE`) · `.gitignore` (`/storage/backups/`)
- `tests/Feature/{DatabaseBackupPushTest,DeployPipelineTest}.php`

## Pontos de revisão

### Operação e determinismo (ADR-001)
- [ ] Idempotência real (2× sem mudar estado); nada depende de relógio local, ordem implícita ou `date` não-UTC em nome de artefato.
- [ ] Falha em qualquer fase **aborta** e o rollback devolve o estado anterior; nada falha "em silêncio".

### Segurança (ADR-019/020/030/035)
- [ ] Nenhum segredo/senha/e-mail em log, stdout de artefato ou arquivo versionado.
- [ ] Dump/backup contém dado pessoal → fora do git, retenção definida, acesso restrito.
- [ ] Cópia remota confere integridade (tamanho/checksum) e não engole erro.

### Banco (ADR-007/042)
- [ ] Nenhum caminho de ETL/legado (`sys_migration`/`legacy_mapping`/`migrate:legacy`).
- [ ] Config do PostgreSQL é a mesma dev/prod; nada de `$isPgsql`/driver branching.
- [ ] `pg_dump`/`pg_restore`/`pg_basebackup` na **mesma** versão do servidor.

### Scripts de shell (D1/D13)
- [ ] `set -Eeuo pipefail`; `sc()` de erro explícito; mensagens acionáveis; nada de `rm -rf`.
- [ ] `shellcheck`-limpo no que for razoável; helpers sem duplicação desnecessária.
- [ ] Caminhos derivados do `.env`/compose (sem porta/host hardcoded).

### Comandos de console (D2/D8/D12)
- [ ] `declare(strict_types=1)`; DI por construtor/injeção (sem `new`/`app()` no corpo).
- [ ] Validação de entrada no comando; `FAILURE` com mensagem clara (sem stack trace crua).
- [ ] `log_event('{module}.{action}')` **sem PII**; auditoria automática quando há mutação Eloquent.
- [ ] Textos de saída em pt_BR (CLI), sem string fixa em view.

### Testes (D13/ADR-038)
- [ ] Teste cobre o caminho feliz **e** as recusas; prova vermelho→verde registrada no `kd`.
- [ ] Estrutural: gate impede a regressão do invariante que a task criou.

### Documentação (D14)
- [ ] Runbook/spec/plano descrevem o que o código faz (comandos, flags, env) — sem promessa não implementada.
- [ ] `docs-check` verde; DoD do plano 06 reflete o estado real.

### Específicos desta task

- [ ] A ordem é **backup → migrate** (gate estrutural em `DeployPipelineTest`).
- [ ] Falha do dump aborta (**não** "segue sem backup").
- [ ] Dump validado (TOC legível), não só "arquivo existe".
- [ ] `storage/backups/` fora do git (LGPD) e retenção não apaga o último.
- [ ] Cópia remota confere tamanho e remove cópia truncada.

---

> Revisar contra `AGENTS.md`, `wiki/adr/` e `wiki/implementation/06-deploy-migracao.md`. Achado real →
> `kd write --anchor` + teste de regressão. **Não** reescrever por gosto.

## Notas da revisão

| Data | Revisor | Resultado | Observações |
|---|---|---|---|
| 2026-10-20 | agente (revisão F6) | 🔧 | **E** (médio, D11/ADR-019) `core:backup:push` aceitava **qualquer** arquivo sob `storage/` (`logs/laravel.log`, payloads) e enviava ao S3, cuja retenção é longa → agora só `backups/*.dump`, com teste (`fact_000raeon`). Verificado OK: ordem backup→migrate, dump validado com `pg_restore --list`, vazio/ilegível aborta, `backups/` no `.gitignore`, retenção não apaga o último, cópia truncada é removida. |