[Plugin] astrbot_plugin_LivelyState

[KonmaKanSinPack]

插件信息

{
  "name": "astrbot_plugin_LivelyState",
  "display_name": "角色状态记忆",
  "desc": "这是一个让角色拥有持续状态记忆的拟人插件：不再每句“重开存档”，而是带着上一刻的心情继续和你说话。",
  "author": "兔子",
  "repo": "https://github.com/KonmaKanSinPack/astrbot_plugin_LivelyState",
  "tags": [
    "拟人",
    "全局记忆"
  ]
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_LivelyState

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

main.py

以下是对 main.py 文件的代码审查报告。该审查基于 Python 3.10、异步环境的执行标准以及 AstrBot 框架规范。

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

• 致命的缩进层级错误：change_current_state 这个由 @filter.llm_tool 装饰的方法被错误地定义在了 LivelyState 类的外部。由于它本质上是全局作用域中的一个独立函数，但第一个参数却命名为 self 并在内部调用了 self._handle_apply(...)，这会导致 LLM 调用该工具时引发严重的 AttributeError 或 TypeError。
  • 修复建议：将 change_current_state 方法及其装饰器整体增加一个缩进级别，将其移入 LivelyState 类内部，使其成为合法的实例方法。

2. 代码质量与编码规范

• 不符合 PEP 8 的命名规范：在 change_current_state 方法的签名中，参数使用了首字母大写（如 Emotion, Energy, Thirst, State）。在 Python 中，方法参数应严格使用小写字母和下划线分隔符（如 emotion, energy 等）。
• 未使用的导入（Dead Imports）：文件顶部包含了大量从未被使用的导入项，不仅增加了阅读负担，还可能拖慢加载速度。例如：from pydantic import Field, ContextWrapper, FunctionTool, ToolExecResult, AstrAgentContext, Tuple, List 等。建议清理无用引用。
• 无意义的空方法：CharacterState.update 方法为空（只有 pass），如果是计划中尚未实现的功能，建议添加 TODO 注释或直接移除。

3. 可维护性与可读性

• 冗余或未使用的死代码：类 LivelyState 中存在几个从未在上下文中被调用过的方法，如 _extract_json_block、send_prompt 以及 get_persona_system_prompt。如果这些方法不作为对外的接口或是后续更新的保留功能，建议将其移除，以降低类的维护复杂度和代码体积。

4. 安全漏洞与潜在缺陷

• 未发现明显的安全漏洞（例如没有未经验证的命令执行、未包含不安全的序列化等）。json_repair 处理读取时的安全性也优于直接使用 eval 或存在风险的方案。

5. 框架适应性检查 (AstrBot 规范)

• ✅ 日志记录：严格按照规范仅使用了 from astrbot.api import logger。
• ✅ 并发模型：未发现阻塞异步事件循环的同步网络 I/O 操作。
• ✅ 数据持久化：完全遵循了使用 StarTools.get_data_dir() 的规范，且正确利用了返回的 Path 对象进行了目录创建（mkdir(parents=True, exist_ok=True)）和读写操作，非常规范。
• ✅ 插件结构与装饰器：filter 导入来源正确；实现了继承自 Star 的类；on_llm_request 钩子具备完全正确的三个参数 (self, event, req) 及 async def 定义。

---

🎭 代码评审团

人物一：Linus Torvalds（Linux 之父）

“我简直不敢相信我的眼睛。你的 change_current_state 函数飘在了类的外面，就像太空垃圾一样！然后你在里面若无其事地写上 self，甚至还沾沾自喜地调用了 self._handle_apply？你以为只要给参数起名叫 self，Python 的解释器就会像保姆一样自动帮你找到对应的类实例吗？！把那个该死的方法缩进调整对，放回你的类里面去。这是最基础的常识，别让代码看起来像是用脚趾头敲出来的。”

人物二：Steve Jobs（苹果公司联合创始人）

“我们要的是优雅，是极致的清晰。但看看这个文件的顶部，堆砌着 FunctionTool、ContextWrapper 这些根本就没用到的导入模块，就像是在漂亮的玻璃面板上留下了脏兮兮的指纹。还有那些首字母大写的参数名 Emotion、Energy……这不够 Pythonic，也不够优美。做减法吧。把你用不到的方法和导入全都删掉，直到这件作品不再有任何累赘。只有当它极其纯粹时，它才能完美工作。”

人物三：GLaDOS（《传送门》系列虚拟 AI）

“哦，看看这个模拟人类情绪和疲惫感的插件，试图用大段的 Prompt 告诉 LLM 它有多‘累’。这真是个感人的算力浪费实验。既然你都有了一个全局状态文件，为什么把时间衰减逻辑完全寄托在大模型的阅读理解能力上？本地代码中的 CharacterState.update 却是个空壳。我建议你把能量流失和情绪衰退的数学模型写在本地计算里，大模型只需要负责生成文本即可。毕竟，人类连简单的加减法都容易算错，何况是你们造出来的语言模型呢。测试完成后，请记得清理掉那些写了又不用的死代码。”

(Disclaimer: 以上评审内容由 AI 自动生成，所涉及人物形象与现实无关，不代表真实人物观点。)

---

🔍 审核摘要

统计信息

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

main.py

---

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

[KonmaKanSinPack]

已修复

[astrbot-plugin-copybara]

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

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