Treat non-alnum fetching separators as idempotent prefixes

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

Author: cursoragent

hyperbrowser/client/polling.py

@@ -136,7 +136,7 @@ def build_fetch_operation_name(operation_name: object) -> str:
         if next_character_index == len(normalized_lower_operation_name):
             return normalized_operation_name
         next_character = normalized_lower_operation_name[next_character_index]
-        if next_character.isspace() or next_character == "?":
+        if not next_character.isalnum():
             return normalized_operation_name
     return build_operation_name(
         _FETCH_OPERATION_NAME_PREFIX,

tests/test_polling.py

@@ -70,6 +70,13 @@ def test_build_fetch_operation_name_is_idempotent_for_sanitized_fetch_separator(
     assert normalized_operation_name.count("Fetching") == 1
 
 
+def test_build_fetch_operation_name_is_idempotent_for_punctuation_separator():
+    operation_name = "Fetching...crawl job 123"
+    normalized_operation_name = build_fetch_operation_name(operation_name)
+
+    assert normalized_operation_name == operation_name
+
+
 def test_build_fetch_operation_name_prefixes_non_separator_fetching_variants():
     operation_name = "FetchingTask"
     normalized_operation_name = build_fetch_operation_name(operation_name)