fix: corrigir tipo de retorno de DiscussionPolicy::uploadHeroImage

[ram0ng1]

O método declarava : ?bool mas $this->allow() retorna a string 'allow' (a constante AbstractPolicy::ALLOW). Resultado: TypeError ao fazer upload de hero image em uma discussão.

Remover a anotação — métodos de policy retornam string|null por contrato do Flarum.

[github-actions]

🔬 Performance benchmark

Home (/)

Métrica	PR	Master (baseline)	Δ
Performance (score)	93	89	🟢 4pp (4.5%)
FCP	681 ms	681 ms	🟢 0 ms (-0.1%)
LCP	1739 ms	1850 ms	🟢 -111 ms (-6.0%)
TBT	0 ms	124 ms	🟢 -124 ms (-100.0%)
CLS	0.000	0.000	(=)
Speed Index	848 ms	1137 ms	🟢 -289 ms (-25.4%)
TTI	1739 ms	1890 ms	🟢 -151 ms (-8.0%)

💡 Recomendações (Lighthouse + dicas Flarum 2 / Avocado)

• Sem compressão de texto — economia ~920 ms · ~1097 KB
  Habilite gzip/brotli no servidor (nginx: gzip on; gzip_types text/css application/javascript; ou brotli on; brotli_types ...). Esse é um setting de host, não da extensão, mas reportar aqui ajuda.
• CSS não utilizado — economia ~480 ms · ~538 KB
  O bundle forum.css carrega tudo de less/forum/**. Divida por rota (Home/Discussion/Tag) via Extend\Frontend->css() condicional no JS, ou use PurgeCSS no webpack para o build de produção.
• JavaScript não utilizado — economia ~460 ms · ~515 KB
  Use import() dinâmico para componentes só usados em rotas específicas (TeamPage, AvocadoSearchPage). O webpack faz split automático se você usar import("./components/TeamPage") dentro do routes ao invés de import estático no topo do index.tsx.
• Cache HTTP curto — 8 resources found
  Os assets versionados em /assets/forum-<hash>.js deveriam ter Cache-Control: public, max-age=31536000, immutable. Configure no nginx/apache, não no Flarum.

/discussions

Métrica	PR	Master (baseline)	Δ
Performance (score)	92	92	(=)
FCP	685 ms	687 ms	🟢 -2 ms (-0.3%)
LCP	1780 ms	1795 ms	🟢 -15 ms (-0.8%)
TBT	0 ms	2 ms	🟢 -2 ms (-100.0%)
CLS	0.000	0.000	(=)
Speed Index	830 ms	870 ms	🟢 -40 ms (-4.6%)
TTI	1780 ms	1795 ms	🟢 -15 ms (-0.8%)

💡 Recomendações (Lighthouse + dicas Flarum 2 / Avocado)

• Sem compressão de texto — economia ~920 ms · ~1103 KB
  Habilite gzip/brotli no servidor (nginx: gzip on; gzip_types text/css application/javascript; ou brotli on; brotli_types ...). Esse é um setting de host, não da extensão, mas reportar aqui ajuda.
• JavaScript não utilizado — economia ~540 ms · ~537 KB
  Use import() dinâmico para componentes só usados em rotas específicas (TeamPage, AvocadoSearchPage). O webpack faz split automático se você usar import("./components/TeamPage") dentro do routes ao invés de import estático no topo do index.tsx.
• CSS não utilizado — economia ~480 ms · ~544 KB
  O bundle forum.css carrega tudo de less/forum/**. Divida por rota (Home/Discussion/Tag) via Extend\Frontend->css() condicional no JS, ou use PurgeCSS no webpack para o build de produção.
• Cache HTTP curto — 9 resources found
  Os assets versionados em /assets/forum-<hash>.js deveriam ter Cache-Control: public, max-age=31536000, immutable. Configure no nginx/apache, não no Flarum.

/tags

Métrica	PR	Master (baseline)	Δ
Performance (score)	93	93	(=)
FCP	686 ms	682 ms	🔴 +4 ms (0.6%)
LCP	1741 ms	1744 ms	🟢 -3 ms (-0.2%)
TBT	0 ms	0 ms	(=)
CLS	0.000	0.000	(=)
Speed Index	776 ms	787 ms	🟢 -10 ms (-1.3%)
TTI	1741 ms	1744 ms	🟢 -3 ms (-0.2%)

💡 Recomendações (Lighthouse + dicas Flarum 2 / Avocado)

• Sem compressão de texto — economia ~920 ms · ~1098 KB
  Habilite gzip/brotli no servidor (nginx: gzip on; gzip_types text/css application/javascript; ou brotli on; brotli_types ...). Esse é um setting de host, não da extensão, mas reportar aqui ajuda.
• JavaScript não utilizado — economia ~530 ms · ~540 KB
  Use import() dinâmico para componentes só usados em rotas específicas (TeamPage, AvocadoSearchPage). O webpack faz split automático se você usar import("./components/TeamPage") dentro do routes ao invés de import estático no topo do index.tsx.
• CSS não utilizado — economia ~520 ms · ~543 KB
  O bundle forum.css carrega tudo de less/forum/**. Divida por rota (Home/Discussion/Tag) via Extend\Frontend->css() condicional no JS, ou use PurgeCSS no webpack para o build de produção.
• Cache HTTP curto — 8 resources found
  Os assets versionados em /assets/forum-<hash>.js deveriam ter Cache-Control: public, max-age=31536000, immutable. Configure no nginx/apache, não no Flarum.

---

🥑 Dicas gerais para acelerar o tema (Flarum 2 + Avocado)

1. Build de produção minificado — confirme que npm run build rodou com mode: production (já é o caso no js/package.json). Webpack tree-shakes import { x } from "flarum/..." se o consumo for explícito.
2. Split por rota — index.tsx importa todos os Components no topo. Trocar para import() dinâmico nas rotas raramente acessadas (TeamPage, AvocadoSearchPage, TagsPage) reduz o forum.js inicial.
3. Less crítico inline — Flarum 2 serve forum.css bloqueando. Considere extrair o CSS above-the-fold (header + 1ª thread visível) e injetar inline via Extend\Frontend->content(InlineCriticalCss::class).
4. Avatares — sirva via <img loading="lazy"> em todos os ThreadCards/PostCards que não estiverem no fold inicial.
5. <link rel="preconnect"> — se você usa CDN para fontes ou avatares S3, adicione preconnects no header.
6. Extend\Frontend->js() rodam síncronos — todo arquivo js/dist/forum.js é parseado no boot. Cada extend() no index.tsx roda antes da primeira pintura.
7. opcache + view cache — no host: opcache.enable=1, opcache.validate_timestamps=0 em produção, e php flarum cache:clear no deploy.
8. HTTP/2 + Brotli no host — o Flarum gera bundles grandes; sem brotli você paga em transferência.

_{Lighthouse desktop, 1 run por URL. 🟢 = melhorou vs master · 🔴 = regrediu.}