Fix PE redirect percent encoding

[kentcdodds]

Summary

• Route progressive enhancement redirects through toRedirectLocation so already-percent-encoded paths are not double encoded.
• Add regression coverage for encoded paths, non-ASCII paths, unsafe redirects, and unencodable input.

Testing

• npx nx run @epic-web/workshop-app:build passes.
• npm --prefix packages/workshop-app run lint passes.
• npx tsx --conditions development -e "import { toRedirectLocation } from './packages/workshop-app/app/utils/pe.tsx'; const cases = [['/foo%20bar','/foo%20bar'], ['/café','/caf%C3%A9'], ['https://example.com/foo%20bar','/'], ['/'+String.fromCharCode(0xd800), '/']]; for (const [input, expected] of cases) { const actual = toRedirectLocation(input); if (actual !== expected) throw new Error(input + ' expected ' + expected + ' got ' + actual); console.log(input.replace(/\\ud800/g, '<surrogate>') + ' => ' + actual); }" passes.
• npm test passed in the git pre-push hook: 21 files, 167 tests.

  

[nx-cloud]

View your CI Pipeline Execution ↗ for commit dc21bc5

Command	Status	Duration	Result
nx run-many --target typecheck	✅ Succeeded	10s	View ↗
nx run-many --target build	✅ Succeeded	40s	View ↗
nx lint	✅ Succeeded	25s	View ↗

---

☁️ Nx Cloud last updated this comment at 2026-05-01 15:30:18 UTC