# F6.I3.T1 — 301 curado à mão (`seo:redirect`)

| Campo | Valor |
|---|---|
| **Fase** | F6 — Deploy e operação |
| **Issue** | F6.I3 — Corte e conteúdo do cliente (manual) |
| **kd task** | `task_00yjz1b0` |
| **Âncora** | `wiki/implementation/06-deploy-migracao.md` |
| **Status da revisão** | 🔧 |

## O que foi implementado

- `seo:redirect` (console): lista por padrão; `--from/--to/--status/--disable/--remove`; valida origem
  (`/`, sem `//`/`..`) e destino (caminho ou `http(s)`); aceita 301/302/307/308; `updateOrCreate` por
  `from_path`; invalida `Cache::forget(SeoRedirect::CACHE_KEY)`; emite `seo.redirect`.
- A rota viva **nunca** é sombreada (`SeoRedirectTest` existente). UI de CMS fica no backlog.

## Arquivos para revisar

- `src/services/Seo/Console/RedirectCommand.php`
- `src/services/Seo/ServiceProvider.php` (registro só em console)
- `src/services/Seo/Test/{SeoRedirectCommandTest,SeoRedirectTest}.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

- [ ] Toda escrita invalida o cache do mapa (senão o 301 velho fica até 300 s).
- [ ] Destino externo/`//`/travessia recusado (open redirect).
- [ ] `--from` inválido **não** cai no "listar por engano" (erro explícito).
- [ ] Rota viva continua vencendo o redirect.

---

> 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) | 🔧 | **A** (alto, D12/ADR-035) `--disable`/`--remove` usavam **query builder** (`update()`/`delete()`), que não emite evento do Eloquent: a desativação e a remoção **não entravam na trilha de auditoria**, e o `Cache::forget` manual mascarava a ausência do hook do model → passou a operar por model (cache pelo `booted()`), com teste que exige `created`/`updated`/`deleted` em `core.core_audit_logs` (`error_006uxfoa`). **H** (médio, D11) aceitava laço (`/a → /a` ou `/a→/b` + `/b→/a`) → navegador em loop de 301: caminhada limitada a 10 saltos recusa (`error_006uxfoa`). **I** (baixo, obs) `--from=/a/` não casa com `/a` no disable/remove (`risk_01urn5zn`). Verificado OK: origem só interna, destino sem `//`/`..`, status na allowlist, rota viva vence, cache invalidado. Testes: `SeoRedirectCommandTest` (2 novos, vermelhos antes). |