[Plugin] astrbot_plugin_shit_detector_plus

[Sisyphbaous-DT-Project]

插件信息

{
  "name": "astrbot_plugin_shit_detector_plus",
  "display_name": "GitHub 搬史专家",
  "desc": "如果你觉得你的 coding plan 额度用不完，不妨试着让 AI 给你自动搬史。一个自动搜索 GitHub 项目，使用 AI 进行\"史\"级评定的趣味插件，支持定时推送精选项目到群聊",
  "author": "C₂₂H₂₅NO₆",
  "repo": "https://github.com/Sisyphbaous-DT-Project/astrbot_plugin_shit_detector_plus",
  "tags": [
    "搬史"
  ],
  "social_link": "https://github.com/Sisyphbaous-DT-Project"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_shit_detector_plus

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

以下是对提供的各个文件进行的综合代码审查报告。对于未列出的文件，均视为审查通过，未发现明显违规或显著的质量缺陷。

文件路径 core/data_store.py

• 数据持久化违规 (框架适应性):
  在 ShitDataStore.__init__ 方法中，处理默认的数据存放路径时，使用了硬编码的路径 Path("data")，并且试图通过不存在的 context.get_data_dir() 获取目录。
  建议: 必须从 astrbot.api.star 导入 StarTools，并调用 StarTools.get_data_dir() 获取标准的插件数据存储目录（注意其返回的是一个 Path 对象），从而避免硬编码路径。

  from astrbot.api.star import StarTools
  
  # 建议的路径获取方式
  self.data_path = StarTools.get_data_dir() / self.DEFAULT_FILENAME

• 潜在的异步阻塞 (可维护性/缺陷):
  在 _load 与 _init_empty_data_sync 方法中，使用了同步的 open(...) 和 json.load/dump 来处理文件，这在初始化阶段可以接受。但在运行时可能被高频触发的地方，如果数据文件变得庞大，同步 I/O 会略微拖慢事件循环，尽管目前使用 run_in_executor 封装了 _save 操作，这是个不错的妥协。

文件路径 utils/repo_downloader.py

• 数据持久化违规 (框架适应性):
  在 RepoDownloader.__init__ 中，定义了硬编码的相对路径 base_dir: str = "data/shit_detector_repos"。由于这是插件私有的临时克隆目录，应当被包含在插件的规范数据持久化目录下。
  建议: 同样改用 StarTools.get_data_dir() / "repos" 或类似的逻辑作为默认根目录。

• 潜在的安全隐患 (安全漏洞与最佳实践):
  在 download 方法里通过 asyncio.create_subprocess_exec 调用 git clone，参数 --branch 的值 branch 是通过字符串直接传入的。由于 create_subprocess_exec 不通过 shell，命令注入的风险较小（例如无法使用 && 拼接多条命令），但如果用户输入的 branch 以 - 开头，可能会导致未预期的 Git 选项被执行（Argument Injection）。
  建议: 使用正则表达式验证 branch 名称，确保它只包含字母、数字、.、_ 等安全字符，或加上 -- 分隔符避免被解析为参数。

文件路径 scheduler/delivery_scheduler.py

• 代码质量与编码规范:
  在 _generate_message 方法内进行了局部延迟导入 from ..utils.repo_downloader import RepoDownloader。虽然这可能是为了规避循环导入，但此方法是基于定时调度高频执行的。
  建议: 将依赖注入到类的初始化过程中（类似于已经传入的 analyzer），以降低耦合度并提高代码执行效率。

• 潜在缺陷:
  在 get_status 函数中，调用了 job = self.scheduler.get_job(self.job_id)，但如果没有配置或找到这个 Job，job 将为 None。紧接着 str(job.next_run_time) if job else None 虽做了保护，但在高并发异步锁状态变迁期间要多留意对 self.scheduler 生命周期的空判断。

文件路径 core/ai_analyzer.py

• 复杂代码块 (可维护性):
  _parse_analyst_response 方法中为了处理大语言模型返回的不同格式 JSON 结果，堆叠了过多的 try-except 与字符串查找逻辑（如正则、查找花括号、剔除 BOM 头等），这导致了圈复杂度（Cyclomatic Complexity）偏高。
  建议: 将解析大语言模型 JSON 格式字符串的方法剥离成一个单独的独立工具函数，以改善 AICodeAnalyzer 的阅读体验并使其更易于编写单元测试。

---

代码评审团

人物一（Linus Torvalds）：
“你以为在 core/data_store.py 里面写一行 Path("data") 框架就会夸你聪明吗？文档里明明白白写着要用 StarTools.get_data_dir()！你们这些把持久化文件像随地大小便一样乱丢的人就是导致代码库像个垃圾场的罪魁祸首。还有那个正则表达式解析 GitHub URL 却不带完整性校验的逻辑，我都懒得开喷了，赶紧去把这些烂摊子修好！”

人物二（Gordon Ramsay）：
“看看这个 RepoDownloader！你在初始化参数里硬塞一个默认路径进去，简直就像在惠灵顿牛排里夹生面团一样恶心！为什么不能乖乖遵循框架的食谱？不过，我得承认，你懂用 aiohttp 和 asyncio.create_subprocess_exec 来处理网络并发与子进程，这让我没有直接把盘子砸在你脸上。味道有点奇怪，但至少厨房没有被你烧掉。”

人物三（乔布斯）：
“伟大的产品在于将细节做到极致。你的双 LLM 流水线架构非常性感，它确实在重新定义我们如何挖掘代码的趣味。但是，你在 delivery_scheduler.py 里面的局部引用破坏了这件艺术品的连贯性。我们不需要一个在运行时到处找扳手的设计。把你的工具（RepoDownloader）在一开始就优雅地组装好，让这个‘搬史’过程像魔术一样自然发生。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
commands/__init__.py
core/__init__.py
core/ai_analyzer.py
core/constants.py
core/data_store.py
core/github_searcher.py
scheduler/__init__.py
scheduler/delivery_scheduler.py
scheduler/search_scheduler.py
utils/__init__.py
utils/github_client.py
utils/helpers.py
utils/repo_downloader.py

---

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

当前仓库：Sisyphbaous-DT-Project/astrbot_plugin_shit_detector_plus
剩余触发次数：4

[Sisyphbaous-DT-Project]

已严肃修改（）@astrpluginreviewer

[astrbot-plugin-copybara]

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

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