Add safe key-display import boundary guard

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

Author: cursoragent

CONTRIBUTING.md

@@ -197,6 +197,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `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_safe_key_display_helper_import_boundary.py` (safe mapping-key display helper import boundary enforcement),
   - `tests/test_safe_key_display_helper_usage.py` (safe mapping-key display helper usage centralization),
   - `tests/test_schema_injection_helper_usage.py` (shared schema injection helper usage enforcement in payload builders),
   - `tests/test_session_operation_metadata_import_boundary.py` (session operation-metadata import boundary enforcement),

tests/test_architecture_marker_usage.py

@@ -129,6 +129,7 @@
     "tests/test_computer_action_payload_helper_usage.py",
     "tests/test_computer_action_request_helper_usage.py",
     "tests/test_computer_action_request_internal_reuse.py",
+    "tests/test_safe_key_display_helper_import_boundary.py",
     "tests/test_safe_key_display_helper_usage.py",
     "tests/test_schema_injection_helper_usage.py",
     "tests/test_session_operation_metadata_import_boundary.py",

tests/test_safe_key_display_helper_import_boundary.py

@@ -0,0 +1,40 @@
+import ast
+from pathlib import Path
+
+import pytest
+
+pytestmark = pytest.mark.architecture
+
+
+EXPECTED_SAFE_KEY_DISPLAY_IMPORTERS = (
+    "hyperbrowser/client/managers/list_parsing_utils.py",
+    "tests/test_mapping_utils.py",
+)
+
+
+def _imports_safe_key_display_helper(module_text: str) -> bool:
+    module_ast = ast.parse(module_text)
+    for node in module_ast.body:
+        if not isinstance(node, ast.ImportFrom):
+            continue
+        if node.module != "hyperbrowser.mapping_utils":
+            continue
+        if any(alias.name == "safe_key_display_for_error" for alias in node.names):
+            return True
+    return False
+
+
+def test_safe_key_display_helper_imports_are_centralized():
+    discovered_modules: list[str] = []
+
+    for module_path in sorted(Path("hyperbrowser").rglob("*.py")):
+        module_text = module_path.read_text(encoding="utf-8")
+        if _imports_safe_key_display_helper(module_text):
+            discovered_modules.append(module_path.as_posix())
+
+    for module_path in sorted(Path("tests").glob("test_*.py")):
+        module_text = module_path.read_text(encoding="utf-8")
+        if _imports_safe_key_display_helper(module_text):
+            discovered_modules.append(module_path.as_posix())
+
+    assert discovered_modules == list(EXPECTED_SAFE_KEY_DISPLAY_IMPORTERS)