refactor(repositories): honest reads in the eight files the ruler could not see (#1087 passo 4)

[rpgmem]

Passo 4 da #1087: os 30 erros que a régua corrigida (#1090) passou a acusar, e a folga do gate de volta a 0.

Nenhum dos 30 era novo. A varredura exigia a substring literal $wpdb-> e nunca olhou os repositórios que ligam wpdb como propriedade — oito arquivos, o AbstractRepository entre eles.

O que os 30 eram de verdade

Lendo-os, não eram majoritariamente "shape de coluna faltando". Eram quatro origens, e a dominante não tem nada a ver com o $wpdb:

Origem	Sites	O que é
Cache de objetos	6 métodos, 5 arquivos	wp_cache_get() devolve mixed, então um cache hit jogava o tipo fora enquanto a leitura do banco logo abaixo estava tipada
Projeções agregadas	3	SELECT user_id, COUNT(*) não é linha; e COUNT()/SUM() voltam como string numérica, ou NULL em tabela vazia
json_decode sobre coluna	3	colunas JSON escritas pelo próprio plugin, lidas de um array<string, mixed>
Projeção de 5 colunas	6	o SELECT id, user_id, email_hash, cpf_hash, rf_hash do writer, agora um @phpstan-type

O caso do cache é o achado que importa. É literalmente o do #1072 — "a shape estava certa; o furo era o wp_cache_get()" — e ele nunca chegou a estes arquivos porque a régua não os enxergava. A cegueira do passo 3 não escondia só uma contagem: escondeu a aplicação de uma correção que o épico já havia feito em outro módulo. Seis métodos ficaram com um defeito conhecido e documentado porque o gate que deveria acusá-los nunca os olhou.

A asserção de cache vai sempre na chave, nunca no get_cache() do trait — o cache é heterogêneo por chave, e só o método sabe o que guardou sob a dele. Foi assim que o #1072 resolveu, e a razão continua valendo.

Duas decisões

AbstractRepository não ganha shape. É base de várias tabelas, então não existe uma forma só. Ali o honesto é declarar o que ele de fato sabe — que o $wpdb devolve chaves string — e ler o id pelo ArrayValue em vez de castar mixed. Inventar um @phpstan-type genérico ali seria a shape desonesta que o #1060 alerta.

Não declarei um SubmissionRow completo. A consolidação da #1091 agora permitiria, porque as 25 colunas viraram obrigatórias — mas isso é melhoria separada, não é o que os 30 pedem, e o CLAUDE.md manda manter cada correção mínima. Fica registrado como possível, não feito.

Três rodadas de medição, não uma

30 → 10 → 1 → 0. E o que sobrou nas duas primeiras foi sempre aquilo que eu já afirmava ter tratado: o furo de cache do url-shortener (rodada 2) e o ramo de cache hit do findByIds(), um método ao lado do findById() que eu havia anotado no mesmo arquivo (rodada 3). Só a medição disse isso — a leitura não disse.

A folga volta a 0, por encolhimento

Que era a única forma que o comentário do ci.yml admitia quando ela subiu. O comentário foi reescrito para registrar por que subiu (denominador errado, não dívida nova), por quanto tempo ficou em 30, e que uma próxima subida precisa passar pela mesma barra.

Row-reading classes: 53
Level-9 errors in them: 0 (allowed: 0)
None. Every class that reads a row declares what the row holds.
PASS

Essa frase agora é verdadeira. Antes do passo 3 ela era impressa com 30 erros de pé.

Validação

• Régua: 53 classes, 0 erros, folga 0 — PASS
• PHPStan nível 8: limpo no repositório inteiro, sem supressão nova
• WPCS e phpcs-tests.xml.dist: limpos — o @param do hasConflictInForm() foi partido em @param array<string, mixed> mais @phpstan-param IdentifierProjection, porque o WPCS exige que o type hint nativo case com o docblock e um alias do PHPStan não existe para ele. É o idioma que outros 45 arquivos do repositório já usam.
• Suíte completa: 7657 testes, 21965 asserções, verde (11m25s)
• Nível 9 do repositório: 2535 → 2505

Refs #1087, #1072, #1090, #1091, #1060

🤖 Generated with Claude Code

https://claude.ai/code/session_012XWx9qJdjZdAq8crxM9GCU

---

Generated by Claude Code

[coveralls]

Coverage Report for CI Build 34071209357

Warning

No base build found for commit 0ad8ef1 on develop.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 89.967%

Details

• Patch coverage: 16 of 16 lines across 6 files are fully covered (100%).

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →

---

Coverage Stats

Relevant Lines:	53944
Covered Lines:	48532
Line Coverage:	89.97%
Coverage Strength:	4.87 hits per line

---

💛 - Coveralls