[Plugin] astrbot_plugin_bangumi_enhance

[united-pooh]

插件信息

{
  "name": "astrbot_plugin_bangumi_enhance",
  "display_name": "Bangumi Enhance",
  "desc": "AstrBot Bangumi 增强版：为 AstrBot 打造的一站式 Bangumi 追番助手。支持番剧/漫画图文搜索、每日放送时刻表查看及集数更新自动提醒。",
  "author": "united_pooh",
  "repo": "https://github.com/united-pooh/astrbot_plugin_bangumi",
  "tags": [
    "动漫",
    "番剧",
    "bangumi",
    "追番"
  ],
  "social_link": "https://github.com/united-pooh"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_bangumi_enhance

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

main.py

1. 插件数据目录规范风险
   在 initialize() 中通过 get_astrbot_data_path() 构造数据路径并传给 EnvManager，未体现使用 StarTools.get_data_dir() 的插件级数据目录规范。若 EnvManager 涉及持久化，可能导致目录不一致或跨插件污染风险。

2. 调试指令存在副作用
   bgm_debug 中固定执行 self.storage.update_subject_episode("525565", 0)，会直接改写生产数据。调试命令不应隐式修改状态，否则容易造成线上数据污染。

3. 异常处理过宽且降级过深
   初始化阶段多个 except Exception 仅日志记录并继续运行，可能导致插件处于“部分可用但行为异常”的状态，后续问题更隐蔽（例如 service/storage 初始化失败后才在命令期报错）。

---

tests/test_subject_renderer.py

1. 日志框架不符合约束
   使用了 from loguru import logger，违反“logger 必须且只能从 astrbot.api 导入”的规则。

2. 测试依赖外部真实网络服务
   rpc_url="https://api.unitedpooh.top/rpc" 使测试变成外部集成测试，受网络波动、第三方可用性影响，导致测试不稳定且不可重复。

3. 测试中硬编码外部服务地址
   外部地址硬编码会增加维护成本，也可能在 CI 环境造成不可控失败。

---

src/config/config_manager.py

1. get_user_agent() 缺少异常保护
   读取 metadata.yaml 时未处理文件不存在/解析失败场景，可能直接抛异常影响插件初始化。

2. 路径构造耦合项目结构
   通过 Path(__file__).resolve().parent.parent.parent 回溯目录定位 metadata.yaml，对目录结构变更敏感，可维护性较差。

---

src/db/repository.py

1. 日志框架不符合约束（严重）
   使用了内置 logging（logging.getLogger("astrbot")），违反“logger 必须且只能从 astrbot.api 导入”的硬性要求。

2. 数据持久化目录不符合规范
   手动拼接 get_astrbot_data_path()/plugin_data/...，未使用 StarTools.get_data_dir()。这与框架约定不一致。

3. 插件目录名硬编码且与主插件名不一致
   目录写死为 "astrbot_plugin_bangumi"，而主注册名是 "astrbot_plugin_bangumi_enhance"，会导致历史目录混用或新旧版本数据错位。

4. 数据库初始化失败后可能出现二次异常
   _init_db() 异常时仅记录日志，不保证 self.Session 可用；后续方法直接 self.Session() 可能触发 AttributeError，错误定位困难。

5. add_subscription 事务原子性不足
   同一操作内多次 commit()，中途失败可能留下半完成状态（例如先建 subject 后建 subscription 失败）。

---

src/render/base_renderer.py

1. RPC 响应解析存在潜在空值异常
   _render_via_rpc() 中：

   result = result.get("result")
   if "image" in result:

   当 result.get("result") 为 None 时会触发 TypeError。应先判空和类型检查。

2. 网络请求缺少更细粒度异常处理
   目前 except Exception 兜底，无法区分超时、连接错误、JSON 解析错误等，影响问题定位和重试策略精度。

3. 频繁创建 aiohttp.ClientSession
   每次 RPC 调用新建 Session，连接复用不足，存在性能损耗；高并发渲染时更明显。

---

src/render/calendar_renderer.py

1. 输入数据被原地修改
   reorder_days() 中对 calendar_data 内元素直接写入 day["is_today"] = True，会产生副作用；若上游复用原对象，可能引发隐蔽逻辑问题。

---

src/render/subject_renderer.py

1. 返回类型标注冗余
   -> Optional[str] | None 重复表达同一含义，降低可读性。

2. 批量渲染并发未限流
   render_batch_subject_cards_to_base64() 对 data_list 直接 asyncio.gather(*tasks)，大量任务时可能压垮本地浏览器/RPC 服务。

3. 批处理容错性不足
   gather 未设置 return_exceptions=True，单个任务异常会导致整批失败，不利于“尽可能返回可用结果”。

---

src/services/base.py

1. 网络层缺少请求超时控制
   _request() 未配置 aiohttp 超时参数，网络异常时可能长时间挂起，影响异步系统吞吐。

2. 每次请求新建 ClientSession（性能问题）
   连接无法复用，TCP/TLS 建连开销高，频繁请求时性能下降明显。

3. 限流状态非并发安全
   last_request_time 在并发协程下共享且无锁，可能出现竞态（过度等待或限流失效）。

4. 重试失败信息可能失真
   若多次是 5xx 并走 continue，last_exception 可能仍为 None，最终抛出的错误信息缺少关键上下文。

---

src/services/calendar.py

1. 返回值缺少结构校验
   get_calendar() 直接返回 _request 数据，未验证是否为 list[dict]；上游若依赖固定结构，异常响应会在更晚阶段爆炸，增加排障成本。

---

代码评审团

人物一：Linus Torvalds（直球硬核风）
“你这 repository.py 里自己起 logging.getLogger，等于绕过了项目统一日志协议。统一约束不是装饰品。还有多次 commit() 的半事务写法，出了问题你就会得到最讨厌的那种 bug：‘看起来成功，其实烂了一半’。把事务一次性提交，日志入口统一，先把地基打平。”

人物二：Steve Jobs（产品与体验洁癖风）
“代码是产品的一部分。tests/test_subject_renderer.py 直接打公网 RPC，这不是测试，这是祈祷。稳定、可重复、可预期，才是优秀工程体验。把外部依赖 mock 掉，你会得到真正能‘放心发布’的测试套件。”

人物三：Tony Stark（工程效率 + 并发火力风）
“asyncio.gather(*tasks) 无上限并发？很酷，直到你的渲染服务冒烟。再加上每次请求新建 ClientSession，这就是‘用反应堆点蜡烛’。加并发信号量、复用会话、加超时，你的系统会从‘能跑’变成‘能扛’。”

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

---

🔍 审核摘要

统计信息

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

main.py
tests/test_subject_renderer.py
src/config/__init__.py
src/config/config_manager.py
src/db/__init__.py
src/db/models.py
src/db/repository.py
src/render/__init__.py
src/render/base_renderer.py
src/render/calendar_renderer.py
src/render/episode_renderer.py
src/render/subject_renderer.py
src/services/__init__.py
src/services/base.py
src/services/calendar.py

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

---

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

[astrbot-plugin-copybara]

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

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