Skip to content

Commit 9d9d53e

Browse files
authored
refactor: use the official version of react-router (#7612)
1 parent 7bf5150 commit 9d9d53e

File tree

17 files changed

+38
-72
lines changed

17 files changed

+38
-72
lines changed

packages/runtime/plugin-runtime/package.json

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -94,13 +94,13 @@
9494
"./router": {
9595
"types": "./dist/types/router/index.d.ts",
9696
"jsnext:source": "./src/router/index.ts",
97+
"react-server": {
98+
"types": "./dist/types/router/runtime/rsc.d.ts",
99+
"jsnext:source": "./src/router/runtime/rsc.ts",
100+
"default": "./dist/esm/router/runtime/rsc.js"
101+
},
97102
"default": "./dist/esm/router/index.js"
98103
},
99-
"./router/rsc": {
100-
"types": "./dist/types/router/runtime/rsc.d.ts",
101-
"jsnext:source": "./src/router/runtime/rsc.ts",
102-
"default": "./dist/esm/router/runtime/rsc.js"
103-
},
104104
"./router/server": {
105105
"types": "./dist/types/router/runtime/server.d.ts",
106106
"jsnext:source": "./src/router/runtime/server.ts",
@@ -111,6 +111,11 @@
111111
"jsnext:source": "./src/router/internal.ts",
112112
"default": "./dist/esm/router/internal.js"
113113
},
114+
"./routerHelper": {
115+
"types": "./dist/types/router/helper.d.ts",
116+
"jsnext:source": "./src/router/runtime/routerHelper.ts",
117+
"default": "./dist/esm/router/runtime/routerHelper.js"
118+
},
114119
"./loadable-bundler-plugin": {
115120
"types": "./dist/types/cli/ssr/loadable-bundler-plugin.d.ts",
116121
"jsnext:source": "./src/cli/ssr/loadable-bundler-plugin.ts",

packages/runtime/plugin-runtime/src/core/browser/hydrate.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,7 @@ export function hydrateRoot(
5959

6060
function stringSSRHydrate() {
6161
// client render and server prefetch use same logic
62-
if (
63-
renderLevel === RenderLevel.CLIENT_RENDER ||
64-
renderLevel === RenderLevel.SERVER_PREFETCH
65-
) {
62+
if (renderLevel === RenderLevel.CLIENT_RENDER) {
6663
return ModernRender(wrapRuntimeContextProvider(App, context));
6764
} else if (renderLevel === RenderLevel.SERVER_RENDER) {
6865
return new Promise<Root | HTMLElement>(resolve => {

packages/runtime/plugin-runtime/src/core/constants.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
export enum RenderLevel {
22
CLIENT_RENDER = 0,
3-
SERVER_PREFETCH = 1,
43
SERVER_RENDER = 2,
54
}
65

packages/runtime/plugin-runtime/src/router/cli/code/templates.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -469,7 +469,7 @@ export const fileSystemRoutes = async ({
469469
await fs.writeJSON(loadersMapFile, loadersMap);
470470

471471
const importRuntimeRouterCode = `
472-
import { createShouldRevalidate, handleRouteModule, handleRouteModuleError} from '@${metaName}/runtime/router';
472+
import { createShouldRevalidate, handleRouteModule, handleRouteModuleError} from '@${metaName}/runtime/routerHelper';
473473
`;
474474
const routeModulesCode = `
475475
if(typeof document !== 'undefined'){

packages/runtime/plugin-runtime/src/router/runtime/index.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,6 @@ export const useRouteLoaderData: typeof useRouteData = (routeId: string) => {
1010
return useRouteData(realRouteId);
1111
};
1212

13-
export {
14-
createShouldRevalidate,
15-
handleRouteModule,
16-
handleRouteModuleError,
17-
} from './routeModule';
18-
1913
export * from './withRouter';
2014

2115
export type { LoaderFunction, LoaderFunctionArgs } from './types';
File renamed without changes.

packages/toolkit/runtime-utils/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,7 +224,7 @@
224224
"@modern-js/utils": "workspace:*",
225225
"@swc/helpers": "^0.5.17",
226226
"lru-cache": "^10.4.3",
227-
"react-router": "0.0.0-experimental-3f68c7c20",
227+
"react-router": "7.8.2",
228228
"serialize-javascript": "^6.0.2"
229229
},
230230
"peerDependencies": {

packages/toolkit/runtime-utils/src/remixRouter.ts

Lines changed: 0 additions & 3 deletions
This file was deleted.
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1-
export * from 'react-router/rsc';
1+
// Here import the server APIs for the react-router
2+
export * from 'react-router';

pnpm-lock.yaml

Lines changed: 18 additions & 42 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)