[Plugin] astrbot_plugin_omnidraw

[diaomin66]

插件信息

{
  "name": "astrbot_plugin_omnidraw",
  "display_name": "万象画卷",
  "desc": "占用低、速度快、配置简洁、质量高的生图插件。支持文生图与图生图、人设自拍；兼容openai；支持 LLM 工具自然调用、指令直接出图、使用权限管理（QQ）。新增视频生成，采用静默幽灵式获取。拥有强大的副脑（提示词优化）功能",
  "author": "雪碧bir",
  "repo": "https://github.com/diaomin66/astrbot_plugin_omnidraw/",
  "tags": [
    "生图",
    "视频",
    "文生图",
    "图生图",
    "异步视频",
    "人设自拍",
    "模型热切换",
    "提示词优化"
  ],
  "social_link": ""
}

插件检查清单

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

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_omnidraw

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

这里是针对您提供的 Python 插件代码的审查报告。遵循“只报告发现的问题”的原则，未列出的文件即未发现明显违规或严重缺陷。

main.py

• 框架适应性检查:
  • 数据持久化: 在 _process_images_to_base64 和 _create_image_component 方法中，严重违反了数据持久化规范。代码硬编码了存储路径：os.path.abspath(os.path.join(os.getcwd(), "data", "plugin_data", "astrbot_plugin_omnidraw", ...))。必须从 astrbot.api.star 导入 StarTools，并调用 StarTools.get_data_dir() 获取标准的数据存储目录。需要注意的是，该方法返回的是一个 Path 对象，请使用 / 操作符或 .joinpath() 方法进行路径拼接，并使用 .mkdir(parents=True, exist_ok=True) 替代 os.makedirs。
• 可维护性与可读性:
  • _process_images_to_base64 方法内部的逻辑过于臃肿。它同时负责了网络图片下载、超时重试、防盗链伪装、本地图片读取以及 Base64 转码等多个职责。建议将其拆分为单一职责的内部异步方法（如 _download_image 和 _encode_local_image），以提高代码的可读性。

models.py

• 框架适应性检查:
  • 数据持久化: from_dict 方法在处理固定形象参考图 ref_path 时，硬编码了 os.path.join(os.getcwd(), "data", "plugin_data", "astrbot_plugin_omnidraw") 这样的绝对路径依赖。请同样替换为 StarTools.get_data_dir() 以获取规范目录，防止因 AstrBot 框架运行目录结构变动导致文件路径寻址失败。

core/video_manager.py

• 潜在缺陷或问题:
  • background_task_runner 方法作为一个在后台挂机运行的独立 task，最外层使用了 except Exception as e: 进行全局兜底，并在其中执行 await event.send(Plain(f"❌ 后台视频渲染引擎发生错误：{str(e)}"))。如果产生异常的根本原因是网络断开导致底层的消息发送模块已经不可用，这里的 event.send() 调用本身会再次抛出异常，从而导致此后台协程抛出未处理的异常并静默崩溃，建议将错误通报发送动作单独用 try-except 包裹。

providers/openai_chat_impl.py

• 可维护性与可读性:
  • 文件顶部导入 BaseProvider 时，使用了一种极其不规范、脆弱的三层 try-except ImportError 嵌套，并且在最底层的 fallback 中硬编码了 from data.plugins.astrbot_plugin_omnidraw.providers.base import BaseProvider。这种基于项目特定文件系统目录结构的绝对路径导入，一旦用户重命名了插件文件夹，代码将直接瘫痪。请信任 Python 的相对导入包机制，直接保留 from .base import BaseProvider 即可。

providers/openai_impl.py

• 可维护性与可读性:
  • 与 openai_chat_impl.py 相同，存在由于不信任包结构而写出的冗长且硬编码的 BaseProvider try-except 导入代码，请将其清理，统一采用相对导入。
• 潜在缺陷或问题:
  • generate_image 在进入编辑通道（edits）组装 aiohttp.FormData() 时，直接将 image_bytes (可能达数兆的图像数据) 装载进表单字段。在多用户高并发情况下，若不限制源图的尺寸，一次性将多个大文件的 bytes 数据存放在内存中可能会导致进程内存急剧上升的性能瓶颈。

---

代码评审团

人物一（Linus Torvalds）：那些丑陋的 try-except ImportError 嵌套和硬编码的 data.plugins... 导入路径是认真的吗？你为什么不干脆把操作系统的 C 盘绝对路径也写上去？如果你的 __init__.py 结构是对的，一个优雅的相对导入就足够了！停止写这种充满妥协和狗皮膏药补丁的代码！

人物二（Elon Musk）：我喜欢 video_manager 里把多个接口节点（chat/generations）串起来做无缝降级轮询的思路，这很符合第一性原理——尽全力保障渲染任务的成功率。不过，如果你的所有节点都挂了，它会抛出 Exception 然后触发异常处理；万一此时你的 event.send() 因为断网又挂了，这颗挂机火箭就在后台无声无息地爆炸且没有任何日志记录。建议在这里增加一些多重冗余机制。

人物三（GLaDOS）：哦，我看到你的“提示词副脑”系统试图用一套庞大的 system prompt 去强迫大语言模型输出完美的、极其精确的 JSON 格式。这真是感人至深的尝试。但是，当你那可怜的模型没能返回有效的 JSON 时，你竟然选择了：默默容忍，直接把人类那种原始、粗糙的简短词汇原样喂给下游，并且仅仅打印了一句“未返回标准 JSON”的警告？多么懦弱的降级策略。你本可以做得更无情一些。

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
constants.py
models.py
utils.py
core/chain_manager.py
core/parser.py
core/persona_manager.py
core/prompt_optimizer.py
core/video_manager.py
providers/__init__.py
providers/base.py
providers/openai_chat_impl.py
providers/openai_impl.py

---

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

当前仓库：diaomin66/astrbot_plugin_omnidraw
剩余触发次数：4

[diaomin66]

@astrpluginreviewer review

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_omnidraw

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

以下是对您提供的万象画卷插件代码的深度审查报告。

main.py

• 安全漏洞与最佳实践：
  • _process_and_save_images 方法中存在裸露的 except: 捕获语句（except: await asyncio.sleep(1)）。在异步环境中，裸露的 except: 会捕获包括 asyncio.CancelledError、KeyboardInterrupt 等在内的 BaseException，这可能会阻碍协程的正常取消机制。建议：明确指定捕获范围，如 except Exception as e:。
  • 在 cmd_video 和 tool_generate_video 方法中，直接调用了 asyncio.create_task(...) 而没有保存对任务的强引用。根据 Python 官方的 asyncio 最佳实践，如果没有任何容器持有该 Task 的引用，它可能会在执行完成前被 Python 的垃圾回收器（Garbage Collector）意外销毁。建议：声明一个集合（如 self.background_tasks = set()），并将任务添加进去，任务完成后再移除自身。

core/prompt_optimizer.py

• 功能实现与逻辑正确性（严重 Bug）：
  • 在 optimize 方法解析 JSON 结果的代码块中：

    prompt_data = json.loads(raw_content)
    # ...
    items = prompt_data.get("results", [])
    if not items and isinstance(prompt_data, list):
        items = prompt_data

    如果 LLM 直接返回了一个 JSON 数组（即 prompt_data 是 list 类型），执行 prompt_data.get("results", []) 时会直接抛出 AttributeError: 'list' object has no attribute 'get'，导致整个方法崩溃降级。建议：应先判断类型，修改为 items = prompt_data.get("results", []) if isinstance(prompt_data, dict) else (prompt_data if isinstance(prompt_data, list) else [])。

providers/openai_chat_impl.py

• 代码质量与编码规范：
  • 顶部的 ImportError 异常处理嵌套过于臃肿且具有隐患：

    except ImportError:
        from data.plugins.astrbot_plugin_omnidraw.providers.base import BaseProvider

    硬编码了插件的绝对路径名 astrbot_plugin_omnidraw。一旦用户修改了插件文件夹的名称，该模块的导入将彻底崩溃。建议：保持一致的相对导入结构（如 from ..providers.base import BaseProvider），统一梳理包层级结构，不要使用硬编码绝对路径作为兜底。

providers/openai_impl.py

• 代码质量与编码规范：
  • 同样存在与 openai_chat_impl.py 相同的硬编码绝对路径导入兜底问题，可维护性较差，存在强耦合风险。

其他文件 (__init__.py, constants.py, models.py, utils.py, core/chain_manager.py, core/parser.py, core/persona_manager.py, core/video_manager.py, providers/__init__.py, providers/base.py)

• 未发现明显的框架规范违背、阻塞网络 I/O、安全漏洞或逻辑错误。代码结构清晰，严格遵守了 astrbot.api 的导入规范和数据持久化路径规范。

---

代码评审团

Linus Torvalds：
“搞什么鬼？prompt_optimizer.py 里那个 prompt_data.get("results") 简直是灾难！如果大语言模型给你扔回来一个 List，你的代码就当场挂掉抛出 AttributeError 了！你不先检查 isinstance(dict) 就敢直接调用 .get？这是编程基础课第一天就该学的内容！还有，永远、永远不要在 asyncio 里写裸露的 except:，你把中断信号和协程取消信号全都吞了，这种代码在生产环境里要是卡死了你都不知道怎么回事！”

Steve Jobs：
“我们在做产品时，最重要的是体验的一致性和底层机制的优雅。你看你的 cmd_video，你创建了一个后台任务 asyncio.create_task，然后就直接把它扔到空气里了？Python 的垃圾回收器会像清理灰尘一样把你的任务干掉！用户的视频可能永远也渲染不出来。你需要给你的艺术品加上画框，把这些后台任务存到一个 set 里保持强引用。保持优雅，好吗？”

GLaDOS：
“哦，真有趣。我看到你的 providers/openai_impl.py 里写着 from data.plugins.astrbot_plugin_omnidraw...。你是在假设人类永远不会重命名你的插件文件夹吗？把绝对路径硬编码到相对模块里，这就像是在我的核心机房里拉了一根随时会被绊倒的网线。如果这是光圈科技的测试，你显然没能拿到蛋糕。不过，你处理 Base64 防盗链的逻辑倒是勉强算个及格的防御机制。请继续你的‘实验’吧。”

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
constants.py
models.py
utils.py
core/chain_manager.py
core/parser.py
core/persona_manager.py
core/prompt_optimizer.py
core/video_manager.py
providers/__init__.py
providers/base.py
providers/openai_chat_impl.py
providers/openai_impl.py

---

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

当前仓库：diaomin66/astrbot_plugin_omnidraw
剩余触发次数：3

[diaomin66]

@astrpluginreviewer review

[astrpluginreviewer]

🤖 AI代码审核报告 for astrbot_plugin_omnidraw

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

以下是对您提供的 AstrBot 插件代码的审查报告。本次审查严格基于 Python 3.10，聚焦于异步编程最佳实践、AstrBot 框架规范及代码安全性。

根据审查要求，仅输出存在发现问题的文件报告。

---

main.py

• 代码质量与编码规范：
  • 裸异常捕获（Bare Except）：在 _process_and_save_images 方法中，使用了 except: await asyncio.sleep(1)。在 Python 和特别是异步编程中，裸异常不仅会捕获常规业务异常，还会拦截 asyncio.CancelledError、KeyboardInterrupt 等系统控制级异常，导致协程无法被正确取消。建议显式指定为 except Exception:。
• 安全漏洞与最佳实践：
  • 服务器端请求伪造 (SSRF) 风险：在 _process_and_save_images 中，代码直接将用户消息中解析出的 URL（img_ref）传递给 aiohttp.ClientSession().get()。如果恶意用户构造特定的内网地址（如 http://127.0.0.1/ 或云服务的元数据地址），可能导致插件作为代理向内网发起探测。建议对传入的 URL 进行基础清洗，或限制其目标 IP 不能为私有地址。
• 潜在缺陷或问题：
  • 未被引用的后台任务：在 cmd_video 与 tool_generate_video 方法中，使用了 asyncio.create_task(...) 来启动后台渲染视频任务，但未对创建的 Task 对象保持强引用。在某些情况下（尤其是垃圾回收介入时），未保留强引用的任务可能在执行中途被意外回收终止。建议将这些 Task 添加到一个类的集合中（如 self._background_tasks.add(task)）并在完成后移除。

core/video_manager.py

• 可维护性与可读性：
  • 异常日志缺乏堆栈信息：在 _fetch_video_from_api 的轮询重试逻辑块中，捕获异常后仅记录了 str(e)（logger.warning(f"⚠️ 接口...尝试失败: {e}...")）。由于涉及不稳定的第三方网络请求，当发生解析崩溃或连接被重置时，仅保留字符串极难定位问题。建议在关键的错误打印中加入 exc_info=True，以便追溯具体报错行。
• 功能实现与逻辑正确性：
  • URL 提取正则脆弱：_extract_url 中的正则 r'(https?://[^\s\]\)"\']+)' 相对宽泛。当大模型返回错乱 Markdown（如包含奇怪字符拼接）时，有可能截断或多提取非法字符。虽已作为备用方案，但应警惕其对无效 URL 的放行。

providers/openai_chat_impl.py

• 安全漏洞与最佳实践：
  • SSRF 风险：_encode_image_to_base64 方法通过 session.get(image_path_or_url) 下载图片时，同样未对 image_path_or_url 的可信度（内/外网、协议）做任何校验，这继承了主逻辑中的潜在安全风险。

providers/openai_impl.py

• 安全漏洞与最佳实践：
  • SSRF 风险：_get_image_bytes 方法的逻辑同上，直接对外部输入的 URL 发起了 GET 请求，需防范恶意的内部局域网探测。

---

代码评审团

Linus Torvalds：
看在上帝的份上，别在你的异步代码里写裸 except: 了。你以为你只是在忽略下载失败？你顺手把事件循环发出的取消信号（CancelledError）也给活埋了！还有，用 asyncio.create_task 派发后台任务却不保存返回的引用？Python 的垃圾回收器可不是你家保姆，它随时可能把没人管的 Task 默默扬了，到时候用户只会盯着屏幕傻等视频。

乔布斯 (Steve Jobs)：
我们追求极致的产品体验与优雅的架构。你可以把这个插件包装得极具艺术感，但内在的边界控制缺乏了一丝美感。看你的图片防盗链拦截机制——想法很好，但如果用户故意喂给你一串内网探测链接，你的服务器就会像个老实人一样乖乖去请求。我们需要“智能”的拦截，不仅要懂绕过防盗链，更要知道哪些门是不该敲的（防 SSRF）。把这一点做好，它才是艺术品。

John Carmack (约翰·卡马克)：
底层循环和网络 I/O 的健壮性是让引擎跑起来的关键。对于 video_manager.py 这种负责兜底与轮询的核心节点，出错了居然只留下一句干瘪的 str(e) 日志？当你在复杂的分布式或三方 API 返回中迷失时，你绝对会怀念完整堆栈。把 exc_info=True 加上，这对你的性能毫无影响，却能在调试陷入泥潭时救你一命。

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

---

🔍 审核摘要

统计信息

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

main.py
__init__.py
constants.py
models.py
utils.py
core/chain_manager.py
core/parser.py
core/persona_manager.py
core/prompt_optimizer.py
core/video_manager.py
providers/__init__.py
providers/base.py
providers/openai_chat_impl.py
providers/openai_impl.py

---

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

当前仓库：diaomin66/astrbot_plugin_omnidraw
剩余触发次数：2

[astrbot-plugin-copybara]

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

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