Skip to content

fix(frontend): remove duplicated streams/streams/[streamId] route and add redirect - #1157

Open
omoh5 wants to merge 1 commit into
LabsCrypt:mainfrom
omoh5:fix/1084-remove-duplicate-stream-route
Open

fix(frontend): remove duplicated streams/streams/[streamId] route and add redirect#1157
omoh5 wants to merge 1 commit into
LabsCrypt:mainfrom
omoh5:fix/1084-remove-duplicate-stream-route

Conversation

@omoh5

@omoh5 omoh5 commented Jul 30, 2026

Copy link
Copy Markdown

Summary

Fixes #1084 — removes the dead /streams/streams/[streamId] route that was a leftover from a previous refactor, and adds a permanent redirect to the canonical /streams/:id route.

Problem

  • frontend/src/app/streams/streams/[streamId]/page.tsx was a duplicated dynamic route alongside frontend/src/app/streams/[id]/page.tsx
  • The code-searcher confirmed zero references to the streams/streams/ path anywhere in the codebase
  • With two competing dynamic routes, Next.js can route to either one, causing confusion for navigation and SEO (duplicate content at two URLs)

Changes

🔴 Removed

  • frontend/src/app/streams/streams/[streamId]/page.tsx (343 lines deleted)
    • Confirmed dead code — no imports, links, or navigation references point to it
    • All links in the codebase use /streams/<id> which resolves to the [id] route

🟢 Added

  • frontend/next.config.ts — permanent redirect (HTTP 308)
    • /streams/streams/:streamId/streams/:streamId
    • Ensures any external links or bookmarks to the old path seamlessly land on the correct page

Verification

  • npx tsc --noEmit — no new errors (pre-existing errors in useIncomingStreams.test.ts are unrelated)
  • Pre-commit lint hook passed
  • Code reviewer confirmed no issues
  • Directory confirmed removed

Out of scope

  • Restructuring the whole streams route tree
  • Changing the stream detail UI

… add redirect

Delete the dead /streams/streams/:streamId route that was a leftover from
refactoring streamId to id. Add a permanent redirect in next.config.ts so
any external links to the old path land on the canonical stream details page.

Closes LabsCrypt#1084
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Frontend] frontend/src/app/streams/streams/[streamId]/page.tsx duplicated route nesting looks like a stray leftover from a refactor

1 participant