[Plugin] astrbot_plugin_iamthinking

[sssn-tech]

插件信息

{
  "name": "astrbot_plugin_iamthinking",
  "display_name": "别吵我在思考",
  "desc": "别吵，我在思考！还在因为思考型模型反应太慢而烦恼？这个插件不能解决问题，但可以提升一点体验。收到用户消息并准备调用 LLM 时，为这条用户消息贴一个表情表示处理中；发送回复后，再贴一个表情表示处理完成。",
  "author": "石上三年",
  "repo": "https://github.com/sssn-tech/astrbot_plugin_iamthinking",
  "social_link": "https://github.com/sssn-tech"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_iamthinking

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

main.py

1. on_llm_response 的第三个参数类型过于宽泛（Any）

   • 你实现了 @filter.on_llm_response()，签名是三参数这一点正确；但第三个参数标注为 Any，不符合“特定对象”语义要求，降低了类型约束与可维护性。
   • 风险：后续维护者难以明确 resp 的实际结构，IDE/类型检查无法有效提示，容易引入误用。
   • 建议：改为框架对应的明确响应类型（如 LLMResponse）。

2. 存在多处“吞异常”式写法，掩盖真实故障原因

   • 位置包括：
     • try: import ... except Exception: AiocqhttpMessageEvent = None
     • _as_int_list 中对元素转换的 except Exception: continue
     • _emoji_like 中 except Exception as e: logger.warning(...)
   • 风险：真实编程错误（例如属性拼写、对象协议变化）会被当作普通异常吞掉，导致问题长期潜伏，仅表现为“功能偶发失效”。
   • 建议：缩窄异常类型（如 ImportError, ValueError, TypeError），并在日志中附带必要上下文（emoji_id/message_id/event类型）。

3. after_message_sent 完成态设置时机存在一致性问题

   • 当前逻辑在执行完“贴完成表情 + 可选移除思考表情”后，无论实际是否全部成功，都直接 event.set_extra("iamthinking_done", True)。
   • 风险：一旦贴表情调用部分失败，状态已被标记为 done，后续不会重试，最终 UI 状态可能与真实意图不一致。
   • 建议：根据调用成功结果决定是否设置 done，或记录失败后允许幂等重试。

4. 异步并发下可能重复执行“完成逻辑”

   • after_message_sent 中 iamthinking_done 的检查与设置不是原子操作，且中间有 await。若该钩子在同一事件上并发触发，可能出现重复贴/取消表情调用。
   • 风险：重复网络请求、状态抖动、日志噪音，极端情况下可能触发平台限流。
   • 建议：在首次进入时尽早设置“处理中锁标记”（例如 iamthinking_finishing），或使用事件级锁保证同一事件只完成一次。

---

代码评审团

人物一：Linus Torvalds（直给型）
“这段代码能跑，但异常处理像拿毛毯盖火警报警器：安静了，不代表没着火。except Exception 到处都是，debug 时会很痛苦。把异常类型收紧，问题会自己浮出来。”

人物二：Martin Fowler（重构派）
“状态流转（active / responded / done）已经有雏形，但完成态与外部调用结果耦合松散。建议把状态机显式化：thinking -> responding -> done/failed，并把失败重试策略写清楚，维护成本会明显下降。”

人物三：Tony Stark（工程效率派）
“你这插件思路很实用，但并发安全像没装防抖模块。after_message_sent 在高并发聊天里可能连点两次‘完成按钮’。加个轻量锁或原子标记，系统稳定性会立刻上一个台阶。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

[sssn-tech]

@astrpluginreviewer review

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_iamthinking

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

main.py

1. 异常捕获范围过窄，异步网络调用可能直接中断钩子链

   • 位置：_emoji_like() 中 await bot.set_msg_emoji_like(...) 的 try/except
   • 问题：当前仅捕获 AttributeError/RuntimeError/TypeError/ValueError。实际异步网络请求中常见的 TimeoutError、ConnectionError、底层 SDK 抛出的通用异常未覆盖，可能导致异常冒泡，影响后续事件处理。
   • 影响：在网络抖动或平台 API 异常时，插件可能出现未预期中断。
   • 建议：至少补充网络相关异常，或在该调用点统一兜底 Exception 并记录详细日志（避免吞掉上下文）。

2. 失败重试缺乏上限，存在重复调用与日志风暴风险

   • 位置：after_message_sent()
   • 问题：当 done_ok/remove_ok 失败时，仅将 iamthinking_finishing 置回 False，但不增加失败计数、不做熔断。若同一事件触发多次 after_message_sent，会持续重试。
   • 影响：在机器人无权限、接口长期失败等场景下，会反复请求平台接口并打大量 warning/debug 日志。
   • 建议：增加最大重试次数或失败后短路标记（如 iamthinking_failed），避免无界重试。

3. 平台判断逻辑过于收缩，可能导致功能“看起来失效”

   • 位置：_is_aiocqhttp()
   • 问题：return bool(event.get_group_id()) 将私聊场景全部排除。插件描述是“为 LLM 对话贴表情提示处理中/完成”，但当前实际只在群聊生效。
   • 影响：用户在私聊中不会生效，且仅有 debug 日志提示，容易被误判为插件故障。
   • 建议：若设计上确实只支持群聊，建议在配置项/文档中显式声明；否则应放宽判定条件。

4. @register 用法有迁移维护成本

   • 位置：类装饰器 @register(...)
   • 问题：按你给出的规范说明，v3.5.20 后 @register 已废弃（虽然仍可用）。继续依赖旧注册方式会增加后续迁移工作。
   • 影响：框架升级时可能引入兼容性维护负担。
   • 建议：中长期可考虑切换到“自动识别继承 Star 类”的注册方式，减少技术债。

---

代码评审团

人物一：Linus Torvalds（直球型）
“你在 _emoji_like 里以为自己把异常都抓了，其实只抓了‘你想到的异常’，没抓‘现实会给你的异常’。网络代码不做完整异常策略，就是把稳定性交给运气。建议：按失败类型分层处理，至少别让钩子链被一次超时直接打断。”

人物二：Ada Lovelace（结构型）
“这段逻辑已经接近一个状态机了（active/responded/finishing/done），但没有‘失败终态’。没有终态就会不断循环尝试。建议加入明确的失败状态与重试上限，让状态转移可证明、可观察、可维护。”

人物三：Tony Stark（产品型）
“功能叫‘LLM 对话提示’，结果私聊直接被 group_id 门禁挡掉，用户只会说：‘它偶尔好使，偶尔装死。’建议把支持范围写进配置或说明，别让行为和产品预期打架。还有，废弃注册方式这事，早点还债，未来少加班。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

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

[sssn-tech]

机器人给出的建议1；2；4已经修改，3添加了readme额外说明。贴表情本来就是QQ群聊特有功能，私聊不支持

[astrbot-plugin-copybara]

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

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