Make fetch operation-name builder idempotent for prefixed labels

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

Author: cursoragent

hyperbrowser/client/polling.py

@@ -124,7 +124,13 @@ def build_operation_name(prefix: object, identifier: object) -> str:
 
 
 def build_fetch_operation_name(operation_name: object) -> str:
-    return build_operation_name(_FETCH_OPERATION_NAME_PREFIX, operation_name)
+    normalized_operation_name = build_operation_name("", operation_name)
+    if normalized_operation_name.startswith(_FETCH_OPERATION_NAME_PREFIX):
+        return normalized_operation_name
+    return build_operation_name(
+        _FETCH_OPERATION_NAME_PREFIX,
+        normalized_operation_name,
+    )
 
 
 def _ensure_boolean_terminal_result(result: object, *, operation_name: str) -> bool:

tests/test_polling.py

@@ -47,6 +47,11 @@ def test_build_fetch_operation_name_prefixes_when_within_length_limit():
     assert build_fetch_operation_name("crawl job 123") == "Fetching crawl job 123"
 
 
+def test_build_fetch_operation_name_is_idempotent_for_prefixed_inputs():
+    operation_name = "Fetching crawl job 123"
+    assert build_fetch_operation_name(operation_name) == operation_name
+
+
 def test_build_fetch_operation_name_truncates_to_preserve_fetch_prefix():
     operation_name = "x" * 200
     fetch_operation_name = build_fetch_operation_name(operation_name)
@@ -71,6 +76,12 @@ def test_build_fetch_operation_name_bounds_very_large_operation_names():
     assert len(operation_name) == 200
 
 
+def test_build_fetch_operation_name_is_idempotent_for_bounded_prefixed_inputs():
+    bounded_prefixed_name = "Fetching " + ("x" * 188) + "..."
+    operation_name = build_fetch_operation_name(bounded_prefixed_name)
+    assert operation_name == bounded_prefixed_name
+
+
 def test_build_fetch_operation_name_handles_unstringifiable_input():
     class _BadOperationName:
         def __str__(self) -> str: