Enforce wait-for-job helper primitive boundary

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

Author: cursoragent

CONTRIBUTING.md

@@ -99,6 +99,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `tests/test_extract_payload_helper_usage.py` (extract start-payload helper usage enforcement),
   - `tests/test_guardrail_ast_utils.py` (shared AST guard utility contract),
   - `tests/test_job_pagination_helper_usage.py` (shared scrape/crawl pagination helper usage enforcement),
+  - `tests/test_job_wait_helper_boundary.py` (centralization boundary enforcement for wait-for-job helper primitives),
   - `tests/test_job_wait_helper_usage.py` (shared wait-for-job defaults helper usage enforcement),
   - `tests/test_makefile_quality_targets.py` (Makefile quality-gate target enforcement),
   - `tests/test_manager_model_dump_usage.py` (manager serialization centralization),

tests/test_architecture_marker_usage.py

@@ -39,6 +39,7 @@
     "tests/test_extract_payload_helper_usage.py",
     "tests/test_examples_naming_convention.py",
     "tests/test_job_pagination_helper_usage.py",
+    "tests/test_job_wait_helper_boundary.py",
     "tests/test_job_wait_helper_usage.py",
     "tests/test_example_sync_async_parity.py",
     "tests/test_example_run_instructions.py",

tests/test_job_wait_helper_boundary.py

@@ -0,0 +1,20 @@
+from pathlib import Path
+
+import pytest
+
+pytestmark = pytest.mark.architecture
+
+
+MANAGERS_DIR = Path("hyperbrowser/client/managers")
+
+
+def test_wait_for_job_result_primitives_are_centralized():
+    violating_modules: list[str] = []
+    for module_path in sorted(MANAGERS_DIR.rglob("*.py")):
+        if module_path.name in {"__init__.py", "job_wait_utils.py"}:
+            continue
+        module_text = module_path.read_text(encoding="utf-8")
+        if "wait_for_job_result(" in module_text or "wait_for_job_result_async(" in module_text:
+            violating_modules.append(module_path.as_posix())
+
+    assert violating_modules == []