pomera-ai-commander 1.2.5 → 1.2.8

package/core/collapsible_panel.py

@@ -0,0 +1,197 @@
+"""
+Collapsible Panel Widget - Reusable collapsible container for UI sections.
+
+This module provides a collapsible panel widget that can be used to hide/show
+sections of the UI, saving screen space while keeping functionality accessible.
+
+Author: Pomera AI Commander Team
+"""
+
+import tkinter as tk
+from tkinter import ttk
+from typing import Optional, Callable
+import logging
+
+
+logger = logging.getLogger(__name__)
+
+
+class CollapsiblePanel(ttk.Frame):
+    """
+    A reusable collapsible panel widget.
+    
+    Features:
+    - Toggle button with chevron icon (▼/▲)
+    - Smooth collapse/expand animation
+    - State persistence via callback
+    - Keyboard shortcut support
+    - Optional title display
+    
+    Usage:
+        panel = CollapsiblePanel(
+            parent,
+            title="Options",
+            collapsed=False,
+            on_state_change=lambda collapsed: save_state(collapsed)
+        )
+        # Add content to panel.content_frame
+        ttk.Label(panel.content_frame, text="Panel content").pack()
+    """
+    
+    # Animation settings
+    ANIMATION_DURATION_MS = 200
+    ANIMATION_STEPS = 10
+    
+    def __init__(
+        self,
+        parent: tk.Widget,
+        title: str = "",
+        collapsed: bool = False,
+        on_state_change: Optional[Callable[[bool], None]] = None,
+        show_title: bool = True,
+        **kwargs
+    ):
+        """
+        Initialize the collapsible panel.
+        
+        Args:
+            parent: Parent widget
+            title: Panel title text
+            collapsed: Initial collapsed state
+            on_state_change: Callback when collapse state changes, receives bool
+            show_title: Whether to show the title text
+            **kwargs: Additional keyword arguments for ttk.Frame
+        """
+        super().__init__(parent, **kwargs)
+        
+        self.title = title
+        self._collapsed = collapsed
+        self._on_state_change = on_state_change
+        self._show_title = show_title
+        self._animation_id: Optional[str] = None
+        self._content_height: int = 0
+        
+        self._create_widgets()
+        self._apply_initial_state()
+    
+    def _create_widgets(self) -> None:
+        """Create the panel widgets."""
+        # Header frame with toggle button
+        self.header_frame = ttk.Frame(self)
+        self.header_frame.pack(fill=tk.X)
+        
+        # Toggle button with chevron
+        self.toggle_btn = ttk.Button(
+            self.header_frame,
+            text=self._get_toggle_text(),
+            command=self.toggle,
+            width=3
+        )
+        self.toggle_btn.pack(side=tk.LEFT, padx=(0, 5))
+        
+        # Title label with inline shortcut hint
+        if self._show_title and self.title:
+            self.title_label = ttk.Label(
+                self.header_frame,
+                text=f"{self.title} (Ctrl+Shift+H)",
+                font=("TkDefaultFont", 9, "bold")
+            )
+            self.title_label.pack(side=tk.LEFT)
+            # Make title clickable too
+            self.title_label.bind("<Button-1>", lambda e: self.toggle())
+        
+        # Content frame (what gets collapsed)
+        self.content_frame = ttk.Frame(self)
+        if not self._collapsed:
+            self.content_frame.pack(fill=tk.BOTH, expand=True, pady=(5, 0))
+    
+    def _get_toggle_text(self) -> str:
+        """Get the toggle button text based on collapsed state."""
+        return "▲" if self._collapsed else "▼"
+    
+    def _apply_initial_state(self) -> None:
+        """Apply the initial collapsed state."""
+        if self._collapsed:
+            self.content_frame.pack_forget()
+        self.toggle_btn.configure(text=self._get_toggle_text())
+    
+    @property
+    def collapsed(self) -> bool:
+        """Get the current collapsed state."""
+        return self._collapsed
+    
+    @collapsed.setter
+    def collapsed(self, value: bool) -> None:
+        """Set the collapsed state."""
+        if value != self._collapsed:
+            self._collapsed = value
+            self._update_state()
+    
+    def toggle(self) -> None:
+        """Toggle the collapsed state."""
+        self._collapsed = not self._collapsed
+        self._update_state()
+        
+        # Notify callback
+        if self._on_state_change:
+            try:
+                self._on_state_change(self._collapsed)
+            except Exception as e:
+                logger.warning(f"Error in on_state_change callback: {e}")
+    
+    def _update_state(self) -> None:
+        """Update the visual state based on collapsed flag."""
+        # Cancel any running animation
+        if self._animation_id:
+            self.after_cancel(self._animation_id)
+            self._animation_id = None
+        
+        # Update toggle button
+        self.toggle_btn.configure(text=self._get_toggle_text())
+        
+        # Show/hide content
+        if self._collapsed:
+            self.content_frame.pack_forget()
+        else:
+            self.content_frame.pack(fill=tk.BOTH, expand=True, pady=(5, 0))
+        
+        logger.debug(f"Panel '{self.title}' collapsed={self._collapsed}")
+    
+    def expand(self) -> None:
+        """Expand the panel if collapsed."""
+        if self._collapsed:
+            self.toggle()
+    
+    def collapse(self) -> None:
+        """Collapse the panel if expanded."""
+        if not self._collapsed:
+            self.toggle()
+    
+    def set_content_widget(self, widget: tk.Widget) -> None:
+        """
+        Set a widget as the panel content.
+        
+        Args:
+            widget: Widget to place inside the content frame
+        """
+        # Clear existing content
+        for child in self.content_frame.winfo_children():
+            child.destroy()
+        
+        # Add new content
+        widget.pack(in_=self.content_frame, fill=tk.BOTH, expand=True)
+    
+    def bind_shortcut(self, root: tk.Tk, shortcut: str = "<Control-Shift-H>") -> None:
+        """
+        Bind a keyboard shortcut to toggle the panel.
+        
+        Args:
+            root: Root window to bind the shortcut to
+            shortcut: Key sequence (default: Ctrl+Shift+H)
+        """
+        root.bind_all(shortcut, lambda e: self.toggle())
+        logger.debug(f"Bound shortcut {shortcut} to panel '{self.title}'")
+
+
+# Module availability flag for import checking
+COLLAPSIBLE_PANEL_AVAILABLE = True

package/core/database_settings_manager.py

@@ -45,12 +45,17 @@ class NestedSettingsProxy:
     def __getitem__(self, key: str) -> Any:
         """Handle nested access like settings["tool_settings"]["Tool Name"]."""
         if key not in self._data:
-            # For tool_settings, create empty tool settings when accessed
+            # For tool_settings, first try to load existing settings from database
             if self.parent_key == "tool_settings":
-                # Initialize empty tool settings
-                self._data[key] = {}
-                # Also save to database
-                self.settings_manager.set_tool_setting(key, "initialized", True)
+                existing_settings = self.settings_manager.get_tool_settings(key)
+                if existing_settings and not (len(existing_settings) == 1 and 'initialized' in existing_settings):
+                    # Found real settings in database - use them
+                    self._data[key] = existing_settings
+                else:
+                    # No existing settings - create empty tool settings
+                    self._data[key] = {}
+                    # Save initialized marker to database
+                    self.settings_manager.set_tool_setting(key, "initialized", True)
             else:
                 raise KeyError(f"Key '{key}' not found in {self.parent_key}")
         
@@ -91,10 +96,16 @@ class NestedSettingsProxy:
     def get(self, key: str, default: Any = None) -> Any:
         """Handle nested_settings.get("key", default) calls."""
         if key not in self._data and self.parent_key == "tool_settings":
-            # For tool_settings, create empty tool settings when accessed via get()
-            self._data[key] = {}
-            # Also save to database
-            self.settings_manager.set_tool_setting(key, "initialized", True)
+            # For tool_settings, first try to load existing settings from database
+            existing_settings = self.settings_manager.get_tool_settings(key)
+            if existing_settings and not (len(existing_settings) == 1 and 'initialized' in existing_settings):
+                # Found real settings in database - use them
+                self._data[key] = existing_settings
+            else:
+                # No existing settings - create empty tool settings
+                self._data[key] = {}
+                # Save initialized marker to database
+                self.settings_manager.set_tool_setting(key, "initialized", True)
         
         value = self._data.get(key, default)
         

package/core/migration_manager.py

@@ -324,8 +324,9 @@ class MigrationManager:
         """
         try:
             with self.connection_manager.transaction() as conn:
-                # Clear existing data
-                self._clear_all_tables(conn)
+                # NOTE: Do NOT clear tables - use INSERT OR REPLACE for upsert semantics
+                # The _clear_all_tables() call was removed because it caused data loss
+                # when save_settings() was called with incomplete/empty data
                 
                 # Migrate core settings
                 self._migrate_core_settings(conn, json_data)
@@ -432,7 +433,7 @@ class MigrationManager:
                 serialized_value = self.converter.serialize_value(value)
                 
                 conn.execute(
-                    "INSERT INTO core_settings (key, value, data_type) VALUES (?, ?, ?)",
+                    "INSERT OR REPLACE INTO core_settings (key, value, data_type) VALUES (?, ?, ?)",
                     (key, serialized_value, data_type)
                 )
     
@@ -454,7 +455,7 @@ class MigrationManager:
                     serialized_value = self.converter.serialize_value(value)
                     
                     conn.execute(
-                        "INSERT INTO tool_settings (tool_name, setting_path, setting_value, data_type) VALUES (?, ?, ?, ?)",
+                        "INSERT OR REPLACE INTO tool_settings (tool_name, setting_path, setting_value, data_type) VALUES (?, ?, ?, ?)",
                         (tool_name, setting_path, serialized_value, data_type)
                     )
             else:
@@ -463,7 +464,7 @@ class MigrationManager:
                 serialized_value = self.converter.serialize_value(tool_config)
                 
                 conn.execute(
-                    "INSERT INTO tool_settings (tool_name, setting_path, setting_value, data_type) VALUES (?, ?, ?, ?)",
+                    "INSERT OR REPLACE INTO tool_settings (tool_name, setting_path, setting_value, data_type) VALUES (?, ?, ?, ?)",
                     (tool_name, 'value', serialized_value, data_type)
                 )
     
@@ -480,7 +481,7 @@ class MigrationManager:
             input_tabs = json_data['input_tabs']
             for i, content in enumerate(input_tabs):
                 conn.execute(
-                    "INSERT INTO tab_content (tab_type, tab_index, content) VALUES (?, ?, ?)",
+                    "INSERT OR REPLACE INTO tab_content (tab_type, tab_index, content) VALUES (?, ?, ?)",
                     ('input', i, content or '')
                 )
         
@@ -489,7 +490,7 @@ class MigrationManager:
             output_tabs = json_data['output_tabs']
             for i, content in enumerate(output_tabs):
                 conn.execute(
-                    "INSERT INTO tab_content (tab_type, tab_index, content) VALUES (?, ?, ?)",
+                    "INSERT OR REPLACE INTO tab_content (tab_type, tab_index, content) VALUES (?, ?, ?)",
                     ('output', i, content or '')
                 )
     
@@ -511,7 +512,7 @@ class MigrationManager:
                     serialized_value = self.converter.serialize_value(value)
                     
                     conn.execute(
-                        "INSERT INTO performance_settings (category, setting_key, setting_value, data_type) VALUES (?, ?, ?, ?)",
+                        "INSERT OR REPLACE INTO performance_settings (category, setting_key, setting_value, data_type) VALUES (?, ?, ?, ?)",
                         (category, setting_key, serialized_value, data_type)
                     )
             else:
@@ -520,7 +521,7 @@ class MigrationManager:
                 serialized_value = self.converter.serialize_value(settings)
                 
                 conn.execute(
-                    "INSERT INTO performance_settings (category, setting_key, setting_value, data_type) VALUES (?, ?, ?, ?)",
+                    "INSERT OR REPLACE INTO performance_settings (category, setting_key, setting_value, data_type) VALUES (?, ?, ?, ?)",
                     (category, 'value', serialized_value, data_type)
                 )
     
@@ -539,7 +540,7 @@ class MigrationManager:
                     serialized_value = self.converter.serialize_value(value)
                     
                     conn.execute(
-                        "INSERT INTO font_settings (font_type, property, value, data_type) VALUES (?, ?, ?, ?)",
+                        "INSERT OR REPLACE INTO font_settings (font_type, property, value, data_type) VALUES (?, ?, ?, ?)",
                         (font_type, property_name, serialized_value, data_type)
                     )
     
@@ -560,7 +561,7 @@ class MigrationManager:
                     self.logger.debug(f"Inserting dialog setting: {category}.{property_name} = {value} (type: {data_type})")
                     
                     conn.execute(
-                        "INSERT INTO dialog_settings (category, property, value, data_type) VALUES (?, ?, ?, ?)",
+                        "INSERT OR REPLACE INTO dialog_settings (category, property, value, data_type) VALUES (?, ?, ?, ?)",
                         (category, property_name, serialized_value, data_type)
                     )
     

package/core/settings_defaults_registry.py

@@ -833,6 +833,14 @@ class SettingsDefaultsRegistry:
                     "description": "Error messages for critical issues (cannot be disabled)",
                     "examples": ["File not found", "Network error", "Invalid configuration"]
                 }
+            },
+            # UI Layout Settings for tool search and collapsible panels
+            "ui_layout": {
+                "options_panel_collapsed": False,
+                "search_bar_collapsed": False,
+                "favorite_tools": [],
+                "recent_tools": [],
+                "recent_tools_max": 10
             }
         }