Archived
审批点后面补上网关 G3,并让引擎**真的走它** —— 顺带给去向旗找到落脚处
两件事,都是「图上画着、实际走不到」这一类。 ## ① 审批点后面没有网关(用户提出) A5 只有一条出边 A5→A6,驳回走向藏在 `reject` 声明里,图上只是一枚牌子。 BPMN 的常规是审批的两个去向都由网关决定 —— 补 G3「审批结果」, `verdict = "pass"` → A6,`verdict = "reject"` → A4(回边)。 补的时候撞上真问题:`onApprovalVerdict` 只在 pass 时走 `advanceFrom`, 而且 payload 里**不含 verdict** —— 于是 G3 没有判据可读。异或分叉的每条出边都得带条件、 没有无条件兜底边,判据一缺就全不成立,流会无声停在 G3。 能让闸过的最省事办法是在 `e-a5-g3` 上喂一枚常量 `verdict: "pass"`。**没那么做** —— 那样 G3 的条件恒真、驳回边永不可达,网关就是个摆设,又是一处「画着两个去向、 其中一个结构上走不到」的谎。 改的是引擎:裁决进载荷,通过与驳回**都先看图**,只有没画驳回边的包才退回 `reject` 声明 (它 schema 必填,且是通用机制)。`advanceFrom` 因此回报「有没有真的推出去」, 并加 `quiet` 参数 —— 「先试边、不成再退回声明」这一步试空是正常分支,不是死路, 不该报 route-dead-end 假警。 `reject` 保留:schema 必填、CONTRACT.md 的不变量、且审批人界面拿它渲染人读文案。 它与 `e-g3-a4` 必须字字对齐,加了测试守。 ## ② 几何重排:两带换行,不回形 原坐标逐点抄自 MVP 的 SVG,是回形走线(row 0 左→右、row 1 右→左、row 2 折回最左)—— 这正是上一轮流程链读成 A1→A7→A2 的根源。重排为两带: 受理·选型·寻源(y 60..100)→ 换行车道 y=190 → 定价·审批·议价·成交(y 310..350)→ 未成交闭环(y 410..450)。每带内 x 随推进次序单调递增。 实测渲染(画布 639×325):带 0 的 A1 57→G1 159→A2 223→G2 325→A3 389→G2J 491, 带 1 的 A4 38→A5 153→G3 242→A6 293→G4 395,**零框重叠**。 两枚回 A4 的卡点同一条车道、同朝向(向上=回退)。 ## ③ 顺带:两枚去向旗终于有地方落 上一轮我在债务簿写「A1 过不了 G1 是诚实状态、不是缺陷」——**判断错了**。 出口的标量字段只从落地事实取,而 `needAlternative` / `needSourcing` 在任何台账对象上都没有列,活动实现了也一样取不到。 新增 `project.intake_verdict`「受理结论」**一单一行**(主键只有询价单号)。 为什么不挂在按料号分行的「统一受理记录」上:`flat` 是把这一单所有落地行 reduce 展开, 六个料号就有六行同名列、最后合并的那行赢 —— 而归并规则是「任一条命中 C 即需选型」, last-write-wins 会静默给出错的去向。测试正反两面都守:结论表主键只能是 rfq_ref, 且**其它对象上不许出现去向旗**。 A2 也要回写这一行,所以 `rfqRef` 进了 G1→A2 的边映射,并**正式声明进 A2 的入参**—— 不声明也能偷渡(闭合校验遇到未声明的目标直接跳过),但那是默许不是正当。 A2 的 contract.version 因此升 0.2.0,包版升 0.5.0。 顺手删了一个陈旧草稿 v2:它是我早前测编辑器留下的旧图(14 节点、无 G3), 而种子刚把 v1 用新图重发布 —— 草稿内容比已发布版更旧,谁发布它 G3 就没了。 删前确认零工作流引用。系统没有「丢弃草稿」的入口,这条记进债务簿。 typecheck ✓ lint ✓ depcruise 374 模块 ✓ api:check ✓ 495 测试 ✓ 发布闸实测:15 节点 / 18 边,边映射闭合校验通过。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KeDBx2PHLavbURXB8LJJ7m
This commit is contained in:
@@ -173,13 +173,22 @@ export function createProcessService(deps: ProcessServiceDeps): ProcessService {
|
||||
await deps.events.publish({ workflowId: record.id, kind: 'approval-raised', detail: { nodeId: node.id, approvalId: approval.id } });
|
||||
}
|
||||
|
||||
/** 从某节点继续流转:选边 → 装配 → 进下一节点(AND 汇合要回齐) */
|
||||
/**
|
||||
* 从某节点继续流转:选边 → 装配 → 进下一节点(AND 汇合要回齐)。
|
||||
*
|
||||
* 返回**有没有真的推出去**。审批点的驳回要用它:图上显式画了驳回出边就走边,
|
||||
* 没画的包才退回 `reject` 声明 —— 调用方据此判断,不必自己再选一遍边。
|
||||
*
|
||||
* `quiet` 为真时不发 `route-dead-end`:那种「先试边、不成再退回声明」的探路
|
||||
* 一条边都没中是**正常分支**,不是死路,报出来就是假警。
|
||||
*/
|
||||
async function advanceFrom(
|
||||
record: WorkflowRecord,
|
||||
pkg: ProcessPackage,
|
||||
fromNodeId: string,
|
||||
payload: Record<string, unknown>,
|
||||
): Promise<void> {
|
||||
quiet = false,
|
||||
): Promise<boolean> {
|
||||
const scope = scopeOf(record);
|
||||
const flat = { ...scope.initiation, ...scope.vars, ...payload };
|
||||
const edges = selectEdges(pkg, fromNodeId, flat, deps.sfeel.condition);
|
||||
@@ -196,7 +205,7 @@ export function createProcessService(deps: ProcessServiceDeps): ProcessService {
|
||||
*/
|
||||
if (edges.length === 0) {
|
||||
const outgoing = pkg.edges.filter((e) => e.from === fromNodeId);
|
||||
if (outgoing.length > 0) {
|
||||
if (outgoing.length > 0 && !quiet) {
|
||||
await deps.events.publish({
|
||||
workflowId: record.id,
|
||||
kind: 'route-dead-end',
|
||||
@@ -255,6 +264,7 @@ export function createProcessService(deps: ProcessServiceDeps): ProcessService {
|
||||
});
|
||||
}
|
||||
}
|
||||
return edges.length > 0;
|
||||
}
|
||||
|
||||
return {
|
||||
@@ -429,16 +439,38 @@ export function createProcessService(deps: ProcessServiceDeps): ProcessService {
|
||||
}
|
||||
await deps.events.publish({ workflowId, kind: 'approval-verdict', detail: { nodeId, verdict } });
|
||||
|
||||
/*
|
||||
* **裁决进载荷**:审批点后面跟网关时,网关判的就是这个字段
|
||||
* (如 G3「审批结果」的 `verdict = "pass"` / `verdict = "reject"`)。
|
||||
*
|
||||
* 从前只有 pass 走 `advanceFrom`,而且 payload 里不含 `verdict` —— 于是审批点后面
|
||||
* 就算画了网关也判不了(异或分叉的每条出边都得带条件、没有无条件兜底边,
|
||||
* 判据一缺就全不成立,流无声停在网关上)。要让它判得了,只能在边上挂一枚
|
||||
* 常量 `verdict: "pass"` 骗过条件 —— 那网关就成了摆设:条件恒真、驳回那条边永不可达。
|
||||
* 图上画着两个去向、其中一个结构上走不到,是在说谎。
|
||||
*/
|
||||
const carried = { ...payload, verdict };
|
||||
|
||||
if (verdict === 'pass') {
|
||||
await advanceFrom(record, pkg, nodeId, payload);
|
||||
await advanceFrom(record, pkg, nodeId, carried);
|
||||
} else {
|
||||
const route = rejectRouteOf(node, scopeOf(record), deps.sfeel.expression);
|
||||
if (route.kind === 'terminate') {
|
||||
await deps.store.updateWorkflow(workflowId, { state: 'terminated', finishedAt: deps.clock.now().toISOString() });
|
||||
} else {
|
||||
// 回指定节点**重开新实例**(同节点、新任务 ID、留痕关联),不复活旧任务
|
||||
const target = pkg.nodes.find((n) => n.id === route.nodeId);
|
||||
if (target !== undefined) await enter(record, pkg, target, { ...route.payload, ...payload });
|
||||
/*
|
||||
* 驳回:**先看图**。图上显式画了驳回出边(审批点后面跟网关)就走边 —— 编排即定契约,
|
||||
* 图是真相。没画的包才退回 `reject` 声明:它是 schema 必填的通用机制,
|
||||
* 不是所有流程包都会把驳回画成边。
|
||||
*
|
||||
* `quiet` 传真:这一步「试边不成」是正常分支,不是死路,不该报 route-dead-end。
|
||||
*/
|
||||
const routedByEdge = await advanceFrom(record, pkg, nodeId, carried, true);
|
||||
if (!routedByEdge) {
|
||||
const route = rejectRouteOf(node, scopeOf(record), deps.sfeel.expression);
|
||||
if (route.kind === 'terminate') {
|
||||
await deps.store.updateWorkflow(workflowId, { state: 'terminated', finishedAt: deps.clock.now().toISOString() });
|
||||
} else {
|
||||
// 回指定节点**重开新实例**(同节点、新任务 ID、留痕关联),不复活旧任务
|
||||
const target = pkg.nodes.find((n) => n.id === route.nodeId);
|
||||
if (target !== undefined) await enter(record, pkg, target, { ...route.payload, ...payload });
|
||||
}
|
||||
}
|
||||
}
|
||||
return this.refreshState(workflowId);
|
||||
|
||||
Reference in New Issue
Block a user