fh-pydantic-form 0.3.3__tar.gz → 0.3.4__tar.gz

{fh_pydantic_form-0.3.3 → fh_pydantic_form-0.3.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fh-pydantic-form
-Version: 0.3.3
+Version: 0.3.4
 Summary: a library to turn any pydantic BaseModel object into a fasthtml/monsterui input form
 Project-URL: Homepage, https://github.com/Marcura/fh-pydantic-form
 Project-URL: Repository, https://github.com/Marcura/fh-pydantic-form

{fh_pydantic_form-0.3.3 → fh_pydantic_form-0.3.4}/RELEASE_NOTES.md

@@ -1,5 +1,9 @@
 # Release Notes
 
+## Version 0.3.4 (2025-07-15)
+
+- **NEW**: Added support for Optional[List[..]] types in form fields
+
 ## Version 0.3.3 (2025-07-09)
 
 - fix bug where label_color was not passed down in ComparisonForm

{fh_pydantic_form-0.3.3 → fh_pydantic_form-0.3.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "fh-pydantic-form"
-version = "0.3.3"
+version = "0.3.4"
 description = "a library to turn any pydantic BaseModel object into a fasthtml/monsterui input form"
 readme = "README.md"
 requires-python = ">=3.10"

{fh_pydantic_form-0.3.3 → fh_pydantic_form-0.3.4}/src/fh_pydantic_form/__init__.py

@@ -87,13 +87,16 @@ def register_default_renderers() -> None:
 
     # Register list renderer for List[*] types
     def is_list_field(field_info):
-        """Check if field is a list type"""
+        """Check if field is a list type, including Optional[List[...]]"""
         annotation = getattr(field_info, "annotation", None)
-        return (
-            annotation is not None
-            and hasattr(annotation, "__origin__")
-            and annotation.__origin__ is list
-        )
+        if annotation is None:
+            return False
+
+        # Handle Optional[List[...]] by unwrapping the Optional
+        underlying_type = _get_underlying_type_if_optional(annotation)
+
+        # Check if the underlying type is a list
+        return get_origin(underlying_type) is list
 
     FieldRendererRegistry.register_type_renderer_with_predicate(
         is_list_field, ListFieldRenderer

{fh_pydantic_form-0.3.3 → fh_pydantic_form-0.3.4}/src/fh_pydantic_form/field_renderers.py

@@ -1523,12 +1523,15 @@ class ListFieldRenderer(BaseFieldRenderer):
         annotation = getattr(self.field_info, "annotation", None)
         item_type = None  # Initialize here to avoid UnboundLocalError
 
+        # Handle Optional[List[...]] by unwrapping the Optional first
+        base_annotation = _get_underlying_type_if_optional(annotation)
+
         if (
-            annotation is not None
-            and hasattr(annotation, "__origin__")
-            and annotation.__origin__ is list
+            base_annotation is not None
+            and hasattr(base_annotation, "__origin__")
+            and base_annotation.__origin__ is list
         ):
-            item_type = annotation.__args__[0]
+            item_type = base_annotation.__args__[0]
 
         if not item_type:
             logger.error(f"Cannot determine item type for list field {self.field_name}")
@@ -1600,10 +1603,20 @@ class ListFieldRenderer(BaseFieldRenderer):
             if self.disabled:
                 add_button_attrs["disabled"] = "true"
 
+            # Differentiate message for Optional[List] vs required List
+            if self.is_optional:
+                empty_message = (
+                    "No items in this optional list. Click 'Add Item' if needed."
+                )
+            else:
+                empty_message = (
+                    "No items in this required list. Click 'Add Item' to create one."
+                )
+
             empty_state = mui.Alert(
                 fh.Div(
                     mui.UkIcon("info", cls="mr-2"),
-                    "No items in this list. Click 'Add Item' to create one.",
+                    empty_message,
                     mui.Button("Add Item", **add_button_attrs),
                     cls="flex flex-col items-start",
                 ),

{fh_pydantic_form-0.3.3 → fh_pydantic_form-0.3.4}/src/fh_pydantic_form/form_parser.py

@@ -7,8 +7,8 @@ from typing import (
     Optional,
     Tuple,
     Union,
-    get_origin,
     get_args,
+    get_origin,
 )
 
 from fh_pydantic_form.type_helpers import (
@@ -438,7 +438,7 @@ def _parse_list_fields(
     list_field_defs: Dict[str, Dict[str, Any]],
     base_prefix: str = "",
     exclude_fields: Optional[List[str]] = None,
-) -> Dict[str, List[Any]]:
+) -> Dict[str, Optional[List[Any]]]:
     """
     Parse list fields from form data by analyzing keys and reconstructing ordered lists.
 
@@ -490,7 +490,7 @@ def _parse_list_fields(
                 list_items_temp[field_name][idx_str][subfield] = value
 
     # Build final lists based on tracked order
-    final_lists = {}
+    final_lists: Dict[str, Optional[List[Any]]] = {}
     for field_name, ordered_indices in list_item_indices_ordered.items():
         field_def = list_field_defs[field_name]
         item_type = field_def["item_type"]
@@ -544,8 +544,12 @@ def _parse_list_fields(
         if field_name in final_lists:
             continue
 
-        # User submitted form with zero items → honour intent with empty list
-        final_lists[field_name] = []
+        # User submitted form with zero items → honour intent with None for Optional[List]
+        field_info = field_def["field_info"]
+        if _is_optional_type(field_info.annotation):
+            final_lists[field_name] = None  # Use None for empty Optional[List]
+        else:
+            final_lists[field_name] = []  # Regular empty list for required fields
 
     return final_lists
 

{fh_pydantic_form-0.3.3 → fh_pydantic_form-0.3.4}/src/fh_pydantic_form/form_renderer.py

@@ -435,7 +435,7 @@ class PydanticForm(Generic[ModelType]):
                     try:
                         default_factory = field_info.default_factory
                         if callable(default_factory):
-                            initial_value = default_factory()
+                            initial_value = default_factory()  # type: ignore[call-arg]
                         else:
                             initial_value = None
                             logger.warning(