Remove loading view and rework connections

.pre-commit-config.yaml

@@ -13,21 +13,27 @@ repos:
         args: ["--autofix", "--no-sort-keys", "--indent", "4"]
     -   id: check-added-large-files
 
+-   repo: https://github.com/asottile/pyupgrade
+    rev: v3.19.1
+    hooks:
+    -   id: pyupgrade
+        args: ["--py312-plus"]
+
 -   repo: https://github.com/astral-sh/ruff-pre-commit
     rev: v0.8.2
     hooks:
     -   id: ruff
         args: [ "--fix" ]
     -   id: ruff-format
 
+-   repo: https://github.com/PyCQA/autoflake
+    rev: v2.3.1
+    hooks:
+    -   id: autoflake
+
 -   repo: https://github.com/pre-commit/mirrors-mypy
     rev: v1.13.0
     hooks:
     -   id: mypy
         args: ["--pretty"]
         additional_dependencies: ["pygobject-stubs", "types-PyYAML", "types-Markdown", "types-requests", "types-pycurl", "types-chardet", "pytest-stub", "types-orjson", "pathvalidate", "requirements-parser", "icoextract", "fvs", "patool", "git+https://gitlab.com/TheEvilSkeleton/vkbasalt-cli.git@main"]
-
--   repo: https://github.com/PyCQA/autoflake
-    rev: v2.3.1
-    hooks:
-    -   id: autoflake

bottles/backend/cabextract.py

@@ -19,7 +19,6 @@
 import shlex
 import shutil
 import subprocess
-from typing import Optional
 
 from bottles.backend.logger import Logger
 
@@ -47,7 +46,7 @@ def run(
         self,
         path: str,
         name: str = "",
-        files: Optional[list] = None,
+        files: list | None = None,
         destination: str = "",
     ):
         if files is None:

bottles/backend/downloader.py

@@ -18,7 +18,6 @@
 import shutil
 import sys
 import time
-from typing import Optional
 
 import requests
 
@@ -38,7 +37,7 @@ class Downloader:
     """
 
     def __init__(
-        self, url: str, file: str, update_func: Optional[TaskStreamUpdateHandler] = None
+        self, url: str, file: str, update_func: TaskStreamUpdateHandler | None = None
     ):
         self.start_time = None
         self.url = url

bottles/backend/globals.py

@@ -18,7 +18,6 @@
 import os
 import shutil
 from pathlib import Path
-from typing import Dict
 from bottles.backend.utils import yaml, json
 
 
@@ -82,12 +81,12 @@ class TrdyPaths:
 vmtouch_available = shutil.which("vmtouch") or False
 base_version = ""
 if os.path.isfile("/app/manifest.json"):
-    with open("/app/manifest.json", mode="r", encoding="utf-8") as file:
+    with open("/app/manifest.json", encoding="utf-8") as file:
         base_version = (
             json.load(file)  # type: ignore
             .get("base-version", "")
             .removeprefix("stable-")
         )
 
 # encoding detection correction, following windows defaults
-locale_encodings: Dict[str, str] = {"ja_JP": "cp932", "zh_CN": "gbk"}
+locale_encodings: dict[str, str] = {"ja_JP": "cp932", "zh_CN": "gbk"}

bottles/backend/managers/component.py

@@ -20,7 +20,6 @@
 import shutil
 import tarfile
 from functools import lru_cache
-from typing import Optional
 
 import pycurl
 
@@ -45,10 +44,9 @@
 
 # noinspection PyTypeChecker
 class ComponentManager:
-    def __init__(self, manager, offline: bool = False):
+    def __init__(self, manager):
         self.__manager = manager
-        self.__repo = manager.repository_manager.get_repo("components", offline)
-        self.__utils_conn = manager.utils_conn
+        self.__repo = manager.repository_manager.get_repo("components")
 
     @lru_cache
     def get_component(self, name: str, plain: bool = False) -> dict:
@@ -59,9 +57,6 @@ def fetch_catalog(self) -> dict:
         Fetch all components from the Bottles repository, mark the installed
         ones and return a dict with the catalog.
         """
-        if not self.__utils_conn.check_connection():
-            return {}
-
         catalog = {
             "runtimes": {},
             "wine": {},
@@ -131,7 +126,7 @@ def download(
         file: str,
         rename: str = "",
         checksum: str = "",
-        func: Optional[TaskStreamUpdateHandler] = None,
+        func: TaskStreamUpdateHandler | None = None,
     ) -> bool:
         """Download a component from the Bottles repository."""
 
@@ -274,7 +269,7 @@ def extract(name: str, component: str, archive: str) -> bool:
             root_dir = tar.getnames()[0]
             tar.extractall(path)
             tar.close()
-        except (tarfile.TarError, IOError, EOFError):
+        except (tarfile.TarError, OSError, EOFError):
             with contextlib.suppress(FileNotFoundError):
                 os.remove(os.path.join(Paths.temp, archive))
             with contextlib.suppress(FileNotFoundError):
@@ -301,7 +296,7 @@ def install(
         self,
         component_type: str,
         component_name: str,
-        func: Optional[TaskStreamUpdateHandler] = None,
+        func: TaskStreamUpdateHandler | None = None,
     ):
         """
         This function is used to install a component. It automatically

bottles/backend/managers/conf.py

@@ -1,16 +1,15 @@
 import os
 from configparser import ConfigParser
-from typing import Optional
 
 from bottles.backend.utils import yaml, json
 
 
-class ConfigManager(object):
+class ConfigManager:
     def __init__(
         self,
-        config_file: Optional[str] = None,
+        config_file: str | None = None,
         config_type: str = "ini",
-        config_string: Optional[str] = None,
+        config_string: str | None = None,
     ):
         self.config_file = config_file
         self.config_string = config_string
@@ -44,10 +43,10 @@ def read(self):
                 # noinspection PyProtectedMember
                 res = config._sections
             elif self.config_type == "json":
-                with open(self.config_file, "r") as f:
+                with open(self.config_file) as f:
                     res = json.load(f)
             elif self.config_type == "yaml" or self.config_type == "yml":
-                with open(self.config_file, "r") as f:
+                with open(self.config_file) as f:
                     res = yaml.load(f)
             else:
                 raise ValueError("Invalid configuration type")
@@ -94,7 +93,7 @@ def write_ini(self):
         with open(self.config_file, "w") as f:
             config.write(f)
 
-    def write_dict(self, config_file: Optional[str] = None):
+    def write_dict(self, config_file: str | None = None):
         if self.config_file is None and config_file is None:
             raise ValueError("No config path specified")
         elif self.config_file is None and config_file is not None:

bottles/backend/managers/data.py

@@ -45,7 +45,7 @@ def __init__(self):
 
     def __get_data(self):
         try:
-            with open(self.__p_data, "r") as s:
+            with open(self.__p_data) as s:
                 self.__data = yaml.load(s)
                 if self.__data is None:
                     raise AttributeError

bottles/backend/managers/dependency.py

@@ -43,10 +43,9 @@
 
 
 class DependencyManager:
-    def __init__(self, manager, offline: bool = False):
+    def __init__(self, manager):
         self.__manager = manager
-        self.__repo = manager.repository_manager.get_repo("dependencies", offline)
-        self.__utils_conn = manager.utils_conn
+        self.__repo = manager.repository_manager.get_repo("dependencies")
 
     @lru_cache
     def get_dependency(self, name: str, plain: bool = False) -> str | dict | bool:
@@ -59,8 +58,6 @@ def fetch_catalog(self) -> dict:
         and return these as a dictionary. It also returns an empty dictionary
         if there are no dependencies or fails to fetch them.
         """
-        if not self.__utils_conn.check_connection():
-            return {}
 
         catalog = {}
         index = self.__repo.catalog
@@ -91,7 +88,9 @@ def install(self, config: BottleConfig, dependency: list) -> Result:
         task_id = TaskManager.add(Task(title=dependency[0]))
 
         logging.info(
-            "Installing dependency [%s] in bottle [%s]." % (dependency[0], config.Name),
+            "Installing dependency [{}] in bottle [{}].".format(
+                dependency[0], config.Name
+            ),
         )
         manifest = self.get_dependency(dependency[0])
         if not manifest:

bottles/backend/managers/epicgamesstore.py

@@ -59,7 +59,7 @@ def get_installed_games(config: BottleConfig) -> list:
         if dat_path is None:
             return []
 
-        with open(dat_path, "r") as dat:
+        with open(dat_path) as dat:
             data = json.load(dat)
 
             for game in data["InstallationList"]:

bottles/backend/managers/installer.py

@@ -18,7 +18,6 @@
 import subprocess
 import uuid
 from functools import lru_cache
-from typing import Optional
 
 import markdown
 import pycurl
@@ -37,10 +36,9 @@
 
 
 class InstallerManager:
-    def __init__(self, manager, offline: bool = False):
+    def __init__(self, manager):
         self.__manager = manager
-        self.__repo = manager.repository_manager.get_repo("installers", offline)
-        self.__utils_conn = manager.utils_conn
+        self.__repo = manager.repository_manager.get_repo("installers")
         self.__component_manager = manager.component_manager
         self.__local_resources = {}
 
@@ -69,8 +67,6 @@ def fetch_catalog(self) -> dict:
         """Fetch the installers catalog from the repository"""
         catalog = {}
         index = self.__repo.catalog
-        if not self.__utils_conn.check_connection():
-            return {}
 
         for installer in index.items():
             catalog[installer[0]] = installer[1]
@@ -358,7 +354,7 @@ def install(
         installer: dict,
         step_fn: callable,
         is_final: bool = True,
-        local_resources: Optional[dict] = None,
+        local_resources: dict | None = None,
     ):
         manifest = self.get_installer(installer[0])
         _config = config

bottles/backend/managers/journal.py

@@ -20,7 +20,6 @@
 import shutil
 import uuid
 from datetime import datetime, timedelta
-from typing import Optional
 
 from bottles.backend.globals import Paths
 from bottles.backend.utils import yaml
@@ -52,7 +51,7 @@ def __get_journal() -> dict:
             with open(JournalManager.path, "w") as f:
                 yaml.dump({}, f)
 
-        with open(JournalManager.path, "r") as f:
+        with open(JournalManager.path) as f:
             try:
                 journal = yaml.load(f)
             except yaml.YAMLError:
@@ -100,7 +99,7 @@ def __clean_old():
         JournalManager.__save_journal(journal)
 
     @staticmethod
-    def __save_journal(journal: Optional[dict] = None):
+    def __save_journal(journal: dict | None = None):
         """Save the journal to the journal file."""
         if journal is None:
             journal = JournalManager.__get_journal()

bottles/backend/managers/library.py

@@ -49,7 +49,7 @@ def load_library(self, silent=False):
             self.__library = {}
             self.save_library()
         else:
-            with open(self.library_path, "r") as library_file:
+            with open(self.library_path) as library_file:
                 self.__library = yaml.load(library_file)
 
         if self.__library is None: