test on local image

.gitea/workflows/release.yml

@@ -1,6 +1,7 @@
 name: Build and Publish Release
 
 on:
+  push: # Для теста лучше использовать push, чтобы не создавать релиз каждый раз
   release:
     types: [published]
 
@@ -8,23 +9,21 @@ jobs:
   build-linux:
     name: Build Linux
     runs-on: ubuntu-latest
+    container:
+      image: git.borderban.ru/borderban/ci-images/py-builder:latest
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
 
-      - name: Set up Python
-        uses: actions/setup-python@v5
+      # Кэшируем pip, чтобы не качать зависимости из requirements.txt каждый раз
+      - name: Cache pip
+        uses: actions/cache@v3
         with:
-          python-version: '3.11'
-
-      - name: Install system dependencies (Tkinter)
-        run: apt-get update && apt-get install -y python3-tk
+          path: ~/.cache/pip
+          key: ${{ runner.os }}-pip-${{ hashFiles('requirements.txt') }}
 
       - name: Install Python dependencies
-        run: |
-          python -m pip install --upgrade pip
-          pip install -r requirements.txt
-          pip install pyinstaller
+        run: pip install -r requirements.txt
 
       - name: Build Linux executable
         run: pyinstaller --onefile --windowed --name factorio-mod-sync-linux main.py
@@ -39,21 +38,20 @@ jobs:
     name: Build Windows
     runs-on: ubuntu-latest
     container:
-      image: tobix/pywine:3.11
+      image: git.borderban.ru/borderban/ci-images/py-builder:latest
     steps:
-      - name: Install Node.js and Git (required for Actions)
-        run: apt-get update && apt-get install -y nodejs git
-
       - name: Checkout code
         uses: actions/checkout@v4
 
       - name: Install Python dependencies (Windows)
         run: |
-          wine python -m pip install --upgrade pip
-          wine pip install -r requirements.txt
-          wine pip install pyinstaller
+          # В твоем образе py-builder мы ставим пакеты обычно, 
+          # а pyinstaller упакует их под Windows благодаря Wine внутри
+          pip install -r requirements.txt
 
       - name: Build Windows executable
+        # Внутри твоего контейнера вызывай сборку. 
+        # Если в образе настроен wine как алиас или через него — используй его.
         run: wine pyinstaller --onefile --windowed --name factorio-mod-sync-windows.exe main.py
 
       - name: Upload Windows artifact
@@ -66,7 +64,8 @@ jobs:
     name: Upload Assets to Release
     needs: [build-linux, build-windows]
     runs-on: ubuntu-latest
-    if: always() && (needs.build-linux.result == 'success' || needs.build-windows.result == 'success')
+    # Этот шаг запустится только если это реально Релиз
+    if: github.event_name == 'release'
     steps:
       - name: Download all artifacts
         uses: actions/download-artifact@v3

main.py

@@ -11,31 +11,11 @@ import threading
 import subprocess
 import re
 
+APP_VERSION = "v1.1.6"
 UPDATE_API_URL = "https://git.borderban.ru/api/v1/repos/BorderBan/client-py/releases/latest"
 SERVER_URL = "https://server1.borderban.ru/mods/"
 CONFIG_FILE = Path.home() / ".factorio_sync_config.json"
 
-def load_app_config():
-    if CONFIG_FILE.exists():
-        try:
-            return json.loads(CONFIG_FILE.read_text())
-        except Exception:
-            pass
-    return {}
-
-def save_app_config(new_data):
-    config = load_app_config()
-    config.update(new_data)
-    try:
-        CONFIG_FILE.write_text(json.dumps(config))
-    except Exception as e:
-        print(f"Ошибка сохранения конфига: {e}")
-
-if not getattr(sys, 'frozen', False):
-    APP_VERSION = "Dev"
-else:
-    APP_VERSION = load_app_config().get("app_version", "Неизвестная версия")
-
 def get_mod_base_name(filename):
     # Пытаемся отсечь версию мода (обычно ИмяМода_1.0.0.zip)
     match = re.match(r"^(.+)_(\d+\.\d+\.\d+)\.zip$", filename)
@@ -82,10 +62,12 @@ def check_and_update(root):
         release_data = response.json()
         latest_version = release_data.get("tag_name")
 
-        if not latest_version:
+        # Проверяем, нужна ли загрузка
+        if not latest_version or latest_version == APP_VERSION:
             update_win.destroy()
             return
 
+        # Определяем имя нужного ассета в зависимости от ОС
         system = platform.system().lower()
         asset_name = None
         if system == "windows":
@@ -97,34 +79,12 @@ def check_and_update(root):
             update_win.destroy()
             return
 
-        target_asset = next((asset for asset in release_data.get("assets", []) if asset.get("name") == asset_name), None)
-        if not target_asset:
-            update_win.destroy()
-            return
-
-        download_url = target_asset.get("browser_download_url")
+        download_url = next((asset.get("browser_download_url") for asset in release_data.get("assets", []) if asset.get("name") == asset_name), None)
 
         if not download_url:
             update_win.destroy()
             return
 
-        remote_size = target_asset.get("size", 0)
-        local_size = os.path.getsize(exe_path)
-        
-        # Если размер файла совпадает с размером из релиза, считаем, что мы уже на этой версии
-        if remote_size and local_size == remote_size:
-            if load_app_config().get("app_version") != latest_version:
-                save_app_config({"app_version": latest_version})
-                global APP_VERSION
-                APP_VERSION = latest_version
-            update_win.destroy()
-            return
-        
-        # На крайний случай проверяем сохраненную версию, если размер не удалось получить
-        if not remote_size and load_app_config().get("app_version") == latest_version:
-            update_win.destroy()
-            return
-
         # Обновляем текст на окне, если обновление найдено
         update_label.config(text=f"Обнаружена новая версия ({latest_version}).\nЗагрузка обновления...\nПожалуйста, подождите.")
         update_win.update()
@@ -144,9 +104,6 @@ def check_and_update(root):
         os.replace(exe_path, old_exe_path)
         os.replace(new_exe_path, exe_path)
 
-        # Сохраняем новую версию в конфиг перед перезапуском
-        save_app_config({"app_version": latest_version})
-
         # Перезапускаем новую версию и выходим из текущей
         subprocess.Popen([exe_path] + sys.argv[1:])
         sys.exit(0)
@@ -162,23 +119,25 @@ class SyncApp:
         self.root.title(f"Factorio Mod Sync {APP_VERSION}")
         self.root.geometry("500x350")
 
-        self.config = load_app_config()
-        if "mods_path" not in self.config:
-            self.config["mods_path"] = ""
-        if "game_path" not in self.config:
-            self.config["game_path"] = ""
-
+        self.config = self.load_config()
         self.setup_ui()
 
+    def load_config(self):
+        if CONFIG_FILE.exists():
+            data = json.loads(CONFIG_FILE.read_text())
+            return {
+                "mods_path": data.get("mods_path", ""),
+                "game_path": data.get("game_path", "")
+            }
+        return {"mods_path": "", "game_path": ""}
+
     def save_config(self, mods_path=None, game_path=None):
-        data_to_save = {}
+        config = self.load_config()
         if mods_path is not None:
-            data_to_save["mods_path"] = mods_path
-            self.config["mods_path"] = mods_path
+            config["mods_path"] = mods_path
         if game_path is not None:
-            data_to_save["game_path"] = game_path
-            self.config["game_path"] = game_path
-        save_app_config(data_to_save)
+            config["game_path"] = game_path
+        CONFIG_FILE.write_text(json.dumps(config))
 
     def setup_ui(self):
         # Выбор пути