Codecov Report

❌ Patch coverage is 66.66667% with 29 lines in your changes missing coverage. Please review.

📢 Thoughts on this report? Let us know!

Code Review：feat(weixin): support incoming image and file attachments (#1765)

变更概述

本 PR 为微信频道新增了图片和文件附件的接收支持：在 WeixinMonitor 中检测 IMAGE_ITEM_TYPE（type=2）和 FILE_ITEM_TYPE（type=4）消息体，从 WeChat CDN 下载原始数据、AES-128-ECB 解密、magic-byte 类型嗅探后保存到本地 weixin-uploads/，并将文件路径以 [Image: /path] 或 [File "name": /path] 格式追加到 IUnifiedIncomingMessage 的 text 字段交给 Agent 处理；新增 72 小时 TTL + 200 MB 容量上限的清理机制。

方案评估

结论：⚠️ 方案有缺陷

整体方案思路合理：将附件路径以 marker 形式注入 text 字段，不改变 IPC/Agent 接口，改动影响面很小。下载 + 解密 + 存储的实现也与现有架构（main process 直接 fs 操作）一致。但 msgId 直接来自 WeChat API 响应，未经过滤就拼入文件路径，存在路径穿越风险，需要修复。

问题清单

🟠 HIGH — msgId 未过滤，存在路径穿越

文件：src/process/channels/plugins/weixin/WeixinMonitor.ts，第 267 行

问题代码：

const msgId = msg.msg_id ?? String(Date.now());
// ...
const fileName = `${msgId}-${idx}-${safeName}${ext}`;
const filePath = path.join(uploadsDir, fileName);

问题说明：msgId 直接取自 WeChat API 响应，未做任何清理。若服务端（或中间人）返回 msg_id: "../../home/user/.ssh/authorized_keys" 等恶意值，path.join 会解析 \.\. 跳出 uploadsDir，导致文件被写入任意路径。declaredName 已经过 safeName 正则清理，msgId 同样需要相同处理。

修复建议：

const rawMsgId = msg.msg_id ?? String(Date.now());
const safeMsgId = rawMsgId.replace(/[^a-zA-Z0-9._-]/g, '_').slice(0, 48);
// ...
const fileName = `${safeMsgId}-${idx}-${safeName}${ext}`;

🔵 LOW — 现有测试名称与新行为不符

文件：tests/unit/channels/weixinMonitor.test.ts，第 101 行

问题代码：

it('does not call agent.chat for non-text items (image type=2)', async () => {

问题说明：PR 改动后，type=2 的 item 已经会进入下载流程，并非因为「非文本消息被跳过」而不触发 agent.chat，而是因为 image_item 为空导致下载抛异常（missing encrypt_query_param）、attachments 为空 + text 为空，才最终跳过。测试名称描述的行为与实际代码路径已不一致，容易误导后续维护者。

修复建议：

it('skips agent.chat when media item has no encrypt_query_param', async () => {

汇总

结论

⚠️ 有条件批准 — 路径穿越问题（HIGH）需要修复后可合并，LOW 级测试名称建议一并更新。

本报告由本地 pr-review skill 生成，包含完整项目上下文，无截断限制。

CONCLUSION: CONDITIONAL
IS_CRITICAL_PATH: false
PR_NUMBER: 1765

PR Fix 验证报告

原始 PR: #1765
修复方式: 直接推送到 feat/weixin-attachments

总结： ✅ 已修复 1 个 | ❌ 未能修复 0 个

🔵 LOW 级别问题（测试名称与新行为不符）已跳过（不阻塞合并）。