Add operation-metadata import boundary guards

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

Author: cursoragent

CONTRIBUTING.md

@@ -127,6 +127,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `tests/test_examples_syntax.py` (example script syntax guardrail),
   - `tests/test_extension_create_helper_usage.py` (extension create-input normalization helper usage enforcement),
   - `tests/test_extension_create_metadata_usage.py` (extension create-helper shared operation-metadata prefix usage enforcement),
+  - `tests/test_extension_operation_metadata_import_boundary.py` (extension operation-metadata import boundary enforcement),
   - `tests/test_extension_operation_metadata_usage.py` (extension manager operation-metadata usage enforcement),
   - `tests/test_extension_parse_usage_boundary.py` (centralized extension list parse-helper usage boundary enforcement),
   - `tests/test_extension_request_function_parse_boundary.py` (extension-request function-level parser boundary enforcement between create/list wrappers),
@@ -188,6 +189,7 @@ This runs lint, format checks, compile checks, tests, and package build.
   - `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_import_boundary.py` (session operation-metadata import boundary enforcement),
   - `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_architecture_marker_usage.py

@@ -90,6 +90,7 @@
     "tests/test_extension_create_metadata_usage.py",
     "tests/test_extract_payload_helper_usage.py",
     "tests/test_examples_naming_convention.py",
+    "tests/test_extension_operation_metadata_import_boundary.py",
     "tests/test_extension_operation_metadata_usage.py",
     "tests/test_extension_parse_usage_boundary.py",
     "tests/test_extension_request_function_parse_boundary.py",
@@ -120,6 +121,7 @@
     "tests/test_computer_action_request_helper_usage.py",
     "tests/test_computer_action_request_internal_reuse.py",
     "tests/test_schema_injection_helper_usage.py",
+    "tests/test_session_operation_metadata_import_boundary.py",
     "tests/test_session_operation_metadata_usage.py",
     "tests/test_session_parse_usage_boundary.py",
     "tests/test_session_profile_update_helper_usage.py",

tests/test_extension_operation_metadata_import_boundary.py

@@ -0,0 +1,33 @@
+from pathlib import Path
+
+import pytest
+
+from tests.test_extension_create_metadata_usage import MODULE_PATH as CREATE_HELPER_MODULE
+from tests.test_extension_operation_metadata_usage import MODULES as MANAGER_MODULES
+
+pytestmark = pytest.mark.architecture
+
+
+EXPECTED_EXTRA_IMPORTERS = (
+    "tests/test_extension_create_metadata_usage.py",
+    "tests/test_extension_operation_metadata.py",
+    "tests/test_extension_operation_metadata_import_boundary.py",
+    "tests/test_extension_operation_metadata_usage.py",
+)
+
+
+def test_extension_operation_metadata_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 "extension_operation_metadata import" in 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 "extension_operation_metadata import" in module_text:
+            discovered_modules.append(module_path.as_posix())
+
+    expected_modules = sorted([*MANAGER_MODULES, CREATE_HELPER_MODULE, *EXPECTED_EXTRA_IMPORTERS])
+    assert discovered_modules == expected_modules

tests/test_session_operation_metadata_import_boundary.py

@@ -0,0 +1,33 @@
+from pathlib import Path
+
+import pytest
+
+from tests.test_session_operation_metadata_usage import MODULES as MANAGER_MODULES
+from tests.test_session_upload_metadata_usage import MODULE_PATH as UPLOAD_HELPER_MODULE
+
+pytestmark = pytest.mark.architecture
+
+
+EXPECTED_EXTRA_IMPORTERS = (
+    "tests/test_session_operation_metadata.py",
+    "tests/test_session_operation_metadata_import_boundary.py",
+    "tests/test_session_operation_metadata_usage.py",
+    "tests/test_session_upload_metadata_usage.py",
+)
+
+
+def test_session_operation_metadata_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 "session_operation_metadata import" in 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 "session_operation_metadata import" in module_text:
+            discovered_modules.append(module_path.as_posix())
+
+    expected_modules = sorted([*MANAGER_MODULES, UPLOAD_HELPER_MODULE, *EXPECTED_EXTRA_IMPORTERS])
+    assert discovered_modules == expected_modules