@steedos-labs/content-compliance 0.8.25 → 0.8.27

package/main/default/triggers/pepsico_material_approval_change.trigger.yml

@@ -6,30 +6,140 @@ when:
 isEnabled: true
 handler: |-
   const { _ } = global;
-  const { doc } = ctx.params;
-  if(ctx.params.isInsert){
-    if(doc.approval_type === 'personnel_review'){
-      if(_.isEmpty(doc.reviewer)){
-        throw new Error('固定审核人缺失, 请确认审批单对应审核人员信息');
-      }
+  const { doc, previousDoc } = ctx.params;
+
+  // 保存审批单时的审核人兜底与校验 (#450)
+  //
+  // 审核人原本只由 amis 字段的 onEvent.change 链回填, 服务端没有任何补偿。
+  // 前端链一断(链首 ajax 是 ignoreError:false、owner_department 没有 change 事件、
+  // 矩阵查不到时 adaptor 返回 {} 把已有值抹掉), 单子就带着空审核人存进来,
+  // 一路走到部门审核才因取不到审核人而中断, 静默卡死且无告警。
+  //
+  // 这里在保存这一刻就按角色矩阵补齐, 补不上就拦下 —— 空审核人的单子根本不该存进库。
+  //
+  // 只管草稿阶段: 审批过程中的修改、退回给提交人修改等都不该被拦,
+  // 那时单子早已在流程里, 再拦只会挡住正常操作。
+
+  // beforeUpdate 的 doc 只含本次变更字段, 必须和 previousDoc 合并后再判,
+  // 否则任何不碰审核人的更新(改个备注)都会被误判成"审核人为空"而拦死。
+  const record = _.assign({}, previousDoc || {}, doc);
+
+  const isBlank = (v) => _.isArray(v) ? _.isEmpty(_.compact(v)) : _.isEmpty(v);
+  // tree-select 在不同配置下可能给出字符串或单元素数组, 查矩阵时统一成字符串
+  const matrixKey = (v) => _.isArray(v) ? _.compact(v)[0] : v;
+
+  // 与 src/const.ts 一致: DCO 分类跳过审核直接归档, 不需要审核人
+  const dcoTypes = _.split(
+    (typeof process !== 'undefined' && process.env && process.env.CONTENT_DCO_TYPES) || 'MD-PO1-PO18',
+    ','
+  );
+
+  // 新建的单子一律是草稿(init 触发器在 beforeInsert 无条件写 current_step=draft),
+  // 直接认 isInsert 而不读 doc.current_step —— 两个 beforeInsert 触发器的执行顺序没有保证。
+  const isDraft = ctx.params.isInsert === true || record.current_step === 'draft';
+
+  const needsReviewers = isDraft
+    && record.review_status !== 'terminated'
+    && !_.includes(dcoTypes, record.classification);
+
+  if(needsReviewers){
+    // 角色矩阵(pepsico_reviewer)与审批单上同名的字段。显式列出是为了圈定范围,
+    // 避免把矩阵行上的其它字段(name / org / remark 等)一起写进审批单。
+    const MATRIX_FIELDS = [
+      'reviewer', 'assistance',
+      'ns_reviewer', 'ns_assistance',
+      'legal_reviewer', 'legal_assistance',
+      'sra_reviewer', 'sra_assistance',
+      'ca_reviewer', 'ca_assistance',
+      'qc_reviewer', 'qc_assistance',
+      'can_pass_users',
+      'pre_reviewer', 'pre_ns_reviewer', 'pre_legal_reviewer', 'pre_sra_reviewer', 'pre_ca_reviewer', 'pre_qc_reviewer'
+    ];
+
+    const classification = matrixKey(record.classification);
+    const product = matrixKey(record.product);
+    const org = matrixKey(record.owner_department);
+
+    let row = null;
+    if(classification && product && org){
+      // 矩阵上这三个字段都是 multiple, 审批单上是单值, 所以 '=' 即"矩阵行包含该值"。
+      // 用 directFind 绕权限: 发起人通常没有角色矩阵的读权限, 走 find 会读不到而误判成"未配置"。
+      const rows = await ctx.getObject('pepsico_reviewer').directFind({
+        filters: [
+          ['classification', '=', classification],
+          ['product', '=', product],
+          ['org', '=', org],
+          ['space', '=', record.space || ctx.params.spaceId]
+        ],
+        sort: 'modified desc',
+        top: 1
+      });
+      row = rows && rows.length > 0 ? rows[0] : null;
     }
 
-    if(doc.approval_type === 'departmental_review'){
-      if(_.isEmpty(doc.ca_reviewer)){
-        throw new Error('CA审核人缺失, 请确认审批单对应审核人员信息');
-      }
-      if(_.isEmpty(doc.legal_reviewer)){
-        throw new Error('Legal审核人缺失, 请确认审批单对应审核人员信息');
-      }
-      if(_.isEmpty(doc.ns_reviewer)){
-        throw new Error('NS审核人缺失, 请确认审批单对应审核人员信息');
-      }
-      if(_.isEmpty(doc.sra_reviewer)){
-        throw new Error('SRA审核人缺失, 请确认审批单对应审核人员信息');
+    if(row){
+      // 只补空, 不覆盖 —— 加急换人、管理员手工调整过的值必须保住, 覆盖会伤存量数据
+      _.each(MATRIX_FIELDS, (field) => {
+        if(isBlank(record[field]) && !isBlank(row[field])){
+          doc[field] = row[field];
+          record[field] = row[field];
+        }
+      });
+    }
+
+    const deptLabels = {
+      ns_reviewer: 'NS',
+      legal_reviewer: 'Legal',
+      sra_reviewer: 'SRA',
+      ca_reviewer: 'CA'
+    };
+    if(record.ec_has_detail_page === 'yes'){
+      deptLabels.qc_reviewer = 'QC';
+    }
+    const missingDepts = _.filter(_.keys(deptLabels), (field) => isBlank(record[field]));
+    const hasPersonnel = !isBlank(record.reviewer);
+
+    let missing = [];
+    if(record.approval_type === 'personnel_review'){
+      if(!hasPersonnel){
+        missing = ['固定'];
       }
-      if(doc.ec_has_detail_page === 'yes' && _.isEmpty(doc.qc_reviewer)){
-        throw new Error('QC审核人缺失, 请确认审批单对应审核人员信息');
+    }else if(record.approval_type === 'departmental_review'){
+      missing = _.map(missingDepts, (field) => deptLabels[field]);
+    }else if(!hasPersonnel && missingDepts.length > 0){
+      // approval_type 为空恰恰是最危险的情况: 它和审核人来自同一条前端链, 链断则两者同时为空。
+      // 旧逻辑在这里直接跳过校验, 空审核人的单子就是这样进来的, 所以不能以它有值为前提。
+      // 此时无法确定后续会走哪条分支(approval_type 由服务端在提交时重算),
+      // 要求至少有一套可用的审核人: 固定人员, 或四部门齐全。
+      missing = _.map(missingDepts, (field) => deptLabels[field]);
+    }
+
+    if(missing.length > 0){
+      // 原始 ID 写系统日志给管理员定位; 发起人就在表单上, 分类/品牌/部门他看得见, 不必回显
+      try {
+        await ctx.getObject('logs').directInsert({
+          name: '保存被拦截: 审核人缺失',
+          level: 'error',
+          data: JSON.stringify({
+            action: 'save-blocked-reviewer-missing',
+            id: record._id || ctx.params.id,
+            approval_name: record.name,
+            is_insert: ctx.params.isInsert === true,
+            missing: missing,
+            matched_pepsico_reviewer: row ? row._id : null,
+            approval_type: record.approval_type || null,
+            classification: classification,
+            product: product,
+            owner_department: org
+          }),
+          space: record.space || ctx.params.spaceId,
+          created: new Date()
+        });
+      } catch (e) {
+        console.error('[pepsico_material_approval_change] 写系统日志失败', e && e.message);
       }
+
+      throw new Error('缺少' + missing.join('、') + '审核人, 请联系管理员为当前「分类 + 品牌/产品 + 发起人部门」的组合配置角色矩阵后重试。');
     }
   }
 
@@ -40,7 +150,7 @@ handler: |-
       }
     }
   }
-  
+
 authentication_type: none
 locked: false
 type: code

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@steedos-labs/content-compliance",
-    "version": "0.8.25",
+    "version": "0.8.27",
     "description": "内容审核系统",
     "main": "package.service.js",
     "scripts": {