[Plugin] astrbot_plugin_unturned_servers

[Emqo]

插件信息

{
    "name": "astrbot_plugin_unturned_servers",
    "display_name": "Unturned 服务器查询",
    "desc": "查询 Unturned 游戏服务器实时状态和在线玩家信息，支持 /status 和 /players 命令",
    "author": "Bugelll",
    "repo": "https://github.com/Bugelll/astrbot_plugin_unturned_servers",
    "tags": ["游戏", "Unturned", "服务器查询"]
  }

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_unturned_servers

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

main.py

代码审查报告

从整体来看，这份代码结构清晰，严格遵守了 AstrBot 的规范要求，异步编程的实践也相当规范，没有发现阻塞性操作或违规导入。在安全性和功能正确性上表现良好。以下主要指出一些可优化的细节和潜在的逻辑瑕疵：

1. 冗余的异常处理逻辑

在 _query_server_info 和 _query_server_players 方法中，你已经使用了 try...except Exception as e: 捕获了异常，并且在失败时返回了 None。
因此，在随后的 asyncio.gather(*tasks, return_exceptions=True) 中，除非发生 BaseException（如 asyncio.CancelledError），否则正常的异常已经被内部消化掉了，返回给 gather 的结果只会是 None 或正常数据。
问题：后面的 if isinstance(result, Exception) or result is None: 判断中，isinstance(result, Exception) 基本是“死代码（Dead Code）”，虽不致命但略显冗余。

2. 并发查询（UDP 突发）潜在问题

在 players 指令中，你把所有的 info_tasks 和 player_tasks 扁平化合并到了同一个 asyncio.gather 中并发执行：

all_results = await asyncio.gather(*all_tasks, return_exceptions=True)

潜在风险：对于同一个服务器，这意味着它会在同一瞬间收到两个 UDP 查询包（Info 和 Players）。部分游戏服务器的网络栈或防火墙可能会对瞬间来自同一源 IP 的多个 UDP 探针限流或丢包，从而增加超时概率。
建议：对于单个服务器，建议编写一个专门的协程封装查询，例如先查 Info，如果在线再查 Players，然后再聚合成单个服务器的状态字典返回。这样既减少了无意义的 Players 查询（如果服务器离线的话），也避免了并发 UDP 拥塞。

3. 配置文件解析的硬编码清理

raw = raw.replace("\\n", "\n")

这种清理方式略显粗暴，主要是因为配置输入可能是转义后的字符串。虽然能工作，但如果玩家的服务器名字里真的包含 \n 这个子串（虽然很少见），会被误伤。不过对于文本解析场景，属于可接受的妥协。

---

代码评审团

Linus Torvalds：（粗暴直率，关注底层逻辑和并发）

“看看你在 players 里的网络请求！你把 ainfo 和 aplayers 打包扔进同一个 gather 里直接并发？你是在做网络压力测试吗？对于同一个土豆游戏服务器，瞬间塞过去两个 UDP 查询，它的包过滤机制可能直接把你的请求当成垃圾丢掉！你为什么不写一个 wrapper 协程，先查 info，如果服务器没挂掉再去查 players？还有，你内层函数都 catch Exception 并返回 None 了，外层还要 isinstance(info, Exception)？你是在指望系统给你抛出一个不在 Exception 继承树里的外星异常吗？Clean up your logic!”

王家卫：（文艺、意识流，关注数据背后的情感与时间）

“每一个服务器都有属于它的 IP 和端口，就像 123.45.67.89,27015。在这个宇宙里，你等了 5 秒，有时候得到的只是一个 None。代码里冰冷地写着 🔴 离线（连接超时），其实不过是它在某个瞬间不想回应你罢了。你用 divmod(total, 3600) 去丈量玩家停留的痕迹，却忘了代码里那些 < 0 时返回的『未知』，也许才是游戏世界里最漫长、最深刻的孤独。”

Gordon Ramsay：（暴躁老哥，关注代码“食材”的处理与规范）

“What is this mess?! raw = raw.replace("\\n", "\n")?! 这就像是你在用微波炉去解冻一块上等的和牛！你期望用户规规矩矩地输入多行配置，结果你还得像个保姆一样去手动清理那些该死的转义字符！还有看看你提取端口的地方 name = f"{host}:{port_str}"，这简直是临时起意的凑合！代码逻辑没有大错，但这代码不够精致，它不是全熟，IT'S RAW! 给我重新把逻辑切分好，别把什么东西都往一个盘子里装！”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

当前仓库：Bugelll/astrbot_plugin_unturned_servers
剩余触发次数：4

[astrbot-plugin-copybara]

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

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