Enforce manager parse helper import boundary

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

Author: cursoragent

CONTRIBUTING.md

@@ -121,6 +121,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `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_helper_import_boundary.py` (manager helper-import boundary enforcement for low-level parse modules),
   - `tests/test_manager_model_dump_usage.py` (manager serialization centralization),
   - `tests/test_manager_parse_boundary.py` (manager response-parse boundary enforcement through shared helper modules),
   - `tests/test_manager_transport_boundary.py` (manager transport boundary enforcement through shared request helpers),

tests/test_architecture_marker_usage.py

@@ -16,6 +16,7 @@
     "tests/test_agent_terminal_status_helper_usage.py",
     "tests/test_guardrail_ast_utils.py",
     "tests/test_manager_model_dump_usage.py",
+    "tests/test_manager_helper_import_boundary.py",
     "tests/test_manager_parse_boundary.py",
     "tests/test_manager_transport_boundary.py",
     "tests/test_mapping_reader_usage.py",

tests/test_manager_helper_import_boundary.py

@@ -0,0 +1,35 @@
+from pathlib import Path
+
+import pytest
+
+pytestmark = pytest.mark.architecture
+
+
+MANAGER_DIRECTORIES = (
+    Path("hyperbrowser/client/managers/sync_manager"),
+    Path("hyperbrowser/client/managers/async_manager"),
+)
+
+DISALLOWED_IMPORT_MARKERS = (
+    "response_utils import",
+    "session_utils import",
+)
+
+
+def test_managers_do_not_import_low_level_parse_modules():
+    violating_modules: list[str] = []
+    for manager_dir in MANAGER_DIRECTORIES:
+        for module_path in sorted(manager_dir.glob("*.py")):
+            module_text = module_path.read_text(encoding="utf-8")
+            if any(marker in module_text for marker in DISALLOWED_IMPORT_MARKERS):
+                violating_modules.append(module_path.as_posix())
+
+        for nested_dir in sorted(
+            path for path in manager_dir.iterdir() if path.is_dir()
+        ):
+            for module_path in sorted(nested_dir.glob("*.py")):
+                module_text = module_path.read_text(encoding="utf-8")
+                if any(marker in module_text for marker in DISALLOWED_IMPORT_MARKERS):
+                    violating_modules.append(module_path.as_posix())
+
+    assert violating_modules == []