# F3.I4.T1 — Core: Lang/{locale} por convenção + locale pela URL

| Campo | Valor |
|---|---|
| **Fase** | F3 — Layout & UX |
| **Issue** | F3.I4 — i18n no módulo (ADR-012) |
| **kd task** | `task_01xmrkgr` (`closed`) |
| **Âncora** | `wiki/implementation/03-layout-ux.md` |
| **Status da revisão** | 🔧 com correções |

## O que foi implementado

- `ModuleDiscoveryServiceProvider` carrega `Lang/{locale}/*.php` (namespace = token) para módulos e services.
- `ResolveLocaleFromUrl` (locale **pela URL**, sem sessão/cookie) + rotas do portal localizadas (`Locales`, `URL::defaults`, `hreflang`/`x-default`, `LocalizedDate` com `intl`).

## Arquivos para revisar

- `src/core/Internal/ModuleDiscovery/ModuleDiscoveryServiceProvider.php`
- `src/core/Localization/Locales.php`
- `src/core/Localization/LocalizedDate.php`
- `src/core/Internal/Routing/AreaRegistrar.php`
- `src/core/Internal/Http/Middleware/ResolveLocaleFromUrl.php`
- `src/core/Internal/ServiceDiscovery/*` (Lang de services)

## Pontos de revisão

### PHP 8.4
- [ ] `declare(strict_types=1)`; tipos nativos (union/enum/`readonly`); sem `mixed` desnecessário.
- [ ] Dependências **injetadas** (nenhum `new`/`app()`/`resolve()` no corpo); sem traits/abstract/generics.
- [ ] `match` em vez de cadeias de `if`; `?->`/`??`; sem código morto; PHPStan `max` limpo.

### Laravel 12
- [ ] Nativo primeiro (Eloquent casts/scopes, `FormRequest`, `Gate`, scheduler, events) — sem reimplementar.
- [ ] Sem closure em rota; resources/rotas nomeadas; `config()` só lê, `env()` só em `config/`.
- [ ] Migração **idempotente**; `down()` → `backup_*` (nunca `DROP`); sem `orderBy` cru quando `latest/oldest` serve.

### HTML semântico + a11y (eMAG/WCAG 2.2 AA)
- [ ] Um `<main>`, headings em nível correto, landmarks; só ARIA quando necessário.
- [ ] Alvos ≥ 24 px (≥ 44 px em ponteiro grosso); foco visível; rótulos em todos os campos; contraste (axe).
- [ ] Funciona **sem JS**; `sniffCSS` sem `fail`/`warn` fora do ledger.

### CSS moderno
- [ ] `oklch`, `clamp`, `rem`/`fr`/`dvh`/`%`; `@layer` (a11y por último); `@property`; subgrid com fallback.
- [ ] Sem `!important`, sem `style="…"` inline, sem hex/rgb (exceto justificado); sem CDN.

### Decisões do projeto (ADRs)
- [ ] Respeita a ADR do escopo (ver cabeçalho) e não escapa das demais (001, 002, 004, 013, 015…).
- [ ] Determinismo: relógio/id injetados, ordem explícita, dados idempotentes (ADR-001).

### i18n (ADR-012)
- [ ] Nenhuma string visível fixa; 4 locales com paridade de chaves; `route()`/`localized_url()` (nunca path literal).

### Observabilidade/segurança/testes
- [ ] Span + `log_event('{module}.{action}')` com atributos `scalar|null`, **sem PII** (ADR-024).
- [ ] Teste que falha sem a mudança (vermelho→verde) + evidência no `kd`; `make quality` 2×.

### Específicos desta task

- [ ] Grupo do portal registrado **duas vezes** (sem prefixo + `{locale?}`); `URL::defaults` mantém `route()`.
- [ ] Middleware remove o parâmetro `locale` (params posicionais).

## Comandos úteis

```bash
make quality            # lint + PHPStan max + gates + testes
make e2e                # Playwright + axe (se a task toca UI)
make check-boundaries   # fronteiras core/services/layout/modules
```

---

> Revisar contra `AGENTS.md`, `wiki/adr/` e o plano citado. Registrar achados no `kd` (`kd write --anchor`) e, se houver correção, teste de regressão.

## Notas da revisão

| Data | Revisor | Resultado | Observações |
|---|---|---|---|
| 2026-10-18 | agente | 🔧 | **A** (**bloqueador**) a view pública do Demo chamava `LocalizedDate::date()` — método **inexistente** — e `/demo/milestones/{id}` respondia **HTTP 500**; o PHPStan não analisa `*.blade.php` e o módulo **não tinha nenhum teste** → corrigido para `long()` (provado 200 em pt_BR/en) + **novo gate** `tests/Feature/BladeStaticCallsTest.php` (`FQCN::metodo()` em Blade resolvido por reflexão; vermelho→verde) + regressão `src/modules/Desenvolvimento/Demo/Test/PublicMilestoneTest.php` — `fact_01b8ggqk`/`fact_00q5rccz` (módulo sem teste: task `task_00nffbim`); **B** (médio) `Locales::normalize()` era **código morto** (zero call sites) e o ignore `shipmonk.deadMethod` o encobria (contra ADR-040 e ADR-012 §5: locale nunca vem de `Accept-Language`/cookie) → removido (`fact_00ieghyi`); **C** (médio) **ADR-004/spec 09 pedem token kebab**, mas o código usa `strtolower` (`dadosabertos`) e o namespace i18n usa `Str::snake` (`dados_abertos`) → três formas, a “busca única” falha → task `task_00h25ff0` (`fact_01n2sr6j`); **D** (obs) `Locales`/`LocalizedDate` são utilitários estáticos que usam `app()->getLocale()` (service locator) — justificado por serem chamados de Blade (`risk_00avmp3u`). Demais pontos da ficha (registro duplo do portal, `URL::defaults`, `forgetParameter`) verificados e cobertos por `I18nTest`. |

> Resultado: `✅ revisado` · `🔧 com correções` · `⏭️ delegado` (com justificativa).