review comments

Author: cpwright

plugins/ui/src/deephaven/ui/object_types/ElementType.py

@@ -1,4 +1,4 @@
-from typing import Any
+from typing import Any, Literal
 
 from deephaven.plugin.object_type import BidirectionalObjectType, MessageStream
 from ..elements import Element
@@ -21,7 +21,7 @@ def name(self) -> str:
         return "deephaven.ui.Element"
 
     @property
-    def authorization_export_behavior(self) -> str:
+    def authorization_export_behavior(self) -> Literal["transform", "unset"]:
         """Declares that deephaven.ui must export its references through the authorization transform.
 
         Server objects (tables, etc.) handed to the client via a deephaven.ui component are run through the
@@ -31,7 +31,9 @@ def authorization_export_behavior(self) -> str:
         enforced (fail secure).
         """
         try:
-            from deephaven.configuration import get_configuration
+            # deephaven.configuration only exists in the 42.x server package; the import is conditional to
+            # maintain compatibility with 41.x, where ImportError is caught and the transform is enforced.
+            from deephaven.configuration import get_configuration  # type: ignore[import-untyped,import-not-found]
 
             if get_configuration().get_bool(
                 _DISABLE_AUTHORIZATION_EXPORT_TRANSFORM_PROPERTY, False