@bleedingdev/modern-js-plugin-tanstack 3.2.0-ultramodern.81 → 3.2.0-ultramodern.83
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
|
@@ -85,12 +85,19 @@ function normalizeModernLoaderResponse(result) {
|
|
|
85
85
|
}
|
|
86
86
|
return normalizeModernLoaderResult(result);
|
|
87
87
|
}
|
|
88
|
-
function pickRouteModuleComponent(routeModule) {
|
|
88
|
+
function pickRouteModuleComponent(routeModule, seen = new Set()) {
|
|
89
89
|
if ('function' == typeof routeModule || routeModule && 'object' == typeof routeModule && '$$typeof' in routeModule) return routeModule;
|
|
90
90
|
if (!routeModule || 'object' != typeof routeModule) return;
|
|
91
|
+
if (seen.has(routeModule)) return;
|
|
92
|
+
seen.add(routeModule);
|
|
91
93
|
const module = routeModule;
|
|
92
|
-
const
|
|
93
|
-
|
|
94
|
+
for (const candidate of [
|
|
95
|
+
module.default,
|
|
96
|
+
module.Component
|
|
97
|
+
]){
|
|
98
|
+
const component = pickRouteModuleComponent(candidate, seen);
|
|
99
|
+
if (component) return component;
|
|
100
|
+
}
|
|
94
101
|
}
|
|
95
102
|
function createServerLazyImportComponent(lazyImport, fallbackComponent) {
|
|
96
103
|
if ("u" > typeof document) return fallbackComponent;
|
|
@@ -55,12 +55,19 @@ function normalizeModernLoaderResponse(result) {
|
|
|
55
55
|
}
|
|
56
56
|
return normalizeModernLoaderResult(result);
|
|
57
57
|
}
|
|
58
|
-
function pickRouteModuleComponent(routeModule) {
|
|
58
|
+
function pickRouteModuleComponent(routeModule, seen = new Set()) {
|
|
59
59
|
if ('function' == typeof routeModule || routeModule && 'object' == typeof routeModule && '$$typeof' in routeModule) return routeModule;
|
|
60
60
|
if (!routeModule || 'object' != typeof routeModule) return;
|
|
61
|
+
if (seen.has(routeModule)) return;
|
|
62
|
+
seen.add(routeModule);
|
|
61
63
|
const module = routeModule;
|
|
62
|
-
const
|
|
63
|
-
|
|
64
|
+
for (const candidate of [
|
|
65
|
+
module.default,
|
|
66
|
+
module.Component
|
|
67
|
+
]){
|
|
68
|
+
const component = pickRouteModuleComponent(candidate, seen);
|
|
69
|
+
if (component) return component;
|
|
70
|
+
}
|
|
64
71
|
}
|
|
65
72
|
function createServerLazyImportComponent(lazyImport, fallbackComponent) {
|
|
66
73
|
if ("u" > typeof document) return fallbackComponent;
|
|
@@ -56,12 +56,19 @@ function normalizeModernLoaderResponse(result) {
|
|
|
56
56
|
}
|
|
57
57
|
return normalizeModernLoaderResult(result);
|
|
58
58
|
}
|
|
59
|
-
function pickRouteModuleComponent(routeModule) {
|
|
59
|
+
function pickRouteModuleComponent(routeModule, seen = new Set()) {
|
|
60
60
|
if ('function' == typeof routeModule || routeModule && 'object' == typeof routeModule && '$$typeof' in routeModule) return routeModule;
|
|
61
61
|
if (!routeModule || 'object' != typeof routeModule) return;
|
|
62
|
+
if (seen.has(routeModule)) return;
|
|
63
|
+
seen.add(routeModule);
|
|
62
64
|
const module = routeModule;
|
|
63
|
-
const
|
|
64
|
-
|
|
65
|
+
for (const candidate of [
|
|
66
|
+
module.default,
|
|
67
|
+
module.Component
|
|
68
|
+
]){
|
|
69
|
+
const component = pickRouteModuleComponent(candidate, seen);
|
|
70
|
+
if (component) return component;
|
|
71
|
+
}
|
|
65
72
|
}
|
|
66
73
|
function createServerLazyImportComponent(lazyImport, fallbackComponent) {
|
|
67
74
|
if ("u" > typeof document) return fallbackComponent;
|
package/package.json
CHANGED
|
@@ -18,7 +18,7 @@
|
|
|
18
18
|
"modern.js",
|
|
19
19
|
"tanstack-router"
|
|
20
20
|
],
|
|
21
|
-
"version": "3.2.0-ultramodern.
|
|
21
|
+
"version": "3.2.0-ultramodern.83",
|
|
22
22
|
"engines": {
|
|
23
23
|
"node": ">=20"
|
|
24
24
|
},
|
|
@@ -88,13 +88,13 @@
|
|
|
88
88
|
"@swc/helpers": "^0.5.21",
|
|
89
89
|
"@tanstack/react-router": "1.170.8",
|
|
90
90
|
"@tanstack/router-core": "1.171.6",
|
|
91
|
-
"@modern-js/plugin": "npm:@bleedingdev/modern-js-plugin@3.2.0-ultramodern.
|
|
92
|
-
"@modern-js/
|
|
93
|
-
"@modern-js/
|
|
94
|
-
"@modern-js/
|
|
91
|
+
"@modern-js/plugin": "npm:@bleedingdev/modern-js-plugin@3.2.0-ultramodern.83",
|
|
92
|
+
"@modern-js/runtime-utils": "npm:@bleedingdev/modern-js-runtime-utils@3.2.0-ultramodern.83",
|
|
93
|
+
"@modern-js/types": "npm:@bleedingdev/modern-js-types@3.2.0-ultramodern.83",
|
|
94
|
+
"@modern-js/utils": "npm:@bleedingdev/modern-js-utils@3.2.0-ultramodern.83"
|
|
95
95
|
},
|
|
96
96
|
"peerDependencies": {
|
|
97
|
-
"@modern-js/runtime": "3.2.0-ultramodern.
|
|
97
|
+
"@modern-js/runtime": "3.2.0-ultramodern.83",
|
|
98
98
|
"react": "^19.2.6",
|
|
99
99
|
"react-dom": "^19.2.6"
|
|
100
100
|
},
|
|
@@ -109,8 +109,8 @@
|
|
|
109
109
|
"@typescript/native-preview": "7.0.0-dev.20260527.2",
|
|
110
110
|
"react": "^19.2.6",
|
|
111
111
|
"react-dom": "^19.2.6",
|
|
112
|
-
"@modern-js/app-tools": "npm:@bleedingdev/modern-js-app-tools@3.2.0-ultramodern.
|
|
113
|
-
"@modern-js/runtime": "npm:@bleedingdev/modern-js-runtime@3.2.0-ultramodern.
|
|
112
|
+
"@modern-js/app-tools": "npm:@bleedingdev/modern-js-app-tools@3.2.0-ultramodern.83",
|
|
113
|
+
"@modern-js/runtime": "npm:@bleedingdev/modern-js-runtime@3.2.0-ultramodern.83",
|
|
114
114
|
"@scripts/rstest-config": "2.66.0"
|
|
115
115
|
},
|
|
116
116
|
"sideEffects": false,
|
package/src/runtime/routeTree.ts
CHANGED
|
@@ -231,6 +231,7 @@ function normalizeModernLoaderResponse(result: unknown): unknown {
|
|
|
231
231
|
|
|
232
232
|
function pickRouteModuleComponent(
|
|
233
233
|
routeModule: unknown,
|
|
234
|
+
seen: Set<unknown> = new Set(),
|
|
234
235
|
): ElementType<Record<string, unknown>> | undefined {
|
|
235
236
|
if (
|
|
236
237
|
typeof routeModule === 'function' ||
|
|
@@ -244,14 +245,17 @@ function pickRouteModuleComponent(
|
|
|
244
245
|
if (!routeModule || typeof routeModule !== 'object') {
|
|
245
246
|
return undefined;
|
|
246
247
|
}
|
|
248
|
+
if (seen.has(routeModule)) {
|
|
249
|
+
return undefined;
|
|
250
|
+
}
|
|
251
|
+
seen.add(routeModule);
|
|
247
252
|
|
|
248
253
|
const module = routeModule as ModernRouteModule;
|
|
249
|
-
const
|
|
250
|
-
|
|
251
|
-
|
|
252
|
-
|
|
253
|
-
|
|
254
|
-
return component as ElementType<Record<string, unknown>>;
|
|
254
|
+
for (const candidate of [module.default, module.Component]) {
|
|
255
|
+
const component = pickRouteModuleComponent(candidate, seen);
|
|
256
|
+
if (component) {
|
|
257
|
+
return component;
|
|
258
|
+
}
|
|
255
259
|
}
|
|
256
260
|
|
|
257
261
|
return undefined;
|
|
@@ -26,7 +26,7 @@ type TestRouteObject = RouteObject & {
|
|
|
26
26
|
hasLoader?: boolean;
|
|
27
27
|
inValidSSRRoute?: boolean;
|
|
28
28
|
isClientComponent?: boolean;
|
|
29
|
-
lazyImport?: () => Promise<
|
|
29
|
+
lazyImport?: () => Promise<unknown>;
|
|
30
30
|
};
|
|
31
31
|
|
|
32
32
|
type TestNestedRoute = NestedRoute & {
|
|
@@ -305,6 +305,40 @@ describe('tanstack route tree from RouteObject[]', () => {
|
|
|
305
305
|
).toContain('Lazy child route ready');
|
|
306
306
|
});
|
|
307
307
|
|
|
308
|
+
test('unwraps nested ESM route module defaults before server rendering', async () => {
|
|
309
|
+
const LazyRouteComponent = () =>
|
|
310
|
+
createElement('main', null, 'Nested lazy child route ready');
|
|
311
|
+
const lazyImport = rstest.fn(async () => ({
|
|
312
|
+
default: {
|
|
313
|
+
default: LazyRouteComponent,
|
|
314
|
+
},
|
|
315
|
+
}));
|
|
316
|
+
const routes: TestRouteObject[] = [
|
|
317
|
+
{
|
|
318
|
+
id: 'root',
|
|
319
|
+
path: '/',
|
|
320
|
+
Component: () => createElement('section', null, createElement(Outlet)),
|
|
321
|
+
children: [
|
|
322
|
+
{
|
|
323
|
+
id: 'lazy',
|
|
324
|
+
path: 'lazy',
|
|
325
|
+
Component: lazy(
|
|
326
|
+
lazyImport as () => Promise<{ default: ComponentType }>,
|
|
327
|
+
),
|
|
328
|
+
lazyImport,
|
|
329
|
+
},
|
|
330
|
+
],
|
|
331
|
+
},
|
|
332
|
+
];
|
|
333
|
+
|
|
334
|
+
const routeTree = createRouteTreeFromRouteObjects(routes);
|
|
335
|
+
const router = await loadRouteTree(routeTree, '/lazy');
|
|
336
|
+
|
|
337
|
+
expect(
|
|
338
|
+
renderToStaticMarkup(createElement(RouterProvider, { router } as never)),
|
|
339
|
+
).toContain('Nested lazy child route ready');
|
|
340
|
+
});
|
|
341
|
+
|
|
308
342
|
test('preserves route handle and maps shouldRevalidate to shouldReload', async () => {
|
|
309
343
|
const shouldRevalidate = rstest.fn(({ nextUrl }: ShouldRevalidateArgs) =>
|
|
310
344
|
nextUrl.pathname.endsWith('/456'),
|