Skip to content

Commit 55d38c8

Browse files
Deserialize each route once when loading the manifest (#17415)
deserializeManifest deserialized every route twice: once for the returned routes array, and again onto the input object, which nothing reads afterwards. Both callers pass a fresh manifest they never read back, and the returned routes are separate copies, so dropping the second pass does not change behavior. Co-authored-by: Matthew Phillips <matthewphillips@cloudflare.com>
1 parent 89e5349 commit 55d38c8

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'astro': patch
3+
---
4+
5+
Deserializes each route once when loading the SSR manifest

packages/astro/src/core/app/manifest.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,6 @@ export function deserializeManifest(
2323
...serializedRoute,
2424
routeData: deserializeRouteData(serializedRoute.routeData),
2525
});
26-
27-
const route = serializedRoute as unknown as RouteInfo;
28-
route.routeData = deserializeRouteData(serializedRoute.routeData);
2926
}
3027
}
3128
if (routesList) {

0 commit comments

Comments
 (0)