claude-mpm 3.9.11__py3-none-any.whl → 4.0.4__py3-none-any.whl

claude_mpm/cli/commands/tickets.py

@@ -10,43 +10,43 @@ to provide a complete ticket management system within the claude-mpm CLI. The co
 mirror the scripts/ticket.py interface for consistency.
 """
 
-import sys
+import json
 import subprocess
-from typing import Optional, List, Dict, Any
-from pathlib import Path
+import sys
+from typing import Any, Dict, List, Optional
 
-from ...core.logger import get_logger
 from ...constants import TicketCommands
+from ...core.logger import get_logger
 
 
 def manage_tickets(args):
     """
     Main ticket command dispatcher.
-    
+
     WHY: This function routes ticket subcommands to their appropriate handlers,
     providing a single entry point for all ticket-related operations.
-    
+
     DESIGN DECISION: We use a subcommand pattern similar to git, allowing for
     intuitive command structure like 'claude-mpm tickets create "title"'.
-    
+
     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:
+    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'):
+        if not hasattr(args, "limit"):
             args.limit = 10
-        if not hasattr(args, 'verbose'):
+        if not hasattr(args, "verbose"):
             args.verbose = False
-    
+
     # Map subcommands to handler functions
     handlers = {
         TicketCommands.CREATE.value: create_ticket,
@@ -59,7 +59,7 @@ def manage_tickets(args):
         TicketCommands.COMMENT.value: add_comment,
         TicketCommands.WORKFLOW.value: update_workflow,
     }
-    
+
     # Execute the appropriate handler
     handler = handlers.get(args.tickets_command)
     if handler:
@@ -70,8 +70,9 @@ def manage_tickets(args):
             return 1
         except Exception as e:
             logger.error(f"Error executing {args.tickets_command}: {e}")
-            if hasattr(args, 'debug') and args.debug:
+            if hasattr(args, "debug") and args.debug:
                 import traceback
+
                 traceback.print_exc()
             return 1
     else:
@@ -82,34 +83,34 @@ def manage_tickets(args):
 def create_ticket(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)
     """
     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()
-    
+
     # 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,
@@ -118,10 +119,10 @@ def create_ticket(args):
         priority=args.priority,
         tags=tags,
         source="claude-mpm-cli",
-        parent_epic=getattr(args, 'parent_epic', None),
-        parent_issue=getattr(args, 'parent_issue', None)
+        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:
@@ -129,9 +130,9 @@ def create_ticket(args):
             print(f"   Priority: {args.priority}")
             if tags:
                 print(f"   Tags: {', '.join(tags)}")
-            if getattr(args, 'parent_epic', None):
+            if getattr(args, "parent_epic", None):
                 print(f"   Parent Epic: {args.parent_epic}")
-            if getattr(args, 'parent_issue', None):
+            if getattr(args, "parent_issue", None):
                 print(f"   Parent Issue: {args.parent_issue}")
         return 0
     else:
@@ -142,82 +143,158 @@ def create_ticket(args):
 def list_tickets(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:
-            from ...services.ticket_manager import TicketManager
-        except ImportError:
-            from claude_mpm.services.ticket_manager import TicketManager
-        
-        ticket_manager = TicketManager()
-        
-        # Get tickets with limit
-        limit = getattr(args, 'limit', 10)
-        tickets = ticket_manager.list_recent_tickets(limit=limit * 2)  # Get extra for filtering
-        
-        # Apply filters if specified
-        filtered_tickets = []
-        for ticket in tickets:
-            # Type filter
-            type_filter = getattr(args, 'type', 'all')
-            if type_filter != 'all':
-                ticket_type = ticket.get('metadata', {}).get('ticket_type', 'unknown')
-                if ticket_type != type_filter:
-                    continue
-            
-            # Status filter
-            status_filter = getattr(args, 'status', 'all')
-            if status_filter != 'all':
-                if ticket.get('status') != status_filter:
-                    continue
-            
-            filtered_tickets.append(ticket)
-            if len(filtered_tickets) >= limit:
-                break
-        
-        if not filtered_tickets:
+            # 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
-        
-        print(f"Recent tickets (showing {len(filtered_tickets)}):")
+
+        # Display pagination info
+        total_shown = len(tickets)
+        print(f"Tickets (page {page}, showing {total_shown} tickets):")
         print("-" * 80)
-        
-        for ticket in filtered_tickets:
+
+        for ticket in tickets:
             # Use emoji to indicate status visually
             status_emoji = {
                 "open": "🔵",
                 "in_progress": "🟡",
                 "done": "🟢",
                 "closed": "⚫",
-                "blocked": "🔴"
-            }.get(ticket.get('status', 'unknown'), "⚪")
-            
+                "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'):
+
+            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")
@@ -232,126 +309,141 @@ def list_tickets(args):
 def view_ticket(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)
     """
     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()
-    ticket = ticket_manager.get_ticket(args.id)
-    
+    # 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 {args.id} not found")
+        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'):
+
+    if ticket.get("tags"):
         print(f"Tags: {', '.join(ticket['tags'])}")
-    
-    if ticket.get('assignees'):
+
+    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'):
+    metadata = ticket.get("metadata", {})
+    if metadata.get("parent_epic"):
         print(f"Parent Epic: {metadata['parent_epic']}")
-    if metadata.get('parent_issue'):
+    if metadata.get("parent_issue"):
         print(f"Parent Issue: {metadata['parent_issue']}")
-    
+
     print(f"\nDescription:")
     print("-" * 40)
-    print(ticket.get('description', 'No description'))
-    
+    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'):
+
+    if args.verbose and ticket.get("metadata"):
         print(f"\nMetadata:")
         print("-" * 40)
-        for key, value in ticket['metadata'].items():
-            if key not in ['parent_epic', 'parent_issue', 'ticket_type']:  # Already shown above
+        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
 
 
 def update_ticket(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
-    
+        updates["status"] = args.status
+
     if args.priority:
-        updates['priority'] = args.priority
-    
+        updates["priority"] = args.priority
+
     if args.description:
-        updates['description'] = " ".join(args.description)
-    
+        updates["description"] = " ".join(args.description)
+
     if args.tags:
-        updates['tags'] = args.tags.split(",")
-    
+        updates["tags"] = args.tags.split(",")
+
     if args.assign:
-        updates['assignees'] = [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(args.id, **updates)
-    
+    success = ticket_manager.update_task(ticket_id, **updates)
+
     if success:
-        print(f"✅ Updated ticket: {args.id}")
+        print(f"✅ Updated ticket: {ticket_id}")
         return 0
     else:
         # Fallback to aitrackdown CLI for status transitions
         if args.status:
             logger.info("Attempting update via aitrackdown CLI")
-            cmd = ["aitrackdown", "transition", args.id, args.status]
-            
+            cmd = ["aitrackdown", "transition", ticket_id, args.status]
+
             # Add comment with other updates
             comment_parts = []
             if args.priority:
@@ -360,180 +452,219 @@ def update_ticket(args):
                 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: {args.id}")
+                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: {args.id}")
+                print(f"❌ Failed to update ticket: {ticket_id}")
                 return 1
         else:
-            print(f"❌ Failed to update ticket: {args.id}")
+            print(f"❌ Failed to update ticket: {ticket_id}")
             return 1
 
 
 def close_ticket(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', None)
-    success = ticket_manager.close_task(args.id, resolution=resolution)
-    
+    resolution = getattr(args, "resolution", getattr(args, "comment", None))
+    success = ticket_manager.close_task(ticket_id, resolution=resolution)
+
     if success:
-        print(f"✅ Closed ticket: {args.id}")
+        print(f"✅ Closed ticket: {ticket_id}")
         return 0
     else:
         # Fallback to aitrackdown CLI
         logger.info("Attempting close via aitrackdown CLI")
-        cmd = ["aitrackdown", "close", args.id]
-        
+        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: {args.id}")
+            print(f"✅ Closed ticket: {ticket_id}")
             return 0
         except subprocess.CalledProcessError:
-            print(f"❌ Failed to close ticket: {args.id}")
+            print(f"❌ Failed to close ticket: {ticket_id}")
             return 1
 
 
 def delete_ticket(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)
     """
     logger = 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:
-        response = input(f"Are you sure you want to delete ticket {args.id}? (y/N): ")
-        if response.lower() != 'y':
+        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", args.id]
+    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: {args.id}")
+        print(f"✅ Deleted ticket: {ticket_id}")
         return 0
     except subprocess.CalledProcessError:
-        print(f"❌ Failed to delete ticket: {args.id}")
+        print(f"❌ Failed to delete ticket: {ticket_id}")
         return 1
 
 
 def search_tickets(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)
     """
     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()
-    
+
     # 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', []))):
-            
+        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 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':
-                if ticket.get('status') != args.status:
+            if args.status != "all":
+                if 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'), "⚪")
-        
+            "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', '')
+        desc = ticket.get("description", "")
         if query in desc.lower():
             # Find and show context around the match
             idx = desc.lower().index(query)
@@ -545,83 +676,95 @@ def search_tickets(args):
             if end < len(desc):
                 snippet = snippet + "..."
             print(f"   {snippet}")
-    
+
     return 0
 
 
 def add_comment(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)
     """
     logger = 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)
-    
+    comment = " ".join(args.comment) if isinstance(args.comment, list) else args.comment
+
     # Use aitrackdown CLI for comments
-    cmd = ["aitrackdown", "comment", args.id, comment]
-    
+    cmd = ["aitrackdown", "comment", ticket_id, comment]
+
     try:
         subprocess.run(cmd, check=True, capture_output=True, text=True)
-        print(f"✅ Added comment to ticket: {args.id}")
+        print(f"✅ Added comment to ticket: {ticket_id}")
         return 0
     except subprocess.CalledProcessError:
-        print(f"❌ Failed to add comment to ticket: {args.id}")
+        print(f"❌ Failed to add comment to ticket: {ticket_id}")
         return 1
 
 
 def update_workflow(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)
     """
     logger = 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
     state_mapping = {
-        'todo': 'open',
-        'in_progress': 'in_progress',
-        'ready': 'ready',
-        'tested': 'tested',
-        'done': 'done',
-        'blocked': 'blocked'
+        "todo": "open",
+        "in_progress": "in_progress",
+        "ready": "ready",
+        "tested": "tested",
+        "done": "done",
+        "blocked": "blocked",
     }
-    
+
     # Use aitrackdown transition command
-    cmd = ["aitrackdown", "transition", args.id, args.state]
-    
-    if getattr(args, 'comment', None):
+    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 {args.id} to: {args.state}")
+        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: {args.id}")
+        print(f"❌ Failed to update workflow state for ticket: {ticket_id}")
         return 1
 
 
@@ -629,12 +772,12 @@ def update_workflow(args):
 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)
+    return list_tickets(args)