Raise error on providing absolute path to local files

Author: AlePouroullis

.gitignore

@@ -5,7 +5,7 @@ poetry.toml
 .ruff_cache/
 .vscode
 .env
-.env.test
 tests/assets/*.jsonl
 tests/assets/*.parquet
+# Ignore humanloop directory which could mistakenly be committed when testing sync functionality as it's used as the default sync directory
 humanloop

src/humanloop/cli/__main__.py

@@ -64,7 +64,7 @@ def load_api_key(env_file: Optional[str] = None) -> str:
 def get_client(
     api_key: Optional[str] = None, env_file: Optional[str] = None, base_url: Optional[str] = None
 ) -> Humanloop:
-    """Get a Humanloop client instance.
+    """Instantiate a Humanloop client for the CLI.
 
     Args:
         api_key: Optional API key provided directly

src/humanloop/overload.py

@@ -108,6 +108,9 @@ def _handle_local_files(
         )
 
     file_type = _get_file_type_from_client(client)
+    if file_type not in SyncClient.SERIALIZABLE_FILE_TYPES:
+        raise HumanloopRuntimeError(f"Local files are not supported for `{file_type}` files.")
+
     # If file_type is already specified in kwargs, it means user provided a PromptKernelRequestParams object
     if file_type in kwargs and not isinstance(kwargs[file_type], str):
         logger.warning(
@@ -117,7 +120,7 @@ def _handle_local_files(
         return kwargs
 
     try:
-        file_content = sync_client.get_file_content(normalized_path, file_type)
+        file_content = sync_client.get_file_content(normalized_path, file_type)  # type: ignore [arg-type] file_type was checked above
         kwargs[file_type] = file_content
     except HumanloopRuntimeError as e:
         raise HumanloopRuntimeError(f"Failed to use local file for `{normalized_path}`: {str(e)}")
@@ -186,7 +189,7 @@ def overload_client(
     use_local_files: bool = False,
 ) -> Any:
     """Overloads client methods to add tracing, local file handling, and evaluation context."""
-    # Store original log method as _log for all clients. Used in flow decorator 
+    # Store original log method as _log for all clients. Used in flow decorator
     if hasattr(client, "log") and not hasattr(client, "_log"):
         client._log = client.log  # type: ignore [attr-defined]
 

src/humanloop/sync/sync_client.py

@@ -3,7 +3,6 @@
 from typing import List, Tuple, TYPE_CHECKING
 from functools import lru_cache
 import typing
-from humanloop.types import FileType
 import time
 from humanloop.error import HumanloopRuntimeError
 import json
@@ -164,6 +163,14 @@ def _normalize_path(self, path: str) -> str:
         # Convert to Path object to handle platform-specific separators
         path_obj = Path(path)
 
+        # Paths are considered absolute on unix-like systems if they start with a forward slash.
+        # This is because we want to ensure seamless toggling between the local and remote filesystems.
+        if path_obj.is_absolute():
+            raise HumanloopRuntimeError(
+                f"Absolute paths are not supported: `{path}`. "
+                f"Paths should be relative to the base directory (`{self.base_dir}`)."
+            )
+
         # Remove extension, convert to string with forward slashes, and remove leading/trailing slashes
         normalized = str(path_obj.with_suffix(""))
         # Replace all backslashes and normalize multiple forward slashes
@@ -208,7 +215,7 @@ def _pull_file(self, path: str | None = None, environment: str | None = None) ->
                 environment=environment,
                 include_raw_file_content=True,
             )
-            
+
             if file.type not in self.SERIALIZABLE_FILE_TYPES:
                 logger.error(f"Unsupported file type: {file.type}")
                 return False
@@ -237,7 +244,7 @@ def _pull_directory(
         Returns:
             Tuple of two lists:
             - First list contains paths of successfully synced files
-            - Second list contains paths of files that failed to sync. 
+            - Second list contains paths of files that failed to sync.
               Failures can occur due to missing content in the response or errors during local file writing.
 
         Raises: