fix: handle optional prop with default

vuejs · Oct 22, 2024 · 4a3bc4e · 4a3bc4e
1 parent 15c6f92
commit 4a3bc4e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/packages-private/dts-test/setupHelpers.test-d.ts b/packages-private/dts-test/setupHelpers.test-d.ts
@@ -456,6 +456,11 @@ describe('defineModel', () => {
     defineModel<() => number>({
       default: () => '',
     })
+    // @ts-expect-error
+    // optional prop with default
+    defineModel<{ id?: number }>({
+      default: () => ({ foo: 'bar' }),
+    })
   }
 
   // @ts-expect-error unknown props option

diff --git a/packages/runtime-core/src/apiSetupHelpers.ts b/packages/runtime-core/src/apiSetupHelpers.ts
@@ -285,7 +285,7 @@ export type DefineModelOptions<T = any, G = T, S = T> = {
  * ```
  */
 export function defineModel<T, M extends PropertyKey = string, G = T, S = T>(
-  options: ({ default: any } | { required: true }) &
+  options: ({ default: NonNullable<unknown> } | { required: true }) &
     PropOptions<T> &
     DefineModelOptions<T, G, S>,
 ): ModelRef<T, M, G, S>
@@ -296,7 +296,7 @@ export function defineModel<T, M extends PropertyKey = string, G = T, S = T>(
 
 export function defineModel<T, M extends PropertyKey = string, G = T, S = T>(
   name: string,
-  options: ({ default: any } | { required: true }) &
+  options: ({ default: NonNullable<unknown> } | { required: true }) &
     PropOptions<T> &
     DefineModelOptions<T, G, S>,
 ): ModelRef<T, M, G, S>