fix(kit): add default values for destructured references in addTypeTemplate hooks

hillaryke · hillaryke · commit 6e3b606eebc2 · 2025-09-15T17:47:24.000+03:00
Fixes issue where nodeReferences, sharedReferences, and references arrays
could be undefined when destructured in prepare:types and nitro:prepare:types
hooks, causing 'Cannot read properties of undefined (reading "push")' errors.

This ensures backward compatibility by providing empty arrays as defaults
when the hook payload doesn't contain these properties.
diff --git a/packages/kit/src/template.ts b/packages/kit/src/template.ts
@@ -80,17 +80,17 @@ export function addTypeTemplate<T> (_template: NuxtTypeTemplate<T>, context?: {
 
   // Add template to types reference
   if (!context || context.nuxt) {
-    nuxt.hook('prepare:types', ({ references }) => {
+    nuxt.hook('prepare:types', ({ references = [] }) => {
       references.push({ path: template.dst })
     })
   }
   if (context?.node) {
-    nuxt.hook('prepare:types', ({ nodeReferences }) => {
+    nuxt.hook('prepare:types', ({ nodeReferences = [] }) => {
       nodeReferences.push({ path: template.dst })
     })
   }
   if (context?.shared) {
-    nuxt.hook('prepare:types', ({ sharedReferences }) => {
+    nuxt.hook('prepare:types', ({ sharedReferences = [] }) => {
       sharedReferences.push({ path: template.dst })
     })
   }
@@ -105,7 +105,7 @@ export function addTypeTemplate<T> (_template: NuxtTypeTemplate<T>, context?: {
   }
 
   if (context?.nitro) {
-    nuxt.hook('nitro:prepare:types', ({ references }) => {
+    nuxt.hook('nitro:prepare:types', ({ references = [] }) => {
       references.push({ path: template.dst })
     })
   }

Original file line number	Diff line number	Diff line change
`@@ -80,17 +80,17 @@ export function addTypeTemplate<T> (_template: NuxtTypeTemplate<T>, context?: {`
`80`	`80`
`81`	`81`	`// Add template to types reference`
`82`	`82`	`if (!context \|\| context.nuxt) {`
`83`		`- nuxt.hook('prepare:types', ({ references }) => {`
	`83`	`+ nuxt.hook('prepare:types', ({ references = [] }) => {`
`84`	`84`	`references.push({ path: template.dst })`
`85`	`85`	`})`
`86`	`86`	`}`
`87`	`87`	`if (context?.node) {`
`88`		`- nuxt.hook('prepare:types', ({ nodeReferences }) => {`
	`88`	`+ nuxt.hook('prepare:types', ({ nodeReferences = [] }) => {`
`89`	`89`	`nodeReferences.push({ path: template.dst })`
`90`	`90`	`})`
`91`	`91`	`}`
`92`	`92`	`if (context?.shared) {`
`93`		`- nuxt.hook('prepare:types', ({ sharedReferences }) => {`
	`93`	`+ nuxt.hook('prepare:types', ({ sharedReferences = [] }) => {`
`94`	`94`	`sharedReferences.push({ path: template.dst })`
`95`	`95`	`})`
`96`	`96`	`}`
`@@ -105,7 +105,7 @@ export function addTypeTemplate<T> (_template: NuxtTypeTemplate<T>, context?: {`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`if (context?.nitro) {`
`108`		`- nuxt.hook('nitro:prepare:types', ({ references }) => {`
	`108`	`+ nuxt.hook('nitro:prepare:types', ({ references = [] }) => {`
`109`	`109`	`references.push({ path: template.dst })`
`110`	`110`	`})`
`111`	`111`	`}`