android-midscene-automation 0.1.33 → 0.1.34

package/CHANGELOG.md

@@ -1,4 +1,8 @@
 # 更新记录
+## v0.1.34
+- 修复同一分支内显式跳转被画布误画为顺序连接的问题，失效跳转目标不再错误连接到相邻节点。
+- 修复判断分支默认返回公共后续节点时画布漏连线、节点重叠的问题，嵌套分支显示与回放路径保持一致。
+
 ## v0.1.33
 - 操作菜单移除“断言存在”和“断言文本”，统一保留“判断存在”及其文本匹配配置；旧脚本中的断言节点仍兼容原有执行行为。
 - “点击坐标”节点支持编辑 X、Y 坐标，保存后回放使用新坐标；默认节点名称同步更新，自定义名称保持不变。

package/README.md

@@ -1229,6 +1229,10 @@ Linux: ~/.local/share/android-midscene-automation
 
 # 项目更新记录
 
+## v0.1.34
+- 修复同一分支内显式跳转被画布误画为顺序连接的问题，失效跳转目标不再错误连接到相邻节点。
+- 修复判断分支默认返回公共后续节点时画布漏连线、节点重叠的问题，嵌套分支显示与回放路径保持一致。
+
 ## v0.1.33
 - 操作菜单移除“断言存在”和“断言文本”，统一保留“判断存在”及其文本匹配配置；旧脚本中的断言节点仍兼容原有执行行为。
 - “点击坐标”节点支持编辑 X、Y 坐标，保存后回放使用新坐标；默认节点名称同步更新，自定义名称保持不变。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "android-midscene-automation",
-  "version": "0.1.33",
+  "version": "0.1.34",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/oooooooko/android-midscene-automation.git"

package/src/appium-recorder/flow-graph.ts

@@ -233,6 +233,24 @@ function addUniqueLink(links: Link[], seen: Set<string>, link: Link) {
   links.push(link);
 }
 
+// 与回放的默认分支返回一致；循环体的虚拟回边不参与有向无环布局。
+function continuationTargetId(items: StepItem[], step: AppiumRecordedStep, seen = new Set<string>()): string {
+  if (seen.has(step.id)) return '';
+  seen.add(step.id);
+  if (step.flow?.successTargetId) return step.flow.successTargetId;
+  if (defaultKind(step) === 'condition') {
+    const index = items.findIndex(item => item.step.id === step.id);
+    const sibling = items.slice(index + 1).find(({ step: candidate }) => (
+      candidate.flow?.parentConditionId === step.flow?.parentConditionId
+      && candidate.flow?.parentBranch === step.flow?.parentBranch
+    ));
+    if (sibling) return sibling.step.id;
+  }
+  const parent = items.find(item => item.step.id === step.flow?.parentConditionId)?.step;
+  if (!parent || (parent.type === 'loop' && step.flow?.parentBranch === 'yes')) return '';
+  return continuationTargetId(items, parent, seen);
+}
+
 function branchConnectionTargetId(items: StepItem[], condition: AppiumRecordedStep, branch: FlowBranch) {
   const branchItems = directBranchItems(items, condition.id, branch);
   if (branchItems.length) {
@@ -240,6 +258,7 @@ function branchConnectionTargetId(items: StepItem[], condition: AppiumRecordedSt
     return defaultKind(last) === 'condition' ? '' : last.flow?.successTargetId || '';
   }
   const targetId = branch === 'yes' ? condition.flow?.yesTargetId : condition.flow?.noTargetId;
+  if (!targetId) return condition.type === 'loop' && branch === 'yes' ? '' : continuationTargetId(items, condition);
   const target = items.find(({ step }) => step.id === targetId)?.step;
   return target && target.flow?.parentConditionId === condition.flow?.parentConditionId ? target.id : '';
 }
@@ -547,11 +566,8 @@ export function buildFlowGraph(
 
     const targetId = item.step.flow?.successTargetId;
     const explicitTarget = targetId ? stepById.get(targetId) : undefined;
-    const explicitMainTarget = explicitTarget && (!explicitTarget.step.flow?.parentConditionId
-      || explicitTarget.step.flow.parentConditionId !== item.step.flow?.parentConditionId)
-      ? explicitTarget
-      : undefined;
-    const target = explicitMainTarget || nextSibling;
+    // 显式跳转优先于顺序关系，包括同一分支内的跳转；失效目标不能悄悄改走相邻节点。
+    const target = targetId ? explicitTarget : nextSibling || stepById.get(continuationTargetId(items, item.step));
     const insertId = addInsertNode(
       item.index,
       branch,
@@ -559,10 +575,6 @@ export function buildFlowGraph(
     );
     addVisibleLink(nodeIdForStep(item.step), insertId, branch);
     if (!target) return;
-    if (branch && explicitMainTarget) {
-      addVisibleLink(insertId, nodeIdForStep(target.step), branch);
-      return;
-    }
     addVisibleLink(insertId, nodeIdForStep(target.step), branch);
   }
 
@@ -742,7 +754,7 @@ export function buildFlowGraph(
   // 公共起点从两侧汇入，不能被最后一条分支连线拉到左列或右列。
   const joins = new Map<string, string>();
   items.forEach(({ step }) => {
-    const target = step.flow?.successTargetId && stepById.get(step.flow.successTargetId);
+    const target = defaultKind(step) === 'condition' && stepById.get(continuationTargetId(items, step));
     if (defaultKind(step) === 'condition' && target && !joins.has(nodeIdForStep(target.step))) {
       joins.set(nodeIdForStep(target.step), nodeIdForStep(step));
     }