lints

Author: schloerke

shiny/input_handler.py

@@ -227,6 +227,12 @@ def _(value: Any, name: ResolvedId, session: Session) -> Any:
 
     restore_ctx_dir = Path(restore_ctx.dir)
 
+    def cleanup_tempdir(tempdir_root: tempfile.TemporaryDirectory[str]):
+        @session.on_ended
+        def _():
+            # Cleanup the temporary directory after the session ends
+            tempdir_root.cleanup()
+
     for f in value_list:
         assert f["datapath"] is not None and isinstance(f["datapath"], str)
 
@@ -238,7 +244,7 @@ def _(value: Any, name: ResolvedId, session: Session) -> Any:
         # Copy the original file to a new temp dir, so that a restored session can't
         # modify the original.
         tempdir_root = tempfile.TemporaryDirectory()
-        session.on_ended(lambda: tempdir_root.cleanup())
+        cleanup_tempdir(tempdir_root)
 
         tempdir = Path(tempdir_root.name) / rand_hex(12)
         tempdir.mkdir(parents=True, exist_ok=True)

shiny/ui/_input_file.py

@@ -102,9 +102,10 @@ def input_file(
                 restored_obj["type"].append(file.get("type", None))
                 restored_obj["datapath"].append(file.get("datapath", None))
             restored_value = to_json_str(restored_obj)
-        except:
+        except Exception:
             warnings.warn(
-                f"Error while restoring file input value for `{resolved_id}`. Resetting to `None`."
+                f"Error while restoring file input value for `{resolved_id}`. Resetting to `None`.",
+                stacklevel=1,
             )
 
     btn_file = span(

tests/playwright/ai_generated_apps/bookmark/input_file/test_input_file_express_bookmarking.py

@@ -1,4 +1,3 @@
-import pytest
 from playwright.sync_api import FilePayload, Page
 
 from shiny.playwright import controller