[Plugin] astrbot_plugin_bazaar

[shitlsh]

插件信息

{
  "name": "astrbot_plugin_bazaar",
  "display_name": "大巴扎小助手",
  "desc": "astrbot_plugin_bazaar 是一个 AstrBot 聊天机器人插件，可以在 QQ / Telegram / 微信等平台中快速查询 The Bazaar 游戏数据，包括怪物、物品、技能信息，以及来自社区的推荐阵容。\n\n查询结果以精美的图片卡片形式呈现，支持中英文双语查询。",
  "author": "钱德勒饼",
  "repo": "https://github.com/shitlsh/astrbot_plugin_bazaar",
  "tags": [
    "bazaar",
    "电子游戏"
  ]
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_bazaar

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

main.py

1. 框架适应性问题

• 注册装饰器过时: 代码中使用了 @register 装饰器。在 AstrBot v3.5.20 版本之后，系统会自动识别继承自 Star 的类并将其作为插件加载，@register 已被废弃。建议移除该装饰器以保持代码简洁并符合最新规范。

2. 安全漏洞与最佳实践

• HTTP Session 频繁创建与销毁: 在 _download_image 和 _fetch_builds 方法中，每次发起请求都会执行 async with aiohttp.ClientSession() as session:。在异步应用中，频繁建立和关闭网络连接会带来巨大的性能开销。最佳实践是在插件类（BazaarPlugin）初始化时创建一个实例级别的持久 Session，并在 terminate(self) 中优雅关闭。

3. 代码质量与编码规范

• 脆弱的 HTML 清理方式: _fetch_builds 中提取文本时使用了链式替换：excerpt_html.replace("<p>", "").replace("</p>", "").replace("<br>", "\n")...。这种方式不仅难以阅读，而且对稍微不规则的 HTML 标签毫无招架之力。建议使用正则表达式（如 re.sub(r'<[^>]+>', '', text)）或内置模块进行更健壮的文本提取。
• 重复的搜索校验逻辑: cmd_monster、cmd_item 和 cmd_skill 指令中，判断“未找到、精确匹配 1 个、部分匹配多个”的 if-elif-else 逻辑高度重复。建议将这部分代码提炼为一个通用的泛型函数，提升可维护性和代码复用率。

---

card_renderer.py

1. 框架适应性问题

• 数据持久化路径硬编码: 在 __init__ 方法中，缓存目录被硬编码为 self.cache_dir = plugin_dir / "data" / "cache"，并执行了写入操作。对于需要持久化保存的数据（如图片缓存），必须通过 from astrbot.api.star import StarTools 并调用 StarTools.get_data_dir() 来获取规范的数据存储目录。

2. 安全漏洞与最佳实践

• 未复用的 aiohttp Session: 与 main.py 相同，_fetch_image 中同样每次请求都在实例化 ClientSession。建议将复用的 Session 通过参数注入到 CardRenderer 中。

3. 潜在缺陷或问题

• 低效的排版计算逻辑: 在 _wrap_text 方法中，为了计算文本换行，采用了逐个字符追加并调用 font.getbbox(test) 的做法。这种 O(N^2) 级别的像素级计算在遇到长文本时性能堪忧，且可能会强行截断英文字母。建议按词组（以空格分隔）进行计算，或使用更成熟的文本排版方式。

4. 可维护性与可读性

• 泛滥的“魔法数字” (Magic Numbers): render_monster_card、render_item_card 等渲染方法动辄上百行，其中充斥着海量的魔法数字（如 y += 22, y += 28, content_width - 30）。一旦未来需要微调 UI 或改变字号，开发者将迷失在这些杂乱无章的加减法中。建议将所有边距、行高、固定偏移量提取为类的常量配置。

---

代码评审团

Linus Torvalds：（揉着眉头，语气极其烦躁）
“我真是不明白，为什么人们在使用异步网络库时总是像得了失忆症一样。你每次去下载一张小图片或者抓取一条数据，都要建立一个新的 aiohttp.ClientSession() 然后再把它撕碎？你知道 TCP 握手和 TLS 协商要浪费多少时间和 CPU 周期吗？建立一个 Session，把它存下来，复用它！还有那个 _wrap_text 里的逐个字符拼凑算宽度的逻辑，天哪，如果这就是你们生成图片的算法，我宁愿蒙上眼睛看命令行文本！”

罗永浩：（叹了口气，推了一下眼镜）
“兄弟，你这个排版代码写得实在是太不体面了。你看这些散落一地的数字，y += 22，padding + 8，它们是什么？它们为什么在这里？当你下个版本想把字体调大一号时，你打算怎么面对这几百行的硬编码？这完全没有所谓的‘工匠精神’。另外，看看你清理 HTML 的那个 .replace() 链条，就像是用狗皮膏药去贴大厦上的裂缝，一点都不优雅。为什么不能好好封装一下呢？”

马斯克：（双手抱胸，思维跳跃）
“这套代码的系统架构不够第一性原理。比如你那个 @register 装饰器，新的引擎明明已经可以自动捕获并点火（自动识别插件类）了，你还留着那个冗余的挂载组件干什么？把它砍掉，减轻起飞重量！还有你的缓存文件，居然硬编码存在插件目录里？这完全没有考虑到星际扩张（跨实例/环境迁移）的需要！统一用 StarTools.get_data_dir()，让框架系统帮你管理存储，才能实现全自动的生态闭环。”

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

---

🔍 审核摘要

统计信息

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

main.py
card_renderer.py

---

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

[shitlsh]

已修复

[astrbot-plugin-copybara]

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

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