code-puppy 0.0.302__py3-none-any.whl → 0.0.323__py3-none-any.whl

code_puppy/command_line/core_commands.py

@@ -145,6 +145,8 @@ def handle_exit_command(command: str) -> bool:
 )
 def handle_agent_command(command: str) -> bool:
     """Handle agent switching."""
+    from rich.text import Text
+
     from code_puppy.agents import (
         get_agent_descriptions,
         get_available_agents,
@@ -201,7 +203,9 @@ def handle_agent_command(command: str) -> bool:
                 )
                 emit_info(f"{new_agent.description}", message_group=group_id)
                 emit_info(
-                    f"[dim]Auto-save session rotated to: {new_session_id}[/dim]",
+                    Text.from_markup(
+                        f"[dim]Auto-save session rotated to: {new_session_id}[/dim]"
+                    ),
                     message_group=group_id,
                 )
             else:
@@ -224,15 +228,19 @@ def handle_agent_command(command: str) -> bool:
             group_id = str(uuid.uuid4())
 
             emit_info(
-                f"[bold green]Current Agent:[/bold green] {current_agent.display_name}",
+                Text.from_markup(
+                    f"[bold green]Current Agent:[/bold green] {current_agent.display_name}"
+                ),
                 message_group=group_id,
             )
             emit_info(
-                f"[dim]{current_agent.description}[/dim]\n", message_group=group_id
+                Text.from_markup(f"[dim]{current_agent.description}[/dim]\n"),
+                message_group=group_id,
             )
 
             emit_info(
-                "[bold magenta]Available Agents:[/bold magenta]", message_group=group_id
+                Text.from_markup("[bold magenta]Available Agents:[/bold magenta]"),
+                message_group=group_id,
             )
             for name, display_name in available_agents.items():
                 description = descriptions.get(name, "No description")
@@ -240,13 +248,15 @@ def handle_agent_command(command: str) -> bool:
                     " [green]← current[/green]" if name == current_agent.name else ""
                 )
                 emit_info(
-                    f"  [cyan]{name:<12}[/cyan] {display_name}{current_marker}",
+                    Text.from_markup(
+                        f"  [cyan]{name:<12}[/cyan] {display_name}{current_marker}"
+                    ),
                     message_group=group_id,
                 )
                 emit_info(f"    {description}", message_group=group_id)
 
             emit_info(
-                "\n[yellow]Usage:[/yellow] /agent <agent-name>",
+                Text.from_markup("\n[yellow]Usage:[/yellow] /agent <agent-name>"),
                 message_group=group_id,
             )
             return True
@@ -292,7 +302,9 @@ def handle_agent_command(command: str) -> bool:
         )
         emit_info(f"{new_agent.description}", message_group=group_id)
         emit_info(
-            f"[dim]Auto-save session rotated to: {new_session_id}[/dim]",
+            Text.from_markup(
+                f"[dim]Auto-save session rotated to: {new_session_id}[/dim]"
+            ),
             message_group=group_id,
         )
         return True

code_puppy/command_line/mcp/edit_command.py

@@ -8,6 +8,8 @@ import logging
 import os
 from typing import List, Optional
 
+from rich.text import Text
+
 from code_puppy.config import MCP_SERVERS_FILE
 from code_puppy.messaging import emit_error, emit_info, emit_warning
 
@@ -39,7 +41,7 @@ class EditCommand(MCPCommandBase):
             # Need a server name
             if not args:
                 emit_info(
-                    "[yellow]Usage: /mcp edit <server_name>[/yellow]",
+                    Text.from_markup("[yellow]Usage: /mcp edit <server_name>[/yellow]"),
                     message_group=group_id,
                 )
                 emit_info(

code_puppy/command_line/mcp/handler.py

@@ -8,6 +8,8 @@ to their respective command modules.
 import logging
 import shlex
 
+from rich.text import Text
+
 from code_puppy.messaging import emit_info
 
 from .add_command import AddCommand
@@ -103,7 +105,8 @@ class MCPCommandHandler(MCPCommandBase):
                 args = shlex.split(args_str)
             except ValueError as e:
                 emit_info(
-                    f"[red]Invalid command syntax: {e}[/red]", message_group=group_id
+                    Text.from_markup(f"[red]Invalid command syntax: {e}[/red]"),
+                    message_group=group_id,
                 )
                 return True
 
@@ -121,7 +124,9 @@ class MCPCommandHandler(MCPCommandBase):
                 return True
             else:
                 emit_info(
-                    f"[yellow]Unknown MCP subcommand: {subcommand}[/yellow]",
+                    Text.from_markup(
+                        f"[yellow]Unknown MCP subcommand: {subcommand}[/yellow]"
+                    ),
                     message_group=group_id,
                 )
                 emit_info(

code_puppy/command_line/mcp/install_command.py

@@ -5,6 +5,8 @@ MCP Install Command - Installs pre-configured MCP servers from the registry.
 import logging
 from typing import List, Optional
 
+from rich.text import Text
+
 from code_puppy.messaging import emit_error, emit_info
 
 from .base import MCPCommandBase
@@ -150,7 +152,9 @@ class InstallCommand(MCPCommandBase):
             required_env_vars = selected_server.get_environment_vars()
             if required_env_vars:
                 emit_info(
-                    "\n[yellow]Required Environment Variables:[/yellow]",
+                    Text.from_markup(
+                        "\n[yellow]Required Environment Variables:[/yellow]"
+                    ),
                     message_group=group_id,
                 )
                 for var in required_env_vars:
@@ -160,7 +164,7 @@ class InstallCommand(MCPCommandBase):
                     current_value = os.environ.get(var, "")
                     if current_value:
                         emit_info(
-                            f"  {var}: [green]Already set[/green]",
+                            Text.from_markup(f"  {var}: [green]Already set[/green]"),
                             message_group=group_id,
                         )
                         env_vars[var] = current_value
@@ -173,7 +177,8 @@ class InstallCommand(MCPCommandBase):
             required_cmd_args = selected_server.get_command_line_args()
             if required_cmd_args:
                 emit_info(
-                    "\n[yellow]Command Line Arguments:[/yellow]", message_group=group_id
+                    Text.from_markup("\n[yellow]Command Line Arguments:[/yellow]"),
+                    message_group=group_id,
                 )
                 for arg_config in required_cmd_args:
                     name = arg_config.get("name", "")

code_puppy/command_line/mcp/logs_command.py

@@ -1,14 +1,21 @@
 """
-MCP Logs Command - Shows recent events/logs for a server.
+MCP Logs Command - Shows server logs from persistent log files.
 """
 
 import logging
-from datetime import datetime
 from typing import List, Optional
 
-from rich.table import Table
+from rich.panel import Panel
+from rich.syntax import Syntax
 from rich.text import Text
 
+from code_puppy.mcp_.mcp_logs import (
+    clear_logs,
+    get_log_file_path,
+    get_log_stats,
+    list_servers_with_logs,
+    read_logs,
+)
 from code_puppy.messaging import emit_error, emit_info
 
 from .base import MCPCommandBase
@@ -22,105 +29,207 @@ class LogsCommand(MCPCommandBase):
     """
     Command handler for showing MCP server logs.
 
-    Shows recent events/logs for a specific MCP server with configurable limit.
+    Shows logs from persistent log files stored in ~/.code_puppy/mcp_logs/.
     """
 
     def execute(self, args: List[str], group_id: Optional[str] = None) -> None:
         """
-        Show recent events/logs for a server.
+        Show logs for a server.
+
+        Usage:
+            /mcp logs                    - List servers with logs
+            /mcp logs <server_name>      - Show last 50 lines
+            /mcp logs <server_name> 100  - Show last 100 lines
+            /mcp logs <server_name> all  - Show all logs
+            /mcp logs <server_name> --clear - Clear logs for server
 
         Args:
-            args: Command arguments, expects [server_name] and optional [limit]
+            args: Command arguments
             group_id: Optional message group ID for grouping related messages
         """
         if group_id is None:
             group_id = self.generate_group_id()
 
+        # No args - list servers with logs
         if not args:
-            emit_info("Usage: /mcp logs <server_name> [limit]", message_group=group_id)
+            self._list_servers_with_logs(group_id)
             return
 
         server_name = args[0]
-        limit = 10  # Default limit
+
+        # Check for --clear flag
+        if len(args) > 1 and args[1] == "--clear":
+            self._clear_logs(server_name, group_id)
+            return
+
+        # Determine number of lines
+        lines = 50  # Default
+        show_all = False
 
         if len(args) > 1:
-            try:
-                limit = int(args[1])
-                if limit <= 0 or limit > 100:
+            if args[1].lower() == "all":
+                show_all = True
+            else:
+                try:
+                    lines = int(args[1])
+                    if lines <= 0:
+                        emit_info(
+                            "Lines must be positive, using default: 50",
+                            message_group=group_id,
+                        )
+                        lines = 50
+                except ValueError:
                     emit_info(
-                        "Limit must be between 1 and 100, using default: 10",
+                        f"Invalid number '{args[1]}', using default: 50",
                         message_group=group_id,
                     )
-                    limit = 10
-            except ValueError:
-                emit_info(
-                    f"Invalid limit '{args[1]}', using default: 10",
-                    message_group=group_id,
-                )
 
+        self._show_logs(server_name, lines if not show_all else None, group_id)
+
+    def _list_servers_with_logs(self, group_id: str) -> None:
+        """List all servers that have log files."""
+        servers = list_servers_with_logs()
+
+        if not servers:
+            emit_info(
+                "📋 No MCP server logs found.\n"
+                "Logs are created when servers are started.",
+                message_group=group_id,
+            )
+            return
+
+        lines = ["📋 **Servers with logs:**\n"]
+
+        for server in servers:
+            stats = get_log_stats(server)
+            size_kb = stats["total_size_bytes"] / 1024
+            size_str = (
+                f"{size_kb:.1f} KB" if size_kb < 1024 else f"{size_kb / 1024:.1f} MB"
+            )
+            rotated = (
+                f" (+{stats['rotated_count']} rotated)"
+                if stats["rotated_count"]
+                else ""
+            )
+            lines.append(
+                f"  • **{server}** - {stats['line_count']} lines, {size_str}{rotated}"
+            )
+
+        lines.append("\n**Usage:** `/mcp logs <server_name> [lines|all]`")
+
+        emit_info("\n".join(lines), message_group=group_id)
+
+    def _show_logs(self, server_name: str, lines: Optional[int], group_id: str) -> None:
+        """
+        Show logs for a specific server.
+
+        Args:
+            server_name: Name of the server
+            lines: Number of lines to show, or None for all
+            group_id: Message group ID
+        """
         try:
-            # Find server by name
+            # Verify server exists in manager
             server_id = find_server_id_by_name(self.manager, server_name)
             if not server_id:
-                emit_info(f"Server '{server_name}' not found", message_group=group_id)
-                suggest_similar_servers(self.manager, server_name, group_id=group_id)
-                return
+                # Server not configured, but might have logs from before
+                stats = get_log_stats(server_name)
+                if not stats["exists"]:
+                    emit_info(
+                        f"Server '{server_name}' not found and has no logs.",
+                        message_group=group_id,
+                    )
+                    suggest_similar_servers(
+                        self.manager, server_name, group_id=group_id
+                    )
+                    return
 
-            # Get server status which includes recent events
-            status = self.manager.get_server_status(server_id)
+            # Read logs
+            log_lines = read_logs(server_name, lines=lines)
 
-            if not status.get("exists", True):
+            if not log_lines:
                 emit_info(
-                    f"Server '{server_name}' status not available",
+                    f"📋 No logs found for server: **{server_name}**\n"
+                    f"Log file: `{get_log_file_path(server_name)}`",
                     message_group=group_id,
                 )
                 return
 
-            recent_events = status.get("recent_events", [])
-
-            if not recent_events:
-                emit_info(
-                    f"No recent events for server: {server_name}",
-                    message_group=group_id,
+            # Get stats for header
+            stats = get_log_stats(server_name)
+            total_lines = stats["line_count"]
+            showing = len(log_lines)
+
+            # Format header
+            if lines is None:
+                header = f"📋 Logs for {server_name} (all {total_lines} lines)"
+            else:
+                header = (
+                    f"📋 Logs for {server_name} (last {showing} of {total_lines} lines)"
                 )
-                return
 
-            # Show events in a table
-            table = Table(title=f"📋 Recent Events for {server_name} (last {limit})")
-            table.add_column("Time", style="dim", no_wrap=True)
-            table.add_column("Event", style="cyan")
-            table.add_column("Details", style="dim")
+            # Format log content with syntax highlighting
+            log_content = "\n".join(log_lines)
+
+            # Create a panel with the logs
+            syntax = Syntax(
+                log_content,
+                "log",
+                theme="monokai",
+                word_wrap=True,
+                line_numbers=False,
+            )
 
-            # Take only the requested number of events
-            events_to_show = (
-                recent_events[-limit:] if len(recent_events) > limit else recent_events
+            panel = Panel(
+                syntax,
+                title=header,
+                subtitle=f"Log file: {get_log_file_path(server_name)}",
+                border_style="dim",
             )
 
-            for event in reversed(events_to_show):  # Show newest first
-                timestamp = datetime.fromisoformat(event["timestamp"])
-                time_str = timestamp.strftime("%H:%M:%S")
-                event_type = event["event_type"]
-
-                # Format details
-                details = event.get("details", {})
-                details_str = details.get("message", "")
-                if not details_str and "error" in details:
-                    details_str = str(details["error"])
-
-                # Color code event types
-                event_style = "cyan"
-                if "error" in event_type.lower():
-                    event_style = "red"
-                elif event_type in ["started", "enabled", "registered"]:
-                    event_style = "green"
-                elif event_type in ["stopped", "disabled"]:
-                    event_style = "yellow"
-
-                table.add_row(
-                    time_str, Text(event_type, style=event_style), details_str or "-"
+            emit_info(panel, message_group=group_id)
+
+            # Show hint for more options
+            if lines is not None and showing < total_lines:
+                emit_info(
+                    Text.from_markup(
+                        f"[dim]💡 Use `/mcp logs {server_name} all` to see all logs, "
+                        f"or `/mcp logs {server_name} <number>` for specific count[/dim]"
+                    ),
+                    message_group=group_id,
                 )
-            emit_info(table, message_group=group_id)
 
         except Exception as e:
             logger.error(f"Error getting logs for server '{server_name}': {e}")
             emit_error(f"Error getting logs: {e}", message_group=group_id)
+
+    def _clear_logs(self, server_name: str, group_id: str) -> None:
+        """
+        Clear logs for a specific server.
+
+        Args:
+            server_name: Name of the server
+            group_id: Message group ID
+        """
+        try:
+            stats = get_log_stats(server_name)
+
+            if not stats["exists"] and stats["rotated_count"] == 0:
+                emit_info(
+                    f"No logs to clear for server: {server_name}",
+                    message_group=group_id,
+                )
+                return
+
+            # Clear the logs
+            clear_logs(server_name, include_rotated=True)
+
+            cleared_count = 1 + stats["rotated_count"]
+            emit_info(
+                f"🗑️  Cleared {cleared_count} log file(s) for **{server_name}**",
+                message_group=group_id,
+            )
+
+        except Exception as e:
+            logger.error(f"Error clearing logs for server '{server_name}': {e}")
+            emit_error(f"Error clearing logs: {e}", message_group=group_id)

code_puppy/command_line/mcp/restart_command.py

@@ -5,6 +5,8 @@ MCP Restart Command - Restarts a specific MCP server.
 import logging
 from typing import List, Optional
 
+from rich.text import Text
+
 from code_puppy.messaging import emit_info
 
 from .base import MCPCommandBase
@@ -72,7 +74,9 @@ class RestartCommand(MCPCommandBase):
                         # Update MCP tool cache immediately so token counts reflect the change
                         agent.update_mcp_tool_cache_sync()
                         emit_info(
-                            "[dim]Agent reloaded with updated servers[/dim]",
+                            Text.from_markup(
+                                "[dim]Agent reloaded with updated servers[/dim]"
+                            ),
                             message_group=group_id,
                         )
                     except Exception as e:
@@ -91,5 +95,6 @@ class RestartCommand(MCPCommandBase):
         except Exception as e:
             logger.error(f"Error restarting server '{server_name}': {e}")
             emit_info(
-                f"[red]Failed to restart server: {e}[/red]", message_group=group_id
+                Text.from_markup(f"[red]Failed to restart server: {e}[/red]"),
+                message_group=group_id,
             )

code_puppy/command_line/mcp/search_command.py

@@ -6,6 +6,7 @@ import logging
 from typing import List, Optional
 
 from rich.table import Table
+from rich.text import Text
 
 from code_puppy.messaging import emit_info, emit_system_message, emit_warning
 
@@ -99,19 +100,24 @@ class SearchCommand(MCPCommandBase):
             emit_system_message(table, message_group=group_id)
             emit_info("\n✓ = Verified  ⭐ = Popular", message_group=group_id)
             emit_info(
-                "[yellow]To install:[/yellow] /mcp install <id>", message_group=group_id
+                Text.from_markup("[yellow]To install:[/yellow] /mcp install <id>"),
+                message_group=group_id,
             )
             emit_info(
-                "[yellow]For details:[/yellow] /mcp search <specific-term>",
+                Text.from_markup(
+                    "[yellow]For details:[/yellow] /mcp search <specific-term>"
+                ),
                 message_group=group_id,
             )
 
         except ImportError:
             emit_info(
-                "[red]Server registry not available[/red]", message_group=group_id
+                Text.from_markup("[red]Server registry not available[/red]"),
+                message_group=group_id,
             )
         except Exception as e:
             logger.error(f"Error searching server registry: {e}")
             emit_info(
-                f"[red]Error searching servers: {e}[/red]", message_group=group_id
+                Text.from_markup(f"[red]Error searching servers: {e}[/red]"),
+                message_group=group_id,
             )

code_puppy/command_line/mcp/start_all_command.py

@@ -6,6 +6,8 @@ import logging
 import time
 from typing import List, Optional
 
+from rich.text import Text
+
 from code_puppy.mcp_.managed_server import ServerState
 from code_puppy.messaging import emit_info
 
@@ -67,20 +69,23 @@ class StartAllCommand(MCPCommandBase):
                 if success:
                     started_count += 1
                     emit_info(
-                        f"  [green]✓ Started: {server_name}[/green]",
+                        Text.from_markup(f"  [green]✓ Started: {server_name}[/green]"),
                         message_group=group_id,
                     )
                 else:
                     failed_count += 1
                     emit_info(
-                        f"  [red]✗ Failed: {server_name}[/red]", message_group=group_id
+                        Text.from_markup(f"  [red]✗ Failed: {server_name}[/red]"),
+                        message_group=group_id,
                     )
 
             # Summary
             emit_info("", message_group=group_id)
             if started_count > 0:
                 emit_info(
-                    f"[green]Started {started_count} server(s)[/green]",
+                    Text.from_markup(
+                        f"[green]Started {started_count} server(s)[/green]"
+                    ),
                     message_group=group_id,
                 )
             if already_running > 0:
@@ -90,7 +95,9 @@ class StartAllCommand(MCPCommandBase):
                 )
             if failed_count > 0:
                 emit_info(
-                    f"[yellow]Failed to start {failed_count} server(s)[/yellow]",
+                    Text.from_markup(
+                        f"[yellow]Failed to start {failed_count} server(s)[/yellow]"
+                    ),
                     message_group=group_id,
                 )
 
@@ -112,7 +119,9 @@ class StartAllCommand(MCPCommandBase):
                     # Update MCP tool cache immediately so token counts reflect the change
                     agent.update_mcp_tool_cache_sync()
                     emit_info(
-                        "[dim]Agent reloaded with updated servers[/dim]",
+                        Text.from_markup(
+                            "[dim]Agent reloaded with updated servers[/dim]"
+                        ),
                         message_group=group_id,
                     )
                 except Exception as e:
@@ -121,5 +130,6 @@ class StartAllCommand(MCPCommandBase):
         except Exception as e:
             logger.error(f"Error starting all servers: {e}")
             emit_info(
-                f"[red]Failed to start servers: {e}[/red]", message_group=group_id
+                Text.from_markup(f"[red]Failed to start servers: {e}[/red]"),
+                message_group=group_id,
             )

code_puppy/command_line/mcp/start_command.py

@@ -6,6 +6,8 @@ import logging
 import time
 from typing import List, Optional
 
+from rich.text import Text
+
 from code_puppy.messaging import emit_error, emit_info, emit_success
 
 from ...agents import get_current_agent
@@ -36,7 +38,7 @@ class StartCommand(MCPCommandBase):
 
         if not args:
             emit_info(
-                "[yellow]Usage: /mcp start <server_name>[/yellow]",
+                Text.from_markup("[yellow]Usage: /mcp start <server_name>[/yellow]"),
                 message_group=group_id,
             )
             return

code_puppy/command_line/mcp/status_command.py

@@ -7,6 +7,7 @@ from datetime import datetime
 from typing import List, Optional
 
 from rich.panel import Panel
+from rich.text import Text
 
 from code_puppy.mcp_.managed_server import ServerState
 from code_puppy.messaging import emit_error, emit_info
@@ -158,7 +159,7 @@ class StatusCommand(MCPCommandBase):
                 status_lines.append(f"[bold]Metadata:[/bold] {len(metadata)} keys")
 
             # Create and show the panel
-            panel_content = "\n".join(status_lines)
+            panel_content = Text.from_markup("\n".join(status_lines))
             panel = Panel(
                 panel_content, title=f"🔌 {server_name} Status", border_style="cyan"
             )

code_puppy/command_line/mcp/stop_all_command.py

@@ -6,6 +6,8 @@ import logging
 import time
 from typing import List, Optional
 
+from rich.text import Text
+
 from code_puppy.mcp_.managed_server import ServerState
 from code_puppy.messaging import emit_info
 
@@ -97,7 +99,9 @@ class StopAllCommand(MCPCommandBase):
                     # Update MCP tool cache immediately so token counts reflect the change
                     agent.update_mcp_tool_cache_sync()
                     emit_info(
-                        "[dim]Agent reloaded with updated servers[/dim]",
+                        Text.from_markup(
+                            "[dim]Agent reloaded with updated servers[/dim]"
+                        ),
                         message_group=group_id,
                     )
                 except Exception as e:

code_puppy/command_line/mcp/stop_command.py

@@ -5,6 +5,8 @@ MCP Stop Command - Stops a specific MCP server.
 import logging
 from typing import List, Optional
 
+from rich.text import Text
+
 from code_puppy.messaging import emit_error, emit_info
 
 from ...agents import get_current_agent
@@ -35,7 +37,7 @@ class StopCommand(MCPCommandBase):
 
         if not args:
             emit_info(
-                "[yellow]Usage: /mcp stop <server_name>[/yellow]",
+                Text.from_markup("[yellow]Usage: /mcp stop <server_name>[/yellow]"),
                 message_group=group_id,
             )
             return