Add model request function transport boundary architecture guard

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

Author: cursoragent

CONTRIBUTING.md

@@ -134,6 +134,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `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_model_request_function_transport_boundary.py` (model-request function-level transport boundary enforcement between parsed wrappers and raw helpers),
   - `tests/test_model_request_internal_reuse.py` (request-helper internal reuse of shared model request helper primitives),
   - `tests/test_model_request_wrapper_internal_reuse.py` (parsed model-request wrapper internal reuse of shared raw response-data helpers),
   - `tests/test_optional_serialization_helper_usage.py` (optional model serialization helper usage enforcement),

tests/test_architecture_marker_usage.py

@@ -24,6 +24,7 @@
     "tests/test_manager_transport_boundary.py",
     "tests/test_mapping_reader_usage.py",
     "tests/test_mapping_keys_access_usage.py",
+    "tests/test_model_request_function_transport_boundary.py",
     "tests/test_model_request_internal_reuse.py",
     "tests/test_model_request_wrapper_internal_reuse.py",
     "tests/test_tool_mapping_reader_usage.py",

tests/test_model_request_function_transport_boundary.py

@@ -0,0 +1,61 @@
+import ast
+from pathlib import Path
+
+import pytest
+
+pytestmark = pytest.mark.architecture
+
+
+MODULE_PATH = Path("hyperbrowser/client/managers/model_request_utils.py")
+
+PARSED_WRAPPER_FUNCTIONS = (
+    "post_model_request",
+    "get_model_request",
+    "delete_model_request",
+    "put_model_request",
+    "post_model_request_async",
+    "get_model_request_async",
+    "delete_model_request_async",
+    "put_model_request_async",
+    "post_model_request_to_endpoint",
+    "post_model_request_to_endpoint_async",
+)
+
+RAW_HELPER_FUNCTIONS = (
+    "post_model_response_data",
+    "get_model_response_data",
+    "delete_model_response_data",
+    "put_model_response_data",
+    "post_model_response_data_async",
+    "get_model_response_data_async",
+    "delete_model_response_data_async",
+    "put_model_response_data_async",
+    "post_model_response_data_to_endpoint",
+    "post_model_response_data_to_endpoint_async",
+)
+
+
+def _collect_function_sources() -> dict[str, str]:
+    module_text = MODULE_PATH.read_text(encoding="utf-8")
+    module_ast = ast.parse(module_text)
+    function_sources: dict[str, str] = {}
+    for node in module_ast.body:
+        if isinstance(node, (ast.FunctionDef, ast.AsyncFunctionDef)):
+            function_source = ast.get_source_segment(module_text, node)
+            if function_source is not None:
+                function_sources[node.name] = function_source
+    return function_sources
+
+
+def test_parsed_model_request_wrappers_do_not_call_transport_directly():
+    function_sources = _collect_function_sources()
+    for function_name in PARSED_WRAPPER_FUNCTIONS:
+        function_source = function_sources[function_name]
+        assert "client.transport." not in function_source
+
+
+def test_raw_model_request_helpers_are_transport_boundary():
+    function_sources = _collect_function_sources()
+    for function_name in RAW_HELPER_FUNCTIONS:
+        function_source = function_sources[function_name]
+        assert "client.transport." in function_source