fix: Critical TUI exit functionality - users can now exit with q, Ctrl+C, Ctrl+Q, or Escape

- Added multiple exit key bindings to ensure users can always exit
- Added explicit quit action handlers
- Wrapped TUI launch in KeyboardInterrupt handler for clean exit
- Fixes critical bug where TUI trapped users with no way to exit

Version bump to 0.3.5

Author: bdmorin

CHANGELOG.md

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.5] - 2025-01-30
+
+### Fixed
+- **CRITICAL**: TUI now properly responds to exit commands (q, Ctrl+C, Ctrl+Q, Escape)
+- Added multiple exit key bindings for better user experience
+- Added explicit quit action handlers
+- Wrapped TUI launch in KeyboardInterrupt handler for clean exit
+
 ## [0.3.4] - 2025-01-30
 
 ### Added

README.md

@@ -9,7 +9,7 @@
 [![CI](https://github.com/bdmorin/eyelet/actions/workflows/ci.yml/badge.svg)](https://github.com/bdmorin/eyelet/actions/workflows/ci.yml)
 [![Status](https://img.shields.io/badge/status-alpha-yellow)](https://github.com/bdmorin/eyelet)
 
-## 🎉 New in v0.3.4: Auto-update Support & Critical Fixes!
+## 🎉 New in v0.3.5: TUI Exit Fix!
 
 ### v0.3.4 Updates
 - **Auto-update support**: `--autoupdate` flag for install-all command

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "eyelet"
-version = "0.3.4"
+version = "0.3.5"
 description = "Hook orchestration system for AI agents with SQLite logging and powerful analytics"
 readme = "README.md"
 authors = [{name = "Brian Morin"}]

src/eyelet/__init__.py

@@ -4,7 +4,7 @@
 Thread through the eyelet!
 """
 
-__version__ = "0.3.4"
+__version__ = "0.3.5"
 __author__ = "Brian Morin"
 
 # Defer imports to avoid circular dependencies

src/eyelet/tui/app.py

@@ -122,6 +122,10 @@ def show_quick_actions(self) -> None:
         """Show quick actions menu"""
         # TODO: Implement quick actions popup
         self.app.notify("⚡ Quick Actions - Coming soon!", severity="information")
+    
+    def action_quit(self) -> None:
+        """Quit the application"""
+        self.app.exit()
 
 
 class EyeletApp(App):
@@ -141,7 +145,10 @@ class EyeletApp(App):
     }
 
     BINDINGS = [
-        Binding("ctrl+q", "quit", "Quit", priority=True),
+        Binding("ctrl+c", "quit", "Quit", priority=True, show=True),
+        Binding("ctrl+q", "quit", "Quit", priority=True, show=True),
+        Binding("q", "quit", "Quit", priority=True, show=True),
+        Binding("escape", "quit", "Exit", priority=True, show=False),
         Binding("ctrl+p", "command_palette", "Command Palette"),
         Binding("ctrl+t", "toggle_theme", "Toggle Theme"),
     ]
@@ -175,9 +182,23 @@ def apply_theme(self) -> None:
     def action_command_palette(self) -> None:
         """Show command palette"""
         self.notify("🎯 Command Palette - Coming soon!", severity="information")
+    
+    def action_quit(self) -> None:
+        """Quit the application immediately"""
+        self.exit()
+    
+    def on_key(self, event: events.Key) -> None:
+        """Handle key events globally"""
+        # Emergency exit on Ctrl+C
+        if event.key == "ctrl+c":
+            self.exit()
 
 
 def launch_tui():
     """Launch the Eyelet TUI"""
-    app = EyeletApp()
-    app.run()
+    try:
+        app = EyeletApp()
+        app.run()
+    except KeyboardInterrupt:
+        # Clean exit on Ctrl+C
+        pass

test_tui_exit.py

@@ -0,0 +1,44 @@
+#!/usr/bin/env python3
+"""Test TUI exit functionality"""
+
+import asyncio
+import sys
+from pathlib import Path
+
+# Add src to path for local testing
+sys.path.insert(0, str(Path(__file__).parent / "src"))
+
+from textual.pilot import Pilot
+from eyelet.tui.app import EyeletApp
+
+
+async def test_exit_keys():
+    """Test that all exit keys work"""
+    app = EyeletApp()
+    
+    async with app.run_test() as pilot:
+        # Test q key
+        await pilot.press("q")
+        # App should have exited, but we can't test that directly
+        # Just make sure no exception was raised
+        
+    print("✓ 'q' key handled")
+    
+    # Test Ctrl+Q
+    app = EyeletApp()
+    async with app.run_test() as pilot:
+        await pilot.press("ctrl+q")
+    print("✓ 'ctrl+q' handled")
+    
+    # Test Escape
+    app = EyeletApp()
+    async with app.run_test() as pilot:
+        await pilot.press("escape")
+    print("✓ 'escape' key handled")
+    
+    print("\n✅ All exit keys working!")
+
+
+if __name__ == "__main__":
+    print("Testing TUI exit functionality...")
+    asyncio.run(test_exit_keys())