Create file routes directly

packages/react-router/src/fileRoute.ts

@@ -1,4 +1,4 @@
-import { createRoute } from './route'
+import { createRoute as createRouteImpl } from './route'
 
 import { useMatch } from './useMatch'
 import { useLoaderDeps } from './useLoaderDeps'
@@ -54,11 +54,15 @@ export function createFileRoute<
   TFullPath extends RouteConstraints['TFullPath'] =
     FileRoutesByPath[TFilePath]['fullPath'],
 >(
+  // eslint-disable-next-line unused-imports/no-unused-vars
   path?: TFilePath,
 ): FileRoute<TFilePath, TParentRoute, TId, TPath, TFullPath>['createRoute'] {
-  return new FileRoute<TFilePath, TParentRoute, TId, TPath, TFullPath>(path, {
-    silent: true,
-  }).createRoute
+  const createRoute = (options?: any) => {
+    const route = createRouteImpl(options)
+    ;(route as any).isRoot = false
+    return route
+  }
+  return createRoute as any
 }
 
 /** 
@@ -151,7 +155,7 @@ export class FileRoute<
         )
       }
     }
-    const route = createRoute(options as any)
+    const route = createRouteImpl(options as any)
     ;(route as any).isRoot = false
     return route as any
   }

packages/react-router/tests/fileRoute.test.ts

@@ -1,14 +1,39 @@
 /* eslint-disable */
-import { describe, it, expect } from 'vitest'
+import { afterEach, describe, it, expect, vi } from 'vitest'
 import {
   getRouteApi,
   createFileRoute,
   createLazyRoute,
   createLazyFileRoute,
   LazyRoute,
   AnyRoute,
+  FileRoute,
 } from '../src'
 
+afterEach(() => {
+  vi.restoreAllMocks()
+})
+
+describe('createFileRoute', () => {
+  it('creates a non-root route without a deprecation warning', () => {
+    const warn = vi.spyOn(console, 'warn').mockImplementation(() => {})
+    // @ts-expect-error
+    const route = createFileRoute('')({})
+
+    expect(route.isRoot).toBe(false)
+    expect(warn).not.toHaveBeenCalled()
+  })
+
+  it('keeps the deprecation warning for direct FileRoute usage', () => {
+    const warn = vi.spyOn(console, 'warn').mockImplementation(() => {})
+    // @ts-expect-error
+    const route = new FileRoute('').createRoute({})
+
+    expect(route.isRoot).toBe(false)
+    expect(warn).toHaveBeenCalledOnce()
+  })
+})
+
 describe('createFileRoute has the same hooks as getRouteApi', () => {
   const routeApi = getRouteApi('foo')
   const hookNames = Object.keys(routeApi).filter((key) => key.startsWith('use'))

packages/solid-router/src/fileRoute.ts

@@ -1,4 +1,4 @@
-import { createRoute } from './route'
+import { createRoute as createRouteImpl } from './route'
 
 import { useMatch } from './useMatch'
 import { useLoaderDeps } from './useLoaderDeps'
@@ -43,11 +43,15 @@ export function createFileRoute<
   TFullPath extends RouteConstraints['TFullPath'] =
     FileRoutesByPath[TFilePath]['fullPath'],
 >(
+  // eslint-disable-next-line unused-imports/no-unused-vars
   path?: TFilePath,
 ): FileRoute<TFilePath, TParentRoute, TId, TPath, TFullPath>['createRoute'] {
-  return new FileRoute<TFilePath, TParentRoute, TId, TPath, TFullPath>(path, {
-    silent: true,
-  }).createRoute
+  const createRoute = (options?: any) => {
+    const route = createRouteImpl(options)
+    ;(route as any).isRoot = false
+    return route
+  }
+  return createRoute as any
 }
 
 /** 
@@ -140,7 +144,7 @@ export class FileRoute<
         )
       }
     }
-    const route = createRoute(options as any)
+    const route = createRouteImpl(options as any)
     ;(route as any).isRoot = false
     return route as any
   }

packages/solid-router/tests/fileRoute.test.ts

@@ -1,13 +1,38 @@
 /* eslint-disable */
-import { describe, it, expect } from 'vitest'
+import { afterEach, describe, it, expect, vi } from 'vitest'
 import {
   getRouteApi,
   createFileRoute,
   createLazyRoute,
   createLazyFileRoute,
   LazyRoute,
+  FileRoute,
 } from '../src'
 
+afterEach(() => {
+  vi.restoreAllMocks()
+})
+
+describe('createFileRoute', () => {
+  it('creates a non-root route without a deprecation warning', () => {
+    const warn = vi.spyOn(console, 'warn').mockImplementation(() => {})
+    // @ts-expect-error
+    const route = createFileRoute('')({})
+
+    expect(route.isRoot).toBe(false)
+    expect(warn).not.toHaveBeenCalled()
+  })
+
+  it('keeps the deprecation warning for direct FileRoute usage', () => {
+    const warn = vi.spyOn(console, 'warn').mockImplementation(() => {})
+    // @ts-expect-error
+    const route = new FileRoute('').createRoute({})
+
+    expect(route.isRoot).toBe(false)
+    expect(warn).toHaveBeenCalledOnce()
+  })
+})
+
 describe('createFileRoute has the same hooks as getRouteApi', () => {
   const routeApi = getRouteApi('foo')
   const hookNames = Object.keys(routeApi).filter((key) => key.startsWith('use'))

packages/vue-router/src/fileRoute.ts

@@ -1,4 +1,4 @@
-import { createRoute } from './route'
+import { createRoute as createRouteImpl } from './route'
 
 import { useMatch } from './useMatch'
 import { useLoaderDeps } from './useLoaderDeps'
@@ -43,11 +43,15 @@ export function createFileRoute<
   TFullPath extends RouteConstraints['TFullPath'] =
     FileRoutesByPath[TFilePath]['fullPath'],
 >(
+  // eslint-disable-next-line unused-imports/no-unused-vars
   path?: TFilePath,
 ): FileRoute<TFilePath, TParentRoute, TId, TPath, TFullPath>['createRoute'] {
-  return new FileRoute<TFilePath, TParentRoute, TId, TPath, TFullPath>(path, {
-    silent: true,
-  }).createRoute
+  const createRoute = (options?: any) => {
+    const route = createRouteImpl(options)
+    ;(route as any).isRoot = false
+    return route
+  }
+  return createRoute as any
 }
 
 /**
@@ -140,7 +144,7 @@ export class FileRoute<
         )
       }
     }
-    const route = createRoute(options as any)
+    const route = createRouteImpl(options as any)
     ;(route as any).isRoot = false
     return route as any
   }

packages/vue-router/tests/fileRoute.test.ts

@@ -1,13 +1,38 @@
 /* eslint-disable */
-import { describe, it, expect } from 'vitest'
+import { afterEach, describe, it, expect, vi } from 'vitest'
 import {
   getRouteApi,
   createFileRoute,
   createLazyRoute,
   createLazyFileRoute,
   LazyRoute,
+  FileRoute,
 } from '../src'
 
+afterEach(() => {
+  vi.restoreAllMocks()
+})
+
+describe('createFileRoute', () => {
+  it('creates a non-root route without a deprecation warning', () => {
+    const warn = vi.spyOn(console, 'warn').mockImplementation(() => {})
+    // @ts-expect-error
+    const route = createFileRoute('')({})
+
+    expect(route.isRoot).toBe(false)
+    expect(warn).not.toHaveBeenCalled()
+  })
+
+  it('keeps the deprecation warning for direct FileRoute usage', () => {
+    const warn = vi.spyOn(console, 'warn').mockImplementation(() => {})
+    // @ts-expect-error
+    const route = new FileRoute('').createRoute({})
+
+    expect(route.isRoot).toBe(false)
+    expect(warn).toHaveBeenCalledOnce()
+  })
+})
+
 describe('createFileRoute has the same hooks as getRouteApi', () => {
   const routeApi = getRouteApi('foo')
   const hookNames = Object.keys(routeApi).filter((key) => key.startsWith('use'))