removed duplicated get_auto_default_for_type, kept in engine_value and imported same in engine_object

Author: Davda-James

python/cocoindex/engine_object.py

@@ -17,7 +17,6 @@
     AnalyzedDictType,
     AnalyzedListType,
     AnalyzedStructType,
-    AnalyzedTypeInfo,
     AnalyzedUnionType,
     EnrichedValueType,
     FieldSchema,
@@ -27,6 +26,7 @@
     is_pydantic_model,
     extract_ndarray_elem_dtype,
 )
+from .engine_value import get_auto_default_for_type
 
 
 T = TypeVar("T")
@@ -37,30 +37,6 @@
     pass
 
 
-def _get_auto_default_for_type(
-    type_info: AnalyzedTypeInfo,
-) -> tuple[Any, bool]:
-    """
-    Get an auto-default value for a type annotation if it's safe to do so.
-
-    Returns:
-        A tuple of (default_value, is_supported) where:
-        - default_value: The default value if auto-defaulting is supported
-        - is_supported: True if auto-defaulting is supported for this type
-    """
-    # Case 1: Nullable types (Optional[T] or T | None)
-    if type_info.nullable:
-        return None, True
-
-    # Case 2: Table types (KTable or LTable) - check if it's a list or dict type
-    if isinstance(type_info.variant, AnalyzedListType):
-        return [], True
-    elif isinstance(type_info.variant, AnalyzedDictType):
-        return {}, True
-
-    return None, False
-
-
 def dump_engine_object(v: Any) -> Any:
     """Recursively dump an object for engine. Engine side uses `Pythonized` to catch."""
     if v is None:
@@ -233,7 +209,7 @@ def load_engine_object(expected_type: Any, v: Any) -> Any:
 
         for name, f_type in missing_fields:
             type_info = analyze_type_info(f_type)
-            auto_default, is_supported = _get_auto_default_for_type(type_info)
+            auto_default, is_supported = get_auto_default_for_type(type_info)
             if is_supported:
                 init_kwargs[name] = auto_default
         return struct_type(**init_kwargs)

python/cocoindex/engine_value.py

@@ -439,7 +439,7 @@ def decode_scalar(value: Any) -> Any | None:
     return lambda value: value
 
 
-def _get_auto_default_for_type(
+def get_auto_default_for_type(
     type_info: AnalyzedTypeInfo,
 ) -> tuple[Any, bool]:
     """
@@ -557,7 +557,7 @@ def make_closure_for_field(
             if default_value is not inspect.Parameter.empty:
                 return lambda _: default_value
 
-            auto_default, is_supported = _get_auto_default_for_type(type_info)
+            auto_default, is_supported = get_auto_default_for_type(type_info)
             if is_supported:
                 warnings.warn(
                     f"Field '{name}' (type {param.annotation}) without default value is missing in input: "