fix: selecting an example from the Command Palette didn't leave the input box focused, breaking Up-arrow history browsing afterward

Real bug found via user testing: the utterance WAS correctly appended
to utterance_history when selecting an example (confirmed by the
existing test) - the actual break was that Up-arrow did nothing
afterward. _navigate_history()'s own focus check ('if self.focused is
not input_widget: return') silently no-ops unless the input box
itself has keyboard focus, and closing the Command Palette doesn't
reliably leave focus there the way typing+Enter naturally does (focus
was already on the input box before Enter was pressed).

_send_utterance() now explicitly re-focuses the input box at the end -
harmless for the typing+Enter path (already focused there), fixes the
palette-selection path where it wasn't guaranteed.

264 tests passing (1 new, confirming focus lands on the input box
after selecting an example). Version bumped to 0.1.24 for release.

Author: andlo

README.md

@@ -205,7 +205,7 @@ Confirmed NOT sufficient on its own under rootless Podman specifically
 Podman rather than a standard root-owned `dockerd` socket, this may
 need more digging into your specific setup.
 
-Images are tagged by version (`:0.1.23`) and `:latest`, built and
+Images are tagged by version (`:0.1.24`) and `:latest`, built and
 published automatically on every release.
 
 ## Why not just fix ovos-cli-client / neon-cli-client?

ovos_tui_client/app.py

@@ -1069,6 +1069,17 @@ def _send_utterance(self, text: str) -> None:
         self.bus.send_utterance(text)
         self.utterance_history.append(text)
         self.history_index = None
+        # Real bug found via user testing: selecting an example from
+        # the Command Palette correctly appended to utterance_history
+        # (confirmed above), but Up-arrow afterward did nothing -
+        # _navigate_history()'s own focus check (`if self.focused is
+        # not input_widget: return`) silently no-ops unless the input
+        # box itself has keyboard focus, and closing the palette
+        # doesn't reliably leave focus there. Explicitly re-focusing
+        # here makes history browsing work right after EITHER path
+        # (typing+Enter already had focus naturally; this covers the
+        # palette-selection path where it wasn't guaranteed).
+        self.query_one("#utterance-input", Input).focus()
 
     def get_system_commands(self, screen: Screen):
         """Surfaces the same actions available via F1/F5-F8 in

tests/test_command_palette.py

@@ -8,7 +8,7 @@
 from unittest.mock import MagicMock, patch
 
 import pytest
-from textual.widgets import RichLog
+from textual.widgets import RichLog, Input
 
 from ovos_tui_client.app import OVOSTUIApp, ServiceCommandProvider, SkillCommandProvider, PipelineCommandProvider, ExampleCommandProvider
 
@@ -274,6 +274,28 @@ async def test_selecting_an_example_sends_it_like_a_typed_utterance(tmp_path):
         assert "what's the weather?" in text
 
 
+@pytest.mark.asyncio
+async def test_selecting_an_example_returns_focus_to_the_input_box(tmp_path):
+    """Real bug found via user testing: selecting an example DID
+    correctly append to utterance_history (confirmed by the sibling
+    test above), but pressing Up afterward did nothing - the Up/Down
+    handler's own focus check silently no-ops unless the input box
+    itself has keyboard focus, and closing the Command Palette doesn't
+    reliably leave focus there. _send_utterance() now explicitly
+    re-focuses the input box, so history browsing works right after
+    selecting an example, not just after typing+Enter."""
+    app = _app_with_fake_bus(tmp_path)
+    app.skill_examples = {"ovos-skill-weather.openvoiceos": ["what's the weather?"]}
+    async with app.run_test() as pilot:
+        provider = ExampleCommandProvider(app.screen)
+        hits = await _collect_hits(provider, "weather")
+        hits[0].command()
+        await pilot.pause()
+
+        input_widget = app.query_one("#utterance-input", Input)
+        assert app.focused is input_widget
+
+
 @pytest.mark.asyncio
 async def test_example_search_has_no_hits_when_cache_is_empty(tmp_path):
     """Not an error - just nothing to offer yet if skill_examples

version.py

@@ -1,5 +1,5 @@
 VERSION_MAJOR = 0
 VERSION_MINOR = 1
-VERSION_BUILD = 23
+VERSION_BUILD = 24
 VERSION_ALPHA = 0
 # END_VERSION_BLOCK