fix: eagerly initialise dependencies cache (#47)

danielroe · web-flow · commit 7b72ba244c18 · 2023-03-22T18:44:37.000+01:00
diff --git a/src/runtime.ts b/src/runtime.ts
@@ -79,7 +79,7 @@ export function getModuleDependencies (id: string, rendererContext: RendererCont
     return rendererContext._dependencies[id]
   }
 
-  const dependencies: ModuleDependencies = {
+  const dependencies: ModuleDependencies = rendererContext._dependencies[id] = {
     scripts: {},
     styles: {},
     preload: {},
@@ -89,7 +89,6 @@ export function getModuleDependencies (id: string, rendererContext: RendererCont
   const meta = rendererContext.manifest[id]
 
   if (!meta) {
-    rendererContext._dependencies[id] = dependencies
     return dependencies
   }
 
@@ -122,7 +121,6 @@ export function getModuleDependencies (id: string, rendererContext: RendererCont
   }
   dependencies.preload = filteredPreload
 
-  rendererContext._dependencies[id] = dependencies
   return dependencies
 }
 
@@ -255,7 +253,7 @@ export function createRenderer (createApp: any, renderOptions: RenderOptions & {
       const app = await _createApp(ssrContext)
       const html = await renderOptions.renderToString(app, ssrContext)
 
-      const wrap = <T extends RenderFunction>(fn: T) => () => fn(ssrContext, rendererContext) as ReturnType<T>
+      const wrap = <T extends RenderFunction> (fn: T) => () => fn(ssrContext, rendererContext) as ReturnType<T>
 
       return {
         html,

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ export function getModuleDependencies (id: string, rendererContext: RendererCont`
`79`	`79`	`return rendererContext._dependencies[id]`
`80`	`80`	`}`
`81`	`81`
`82`		`- const dependencies: ModuleDependencies = {`
	`82`	`+ const dependencies: ModuleDependencies = rendererContext._dependencies[id] = {`
`83`	`83`	`scripts: {},`
`84`	`84`	`styles: {},`
`85`	`85`	`preload: {},`
`@@ -89,7 +89,6 @@ export function getModuleDependencies (id: string, rendererContext: RendererCont`
`89`	`89`	`const meta = rendererContext.manifest[id]`
`90`	`90`
`91`	`91`	`if (!meta) {`
`92`		`- rendererContext._dependencies[id] = dependencies`
`93`	`92`	`return dependencies`
`94`	`93`	`}`
`95`	`94`
`@@ -122,7 +121,6 @@ export function getModuleDependencies (id: string, rendererContext: RendererCont`
`122`	`121`	`}`
`123`	`122`	`dependencies.preload = filteredPreload`
`124`	`123`
`125`		`- rendererContext._dependencies[id] = dependencies`
`126`	`124`	`return dependencies`
`127`	`125`	`}`
`128`	`126`
`@@ -255,7 +253,7 @@ export function createRenderer (createApp: any, renderOptions: RenderOptions & {`
`255`	`253`	`const app = await _createApp(ssrContext)`
`256`	`254`	`const html = await renderOptions.renderToString(app, ssrContext)`
`257`	`255`
`258`		`- const wrap = <T extends RenderFunction>(fn: T) => () => fn(ssrContext, rendererContext) as ReturnType<T>`
	`256`	`+ const wrap = <T extends RenderFunction> (fn: T) => () => fn(ssrContext, rendererContext) as ReturnType<T>`
`259`	`257`
`260`	`258`	`return {`
`261`	`259`	`html,`