Apply suggestions from code review

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Author: FichteFoll

plugin/history.py

@@ -117,9 +117,15 @@ def sync_selection_history(
 
     if prune_removed_sheets:
         for group_key, group_state_stack in list(groups.items()):
-            updated_stack = prune_history_stack_for_live_sheets(
-                window, int(group_key), group_state_stack
-            )
+            try:
+                group = int(group_key)
+            except (TypeError, ValueError):
+                # Unexpected/corrupt key; drop it to avoid crashing.
+                del groups[group_key]
+                changed = True
+                continue
+
+            updated_stack = prune_history_stack_for_live_sheets(window, group, group_state_stack)
             if updated_stack != group_state_stack:
                 changed = True
                 if updated_stack:
@@ -227,11 +233,10 @@ def prune_group_state(
 
 def prune_history_stack(
     group_state_stack: list[GroupSelectionState],
-    removed_identities: list[SheetIdentity] = [],
+    removed_identities: list[SheetIdentity] | None = None,
 ) -> list[GroupSelectionState]:
     updated_stack: list[GroupSelectionState] = []
-    seen_identities: list[SheetIdentity] = [*removed_identities]
-
+    seen_identities: list[SheetIdentity] = list(removed_identities or [])
     for group_state in group_state_stack:
         pruned_group_state = prune_group_state(
             group_state,

plugin/session.py

@@ -6,9 +6,7 @@
 
 
 def show_panel(window, state) -> None:
-    entries = state.session_entries
-    items = [entry.caption for entry in entries]
-    state.session_selected_index = min(len(items), 1)
+    state.session_selected_index = 1 if len(items) > 1 else 0
 
     def on_select(index: int) -> None:
         if index < 0:
@@ -41,7 +39,9 @@ def on_highlight(index: int) -> None:
 
 
 def preview_entry(window, state: TabStackWindowState) -> None:
-    if state.session_entries is None or len(state.session_entries) < state.session_selected_index:
+    if state.session_entries is None:
+        return
+    if not 0 <= state.session_selected_index < len(state.session_entries):
         return
     entry = state.session_entries[state.session_selected_index]
     apply_group_selection(window, entry.selection, state.session_group)