refactor: Clean up string concatenation and fix comment formatting

Author: Cyber-Syntax

AGENTS.md

@@ -5,19 +5,21 @@ This file provides guidance to agents when working with code in this repository.
 ## General Guidelines
 
 1. Modern Python First: Use Python 3.12+ features extensively - built-in generics, pattern matching, and dataclasses.
-2. Async-First Architecture: All I/O operations must be async. Use modern async patterns like `asyncio.TaskGroup` for concurrency.
-3. Type Safety: Full type annotations on all functions including return types. Use modern syntax (`dict[str, int]`, `str | None`).
-4. KISS Principle: Aim for simplicity and clarity. Avoid unnecessary abstractions or metaprogramming.
-5. DRY with Care: Reuse code appropriately but avoid over-engineering. Each command handler has single responsibility.
-6. Performance-Conscious: Use `@dataclass(slots=True)` when object count justifies it, orjson for JSON, and async-safe patterns over explicit locks.
+2. KISS Principle: Aim for simplicity and clarity. Avoid unnecessary abstractions or metaprogramming.
+3. DRY with Care: Reuse code appropriately but avoid over-engineering. Each command handler has single responsibility.
 
-## Activate venv before any test execution
+## Test after any change
 
-Unit test located in `tests/` directory
+1. Activate venv before any test execution:
 
 ```bash
 source .venv/bin/activate
-pytest -v -qa --strict-markers
+```
+
+2. Run pytest with following command to ensure all tests pass:
+
+```bash
+pytest -v -q --strict-markers
 ```
 
 ## Run the application
@@ -27,10 +29,3 @@ python run.py --help
 python run.py install appflowy
 python run.py update appflowy
 ```
-
-## More information about cli
-
-- Read developers.md for an overview of the architecture, api structure etc.
-    - [docs/developers.md](docs/developers.md)
-- Read wiki for detailed information about configuration, commands etc.
-    - [docs/wiki.md](docs/wiki.md)

my_unicorn/github_client.py

@@ -4,7 +4,6 @@
 information, extract AppImage assets, and manage GitHub-specific operations.
 """
 
-import re
 from dataclasses import dataclass
 from typing import Any, TypedDict, cast
 from urllib.parse import urlparse
@@ -17,8 +16,8 @@
 from .services.progress import get_progress_service
 from .utils import (
     extract_and_validate_version,
-    is_checksum_file,
     get_checksum_file_format_type,
+    is_checksum_file,
 )
 
 logger = get_logger(__name__)
@@ -198,6 +197,7 @@ async def _try_fetch_stable_release(
 
         Raises:
             aiohttp.ClientError: If there are actual API/network errors
+
         """
         try:
             # Check cache first (unless bypassed)
@@ -289,6 +289,7 @@ async def _try_fetch_prerelease(
 
         Raises:
             aiohttp.ClientError: If there are actual API/network errors
+
         """
         try:
             # Check cache first (unless bypassed)
@@ -699,7 +700,7 @@ def extract_appimage_assets(self, release_data: GitHubReleaseDetails) -> list[Gi
             if asset["name"].endswith(".AppImage") or asset["name"].endswith(".appimage")
         ]
 
-    #FIXME: too many branches
+    # FIXME: too many branches
     def select_best_appimage(
         self,
         release_data: GitHubReleaseDetails,
@@ -793,7 +794,7 @@ def select_best_appimage(
         # Fallback: return first candidate
         return candidates[0]
 
-    #FIXME: unused, move the auth or make this to check the available rate status
+    # FIXME: unused, move the auth or make this to check the available rate status
     # and use in the github api requests to prevent limit errors
     async def check_rate_limit(self) -> dict[str, Any]:
         """Check current rate limit status.
@@ -810,7 +811,7 @@ async def check_rate_limit(self) -> dict[str, Any]:
             data = await response.json()
             return data
 
-    #FIXME: unused? checkout parser.py or url install template method
+    # FIXME: unused? checkout parser.py or url install template method
     @staticmethod
     def parse_repo_url(repo_url: str) -> tuple[str, str]:
         """Parse GitHub repository URL to extract owner and repo.
@@ -868,7 +869,7 @@ async def get_default_branch(self) -> str:
                     f"Fetched default branch for {self.owner}/{self.repo}"
                 )
 
-            #FIXME: return str not Any
+            # FIXME: return str not Any
             return data.get("default_branch", "main")
 
     def build_icon_url(self, icon_path: str, branch: str | None = None) -> str:

my_unicorn/self_update.py

@@ -163,8 +163,12 @@ async def get_latest_release(self) -> dict[str, Any] | None:
             if e.status == HTTP_FORBIDDEN:
                 logger.error("GitHub API rate limit exceeded")
                 print(
-                    "GitHub Rate limit exceeded. Please try again later within 1 hour "
-                    + "or use different network/VPN."
+                    "".join(
+                        [
+                            "GitHub Rate limit exceeded. Please try again later ",
+                            "within 1 hour or use different network/VPN.",
+                        ]
+                    )
                 )
             else:
                 logger.error("GitHub API error: %s", e)
@@ -195,8 +199,12 @@ async def check_for_update(self) -> bool:
         if not latest_version_tag:
             logger.error("Malformed release data - no tag_name found")
             print(
-                "Malformed release data! Reinstall manually or "
-                + "open an issue on GitHub for help!"
+                "".join(
+                    [
+                        "Malformed release data! Reinstall manually or ",
+                        "open an issue on GitHub for help!",
+                    ]
+                )
             )
             return False