Add cross-module request wrapper internal reuse guard

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

Author: cursoragent

CONTRIBUTING.md

@@ -154,6 +154,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `tests/test_readme_examples_listing.py` (README example-listing consistency enforcement),
   - `tests/test_request_helper_parse_import_boundary.py` (request-helper import boundary enforcement for direct response parsing imports),
   - `tests/test_request_helper_transport_boundary.py` (request-helper transport boundary enforcement through shared model request helpers),
+  - `tests/test_request_wrapper_internal_reuse.py` (request-wrapper internal reuse of shared model request helpers across profile/team/extension/computer-action modules),
   - `tests/test_response_parse_usage_boundary.py` (centralized `parse_response_model(...)` usage boundary enforcement),
   - `tests/test_schema_injection_helper_usage.py` (shared schema injection helper usage enforcement in payload builders),
   - `tests/test_session_operation_metadata_usage.py` (session manager operation-metadata usage enforcement),

tests/test_architecture_marker_usage.py

@@ -55,6 +55,7 @@
     "tests/test_readme_examples_listing.py",
     "tests/test_request_helper_parse_import_boundary.py",
     "tests/test_request_helper_transport_boundary.py",
+    "tests/test_request_wrapper_internal_reuse.py",
     "tests/test_response_parse_usage_boundary.py",
     "tests/test_examples_syntax.py",
     "tests/test_docs_python3_commands.py",

tests/test_request_wrapper_internal_reuse.py

@@ -0,0 +1,65 @@
+import ast
+from pathlib import Path
+
+import pytest
+
+pytestmark = pytest.mark.architecture
+
+
+MODULE_WRAPPER_EXPECTATIONS = {
+    "hyperbrowser/client/managers/profile_request_utils.py": {
+        "create_profile_resource": ("post_model_request(",),
+        "get_profile_resource": ("get_model_request(",),
+        "delete_profile_resource": ("delete_model_request(",),
+        "list_profile_resources": ("get_model_request(",),
+        "create_profile_resource_async": ("post_model_request_async(",),
+        "get_profile_resource_async": ("get_model_request_async(",),
+        "delete_profile_resource_async": ("delete_model_request_async(",),
+        "list_profile_resources_async": ("get_model_request_async(",),
+    },
+    "hyperbrowser/client/managers/team_request_utils.py": {
+        "get_team_resource": ("get_model_request(",),
+        "get_team_resource_async": ("get_model_request_async(",),
+    },
+    "hyperbrowser/client/managers/computer_action_request_utils.py": {
+        "execute_computer_action_request": ("post_model_request_to_endpoint(",),
+        "execute_computer_action_request_async": (
+            "post_model_request_to_endpoint_async(",
+        ),
+    },
+    "hyperbrowser/client/managers/extension_request_utils.py": {
+        "create_extension_resource": ("post_model_request(",),
+        "create_extension_resource_async": ("post_model_request_async(",),
+        "list_extension_resources": (
+            "get_model_response_data(",
+            "parse_extension_list_response_data(",
+        ),
+        "list_extension_resources_async": (
+            "get_model_response_data_async(",
+            "parse_extension_list_response_data(",
+        ),
+    },
+}
+
+
+def _collect_module_function_sources(module_path: str) -> dict[str, str]:
+    module_text = Path(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_request_wrappers_delegate_to_expected_shared_helpers():
+    for module_path, wrapper_expectations in MODULE_WRAPPER_EXPECTATIONS.items():
+        function_sources = _collect_module_function_sources(module_path)
+        for wrapper_name, expected_markers in wrapper_expectations.items():
+            wrapper_source = function_sources[wrapper_name]
+            for expected_marker in expected_markers:
+                assert expected_marker in wrapper_source
+            assert "client.transport." not in wrapper_source
+            assert "parse_response_model(" not in wrapper_source