@steedos-labs/plugin-workflow 3.0.51 → 3.0.53

package/main/default/routes/am.router.js

@@ -183,7 +183,7 @@ router.post('/am/forms', async function (req, res) {
             });
             let companyId = currentSpaceUser.company_id;
             let newForm = {
-                _id: form["id"],
+                _id: form["id"] || form["_id"],
                 name: form["name"],
                 state: "disabled",
                 is_deleted: false,
@@ -204,7 +204,7 @@ router.post('/am/forms', async function (req, res) {
                 newForm.company_id = companyId;
             }
             let current = {
-                _id: form["current"]["id"],
+                _id: form["current"]["id"] || form["current"]["_id"],
                 name_forumla: name_forumla,
                 form: newForm._id,
                 _rev: 1,
@@ -274,7 +274,7 @@ router.post('/am/forms', async function (req, res) {
             let flow = {
                 _id: _makeNewID(),
                 space: form["space"],
-                form: form["id"],
+                form: form["id"] || form["_id"],
                 name_formula: "",
                 code_formula: "",
                 is_valid: true,
@@ -323,7 +323,7 @@ router.post('/am/forms', async function (req, res) {
                 steps: await designerManager.makeSteps(userId, newForm.current.fields, language),
                 _rev: 1,
                 flow: flow._id,
-                form_version: form["current"]["id"],
+                form_version: form["current"]["id"] || form["current"]["_id"],
                 created: now,
                 created_by: userId,
                 modified: now,
@@ -359,7 +359,7 @@ router.put('/am/forms', async function (req, res) {
             // 执行者的身份校验
             await designerManager.checkSpaceUserBeforeUpdate(form['space'], userId, req.user.roles)
             // 更新表单
-            await designerManager.updateForm(form["id"], form, updatedForms, updatedFlows, userId)
+            await designerManager.updateForm(form["id"] || form["_id"], form, updatedForms, updatedFlows, userId)
         }
         res.send({
             "ChangeSet": {
@@ -370,6 +370,7 @@ router.put('/am/forms', async function (req, res) {
             }
         });
     } catch (error) {
+        console.log(error)
         res.status(500).send(error.message)
     }
 })
@@ -394,7 +395,7 @@ router.delete('/am/forms', async function (req, res) {
         let now = new Date();
         for (let i = 0; i < data['Forms'].length; i++) {
             let form = data['Forms'][i];
-            let f = await formCollection.findOne({_id: form["id"]});
+            let f = await formCollection.findOne({_id: form["id"] || form["_id"]});
             // 执行者的身份校验
             await designerManager.checkSpaceUserBeforeUpdate(f.space, userId, req.user.roles)
 
@@ -404,7 +405,7 @@ router.delete('/am/forms', async function (req, res) {
             // 删除表单对应流程
             let flows = await flowCollection.find({
                 space: spaceId,
-                form: form["id"]
+                form: form["id"] || form["_id"]
             }).toArray();
             for (let flow_fd of flows) {
                 let deleted_flow = Object.assign({}, flow_fd);
@@ -419,7 +420,7 @@ router.delete('/am/forms', async function (req, res) {
             if (recordCollection) {
                 let records = await recordCollection.find({
                     space: spaceId,
-                    form: form["id"]
+                    form: form["id"] || form["_id"]
                 }).toArray();
                 for (let record_fd of records) {
                     let deleted_record = Object.assign({}, record_fd);
@@ -434,7 +435,7 @@ router.delete('/am/forms', async function (req, res) {
             // 删除表单对应申请单
             let instances = await instanceCollection.find({
                 space: spaceId,
-                form: form["id"]
+                form: form["id"] || form["_id"]
             }).toArray();
             for (let instance_fd of instances) {
                 let deleted_instance = Object.assign({}, instance_fd);
@@ -446,7 +447,7 @@ router.delete('/am/forms', async function (req, res) {
             }
 
             // 删除instance_tasks
-            await instanceCollection.deleteMany({ form: form["id"] })
+            await instanceCollection.deleteMany({ form: form["id"] || form["_id"] })
 
             // 删除表单
             let deleted_form = Object.assign({}, f);
@@ -454,7 +455,7 @@ router.delete('/am/forms', async function (req, res) {
             deleted_form['deleted_by'] = userId;
             await dFormsCollection.insert(deleted_form);
             deletedForms.push(deleted_form);
-            await formCollection.deleteOne({_id: form["id"]})
+            await formCollection.deleteOne({_id: form["id"] || form["_id"]})
         }
 
         res.send({
@@ -483,7 +484,7 @@ router.put('/am/flows', async function (req, res) {
             let flowCome = data['Flows'][i];
             let spaceId = flowCome["space"];
             let formId = flowCome["form"];
-            let flowId = flowCome["id"];
+            let flowId = flowCome["id"] || flowCome["_id"];
             let upgraded = flowCome['upgraded'];
             delete flowCome['decription'];
             let now = new Date();
@@ -501,7 +502,7 @@ router.put('/am/flows', async function (req, res) {
             // 某步骤被删除后，删除同流程的“指定历史步骤”属性中被引用的步骤id（仅限于流程的最新版)
             let clientStepIds = []
             _.each(flowCome['current']['steps'], function (step) {
-                clientStepIds.push(step['id']);
+                clientStepIds.push(step['_id'] || step['id']);
             })
 
             _.each(flowCome['current']['steps'], function (step) {
@@ -524,12 +525,16 @@ router.put('/am/flows', async function (req, res) {
 
             // 由于前台传的是id而非_id，故比较时将id转为_id
             _.each(flowCome['current']['steps'], function (step) {
-                step['_id'] = step['id'];
-                delete step['id'];
+                if (step['id']) {
+                    step['_id'] = step['id'];
+                    delete step['id'];
+                }
                 if (step['lines']) {
                     _.each(step['lines'], function (line) {
-                        line['_id'] = line['id'];
-                        delete line['id'];
+                        if (line['id']) {
+                            line['_id'] = line['id'];
+                            delete line['id'];
+                        }
                     })
                 }
             })
@@ -647,6 +652,7 @@ router.put('/am/flows', async function (req, res) {
             }
         });
     } catch (error) {
+        console.log(error);
         res.status(500).send(error.message)
     }
 })
@@ -664,7 +670,7 @@ router.put('/am/flows/state', async function (req, res) {
             let flowCome = data['Flows'][i];
             let spaceId = flowCome["space"];
             let formId = flowCome["form"];
-            let flowId = flowCome["id"];
+            let flowId = flowCome["id"] || flowCome["_id"];
             let now = new Date();
             let flowUpdateObj = {
                 $set: {}

package/main/default/routes/api_workflow_ai_form_design.router.js

@@ -39,7 +39,7 @@ router.post('/am/ai/form-design', async function auth(req, res, next) {
     const baseURL = process.env.WORKFLOW_AI_BASE_URL || 'https://api.openai.com/v1';
     const model = process.env.WORKFLOW_AI_MODEL || 'gpt-4o';
 
-    const { fields, events, userRequest, images, instance_template, form_script, flow_events } = req.body;
+    const { fields, events, userRequest, images, instance_template, form_script, flow_events, name_forumla } = req.body;
 
     if (!userRequest) {
       return res.status(400).json({ error: '请输入需求描述' });
@@ -120,6 +120,7 @@ router.post('/am/ai/form-design', async function auth(req, res, next) {
   - 示例: "\${RMB(assessed_value)}" —— 人民币大写（推荐）
   - 示例: "\${TEXT(amount, \"rmb\")}" —— 等同于 RMB()，两者都可用，不要用 TEXT(v, "[DBNum2]...") 的 Excel 格式
   - 注意：公式内字段名直接写，不加 \${} 嵌套，正确："\${RMB(amount)}"，错误："\${RMB(\${amount})}"
+  - ⚠️ 子表字段引用规则：在主表公式中对子表（table）的子字段使用聚合函数时，直接使用子字段名，不需要加子表名前缀。系统会自动从子表数据中提取该列进行聚合。正确："\${SUM(amount)}"（amount 是子表的子字段名），错误："\${SUM(items.amount)}"（不要用 子表名.子字段名 的写法）。示例：子表 "items" 有子字段 "unit_price" 和 "quantity"，主表求合计公式写 "\${SUM(unit_price)}"，不要写 "\${SUM(items.unit_price)}"。
 
 - section 类型（分组标题）的额外字段：
   - colorScheme: 字符串，预设配色方案名称。可选值："indigo"(靛蓝), "emerald"(翠绿), "amber"(琥珀), "rose"(玫红), "sky"(天蓝), "violet"(紫罗兰), "slate"(石板灰), "orange"(橙色), "teal"(青色)。设置后分组标题会显示对应的主题色
@@ -401,9 +402,29 @@ if (field.name === 'quantity' || field.name === 'unit_price') {
     "onValueChange": "脚本代码或空字符串",
     "onSubmit": "脚本代码或空字符串"
   },
+  "nameFormula": "标题公式（仅升级场景，将旧版标题公式中的字段引用更新为新版字段 name）",
   "migrationLog": ["迁移记录，仅升级场景需要"]
 }
 
+### nameFormula（标题公式，升级场景）
+当用户提供了旧版标题公式（name_forumla）时，需要将其中的字段引用更新为新版字段 name。
+- 旧版标题公式格式：用 {字段名} 引用字段值，如 "{申请人} 的 {类型} 申请"
+- 升级时**只替换 {旧字段名} → {新字段name}**，如 "{applicant_name} 的 {request_type} 申请"
+- ⚠️ **公式中花括号外的所有字符必须原样保留**，包括但不限于：双引号 "、单引号 '、中文引号""''、括号、标点符号、空格等。只有 {} 内的字段名需要替换，其他一切文本保持不变
+- 示例：旧版 \`"项目" + {项目名称}\` → 新版 \`"项目" + {project_name}\`（注意引号 " 和加号 + 必须保留）
+- 示例：旧版 \`{申请人}"提交的"{类型}"申请"\` → 新版 \`{applicant_name}"提交的"{request_type}"申请"\`（注意引号必须保留）
+- 示例：旧版 \`{项目名称}（{申请日期}）\` → 新版 \`{project_name}（{apply_date}）\`（注意中文括号必须保留）
+- 如果用户未提供旧版标题公式，则 nameFormula 字段可省略或返回空字符串
+
+#### name_forumla 运行时格式规范
+nameFormula 是一个 JS 表达式，运行时被包装为 \`return <nameFormula>\`，其中 \`{fieldCode}\` 会被替换为 \`(values.fieldCode || '')\`，\`{applicant.xxx}\` 会被替换为 \`(applicant.xxx || '')\`。
+**规则：所有非 {xxx} 的文本（包括中文、符号、空格）必须用引号包裹，多段用 + 拼接。**
+- 正确：\`"合同名称" + {project_name}\`
+- 正确：\`{applicant_name} + "-" + {project_name}\`
+- 错误：\`合同名称{project_name}\` — JS 语法错误
+- 错误：\`合同名称 + {project_name}\` — JS 语法错误
+- 错误：\`{project_name} 审批单\` — JS 语法错误
+
 ### migrationLog（迁移记录）
 当用户提供了 flow_events、form_script、instance_template 中包含业务逻辑（如校验、赋值、显隐控制等脚本代码）时，必须在 migrationLog 数组中为每条被迁移的逻辑记录一条说明，格式：
   "<来源>: <原逻辑简述> → <迁移到的具体位置>"
@@ -442,6 +463,9 @@ if (field.name === 'quantity' || field.name === 'unit_price') {
     if (flow_events) {
       extraSections += `\n\n## 流程事件 (flow_events)\n\`\`\`javascript\n${flow_events}\n\`\`\``;
     }
+    if (name_forumla) {
+      extraSections += `\n\n## 旧版标题公式 (name_forumla)\n${name_forumla}\n\n说明：标题公式用 {字段名} 引用字段值，升级时需将其中的 {旧字段名} 替换为 {新版字段name}，输出到 nameFormula 字段。**只替换 {} 花括号内的字段名，花括号外的所有文本（包括引号 "、单引号 '、中文引号""''、括号、其他标点符号和空格等）必须原样保留，禁止增删改。**`;
+    }
 
     const userContent = `## 当前表单字段
 ${fieldsJson}

package/main/default/routes/api_workflow_ai_form_design_stream.router.js

@@ -90,7 +90,7 @@ router.post('/am/ai/form-design-stream', async function auth(req, res, next) {
       extraSections += `\n\n## 流程事件 (flow_events)\n\`\`\`javascript\n${flow_events}\n\`\`\``;
     }
     if (name_forumla) {
-      extraSections += `\n\n## 旧版标题公式 (name_forumla)\n${name_forumla}\n\n说明：标题公式用 {字段名} 引用字段值，升级时需将其中的 {旧字段名} 替换为 {新版字段name}，输出到 nameFormula 字段。`;
+      extraSections += `\n\n## 旧版标题公式 (name_forumla)\n${name_forumla}\n\n说明：标题公式用 {字段名} 引用字段值，升级时需将其中的 {旧字段名} 替换为 {新版字段name}，输出到 nameFormula 字段。**只替换 {} 花括号内的字段名，花括号外的所有文本（包括引号 \"、单引号 '、中文引号""''、括号、其他标点符号和空格等）必须原样保留，禁止增删改。**`;
     }
 
     const userContent = `## 当前表单字段\n${fieldsJson}\n\n## 当前事件脚本\n${eventsJson}${extraSections}\n\n## 用户需求\n${userRequest}\n\n请返回修改后的完整 JSON 对象（包含 tableColumns、fields 和 events）。`;
@@ -476,6 +476,7 @@ function buildSystemPrompt() {
   - 示例: "\${RMB(assessed_value)}" —— 人民币大写（推荐）
   - 示例: "\${TEXT(amount, \\"rmb\\")}" —— 等同于 RMB()，两者都可用，不要用 TEXT(v, "[DBNum2]...") 的 Excel 格式
   - 注意：公式内字段名直接写，不加 \${} 嵌套，正确："\${RMB(amount)}"，错误："\${RMB(\${amount})}"
+  - ⚠️ 子表字段引用规则：在主表公式中对子表（table）的子字段使用聚合函数时，直接使用子字段名，不需要加子表名前缀。系统会自动从子表数据中提取该列进行聚合。正确："\${SUM(amount)}"（amount 是子表的子字段名），错误："\${SUM(items.amount)}"（不要用 子表名.子字段名 的写法）。示例：子表 "items" 有子字段 "unit_price" 和 "quantity"，主表求合计公式写 "\${SUM(unit_price)}"，不要写 "\${SUM(items.unit_price)}"。
 
 - section 类型（分组标题）的额外字段：
   - colorScheme: 字符串，预设配色方案名称。可选值："indigo"(靛蓝), "emerald"(翠绿), "amber"(琥珀), "rose"(玫红), "sky"(天蓝), "violet"(紫罗兰), "slate"(石板灰), "orange"(橙色), "teal"(青色)。设置后分组标题会显示对应的主题色
@@ -779,10 +780,22 @@ if (field.name === 'quantity' || field.name === 'unit_price') {
 ### nameFormula（标题公式，升级场景）
 当用户提供了旧版标题公式（name_forumla）时，需要将其中的字段引用更新为新版字段 name。
 - 旧版标题公式格式：用 {字段名} 引用字段值，如 "{申请人} 的 {类型} 申请"
-- 升级时将 {旧字段名} 替换为 {新字段name}，如 "{applicant_name} 的 {request_type} 申请"
-- 公式中的纯文本部分保持不变
+- 升级时**只替换 {旧字段名} → {新字段name}**，如 "{applicant_name} 的 {request_type} 申请"
+- ⚠️ **公式中花括号外的所有字符必须原样保留**，包括但不限于：双引号 "、单引号 '、中文引号""''、括号、标点符号、空格等。只有 {} 内的字段名需要替换，其他一切文本保持不变
+- 示例：旧版 \`"项目" + {项目名称}\` → 新版 \`"项目" + {project_name}\`（注意引号 " 和加号 + 必须保留）
+- 示例：旧版 \`{申请人}"提交的"{类型}"申请"\` → 新版 \`{applicant_name}"提交的"{request_type}"申请"\`（注意引号必须保留）
+- 示例：旧版 \`{项目名称}（{申请日期}）\` → 新版 \`{project_name}（{apply_date}）\`（注意中文括号必须保留）
 - 如果用户未提供旧版标题公式，则 nameFormula 字段可省略或返回空字符串
 
+#### name_forumla 运行时格式规范
+nameFormula 是一个 JS 表达式，运行时被包装为 \`return <nameFormula>\`，其中 \`{fieldCode}\` 会被替换为 \`(values.fieldCode || '')\`，\`{applicant.xxx}\` 会被替换为 \`(applicant.xxx || '')\`。
+**规则：所有非 {xxx} 的文本（包括中文、符号、空格）必须用引号包裹，多段用 + 拼接。**
+- 正确：\`"合同名称" + {project_name}\`
+- 正确：\`{applicant_name} + "-" + {project_name}\`
+- 错误：\`合同名称{project_name}\` — JS 语法错误
+- 错误：\`合同名称 + {project_name}\` — JS 语法错误
+- 错误：\`{project_name} 审批单\` — JS 语法错误
+
 ### migrationLog（迁移记录）
 当用户提供了 flow_events、form_script、instance_template 中包含业务逻辑（如校验、赋值、显隐控制等脚本代码）时，必须在 migrationLog 数组中为每条被迁移的逻辑记录一条说明，格式：
   "<来源>: <原逻辑简述> → <迁移到的具体位置>"

package/main/default/routes/api_workflow_next_step.router.js

@@ -310,7 +310,7 @@ const calcSteps = async function(instance, flow, flowVersionId, formFields, valu
 	
 	_steps.push(step);
 	try {
-        var nextSteps = await getNextSteps(flow, flowVersionId, instance, step, judge, values, null, null, instance.submitter);
+        var nextSteps = await getNextSteps(flow, flowVersionId, instance, step, judge, values, null, true, instance.submitter);
         const recursiveResults = await Promise.all(nextSteps.map(async (nextStep) => {
             try {
                 if (!_.includes(track, nextStep._id)) {

package/main/default/routes/flow_form_design.ejs

@@ -628,7 +628,7 @@
           for (const sField of field.fields) {
             if (sField.type != "hidden") {
               sField.permission = field.permission
-              const column = await getFieldEditTpl(sField, true);
+              const column = await getFieldEditTpl(sField, null);
               tpl.columns.push(column);
             }
           }
@@ -641,7 +641,7 @@
           tpl.body = []
           if (field.fields) {
             for (let i = 0; i < field.fields.length; i++) {
-              let fieldsItem = await getFieldEditTpl(field.fields[i], true)
+              let fieldsItem = await getFieldEditTpl(field.fields[i], tableFieldMap)
               tpl.body.push(fieldsItem)
             }
           }

package/main/default/test/FORMULA_SCAN_GUIDE.md

@@ -205,9 +205,12 @@ reports/formula-scan/
 | E3   | ERROR | 转换结果中含裸 `{`（非合法 `${...}` 表达式） |
 | E4   | ERROR | mapFormula 抛出异常 |
 | E5   | ERROR | 聚合函数引用子表字段但转换结果不含对应 ARRAYMAP（子表识别失败） |
+| E6   | ERROR | section 嵌套公式字段设计器兼容性：引擎转换结果与设计器传参不一致 |
 | W1   | WARN  | 公式含除法且除数是字段引用（除零风险） |
 | W2   | WARN  | 同表单内不同字段 getSafeCode 后结果相同（字段名冲突） |
 
+> **E6 说明**：检测 `flow_form_design.ejs` 的 section 子字段是否正确传递 `tableFieldMap`。模拟设计器行为（传 `true`）并与正确结果对比，不一致则报 ERROR。详见 SCAN_HISTORY v9→v10。
+
 ---
 
 ## 报告结构说明

package/main/default/test/reports/formula-scan/SCAN_HISTORY.md

@@ -197,3 +197,60 @@ v8 与 v7 对比完全一致（0 ERROR），说明：
 | KNOWN_ISSUE | 4 | 4 |
 | WARN | 206 | 206 |
 | PASS | 1949 | 1949 |
+
+---
+
+## v9 → v10：Bug 9 — 设计器 section 子字段未传递 tableFieldMap
+
+- **扫描日期**: 2026-03-27
+- **修正前报告**: `scan-result-v9-20260327.json`（6 ERROR, 4 KNOWN_ISSUE, 206 WARN）
+- **修正后报告**: `scan-result-v10-20260327.json`（0 ERROR, 4 KNOWN_ISSUE, 206 WARN）
+- **关联 Issue**: [#538](https://github.com/steedos/steedos-plugins/issues/538)
+- **修改文件**: `flow_form_design.ejs`
+
+### Bug 9 根因
+
+`flow_form_design.ejs` 的 `getFieldEditTpl` 函数中：
+- **section case（第 644 行）**：调用 `getFieldEditTpl(field.fields[i], true)` — 传递布尔值 `true` 而非 `tableFieldMap`
+- **table case（第 631 行）**：调用 `getFieldEditTpl(sField, true)` — 同样传递 `true`
+
+在 `_mapFormula` 内部，`tableFieldMap` 为 `true` 时，`true[fieldCode]` 永远返回 `undefined`，导致子表字段识别失败，走了非子表路径（`getSafeCode` 而非 `ARRAYMAP`）。
+
+> **为什么之前的扫描器没发现**：扫描器自己通过 `getTableFieldMap(fields)` 构建正确的 `tableFieldMap` 传给 `mapFormula`，测试的是公式引擎本身。但设计器的调用链在 section 递归时传参错误，属于**调用者 bug**而非引擎 bug。
+
+### 修复内容
+
+- section case：`getFieldEditTpl(field.fields[i], true)` → `getFieldEditTpl(field.fields[i], tableFieldMap)`
+- table case：`getFieldEditTpl(sField, true)` → `getFieldEditTpl(sField, null)`（子表内字段不需要 tableFieldMap）
+
+### 检测方法（E6 规则，一次性诊断）
+
+为发现此类问题，临时添加 E6 规则到扫描器：
+- 追踪字段是否在 section 内（`inSection` 标志）
+- 对 section 内的公式字段，模拟设计器行为调用 `mapFormula(formula, true)`
+- 比较与正确结果是否一致，不一致则报 ERROR
+
+E6 是一次性诊断规则 — 发现问题后修复设计器，然后移除 E6（因为它测试的是调用者行为而非引擎行为）。但 `inSection` 追踪保留在 `collectFormulaFields` 中，供未来诊断使用。
+
+### 修复了 6 个 E6 ERROR
+
+| # | 表单名称 | formId | 字段 | 原始公式 | 版本 |
+|---|---------|--------|------|---------|------|
+| 1 | 【公司本部】抚顺石化公司车辆维修审批单 | `9a40a1a2049014ff074a4961` | 配件金额 | `{数量}*{单价}` | history |
+| 2 | 【乙烯部】抚顺石化公司车辆维修审批单 | `2a36450a56ecd64af98506ba` | 配件金额 | `{数量}*{单价}` | history |
+| 3 | 【公司办公室】抚顺石化公司车辆维修费用结算审批单 | `2c3a2c7cbbf618d95a40376b` | 配件金额 | `{数量}*{单价}` | history |
+| 4 | 抚顺石化公司物资采购概预算调整审批单 | `cRJknoGMcLa2raizW` | 拟调整概预算金额合计 | `sum({拟调整概预算金额})` | **current** |
+| 5 | 抚顺石化公司物资采购概预算调整审批单 | `cRJknoGMcLa2raizW` | 拟调整概预算总金额 | `sum({拟调整概预算金额})` | history |
+| 6 | 抚顺石化公司化工三剂采购概预算调整审批单 | `17d3b669d8880cd32b89ebfb` | 拟调整概预算金额合计 | `sum({拟调整概预算金额（元）})` | **current** |
+
+> **影响面**: 4 个 history + 2 个 **current**（#4 和 #6 影响在用表单的设计器预览）
+
+### 扫描统计变化
+
+| 指标 | v8 | v9（修正前） | v10（修正后） |
+|------|-----|------------|-------------|
+| ERROR | 0 | 6 | 0 |
+| KNOWN_ISSUE | 4 | 4 | 4 |
+| WARN | 206 | 206 | 206 |
+| PASS | 1949 | 1943 | 1949 |
+| current ERROR | 0 | 2 | 0 |