[Plugin] astrbot_plugin_DUT_Notices

[Lentinel]

插件信息

{
  "name": "astrbot_plugin_DUT_Notices",
  "display_name": "大连理工大学通知新闻推送",
  "desc": "聚合爬取大连理工大学多个通知站点（以开发区校区为主）最新内容，向订阅会话推送最新通知。",
  "author": "Lentinel",
  "repo": "https://github.com/Lentinel/astrbot_plugin_DUT_Notices",
  "social_link": "https://github.com/Lentinel"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_DUT_Notices

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

main.py

1. [高] _resolve_source_from_event 存在未定义变量风险

   • 代码中日志使用了 SOURCES：
     logger.info(... {[s['key'] for s in SOURCES]})
   • 但 main.py 未导入 SOURCES，当 resolve_source(text) 返回 None 时会触发 NameError，导致本应返回“参数错误”的流程中断。

2. [高] 指定来源相关指令参数解析逻辑实际不可用

   • subscribe_source / unsubscribe_source / latest_source 都调用：
     extract_command_args(event, "")
   • 传入空 command_name 与 command_utils.py 当前实现组合后，会导致几乎恒返回空字符串，最终 source is None，这些命令基本无法正确解析来源参数。

3. [高] “增量推送”与实现不一致，存在重复推送/刷屏风险

   • _run_check(push=True) 每次轮询都会对 fetch_notices() 结果全部执行 _push_new_items()。
   • 没有“已推送 ID”持久化去重状态，重启或定时轮询会反复推送同一批历史通知。

4. [中] 轮询任务可能重复启动

   • initialize() 直接 create_task，未检查已有任务是否存活。
   • 若插件被重复初始化（热重载/异常恢复场景），可能出现并行多轮询任务，导致重复抓取和重复推送。

5. [中] self.config 的类型与调用方式不一致

   • __init__ 标注 config: dict[str, Any] | None，且默认 self.config = {}。
   • 但管理员命令中直接调用 self.config.save_config()，若运行时传入的是普通 dict 会抛 AttributeError。

6. [低] 动态模块加载方式增加维护风险

   • _load_local_module 通过裸模块名写入 sys.modules（如 sources, rss_service）。
   • 易与同名模块冲突，且热重载/测试环境下调试复杂度较高。

---

command_utils.py

1. [高] extract_command_args 对空命令名缺乏防御，导致解析异常

   • 当 command_name == "" 时，prefixes 含空字符串，后续逻辑会退化，最终难以返回预期参数。
   • 该问题已在 main.py 中实际触发，导致来源解析命令不可用。

2. [中] 参数提取策略存在误判风险

   • 分支：
     if "dlut" not in lowered and command_name not in lowered: return normalized
   • 对非严格命令输入可能直接把整句当参数，命令边界不清晰，容易误解析到非目标文本。

---

parsers.py

未发现需报告的问题。

---

rss_service.py

1. [高] 数据持久化路径未按规范使用 StarTools.get_data_dir()

   • 当前通过 astrbot.core.utils.astrbot_path.get_astrbot_data_path() 或硬编码 Path("data")/... 组装目录。
   • 按给定审查规范，插件持久化目录应通过 StarTools.get_data_dir() 获取。

2. [高] RSS 写入前未确保目录存在

   • tree.write(self.rss_file_path, ...) 前未 mkdir(parents=True, exist_ok=True)。
   • 在首次运行或目录缺失场景会直接写入失败。

3. [中] 日期正则对中文日期兼容性存在缺陷

   • DATE_PATTERN 使用了 年 、月 （带空格）的模式：
     (?:年 |[-/.])、(?:月 |[-/.])
   • 常见格式如 2024年3月1日（无空格）可能无法匹配，导致发布日期提取失败并回落到“当前时间”，影响排序和去重稳定性。

4. [低] 并发抓取缺少上限控制

   • asyncio.gather(*all_sources) 对全部来源并发请求，没有信号量限流。
   • 来源数量增加时，可能造成瞬时连接数过高，提升网络抖动与失败概率。

5. [低] 存在未使用代码，增加维护负担

   • _collect_sibling_text、_node_text、_request_headers 的 source 参数当前未被有效使用。

---

sources.py

1. [中] 部分抓取源使用明文 HTTP，存在链路篡改风险
   • cgbmis_* 相关 url/base_url 使用 http://。
   • 在不可信网络环境下，抓取内容可被中间人篡改，影响推送可信度。

---

subscription_store.py

未发现需报告的问题。

---

代码评审团

人物一：Linus Torvalds
“你这套异步框架搭得不差，但 增量推送 没有状态去重这事儿很致命——系统会诚实地重复干蠢事。先把‘是否已推送’做成明确、可持久化的状态机，再谈优雅。”

人物二：Steve Jobs
“用户体验里最糟糕的一种是：命令看起来存在，但永远解析不到参数。subscribe_source 这类命令就是这样。产品层面这叫‘功能幻觉’，修复参数解析是第一优先级。”

人物三：Tony Stark（钢铁侠）
“我喜欢自动化，但不喜欢‘自动炸自己’。轮询任务能重复启动、路径又没保证存在，这种组合在生产上很容易制造‘半夜告警派对’。加任务幂等保护 + 写入前建目录，马上能降噪。”

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

---

🔍 审核摘要

统计信息

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

main.py
command_utils.py
parsers.py
rss_service.py
sources.py
subscription_store.py

---

此报告由AI自动生成，旨在提供初步反馈和改进建议，不能完全替代人工审核。最终决策以社区维护者的人工审核为准。目前自动审核（仓库地址）处于试验阶段，如遇问题请向维护者反馈。评论 @astrpluginreviewer /review 来重新触发 review，请不要重复提交插件发布 issue。

当前仓库：Lentinel/astrbot_plugin_DUT_Notices
剩余触发次数：4

[astrbot-plugin-copybara]

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

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