Enforce manager parse helper boundary

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

Author: cursoragent

CONTRIBUTING.md

@@ -122,6 +122,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `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_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_mapping_keys_access_usage.py` (centralized key-iteration boundaries),
   - `tests/test_mapping_reader_usage.py` (shared mapping-read parser usage),

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_parse_boundary.py",
     "tests/test_manager_transport_boundary.py",
     "tests/test_mapping_reader_usage.py",
     "tests/test_mapping_keys_access_usage.py",

tests/test_manager_parse_boundary.py

@@ -0,0 +1,36 @@
+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_PARSE_MARKERS = (
+    "parse_response_model(",
+    "parse_session_response_model(",
+    "parse_session_recordings_response_data(",
+)
+
+
+def test_managers_route_parsing_through_shared_helpers():
+    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_PARSE_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_PARSE_MARKERS):
+                    violating_modules.append(module_path.as_posix())
+
+    assert violating_modules == []