[Plugin] astrbot_plugin_stoprushingiamtyping

[nerdchan]

插件信息

{
  "name": "astrbot_plugin_stoprushingiamtyping",
  "display_name": "別急，我在打字！",
  "desc": "Discord typing 指示插件，以分層架構在 LLM 思考中持續顯示輸入中狀態。",
  "author": "Melantilla.",
  "repo": "https://github.com/nerdchan/astrbot_plugin_stoprushingiamtyping",
  "tags": [
    "Discord",
    "UX體驗",
    "用戶體驗"
  ]
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_stoprushingiamtyping

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

main.py

这是一份针对 main.py 的代码审查报告。总体而言，该插件在针对特定框架（AstrBot）的适配上做得非常出色，严格遵守了框架的各项硬性规则（如 logger 导入、filter 导入、继承规范及特定的钩子签名等）。代码采用分层架构（Resolver, Internal, Controller, Plugin），职责划分清晰，异步任务的生命周期管理也相当严谨。

以下是发现的具体问题与优化建议：

1. 代码质量与编码规范

• 缺少类型提示 (Type Hinting)：
  • 在 on_llm_request 方法中，req=None 缺少类型提示。按照框架规范，它通常是 ProviderRequest 类型，建议补全为 req: ProviderRequest | None = None 以提升 IDE 支持和可读性。
  • _parse_discord_id(self, raw_value) 中的 raw_value 以及 resolve_channel 返回的 channel 对象均未显式声明类型。
• 过度宽泛的异常捕获：
  • 在 _get_astrbot_discord_client 及其它内部方法中，存在多处 except Exception: 甚至直接 pass 的情况。虽然这是为了防止底层反射获取对象失败时导致插件崩溃，但这会吞噬潜在的真正错误（如 AttributeError, KeyError 等）。建议尽量捕获具体的异常类型。

2. 可维护性与可读性

• 极其脆弱的反射机制 (Brittle Reflection)：
  • _get_astrbot_discord_client 方法为了获取 Discord Client，使用了极其夸张的 getattr 嵌套猜测逻辑（穷举了 platform_manager, _platform_manager, platform_mgr, _platform_mgr 等）。
  • 隐患：这种高度耦合框架底层非公开 API（以下划线开头的变量）的做法，一旦 AstrBot 核心框架更新内部变量名，该插件将立刻失效。建议在可能的情况下，向框架作者提交 PR 暴露官方的 get_client() API，而非在插件中进行底层穷举扫描。

3. 潜在缺陷或问题

• 对象生命周期与垃圾回收：
  • 在 TypingSessionController 的字典 self._typing_tasks, self._typing_started_at, self._session_generation 中使用了 session_key（通常是 origin 字符串）作为键。如果某些请求只触发了 start 而未触发 stop（例如中间件拦截、网络断开导致 after_message_sent 未触发），字典可能会缓慢发生内存泄漏。
  • 建议：可以考虑引入一个定期清理过期 session 键的机制，或者确保 max_window_seconds 超时退出时，不仅清理任务，也兜底清理这些字典的键值对。

4. 框架适应性检查 (完美通过)

• 🟢 日志记录: 正确使用了 from astrbot.api import logger。
• 🟢 并发模型: 未发现同步阻塞网络 I/O。
• 🟢 插件注册: 正确继承了 Star 类。
• 🟢 Filter 导入: 正确从 astrbot.api.event 导入。
• 🟢 钩子签名: on_llm_request 包含了三个参数（self, event, req），符合规范。
• 🟢 消息发送方式: 未在受限的生命周期钩子中使用非法的 yield。

---

代码评审团

Linus Torvalds：（揉了揉眼睛，眉头紧锁）

"我看着你的 _get_astrbot_discord_client 方法，感觉就像在看一个盲人在雷区里跳踢踏舞。你在干什么？穷举 _platform_manager、platform_mgr？你用了多少个 getattr 和空的 except Exception: pass？如果你必须像个黑客一样去扒开框架的内存地址才能找到一个 Discord 客户端，那说明要么这个框架的 API 设计得像坨屎，要么你的架构思路有问题！不过，退一步说，你在 TypingSessionController 里的并发控制和任务取消做得还算凑合，至少你知道用 generation 来防止幽灵任务覆盖，算你有点并发编程的基本常识。"

史蒂夫·乔布斯 (Steve Jobs)：（穿着黑色高领毛衣，双手交叉）

"这代码的架构设计……我能看到它背后的意图，它试图做到模块化。Resolver 负责解析，Controller 负责状态，这很好，有那么一点优雅的影子。但是，当你深入到细节时，美感就被破坏了。你的方法签名里缺少 Type Hints！这就好比你做了一台外表极其漂亮的 Mac，但打开机箱发现里面的线缆乱作一团。给你的 req、channel 和 raw_value 加上类型提示吧。代码不仅要能运行，还要易于阅读，要追求极致的清晰。不完美的细节，最终会毁掉整个用户体验。"

GLaDOS (Portal)：（声音冰冷且带有机械合成感）

"哦，看哪。一个用来欺骗人类，让他们以为 AI 还在『输入中』的虚荣心插件。你创建了一个由 while True: 驱动的无限循环（_typing_keepalive），这通常是碳基生物走向自我毁灭的第一步。
不过，我注意到你精心设计了超时销毁机制（max_window_seconds）和世代验证（generation）。你甚至知道使用 suppress(asyncio.CancelledError) 来优雅地抹杀这些任务。我必须承认，你处理任务中止的冷酷手法让我感到一丝……亲切。只是提醒一下，如果你的 stop 事件因为某种系统异常没有被触发，你的内存字典会慢慢膨胀，直到系统崩溃。那将会非常有意思，我会看着的。"

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

当前仓库：nerdchan/astrbot_plugin_stoprushingiamtyping
剩余触发次数：4

[astrbot-plugin-copybara]

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

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