fix bug between rename speaker and rename multiple speakers

ui.py

@@ -114,21 +114,27 @@ def render_rename_sidebar(currFile, speakerNames, all_speaker_tokens):
     st.sidebar.divider()
 
     def _on_grename_change(idx):
+        # Write the widget's new value into the data model first
         st.session_state.globalRenames[idx]["speakers"] = list(
             st.session_state[_global_rename_key(idx)]
         )
+        # Remove the changed entry's widget key so it re-seeds cleanly on
+        # the next render (avoids Streamlit "cannot modify instantiated widget" errors)
+        st.session_state.pop(_global_rename_key(idx), None)
         enforceGlobalRenameExclusivity(idx)
 
-    # Build claimed set from widget keys (not entry["speakers"] data) so it
-    # matches exactly what Streamlit is actually displaying as selected.
+    # Build claimed set from the data model (entry["speakers"]) as the canonical
+    # source of truth.  Widget keys may be absent after a mutation (we pop them
+    # intentionally), so falling back to them can produce a stale claimed set.
     all_claimed = {
         token
-        for i, e in enumerate(st.session_state.globalRenames)
-        for token in st.session_state.get(_global_rename_key(i), e["speakers"])
+        for entry in st.session_state.globalRenames
+        for token in entry["speakers"]
     }
 
     for idx, entry in enumerate(st.session_state.globalRenames):
         grkey = _global_rename_key(idx)
+        # Seed the widget key from entry["speakers"] if it was cleared by a mutation
         if grkey not in st.session_state:
             st.session_state[grkey] = list(entry["speakers"])
         label_col, trash_col = st.sidebar.columns([5, 1])
@@ -138,11 +144,10 @@ def render_rename_sidebar(currFile, speakerNames, all_speaker_tokens):
             on_click=removeGlobalRename, args=(idx,),
             help=f"Remove '{entry['name']}'",
         )
-        # Use the widget's current value (not entry["speakers"]) as this entry's
-        # owned tokens — they can drift, and the widget value is what Streamlit
-        # actually displays. This prevents Streamlit silently dropping selections
-        # when available_tokens is computed from stale entry["speakers"] data.
-        this_entry_tokens = set(st.session_state.get(grkey, entry["speakers"]))
+        # Use entry["speakers"] (the data model) as this entry's owned tokens.
+        # Widget keys are popped during mutations, so using them here would give
+        # a stale empty set and incorrectly exclude tokens from available_tokens.
+        this_entry_tokens = set(entry["speakers"])
         available_tokens = [
             t for t in all_speaker_tokens
             if t not in all_claimed or t in this_entry_tokens