refactor: 统一变量命名并优化代码结构

- 将函数名从 handle_admin_command 改为 admin_command_handler 以保持命名一致性
- 将变量名 comm_prefixes 改为 command_prefixes 以提高可读性
- 重命名 full_cmd 为 command_parts 和 cmd_name 为 command_name 以明确用途
- 简化 WebSocket 相关代码,移除未使用的导入
- 优化 main.py 中的初始化逻辑和变量命名
This commit is contained in:
2026-01-07 23:02:15 +08:00
parent 56b1014419
commit c3b3541694
5 changed files with 39 additions and 41 deletions

View File

@@ -116,15 +116,15 @@ class MessageHandler(BaseHandler):
if not prefix_found: if not prefix_found:
return return
full_cmd = raw_text[len(prefix_found):].split() command_parts = raw_text[len(prefix_found):].split()
if not full_cmd: if not command_parts:
return return
cmd_name = full_cmd[0] command_name = command_parts[0]
args = full_cmd[1:] args = command_parts[1:]
if cmd_name in self.commands: if command_name in self.commands:
command_info = self.commands[cmd_name] command_info = self.commands[command_name]
func = command_info["func"] func = command_info["func"]
permission = command_info.get("permission") permission = command_info.get("permission")
override_check = command_info.get("override_permission_check", False) override_check = command_info.get("override_permission_check", False)

View File

@@ -12,7 +12,7 @@ from ..handlers.event_handler import MessageHandler, NoticeHandler, RequestHandl
# 从配置中获取命令前缀 # 从配置中获取命令前缀
comm_prefixes = global_config.bot.get("command", ("/",)) command_prefixes = global_config.bot.get("command", ("/",))
class CommandManager: class CommandManager:
@@ -133,11 +133,11 @@ class CommandManager:
# --- 全局单例 --- # --- 全局单例 ---
# 确保前缀配置是元组格式 # 确保前缀配置是元组格式
if isinstance(comm_prefixes, list): if isinstance(command_prefixes, list):
comm_prefixes = tuple(comm_prefixes) command_prefixes = tuple(command_prefixes)
elif isinstance(comm_prefixes, str): elif isinstance(command_prefixes, str):
comm_prefixes = (comm_prefixes,) command_prefixes = (command_prefixes,)
# 实例化全局唯一的命令管理器 # 实例化全局唯一的命令管理器
matcher = CommandManager(prefixes=comm_prefixes) matcher = CommandManager(prefixes=command_prefixes)

View File

@@ -13,9 +13,7 @@ WebSocket 连接。它是整个机器人框架的底层通信基础。
""" """
import asyncio import asyncio
import json import json
import traceback
import uuid import uuid
from datetime import datetime
import websockets import websockets
@@ -78,7 +76,7 @@ class WS:
logger.info(f"{self.reconnect_interval}秒后尝试重连...") logger.info(f"{self.reconnect_interval}秒后尝试重连...")
await asyncio.sleep(self.reconnect_interval) await asyncio.sleep(self.reconnect_interval)
async def _listen_loop(self, websocket): async def _listen_loop(self, websocket_connection):
""" """
核心监听循环,处理所有接收到的 WebSocket 消息。 核心监听循环,处理所有接收到的 WebSocket 消息。
@@ -86,9 +84,9 @@ class WS:
判断是 API 响应还是上报的事件,然后分发给相应的处理逻辑。 判断是 API 响应还是上报的事件,然后分发给相应的处理逻辑。
Args: Args:
websocket: 当前活动的 WebSocket 连接对象。 websocket_connection: 当前活动的 WebSocket 连接对象。
""" """
async for message in websocket: async for message in websocket_connection:
try: try:
data = json.loads(message) data = json.loads(message)
@@ -110,7 +108,7 @@ class WS:
except Exception as e: except Exception as e:
logger.exception(f"解析消息异常: {e}") logger.exception(f"解析消息异常: {e}")
async def on_event(self, raw_data: dict): async def on_event(self, event_data: dict):
""" """
事件处理和分发层。 事件处理和分发层。
@@ -121,11 +119,11 @@ class WS:
4. 将事件对象传递给 `CommandManager` (`matcher`) 进行后续处理。 4. 将事件对象传递给 `CommandManager` (`matcher`) 进行后续处理。
Args: Args:
raw_data (dict): 从 WebSocket 接收到的原始事件字典。 event_data (dict): 从 WebSocket 接收到的原始事件字典。
""" """
try: try:
# 使用工厂创建事件对象 # 使用工厂创建事件对象
event = EventFactory.create_event(raw_data) event = EventFactory.create_event(event_data)
event.bot = self.bot # 注入 Bot 实例 event.bot = self.bot # 注入 Bot 实例
# 打印日志 # 打印日志

36
main.py
View File

@@ -7,11 +7,6 @@ import asyncio
import os import os
import sys import sys
import time import time
# 将项目根目录添加到 sys.path
ROOT_DIR = os.path.dirname(os.path.abspath(__file__))
sys.path.insert(0, ROOT_DIR)
from watchdog.observers import Observer from watchdog.observers import Observer
from watchdog.events import FileSystemEventHandler from watchdog.events import FileSystemEventHandler
@@ -22,7 +17,14 @@ from core.managers.admin_manager import admin_manager
from core.ws import WS from core.ws import WS
from core.managers.plugin_manager import load_all_plugins from core.managers.plugin_manager import load_all_plugins
from core.managers.redis_manager import redis_manager from core.managers.redis_manager import redis_manager
from core.utils.executor import run_in_thread_pool from core.utils.executor import run_in_thread_pool, initialize_executor
from core.config_loader import global_config as config
# 将项目根目录添加到 sys.path
ROOT_DIR = os.path.dirname(os.path.abspath(__file__))
sys.path.insert(0, ROOT_DIR)
class PluginReloadHandler(FileSystemEventHandler): class PluginReloadHandler(FileSystemEventHandler):
@@ -42,21 +44,21 @@ class PluginReloadHandler(FileSystemEventHandler):
self.last_reload_time = 0 self.last_reload_time = 0
self.cooldown = 1.0 # 冷却时间,防止短时间内多次重载 self.cooldown = 1.0 # 冷却时间,防止短时间内多次重载
def on_any_event(self, event): def on_any_event(self, file_system_event):
""" """
处理所有文件事件 处理所有文件事件
:param event: watchdog 事件对象 :param file_system_event: watchdog 事件对象
""" """
if event.is_directory: if file_system_event.is_directory:
return return
# 只监控 py 文件 # 只监控 py 文件
if not event.src_path.endswith(".py"): if not file_system_event.src_path.endswith(".py"):
return return
# 过滤掉一些临时文件 # 过滤掉一些临时文件
if "__pycache__" in event.src_path: if "__pycache__" in file_system_event.src_path:
return return
# 简单的防抖动 # 简单的防抖动
@@ -66,7 +68,7 @@ class PluginReloadHandler(FileSystemEventHandler):
self.last_reload_time = current_time self.last_reload_time = current_time
logger.info(f"检测到文件变更: {event.src_path}") logger.info(f"检测到文件变更: {file_system_event.src_path}")
logger.info("正在重载插件...") logger.info("正在重载插件...")
try: try:
@@ -112,13 +114,11 @@ async def main():
logger.warning(f"插件目录不存在 {plugin_path}") logger.warning(f"插件目录不存在 {plugin_path}")
try: try:
bot = WS() websocket_client = WS()
# 初始化代码执行器 # 初始化代码执行器
from core.config_loader import global_config as config code_executor = initialize_executor(websocket_client, config)
from core.utils.executor import initialize_executor websocket_client.bot.code_executor = code_executor # 将执行器实例附加到 bot.bot 对象上
code_executor = initialize_executor(bot, config)
bot.bot.code_executor = code_executor # 将执行器实例附加到 bot.bot 对象上
# 启动代码执行器的后台 worker # 启动代码执行器的后台 worker
logger.debug("[Main] 检查是否需要启动代码执行 Worker...") logger.debug("[Main] 检查是否需要启动代码执行 Worker...")
@@ -128,7 +128,7 @@ async def main():
else: else:
logger.warning("[Main] 未启动代码执行 Worker因为 Docker 客户端未初始化或连接失败。") logger.warning("[Main] 未启动代码执行 Worker因为 Docker 客户端未初始化或连接失败。")
await bot.connect() await websocket_client.connect()
finally: finally:
if observer.is_alive(): if observer.is_alive():
observer.stop() observer.stop()

View File

@@ -20,7 +20,7 @@ __plugin_meta__ = {
@matcher.command("admin", permission=MessageEvent.ADMIN) @matcher.command("admin", permission=MessageEvent.ADMIN)
async def handle_admin_command(bot: Bot, event: MessageEvent, args: list[str]): async def admin_command_handler(bot: Bot, event: MessageEvent, args: list[str]):
""" """
处理 /admin 指令 处理 /admin 指令
@@ -40,8 +40,8 @@ async def handle_admin_command(bot: Bot, event: MessageEvent, args: list[str]):
await event.reply("当前没有设置任何管理员。") await event.reply("当前没有设置任何管理员。")
return return
admin_list_str = "\n".join(str(admin_id) for admin_id in admins) admin_list_text = "\n".join(str(admin_id) for admin_id in admins)
await event.reply(f"当前管理员列表 ({len(admins)}):\n{admin_list_str}") await event.reply(f"当前管理员列表 ({len(admins)}):\n{admin_list_text}")
return return
if action in ("add", "remove"): if action in ("add", "remove"):