v0.5.10: Actually fix wake word switching - set id attribute on loaded models

index.html

@@ -97,6 +97,15 @@
 				<h2>Changelog</h2>
 				<div class="how-to-use changelog-container">
 					<div class="changelog-scroll">
+						<div class="changelog-entry">
+							<span class="version">v0.5.10</span>
+							<span class="date">2026-01-10</span>
+							<ul>
+								<li>Fix: Wake word models now have 'id' attribute set correctly</li>
+								<li>Fix: Wake word switching from Home Assistant now actually works</li>
+								<li>Fix: _update_wake_words_list properly filters by active_wake_words</li>
+							</ul>
+						</div>
 						<div class="changelog-entry">
 							<span class="version">v0.5.9</span>
 							<span class="date">2026-01-10</span>

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "reachy_mini_ha_voice"
-version = "0.5.9"
+version = "0.5.10"
 description = "Home Assistant Voice Assistant for Reachy Mini"
 readme = "README.md"
 requires-python = ">=3.10"

reachy_mini_ha_voice/satellite.py

@@ -322,7 +322,10 @@ class VoiceSatelliteProtocol(APIServer):
                     self.state.available_wake_words[wake_word_id] = model_info
 
                 _LOGGER.debug("Loading wake word: %s", model_info.wake_word_path)
-                self.state.wake_words[wake_word_id] = model_info.load()
+                loaded_model = model_info.load()
+                # Set id attribute on the model for later identification
+                setattr(loaded_model, 'id', wake_word_id)
+                self.state.wake_words[wake_word_id] = loaded_model
                 _LOGGER.info("Wake word loaded: %s", wake_word_id)
                 active_wake_words.add(wake_word_id)
                 # Don't break - load ALL requested wake words, not just the first one

reachy_mini_ha_voice/voice_assistant.py

@@ -521,7 +521,10 @@ class VoiceAssistantService:
 
                 try:
                     _LOGGER.debug("Loading wake model: %s", wake_word_id)
-                    wake_models[wake_word_id] = wake_word.load()
+                    loaded_model = wake_word.load()
+                    # Set id attribute on the model for later identification
+                    setattr(loaded_model, 'id', wake_word_id)
+                    wake_models[wake_word_id] = loaded_model
                     active_wake_words.add(wake_word_id)
                 except Exception as e:
                     _LOGGER.warning("Failed to load wake model %s: %s", wake_word_id, e)
@@ -532,7 +535,10 @@ class VoiceAssistantService:
             if wake_word:
                 try:
                     _LOGGER.debug("Loading default wake model: %s", self.wake_model)
-                    wake_models[self.wake_model] = wake_word.load()
+                    loaded_model = wake_word.load()
+                    # Set id attribute on the model for later identification
+                    setattr(loaded_model, 'id', self.wake_model)
+                    wake_models[self.wake_model] = loaded_model
                     active_wake_words.add(self.wake_model)
                 except Exception as e:
                     _LOGGER.error("Failed to load default wake model: %s", e)
@@ -641,16 +647,20 @@ class VoiceAssistantService:
         if (not ctx.wake_words) or (self._state.wake_words_changed and self._state.wake_words):
             self._state.wake_words_changed = False
             ctx.wake_words.clear()
-            ctx.wake_words.extend([
-                ww for ww in self._state.wake_words.values()
-                if ww.id in self._state.active_wake_words
-            ])
+            # state.wake_words is Dict[str, MicroWakeWord/OpenWakeWord]
+            # We need to filter by active_wake_words (which contains the IDs/keys)
+            for ww_id, ww_model in self._state.wake_words.items():
+                if ww_id in self._state.active_wake_words:
+                    # Ensure the model has an 'id' attribute for later use
+                    if not hasattr(ww_model, 'id'):
+                        setattr(ww_model, 'id', ww_id)
+                    ctx.wake_words.append(ww_model)
 
             ctx.has_oww = any(isinstance(ww, OpenWakeWord) for ww in ctx.wake_words)
             if ctx.has_oww and ctx.oww_features is None:
                 ctx.oww_features = OpenWakeWordFeatures.from_builtin()
 
-            _LOGGER.debug("Wake words updated: %s", [ww.id for ww in ctx.wake_words])
+            _LOGGER.info("Active wake words updated: %s", list(self._state.active_wake_words))
 
     def _get_reachy_audio_chunk(self) -> Optional[bytes]:
         """Get audio chunk from Reachy Mini's microphone.