Merge branch 'main' into main

.github/workflows/bot_check.yml

@@ -6,12 +6,14 @@ on:
     paths:
       - zhenxun/**
       - tests/**
+      - .github/workflows/bot_check.yml
       - bot.py
   pull_request:
     branches: ["main"]
     paths:
       - zhenxun/**
       - tests/**
+      - .github/workflows/bot_check.yml
       - bot.py
 
 jobs:
@@ -63,9 +65,9 @@ jobs:
         run: |
           poetry run sudo apt-get update
           poetry run sudo apt-get install -y libgstreamer-plugins-base1.0-0 libgstreamer1.0-0 gstreamer1.0-plugins-base gstreamer1.0-plugins-good gstreamer1.0-plugins-bad gstreamer1.0-libav flite x264 libx264-dev
-          poetry run sudo playwright install-deps 
           poetry run pip install playwright
-          poetry run sudo playwright install
+          poetry run playwright install-deps
+          poetry run playwright install
 
       - name: Run tests
         run: poetry run pytest --cov=zhenxun --cov-report xml

tests/builtin_plugins/check/test_check.py

@@ -1,3 +1,4 @@
+
 from collections.abc import Callable
 from pathlib import Path
 import platform
@@ -229,3 +230,7 @@ async def test_check_arm(
     mock_template_to_pic.assert_awaited_once()
     mock_build_message.assert_called_once_with(mock_template_to_pic_return)
     mock_build_message_return.send.assert_awaited_once()
+
+
+
+

tests/builtin_plugins/plugin_store/test_update_all_plugin.py

@@ -57,7 +57,9 @@ async def test_update_all_plugin_basic_need_update(
         )
         ctx.should_call_send(
             event=event,
-            message=Message(message="已更新插件 识图\n共计1个插件! 重启后生效"),
+            message=Message(
+                message="--已更新1个插件 0个失败 1个成功--\n* 以下插件更新成功:\n\t- 识图\n重启后生效"  # noqa: E501
+            ),
             result=None,
             bot=bot,
         )

zhenxun/builtin_plugins/plugin_store/data_source.py

@@ -256,10 +256,9 @@ class ShopManage:
                 result = await AsyncHttpx.gather_download_file(
                     req_download_urls, req_paths
                 )
-                for _id, success in enumerate(result):
+                for success in result:
                     if not success:
                         raise Exception("插件依赖文件下载失败")
-                else:
                 logger.debug(f"插件依赖文件列表: {req_paths}", "插件管理")
                 install_requirement(plugin_path)
             return True
@@ -389,9 +388,12 @@ class ShopManage:
         """
         data: dict[str, StorePluginInfo] = await cls.get_data()
         plugin_list = list(data.keys())
-        update_list = []
+        update_failed_list = []
+        update_success_list = []
+        result = "--已更新{}个插件 {}个失败 {}个成功--"
         logger.info(f"尝试更新全部插件 {plugin_list}", "插件管理")
         for plugin_key in plugin_list:
+            try:
                 plugin_info = data[plugin_key]
                 plugin_list = await cls.get_loaded_plugins("module", "version")
                 suc_plugin = {p[0]: (p[1] or "Unknown") for p in plugin_list}
@@ -412,11 +414,27 @@ class ShopManage:
                     plugin_info.is_dir,
                     is_external,
                 )
-            update_list.append(plugin_key)
+                update_success_list.append(plugin_key)
-        if len(update_list) == 0:
+            except Exception as e:
+                logger.error(f"更新插件 {plugin_key} 失败: {e}", "插件管理")
+                update_failed_list.append(plugin_key)
+        if not update_success_list and not update_failed_list:
             return "全部插件已是最新版本"
-        return "已更新插件 {}\n共计{}个插件! 重启后生效".format(
+        if update_success_list:
-            "\n- ".join(update_list), len(update_list)
+            result += "\n* 以下插件更新成功:\n\t- {}".format(
+                "\n\t- ".join(update_success_list)
+            )
+        if update_failed_list:
+            result += "\n* 以下插件更新失败:\n\t- {}".format(
+                "\n\t- ".join(update_failed_list)
+            )
+        return (
+            result.format(
+                len(update_success_list) + len(update_failed_list),
+                len(update_failed_list),
+                len(update_success_list),
+            )
+            + "\n重启后生效"
         )
 
     @classmethod