[Plugin] astrbot_plugin_context_undo

[lumingya]

插件信息

{
  "name": "astrbot_plugin_context_undo",
  "display_name": "上下文撤回",
  "desc": "撤回最近一次或多次的ai回复上下文，避免ai被不良信息影响。",
  "author": "lumingya",
  "repo": "https://github.com/lumingya/astrbot_plugin_context_undo"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_context_undo

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

main.py

1. 潜在逻辑错误：发送结果后仍允许继续调用 LLM

   • 位置：_send_plain_result()
   • 问题：该函数里调用了 event.should_call_llm(True)。
     对于“撤回/回滚”这类管理指令，通常预期是插件直接回复并终止后续 LLM 对话流程。这里设为 True 有机会导致：
     • 指令回复后，LLM 仍继续处理原消息；
     • 产生重复响应或污染会话上下文。
   • 建议：确认框架语义后，按意图改为阻断后续 LLM 流程（通常是 False）。

2. 误触发清理风险：普通聊天文本可能触发上下文栈清空

   • 位置：watch_session_management_commands()
   • 问题：_CLEAR_STACK_COMMANDS 包含 "reset", "del", "delete"（无前缀），且仅通过首 token 匹配。
     用户正常发言如“reset 一下思路”也可能触发清理逻辑，导致撤回栈/LTM 状态被清空。
   • 建议：仅匹配明确命令（如 /reset），或增加“是否命令消息”的判定再执行清理。

3. 并发一致性问题：读-改-写无互斥，存在丢数据风险

   • 位置：commit_turn()、_rollback_to_turn()、_save_turn_stack()、_save_ltm_state() 等
   • 问题：异步环境下多个事件可并行到达，同一 umo + conversation_id 上对栈和 LTM 的更新是“先读后写”，缺少锁或 CAS 机制。
     结果可能出现：
     • turn 栈覆盖（后写覆盖先写）；
     • rollback 与 commit 交错导致状态撕裂。
   • 建议：按会话粒度加 asyncio.Lock，把关键读-改-写包成原子区间。

4. 健壮性问题：脏数据可能导致异常中断

   • 位置：_clean_turn_stack()
   • 问题：int(item.get("turn_id", 0) or 0)、int(item.get("created_at", 0) or 0) 对非法字符串（如 "abc"）会抛 ValueError。
     KV 数据若损坏或被旧版本写入非预期类型，会直接中断流程。
   • 建议：对 int() 转换加 try/except 并跳过坏记录，避免插件整体失效。

5. 健壮性问题：_load_turn_stack() 对异常类型缺少保护

   • 位置：_load_turn_stack()
   • 问题：if len(cleaned_stack) != len(raw_stack): 默认 raw_stack 可取 len()。若 KV 返回意外类型（如 int），这里会 TypeError。
   • 建议：仅在 isinstance(raw_stack, list) 时比较长度；否则直接视为脏数据并重置保存。

6. 快照不是深拷贝，存在历史回滚快照漂移风险

   • 位置：_build_pending_turn() -> history_before = self._coerce_history(...)
   • 问题：_coerce_history() 仅过滤并返回原 dict 引用（浅层）。若后续对同一 dict 发生原地修改，history_before 记录会被“连带改变”，影响回滚准确性。
   • 建议：在保存快照时做深拷贝（至少对 list[dict] 结构）。

7. 类型鲁棒性不足：工具调用名拼接可能异常显示

   • 位置：_build_assistant_preview()
   • 问题：", ".join(resp.tools_call_name) 假设它是字符串列表。若实际是单个字符串，会被按字符 join（例如 "abc" -> "a, b, c"）。
   • 建议：对 tools_call_name 做类型分支处理（str / list[str]）。

---

代码评审团

人物一：Linus Torvalds
“你这逻辑在单线程脑补里很顺，但异步环境不是童话世界。commit_turn 和 rollback 没有会话级锁，典型 race condition 温床。先把状态一致性打牢，再谈功能优雅。”

人物二：Steve Jobs
“用户体验上最怕‘误操作且不可见’。reset/del/delete 被普通文本误触发清空，是会让人抓狂的隐形交互 bug。把命令语义做得更明确，体验会瞬间高级很多。”

人物三：Deadpool（死侍）
“你都主动 send 结果了，还 should_call_llm(True)？这像我一边说‘闭嘴’，一边递麦克风。结论：机器人可能会加戏，剧情会失控。让流程只说一次人话。”

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

---

🔍 审核摘要

统计信息

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

main.py

---

此报告由AI自动生成，旨在提供初步反馈和改进建议，不能完全替代人工审核。最终决策以社区维护者的人工审核为准。目前自动审核（仓库地址）处于试验阶段，如遇问题请向维护者反馈。评论 @astrpluginreviewer /review 来重新触发 review，请不要重复提交插件发布 issue。

当前仓库：lumingya/astrbot_plugin_context_undo
剩余触发次数：4

[lumingya]

@astrpluginreviewer /review

[astrbot-plugin-copybara]

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

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