Enforce agent helper internal reuse boundaries

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

Author: cursoragent

CONTRIBUTING.md

@@ -80,6 +80,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `tests/test_agent_helper_boundary.py` (agent manager boundary enforcement for shared request/response helpers),
   - `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_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_internal_reuse.py

@@ -0,0 +1,40 @@
+from pathlib import Path
+
+import pytest
+
+pytestmark = pytest.mark.architecture
+
+
+def test_agent_start_helpers_reuse_job_request_helpers():
+    module_text = Path("hyperbrowser/client/managers/agent_start_utils.py").read_text(
+        encoding="utf-8"
+    )
+    assert "job_request_utils import start_job, start_job_async" in module_text
+    assert "start_job(" in module_text
+    assert "start_job_async(" in module_text
+    assert "client.transport.post(" not in module_text
+    assert "parse_response_model(" not in module_text
+
+
+def test_agent_task_read_helpers_reuse_job_request_helpers():
+    module_text = Path(
+        "hyperbrowser/client/managers/agent_task_read_utils.py"
+    ).read_text(encoding="utf-8")
+    assert "job_request_utils import" in module_text
+    assert "get_job(" in module_text
+    assert "get_job_status(" in module_text
+    assert "get_job_async(" in module_text
+    assert "get_job_status_async(" in module_text
+    assert "client.transport.get(" not in module_text
+    assert "parse_response_model(" not in module_text
+
+
+def test_agent_stop_helpers_reuse_job_request_helpers():
+    module_text = Path("hyperbrowser/client/managers/agent_stop_utils.py").read_text(
+        encoding="utf-8"
+    )
+    assert "job_request_utils import put_job_action, put_job_action_async" in module_text
+    assert "put_job_action(" in module_text
+    assert "put_job_action_async(" in module_text
+    assert "client.transport.put(" not in module_text
+    assert "parse_response_model(" not in module_text

tests/test_architecture_marker_usage.py

@@ -10,6 +10,7 @@
     "tests/test_agent_helper_boundary.py",
     "tests/test_agent_operation_metadata_usage.py",
     "tests/test_agent_payload_helper_usage.py",
+    "tests/test_agent_request_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",