[Plugin] astrbot_plugin_sl_query

[DNTOF]

插件信息

{
  "name": "astrbot_plugin_sl_query",
  "display_name": "SCP:SL 服务器查询插件",
  "desc": "通过 ServerKey 查询 SCP:SL 服务器信息",
  "author": "DNT_OF",
  "repo": "https://github.com/DNTOF/astrbot_plugin_sl_query",
  "tags": [
    "api"
  ],
  "social_link": "https://github.com/DNTOF"
}

插件检查清单

• 我的插件经过完整的测试
• 我的插件不包含恶意代码
• 我已阅读并同意遵守该项目的 行为准则。

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_sl_query

您好！我已经对你提交的插件代码进行了初步自动化审核，作为初步参考:

main.py

1. 潜在缺陷与最佳实践：资源生命周期管理 (__del__ 方法)

• 问题描述：在 SLQueryPlugin 类中使用了 __del__ 魔法方法来关闭 SQLite 数据库连接 (self.conn.close())。在 Python 中，依赖 __del__ 进行关键资源的释放是极不推荐的，因为垃圾回收（GC）何时调用 __del__ 是不确定的，甚至在解释器退出时可能会引发复杂的异常（例如全局变量已被销毁）。
• 建议：建议直接在需要操作数据库的方法内按需创建和关闭连接（使用 with sqlite3.connect(...) ），或提供一个明确的生命周期清理方法，避免强依赖不可靠的 __del__ 机制。

2. 潜在缺陷：多协程并发下的 SQLite 长连接风险

• 问题描述：在 _init_db 中，插件初始化了一个长期的 SQLite 连接 self.conn，并开启了 check_same_thread=False。虽然框架免去了文件 I/O 阻塞检查，但在高并发的异步环境中，多个协程同时通过这个单一的长连接对数据库进行写入或删除（如 _add_or_update_server）时，极易触发 SQLite 的 database is locked 错误。
• 建议：采用 aiosqlite 库以适配异步环境的数据库操作，或者每次操作时申请短暂的新连接。

3. 性能与最佳实践：aiohttp.ClientSession 未复用

• 问题描述：在 _fetch_server_info 方法中，每次调用都会通过 async with aiohttp.ClientSession() as session: 创建和销毁一个新的 Session。这种模式无法利用 HTTP Keep-Alive 和底层的连接池，增加了建立 TCP/TLS 连接的网络延迟和系统开销。
• 建议：在类的初始化阶段（或第一次发起请求时）创建一个全局的 ClientSession 并复用它。

4. 功能实现与逻辑正确性：指令参数解析存在缺陷

• 问题描述：在 cmd_bind 命令的参数解析逻辑中，使用了 for part in params_str.split(): 强行根据空格进行切分。这会导致如果用户传入的 name 包含空格（例如 /bind id=123 key=abc name=My Server），参数会被错误切割为 ['id=123', 'key=abc', 'name=My', 'Server']，使得最终绑定的名字被截断为 My，丢失了 Server 部分。
• 建议：修改参数解析逻辑，可使用正则表达式提取 key=value，或支持带引号包裹的字符串解析，以提升用户体验与程序鲁棒性。

5. 代码质量与编码规范：缺少合理的空行

• 问题描述：文件中的类方法之间（如 _get_all_servers, _get_server, _add_or_update_server 等）没有空行分隔，代码结构紧凑，阅读体验较差，违反了 PEP 8 中关于“在类方法之间保留一个空行”的建议。

---

代码评审团

Linus Torvalds：你居然在 Python 里用 __del__ 来管理数据库连接的生命周期？这是在写 C++ 吗？！Python 的垃圾回收器调用它的时机根本不可预测，随时可能让你的连接变成僵尸，或者在解释器退出时甩你一脸错误。另外，在异步框架里直接持有单例的 sqlite3 长连接，你最好祈祷并发高的时候它别给你报 "database is locked" 的错。

Steve Jobs：这代码提供的体验简直不完美！看看你的 cmd_bind，居然直接用 .split() 粗暴地切分参数。如果用户的服务器名字叫 "Main Server"，你的代码就把它硬生生切成了 "Main"，把 "Server" 像垃圾一样扔掉。我们要的是符合直觉的完美交互，你需要一套更聪明的解析器，哪怕是正则表达式或者支持引号包裹，也比这种随意拼凑的逻辑好一万倍。

Rick Sanchez：听着，莫蒂，这家伙在 _fetch_server_info 里干了什么？他每次发请求都要新建一个 aiohttp.ClientSession()！这就好像你为了喝一口水，每次都要凭空创造一个全新的微型宇宙，喝完再把它彻底毁灭！建立连接池是有意义的，懂吗？你应该把 Session 放在实例里复用，别再做毫无意义的 CPU 循环浪费了，我的天哪。

(Disclaimer: 以上评审内容由 AI 自动生成，所涉及人物形象与现实无关，不代表真实人物观点。如果给出的建议无关痛痒请忽略。)

---

🔍 审核摘要

统计信息

• 仓库文件总数: 1 个 Python 文件
• 已选择审核文件: 1 / 1
  已发送至 AI 审核的文件清单

main.py

---

此报告由AI自动生成，旨在提供初步反馈和改进建议，不能完全替代人工审核。最终决策以社区维护者的人工审核为准。目前自动审核（仓库地址）处于试验阶段，如遇问题请向维护者反馈。

[astrbot-plugin-copybara]

插件信息已同步到集合仓库。
PR: AstrBotDevs/AstrBot_Plugins_Collection#287
已自动合并 PR #287

该插件已上架。
This plugin is now published.