[Plugin] astrbot_plugin_atrifeed

[Heximiao]

插件信息

{
  "name": "astrbot_plugin_atrifeed",
  "display_name": "亚托莉养成（atri)",
  "desc": "「我可是高性能的！」亚托莉养成计划。内置好感度羁绊、动态数据看板、投喂反馈及表情包互动，还有剧情系统。请小心呵护，不然亚托莉也是会生气的哦！",
  "author": "何夕",
  "repo": "https://github.com/Heximiao/astrbot_plugin_atrifeed",
  "tags": [
    "亚托莉",
    "atri",
    "娱乐",
    "gal"
  ],
  "social_link": "https://github.com/Heximiao"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_atrifeed

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

main.py

1. 数据持久化目录未按规范使用 StarTools.get_data_dir()

   • 当前通过 get_astrbot_plugin_data_path()/get_astrbot_data_path() 拼接目录，而审查规范要求持久化目录统一走 from astrbot.api.star import StarTools + StarTools.get_data_dir()。
   • 这会导致插件数据目录策略不一致，增加迁移与维护成本。

2. 关键词命令分支存在不可达逻辑

   • on_keyword_msg 中：

     if is_potential_cmd:
         return
     if route is None:
         route = self._keyword_router.match_command_route(message_str)

   • 对命令前缀消息（/ ! ！）会提前 return，match_command_route 永远不会执行，导致“命令式关键词路由”失效。

3. 异常处理过宽且可能掩盖真实错误

   • 通过 except TypeError 来区分“异步生成器/协程”：

     async for result in handler(event):
     ...
     except TypeError:
         result = await handler(event)

   • 若 handler 内部真实抛出 TypeError（业务 bug），会被误判成调用方式问题并吞掉定位线索。

4. 导入与依赖管理混乱，降低可维护性

   • 存在多处未使用导入（如 random, Path, datetime, Comp, register, AtriDB, AtriShopDB 等）。
   • from .src.command.feeding import * 使用通配符导入，不利于静态分析、命名冲突排查与代码阅读。

5. 重复调用拉黑检查

   • on_at_abuse_monitor 中在入口和 if is_at_me... 内重复 self.is_blocked(event)，存在冗余判断。

---

src/ban.py

1. 边界条件缺失：event.message_str 可能为 None

   • is_apology_msg = "亚托莉我错了对不起" in message_str
   • 在 ALL 事件下若消息不是文本，message_str 为 None 时会抛 TypeError。

2. 未使用导入

   • import random 未使用。

---

src/command/abuse.py

1. 数据目录使用不规范（持久化风险）

   • 通过 plugin_root/data 硬编码路径初始化 AtriBayesFilter：

     data_path = os.path.join(plugin_root, "data")

   • 若该过滤器有持久化行为，应改为 StarTools.get_data_dir() 提供的标准插件数据目录。

2. 边界条件缺失：ALL 事件下可能空消息

   • clean_msg = event.message_str.strip() 若 message_str is None 会异常。

3. 重复定义敏感词列表，维护成本高

   • BAD_WORDS 与 HARD_WORDS 内容重复，且 check_abuse 未被调用，形成死代码+双源配置问题。

---

src/command/backpack.py

1. 未使用导入
   • from astrbot.api import logger 未使用。

---

src/command/dice.py

1. 未使用变量/参数

   • economy 变量未使用。
   • curr_dir 参数未使用。

2. 直接依赖 DB 私有接口

   • 使用 db._get_conn()、db._format_gid()，破坏封装，后续 DB 实现调整时风险高。

---

src/command/feeding.py

1. 可变默认参数

   • get_pic_path(curr_dir: str, folder_types: list = ["emoji"])
   • Python 3.10 下依然是经典陷阱，虽当前未原地修改，但属于高风险坏味道。

2. 边界条件缺失

   • run_feed_fruit_logic 中 msg = event.message_str，随后 if emoji in msg；
   • 在非文本或空消息场景可能触发 TypeError。

3. 重复代码明显

   • get_pic_path 与 yield_random_folder_pic 的 path_map 与遍历逻辑基本重复，维护时容易出现一处改动另一处遗漏。

---

src/command/gig.py

1. 群 ID 获取方式不稳健

   • 使用 event.message_obj.group_id，而非统一接口 event.get_group_id()；
   • 在私聊或不同适配器下可能触发属性不存在/空值异常。

2. 冷却时间计算存在逻辑错误

   • 每次判定都重新 random.randint(5, 8)，会导致同一用户同一轮冷却时长动态变化，表现为“剩余时间忽长忽短”。

3. 未使用导入

   • get_pic_path 被导入但未使用。

4. 异常捕获后丢失调试信息

   • except Exception as e: 中未记录 e，排障困难。

---

src/command/help.py

1. 日志模块使用违反框架约束（严重）
   • 使用了：

     import logging
     logger = logging.getLogger("astrbot")

   • 规范要求 logger 必须且只能来自 from astrbot.api import logger。

---

src/command/my_atri.py

1. 直接依赖 DB 私有接口

   • 使用 db._format_gid、db._get_conn，封装泄漏，影响可维护性。

2. 未使用变量

   • is_blocked、mood_code 赋值后未使用。

3. 异常处理后未反馈用户

   • 渲染失败时仅 logger.error，用户侧无明确失败提示，造成“无响应”体验。

---

src/command/other_emoji.py

1. 资源存在性未校验
   • Comp.Image.fromFileSystem(image_path) 前未检查文件是否存在，资源缺失时可能导致运行时错误。

---

src/command/radish.py

1. 函数内导入
   • import random 放在函数内且非条件依赖，影响代码一致性与可读性（应模块级导入）。

---

代码评审团

人物一：Linus Torvalds（毒舌务实流）
“except TypeError 去判断调用方式？这像是拿锤子修 CPU。你把真正的业务错误也吞了。建议：明确区分 handler 类型（inspect.isasyncgenfunction / 统一包装接口），别靠异常猜测流程。”

人物二：Steve Jobs（产品体验偏执流）
“用户点了功能却没反馈，是最糟糕的体验。比如渲染失败只打日志不回消息，这会让人怀疑机器人卡死。建议：所有失败路径都给用户稳定、简洁、可行动的提示。”

人物三：Tony Stark（工程化效率流）
“你这重复代码有点像手搓 Mark I 战甲：能飞，但不好维护。path_map 重复、敏感词双份、私有 DB 接口直连。建议：抽公共函数、合并配置源、通过 DB 公共方法访问数据，后期升级会轻松很多。”

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

---

🔍 审核摘要

统计信息

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

main.py
keyword_trigger.py
src/__init__.py
src/ban.py
src/constants.py
src/command/__init__.py
src/command/abuse.py
src/command/backpack.py
src/command/dice.py
src/command/feeding.py
src/command/gig.py
src/command/help.py
src/command/my_atri.py
src/command/other_emoji.py
src/command/radish.py

注意：由于项目规模或Token限制，本次仅审核了部分优先文件。以上报告内容由 AI 直接生成。

---

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

当前仓库：Heximiao/astrbot_plugin_atrifeed
剩余触发次数：4