fix(redirects): drop self-redirects at load time instead of serving a loop

[igoramf]

Primeiro de três PRs saídos do mesmo incidente. Este é o fix do bug; os outros dois estão listados no fim.

O bug

Uma regra cuja origem e destino resolvem para a mesma página responde toda request com um redirect para a URL que o browser acabou de pedir → ERR_TOO_MANY_REDIRECTS.

Exports de migração em massa trazem essas linhas prontas, e o normalizePath fabrica mais: ele reduz um from absoluto a new URL(p).pathname, e pathname não inclui a query. Então uma linha que só queria limpar uma URL legada de categoria:

https://www.montecarlo.com.br/aliancas?map=category-1,/aliancas,PERMANENT

vira exact["/aliancas"] = { to: "/aliancas" } e derruba a página inteira.

Aconteceu duas vezes no mesmo storefront, a partir do mesmo CSV:

Linha	Página derrubada
http://blog.montecarlo.com.br/,/,PERMANENT	home
https://www.montecarlo.com.br/aliancas?map=category-1,/aliancas,PERMANENT	/aliancas

Nas duas vezes o contorno foi apagar a linha do CSV. Isso não escala: são ~3000 linhas de dump, e qualquer sync futuro reintroduz.

Por que o loader Fresh não tinha isso

deco-cx/apps@0.161.0/website/loaders/redirectsFromCsv.ts, que este SDK substituiu:

if (!from || !to || from === to) continue;

O guard existia. O port perdeu. Este PR devolve, comparando depois da normalização — é justamente aí que as linhas acima colapsam.

Comportamento

Regra	Antes	Depois
/a → /a	loop	descartada no load
https://site/x?q=1 → /x	loop	descartada no load
http://blog.site/ → /	loop	descartada no load
/a/ → /a, /A → /a	loop	descartada no load
/x → /x?ref=1	loop (o destino recasa)	descartada no load
/x → https://other.site/x	ok	ok — host difere
/old → /new, /blog/* → /news/*	ok	ok

O host é comparado sempre que os dois lados carregam um, então redirect cross-origin para o mesmo path continua disparando.

Testes

packages/blocks/src/sdk/redirects.test.ts, 9 casos montados sobre as linhas reais do CSV do incidente. 7 falham no main:

git stash -- packages/blocks/src/sdk/redirects.ts
vitest run packages/blocks/src/sdk/redirects.test.ts
→ Tests  7 failed | 2 passed (9)

Com o patch: 9/9. Suíte do pacote: 756 passed (56 arquivos).

Trade-off que este PR assume

O guard descarta a regra em vez de consertá-la. ?map=category-1 → /aliancas é um redirect legítimo, que o Fresh honra hoje casando a href inteira (isHref: true), e aqui ele morre silenciosamente. É a escolha certa para um fix de loop — melhor perder um redirect do que derrubar uma página — mas é perda de comportamento.

Restaurar isso é o assunto do PR seguinte, empilhado neste:

• feat(redirects): honor the query a redirect source was scoped to #520 — suporte a regras com query (Redirect.search, matching por query no matchRedirect). Devolve o comportamento que o Fresh tem hoje.
• fix(redirects): respect quoted CSV fields and read the type case-insensitively #521 — dois bugs vizinhos do parser de CSV, independentes deste: split que ignora aspas (643 regras com destino lixo num CSV real) e PERMANENT maiúsculo virando 302 (~3000 redirects rebaixados de 301).

🤖 Generated with Claude Code