mcli-framework 7.12.2__py3-none-any.whl → 7.12.4__py3-none-any.whl

mcli/ml/trading/trading_service.py

@@ -1,27 +1,24 @@
-"""Trading service for managing portfolios and executing trades"""
+"""Trading service for managing portfolios and executing trades."""
 
 import logging
 from datetime import datetime, timedelta
 from decimal import Decimal
-from typing import Dict, List, Optional, Tuple, Union
+from typing import Dict, List, Optional
 from uuid import UUID
 
 import pandas as pd
-from sqlalchemy import and_, desc, func
+from sqlalchemy import desc
 from sqlalchemy.orm import Session
 
 from mcli.ml.trading.alpaca_client import AlpacaTradingClient, create_trading_client
 from mcli.ml.trading.models import (
     OrderCreate,
     OrderResponse,
-    OrderSide,
     OrderStatus,
     OrderType,
     Portfolio,
     PortfolioCreate,
     PortfolioPerformanceSnapshot,
-    PortfolioResponse,
-    PortfolioType,
     Position,
     PositionResponse,
     PositionSide,
@@ -36,7 +33,7 @@ logger = logging.getLogger(__name__)
 
 
 class TradingService:
-    """Service for managing trading operations"""
+    """Service for managing trading operations."""
 
     def __init__(self, db_session: Session):
         self.db = db_session
@@ -44,7 +41,7 @@ class TradingService:
     def create_trading_account(
         self, user_id: UUID, account_data: TradingAccountCreate
     ) -> TradingAccount:
-        """Create a new trading account"""
+        """Create a new trading account."""
         try:
             account = TradingAccount(
                 user_id=user_id,
@@ -71,15 +68,15 @@ class TradingService:
             raise
 
     def get_trading_account(self, account_id: UUID) -> Optional[TradingAccount]:
-        """Get trading account by ID"""
+        """Get trading account by ID."""
         return (
             self.db.query(TradingAccount)
-            .filter(TradingAccount.id == account_id, TradingAccount.is_active == True)
+            .filter(TradingAccount.id == account_id, TradingAccount.is_active is True)
             .first()
         )
 
     def create_portfolio(self, account_id: UUID, portfolio_data: PortfolioCreate) -> Portfolio:
-        """Create a new portfolio"""
+        """Create a new portfolio."""
         try:
             portfolio = Portfolio(
                 trading_account_id=account_id,
@@ -103,24 +100,24 @@ class TradingService:
             raise
 
     def get_portfolio(self, portfolio_id: UUID) -> Optional[Portfolio]:
-        """Get portfolio by ID"""
+        """Get portfolio by ID."""
         return (
             self.db.query(Portfolio)
-            .filter(Portfolio.id == portfolio_id, Portfolio.is_active == True)
+            .filter(Portfolio.id == portfolio_id, Portfolio.is_active is True)
             .first()
         )
 
     def get_user_portfolios(self, user_id: UUID) -> List[Portfolio]:
-        """Get all portfolios for a user"""
+        """Get all portfolios for a user."""
         return (
             self.db.query(Portfolio)
             .join(TradingAccount)
-            .filter(TradingAccount.user_id == user_id, Portfolio.is_active == True)
+            .filter(TradingAccount.user_id == user_id, Portfolio.is_active is True)
             .all()
         )
 
     def create_alpaca_client(self, account: TradingAccount) -> AlpacaTradingClient:
-        """Create Alpaca client for trading account"""
+        """Create Alpaca client for trading account."""
         if not account.alpaca_api_key or not account.alpaca_secret_key:
             raise ValueError("Alpaca credentials not configured for this account")
 
@@ -131,7 +128,7 @@ class TradingService:
         )
 
     def sync_portfolio_with_alpaca(self, portfolio: Portfolio) -> bool:
-        """Sync portfolio data with Alpaca"""
+        """Sync portfolio data with Alpaca."""
         try:
             account = self.get_trading_account(portfolio.trading_account_id)
             if not account:
@@ -168,7 +165,7 @@ class TradingService:
             return False
 
     def _sync_positions(self, portfolio: Portfolio, alpaca_positions: List):
-        """Sync positions with Alpaca data"""
+        """Sync positions with Alpaca data."""
         # Clear existing positions
         self.db.query(Position).filter(Position.portfolio_id == portfolio.id).delete()
 
@@ -191,7 +188,7 @@ class TradingService:
             self.db.add(position)
 
     def _create_performance_snapshot(self, portfolio: Portfolio):
-        """Create daily performance snapshot"""
+        """Create daily performance snapshot."""
         snapshot = PortfolioPerformanceSnapshot(
             portfolio_id=portfolio.id,
             snapshot_date=datetime.utcnow().replace(hour=0, minute=0, second=0, microsecond=0),
@@ -209,7 +206,7 @@ class TradingService:
         self.db.add(snapshot)
 
     def _get_positions_data(self, portfolio_id: UUID) -> Dict:
-        """Get positions data for snapshot"""
+        """Get positions data for snapshot."""
         positions = self.db.query(Position).filter(Position.portfolio_id == portfolio_id).all()
         return {
             pos.symbol: {
@@ -227,7 +224,7 @@ class TradingService:
     def place_order(
         self, portfolio_id: UUID, order_data: OrderCreate, check_risk: bool = True
     ) -> TradingOrder:
-        """Place a trading order"""
+        """Place a trading order."""
         try:
             portfolio = self.get_portfolio(portfolio_id)
             if not portfolio:
@@ -311,7 +308,7 @@ class TradingService:
             raise
 
     def get_portfolio_positions(self, portfolio_id: UUID) -> List[PositionResponse]:
-        """Get all positions for a portfolio"""
+        """Get all positions for a portfolio."""
         positions = self.db.query(Position).filter(Position.portfolio_id == portfolio_id).all()
         return [
             PositionResponse(
@@ -337,7 +334,7 @@ class TradingService:
     def get_portfolio_orders(
         self, portfolio_id: UUID, status: Optional[OrderStatus] = None
     ) -> List[OrderResponse]:
-        """Get orders for a portfolio"""
+        """Get orders for a portfolio."""
         query = self.db.query(TradingOrder).filter(TradingOrder.portfolio_id == portfolio_id)
         if status:
             query = query.filter(TradingOrder.status == status)
@@ -370,7 +367,7 @@ class TradingService:
         ]
 
     def get_portfolio_performance(self, portfolio_id: UUID, days: int = 30) -> pd.DataFrame:
-        """Get portfolio performance history"""
+        """Get portfolio performance history."""
         end_date = datetime.utcnow()
         start_date = end_date - timedelta(days=days)
 
@@ -417,7 +414,7 @@ class TradingService:
         position_size: Optional[float] = None,
         expires_hours: int = 24,
     ) -> TradingSignal:
-        """Create a trading signal"""
+        """Create a trading signal."""
         try:
             expires_at = (
                 datetime.utcnow() + timedelta(hours=expires_hours) if expires_hours > 0 else None
@@ -450,12 +447,12 @@ class TradingService:
             raise
 
     def get_active_signals(self, portfolio_id: UUID) -> List[TradingSignalResponse]:
-        """Get active trading signals for a portfolio"""
+        """Get active trading signals for a portfolio."""
         signals = (
             self.db.query(TradingSignal)
             .filter(
                 TradingSignal.portfolio_id == portfolio_id,
-                TradingSignal.is_active == True,
+                TradingSignal.is_active is True,
                 TradingSignal.expires_at > datetime.utcnow(),
             )
             .order_by(desc(TradingSignal.created_at))
@@ -483,7 +480,7 @@ class TradingService:
         ]
 
     def calculate_portfolio_metrics(self, portfolio_id: UUID) -> Dict:
-        """Calculate portfolio performance metrics"""
+        """Calculate portfolio performance metrics."""
         portfolio = self.get_portfolio(portfolio_id)
         if not portfolio:
             return {}

mcli/ml/training/__init__.py

@@ -1,4 +1,4 @@
-"""ML model training module"""
+"""ML model training module."""
 
 from .train_model import PoliticianTradingNet, fetch_training_data
 from .train_model import main as train_model

mcli/ml/training/train.py

@@ -9,7 +9,6 @@ from mcli.lib.ui.styling import error, info, success
 @click.group(name="mcli-train", help="Model training CLI for MCLI ML system")
 def cli():
     """Main CLI group for model training."""
-    pass
 
 
 @cli.command(name="model", help="Train a model")

mcli/public/oi/oi.py

@@ -5,10 +5,9 @@ from mcli.lib.shell.shell import get_shell_script_path, shell_exec
 
 @click.group(name="oi")
 def oi():
-    """Create an alpha tunnel using the instance"""
+    """Create an alpha tunnel using the instance."""
     scripts_path = get_shell_script_path("oi", __name__)
     shell_exec(scripts_path, "oi")
-    pass
 
 
 if __name__ == "__main__":

mcli/self/completion_cmd.py

@@ -10,19 +10,16 @@ from pathlib import Path
 
 import click
 
-from mcli.lib.ui.styling import success
-
 
 @click.group(name="completion")
 def completion():
-    """Shell completion utilities"""
-    pass
+    """Shell completion utilities."""
 
 
 @completion.command(name="bash")
 @click.pass_context
 def bash_completion(ctx):
-    """Generate bash completion script"""
+    """Generate bash completion script."""
     from click.shell_completion import BashComplete
 
     # Get the root CLI app
@@ -39,7 +36,7 @@ def bash_completion(ctx):
 @completion.command(name="zsh")
 @click.pass_context
 def zsh_completion(ctx):
-    """Generate zsh completion script"""
+    """Generate zsh completion script."""
     from click.shell_completion import ZshComplete
 
     # Get the root CLI app
@@ -56,7 +53,7 @@ def zsh_completion(ctx):
 @completion.command(name="fish")
 @click.pass_context
 def fish_completion(ctx):
-    """Generate fish completion script"""
+    """Generate fish completion script."""
     from click.shell_completion import FishComplete
 
     # Get the root CLI app
@@ -78,8 +75,7 @@ def fish_completion(ctx):
 )
 @click.pass_context
 def install_completion(ctx, shell):
-    """Install shell completion for the current user"""
-    import subprocess
+    """Install shell completion for the current user."""
 
     # Auto-detect shell if not specified
     if not shell:
@@ -180,7 +176,7 @@ def install_completion(ctx, shell):
 
 @completion.command(name="status")
 def completion_status():
-    """Check current shell completion status"""
+    """Check current shell completion status."""
     current_shell = os.environ.get("SHELL", "unknown")
     shell_name = Path(current_shell).name if current_shell != "unknown" else "unknown"
 

mcli/self/logs_cmd.py

@@ -2,9 +2,7 @@
 Log streaming and management commands
 """
 
-import os
 import subprocess
-import sys
 import time
 from datetime import datetime
 from pathlib import Path
@@ -12,8 +10,6 @@ from typing import Optional
 
 import click
 from rich.console import Console
-from rich.live import Live
-from rich.panel import Panel
 from rich.text import Text
 
 from mcli.lib.paths import get_logs_dir
@@ -23,16 +19,15 @@ console = Console()
 
 @click.group(name="logs")
 def logs_group():
-    """Stream and manage MCLI log files"""
-    pass
+    """Stream and manage MCLI log files."""
 
 
 @logs_group.command(name="location")
 def show_location():
-    """Show the location of the logs directory"""
+    """Show the location of the logs directory."""
     logs_dir = get_logs_dir()
     console.print(f"📁 [cyan]Logs directory:[/cyan] {logs_dir}")
-    console.print(f"   [dim]Set MCLI_HOME environment variable to change location[/dim]")
+    console.print("   [dim]Set MCLI_HOME environment variable to change location[/dim]")
 
     # Show if directory exists and has files
     if logs_dir.exists():
@@ -40,9 +35,9 @@ def show_location():
         if log_files:
             console.print(f"   [green]✓ {len(log_files)} log file(s) found[/green]")
         else:
-            console.print(f"   [yellow]⚠ Directory exists but no log files yet[/yellow]")
+            console.print("   [yellow]⚠ Directory exists but no log files yet[/yellow]")
     else:
-        console.print(f"   [yellow]⚠ Directory will be created on first use[/yellow]")
+        console.print("   [yellow]⚠ Directory will be created on first use[/yellow]")
 
 
 @logs_group.command(name="stream")
@@ -111,7 +106,7 @@ def stream_logs(type: str, lines: int, follow: bool):
 @logs_group.command(name="list")
 @click.option("--date", "-d", help="Show logs for specific date (YYYYMMDD format)")
 def list_logs(date: Optional[str]):
-    """List available log files"""
+    """List available log files."""
     logs_dir = get_logs_dir()
     _list_available_logs(logs_dir, date)
 
@@ -207,7 +202,7 @@ def tail_logs(log_type: str, lines: int, date: Optional[str], follow: bool):
     "--context", "-C", type=int, default=3, help="Lines of context around matches (default: 3)"
 )
 def grep_logs(pattern: str, type: str, date: Optional[str], context: int):
-    """Search for patterns in log files"""
+    """Search for patterns in log files."""
     logs_dir = get_logs_dir()
 
     # Use provided date or default to today
@@ -251,7 +246,7 @@ def grep_logs(pattern: str, type: str, date: Optional[str], context: int):
 @click.option("--older-than", type=int, help="Clear logs older than N days")
 @click.option("--confirm", "-y", is_flag=True, help="Skip confirmation prompt")
 def clear_logs(older_than: Optional[int], confirm: bool):
-    """Clear old log files"""
+    """Clear old log files."""
     logs_dir = get_logs_dir()
 
     # Find log files
@@ -273,7 +268,7 @@ def clear_logs(older_than: Optional[int], confirm: bool):
             return
 
     # Show what will be deleted
-    console.print(f"📋 **Log files to clear:**", style="yellow")
+    console.print("📋 **Log files to clear:**", style="yellow")
     total_size = 0
     for log_file in log_files:
         size = log_file.stat().st_size
@@ -283,7 +278,7 @@ def clear_logs(older_than: Optional[int], confirm: bool):
     console.print(f"\n📊 **Total size:** {_format_file_size(total_size)}", style="cyan")
 
     # Confirm deletion
-    if not confirm:
+    if not confirm:  # noqa: SIM102
         if not click.confirm(f"\nDelete {len(log_files)} log files?"):
             console.print("❌ Operation cancelled", style="yellow")
             return
@@ -304,7 +299,7 @@ def clear_logs(older_than: Optional[int], confirm: bool):
 
 
 def _stream_single_file(log_file: Path, lines: int, follow: bool):
-    """Stream a single log file"""
+    """Stream a single log file."""
     console.print(f"📡 **Streaming {log_file.name}**", style="cyan")
     console.print("Press Ctrl+C to stop\n")
 
@@ -332,7 +327,7 @@ def _stream_single_file(log_file: Path, lines: int, follow: bool):
 
 
 def _stream_multiple_files(log_files: list, lines: int, follow: bool):
-    """Stream multiple log files simultaneously"""
+    """Stream multiple log files simultaneously."""
     console.print(f"📡 **Streaming {len(log_files)} log files**", style="cyan")
     for log_file in log_files:
         console.print(f"   • {log_file.name}")
@@ -340,7 +335,7 @@ def _stream_multiple_files(log_files: list, lines: int, follow: bool):
 
     # Use multitail or custom implementation
     # For simplicity, we'll cycle through files
-    try:
+    try:  # noqa: SIM105
         while True:
             for log_file in log_files:
                 if log_file.exists():
@@ -365,7 +360,7 @@ def _stream_multiple_files(log_files: list, lines: int, follow: bool):
 
 
 def _format_log_line(line: str) -> Text:
-    """Format a log line with syntax highlighting"""
+    """Format a log line with syntax highlighting."""
     text = Text()
 
     # Color-code by log level
@@ -384,7 +379,7 @@ def _format_log_line(line: str) -> Text:
 
 
 def _list_available_logs(logs_dir: Path, date_filter: Optional[str] = None):
-    """List available log files"""
+    """List available log files."""
     log_files = sorted(logs_dir.glob("mcli*.log"))
 
     if not log_files:
@@ -409,7 +404,7 @@ def _list_available_logs(logs_dir: Path, date_filter: Optional[str] = None):
 
 
 def _search_log_file(log_file: Path, pattern: str, context: int) -> list:
-    """Search for pattern in log file with context"""
+    """Search for pattern in log file with context."""
     matches = []
 
     try:
@@ -425,7 +420,7 @@ def _search_log_file(log_file: Path, pattern: str, context: int) -> list:
                 context_lines = []
                 for j in range(start, end):
                     prefix = ">>> " if j == i else "    "
-                    style = "bright_yellow" if j == i else "dim"
+                    _style = "bright_yellow" if j == i else "dim"  # noqa: F841
                     context_lines.append(f"{prefix}{lines[j].rstrip()}")
 
                 matches.append("\n".join(context_lines) + "\n")
@@ -437,7 +432,7 @@ def _search_log_file(log_file: Path, pattern: str, context: int) -> list:
 
 
 def _format_file_size(size_bytes: int) -> str:
-    """Format file size in human readable format"""
+    """Format file size in human readable format."""
     for unit in ["B", "KB", "MB", "GB"]:
         if size_bytes < 1024.0:
             return f"{size_bytes:.1f} {unit}"
@@ -447,5 +442,5 @@ def _format_file_size(size_bytes: int) -> str:
 
 # Register with main CLI
 def register_logs_commands(cli):
-    """Register logs commands with the main CLI"""
+    """Register logs commands with the main CLI."""
     cli.add_command(logs_group)

mcli/self/migrate_cmd.py

@@ -7,7 +7,6 @@ Handles migrations between different versions of mcli, including:
 - Command structure changes
 """
 
-import json
 import shutil
 from datetime import datetime
 from pathlib import Path
@@ -15,11 +14,10 @@ from typing import List, Tuple
 
 import click
 from rich.console import Console
-from rich.panel import Panel
 from rich.table import Table
 
 from mcli.lib.logger.logger import get_logger
-from mcli.lib.ui.styling import error, info, success, warning
+from mcli.lib.ui.styling import error, info, success
 
 logger = get_logger(__name__)
 console = Console()
@@ -82,9 +80,7 @@ def get_migration_status() -> dict:
         }
 
         if local_old.exists():
-            files = [
-                f for f in local_old.iterdir() if f.is_file() and not f.name.startswith(".")
-            ]
+            files = [f for f in local_old.iterdir() if f.is_file() and not f.name.startswith(".")]
             status["local"]["files_to_migrate"] = [f.name for f in files]
             status["local"]["needs_migration"] = len(files) > 0
 
@@ -289,19 +285,19 @@ def migrate_command(dry_run: bool, force: bool, status: bool, scope: str):
 
         # Show global status
         global_status = migration_status["global"]
-        console.print(f"\n[bold]Global (~/.mcli)[/bold]")
+        console.print("\n[bold]Global (~/.mcli)[/bold]")
         console.print(f"  Old location: {global_status['old_dir_path']}")
         console.print(f"    Exists: {'✓ Yes' if global_status['old_dir_exists'] else '✗ No'}")
         console.print(f"  New location: {global_status['new_dir_path']}")
         console.print(f"    Exists: {'✓ Yes' if global_status['new_dir_exists'] else '✗ No'}")
 
         if global_status["needs_migration"]:
-            console.print(f"  [yellow]⚠ Migration needed[/yellow]")
+            console.print("  [yellow]⚠ Migration needed[/yellow]")
             console.print(f"  Files to migrate: {len(global_status['files_to_migrate'])}")
         elif global_status["migration_done"]:
-            console.print(f"  [green]✓ Migration completed[/green]")
+            console.print("  [green]✓ Migration completed[/green]")
         else:
-            console.print(f"  [green]✓ No migration needed[/green]")
+            console.print("  [green]✓ No migration needed[/green]")
 
         # Show local status if in git repo
         if migration_status["local"]:
@@ -313,12 +309,12 @@ def migrate_command(dry_run: bool, force: bool, status: bool, scope: str):
             console.print(f"    Exists: {'✓ Yes' if local_status['new_dir_exists'] else '✗ No'}")
 
             if local_status["needs_migration"]:
-                console.print(f"  [yellow]⚠ Migration needed[/yellow]")
+                console.print("  [yellow]⚠ Migration needed[/yellow]")
                 console.print(f"  Files to migrate: {len(local_status['files_to_migrate'])}")
             elif local_status["migration_done"]:
-                console.print(f"  [green]✓ Migration completed[/green]")
+                console.print("  [green]✓ Migration completed[/green]")
             else:
-                console.print(f"  [green]✓ No migration needed[/green]")
+                console.print("  [green]✓ No migration needed[/green]")
 
         # Show files to migrate if any
         all_files = global_status.get("files_to_migrate", [])
@@ -326,7 +322,7 @@ def migrate_command(dry_run: bool, force: bool, status: bool, scope: str):
             all_files.extend(migration_status["local"].get("files_to_migrate", []))
 
         if all_files:
-            console.print(f"\n[bold]Files to migrate:[/bold]")
+            console.print("\n[bold]Files to migrate:[/bold]")
             table = Table()
             table.add_column("Location", style="cyan")
             table.add_column("File Name", style="yellow")
@@ -339,7 +335,7 @@ def migrate_command(dry_run: bool, force: bool, status: bool, scope: str):
                     table.add_row("Local", filename)
 
             console.print(table)
-            console.print(f"\n[dim]Run 'mcli self migrate' to perform migration[/dim]")
+            console.print("\n[dim]Run 'mcli self migrate' to perform migration[/dim]")
 
         return
 
@@ -356,19 +352,25 @@ def migrate_command(dry_run: bool, force: bool, status: bool, scope: str):
     console.print("\n[bold cyan]Migration Plan[/bold cyan]")
 
     if scope in ["global", "all"] and global_status["needs_migration"]:
-        console.print(f"\n[bold]Global:[/bold]")
+        console.print("\n[bold]Global:[/bold]")
         console.print(f"  Source: [cyan]{global_status['old_dir_path']}[/cyan]")
         console.print(f"  Target: [cyan]{global_status['new_dir_path']}[/cyan]")
         console.print(f"  Files: [yellow]{len(global_status['files_to_migrate'])}[/yellow]")
 
-    if scope in ["local", "all"] and migration_status["local"] and migration_status["local"]["needs_migration"]:
-        console.print(f"\n[bold]Local:[/bold]")
+    if (
+        scope in ["local", "all"]
+        and migration_status["local"]
+        and migration_status["local"]["needs_migration"]
+    ):
+        console.print("\n[bold]Local:[/bold]")
         console.print(f"  Source: [cyan]{migration_status['local']['old_dir_path']}[/cyan]")
         console.print(f"  Target: [cyan]{migration_status['local']['new_dir_path']}[/cyan]")
-        console.print(f"  Files: [yellow]{len(migration_status['local']['files_to_migrate'])}[/yellow]")
+        console.print(
+            f"  Files: [yellow]{len(migration_status['local']['files_to_migrate'])}[/yellow]"
+        )
 
     if dry_run:
-        console.print(f"\n[yellow]DRY RUN MODE - No changes will be made[/yellow]")
+        console.print("\n[yellow]DRY RUN MODE - No changes will be made[/yellow]")
 
     # Perform migration
     success_flag, message = migrate_commands_to_workflows(dry_run=dry_run, force=force, scope=scope)

mcli/self/redis_cmd.py

@@ -14,13 +14,12 @@ logger = get_logger(__name__)
 
 @click.group(name="redis")
 def redis_group():
-    """Manage Redis cache service"""
-    pass
+    """Manage Redis cache service."""
 
 
 @redis_group.command(name="start")
 def start_redis():
-    """Start Redis server"""
+    """Start Redis server."""
 
     async def _start():
         service = await get_redis_service()
@@ -51,7 +50,7 @@ def start_redis():
 
 @redis_group.command(name="stop")
 def stop_redis():
-    """Stop Redis server"""
+    """Stop Redis server."""
 
     async def _stop():
         service = await get_redis_service()
@@ -73,7 +72,7 @@ def stop_redis():
 
 @redis_group.command(name="restart")
 def restart_redis():
-    """Restart Redis server"""
+    """Restart Redis server."""
 
     async def _restart():
         service = await get_redis_service()
@@ -83,7 +82,7 @@ def restart_redis():
 
         if success:
             click.echo("✅ Redis server restarted successfully")
-            status = await service.get_status()
+            await service.get_status()
             click.echo(f"   Connection URL: {await service.get_connection_url()}")
         else:
             click.echo("❌ Failed to restart Redis server")
@@ -93,7 +92,7 @@ def restart_redis():
 
 @redis_group.command(name="status")
 def redis_status():
-    """Show Redis server status"""
+    """Show Redis server status."""
 
     async def _status():
         service = await get_redis_service()
@@ -133,7 +132,7 @@ def redis_status():
 
 @redis_group.command(name="test")
 def test_redis():
-    """Test Redis connection and performance"""
+    """Test Redis connection and performance."""
 
     async def _test():
         service = await get_redis_service()
@@ -162,7 +161,7 @@ def test_redis():
 
 @redis_group.command(name="info")
 def redis_info():
-    """Show detailed Redis server information"""
+    """Show detailed Redis server information."""
 
     async def _info():
         service = await get_redis_service()
@@ -235,7 +234,7 @@ def redis_info():
 @redis_group.command(name="logs")
 @click.option("--lines", "-n", default=20, help="Number of log lines to show")
 def redis_logs(lines):
-    """Show Redis server logs"""
+    """Show Redis server logs."""
 
     async def _logs():
         service = await get_redis_service()
@@ -265,5 +264,5 @@ def redis_logs(lines):
 
 # Register with main CLI
 def register_redis_commands(cli):
-    """Register Redis commands with the main CLI"""
+    """Register Redis commands with the main CLI."""
     cli.add_command(redis_group)