# F6.I2.T3 — Retenção de backups (local + fora do host) e ensaio de DR

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

## O que foi implementado

- Local: `BACKUP_KEEP` no `up_prod.sh` (ordem por nome UTC, mantém os N mais novos).
- Remoto: `core:backup:prune --keep=N [--dry-run]` (nunca apaga o último; só mexe em `backups/db-*.dump`),
  executado no deploy após o push (falha na poda **não** derruba o deploy — o backup local já está validado).
- DR documentado no runbook §3: dump remoto + PITR + verificação por `go-live-check`.

## Arquivos para revisar

- `src/core/Internal/System/Storage/Console/PruneDatabaseBackupsCommand.php`
- `up_prod.sh`
- `tests/Feature/DatabaseBackupRetentionTest.php`
- `wiki/runbooks/deploy-e-recuperacao.md`

## 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

- [ ] Ordem determinística pelo **nome** (não por mtime do objeto).
- [ ] `--keep < 1` recusado (nunca zera a retenção).
- [ ] `--dry-run` não apaga; arquivos que não são dump ficam intocados.
- [ ] Retenção alinhada à LGPD (ADR-019): prazo definido, sem PII em nome de arquivo além do instante.

---

> 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) | 🔧 | **D** (médio, D12) `core:backup:prune --dry-run` registrava `core.backup_prune` **sem ter podado nada** (entrada falsa na trilha técnica) e usava `id` = contagem → dry-run deixou de logar, `id` passou a ser o disco, e remoção que falha devolve `FAILURE` (`fact_000raeon`). Verificado OK: ordem por nome UTC (nunca mtime), `--keep < 1` recusado, só `backups/db-*.dump` tocados, deploy não morre por falha de poda. Testes: `DatabaseBackupRetentionTest` (6, com `Log::spy`), vermelho antes. |