Files

1.9 KiB

final_review 角色越权上传重复文件

问题描述

review-contract.yamlfinal_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.yamlfinal_review 角色 procedure,删除上传相关步骤,只保留检查 + 通知 Doro 的逻辑。属于 workflow YAML 改动,需要 WeiWei 确认。