Skip to content

[ftl] Include cause of route types#92056

Merged
eps1lon merged 2 commits intocanaryfrom
sebbie/ftl/typed-links-cause
Mar 30, 2026
Merged

[ftl] Include cause of route types#92056
eps1lon merged 2 commits intocanaryfrom
sebbie/ftl/typed-links-cause

Conversation

@eps1lon
Copy link
Copy Markdown
Member

@eps1lon eps1lon commented Mar 28, 2026

Targetted at debugging (too) complex route types.

Helps debugging which route type was caused by which route file. This is especially helpful with non-trivial routes (route groups, breadcrumbs) and redirects/rewrites.

Redirect/rewrites won't have a helpful cause. For now the most helfpul info for me was knowing which file route it belongs to and if it's a redirect or not.

First commit is a refactor getting rid of intermediate arrays in favor of a more simpler programming model with for-loops and visited tracker.

@nextjs-bot
Copy link
Copy Markdown
Collaborator

nextjs-bot commented Mar 28, 2026

Failing test suites

Commit: 2da9811 | About building and testing Next.js

@eps1lon eps1lon marked this pull request as ready for review March 28, 2026 19:01
@eps1lon eps1lon enabled auto-merge (squash) March 28, 2026 22:33
@eps1lon eps1lon force-pushed the sebbie/ftl/typed-links-cause branch from 050629f to 2da9811 Compare March 28, 2026 23:02
@eps1lon
Copy link
Copy Markdown
Member Author

eps1lon commented Mar 30, 2026

Failures are unrelated

@eps1lon eps1lon disabled auto-merge March 30, 2026 16:57
@eps1lon eps1lon merged commit 49670f8 into canary Mar 30, 2026
655 of 671 checks passed
@eps1lon eps1lon deleted the sebbie/ftl/typed-links-cause branch March 30, 2026 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants