[Plugin] astrbot_plugin_ctfpusher

[btop251]

插件信息

{
  "name": "astrbot_ctfpusher_plugin",
  "display_name": "CTF 赛题及赛事推送助手",
  "desc": "基于 CTFTime 与 NSSCTF 的全功能赛题推送、赛事检索、过滤与订阅提醒插件，支持精细化管理和北京时间自动转换。",
  "author": "btop251",
  "repo": "https://github.com/btop251/astrbot_ctfpush_plugin",
  "version": "3.2.0",
  "social_link": "btop251.top"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_ctfpusher_plugin

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

main.py

1. 数据持久化路径不规范

   • 代码将数据库路径硬编码为 os.path.join(base_dir, "data", "data.db")。
   • 按框架规范，持久化数据应通过 StarTools.get_data_dir() 获取插件数据目录（Path 对象）再拼接文件名。

2. 功能逻辑与命名不一致（NSSCTF 拉取为“模拟数据”）

   • fetch_nssctf_problem() 中声明“从 NSSCTF 获取题目”，但实际未发起网络请求，返回的是随机 mock 数据。
   • api_url、params 定义后未使用，容易误导维护者和用户对功能真实性的预期。

3. 代码质量问题（未使用导入）

   • aiohttp、json 在本文件中未被使用，增加噪音并降低可维护性。

---

ctf_plugin/main.py

1. 不满足 main.py 结构专项要求
   • 该文件中没有继承 Star 的主插件类，也没有 @register(...) 注册入口。
   • 也没有 from astrbot.api.event import filter 及任何 @filter 事件监听器。
   • 按你给定的 main.py 检查规则，这会被判定为结构不合规（即使文件注释说明是“兼容占位模块”）。

---

ctf_plugin/init.py

• 未发现需要报告的问题。

---

ctf_plugin/config_manager.py

1. 日志组件不符合框架要求
   • 使用了 import logging + logging.getLogger(__name__)。
   • 规则要求 logger 必须且只能来自 from astrbot.api import logger。

---

ctf_plugin/utils.py

1. 发送者识别存在潜在逻辑错误
   • extract_sender() 的候选字段包含 get_self_id，在缺少用户字段时可能回退到“机器人自身ID”，导致订阅绑定到错误主体（错误用户/错误会话）。

---

ctf_plugin/data_sources/init.py

• 未发现需要报告的问题。

---

ctf_plugin/data_sources/aggregator.py

1. 日志组件不符合框架要求
   • 使用了 import logging + logging.getLogger(__name__)，不符合必须使用 astrbot.api.logger 的约束。

---

ctf_plugin/data_sources/base.py

• 未发现需要报告的问题。

---

ctf_plugin/data_sources/ctftime.py

1. 日志组件不符合框架要求

   • 使用了 import logging + logging.getLogger(__name__)，违反日志规范。

2. 数据清洗鲁棒性不足，单条脏数据可影响整批结果

   • _normalize_event_data() 中 float(raw_data.get("weight", 0.0) or 0.0) 若遇到非数值字符串会抛异常。
   • 上层在 for item in payload 外层统一 try，一旦某条异常会中断后续全部赛事清洗，导致结果不完整。

---

ctf_plugin/db/models.py

• 未发现需要报告的问题。

---

ctf_plugin/db/repository.py

1. 退订逻辑存在误删风险

   • remove_subscription() 使用 like(f"%{event_id}") 按后缀模糊匹配 event_id。
   • 可能删除非目标赛事（如退订 12 时误命中 112、9912），属于明显逻辑缺陷。

2. 订阅唯一键构造缺乏输入校验

   • unique_id = f"{source}:{id}" 在 source/id 缺失时可能产生 ":" 这类无效键，带来数据污染与冲突风险。

---

ctf_plugin/db/sqlite.py

1. scoped_session 清理不完整，存在会话泄漏风险
   • session_scope() 中仅 session.close()，未调用 scoped_session.remove()。
   • 在线程池/异步转线程场景下，线程局部会话注册可能残留，长期运行可能引发连接与会话管理问题。

---

ctf_plugin/services/subscription.py

1. 消息发送链路缺少关键空值保护
   • _send_to_subscriber() 直接使用 self.context.get_platform_manager().get_default_provider()，未检查 provider 是否为空。
   • 在平台未就绪或 provider 失效时会抛异常，影响提醒任务稳定性。

---

代码评审团

人物一：Linus Torvalds（直球毒舌流）
你这日志体系是“多国部队”——有的文件用 astrbot.api.logger，有的文件还在 logging.getLogger。这不是风格问题，是维护灾难。先统一日志入口，不然你连问题在哪一层发生都难快速定位。

人物二：Steve Jobs（产品体验流）
fetch_nssctf_problem() 对用户说“正在拉取”，实际在本地随机生成。用户体验上这叫“承诺与现实断层”。要么明确写“演示模式”，要么接真 API；产品最怕“看起来能用，但不是它说的那样”。

人物三：Tony Stark（工程系统流）
remove_subscription() 的 %{event_id} 模糊删法有点“激光炮打蚊子”——快是快，但容易把不该删的也轰掉。改成精确匹配唯一键，外加参数校验；另外 scoped_session.remove() 也补上，不然你的后台任务迟早会被“隐形资源泄漏”反噬。

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

---

🔍 审核摘要

统计信息

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

main.py
ctf_plugin/main.py
ctf_plugin/__init__.py
ctf_plugin/config_manager.py
ctf_plugin/utils.py
ctf_plugin/data_sources/__init__.py
ctf_plugin/data_sources/aggregator.py
ctf_plugin/data_sources/base.py
ctf_plugin/data_sources/ctftime.py
ctf_plugin/db/models.py
ctf_plugin/db/repository.py
ctf_plugin/db/sqlite.py
ctf_plugin/services/subscription.py

---

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

当前仓库：btop251/astrbot_ctfpush_plugin
剩余触发次数：3

[astrbot-plugin-copybara]

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

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