Enforce string paths in file path utility

Co-authored-by: Shri Sukhani <shrisukhani@users.noreply.github.com>

Author: cursoragent

hyperbrowser/client/file_utils.py

@@ -18,6 +18,8 @@ def ensure_existing_file_path(
             "file_path must be a string or os.PathLike object",
             original_error=exc,
         ) from exc
+    if not isinstance(normalized_path, str):
+        raise HyperbrowserError("file_path must resolve to a string path")
     if not os.path.exists(normalized_path):
         raise HyperbrowserError(missing_file_message)
     if not os.path.isfile(normalized_path):

tests/test_file_utils.py

@@ -61,3 +61,12 @@ def test_ensure_existing_file_path_rejects_invalid_path_type():
             missing_file_message="missing",
             not_file_message="not-file",
         )
+
+
+def test_ensure_existing_file_path_rejects_non_string_fspath_results():
+    with pytest.raises(HyperbrowserError, match="file_path must resolve to a string"):
+        ensure_existing_file_path(
+            b"/tmp/bytes-path",  # type: ignore[arg-type]
+            missing_file_message="missing",
+            not_file_message="not-file",
+        )