# F6.I3.T2 — Usuários recriados no CMS (`core:user:create`)

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

## O que foi implementado

- `core:user:create`: Argon2id (cast `hashed`), `--name`, `--role` por slug, `--password`, `--force`
  (atualiza e reativa), senha gerada quando não interativo, perfil `suporte` **só** para
  `suporte@maxima.inf.br` (ADR-033), sem perfil = deny-by-default.
- `log_event('core.user_create')` só com o **id** (sem e-mail/senha — ADR-024); mutação Eloquent entra
  na trilha automaticamente (ADR-035).

## Arquivos para revisar

- `src/core/Area/Cms/Console/CreateUserCommand.php`
- `src/core/CoreServiceProvider.php` (registro)
- `tests/Feature/CmsUserCommandTest.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

- [ ] Nenhuma PII no `log_event` (nem e-mail, nem nome, nem senha).
- [ ] Senha nunca ecoada em log; gerada só quando não há terminal.
- [ ] Perfil reservado recusado fora da identidade de suporte.
- [ ] Sem perfil = sem acesso (deny-by-default), com aviso claro.

---

> 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) | 🔧 | **C** (alto, D11/ADR-020) `--password` **não validava tamanho** (só o prompt exigia 12) → senha fraca entrava pela flag; agora há um `MIN_PASSWORD` único para os dois caminhos, com teste (`fact_000raeon`). **I** (baixo, D12) na atualização o evento era `core.user_create` com `action=user_update` (dois nomes para o mesmo evento) → passou a `core.{action}`, com teste. **J** (obs) `--password=` fica no histórico do shell e o comando não case-folda e-mail já existente criado fora dele (`risk_01urn5zn`). Verificado OK: Argon2id, um perfil, `suporte` só para a identidade de suporte, `--force` reativa soft delete, sem perfil = deny-by-default, log sem PII. Testes: `CmsUserCommandTest` (8, 2 novos vermelhos antes). |