Refine list parsing caller and runtime inventories

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

Author: cursoragent

tests/test_list_parsing_helper_usage.py

@@ -7,23 +7,26 @@
 pytestmark = pytest.mark.architecture
 
 HYPERBROWSER_ROOT = Path(__file__).resolve().parents[1] / "hyperbrowser"
-ALLOWED_PARSE_MAPPING_LIST_CALL_FILES = {
+LIST_PARSING_HELPER_RUNTIME_MODULES = {
     Path("client/managers/extension_utils.py"),
     Path("client/managers/list_parsing_utils.py"),
     Path("client/managers/session_utils.py"),
 }
-ALLOWED_READ_PLAIN_LIST_CALL_FILES = {
-    Path("client/managers/extension_utils.py"),
-    Path("client/managers/list_parsing_utils.py"),
-    Path("client/managers/session_utils.py"),
+LIST_PARSING_HELPER_CALLER_MODULES = {
+    path
+    for path in LIST_PARSING_HELPER_RUNTIME_MODULES
+    if path != Path("client/managers/list_parsing_utils.py")
 }
+ALLOWED_PARSE_MAPPING_LIST_CALL_FILES = LIST_PARSING_HELPER_CALLER_MODULES
+ALLOWED_READ_PLAIN_LIST_CALL_FILES = LIST_PARSING_HELPER_CALLER_MODULES
 
 
 def _python_files() -> list[Path]:
     return sorted(HYPERBROWSER_ROOT.rglob("*.py"))
 
 
 def test_parse_mapping_list_items_usage_is_centralized():
+    files_with_calls: set[Path] = set()
     violations: list[str] = []
 
     for path in _python_files():
@@ -32,15 +35,18 @@ def test_parse_mapping_list_items_usage_is_centralized():
         helper_calls = collect_name_call_lines(module, "parse_mapping_list_items")
         if not helper_calls:
             continue
+        files_with_calls.add(relative_path)
         if relative_path in ALLOWED_PARSE_MAPPING_LIST_CALL_FILES:
             continue
         for line in helper_calls:
             violations.append(f"{relative_path}:{line}")
 
     assert violations == []
+    assert files_with_calls == ALLOWED_PARSE_MAPPING_LIST_CALL_FILES
 
 
 def test_read_plain_list_items_usage_is_centralized():
+    files_with_calls: set[Path] = set()
     violations: list[str] = []
 
     for path in _python_files():
@@ -49,9 +55,11 @@ def test_read_plain_list_items_usage_is_centralized():
         helper_calls = collect_name_call_lines(module, "read_plain_list_items")
         if not helper_calls:
             continue
+        files_with_calls.add(relative_path)
         if relative_path in ALLOWED_READ_PLAIN_LIST_CALL_FILES:
             continue
         for line in helper_calls:
             violations.append(f"{relative_path}:{line}")
 
     assert violations == []
+    assert files_with_calls == ALLOWED_READ_PLAIN_LIST_CALL_FILES

tests/test_list_parsing_helpers_import_boundary.py

@@ -3,8 +3,7 @@
 import pytest
 
 from tests.test_list_parsing_helper_usage import (
-    ALLOWED_PARSE_MAPPING_LIST_CALL_FILES,
-    ALLOWED_READ_PLAIN_LIST_CALL_FILES,
+    LIST_PARSING_HELPER_RUNTIME_MODULES,
 )
 
 pytestmark = pytest.mark.architecture
@@ -33,10 +32,7 @@ def test_list_parsing_helper_imports_are_centralized():
 
     expected_runtime_modules = sorted(
         f"hyperbrowser/{path.as_posix()}"
-        for path in {
-            *ALLOWED_PARSE_MAPPING_LIST_CALL_FILES,
-            *ALLOWED_READ_PLAIN_LIST_CALL_FILES,
-        }
+        for path in LIST_PARSING_HELPER_RUNTIME_MODULES
         if path != Path("client/managers/list_parsing_utils.py")
     )
     expected_modules = sorted(