[Plugin] astrbot_plugin_self_code

[murphys7017]

插件信息

{
  "name": "astrbot_plugin_self_code",
  "display_name": "Self Code Assistant",
  "desc": "一个用于 AstrBot 的 AI 编程助手插件。它把聊天指令转为 codex CLI 操作，在 workspace 中生成/修改插件代码，并支持测试和部署。",
  "author": "YakumoAki",
  "repo": "https://github.com/murphys7017/astrbot_plugin_self_code",
  "social_link": "https://www.xiaohongshu.com/user/profile/6986d59f0000000024012e78"
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_self_code

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

main.py

1. 停止确认有效期逻辑未生效（功能正确性）
   pending_stop_confirm_until 在 _should_confirm_stop 与自然语言退出分支中被设置，但全局没有任何地方校验该时间戳。结果是“60 秒内确认”的设计并未真正执行，/codexdev stop confirm 在任意时间都会生效。

2. codex_bin 配置未贯通（功能一致性）
   多处调用 dev_session.handle_message(...) 时未传入可配置的 codex_bin，实际会始终走默认 "codex"。如果用户在配置中设置了自定义二进制路径，该设置在主流程中不会生效。

3. 数据持久化目录规范未按要求使用（框架适配）
   持久化相关能力通过 plugin_root 派生路径（经 get_runtime_root）管理，而不是通过 StarTools.get_data_dir() 获取标准数据目录。与要求的持久化约束不一致。

---

core/dev_session.py

1. “只读日志检查”缺少技术强约束（安全/逻辑）
   _run_log_inspection_pipeline 仅通过 prompt 要求“Do NOT modify code or files”，但没有在执行后做文件变更校验或回滚机制。若模型误操作，仍可能修改工作区文件，和“只分析不修复”的预期不完全一致。

2. 数据持久化路径来源不符合指定方式（框架适配）
   DevSessionManager 依赖 get_runtime_root(plugin_root)，未使用 StarTools.get_data_dir() 作为规范持久化入口。

---

core/local_skills.py

1. 持久化路径硬编码到插件目录（框架适配）
   self.plugin_root / "data" / "local_skills" 直接写在插件目录下，不符合通过 StarTools.get_data_dir() 取得标准持久化目录的要求。

2. 并发写入缺少互斥保护（潜在缺陷）
   create_or_update_skill、_backup_existing_skill 等文件写操作在并发场景下无锁，可能出现覆盖、备份错位或部分写入竞争问题。

---

core/prompt_builder.py

1. 本地 skills 读取路径未走规范数据目录（框架适配）
   _load_local_skills_reference 使用 plugin_root / "data" / "local_skills"，未通过 StarTools.get_data_dir() 获取规范目录，存在目录约定不一致风险。

---

core/skills_cache.py

1. 压缩包解压存在 Zip Slip 风险（安全）
   _download_and_build_index 中 archive.extractall(extract_dir) 未做成员路径校验。理论上恶意 zip 可借路径穿越写出目标目录外。

2. 同步网络 I/O 设计在异步系统中存在误用风险（并发模型）
   urlopen 下载与 GitHub API 访问均为同步网络调用；虽然当前主流程有用 asyncio.to_thread 包裹更新动作，但该类方法本身可被直接调用，一旦在事件循环线程直接执行会阻塞。

3. 持久化目录获取未遵循指定接口（框架适配）
   缓存目录基于 get_runtime_root 推导，未按要求从 StarTools.get_data_dir() 取得标准路径。

---

core/tester.py

1. 异步 reload 任务“未等待即返回成功”（逻辑正确性）
   reload_plugin 中若 manager.reload() 返回协程且当前有事件循环，仅 create_task 后立即返回成功。真实 reload 失败会在后台发生，调用方会收到假阳性结果。

2. 基础测试会执行生成插件代码，隔离不足（安全）
   importlib 直接执行 sandbox 内 main.py，本质是执行不可信代码；当前仅目录级“沙箱”，非进程/权限级隔离，存在宿主进程风险。

3. 持久化路径来源不符合规范（框架适配）
   依赖 get_runtime_root，未使用 StarTools.get_data_dir()。

---

core/utils.py

1. 未使用 StarTools.get_data_dir()（框架适配关键问题）
   get_runtime_root 通过 astrbot.core.utils.path_utils 或目录猜测逻辑解析数据目录，未采用要求的官方持久化入口 StarTools.get_data_dir()。

2. 路径推断回退逻辑不稳健（潜在缺陷）
   _resolve_astrbot_data_dir 的 plugin_root.parent.parent / (plugin_root / "data") 回退可能在不同部署结构下指向非预期目录，导致数据散落或读取错位。

---

core/workspace.py

1. 持久化目录未通过规定接口获取（框架适配）
   工作区路径依赖 get_runtime_root 推导，未使用 StarTools.get_data_dir()，与数据目录规范不一致。

---

代码评审团

人物一：Linus Torvalds（直球毒舌型）
“你们的 stop confirm 计时器像是写了个‘心理安慰剂’：变量有了，执法没了。要么删掉，要么在每次确认时做硬校验，别让状态字段变成注释。”

人物二：Steve Jobs（体验偏执型）
“用户最怕‘看起来成功，实际上失败’。reload create_task 后立刻回 success，会把信任感打碎。产品体验上，这比报错更糟糕——至少报错是真话。”

人物三：Geralt of Rivia（猎魔人式风险嗅觉）
“我闻到了‘解压即信任’的味道。extractall 不做路径审计，就像让陌生术士进你城堡地下室。先验每个成员路径，再落盘，这种怪物得先封印后对话。”

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

---

🔍 审核摘要

统计信息

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

main.py
core/plugin_template/main.py
core/__init__.py
core/codex_runner.py
core/dev_session.py
core/llm_tools.py
core/local_skills.py
core/prompt_builder.py
core/skills_cache.py
core/tester.py
core/utils.py
core/workspace.py

---

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

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

[astrbot-plugin-copybara]

插件信息已同步到集合仓库。
PR: AstrBotDevs/AstrBot_Plugins_Collection#323
已创建 PR #323，自动合并失败，请人工处理。

[astrbot-plugin-copybara]

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

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