@bleedingdev/modern-js-plugin-tanstack 3.2.0-ultramodern.120 → 3.2.0-ultramodern.121

package/src/cli/routeSplitting.ts

@@ -10,35 +10,8 @@ export type TanstackRsbuildRouteSplittingProfile = {
       splitRouteChunks: boolean;
     };
   };
-  modernRouteChunks: {
-    enabled: boolean;
-    owner: 'modern';
-  };
-  builderChunkSplit: {
-    owner: 'modern-rsbuild';
-    preserved: true;
-  };
-  tanstackStartRspackSplitter: {
-    compatible: boolean;
-    reason: string;
-    clientDeleteNodes: string[];
-    routeFactoryCalls: string[];
-  };
 };
 
-const TANSTACK_START_ROUTE_FACTORY_CALLS = [
-  'createFileRoute',
-  'createRootRoute',
-  'createRootRouteWithContext',
-] as const;
-
-const TANSTACK_START_ROUTE_FACTORY_REGEX =
-  /\b(createFileRoute|createRootRoute|createRootRouteWithContext)\s*(?:<|\()/;
-
-export function isTanstackStartRouteModuleSource(source: string) {
-  return TANSTACK_START_ROUTE_FACTORY_REGEX.test(source);
-}
-
 export function resolveTanstackRouteCodeSplittingEnabled(
   option?: TanstackRouteCodeSplittingOption,
 ) {
@@ -49,6 +22,12 @@ export function resolveTanstackRouteCodeSplittingEnabled(
   return option?.enabled ?? true;
 }
 
+/**
+ * Route chunking for TanStack entries is owned by Modern's
+ * `output.splitRouteChunks`; the TanStack Start Rspack splitter does not
+ * apply because Modern generates TanStack route trees from Modern route
+ * metadata rather than TanStack file-route factory modules.
+ */
 export function createTanstackRsbuildRouteSplittingProfile(opts: {
   routeCodeSplitting?: TanstackRouteCodeSplittingOption;
 }): TanstackRsbuildRouteSplittingProfile {
@@ -60,22 +39,5 @@ export function createTanstackRsbuildRouteSplittingProfile(opts: {
         ),
       },
     },
-    modernRouteChunks: {
-      enabled: resolveTanstackRouteCodeSplittingEnabled(
-        opts.routeCodeSplitting,
-      ),
-      owner: 'modern',
-    },
-    builderChunkSplit: {
-      owner: 'modern-rsbuild',
-      preserved: true,
-    },
-    tanstackStartRspackSplitter: {
-      compatible: false,
-      reason:
-        'TanStack Start Rsbuild route splitting is tied to TanStack file-route factory modules; Modern generates TanStack route trees from Modern route metadata and owns route chunking through output.splitRouteChunks.',
-      clientDeleteNodes: ['ssr', 'server', 'headers'],
-      routeFactoryCalls: [...TANSTACK_START_ROUTE_FACTORY_CALLS],
-    },
   };
 }

package/src/cli/tanstackTypes.ts

@@ -1,31 +1,10 @@
 // @effect-diagnostics asyncFunction:off nodeBuiltinImport:off strictBooleanExpressions:off
 import type { AppToolsContext } from '@modern-js/app-tools';
+import { getPathWithoutExt, makeLegalIdentifier } from '@modern-js/runtime/cli';
 import type { NestedRouteForCli, PageRoute } from '@modern-js/types';
-import { findExists, formatImportPath, fs, slash } from '@modern-js/utils';
+import { findExists, formatImportPath, slash } from '@modern-js/utils';
 import path from 'path';
 
-const reservedWords =
-  'break case class catch const continue debugger default delete do else export extends finally for function if import in instanceof let new return super switch this throw try typeof var void while with yield enum await implements package protected static interface private public';
-const builtins =
-  'arguments Infinity NaN undefined null true false eval uneval isFinite isNaN parseFloat parseInt decodeURI decodeURIComponent encodeURI encodeURIComponent escape unescape Object Function Boolean Symbol Error EvalError InternalError RangeError ReferenceError SyntaxError TypeError URIError Number Math Date String RegExp Array Int8Array Uint8Array Uint8ClampedArray Int16Array Uint16Array Int32Array Float32Array Float64Array Map Set WeakMap WeakSet SIMD ArrayBuffer DataView JSON Promise Generator GeneratorFunction Reflect Proxy Intl';
-const forbidList = new Set<string>(`${reservedWords} ${builtins}`.split(' '));
-
-function makeLegalIdentifier(str: string) {
-  const identifier = str
-    .replace(/-(\w)/g, (_, letter) => letter.toUpperCase())
-    .replace(/[^$_a-zA-Z0-9]/g, '_');
-
-  if (/\d/.test(identifier[0]) || forbidList.has(identifier)) {
-    return `_${identifier}`;
-  }
-  return identifier || '_';
-}
-
-function getPathWithoutExt(filename: string) {
-  const extname = path.extname(filename);
-  return extname ? filename.slice(0, -extname.length) : filename;
-}
-
 const JS_OR_TS_EXTS = [
   '.js',
   '.jsx',
@@ -188,6 +167,19 @@ function paramsTypeForCanonicalPath(canonicalPath: string): string {
     : 'Record<string, never>';
 }
 
+export type CollectCanonicalRoutesOptions = {
+  /**
+   * Whether a leading `:lang`/`:locale`/`:language` route param may be
+   * treated as an i18n locale prefix. This MUST only be enabled when
+   * `@modern-js/plugin-i18n` is actually installed: the emitted
+   * `declare module '@modern-js/plugin-i18n/runtime'` augmentation breaks
+   * typechecking (TS2664) for apps that hand-roll a `/:lang/` param without
+   * the plugin. Routes carrying `modernCanonicalPath` metadata are always
+   * honored — only plugin-i18n produces them.
+   */
+  localeParamHeuristic?: boolean;
+};
+
 /**
  * Derive the canonical (language-agnostic) route map for an entry: the
  * leading locale param is stripped and localized physical variants (routes
@@ -198,7 +190,9 @@ function paramsTypeForCanonicalPath(canonicalPath: string): string {
  */
 export function collectCanonicalRoutesForEntry(
   routes: (NestedRouteForCli | PageRoute)[],
+  options: CollectCanonicalRoutesOptions = {},
 ): Record<string, string> | null {
+  const { localeParamHeuristic = true } = options;
   const canonicalParams = new Map<string, string>();
   let hasI18nSurface = false;
 
@@ -229,7 +223,11 @@ export function collectCanonicalRoutesForEntry(
         .replace(/\[(.+?)\]/g, ':$1')
         .split('/')
         .filter(Boolean);
-      if (parentPath === '' && LOCALE_PARAM_SEGMENTS.has(segments[0])) {
+      if (
+        localeParamHeuristic &&
+        parentPath === '' &&
+        LOCALE_PARAM_SEGMENTS.has(segments[0])
+      ) {
         hasI18nSurface = true;
         segments.shift();
       }
@@ -268,29 +266,6 @@ export function collectCanonicalRoutesForEntry(
   );
 }
 
-export async function isTanstackRouterFrameworkEnabled(
-  appContext: AppToolsContext,
-): Promise<boolean> {
-  const runtimeConfigBase = path.join(
-    appContext.srcDirectory,
-    appContext.runtimeConfigFile,
-  );
-  const runtimeConfigFile = findExists(
-    JS_OR_TS_EXTS.map(ext => `${runtimeConfigBase}${ext}`),
-  );
-  if (!runtimeConfigFile) {
-    return false;
-  }
-
-  try {
-    const content = await fs.readFile(runtimeConfigFile, 'utf-8');
-    // Heuristic: allow both single and double quotes, tolerate whitespace/newlines.
-    return /framework\s*:\s*['"]tanstack['"]/.test(content);
-  } catch {
-    return false;
-  }
-}
-
 export async function generateTanstackRouterTypesSourceForEntry(opts: {
   appContext: AppToolsContext;
   entryName: string;
@@ -324,13 +299,50 @@ export async function generateTanstackRouterTypesSourceForEntry(opts: {
   const statements: string[] = [];
 
   const loaderImportMap = new Map<string, string>();
+  const componentImportMap = new Map<string, Promise<string | null>>();
   const searchContractImportMap = new Map<string, string>();
   const usedRouteVarNames = new Set<string>();
   let loaderIndex = 0;
+  let componentIndex = 0;
   let validateSearchIndex = 0;
   let loaderDepsIndex = 0;
   let routeIndex = 0;
 
+  const getImportNameForComponent = (
+    componentPath: unknown,
+  ): Promise<string | null> => {
+    if (typeof componentPath !== 'string' || componentPath.length === 0) {
+      return Promise.resolve(null);
+    }
+
+    // Cache the in-flight promise: sibling routes sharing a component module
+    // are generated concurrently and must reuse one import.
+    let pendingImportName = componentImportMap.get(componentPath);
+    if (!pendingImportName) {
+      pendingImportName = (async () => {
+        // Resolve through the same machinery as loaders: the raw `_component`
+        // value carries the internal `@_modern_js_src` alias, which the app's
+        // tsconfig does not map — the generated file must use relative
+        // imports.
+        const resolvedNoExt = await resolveRouteModuleNoExt(componentPath);
+        if (!resolvedNoExt) {
+          return null;
+        }
+
+        const relImport = normalizeRelativeImport(
+          path.relative(outDir, resolvedNoExt),
+        );
+
+        const componentName = `component_${componentIndex++}`;
+        imports.push(`import ${componentName} from ${quote(relImport)};`);
+        return componentName;
+      })();
+      componentImportMap.set(componentPath, pendingImportName);
+    }
+
+    return pendingImportName;
+  };
+
   const resolveRouteModuleNoExt = async (aliasedNoExtPath: string) => {
     const prefix = `${appContext.internalSrcAlias}/`;
     let absNoExt: string;
@@ -475,6 +487,13 @@ export async function generateTanstackRouterTypesSourceForEntry(opts: {
 
     const routeOpts: string[] = [`getParentRoute: () => ${parentVar},`];
 
+    const componentName = await getImportNameForComponent(
+      (route as any)._component,
+    );
+    if (componentName) {
+      routeOpts.push(`component: ${componentName},`);
+    }
+
     if (isPathlessLayout(route)) {
       const id = (route as any).id as string | undefined;
       routeOpts.push(`id: ${quote(id || 'pathless')},`);
@@ -564,6 +583,14 @@ export async function generateTanstackRouterTypesSourceForEntry(opts: {
   );
 
   const rootOpts: string[] = [];
+
+  const rootComponentName = await getImportNameForComponent(
+    (rootModern as any)?._component,
+  );
+  if (rootComponentName) {
+    rootOpts.push(`component: ${rootComponentName},`);
+  }
+
   if (rootLoaderName) {
     rootOpts.push(
       `loader: modernLoaderToTanstack({ hasSplat: false }, ${rootLoaderName}),`,
@@ -581,178 +608,15 @@ export async function generateTanstackRouterTypesSourceForEntry(opts: {
 
 import {
   createMemoryHistory,
-  modernTanstackRouterFastDefaults,
   createRootRouteWithContext,
   createRoute,
   createRouter,
-  notFound,
-  redirect,
+  createRouteStaticData,
+  type ModernRouterContext,
+  modernLoaderToTanstack,
+  modernTanstackRouterFastDefaults,
 } from '@modern-js/plugin-tanstack/runtime';
 
-type ModernRouterContext = {
-  request?: Request;
-  requestContext?: unknown;
-};
-
-function isResponse(value: unknown): value is Response {
-  return (
-    value != null &&
-    typeof value === 'object' &&
-    typeof (value as any).status === 'number' &&
-    typeof (value as any).headers === 'object'
-  );
-}
-
-const redirectStatusCodes = new Set([301, 302, 303, 307, 308]);
-function isRedirectResponse(res: Response) {
-  return redirectStatusCodes.has(res.status);
-}
-
-function throwTanstackRedirect(location: string) {
-  const target = location.length > 0 ? location : '/';
-  try {
-    void new URL(target);
-    throw redirect({ href: target });
-  } catch {
-    throw redirect({ to: target });
-  }
-}
-
-function mapParamsForModernLoader(params: Record<string, string>, hasSplat: boolean) {
-  if (!hasSplat) {
-    return params;
-  }
-
-  const { _splat, ...rest } = params as any;
-  if (typeof _splat !== 'undefined') {
-    return { ...rest, '*': _splat };
-  }
-  return rest;
-}
-
-function createRouteStaticData(opts: {
-  modernRouteId?: string;
-  modernRouteAction?: unknown;
-  modernRouteLoader?: unknown;
-}) {
-  const staticData: {
-    modernRouteId?: string;
-    modernRouteAction?: unknown;
-    modernRouteLoader?: unknown;
-  } = {};
-
-  if (typeof opts.modernRouteId === 'string' && opts.modernRouteId.length > 0) {
-    staticData.modernRouteId = opts.modernRouteId;
-  }
-
-  if (typeof opts.modernRouteLoader !== 'undefined') {
-    staticData.modernRouteLoader = opts.modernRouteLoader;
-  }
-
-  if (typeof opts.modernRouteAction !== 'undefined') {
-    staticData.modernRouteAction = opts.modernRouteAction;
-  }
-
-  return staticData;
-}
-
-function getLoaderSignal(ctx: any): AbortSignal {
-  const abortSignal = ctx?.abortController?.signal;
-  if (abortSignal instanceof AbortSignal) {
-    return abortSignal;
-  }
-  if (ctx?.signal instanceof AbortSignal) {
-    return ctx.signal;
-  }
-  return new AbortController().signal;
-}
-
-function getLoaderHref(ctx: any): string {
-  if (typeof ctx?.location === 'string') {
-    return ctx.location;
-  }
-
-  const publicHref = ctx?.location?.publicHref;
-  if (typeof publicHref === 'string') {
-    return publicHref;
-  }
-
-  const href = ctx?.location?.href;
-  if (typeof href === 'string') {
-    return href;
-  }
-
-  const urlHref = ctx?.location?.url?.href;
-  return typeof urlHref === 'string' ? urlHref : '';
-}
-
-function getLoaderParams(ctx: any): Record<string, string> {
-  return typeof ctx?.params === 'object' && ctx.params !== null ? ctx.params : {};
-}
-
-function handleModernLoaderResult<LoaderResult>(result: LoaderResult): LoaderResult {
-  if (isResponse(result)) {
-    if (isRedirectResponse(result)) {
-      const location = result.headers.get('Location') ?? '/';
-      throwTanstackRedirect(location);
-    }
-    if (result.status === 404) {
-      throw notFound();
-    }
-  }
-
-  return result;
-}
-
-function handleModernLoaderError(err: unknown): never {
-  if (isResponse(err)) {
-    if (isRedirectResponse(err)) {
-      const location = err.headers.get('Location') ?? '/';
-      throwTanstackRedirect(location);
-    }
-    if (err.status === 404) {
-      throw notFound();
-    }
-  }
-
-  throw err;
-}
-
-function modernLoaderToTanstack<TLoader extends (args: any) => any>(
-  opts: { hasSplat: boolean },
-  modernLoader: TLoader,
-) {
-  type LoaderResult = Awaited<ReturnType<TLoader>>;
-
-  return (ctx: any): Promise<LoaderResult> => {
-    try {
-      const signal = getLoaderSignal(ctx);
-      const baseRequest: Request | undefined =
-        ctx?.context?.request instanceof Request ? ctx.context.request : undefined;
-
-      const href = getLoaderHref(ctx);
-
-      const request = baseRequest !== undefined
-        ? new Request(baseRequest, { signal })
-        : new Request(href, { signal });
-
-      const params = mapParamsForModernLoader(getLoaderParams(ctx), opts.hasSplat);
-
-      return Promise.resolve(
-        (modernLoader as any)({
-          request,
-          params,
-          context: ctx?.context?.requestContext,
-        }),
-      )
-        .then((result: LoaderResult) => handleModernLoaderResult(result))
-        .catch(handleModernLoaderError);
-    } catch (err) {
-      handleModernLoaderError(err);
-    }
-  };
-}
-
 ${imports.join('\n')}
 
 export const rootRoute = createRootRouteWithContext<ModernRouterContext>()({

package/src/runtime/hooks.ts

@@ -1,20 +1,10 @@
-import { createSyncHook } from '@modern-js/plugin';
-import type { TRuntimeContext } from '@modern-js/runtime/context';
-import type { RouteObject } from '@modern-js/runtime-utils/router';
-import type { RouterLifecycleContext } from './lifecycle';
-
-const modifyRoutes = createSyncHook<(routes: RouteObject[]) => RouteObject[]>();
-const onBeforeCreateRoutes =
-  createSyncHook<(context: TRuntimeContext) => void>();
-const onBeforeCreateRouter =
-  createSyncHook<(context: RouterLifecycleContext) => void>();
-const onAfterCreateRouter =
-  createSyncHook<(context: RouterLifecycleContext) => void>();
-const onBeforeHydrateRouter =
-  createSyncHook<(context: RouterLifecycleContext) => void>();
-const onAfterHydrateRouter =
-  createSyncHook<(context: RouterLifecycleContext) => void>();
-
+/**
+ * The router hooks are owned by @modern-js/runtime — this module re-exports
+ * the canonical instances through the `/context` seam, so the TanStack
+ * provider taps and calls the exact same hooks the built-in router wrapper
+ * registers. Creating separate hook instances here would silently split the
+ * hook registry between the wrapper and this provider.
+ */
 export {
   modifyRoutes,
   onAfterCreateRouter,
@@ -22,13 +12,6 @@ export {
   onBeforeCreateRouter,
   onBeforeCreateRoutes,
   onBeforeHydrateRouter,
-};
-
-export type RouterExtendsHooks = {
-  modifyRoutes: typeof modifyRoutes;
-  onBeforeCreateRoutes: typeof onBeforeCreateRoutes;
-  onBeforeCreateRouter: typeof onBeforeCreateRouter;
-  onAfterCreateRouter: typeof onAfterCreateRouter;
-  onBeforeHydrateRouter: typeof onBeforeHydrateRouter;
-  onAfterHydrateRouter: typeof onAfterHydrateRouter;
-};
+  type RouterExtendsHooks,
+  routerProviderRegistryHooks,
+} from '@modern-js/runtime/context';

package/src/runtime/index.tsx

@@ -1,104 +1,6 @@
-export type * from '@tanstack/react-router';
-export {
-  Asset,
-  Await,
-  Block,
-  CatchBoundary,
-  CatchNotFound,
-  ClientOnly,
-  cleanPath,
-  composeRewrites,
-  createBrowserHistory,
-  createControlledPromise,
-  createFileRoute,
-  createHashHistory,
-  createHistory,
-  createLazyFileRoute,
-  createLazyRoute,
-  createLink,
-  createMemoryHistory,
-  createRootRoute,
-  createRootRouteWithContext,
-  createRoute,
-  createRouteMask,
-  createRouter,
-  createRouterConfig,
-  createSerializationAdapter,
-  DEFAULT_PROTOCOL_ALLOWLIST,
-  DefaultGlobalNotFound,
-  deepEqual,
-  defaultParseSearch,
-  defaultStringifySearch,
-  defer,
-  ErrorComponent,
-  FileRoute,
-  FileRouteLoader,
-  functionalUpdate,
-  getRouteApi,
-  HeadContent,
-  interpolatePath,
-  isMatch,
-  isNotFound,
-  isPlainArray,
-  isPlainObject,
-  isRedirect,
-  joinPaths,
-  LazyRoute,
-  lazyFn,
-  lazyRouteComponent,
-  linkOptions,
-  Match,
-  Matches,
-  MatchRoute,
-  Navigate,
-  NotFoundRoute,
-  notFound,
-  parseSearchWith,
-  RootRoute,
-  Route,
-  RouteApi,
-  Router,
-  RouterContextProvider,
-  RouterProvider,
-  reactUse,
-  redirect,
-  replaceEqualDeep,
-  resolvePath,
-  retainSearchParams,
-  rootRouteId,
-  rootRouteWithContext,
-  ScriptOnce,
-  Scripts,
-  ScrollRestoration,
-  SearchParamError,
-  stringifySearchWith,
-  stripSearchParams,
-  trimPath,
-  trimPathLeft,
-  trimPathRight,
-  useAwaited,
-  useBlocker,
-  useCanGoBack,
-  useChildMatches,
-  useElementScrollRestoration,
-  useHydrated,
-  useLayoutEffect,
-  useLinkProps,
-  useLoaderData,
-  useLoaderDeps,
-  useLocation,
-  useMatch,
-  useMatches,
-  useMatchRoute,
-  useNavigate,
-  useParams,
-  useParentMatches,
-  useRouteContext,
-  useRouter,
-  useRouterState,
-  useSearch,
-  useTags,
-} from '@tanstack/react-router';
+import './register';
+
+export * from '@tanstack/react-router';
 export type {
   Fetcher,
   FetcherState,
@@ -111,6 +13,13 @@ export {
   RouteActionResponseError,
   useFetcher,
 } from './dataMutation';
+export type { ModernRouterContext } from './loaderBridge';
+export {
+  createRouteStaticData,
+  isAbsoluteUrl,
+  modernLoaderToTanstack,
+  throwTanstackRedirect,
+} from './loaderBridge';
 export { Outlet } from './outlet';
 export {
   tanstackRouterPlugin,
@@ -122,12 +31,8 @@ export type {
   PrefetchBehavior,
 } from './prefetchLink';
 export { Link, NavLink } from './prefetchLink';
-export type {
-  AnyCompositeComponent,
-  AnyRenderableServerComponent,
-  CompositeComponentProps,
-} from './rsc/client';
-export { CompositeComponent } from './rsc/client';
+export type { TanstackRouterState } from './state';
+export { getTanstackRouterState } from './state';
 export type { RouterConfig } from './types';
 export {
   getModernTanstackRouterFastDefaults,