refactor: improve PathData logging and type hints

- Add debug logs for set_value and get_value methods
- Fix type hint for create_value_fn using Callable from typing
- Format function signature for better readability

The debug logs follow the Helm plugin convention and provide
better observability for value operations. Type hints now
properly specify the function signature for create_value_fn.

Author: ritwik-g

helm_values_manager/models/path_data.py

@@ -5,7 +5,7 @@
 metadata and values associated with a specific configuration path.
 """
 
-from typing import Any, Dict, Iterator, Optional
+from typing import Any, Callable, Dict, Iterator, Optional
 
 from helm_values_manager.models.value import Value
 from helm_values_manager.utils.logger import logger
@@ -61,6 +61,7 @@ def set_value(self, environment: str, value: Value) -> None:
             environment (str): The environment to set the value for
             value (Value): The Value object to set
         """
+        logger.debug("Setting value for path %s in environment %s", self.path, environment)
         self._values[environment] = value
 
     def get_value(self, environment: str, resolve: bool = False) -> Optional[str]:
@@ -74,8 +75,10 @@ def get_value(self, environment: str, resolve: bool = False) -> Optional[str]:
         Returns:
             Optional[str]: The value if found, None otherwise
         """
+        logger.debug("Getting value for path %s in environment %s", self.path, environment)
         value = self._values.get(environment)
         if value is None:
+            logger.debug("No value found for path %s in environment %s", self.path, environment)
             return None
 
         return value.get(resolve=resolve)
@@ -103,13 +106,15 @@ def to_dict(self) -> Dict[str, Any]:
         }
 
     @classmethod
-    def from_dict(cls, data: Dict[str, Any], create_value_fn: callable) -> "PathData":
+    def from_dict(
+        cls, data: Dict[str, Any], create_value_fn: Callable[[str, str, Dict[str, Any]], Value]
+    ) -> "PathData":
         """
         Create a PathData instance from a dictionary.
 
         Args:
             data (Dict[str, Any]): Dictionary containing PathData data
-            create_value_fn: Function to create Value instances
+            create_value_fn: Function to create Value instances. Takes (path, env, value_data) and returns Value
 
         Returns:
             PathData: New PathData instance