chat-console 0.3.8__py3-none-any.whl → 0.3.91__py3-none-any.whl

app/config.py

@@ -175,35 +175,38 @@ CONFIG = load_config()
 
 # --- Dynamically update Anthropic models after initial load ---
 def update_anthropic_models(config):
-    """Fetch models from Anthropic API and update the config dict."""
+    """Update the config with Anthropic models."""
     if AVAILABLE_PROVIDERS["anthropic"]:
         try:
-            from app.api.anthropic import AnthropicClient # Import here to avoid circular dependency at top level
-            client = AnthropicClient()
-            fetched_models = client.get_available_models() # This now fetches (or uses fallback)
-
-            if fetched_models:
-                # Remove old hardcoded anthropic models first
-                models_to_remove = [
-                    model_id for model_id, info in config["available_models"].items()
-                    if info.get("provider") == "anthropic"
-                ]
-                for model_id in models_to_remove:
-                    del config["available_models"][model_id]
-
-                # Add fetched models
-                for model in fetched_models:
-                    config["available_models"][model["id"]] = {
-                        "provider": "anthropic",
-                        "max_tokens": 4096, # Assign a default max_tokens
-                        "display_name": model["name"]
-                    }
-                print(f"Updated Anthropic models in config: {[m['id'] for m in fetched_models]}") # Add print statement
-            else:
-                 print("Could not fetch or find Anthropic models to update config.") # Add print statement
-
+            # Instead of calling an async method, use a hardcoded fallback list
+            # that matches what's in the AnthropicClient class
+            fallback_models = [
+                {"id": "claude-3-opus-20240229", "name": "Claude 3 Opus"},
+                {"id": "claude-3-sonnet-20240229", "name": "Claude 3 Sonnet"},
+                {"id": "claude-3-haiku-20240307", "name": "Claude 3 Haiku"},
+                {"id": "claude-3-5-sonnet-20240620", "name": "Claude 3.5 Sonnet"},
+                {"id": "claude-3-7-sonnet-20250219", "name": "Claude 3.7 Sonnet"},
+            ]
+            
+            # Remove old models first
+            models_to_remove = [
+                model_id for model_id, info in config["available_models"].items()
+                if info.get("provider") == "anthropic"
+            ]
+            for model_id in models_to_remove:
+                del config["available_models"][model_id]
+                
+            # Add the fallback models
+            for model in fallback_models:
+                config["available_models"][model["id"]] = {
+                    "provider": "anthropic",
+                    "max_tokens": 4096,
+                    "display_name": model["name"]
+                }
+            print(f"Updated Anthropic models in config with fallback list")
+            
         except Exception as e:
-            print(f"Error updating Anthropic models in config: {e}") # Add print statement
+            print(f"Error updating Anthropic models in config: {e}")
             # Keep existing config if update fails
 
     return config

app/main.py

@@ -20,10 +20,10 @@ file_handler = logging.FileHandler(debug_log_file)
 file_handler.setFormatter(logging.Formatter('%(asctime)s - %(name)s - %(levelname)s - %(message)s'))
 
 # Get the logger and add the handler
-debug_logger = logging.getLogger("chat-cli-debug")
+debug_logger = logging.getLogger()  # Root logger
 debug_logger.setLevel(logging.DEBUG)
 debug_logger.addHandler(file_handler)
-# Prevent propagation to the root logger (which would print to console)
+# CRITICAL: Force all output to the file, not stdout
 debug_logger.propagate = False
 
 # Add a convenience function to log to this file
@@ -1010,11 +1010,15 @@ class SimpleChatApp(App): # Keep SimpleChatApp class definition
                 self._loading_animation_task.cancel()
             self._loading_animation_task = None
             try:
+                # Explicitly hide loading indicator
                 loading = self.query_one("#loading-indicator")
                 loading.add_class("hidden")
+                loading.remove_class("model-loading")  # Also remove model-loading class if present
+                self.refresh(layout=True)  # Force a refresh to ensure UI updates
                 self.query_one("#message-input").focus()
-            except Exception:
-                pass # Ignore UI errors during cleanup
+            except Exception as ui_err:
+                debug_log(f"Error hiding loading indicator: {str(ui_err)}")
+                log.error(f"Error hiding loading indicator: {str(ui_err)}")
 
     # Rename this method slightly to avoid potential conflicts and clarify purpose
     async def _handle_generation_result(self, worker: Worker[Optional[str]]) -> None:
@@ -1043,6 +1047,15 @@ class SimpleChatApp(App): # Keep SimpleChatApp class definition
                 debug_log(f"Error in generation worker: {error}")
                 log.error(f"Error in generation worker: {error}")
                 
+                # Explicitly hide loading indicator
+                try:
+                    loading = self.query_one("#loading-indicator")
+                    loading.add_class("hidden")
+                    loading.remove_class("model-loading")  # Also remove model-loading class if present
+                except Exception as ui_err:
+                    debug_log(f"Error hiding loading indicator: {str(ui_err)}")
+                    log.error(f"Error hiding loading indicator: {str(ui_err)}")
+                
                 # Sanitize error message for UI display
                 error_str = str(error)
                 
@@ -1069,6 +1082,9 @@ class SimpleChatApp(App): # Keep SimpleChatApp class definition
                 debug_log(f"Adding error message: {user_error}")
                 self.messages.append(Message(role="assistant", content=user_error))
                 await self.update_messages_ui()
+                
+                # Force a refresh to ensure UI updates
+                self.refresh(layout=True)
 
             elif worker.state == "success":
                 full_response = worker.result

app/ui/chat_interface.py

@@ -120,54 +120,61 @@ class MessageDisplay(Static): # Inherit from Static instead of RichLog
         self.update(self._format_content(self.message.content))
         
     async def update_content(self, content: str) -> None:
-        """Update the message content using Static.update() with optimizations for streaming"""
-        # Debug print to verify method is being called with content
-        print(f"MessageDisplay.update_content called with content length: {len(content)}")
-        
-        # Quick unchanged content check to avoid unnecessary updates
-        if self.message.content == content:
-            print("Content unchanged, skipping update")
-            return
-            
-        # Special handling for "Thinking..." to ensure it gets replaced
-        if self.message.content == "Thinking..." and content:
-            print("Replacing 'Thinking...' with actual content")
-            # Force a complete replacement rather than an append
-            self.message.content = ""
-            # Add a debug print to confirm this branch is executed
-            print("CRITICAL FIX: Replacing 'Thinking...' placeholder with actual content")
-            
-        # Update the stored message object content
-        self.message.content = content
-        
-        # Format with fixed-width placeholder to minimize layout shifts
-        # This avoids text reflowing as new tokens arrive
-        formatted_content = self._format_content(content)
-        
-        # Use a direct update that forces refresh - critical fix for streaming
-        # This ensures content is immediately visible
-        print(f"Updating widget with formatted content length: {len(formatted_content)}")
-        self.update(formatted_content, refresh=True)
-        
-        # Force app-level refresh and scroll to ensure visibility
-        try:
-            # Always force app refresh for every update
-            if self.app:
-                # Force a full layout refresh to ensure content is visible
-                self.app.refresh(layout=True)
+        """Update the message content."""
+        import logging
+        logger = logging.getLogger(__name__)
+        logger.debug(f"MessageDisplay.update_content called with content length: {len(content)}")
+        
+        # Use a lock to prevent race conditions during updates
+        if not hasattr(self, '_update_lock'):
+            self._update_lock = asyncio.Lock()
+        
+        async with self._update_lock:
+            # For initial update from "Thinking..."
+            if self.message.content == "Thinking..." and content:
+                logger.debug("Replacing 'Thinking...' with initial content")
+                self.message.content = content  # Update the stored content
+                formatted = self._format_content(content)
+                self.update(formatted, refresh=True)
                 
-                # Find the messages container and scroll to end
-                containers = self.app.query("ScrollableContainer")
-                for container in containers:
-                    if hasattr(container, 'scroll_end'):
-                        container.scroll_end(animate=False)
+                # Force a clean layout update
+                try:
+                    if self.app:
+                        self.app.refresh(layout=True)
+                        await asyncio.sleep(0.05)  # Small delay for layout to update
                         
-                # Add an additional refresh after scrolling
-                self.app.refresh(layout=True)
-        except Exception as e:
-            # Log the error and fallback to local refresh
-            print(f"Error refreshing app: {str(e)}")
-            self.refresh(layout=True)
+                        # Find container and scroll
+                        messages_container = self.app.query_one("#messages-container")
+                        if messages_container:
+                            messages_container.scroll_end(animate=False)
+                except Exception as e:
+                    logger.error(f"Error in initial UI update: {str(e)}")
+                return
+                
+            # Quick unchanged content check to avoid unnecessary updates
+            if self.message.content == content:
+                logger.debug("Content unchanged, skipping update")
+                return
+                
+            # For subsequent updates
+            if self.message.content != content:
+                self.message.content = content
+                formatted = self._format_content(content)
+                self.update(formatted, refresh=True)
+                
+                # Use a more targeted refresh approach
+                try:
+                    if self.app:
+                        self.app.refresh(layout=False)  # Lightweight refresh first
+                        # Find container and scroll
+                        messages_container = self.app.query_one("#messages-container")
+                        if messages_container:
+                            messages_container.scroll_end(animate=False)
+                            
+                        # Final full refresh only at end
+                        self.app.refresh(layout=True)
+                except Exception as e:
+                    logger.error(f"Error refreshing UI: {str(e)}")
         
     def _format_content(self, content: str) -> str:
         """Format message content with timestamp and handle markdown links"""
@@ -191,8 +198,8 @@ class MessageDisplay(Static): # Inherit from Static instead of RichLog
         # But keep our timestamp markup
         timestamp_markup = f"[dim]{timestamp}[/dim]"
         
-        # Debug print to verify content is being formatted
-        print(f"Formatting content: {len(content)} chars")
+        # Use proper logging instead of print
+        logger.debug(f"Formatting content: {len(content)} chars")
         
         return f"{timestamp_markup} {content}"