[feat] clean up type hints to use modern Python 3.9+ syntax with __future__ annotations

- Remove unnecessary # noqa comments
- Replace List[T] with list[T] and Dict[K,V] with dict[K,V]
- Replace Optional[T] with T  < /dev/null |  None
- Add 'from __future__ import annotations' to pr_cache.py
- Remove unused typing imports

Author: christian-byrne

comfy_cli/command/launch.py

@@ -6,7 +6,6 @@
 import sys
 import threading
 import uuid
-from typing import List, Optional  # noqa: UP035
 
 import typer
 from rich import print
@@ -121,8 +120,8 @@ def redirector_stdout():
 
 def launch(
     background: bool = False,
-    extra: Optional[List[str]] = None,  # noqa: UP006, UP045
-    frontend_pr: Optional[str] = None,  # noqa: UP045
+    extra: list[str] | None = None,
+    frontend_pr: str | None = None,
 ):
     check_for_updates()
     resolved_workspace = workspace_manager.workspace_path

comfy_cli/pr_cache.py

@@ -4,11 +4,12 @@
 quick switching between different PR versions without rebuilding.
 """
 
+from __future__ import annotations
+
 import json
 import shutil
 from datetime import datetime, timedelta
 from pathlib import Path
-from typing import Dict, List, Optional  # noqa: UP035
 
 from rich import print as rprint
 
@@ -95,7 +96,7 @@ def save_cache_info(self, pr_info, cache_path: Path) -> None:
         # Enforce cache limits after saving new cache
         self.enforce_cache_limits()
 
-    def get_cached_frontend_path(self, pr_info) -> Optional[Path]:
+    def get_cached_frontend_path(self, pr_info) -> Path | None:
         """Get path to cached frontend build if valid"""
         cache_path = self.get_frontend_cache_path(pr_info)
         dist_path = cache_path / "repo" / "dist"
@@ -104,7 +105,7 @@ def get_cached_frontend_path(self, pr_info) -> Optional[Path]:
             return dist_path
         return None
 
-    def _load_cache_info(self, cache_dir: Path) -> Optional[dict]:
+    def _load_cache_info(self, cache_dir: Path) -> dict | None:
         """Load cache info from a directory."""
         info_path = self.get_cache_info_path(cache_dir)
         if not info_path.exists():
@@ -127,7 +128,7 @@ def _clean_specific_pr_cache(self, frontend_cache: Path, pr_number: int) -> None
                 shutil.rmtree(cache_dir)
                 break
 
-    def clean_frontend_cache(self, pr_number: Optional[int] = None) -> None:
+    def clean_frontend_cache(self, pr_number: int | None = None) -> None:
         """Clean frontend cache (specific PR or all)."""
         frontend_cache = self.cache_dir / "frontend"
         if not frontend_cache.exists():
@@ -145,15 +146,15 @@ def _calculate_cache_size_mb(self, cache_dir: Path) -> float:
         total_size = sum(f.stat().st_size for f in cache_dir.rglob("*") if f.is_file())
         return total_size / (1024 * 1024)
 
-    def _get_cache_info_with_metadata(self, cache_dir: Path) -> Optional[dict]:
+    def _get_cache_info_with_metadata(self, cache_dir: Path) -> dict | None:
         """Get cache info with additional metadata like path and size."""
         info = self._load_cache_info(cache_dir)
         if info:
             info["cache_path"] = str(cache_dir)
             info["size_mb"] = self._calculate_cache_size_mb(cache_dir)
         return info
 
-    def list_cached_frontends(self) -> List[Dict]:  # noqa: UP006
+    def list_cached_frontends(self) -> list[dict]:
         """List all cached frontend PRs."""
         frontend_cache = self.cache_dir / "frontend"
         if not frontend_cache.exists():
@@ -177,7 +178,7 @@ def _is_cache_expired(self, cached_at: str) -> bool:
         except (ValueError, TypeError):
             return True  # Consider invalid timestamps as expired
 
-    def _get_expired_items(self, cached_items: List[Dict]) -> List[Dict]:  # noqa: UP006
+    def _get_expired_items(self, cached_items: list[dict]) -> list[dict]:
         """Get list of expired cache items."""
         expired = []
         for item in cached_items:
@@ -186,7 +187,7 @@ def _get_expired_items(self, cached_items: List[Dict]) -> List[Dict]:  # noqa: U
                 expired.append(item)
         return expired
 
-    def _get_excess_items(self, cached_items: List[Dict], expired_items: List[Dict]) -> List[Dict]:  # noqa: UP006
+    def _get_excess_items(self, cached_items: list[dict], expired_items: list[dict]) -> list[dict]:
         """Get list of items that exceed the maximum cache limit."""
         remaining_items = [item for item in cached_items if item not in expired_items]
         if len(remaining_items) > self.max_cache_items: