[Plugin] astrbot_plugin_ancient_poem

[juangchuank-ops]

插件信息

{
  "name": "astrbot_plugin_ancient_poem",
  "display_name": "古诗插件",
  "desc": "这是一个 AstrBot 插件，用于随机获取并发送一首古诗词。",
  "author": "花殇",
  "repo": "https://github.com/juangchuank-ops/astrbot_plugin_ancient_poem",
  "tags": [
    "古诗"
  ]
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_ancient_poem

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

main.py

这份代码整体质量非常高，异常处理全面（特别是对 asyncio.CancelledError 的处理体现了良好的异步编程素养），并且完全遵守了 AstrBot 的各项 API 规范。以下是审查中发现的少数可优化项和潜在逻辑瑕疵：

1. 代码质量与编码规范

• 类型注解与实际处理冗余：
  在 get_poem(self, event: AstrMessageEvent, count: int = 1) 的签名中，count 已经被注解为 int。然而在方法内部，又使用了 try...except (ValueError, TypeError): count = int(count) 进行防御性转换。
  • 问题：如果 AstrBot 框架依赖类型注解自动转换参数，这里就是多余的；如果框架将用户输入作为纯字符串传入，那么签名中的 int 注解具有误导性（应为 str 并在内部转换，或使用 Union[int, str]）。建议查明框架处理机制，保持签名与内部处理逻辑的一致性。

2. 功能实现与逻辑正确性

• API 缓存导致重复率升高的问题：
  向 hitokoto.cn 连续发送高频的相同 GET 请求时，由于其背后的 CDN 节点缓存机制，极有可能在短时间内返回完全相同的诗句。
  • 问题：代码中使用 await asyncio.sleep(0.2) 并在内部使用 sent_contents 进行去重，这是一种“被动防御”。在极端情况下，CDN 持续返回相同结果，会导致循环快速消耗 max_attempts 额度。
  • 建议：在 URL 中动态加入时间戳或随机数以穿透缓存（Cache Busting），例如：url = f"https://v1.hitokoto.cn/?c=i&_t={asyncio.get_event_loop().time()}"，这样可以大幅降低触发去重逻辑的概率。

3. 潜在缺陷或问题

• terminate 方法的隐式依赖：
  你编写了 terminate(self) 方法来优雅关闭 aiohttp.ClientSession。
  • 问题：虽然这在大多数框架中是标准生命周期钩子，但需要绝对确认 AstrBot 框架在热重载或卸载插件时必定会自动调用名为 terminate 的方法。如果框架不支持或方法名不对，会导致 Session 泄漏。如果框架不支持，建议使用 async with aiohttp.ClientSession() 在每次命令中管理上下文，由于你限定了最多5次请求，这里的开销是可以接受的。

---

代码评审团

Linus Torvalds：（极客、暴躁、直击系统与性能痛点）

“我看了你的循环。你在一个 while 循环里做网络请求，然后还乖乖地 await asyncio.sleep(0.2) 挂起等待？你是按小时收费的吗？用户要 5 首诗，你就串行地发 5 次请求，中间还要睡一觉！为什么不用 asyncio.gather 并发把请求全打出去？就算考虑到重复问题，你并发发 8 个请求，拿到结果后本地去重挑 5 个不就行了？不过，看在你居然知道捕获并且 raise asyncio.CancelledError 的份上，我今天就不骂脏话了。这证明你至少读过一点异步编程的手册。”

Gordon Ramsay：（主厨视角、苛刻、关注逻辑与结构的“味道”）

“过来看看这个 count 参数！你在菜单（函数签名）上写着这道菜是牛肉（int），结果到了后厨（函数体），你又假装它可能是块臭鞋底（TypeError, ValueError），非要再把它强行塞进绞肉机里转一圈！It's bloody redundant! 如果框架给你的是整数，你这叫脱裤子放屁；如果框架给你的是字符串，你的签名就是在欺骗顾客！清理掉这些杂质，让你的代码保持纯粹！”

GLaDOS：（光圈科技AI，冷漠、被动攻击、关注边界和测试）

“测试结果表明，这名人类开发者试图用 set() 来处理 API 返回的重复数据。多可爱啊，你觉得 API 会感激你的温柔吗？它只会在 0.2 秒的间隔内给你吐出 5 次一模一样的 JSON 缓存。你为什么不给 URL 加个时间戳参数呢？哦，我忘了，人类的脑容量通常不足以同时思考‘网络延迟’和‘CDN缓存’两个维度的问题。不过没关系，至少你在拿不到数据的时候，会老老实实地告诉用户获取失败，而不是让整个系统崩溃并释放致命神经毒气。这算个进步。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

当前仓库：juangchuank-ops/astrbot_plugin_ancient_poem
剩余触发次数：2

[astrbot-plugin-copybara]

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

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