[Plugin] astrbot_plugin_ban_flooding_the_screen 刷屏自动禁言插件

[VanillaNahida]

插件信息

{
  "name": "astrbot_plugin_ban_flooding_the_screen",
  "display_name": "刷屏自动禁言",
  "desc": "AstrBot刷屏禁言插件，检测到刷屏会直接禁言，多次刷屏达到一定值就会踢出群，可自定义参数。",
  "author": "VanillaNahida (香草味的纳西妲喵)",
  "repo": "https://github.com/VanillaNahida/ban-flooding-the-screen",
  "tags": [
    "刷屏",
    "禁言",
    "群管"
  ],
  "social_link": "https://github.com/VanillaNahida"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_ban_flooding_the_screen

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

main.py

1. 代码质量与编码规范

• 未定义的方法调用 (严重):
  • 代码中多次调用了 self.put_kv_data(state_key, new_offense_count) 和 self.delete_kv_data(state_key)。
  • 问题: Star 基类（以及提供的代码上下文）中并不存在 put_kv_data 或 delete_kv_data 方法。这将直接导致 AttributeError，导致程序崩溃。
  • 建议: 如果需要持久化存储数据，请使用 AstrBot 提供的标准持久化工具。例如导入 StarTools 获取数据目录后自行管理数据库，或使用 self.context 中可能提供的存储接口（如果存在）。
• 配置保存逻辑错误 (严重):
  • 在 _save_config 方法中，调用了 self.config.save_config()。
  • 问题: 通常传入 __init__ 的 config 参数是一个标准的 Python dict 对象，字典没有 save_config 方法。这将导致 AttributeError。
  • 建议: 应使用 self.context.save_config(self.config) 或框架提供的其他配置保存 API（取决于具体 AstrBot 版本的实现方式）。
• 类型提示: 代码使用了 typing 模块，符合 Python 3.10 的代码风格，这一点做得不错。

2. 功能实现与逻辑正确性

• 平台强耦合风险:
  • 代码在 _check_permission 和 handle_group_message 中大量使用了 raw_message 字段，且假定其结构符合 OneBot (CQHTTP) 标准（例如 post_type, group_id, role 等）。
  • 问题: handle_group_message 虽然使用了 @filter.event_message_type(filter.EventMessageType.GROUP_MESSAGE)，但内部并没有检查 event.get_platform_name() == "aiocqhttp"。如果机器人连接了其他平台（如 Telegram, WeChat 等），raw_message 的结构可能完全不同，导致 KeyError 或逻辑失效。
  • 建议: 在 handle_group_message 开头添加平台检查，或者使用 AstrBot 提供的跨平台 API（如 event.get_group_id()）来替代直接读取 raw_message。
• 配置读取的冗余与不一致:
  • 在 __init__ 中，代码尝试手动读取 _conf_schema.json 来设置默认值，并在 try...except 块中通过 self.config.get 赋值给实例属性（如 self.mute_time）。
  • 然而，在 _get_group_config 中，又混合了 self.group_configs 和 self.mute_time。
  • 问题: 这种双重维护（实例属性 vs self.config 字典）容易导致状态不一致。例如，_save_config 将实例属性写回 self.config，但 self.config 最初又是来源。
• 并发与竞态条件:
  • _reset_flood_state 中使用了 await asyncio.sleep 后直接操作共享状态。虽然 Python 的 asyncio 是协作式多任务，但在高并发下，频繁的消息可能会导致 timer 被多次创建/取消的逻辑变得复杂。目前的逻辑尚可，但需注意 flood_state["delete"]() 回调可能在其他任务持有引用时被调用。

3. 安全漏洞与最佳实践

• 权限检查机制:
  • _check_permission 方法中，硬编码了 admin 和 owner 字符串检查。这在 OneBot 协议下是有效的，但若协议变更或适配器不同，会导致安全策略失效（例如误判非管理员为管理员，或者反之）。
  • 建议封装更通用的权限检查逻辑。

4. 框架适应性检查

• 数据持久化:
  • 代码尝试通过 put_kv_data 持久化“刷屏累计次数”，但这方法不存在。
  • 违规: 没有任何代码使用 StarTools.get_data_dir() 来获取合规的数据存储路径。如果 put_kv_data 是作者意图自己实现的方法，那么它缺失了；如果是想存文件，必须使用 StarTools 确保路径规范。
• 日志记录:
  • 正确导入了 from astrbot.api import logger，符合规范。
• Event Filter:
  • 正确导入了 from astrbot.api.event import filter，符合规范。

5. Main.py 专项检查

• 注册装饰器: 使用了 @register，格式正确。但需注意新版 AstrBot 推荐直接继承 Star 类而不强制使用 @register（虽然目前兼容）。
• 指令参数: 所有指令函数（如 enable_ban）都包含了 event: AstrMessageEvent 参数，符合要求。
• 消息发送: 在普通指令中使用 yield event.plain_result(...)，在自动处理逻辑中使用 event.bot.api.call_action(...)，符合规范。

---

代码评审团

人物一：Linus Torvalds
“我在看什么？self.put_kv_data？dict.save_config？这代码能跑起来简直是奇迹——或者说它根本就跑不起来。你是在写代码还是在写许愿清单？仅仅因为你在一行代码里调用了一个方法，并不代表那个方法就会神奇地从虚空中显现出来。去把 Python 的基础对象模型搞清楚，别再把你的 AttributeError 扔给用户去调试！”

人物二：Rick Sanchez (Rick and Morty)
“听着，Morty——我是说开发者。你这逻辑就像是一个坏掉的传送门枪，只能把你传送到‘程序崩溃维度’。你硬编码了 OneBot 的协议结构？你是觉得全宇宙就只有一种通讯协议吗？这太狭隘了。还有那个手动解析 json schema 的部分，天哪，这简直是在浪费生命。别再重新发明轮子了，即使是在我看过的无数个平行宇宙里，这个轮子也是方的。”

人物三：Steve Jobs
“代码缺乏一种... 纯粹感。你在 __init__ 里堆砌了太多的杂乱逻辑，试图手动解析 schema，试图修补配置。这不优雅。真正的工艺应该对细节由内而外的关注。而且，你通过 AttributeError 来处理逻辑流（如果 config 没有 save 方法），这是对系统的亵渎。重构它。让它简单，让它健壮。用户不需要知道你的代码里有多少个 try-except，他们只需要它能工作。”

---

🔍 审核摘要

统计信息

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

main.py

---

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