feat: implement HTTP 301 redirects for capitalized valid SPA pages in edge worker - #784
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
equipose | 2b4ee80 | Commit Preview URL Branch Preview URL |
Aug 13 2026, 01:35 PM |
|
[CI/CD Fix] Fixed the Angular test compilation failure. The Cloudflare Edge Worker spec file (src/worker.spec.ts) was being picked up and compiled by the Angular unit-test builder (ng test). Since the worker file imports the generated Cloudflare _headers file which contains native path-matching syntax (such as /*) that isn't terminated like a JavaScript multi-line comment, the compiler failed with "Expected */ to terminate multi-line comment". Excluding worker files from the browser-focused Angular unit-test builder via angular.json and tsconfig.spec.json fixes the compilation issue while keeping worker tests covered natively by Vitest (pnpm test:unit). |
|
[CI/CD Fix] Resolved the setup/linter CI failure. The Cloudflare worker spec file (src/worker.spec.ts) was excluded from both tsconfig.app.json and tsconfig.spec.json to prevent Angular test compilation errors (due to path-matching comments in imported Cloudflare _headers files). However, this caused ESLint's TypeScript project service to throw a parsing error as these worker files were not part of any referenced TypeScript project. To fix this cleanly, I created a dedicated |
|
…ges with spec coverage
…s to resolve ESLint project service parsing error
da5f365 to
2b4ee80
Compare
|
Hi @fderuiter, I have successfully addressed your instructions from your previous comment:
The PR is now fully up to date, rebased, and all unit/integration tests are verified green! |
|
🎉 This PR is included in version 1.51.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary of Changes
src/worker.tsto solve case-sensitive router match failures causing soft 404 crawl errors.src/worker.spec.tsmatching all functional criteria.