Review a DeepSeek Harness plug…

# dsh-image-tools v1.2.0 审查反馈

> ⚠️ 说明：你未附上实际的 Bundle 代码/manifest 文件，我无法直接核对你的具体配置。以下基于 DeepSeek Harness 插件规范的通用审查要点，给出“高概率踩坑清单”，请对照自查。如贴出具体 JSON，可给出逐行审查。

一、tool_schema 常见问题

1. Schema 结构完整性

• inputSchema 必须是完整 JSON Schema（type: object + properties + required），缺失 required 会导致模型调用时参数随机
• outputSchema 若声明，需与实际返回结构一致，声明不符比不声明更糟

2. 图片类工具特有陷阱

• 图片参数建议用 string（URL 或 base64）+ format 提示，不要用 type: "image" 这类非标准类型
• base64 大图注意描述中标注大小上限（用区间表述，如“建议 ≤ 数 MB”），避免模型超限调用
• description 要写清输入是“URL / 本地路径 / base64”，这是图片工具最高频的模型误调用来源

3. 命名与版本一致性

• 工具名建议 dsh_image_ 前缀统一，与插件名对齐
• v1.2.0 若新增/删除工具，schema 列表必须同步，version 字段与 changelog 一致

二、权限声明审查要点

1. 最小权限原则

• 图片工具通常涉及：文件系统读取（本地图片）、网络请求（下载/上传）、临时写入（输出结果）
• 常见错误：声明了 fs.write 全盘权限但只写临时目录 → 应收窄为特定目录/路径模式（glob 白名单）
• 若 v1.2.0 新增了权限，必须在更新说明中明示，否则有合规风险

2. 声明与实际行为一致性（重点）

• 静态声明权限 ≠ 实际拦截。确认 Harness 运行时校验覆盖所有代码路径，包括异步回调里的网络请求
• 检查是否有隐式权限依赖：如通过子进程调用 ffmpeg/ImageMagick，会绕过声明的文件/网络权限

3. 敏感操作

• 图片上传到外部服务需声明 network 权限 + 在工具描述中向用户明示数据外发
• 删除文件的权限（若有）应单独声明并要求确认

三、建议自查步骤

1. 用 Harness 自带的 schema 校验器跑一遍 Bundle

2. 对比 v1.1.x → v1.2.0 的 diff，确认新工具/新权限均有声明和文档

3. 构造边界调用（空参数、超大 base64、非法 URL）实测权限拦截是否生效

如需逐行审查，请贴出 bundle.json / manifest 和 tool_schema 片段。