[Plugin] astrbot_plugin_self_open_llm_vtuber

[murphys7017]

插件信息

{
  "name": "astrbot_plugin_self_open_llm_vtuber",
  "display_name": "Alice的桌面分身",
  "desc": "让你的bot在桌面或者web可以看到你、看到你在干什么、开口说话并实时做表情动作，支持语音输入、语音打断与互动联动。",
  "author": "YakumoAki",
  "repo": "https://github.com/murphys7017/astrbot_plugin_self_open_llm_vtuber"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_self_open_llm_vtuber

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

main.py

1. PEP 8 规范问题
   import 语句与类定义之间缺少空行（顶层类定义前应保留 2 个空行）。

2. 可维护性问题（隐式副作用导入）
   在 __init__ 内部执行 from .platform_adapter import OLVPetPlatformAdapter，但未直接使用该符号，属于“靠导入触发副作用”的写法。
   这会降低可读性：后续维护者难以一眼看出此处是为了触发注册逻辑，而非正常依赖使用。

---

platform_adapter.py

1. 职责过重，可维护性较差
   OLVPetPlatformAdapter 同时承担了：WebSocket 服务管理、消息协议解析、会话状态机、音频缓存、图片落盘、安全路径校验、表情规划、性能打点等多种职责。
   单类体量过大，后续修改容易引发连锁影响，测试与定位问题成本高。

2. 潜在性能瓶颈（音频缓冲拼接）
   _handle_audio_data / _handle_raw_audio_data 中对 numpy 数组反复 np.append，每次都会重新分配并拷贝，连续流式音频场景下可能退化为明显的 O(n²) 开销。

3. 潜在异常未覆盖（缓存清理阶段）
   _cleanup_audio_cache 中 trimmable_files 的生成与排序使用了 entry.stat()，该阶段未做异常保护；若文件在遍历过程中被并发删除/替换，可能抛出 OSError 并中断清理逻辑。

4. 路径策略存在部署风险
   音频/图片缓存目录固定在插件代码目录下（PLUGIN_DIR/olv/cache/...）。在只读部署、容器镜像只读层或受限权限环境中，运行时写入可能失败，导致功能异常。

5. 并发一致性风险（锁使用不一致）
   _handle_client 的 finally 中直接重置 self._audio_buffer，但其他路径对同一资源使用了 _audio_buffer_lock。
   锁粒度不一致会带来竞态窗口，可能出现缓冲状态被覆盖或丢帧的边界问题。

---

platform_event.py

1. 潜在重复发送/语义冲突
   send() 中先调用 self.adapter.emit_message_chain(...)，随后又调用 await super().send(message)。
   若父类 send 仍会走平台发送链路，则可能导致同一消息重复下发（或至少造成发送语义重叠），属于逻辑风险点。

---

static_resources.py

1. CORS 过度开放
   end_headers() 中将 Access-Control-Allow-Origin 设为 *，并允许所有请求头。
   在本地服务被其他站点脚本探测/调用的场景下，攻击面会增大；建议最小化来源与头部范围。

---

adapter/audio_runtime.py

1. 全局导入路径修改的并发风险
   temporary_olv_import_path() 通过修改全局 sys.path 实现动态导入。
   在异步/并发场景下，这种全局可变状态可能影响同进程其他导入逻辑，造成不可预期行为（导入污染、竞争条件、错误模块命中）。

---

代码评审团

人物一：Linus Torvalds（直球型）
“platform_event.py 这个 send() 先自发一遍、再 super().send() 一遍，像是‘我怕你收不到，所以我发两次’。代码要么定义单一路径，要么明确幂等，不然线上排查重复消息会把人逼疯。建议明确事件层和平台层谁是唯一发送入口。”

人物二：Steve Jobs（产品体验导向）
“OLVPetPlatformAdapter 现在像一台把所有零件焊死在一起的机器：能跑，但不优雅。把网络接入、媒体处理、表达规划拆开，你会得到更可预测的行为和更快的迭代速度。优秀的软件不是功能堆砌，而是边界清晰。”

人物三：Geralt of Rivia（猎魔人式风险嗅觉）
“我闻到了 sys.path 的味道（audio_runtime.py）。全局路径临时改写就像在夜里开门不设岗：短期能进人，长期也会进怪。建议把动态导入隔离在更可控的加载机制里，至少避免影响其他并发任务的导入现场。”

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

---

🔍 审核摘要

统计信息

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

main.py
platform_adapter.py
platform_event.py
static_resources.py
adapter/__init__.py
adapter/audio_runtime.py
adapter/base_expression_fallback.py
adapter/base_expression_planner.py
adapter/chat_buffer.py
adapter/payload_builder.py
adapter/plugin_runtime.py
adapter/protocol.py
adapter/session_state.py

---

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

当前仓库：murphys7017/astrbot_plugin_self_open_llm_vtuber
剩余触发次数：3

[astrbot-plugin-copybara]

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

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