* perf: cortar 157 KiB do caminho crítico e medir o LCP de forma reprodutível (MAN-109) A auditoria do PR #34 mediu LCP acima da meta de 2,5 s da SPEC §6.6 em todas as páginas no mobile, mas ficou registrada apenas como tabela num comentário do Linear — `storage/app/lighthouse` é gitignored, então não havia artefato para comparar depois. Esta entrega mede de novo, encontra a causa dominante e corta o que dava para cortar. ## Fontes servidas em dobro (88 KiB) Bunny entrega cada peso de EB Garamond em woff2 e woff, e o plugin de fontes emitia uma regra `@font-face` para cada, woff2 primeiro e woff depois. Duas regras com a mesma família, peso, estilo e unicode-range fazem a última vencer: o navegador renderizava a partir dos woff e descartava os woff2 pré-carregados. O log de rede da home prova: 3 woff em prioridade VeryHigh (88 KiB), a mais alta da página e à frente do elemento de LCP, mais 3 woff2 em High (74 KiB) baixados só por causa do `<link rel="preload">`. 162 KiB de tráfego para 74 KiB de fonte útil. woff2 é suportado por todo navegador que este site atende desde 2016, então as regras woff eram peso morto, não fallback. O plugin `amare:fonts-woff2-only` remove as regras do CSS e do manifest e tira os arquivos do bundle. É o que derruba o FCP de 1,51 s para 0,91 s em todas as páginas. ## Ativos de marca reencodados (102 KiB) — absorve MAN-122 O logotipo era servido a 512 px de largura para renderizar em 48 px (lockup, cabeçalho e rodapé de toda página) e 32 px (mark, home), com `loading="eager"`. Reencodados a 3× do maior render: lockup 149×144 (84 → 14 KiB) e mark 191×96 (42 → 10 KiB), mesmos nomes de arquivo para não invalidar cache. As variantes `on-dark` foram reencodadas junto por consistência; nenhuma view as usa hoje. Os 16 baselines visuais foram regenerados no runner Linux e o diff é imperceptível a 2× de zoom — mesma forma, mesma cor, menos bytes. ## Efeito medido Mobile, mediana de 3 execuções por página: home 4,58 → 3,39 s; portfolio 3,98 → 1,58 s; portfolio-detalhe 3,98 → 3,01 s; servicos 3,68 → 2,63 s; sobre 3,01 → 1,58 s; contato 2,55 → 1,51 s. Desktop passa com folga em todas (máximo 0,79 s). Acessibilidade, boas práticas e SEO seguem 100, CLS 0,000 e TBT 0 ms. `contato`, `portfolio` e `sobre` entraram na meta. `servicos`, `portfolio-detalhe` e `home` continuam fora, e o que falta está identificado: o elemento de LCP da home é a imagem do hero em JPEG q82 (143 KiB), e `Improve image delivery` estima 0,90 s de ganho restante. O lever é variante WebP em ResponsiveImage com `<picture>` — fora deste escopo porque mexe no pipeline de mídia que o CMS usa para upload. ## Ferramental - `scripts/perf/lighthouse.sh` ganha `/portfolio/{slug}`, passada desktop, mediana de 3 execuções e uma guarda de HTTP 200 antes de auditar. Uma rota em 404 produz relatório com score alto: a rota mais pesada do site apareceria como ótima e ninguém notaria. - `scripts/perf/summarize-lighthouse.mjs` extrai do JSON o elemento de LCP, a decomposição em fases e as requests até o LCP. - `docker/ci-runner.Dockerfile` e `scripts/test/visual-update-ci.sh` versionam a receita de regeneração de baselines, que existia só como checklist. - `CLAUDE.md` corrigido: Pest Browser serve a aplicação de um servidor Amp in-process, não do FrankenPHP. A paridade que importa nos baselines é Linux vs macOS, não o runtime HTTP. - Sem gate de performance no CI (§14.1 e §22). No lugar, orçamento de bytes para os ativos de marca e asserção de que o build só emite woff2. Staging não foi medido: a origem responde 303 para blocked.teams.cloudflare.com a partir da rede corporativa, inclusive em `/up`. Duas hipóteses seguem não validadas por dependerem de `FILESYSTEM_DISK=r2` — os ~4 round trips a R2 por imagem que `x-media.image` faz sem cache, e a falta de `preconnect` para a origem cross-origin de mídia. Ambas documentadas em docs/evidence/lighthouse/README.md. Co-Authored-By: Claude noreply@anthropic.com AI-Assisted: yes AI-Tool: claude-code * perf: servir imagens de conteúdo em webp e descrever sizes corretamente (MAN-109) Fecha a meta de LCP ≤ 2,5 s da SPEC §6.6 em todas as páginas, nos dois presets. Continuação direta do commit anterior: com fontes e marca resolvidas, o elemento de LCP de toda página no mobile passou a ser a imagem do hero, e o Load Delay de 1,88 s era contenção de banda pura. ## Variantes webp `ResponsiveImage::generate()` escreve uma variante `.webp` ao lado de cada variante no formato original, e `x-media.image` a oferece num `<source type="image/webp">`. O `<img>` continua apontando para o formato original: o `<source>` é preferência, não substituição, então nada quebra em quem não decodifica webp, e mídia antiga sem irmãos webp renderiza `<img>` puro como antes. O `<picture>` recebe `display: contents` porque os chamadores estilizam o `<img>` com classes como `h-full w-full object-cover` que resolvem contra o pai grid ou flex — um wrapper inline quebraria isso. O nome do arquivo acrescenta a extensão em vez de trocá-la (`photo-720.jpg.webp`): uploads usam UUID, então colisão já era improvável, mas `photo-720.webp` colidiria com a variante webp de um `photo.png`. ## sizes que descreve a realidade Nenhuma imagem do site ocupa a viewport inteira — todas ficam dentro de `container-amare`, que reserva 1,5rem de padding de cada lado. Declarar `100vw` fazia uma viewport de 412 px em DPR 1,75 pedir 721 px e pular para a variante de 960 para desenhar uma caixa de 637 px. Errar por um pixel custava um terço a mais de bytes em toda página. Com `calc(100vw - 3rem)` a home passa a usar a variante de 720: 74 KiB, contra 143 KiB no início da investigação. Foi também por isso que 720 entrou em `ResponsiveImage::WIDTHS` — sem ela o salto de 480 para 960 é grande demais para a viewport mobile mais comum. ## Efeito medido Mobile, mediana de 3 execuções: home 3,39 → 2,49 s; portfolio-detalhe 3,01 → 2,18 s; servicos 2,63 → 2,03 s; portfolio 1,58 s; sobre 1,58 s; contato 1,50 s. Desktop no máximo 0,65 s. Peso total da home 798 → 347 KiB. Contra o início da investigação (`2e43fde`): home 4,58 → 2,49 s com score de performance 83 → 98. A home passa **em cima da linha** — a pior das três execuções deu 2,57 s. Está documentado como aprovada por margem, não com folga, e os levers restantes estão listados em docs/evidence/lighthouse/README.md em ordem de custo. Os 16 baselines visuais não mudaram: nenhum seeder gera variantes e `media:generate-variants` não roda no runner visual, então naquele ambiente `availableVariants()` volta vazio e o componente renderiza `<img>` puro. A cobertura do caminho com variantes fica em MediaImageComponentTest e ResponsiveImageTest, não nos baselines — anotado como lacuna conhecida. Co-Authored-By: Claude noreply@anthropic.com AI-Assisted: yes AI-Tool: claude-code * docs: registrar a dependência do deploy e corrigir os commits nas evidências Três correções de honestidade nos artefatos de medição. O campo `Commit:` era gravado com o HEAD do momento da coleta, que é sempre anterior ao commit que contém as mudanças medidas — as duas passadas pós-correção diziam `2e43fde` e `65919d4` quando as árvores medidas viraram `65919d4` e `9ab2beb`. Corrigido à mão nos dois arquivos, e o script passa a marcar `+alterações não commitadas` quando a árvore está suja, para o artefato não voltar a afirmar o que não é. O ganho das imagens não aparece em staging nem em produção antes de `media:generate-variants` rodar: lá o disco é `r2` e as variantes webp e a largura 720 ainda não existem para a mídia publicada. O serviço `migrate` do docker-compose.deploy.yml roda o comando sem condição a cada deploy e ele regenera mesmo quando já há variantes, então o primeiro deploy resolve — mas estava implícito e agora está escrito. E o custo que as variantes webp introduzem no lado servidor está registrado com o número certo: `x-media.image` faz 8 `exists()` por imagem no lugar de 3, o que em `r2` são ~9 round trips remotos por imagem e ~36 na home. Isso agrava a hipótese não validada de TTFB em staging em vez de melhorá-la, e promove o cache desses metadados a item mais urgente da lista. Co-Authored-By: Claude noreply@anthropic.com AI-Assisted: yes AI-Tool: claude-code * fix: cache responsive image metadata for R2 * test: atualizar baselines visuais Linux --------- Co-authored-by: manoel.neto <manoel.neto@creditas.com>
191 lines
7.4 KiB
PHP
191 lines
7.4 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace Tests\Unit;
|
|
|
|
use App\Support\ResponsiveImage;
|
|
use Illuminate\Support\Facades\Cache;
|
|
use Illuminate\Support\Facades\Storage;
|
|
use Intervention\Image\Drivers\Gd\Driver;
|
|
use Intervention\Image\ImageManager;
|
|
use RuntimeException;
|
|
use Tests\TestCase;
|
|
|
|
class ResponsiveImageTest extends TestCase
|
|
{
|
|
protected function setUp(): void
|
|
{
|
|
parent::setUp();
|
|
|
|
config(['cache.default' => 'array']);
|
|
Cache::store('array')->flush();
|
|
}
|
|
|
|
public function test_generates_named_variants_and_removes_them_with_original(): void
|
|
{
|
|
Storage::fake('public');
|
|
|
|
$manager = new ImageManager(new Driver);
|
|
$original = $manager->create(2000, 1200)->toJpeg(quality: 90);
|
|
$path = 'content/abc123.jpg';
|
|
|
|
Storage::disk('public')->put($path, (string) $original);
|
|
|
|
ResponsiveImage::generate($path, 'public');
|
|
|
|
Storage::disk('public')->assertExists('content/abc123-480.jpg');
|
|
Storage::disk('public')->assertExists('content/abc123-960.jpg');
|
|
Storage::disk('public')->assertExists('content/abc123-1440.jpg');
|
|
|
|
$variant480 = (new ImageManager(new Driver))->read(Storage::disk('public')->path('content/abc123-480.jpg'));
|
|
$this->assertSame(480, $variant480->width());
|
|
|
|
ResponsiveImage::delete($path, 'public');
|
|
|
|
Storage::disk('public')->assertMissing($path);
|
|
Storage::disk('public')->assertMissing('content/abc123-480.jpg');
|
|
Storage::disk('public')->assertMissing('content/abc123-960.jpg');
|
|
Storage::disk('public')->assertMissing('content/abc123-1440.jpg');
|
|
}
|
|
|
|
public function test_replace_deletes_previous_variants_and_generates_for_new_original(): void
|
|
{
|
|
Storage::fake('public');
|
|
|
|
$manager = new ImageManager(new Driver);
|
|
Storage::disk('public')->put('content/old.jpg', (string) $manager->create(1600, 900)->toJpeg());
|
|
ResponsiveImage::generate('content/old.jpg', 'public');
|
|
|
|
Storage::disk('public')->put('content/new.jpg', (string) $manager->create(1800, 1000)->toJpeg());
|
|
|
|
ResponsiveImage::replace('content/old.jpg', 'content/new.jpg', 'public');
|
|
|
|
Storage::disk('public')->assertMissing('content/old.jpg');
|
|
Storage::disk('public')->assertMissing('content/old-480.jpg');
|
|
Storage::disk('public')->assertExists('content/new.jpg');
|
|
Storage::disk('public')->assertExists('content/new-480.jpg');
|
|
Storage::disk('public')->assertExists('content/new-960.jpg');
|
|
Storage::disk('public')->assertExists('content/new-1440.jpg');
|
|
}
|
|
|
|
public function test_generates_a_webp_sibling_for_every_variant(): void
|
|
{
|
|
Storage::fake('public');
|
|
|
|
$manager = new ImageManager(new Driver);
|
|
$path = 'content/photo.jpg';
|
|
|
|
Storage::disk('public')->put($path, (string) $manager->create(2000, 1200)->toJpeg(quality: 90));
|
|
|
|
ResponsiveImage::generate($path, 'public');
|
|
|
|
foreach (ResponsiveImage::WIDTHS as $width) {
|
|
Storage::disk('public')->assertExists("content/photo-{$width}.jpg.webp");
|
|
}
|
|
|
|
$this->assertSame(
|
|
ResponsiveImage::WIDTHS,
|
|
array_column(ResponsiveImage::availableWebpVariants($path, 'public'), 'width'),
|
|
);
|
|
|
|
// The original format stays available so a browser that cannot decode
|
|
// webp still has something to fall back to through the <img>.
|
|
foreach (ResponsiveImage::WIDTHS as $width) {
|
|
Storage::disk('public')->assertExists("content/photo-{$width}.jpg");
|
|
}
|
|
|
|
ResponsiveImage::delete($path, 'public');
|
|
|
|
foreach (ResponsiveImage::WIDTHS as $width) {
|
|
Storage::disk('public')->assertMissing("content/photo-{$width}.jpg.webp");
|
|
}
|
|
}
|
|
|
|
public function test_webp_originals_do_not_get_a_duplicate_webp_sibling(): void
|
|
{
|
|
Storage::fake('public');
|
|
|
|
$manager = new ImageManager(new Driver);
|
|
$path = 'content/already.webp';
|
|
|
|
Storage::disk('public')->put($path, (string) $manager->create(1600, 900)->toWebp());
|
|
|
|
ResponsiveImage::generate($path, 'public');
|
|
|
|
Storage::disk('public')->assertExists('content/already-480.webp');
|
|
Storage::disk('public')->assertMissing('content/already-480.webp.webp');
|
|
$this->assertSame([], ResponsiveImage::availableWebpVariants($path, 'public'));
|
|
}
|
|
|
|
public function test_metadata_is_cached_after_the_first_storage_read(): void
|
|
{
|
|
Storage::fake('public');
|
|
|
|
$manager = new ImageManager(new Driver);
|
|
$path = 'content/cached.jpg';
|
|
Storage::disk('public')->put($path, (string) $manager->create(1600, 900)->toJpeg());
|
|
ResponsiveImage::generate($path, 'public');
|
|
|
|
$metadata = ResponsiveImage::metadata($path, 'public');
|
|
|
|
$this->assertSame(['width' => 1600, 'height' => 900], [
|
|
'width' => $metadata['width'],
|
|
'height' => $metadata['height'],
|
|
]);
|
|
$this->assertSame(ResponsiveImage::WIDTHS, array_column($metadata['variants'], 'width'));
|
|
$this->assertSame(ResponsiveImage::WIDTHS, array_column($metadata['webp_variants'], 'width'));
|
|
|
|
Storage::disk('public')->delete(array_merge(
|
|
[$path],
|
|
array_map(fn (int $width): string => ResponsiveImage::variantPath($path, $width), ResponsiveImage::WIDTHS),
|
|
array_map(fn (int $width): string => ResponsiveImage::webpVariantPath($path, $width), ResponsiveImage::WIDTHS),
|
|
));
|
|
|
|
$this->assertSame($metadata, ResponsiveImage::metadata($path, 'public'));
|
|
}
|
|
|
|
public function test_generation_populates_metadata_and_deletion_invalidates_it(): void
|
|
{
|
|
Storage::fake('public');
|
|
|
|
$manager = new ImageManager(new Driver);
|
|
$path = 'content/generated.jpg';
|
|
Storage::disk('public')->put($path, (string) $manager->create(1400, 800)->toJpeg());
|
|
|
|
ResponsiveImage::generate($path, 'public');
|
|
|
|
$this->assertSame(['width' => 1400, 'height' => 800], ResponsiveImage::dimensions($path, 'public'));
|
|
$this->assertSame(ResponsiveImage::WIDTHS, array_column(ResponsiveImage::availableVariants($path, 'public'), 'width'));
|
|
|
|
ResponsiveImage::delete($path, 'public');
|
|
|
|
$this->assertNull(ResponsiveImage::metadata($path, 'public'));
|
|
}
|
|
|
|
public function test_replacement_invalidates_old_metadata_and_populates_new_metadata(): void
|
|
{
|
|
Storage::fake('public');
|
|
|
|
$manager = new ImageManager(new Driver);
|
|
Storage::disk('public')->put('content/old-metadata.jpg', (string) $manager->create(1600, 900)->toJpeg());
|
|
ResponsiveImage::generate('content/old-metadata.jpg', 'public');
|
|
ResponsiveImage::metadata('content/old-metadata.jpg', 'public');
|
|
|
|
Storage::disk('public')->put('content/new-metadata.jpg', (string) $manager->create(1200, 675)->toJpeg());
|
|
ResponsiveImage::replace('content/old-metadata.jpg', 'content/new-metadata.jpg', 'public');
|
|
|
|
$this->assertNull(ResponsiveImage::metadata('content/old-metadata.jpg', 'public'));
|
|
$this->assertSame(['width' => 1200, 'height' => 675], ResponsiveImage::dimensions('content/new-metadata.jpg', 'public'));
|
|
}
|
|
|
|
public function test_metadata_falls_back_safely_when_the_cache_is_unavailable(): void
|
|
{
|
|
Cache::shouldReceive('get')
|
|
->once()
|
|
->andThrow(new RuntimeException('Cache unavailable'));
|
|
|
|
$this->assertNull(ResponsiveImage::metadata('content/unavailable.jpg', 'public'));
|
|
}
|
|
}
|