Add agent request wrapper internal reuse architecture guard

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

Author: cursoragent

CONTRIBUTING.md

@@ -81,6 +81,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `tests/test_agent_operation_metadata_usage.py` (shared agent operation-metadata usage enforcement),
   - `tests/test_agent_payload_helper_usage.py` (shared agent start-payload helper usage enforcement),
   - `tests/test_agent_request_internal_reuse.py` (shared agent helper internal reuse of shared job request helpers),
+  - `tests/test_agent_request_wrapper_internal_reuse.py` (agent request-wrapper internal reuse of shared job request helpers),
   - `tests/test_agent_route_builder_usage.py` (shared agent read/stop route-builder usage enforcement),
   - `tests/test_agent_start_helper_usage.py` (shared agent start-request helper usage enforcement),
   - `tests/test_agent_stop_helper_usage.py` (shared agent stop-request helper usage enforcement),

tests/test_agent_request_wrapper_internal_reuse.py

@@ -0,0 +1,34 @@
+import pytest
+
+from tests.ast_function_source_utils import collect_function_sources
+
+pytestmark = pytest.mark.architecture
+
+
+MODULE_WRAPPER_EXPECTATIONS = {
+    "hyperbrowser/client/managers/agent_start_utils.py": {
+        "start_agent_task": ("start_job(",),
+        "start_agent_task_async": ("start_job_async(",),
+    },
+    "hyperbrowser/client/managers/agent_task_read_utils.py": {
+        "get_agent_task": ("get_job(",),
+        "get_agent_task_status": ("get_job_status(",),
+        "get_agent_task_async": ("get_job_async(",),
+        "get_agent_task_status_async": ("get_job_status_async(",),
+    },
+    "hyperbrowser/client/managers/agent_stop_utils.py": {
+        "stop_agent_task": ("put_job_action(", 'action_suffix="/stop"'),
+        "stop_agent_task_async": ("put_job_action_async(", 'action_suffix="/stop"'),
+    },
+}
+
+
+def test_agent_request_wrappers_delegate_to_expected_job_helpers():
+    for module_path, wrapper_expectations in MODULE_WRAPPER_EXPECTATIONS.items():
+        function_sources = collect_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

tests/test_architecture_marker_usage.py

@@ -11,6 +11,7 @@
     "tests/test_agent_operation_metadata_usage.py",
     "tests/test_agent_payload_helper_usage.py",
     "tests/test_agent_request_internal_reuse.py",
+    "tests/test_agent_request_wrapper_internal_reuse.py",
     "tests/test_agent_route_builder_usage.py",
     "tests/test_agent_start_helper_usage.py",
     "tests/test_agent_task_read_helper_usage.py",