refactor: clean up imports and improve code readability across multiple files

Author: kalfasyan

src/vresto/services/sentinel_stream.py

@@ -8,14 +8,12 @@
 from __future__ import annotations
 
 import contextlib
-import hashlib
 import os
 import re
 import threading
 from pathlib import Path
-from typing import List, Optional, Tuple
+from typing import Optional
 
-import numpy as np
 from loguru import logger
 
 CDSE_S3_BUCKET = "eodata"
@@ -141,8 +139,6 @@ def get_quicklook_url(self, product_name: str) -> Optional[str]:
         """
         # CDSE OData quicklook endpoint
         clean_name = product_name.replace(".SAFE", "")
-        url = f"https://catalogue.dataspace.copernicus.eu/odata/v1/Products?$filter=Name eq '{clean_name}.SAFE'&$expand=Assets"
-        # For direct quicklook, use the standard pattern
         quicklook_url = f"https://catalogue.dataspace.copernicus.eu/odata/v1/Assets('{clean_name}.SAFE')/quicklook"
         return quicklook_url
 
@@ -194,6 +190,7 @@ def find_tci_path_in_product(self, s3_path: str, tile_code: str) -> Optional[str
         """
         try:
             import boto3
+
             from vresto.api.config import CopernicusConfig
 
             config = CopernicusConfig()

src/vresto/services/tiles.py

@@ -229,7 +229,7 @@ def _create_client(
 
     def _shutdown_client_obj(self, name: str, client) -> None:
         """Shut down a single client object (caller must have already removed from _clients)."""
-        url = self._urls.pop(name, None)
+        self._urls.pop(name, None)
         vrt = self._vrts.pop(name, None)
 
         if client:

src/vresto/ui/widgets/map_search_tab.py

@@ -12,7 +12,8 @@
     COLLECTION_PRODUCT_LEVELS,
     get_product_capabilities,
 )
-from vresto.services.mgrs_grid import compute_visible_tiles_geojson, is_available as mgrs_available
+from vresto.services.mgrs_grid import compute_visible_tiles_geojson
+from vresto.services.mgrs_grid import is_available as mgrs_available
 from vresto.services.sentinel_stream import sentinel_stream_service
 from vresto.services.tiles import tile_pool
 from vresto.ui.widgets.activity_log import ActivityLogWidget
@@ -245,7 +246,6 @@ async def _stream_tile(self, tile_code: str):
             if thumb_url and thumb_url.startswith("https://"):
                 # Add as image overlay (approximation — use tile layer for now)
                 self._add_message(f"🖼️ Loading quicklook for {tile_code}...")
-                quicklook_shown = True
 
         # Step 2: Stream full TCI in background
         self._add_message(f"📡 Streaming TCI for {tile_code} ({product.sensing_date})...")
@@ -329,9 +329,10 @@ async def _search_product_for_tile(self, tile_code: str) -> Optional[ProductInfo
         search_code = tile_code if tile_code.startswith("T") else f"T{tile_code}"
 
         def _query():
-            from vresto.api.auth import CopernicusAuth
             import requests
 
+            from vresto.api.auth import CopernicusAuth
+
             config = CopernicusConfig()
             auth = CopernicusAuth(config=config)
             headers = auth.get_headers()

tests/test_services/test_tiles.py

@@ -1,11 +1,11 @@
 import os
 import tempfile
-from unittest.mock import MagicMock, patch
+from unittest.mock import patch
 
 import pytest
 import rasterio
 
-from vresto.services.tiles import TileManager, TilePool, tile_pool
+from vresto.services.tiles import TileManager, TilePool
 
 
 @pytest.fixture