Skip to content

chore(routeTree): regenerate with the current router version - #1505

Merged
tannerlinsley merged 1 commit into
mainfrom
chore/regenerate-route-tree
Nov 12, 2026
Merged

tannerlinsley merged 1 commit into
mainfrom
chore/regenerate-route-tree

Conversation

@sukvvon

@sukvvon sukvvon commented Nov 12, 2026 •

Copy link
Copy Markdown
Member

Running pnpm run dev rewrites src/routeTree.gen.ts and drops four blank lines, so the working tree is dirty from the moment the dev server starts. The committed file was generated by an older version of the generator; this commit is just its current output.

 import type { getRouter } from './router.tsx'
-
 import type { startInstance } from './start.ts'

Not a formatting issue — oxfmt leaves the committed file byte-identical, so the blank lines come from the generator itself.

With this applied, the tree stays clean while the dev server is running.

Worth a look separately

src/routeTree.gen.ts is listed in .gitignore (line 38, added in d335de12) but is still tracked, so the ignore rule has no effect and the file has been committed many times since. If the intent was to stop tracking it, git rm --cached would be the real fix — but src/router.tsx imports it directly, so that needs a check that every build path regenerates it first. Happy to open that separately if it is wanted.

Summary by CodeRabbit

  • Refactor
    • Updated generated route type declarations; no user-visible behavior changes.

@changeset-bot

changeset-bot Bot commented Nov 12, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: f830b0e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@sukvvon sukvvon self-assigned this Nov 12, 2026
@coderabbitai

coderabbitai Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The generated route tree reformats the Start module augmentation. Its SSR, router, and configuration registrations remain unchanged.

Changes: Route tree declarations

Layer / File(s) Summary
Start module augmentation
src/routeTree.gen.ts
The generated declaration registers SSR, router, and configuration types. The registrations match the previous declarations.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Suggested reviewers: harry-whorlow

Merge Risk: Minimal · up to f830b

This change commits the output the router generator already produces, so the working tree stays clean when the dev server runs. It has no runtime effect and is safe to merge.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title names a chore that regenerates src/routeTree.gen.ts with the current router version. The raw summary confirms the change is a generated route tree that adds a @tanstack/react-start module au…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title names a chore that regenerates src/routeTree.gen.ts with the current router version. The raw summary confirms the change is a generated route tree that adds a @tanstack/react-start module augmentation. The objectives describe the same regeneration, which removes four blank lines and makes the working tree dirty. The title is short, specific, and matches the main change. It does not mention the unresolved question about removing the file from tracking, which is a secondary point.

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 06f0add2-51df-4c8d-b3c8-c87c69c5bc9c
Commits

Reviewing files that changed from the base of the PR and between f00c509 and f830b0e.

Files selected for processing: 1

  • src/routeTree.gen.ts
Files with no reviewable changes: 1

  • src/routeTree.gen.ts

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 2 remain after this review.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
tanstack-com f830b0e Commit Preview URL

Branch Preview URL
Nov 12 2026, 09:27 AM

@tannerlinsley
tannerlinsley merged commit fe0b1ba into main Nov 12, 2026
8 checks passed
@tannerlinsley
tannerlinsley deleted the chore/regenerate-route-tree branch November 12, 2026 09:44
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.

2 participants