claude-mpm 4.1.4__py3-none-any.whl → 4.1.6__py3-none-any.whl

claude_mpm/cli/commands/tickets.py

@@ -11,24 +11,37 @@ DESIGN DECISIONS:
 - Maintain backward compatibility with existing ai-trackdown integration
 - Support multiple output formats (json, yaml, table, text)
 - Implement full CRUD operations plus search and workflow management
+- Use service-oriented architecture to separate concerns
 """
 
-import json
-import subprocess
 import sys
 from typing import Optional
 
 from ...constants import TicketCommands
-from ...core.logger import get_logger
+from ...services.ticket_services import (
+    TicketCRUDService,
+    TicketFormatterService,
+    TicketSearchService,
+    TicketValidationService,
+    TicketWorkflowService,
+)
 from ..shared import BaseCommand, CommandResult
 
 
 class TicketsCommand(BaseCommand):
-    """Tickets command using shared utilities."""
+    """Tickets command using shared utilities and service-oriented architecture."""
 
     def __init__(self):
+        """Initialize the tickets command with services."""
         super().__init__("tickets")
 
+        # Initialize services using dependency injection
+        self.crud_service = TicketCRUDService()
+        self.formatter = TicketFormatterService()
+        self.validator = TicketValidationService()
+        self.search_service = TicketSearchService()
+        self.workflow_service = TicketWorkflowService()
+
     def validate_args(self, args) -> Optional[str]:
         """Validate command arguments."""
         if not hasattr(args, "tickets_command") or not args.tickets_command:
@@ -67,123 +80,381 @@ class TicketsCommand(BaseCommand):
             return CommandResult.error_result(f"Error executing tickets command: {e}")
 
     def _create_ticket(self, args) -> CommandResult:
-        """Create a new ticket."""
+        """Create a new ticket using the CRUD service."""
         try:
-            exit_code = create_ticket_legacy(args)
-            if exit_code == 0:
-                return CommandResult.success_result("Ticket created successfully")
-            return CommandResult.error_result(
-                "Failed to create ticket", exit_code=exit_code
+            # Prepare parameters
+            description = self.validator.sanitize_description(args.description)
+            tags = self.validator.sanitize_tags(args.tags)
+
+            # Validate creation parameters
+            params = {"title": args.title, "type": args.type, "priority": args.priority}
+            valid, error = self.validator.validate_create_params(params)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Create ticket via service
+            result = self.crud_service.create_ticket(
+                title=args.title,
+                ticket_type=args.type,
+                priority=args.priority,
+                description=description,
+                tags=tags,
+                parent_epic=getattr(args, "parent_epic", None),
+                parent_issue=getattr(args, "parent_issue", None),
             )
+
+            if result["success"]:
+                # Format and display output
+                output_lines = self.formatter.format_ticket_created(
+                    result["ticket_id"],
+                    verbose=args.verbose,
+                    type=args.type,
+                    priority=args.priority,
+                    tags=tags,
+                    parent_epic=getattr(args, "parent_epic", None),
+                    parent_issue=getattr(args, "parent_issue", None),
+                )
+                for line in output_lines:
+                    print(line)
+                return CommandResult.success_result(result["message"])
+            print(self.formatter.format_error(result["error"]))
+            return CommandResult.error_result(result["error"])
+
         except Exception as e:
             self.logger.error(f"Error creating ticket: {e}")
             return CommandResult.error_result(f"Error creating ticket: {e}")
 
     def _list_tickets(self, args) -> CommandResult:
-        """List tickets."""
+        """List tickets using the CRUD service."""
         try:
-            exit_code = list_tickets_legacy(args)
-            if exit_code == 0:
-                return CommandResult.success_result("Tickets listed successfully")
-            return CommandResult.error_result(
-                "Failed to list tickets", exit_code=exit_code
+            # Get pagination parameters
+            page = getattr(args, "page", 1)
+            page_size = getattr(args, "page_size", 20)
+            limit = getattr(args, "limit", page_size)
+
+            # Validate pagination
+            valid, error = self.validator.validate_pagination(page, page_size)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Get filters
+            type_filter = getattr(args, "type", None) or "all"
+            status_filter = getattr(args, "status", None) or "all"
+
+            # List tickets via service
+            result = self.crud_service.list_tickets(
+                limit=limit,
+                page=page,
+                page_size=page_size,
+                type_filter=type_filter,
+                status_filter=status_filter,
             )
+
+            if result["success"]:
+                # Format and display output
+                output_lines = self.formatter.format_ticket_list(
+                    result["tickets"],
+                    page=page,
+                    page_size=page_size,
+                    verbose=getattr(args, "verbose", False),
+                )
+                for line in output_lines:
+                    print(line)
+                return CommandResult.success_result("Tickets listed successfully")
+            print(self.formatter.format_error(result["error"]))
+            return CommandResult.error_result(result["error"])
+
         except Exception as e:
             self.logger.error(f"Error listing tickets: {e}")
             return CommandResult.error_result(f"Error listing tickets: {e}")
 
     def _view_ticket(self, args) -> CommandResult:
-        """View a specific ticket."""
+        """View a specific ticket using the CRUD service."""
         try:
-            exit_code = view_ticket_legacy(args)
-            if exit_code == 0:
+            # Get ticket ID
+            ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
+
+            # Validate ticket ID
+            valid, error = self.validator.validate_ticket_id(ticket_id)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Get ticket via service
+            ticket = self.crud_service.get_ticket(ticket_id)
+
+            if ticket:
+                # Format and display output
+                output_lines = self.formatter.format_ticket_detail(
+                    ticket, verbose=getattr(args, "verbose", False)
+                )
+                for line in output_lines:
+                    print(line)
                 return CommandResult.success_result("Ticket viewed successfully")
-            return CommandResult.error_result(
-                "Failed to view ticket", exit_code=exit_code
-            )
+            error_msg = f"Ticket {ticket_id} not found"
+            print(self.formatter.format_error(error_msg))
+            return CommandResult.error_result(error_msg)
+
         except Exception as e:
             self.logger.error(f"Error viewing ticket: {e}")
             return CommandResult.error_result(f"Error viewing ticket: {e}")
 
     def _update_ticket(self, args) -> CommandResult:
-        """Update a ticket."""
+        """Update a ticket using the CRUD service."""
         try:
-            exit_code = update_ticket_legacy(args)
-            if exit_code == 0:
-                return CommandResult.success_result("Ticket updated successfully")
-            return CommandResult.error_result(
-                "Failed to update ticket", exit_code=exit_code
+            # Get ticket ID
+            ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
+
+            # Validate ticket ID
+            valid, error = self.validator.validate_ticket_id(ticket_id)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Prepare update parameters
+            description = None
+            if args.description:
+                description = self.validator.sanitize_description(args.description)
+
+            tags = None
+            if args.tags:
+                tags = self.validator.sanitize_tags(args.tags)
+
+            assignees = None
+            if args.assign:
+                assignees = [args.assign]
+
+            # Validate update parameters
+            update_params = {}
+            if args.status:
+                update_params["status"] = args.status
+            if args.priority:
+                update_params["priority"] = args.priority
+
+            valid, error = self.validator.validate_update_params(update_params)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Update ticket via service
+            result = self.crud_service.update_ticket(
+                ticket_id=ticket_id,
+                status=args.status,
+                priority=args.priority,
+                description=description,
+                tags=tags,
+                assignees=assignees,
             )
+
+            if result["success"]:
+                print(self.formatter.format_operation_result("update", ticket_id, True))
+                return CommandResult.success_result(result["message"])
+            print(self.formatter.format_operation_result("update", ticket_id, False))
+            return CommandResult.error_result(result["error"])
+
         except Exception as e:
             self.logger.error(f"Error updating ticket: {e}")
             return CommandResult.error_result(f"Error updating ticket: {e}")
 
     def _close_ticket(self, args) -> CommandResult:
-        """Close a ticket."""
+        """Close a ticket using the CRUD service."""
         try:
-            exit_code = close_ticket_legacy(args)
-            if exit_code == 0:
-                return CommandResult.success_result("Ticket closed successfully")
-            return CommandResult.error_result(
-                "Failed to close ticket", exit_code=exit_code
-            )
+            # Get ticket ID
+            ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
+
+            # Validate ticket ID
+            valid, error = self.validator.validate_ticket_id(ticket_id)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Get resolution
+            resolution = getattr(args, "resolution", getattr(args, "comment", None))
+
+            # Close ticket via service
+            result = self.crud_service.close_ticket(ticket_id, resolution)
+
+            if result["success"]:
+                print(self.formatter.format_operation_result("close", ticket_id, True))
+                return CommandResult.success_result(result["message"])
+            print(self.formatter.format_operation_result("close", ticket_id, False))
+            return CommandResult.error_result(result["error"])
+
         except Exception as e:
             self.logger.error(f"Error closing ticket: {e}")
             return CommandResult.error_result(f"Error closing ticket: {e}")
 
     def _delete_ticket(self, args) -> CommandResult:
-        """Delete a ticket."""
+        """Delete a ticket using the CRUD service."""
         try:
-            exit_code = delete_ticket_legacy(args)
-            if exit_code == 0:
-                return CommandResult.success_result("Ticket deleted successfully")
-            return CommandResult.error_result(
-                "Failed to delete ticket", exit_code=exit_code
-            )
+            # Get ticket ID
+            ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
+
+            # Validate ticket ID
+            valid, error = self.validator.validate_ticket_id(ticket_id)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Confirm deletion unless forced
+            if not args.force:
+                sys.stdout.flush()
+
+                # Check if we're in a TTY environment
+                if not sys.stdin.isatty():
+                    print(
+                        f"Are you sure you want to delete ticket {ticket_id}? (y/N): ",
+                        end="",
+                        flush=True,
+                    )
+                    try:
+                        response = sys.stdin.readline().strip().lower()
+                        response = response.replace("\r", "").replace("\n", "").strip()
+                    except (EOFError, KeyboardInterrupt):
+                        response = "n"
+                else:
+                    try:
+                        response = (
+                            input(
+                                f"Are you sure you want to delete ticket {ticket_id}? (y/N): "
+                            )
+                            .strip()
+                            .lower()
+                        )
+                    except (EOFError, KeyboardInterrupt):
+                        response = "n"
+
+                if response != "y":
+                    print("Deletion cancelled")
+                    return CommandResult.success_result("Deletion cancelled")
+
+            # Delete ticket via service
+            result = self.crud_service.delete_ticket(ticket_id, args.force)
+
+            if result["success"]:
+                print(self.formatter.format_operation_result("delete", ticket_id, True))
+                return CommandResult.success_result(result["message"])
+            print(self.formatter.format_operation_result("delete", ticket_id, False))
+            return CommandResult.error_result(result["error"])
+
         except Exception as e:
             self.logger.error(f"Error deleting ticket: {e}")
             return CommandResult.error_result(f"Error deleting ticket: {e}")
 
     def _search_tickets(self, args) -> CommandResult:
-        """Search tickets."""
+        """Search tickets using the search service."""
         try:
-            exit_code = search_tickets_legacy(args)
-            if exit_code == 0:
-                return CommandResult.success_result("Tickets searched successfully")
-            return CommandResult.error_result(
-                "Failed to search tickets", exit_code=exit_code
+            # Validate search query
+            valid, error = self.validator.validate_search_query(args.query)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Search tickets via service
+            tickets = self.search_service.search_tickets(
+                query=args.query,
+                type_filter=args.type if args.type else "all",
+                status_filter=args.status if args.status else "all",
+                limit=args.limit,
             )
+
+            # Format and display results
+            output_lines = self.formatter.format_search_results(
+                tickets, args.query, show_snippets=True
+            )
+            for line in output_lines:
+                print(line)
+
+            return CommandResult.success_result("Tickets searched successfully")
+
         except Exception as e:
             self.logger.error(f"Error searching tickets: {e}")
             return CommandResult.error_result(f"Error searching tickets: {e}")
 
     def _add_comment(self, args) -> CommandResult:
-        """Add a comment to a ticket."""
+        """Add a comment to a ticket using the workflow service."""
         try:
-            exit_code = add_comment_legacy(args)
-            if exit_code == 0:
-                return CommandResult.success_result("Comment added successfully")
-            return CommandResult.error_result(
-                "Failed to add comment", exit_code=exit_code
-            )
+            # Get ticket ID
+            ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
+
+            # Validate ticket ID
+            valid, error = self.validator.validate_ticket_id(ticket_id)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Prepare comment
+            comment = self.validator.sanitize_description(args.comment)
+
+            # Validate comment
+            valid, error = self.validator.validate_comment(comment)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Add comment via service
+            result = self.workflow_service.add_comment(ticket_id, comment)
+
+            if result["success"]:
+                print(
+                    self.formatter.format_operation_result("comment", ticket_id, True)
+                )
+                return CommandResult.success_result(result["message"])
+            print(self.formatter.format_operation_result("comment", ticket_id, False))
+            return CommandResult.error_result(result["error"])
+
         except Exception as e:
             self.logger.error(f"Error adding comment: {e}")
             return CommandResult.error_result(f"Error adding comment: {e}")
 
     def _update_workflow(self, args) -> CommandResult:
-        """Update workflow state."""
+        """Update workflow state using the workflow service."""
         try:
-            exit_code = update_workflow_legacy(args)
-            if exit_code == 0:
-                return CommandResult.success_result("Workflow updated successfully")
-            return CommandResult.error_result(
-                "Failed to update workflow", exit_code=exit_code
+            # Get ticket ID
+            ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
+
+            # Validate ticket ID
+            valid, error = self.validator.validate_ticket_id(ticket_id)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Validate workflow state
+            valid, error = self.validator.validate_workflow_state(args.state)
+            if not valid:
+                print(self.formatter.format_error(error))
+                return CommandResult.error_result(error)
+
+            # Get optional comment
+            comment = getattr(args, "comment", None)
+
+            # Update workflow via service
+            result = self.workflow_service.transition_ticket(
+                ticket_id, args.state, comment
             )
+
+            if result["success"]:
+                print(
+                    self.formatter.format_operation_result(
+                        "workflow", ticket_id, True, result["message"]
+                    )
+                )
+                return CommandResult.success_result(result["message"])
+            print(self.formatter.format_operation_result("workflow", ticket_id, False))
+            return CommandResult.error_result(result["error"])
+
         except Exception as e:
             self.logger.error(f"Error updating workflow: {e}")
             return CommandResult.error_result(f"Error updating workflow: {e}")
 
 
+# ========================================
+# Backward compatibility functions
+# ========================================
+
+
 def manage_tickets(args):
     """
     Main entry point for tickets command.
@@ -211,755 +482,65 @@ def list_tickets(args):
     return manage_tickets(args)
 
 
-def manage_tickets_legacy(args):
-    """
-    Legacy ticket command dispatcher.
-
-    WHY: This contains the original manage_tickets logic, preserved during migration
-    to BaseCommand pattern. Will be gradually refactored into the TicketsCommand class.
-
-    DESIGN DECISION: We use a subcommand pattern similar to git, allowing for
-    intuitive command structure like 'claude-mpm tickets create "title"'.
+# ========================================
+# Legacy function stubs for compatibility
+# ========================================
 
-    Args:
-        args: Parsed command line arguments with 'tickets_command' attribute
 
-    Returns:
-        Exit code (0 for success, non-zero for errors)
-    """
-    logger = get_logger("cli.tickets")
-
-    # Handle case where no subcommand is provided - default to list
-    if not hasattr(args, "tickets_command") or not args.tickets_command:
-        # Default to list command for backward compatibility
-        args.tickets_command = TicketCommands.LIST.value
-        # Set default limit if not present
-        if not hasattr(args, "limit"):
-            args.limit = 10
-        if not hasattr(args, "verbose"):
-            args.verbose = False
-
-    # Map subcommands to handler functions
-    handlers = {
-        TicketCommands.CREATE.value: create_ticket_legacy,
-        TicketCommands.LIST.value: list_tickets_legacy,
-        TicketCommands.VIEW.value: view_ticket_legacy,
-        TicketCommands.UPDATE.value: update_ticket_legacy,
-        TicketCommands.CLOSE.value: close_ticket_legacy,
-        TicketCommands.DELETE.value: delete_ticket_legacy,
-        TicketCommands.SEARCH.value: search_tickets_legacy,
-        TicketCommands.COMMENT.value: add_comment_legacy,
-        TicketCommands.WORKFLOW.value: update_workflow_legacy,
-    }
-
-    # Execute the appropriate handler
-    handler = handlers.get(args.tickets_command)
-    if handler:
-        try:
-            return handler(args)
-        except KeyboardInterrupt:
-            logger.info("Operation cancelled by user")
-            return 1
-        except Exception as e:
-            logger.error(f"Error executing {args.tickets_command}: {e}")
-            if hasattr(args, "debug") and args.debug:
-                import traceback
-
-                traceback.print_exc()
-            return 1
-    else:
-        logger.error(f"Unknown ticket command: {args.tickets_command}")
-        return 1
+def manage_tickets_legacy(args):
+    """Legacy wrapper - redirects to new implementation."""
+    return manage_tickets(args)
 
 
 def create_ticket_legacy(args):
-    """
-    Create a new ticket.
-
-    WHY: Users need to create tickets to track work items, bugs, and features.
-    This command provides a streamlined interface for ticket creation.
-
-    DESIGN DECISION: We parse description from remaining args to allow natural
-    command line usage like: tickets create "title" -d This is a description
-
-    Args:
-        args: Arguments with title, type, priority, description, tags, etc.
-
-    Returns:
-        Exit code (0 for success, non-zero for errors)
-    """
-    get_logger("cli.tickets")
-
-    try:
-        from ...services.ticket_manager import TicketManager
-    except ImportError:
-        from claude_mpm.services.ticket_manager import TicketManager
-
-    ticket_manager = TicketManager()
-
-    # Parse description from remaining args or use default
-    description = " ".join(args.description) if args.description else ""
-
-    # Parse tags
-    tags = args.tags.split(",") if args.tags else []
-
-    # Create ticket with all provided parameters
-    ticket_id = ticket_manager.create_ticket(
-        title=args.title,
-        ticket_type=args.type,
-        description=description,
-        priority=args.priority,
-        tags=tags,
-        source="claude-mpm-cli",
-        parent_epic=getattr(args, "parent_epic", None),
-        parent_issue=getattr(args, "parent_issue", None),
-    )
-
-    if ticket_id:
-        print(f"✅ Created ticket: {ticket_id}")
-        if args.verbose:
-            print(f"   Type: {args.type}")
-            print(f"   Priority: {args.priority}")
-            if tags:
-                print(f"   Tags: {', '.join(tags)}")
-            if getattr(args, "parent_epic", None):
-                print(f"   Parent Epic: {args.parent_epic}")
-            if getattr(args, "parent_issue", None):
-                print(f"   Parent Issue: {args.parent_issue}")
-        return 0
-    print("❌ Failed to create ticket")
-    return 1
+    """Legacy wrapper - uses new service implementation."""
+    args.tickets_command = TicketCommands.CREATE.value
+    return manage_tickets(args)
 
 
 def list_tickets_legacy(args):
-    """
-    List recent tickets with optional filtering.
-
-    WHY: Users need to review tickets created during Claude sessions. This command
-    provides a quick way to see recent tickets with their status and metadata.
-
-    DESIGN DECISION: We show tickets in a compact format with emoji status indicators
-    for better visual scanning. Filters allow focusing on specific ticket types/statuses.
-
-    Args:
-        args: Arguments with limit, type filter, status filter, verbose flag
-
-    Returns:
-        Exit code (0 for success, non-zero for errors)
-    """
-    logger = get_logger("cli.tickets")
-
-    try:
-        # Get pagination parameters
-        page = getattr(args, "page", 1)
-        page_size = getattr(args, "page_size", 20)
-        limit = getattr(args, "limit", page_size)  # Use page_size as default limit
-
-        # Validate pagination parameters
-        if page < 1:
-            print("❌ Page number must be 1 or greater")
-            return 1
-        if page_size < 1:
-            print("❌ Page size must be 1 or greater")
-            return 1
-
-        # Try to use ai-trackdown CLI directly for better pagination support
-        tickets = []
-        try:
-            # Build aitrackdown command with pagination
-            cmd = ["aitrackdown", "status", "tasks"]
-
-            # Calculate offset for pagination
-            offset = (page - 1) * page_size
-            total_needed = offset + page_size
-
-            # Request more tickets than needed to handle filtering
-            cmd.extend(["--limit", str(total_needed * 2)])
-
-            # Add filters
-            type_filter = getattr(args, "type", None) or "all"
-            if type_filter != "all" and type_filter is not None:
-                cmd.extend(["--type", type_filter])
-
-            status_filter = getattr(args, "status", None) or "all"
-            if status_filter != "all" and status_filter is not None:
-                cmd.extend(["--status", status_filter])
-
-            # Execute command
-            result = subprocess.run(cmd, capture_output=True, text=True, check=True)
-
-            # Parse JSON output
-            if result.stdout.strip():
-                try:
-                    all_tickets = json.loads(result.stdout)
-                    if isinstance(all_tickets, list):
-                        # Apply pagination
-                        start_idx = offset
-                        end_idx = start_idx + page_size
-                        tickets = all_tickets[start_idx:end_idx]
-                    else:
-                        tickets = []
-                except json.JSONDecodeError:
-                    logger.warning(
-                        "Failed to parse aitrackdown JSON output, falling back to stub"
-                    )
-                    tickets = []
-
-        except (subprocess.CalledProcessError, FileNotFoundError) as e:
-            logger.warning(f"aitrackdown command failed: {e}, falling back to stub")
-            # Fallback to stub implementation
-            try:
-                from ...services.ticket_manager import TicketManager
-            except ImportError:
-                from claude_mpm.services.ticket_manager import TicketManager
-
-            ticket_manager = TicketManager()
-            all_tickets = ticket_manager.list_recent_tickets(limit=limit * 2)
-
-            # Apply filters and pagination manually for stub
-            filtered_tickets = []
-            for ticket in all_tickets:
-                # Type filter
-                if type_filter != "all":
-                    ticket_type = ticket.get("metadata", {}).get(
-                        "ticket_type", "unknown"
-                    )
-                    if ticket_type != type_filter:
-                        continue
-
-                # Status filter
-                if status_filter != "all":
-                    if ticket.get("status") != status_filter:
-                        continue
-
-                filtered_tickets.append(ticket)
-
-            # Apply pagination
-            start_idx = (page - 1) * page_size
-            end_idx = start_idx + page_size
-            tickets = filtered_tickets[start_idx:end_idx]
-
-        if not tickets:
-            print("No tickets found matching criteria")
-            if page > 1:
-                print(f"Try a lower page number (current: {page})")
-            return 0
-
-        # Display pagination info
-        total_shown = len(tickets)
-        print(f"Tickets (page {page}, showing {total_shown} tickets):")
-        print("-" * 80)
-
-        for ticket in tickets:
-            # Use emoji to indicate status visually
-            status_emoji = {
-                "open": "🔵",
-                "in_progress": "🟡",
-                "done": "🟢",
-                "closed": "⚫",
-                "blocked": "🔴",
-            }.get(ticket.get("status", "unknown"), "⚪")
-
-            print(f"{status_emoji} [{ticket['id']}] {ticket['title']}")
-
-            if getattr(args, "verbose", False):
-                ticket_type = ticket.get("metadata", {}).get("ticket_type", "task")
-                print(
-                    f"   Type: {ticket_type} | Status: {ticket['status']} | Priority: {ticket['priority']}"
-                )
-                if ticket.get("tags"):
-                    print(f"   Tags: {', '.join(ticket['tags'])}")
-                print(f"   Created: {ticket['created_at']}")
-                print()
-
-        # Show pagination navigation hints
-        if total_shown == page_size:
-            print("-" * 80)
-            print(f"📄 Page {page} | Showing {total_shown} tickets")
-            print(
-                f"💡 Next page: claude-mpm tickets list --page {page + 1} --page-size {page_size}"
-            )
-            if page > 1:
-                print(
-                    f"💡 Previous page: claude-mpm tickets list --page {page - 1} --page-size {page_size}"
-                )
-
-        return 0
-
-    except ImportError:
-        logger.error("ai-trackdown-pytools not installed")
-        print("Error: ai-trackdown-pytools not installed")
-        print("Install with: pip install ai-trackdown-pytools")
-        return 1
-    except Exception as e:
-        logger.error(f"Error listing tickets: {e}")
-        print(f"Error: {e}")
-        return 1
+    """Legacy wrapper - uses new service implementation."""
+    args.tickets_command = TicketCommands.LIST.value
+    return manage_tickets(args)
 
 
 def view_ticket_legacy(args):
-    """
-    View a specific ticket in detail.
-
-    WHY: Users need to see full ticket details including description, metadata,
-    and all associated information for understanding context and status.
-
-    Args:
-        args: Arguments with ticket id and verbose flag
-
-    Returns:
-        Exit code (0 for success, non-zero for errors)
-    """
-    get_logger("cli.tickets")
-
-    try:
-        from ...services.ticket_manager import TicketManager
-    except ImportError:
-        from claude_mpm.services.ticket_manager import TicketManager
-
-    ticket_manager = TicketManager()
-    # Handle both 'id' and 'ticket_id' attributes for compatibility
-    ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
-    if not ticket_id:
-        print("❌ No ticket ID provided")
-        return 1
-    ticket = ticket_manager.get_ticket(ticket_id)
-
-    if not ticket:
-        print(f"❌ Ticket {ticket_id} not found")
-        return 1
-
-    print(f"Ticket: {ticket['id']}")
-    print("=" * 80)
-    print(f"Title: {ticket['title']}")
-    print(f"Type: {ticket.get('metadata', {}).get('ticket_type', 'unknown')}")
-    print(f"Status: {ticket['status']}")
-    print(f"Priority: {ticket['priority']}")
-
-    if ticket.get("tags"):
-        print(f"Tags: {', '.join(ticket['tags'])}")
-
-    if ticket.get("assignees"):
-        print(f"Assignees: {', '.join(ticket['assignees'])}")
-
-    # Show parent references if they exist
-    metadata = ticket.get("metadata", {})
-    if metadata.get("parent_epic"):
-        print(f"Parent Epic: {metadata['parent_epic']}")
-    if metadata.get("parent_issue"):
-        print(f"Parent Issue: {metadata['parent_issue']}")
-
-    print("\nDescription:")
-    print("-" * 40)
-    print(ticket.get("description", "No description"))
-
-    print(f"\nCreated: {ticket['created_at']}")
-    print(f"Updated: {ticket['updated_at']}")
-
-    if args.verbose and ticket.get("metadata"):
-        print("\nMetadata:")
-        print("-" * 40)
-        for key, value in ticket["metadata"].items():
-            if key not in [
-                "parent_epic",
-                "parent_issue",
-                "ticket_type",
-            ]:  # Already shown above
-                print(f"  {key}: {value}")
-
-    return 0
+    """Legacy wrapper - uses new service implementation."""
+    args.tickets_command = TicketCommands.VIEW.value
+    return manage_tickets(args)
 
 
 def update_ticket_legacy(args):
-    """
-    Update a ticket's properties.
-
-    WHY: Tickets need to be updated as work progresses, priorities change,
-    or additional information becomes available.
-
-    DESIGN DECISION: For complex updates, we delegate to aitrackdown CLI
-    for operations not directly supported by our TicketManager interface.
-
-    Args:
-        args: Arguments with ticket id and update fields
-
-    Returns:
-        Exit code (0 for success, non-zero for errors)
-    """
-    logger = get_logger("cli.tickets")
-
-    try:
-        from ...services.ticket_manager import TicketManager
-    except ImportError:
-        from claude_mpm.services.ticket_manager import TicketManager
-
-    ticket_manager = TicketManager()
-
-    # Handle both 'id' and 'ticket_id' attributes for compatibility
-    ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
-    if not ticket_id:
-        print("❌ No ticket ID provided")
-        return 1
-
-    # Build update dictionary
-    updates = {}
-
-    if args.status:
-        updates["status"] = args.status
-
-    if args.priority:
-        updates["priority"] = args.priority
-
-    if args.description:
-        updates["description"] = " ".join(args.description)
-
-    if args.tags:
-        updates["tags"] = args.tags.split(",")
-
-    if args.assign:
-        updates["assignees"] = [args.assign]
-
-    if not updates:
-        print("❌ No updates specified")
-        return 1
-
-    # Try to update using TicketManager
-    success = ticket_manager.update_task(ticket_id, **updates)
-
-    if success:
-        print(f"✅ Updated ticket: {ticket_id}")
-        return 0
-    # Fallback to aitrackdown CLI for status transitions
-    if args.status:
-        logger.info("Attempting update via aitrackdown CLI")
-        cmd = ["aitrackdown", "transition", ticket_id, args.status]
-
-        # Add comment with other updates
-        comment_parts = []
-        if args.priority:
-            comment_parts.append(f"Priority: {args.priority}")
-        if args.assign:
-            comment_parts.append(f"Assigned to: {args.assign}")
-        if args.tags:
-            comment_parts.append(f"Tags: {args.tags}")
-
-        if comment_parts:
-            comment = " | ".join(comment_parts)
-            cmd.extend(["--comment", comment])
-
-        try:
-            subprocess.run(cmd, check=True, capture_output=True, text=True)
-            print(f"✅ Updated ticket: {ticket_id}")
-            return 0
-        except subprocess.CalledProcessError as e:
-            logger.error(f"Failed to update via CLI: {e}")
-            print(f"❌ Failed to update ticket: {ticket_id}")
-            return 1
-    else:
-        print(f"❌ Failed to update ticket: {ticket_id}")
-        return 1
+    """Legacy wrapper - uses new service implementation."""
+    args.tickets_command = TicketCommands.UPDATE.value
+    return manage_tickets(args)
 
 
 def close_ticket_legacy(args):
-    """
-    Close a ticket.
-
-    WHY: Tickets need to be closed when work is completed or no longer relevant.
-
-    Args:
-        args: Arguments with ticket id and optional resolution
-
-    Returns:
-        Exit code (0 for success, non-zero for errors)
-    """
-    logger = get_logger("cli.tickets")
-
-    try:
-        from ...services.ticket_manager import TicketManager
-    except ImportError:
-        from claude_mpm.services.ticket_manager import TicketManager
-
-    ticket_manager = TicketManager()
-
-    # Handle both 'id' and 'ticket_id' attributes for compatibility
-    ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
-    if not ticket_id:
-        print("❌ No ticket ID provided")
-        return 1
-
-    # Try to close using TicketManager
-    resolution = getattr(args, "resolution", getattr(args, "comment", None))
-    success = ticket_manager.close_task(ticket_id, resolution=resolution)
-
-    if success:
-        print(f"✅ Closed ticket: {ticket_id}")
-        return 0
-    # Fallback to aitrackdown CLI
-    logger.info("Attempting close via aitrackdown CLI")
-    cmd = ["aitrackdown", "close", ticket_id]
-
-    if resolution:
-        cmd.extend(["--comment", resolution])
-
-    try:
-        subprocess.run(cmd, check=True, capture_output=True, text=True)
-        print(f"✅ Closed ticket: {ticket_id}")
-        return 0
-    except subprocess.CalledProcessError:
-        print(f"❌ Failed to close ticket: {ticket_id}")
-        return 1
+    """Legacy wrapper - uses new service implementation."""
+    args.tickets_command = TicketCommands.CLOSE.value
+    return manage_tickets(args)
 
 
 def delete_ticket_legacy(args):
-    """
-    Delete a ticket.
-
-    WHY: Sometimes tickets are created in error or are no longer needed
-    and should be removed from the system.
-
-    DESIGN DECISION: We delegate to aitrackdown CLI as deletion is a
-    destructive operation that should use the official tool.
-
-    Args:
-        args: Arguments with ticket id and force flag
-
-    Returns:
-        Exit code (0 for success, non-zero for errors)
-    """
-    get_logger("cli.tickets")
-
-    # Handle both 'id' and 'ticket_id' attributes for compatibility
-    ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
-    if not ticket_id:
-        print("❌ No ticket ID provided")
-        return 1
-
-    # Confirm deletion unless forced
-    if not args.force:
-        sys.stdout.flush()  # Ensure prompt is displayed before input
-
-        # Check if we're in a TTY environment for proper input handling
-        if not sys.stdin.isatty():
-            # In non-TTY environment (like pipes), use readline
-            print(
-                f"Are you sure you want to delete ticket {ticket_id}? (y/N): ",
-                end="",
-                flush=True,
-            )
-            try:
-                response = sys.stdin.readline().strip().lower()
-                # Handle various line endings and control characters
-                response = response.replace("\r", "").replace("\n", "").strip()
-            except (EOFError, KeyboardInterrupt):
-                response = "n"
-        else:
-            # In TTY environment, use normal input()
-            try:
-                response = (
-                    input(
-                        f"Are you sure you want to delete ticket {ticket_id}? (y/N): "
-                    )
-                    .strip()
-                    .lower()
-                )
-            except (EOFError, KeyboardInterrupt):
-                response = "n"
-
-        if response != "y":
-            print("Deletion cancelled")
-            return 0
-
-    # Use aitrackdown CLI for deletion
-    cmd = ["aitrackdown", "delete", ticket_id]
-    if args.force:
-        cmd.append("--force")
-
-    try:
-        subprocess.run(cmd, check=True, capture_output=True, text=True)
-        print(f"✅ Deleted ticket: {ticket_id}")
-        return 0
-    except subprocess.CalledProcessError:
-        print(f"❌ Failed to delete ticket: {ticket_id}")
-        return 1
+    """Legacy wrapper - uses new service implementation."""
+    args.tickets_command = TicketCommands.DELETE.value
+    return manage_tickets(args)
 
 
 def search_tickets_legacy(args):
-    """
-    Search tickets by query string.
-
-    WHY: Users need to find specific tickets based on content, tags, or other criteria.
-
-    DESIGN DECISION: We perform simple text matching on ticket data. For more advanced
-    search, users should use the aitrackdown CLI directly.
-
-    Args:
-        args: Arguments with search query and filters
-
-    Returns:
-        Exit code (0 for success, non-zero for errors)
-    """
-    get_logger("cli.tickets")
-
-    try:
-        from ...services.ticket_manager import TicketManager
-    except ImportError:
-        from claude_mpm.services.ticket_manager import TicketManager
-
-    ticket_manager = TicketManager()
-
-    # Get all available tickets for searching
-    all_tickets = ticket_manager.list_recent_tickets(limit=100)
-
-    # Search tickets
-    query = args.query.lower()
-    matched_tickets = []
-
-    for ticket in all_tickets:
-        # Check if query matches title, description, or tags
-        if (
-            query in ticket.get("title", "").lower()
-            or query in ticket.get("description", "").lower()
-            or any(query in tag.lower() for tag in ticket.get("tags", []))
-        ):
-            # Apply type filter
-            if args.type != "all":
-                ticket_type = ticket.get("metadata", {}).get("ticket_type", "unknown")
-                if ticket_type != args.type:
-                    continue
-
-            # Apply status filter
-            if args.status != "all" and ticket.get("status") != args.status:
-                continue
-
-            matched_tickets.append(ticket)
-            if len(matched_tickets) >= args.limit:
-                break
-
-    if not matched_tickets:
-        print(f"No tickets found matching '{args.query}'")
-        return 0
-
-    print(f"Search results for '{args.query}' (showing {len(matched_tickets)}):")
-    print("-" * 80)
-
-    for ticket in matched_tickets:
-        status_emoji = {
-            "open": "🔵",
-            "in_progress": "🟡",
-            "done": "🟢",
-            "closed": "⚫",
-            "blocked": "🔴",
-        }.get(ticket.get("status", "unknown"), "⚪")
-
-        print(f"{status_emoji} [{ticket['id']}] {ticket['title']}")
-
-        # Show snippet of description if it contains the query
-        desc = ticket.get("description", "")
-        if query in desc.lower():
-            # Find and show context around the match
-            idx = desc.lower().index(query)
-            start = max(0, idx - 30)
-            end = min(len(desc), idx + len(query) + 30)
-            snippet = desc[start:end]
-            if start > 0:
-                snippet = "..." + snippet
-            if end < len(desc):
-                snippet = snippet + "..."
-            print(f"   {snippet}")
-
-    return 0
+    """Legacy wrapper - uses new service implementation."""
+    args.tickets_command = TicketCommands.SEARCH.value
+    return manage_tickets(args)
 
 
 def add_comment_legacy(args):
-    """
-    Add a comment to a ticket.
-
-    WHY: Comments allow tracking progress, decisions, and additional context
-    on tickets over time.
-
-    DESIGN DECISION: We delegate to aitrackdown CLI as it has proper comment
-    tracking infrastructure.
-
-    Args:
-        args: Arguments with ticket id and comment text
-
-    Returns:
-        Exit code (0 for success, non-zero for errors)
-    """
-    get_logger("cli.tickets")
-
-    # Handle both 'id' and 'ticket_id' attributes for compatibility
-    ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
-    if not ticket_id:
-        print("❌ No ticket ID provided")
-        return 1
-
-    # Join comment parts into single string
-    comment = " ".join(args.comment) if isinstance(args.comment, list) else args.comment
-
-    # Use aitrackdown CLI for comments
-    cmd = ["aitrackdown", "comment", ticket_id, comment]
-
-    try:
-        subprocess.run(cmd, check=True, capture_output=True, text=True)
-        print(f"✅ Added comment to ticket: {ticket_id}")
-        return 0
-    except subprocess.CalledProcessError:
-        print(f"❌ Failed to add comment to ticket: {ticket_id}")
-        return 1
+    """Legacy wrapper - uses new service implementation."""
+    args.tickets_command = TicketCommands.COMMENT.value
+    return manage_tickets(args)
 
 
 def update_workflow_legacy(args):
-    """
-    Update ticket workflow state.
-
-    WHY: Workflow states track the progress of tickets through defined stages
-    like todo, in_progress, ready, tested, done.
-
-    DESIGN DECISION: We use aitrackdown's transition command for workflow updates
-    as it maintains proper state machine transitions.
-
-    Args:
-        args: Arguments with ticket id, new state, and optional comment
-
-    Returns:
-        Exit code (0 for success, non-zero for errors)
-    """
-    get_logger("cli.tickets")
-
-    # Handle both 'id' and 'ticket_id' attributes for compatibility
-    ticket_id = getattr(args, "ticket_id", getattr(args, "id", None))
-    if not ticket_id:
-        print("❌ No ticket ID provided")
-        return 1
-
-    # Map workflow states to status if needed
-
-    # Use aitrackdown transition command
-    cmd = ["aitrackdown", "transition", ticket_id, args.state]
-
-    if getattr(args, "comment", None):
-        cmd.extend(["--comment", args.comment])
-
-    try:
-        subprocess.run(cmd, check=True, capture_output=True, text=True)
-        print(f"✅ Updated workflow state for {ticket_id} to: {args.state}")
-        return 0
-    except subprocess.CalledProcessError:
-        print(f"❌ Failed to update workflow state for ticket: {ticket_id}")
-        return 1
-
-
-# Maintain backward compatibility with the old list_tickets function signature
-def list_tickets_legacy(args):
-    """
-    Legacy list_tickets function for backward compatibility.
-
-    WHY: The old CLI interface expected a simple list_tickets function.
-    This wrapper maintains that interface while using the new implementation.
-
-    Args:
-        args: Parsed command line arguments with 'limit' attribute
-    """
-    # Call the new list_tickets function
-    return list_tickets(args)
+    """Legacy wrapper - uses new service implementation."""
+    args.tickets_command = TicketCommands.WORKFLOW.value
+    return manage_tickets(args)