Normalize trailing prefix whitespace in operation-name builder

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

Author: cursoragent

hyperbrowser/client/polling.py

@@ -83,6 +83,12 @@ def build_operation_name(prefix: object, identifier: object) -> str:
         for character in normalized_prefix
     )
     normalized_prefix = normalized_prefix.lstrip()
+    has_trailing_whitespace = (
+        bool(normalized_prefix) and normalized_prefix[-1].isspace()
+    )
+    normalized_prefix = normalized_prefix.rstrip()
+    if has_trailing_whitespace and normalized_prefix:
+        normalized_prefix = f"{normalized_prefix} "
     raw_identifier = _coerce_operation_name_component(identifier, fallback="unknown")
     normalized_identifier = raw_identifier.strip()
     if not normalized_identifier:

tests/test_polling.py

@@ -172,6 +172,24 @@ def test_build_operation_name_trims_leading_whitespace_from_prefix_text():
     assert operation_name == "crawl job 123"
 
 
+def test_build_operation_name_collapses_trailing_prefix_whitespace():
+    operation_name = build_operation_name(
+        "crawl job    ",
+        "123",
+    )
+
+    assert operation_name == "crawl job 123"
+
+
+def test_build_operation_name_handles_large_trailing_prefix_whitespace():
+    operation_name = build_operation_name(
+        "a" + (" " * 500),
+        "b",
+    )
+
+    assert operation_name == "a b"
+
+
 def test_build_operation_name_uses_full_length_budget_after_prefix_lstrip():
     operation_name = build_operation_name(
         "   crawl job ",