Add session recordings follow-redirect boundary guard

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

Author: cursoragent

CONTRIBUTING.md

@@ -161,6 +161,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `tests/test_session_operation_metadata_usage.py` (session manager operation-metadata usage enforcement),
   - `tests/test_session_parse_usage_boundary.py` (centralized session parse-helper usage boundary enforcement),
   - `tests/test_session_profile_update_helper_usage.py` (session profile-update parameter helper usage enforcement),
+  - `tests/test_session_recordings_follow_redirects_boundary.py` (session recordings wrapper follow-redirect enforcement boundary),
   - `tests/test_session_request_helper_usage.py` (session manager request-helper usage enforcement),
   - `tests/test_session_request_internal_reuse.py` (session request-helper internal reuse of shared model raw request helpers),
   - `tests/test_session_request_wrapper_internal_reuse.py` (parsed session-request wrapper internal reuse of session resource helpers),

tests/test_architecture_marker_usage.py

@@ -93,6 +93,7 @@
     "tests/test_session_operation_metadata_usage.py",
     "tests/test_session_parse_usage_boundary.py",
     "tests/test_session_profile_update_helper_usage.py",
+    "tests/test_session_recordings_follow_redirects_boundary.py",
     "tests/test_session_request_helper_usage.py",
     "tests/test_session_request_internal_reuse.py",
     "tests/test_session_request_wrapper_internal_reuse.py",

tests/test_session_recordings_follow_redirects_boundary.py

@@ -0,0 +1,33 @@
+import ast
+from pathlib import Path
+
+import pytest
+
+pytestmark = pytest.mark.architecture
+
+
+MODULE_PATH = Path("hyperbrowser/client/managers/session_request_utils.py")
+
+
+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_sync_session_recordings_wrapper_enforces_follow_redirects():
+    function_source = _collect_function_sources()["get_session_recordings"]
+    assert "get_session_resource(" in function_source
+    assert "follow_redirects=True" in function_source
+
+
+def test_async_session_recordings_wrapper_enforces_follow_redirects():
+    function_source = _collect_function_sources()["get_session_recordings_async"]
+    assert "get_session_resource_async(" in function_source
+    assert "follow_redirects=True" in function_source