-
Notifications
You must be signed in to change notification settings - Fork 0
refactor: migrate codebase to TanStack Router #25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
chiptus
merged 43 commits into
main
from
claude/migrate-tanstack-router-015h5PFMhsh3FgDeb2uEpKgX
Apr 14, 2026
Merged
Changes from all commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
351522a
refactor: migrate from react-router-dom to @tanstack/react-router
claude 39e1831
fix: complete TanStack Router migration
claude 228ab97
fix: resolve all remaining TypeScript errors in TanStack Router migra…
claude 7c7308b
fix: resolve all TypeScript errors in TanStack Router migration
claude ac74153
refactor: improve TanStack Router type safety and add Zod validation
claude 9fbb823
fix: improve navigation type safety in admin pages
claude 3caa448
refactor: eliminate as any casts from route navigation
claude a7f1e5f
refactor: use TanStack Router relative navigation with from property
claude 4d50bc8
refactor: eliminate all 'as any' casts from navigation hooks
claude 34240bc
fix: properly type assert search params with strict: false
claude a8bf05a
refactor: remove type assertions from search params using runtime val…
claude 05bcd45
refactor: simplify search param access by relying on router-level val…
claude 5e4f128
refactor: remove strict: false from all useParams and useRouteContext…
claude e0c554c
refactor: remove 'as string' type assertions from navigation params
claude a2e18a0
fix: add optional chaining for params in beforeLoad functions
claude 6cf9b51
ci: add lint workflow for typecheck and linting
chiptus 6709260
fix: resolve all TypeScript errors for TanStack Router hooks
claude 30073b5
fix
chiptus 2028bfe
refactor: address PR review feedback
claude 193a12a
refactor: address additional PR review feedback
claude 10db104
fix timeline state
chiptus caaff88
refactor: apply strict route typing pattern to useUrlState
claude e282244
refactor: address PR review comments
claude eb59071
fix: address remaining PR review comments
claude 927599a
refactor: implement TanStack Router best practices
claude de6f132
fix: add validateSearch schema to set detail route
claude f792448
fix: restructure sets routes to fix detail page routing
claude 48a7d88
fix: resolve router state errors and simplify useUrlState
claude 096c7ea
fix: properly type router context and make useUrlState strict
claude e38dbd2
chore: regenerate route tree with parent sets route
claude c24cd7a
Migrate FestivalEditionContext to TanStack Router params
claude 3ecd737
Fix subdomain routing to auto-navigate to festival page
claude 4524c0c
Use TanStack Router URL rewrites for subdomain routing
claude 8441f7c
Fix URL rewrite bugs: double slash and wrong destructuring
claude f0a8dfe
Simplify FestivalEditionContext: drop useMatches and getSubdomainInfo
claude ed19af5
Remove getSubdomainInfo from EditionSelection
claude 613c44f
Move FestivalEditionProvider to festival layout route
claude 5fe2cfd
Revert "Move FestivalEditionProvider to festival layout route"
claude 1472bb6
Move FestivalEditionProvider to festival layout route with data prelo…
claude 47465d5
Fix crash on non-festival pages + drop basePath from context
claude 1638a01
Fix Copilot review issues: localhost URL rewrite, CSVImportPage routi…
claude f1b7e49
Fix Copilot review issues: type safety and code deduplication
claude c20184d
Remove remaining unnecessary type casts and bind components to routes
claude File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
| name: Lint | ||
|
|
||
| on: | ||
| push: | ||
| branches: [main, develop] | ||
| pull_request: | ||
| branches: [main, develop] | ||
|
|
||
| jobs: | ||
| lint: | ||
| name: Typecheck and Lint | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 5 | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - uses: pnpm/action-setup@v4 | ||
|
|
||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 22 | ||
| cache: "pnpm" | ||
|
|
||
| - name: Install dependencies | ||
| run: pnpm install --frozen-lockfile | ||
|
|
||
| - name: Run typecheck | ||
| run: pnpm run typecheck | ||
|
|
||
| - name: Run lint | ||
| run: pnpm run lint |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,2 +1 @@ | ||
| npx lint-staged | ||
| pnpm run typecheck |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,4 +8,5 @@ yarn.lock | |
| .vscode | ||
| .idea | ||
| coverage | ||
| *.log | ||
| *.log | ||
| src/routeTree.gen.ts | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.