fix: Don't skip required undefined properties on object

Author: orimay

src/value/default/from-object.ts

@@ -29,9 +29,8 @@ THE SOFTWARE.
 // deno-fmt-ignore-file
 // deno-lint-ignore-file
 
-import type { TProperties, TObject } from '../../type/index.ts'
+import { type TProperties, type TObject, IsOptional } from '../../type/index.ts'
 import { Guard } from '../../guard/index.ts'
-import { FromDefault } from './from-default.ts'
 import { FromType } from './from-type.ts'
 
 import { IsAdditionalProperties } from '../../schema/types/index.ts'
@@ -46,7 +45,7 @@ export function FromObject(context: TProperties, type: TObject, value: unknown):
     // yielded a non undefined result. Here we interpret an undefined result as
     // a non assignable property and continue.
     const propertyValue = FromType(context, type.properties[key], value[key])
-    if (Guard.IsUndefined(propertyValue)) continue
+    if (Guard.IsUndefined(propertyValue) && (IsOptional(type.properties[key]) || !('default' in type.properties[key]))) continue
     value[key] = FromType(context, type.properties[key], value[key])
   }
   // return if not additional properties

test/typebox/runtime/value/default/object.ts

@@ -260,3 +260,13 @@ Test('Should Default 22', () => {
   const R = Value.Default(X, { x: 2 })
   Assert.IsEqual(R, { x: 2 })
 })
+Test('Should Default 23', () => {
+  const X = Type.Object({ x: Type.Undefined({ default: undefined }) })
+  const R = Value.Default(X, {})
+  Assert.IsEqual(R, { x: undefined })
+})
+Test('Should Default 24', () => {
+  const X = Type.Object({ x: Type.Optional({ default: undefined }) })
+  const R = Value.Default(X, {})
+  Assert.IsEqual(R, {})
+})