27 lines
1.9 KiB
Markdown
27 lines
1.9 KiB
Markdown
# final_review 角色越权上传重复文件
|
|
|
|
## 问题描述
|
|
`review-contract.yaml` 的 `final_review` 角色在完成终审后，会自行上传文件到 Nextcloud，与 `deliverer` 角色已上传的文件形成重复。
|
|
|
|
## 表现
|
|
- 任务交付/ 中出现同一合同的两个版本，文件大小完全一致（内容相同）
|
|
- final_review 上传的版本使用完全不同的命名格式（如 `朱家角-巷泽...-修订版-邱庭-20260629.docx`），与 deliverer 的 `【修】合同_朱家角.docx` 不一致
|
|
- 还可能额外生成审查意见 .txt 文件（final_review 自行生成的，非 editor 产出的 companion）
|
|
|
|
## 已确认的案例（2026-06-29）
|
|
1. 巷泽居委会办公家具采购项目合同 — deliverer 上传 `【修】合同_朱家角.docx`，final_review 又上传 `朱家角-巷泽居委会办公家具采购项目合同-修订版-邱庭-20260629.docx`
|
|
2. 医疗急救中心救护车采购合同 — deliverer 上传 `【修】医疗急救中心救护车采购合同.pdf`，final_review 又上传 `【修】上海市青浦区医疗急救中心救护车采购合同-邱庭-20260629.pdf` + `上海市青浦区医疗急救中心救护车采购合同-审查意见-邱庭-20260629.txt`
|
|
|
|
## 判别方法
|
|
- `stat` 比较文件大小：完全一致 = 重复
|
|
- 命名格式不同：`【修】原名.docx` vs `前缀-全名-修订版-邱庭-日期.docx`
|
|
|
|
## 处理方式
|
|
保留 deliverer 上传的版本（命名符合 `【修】+ 原始文件名` 规则），删除 final_review 的重复文件。
|
|
|
|
## 根因
|
|
`final_review` 角色的 procedure 中包含了上传文件的逻辑（它从 /tmp/contract-review/ 读取文件并上传），但 deliverer 已经做过这一步。两个角色都上传 = 双份。
|
|
|
|
## 修复方向
|
|
修改 `review-contract.yaml` 的 `final_review` 角色 procedure，删除上传相关步骤，只保留检查 + 通知 Doro 的逻辑。属于 workflow YAML 改动，需要 WeiWei 确认。
|