@bpmnkit/plugins 0.0.11 → 0.0.13

package/dist/token-highlight/index.js

@@ -53,22 +53,63 @@ export function createTokenHighlightPlugin() {
             }
         }
         // Edge highlights
-        // • active edge  : source visited, target active  (token just left source, entering target)
-        // • visited edge : source visited, target visited (token has fully traversed this edge)
-        // This correctly handles exclusive gateways: only the taken branch's target enters
-        // visited/active, so only the taken edge is highlighted.
-        for (const [flowId, { sourceRef, targetRef }] of flowIndex) {
-            const el = edgeEl(flowId);
-            if (el === undefined)
-                continue;
-            const srcVisited = visitedIds.has(sourceRef);
-            if (!srcVisited)
-                continue;
-            if (activeIds.has(targetRef)) {
-                el.classList.add("bpmnkit-token-edge-active");
+        // Prefer direct sequence-flow tracking: Camunda's element-instances API returns
+        // sequence flow element instances (with their flow ID), so visitedIds/activeIds
+        // will contain the actual flow IDs that were traversed.
+        // Fallback to source/target heuristic only when no flow IDs are present (older
+        // engines or environments that don't track sequence flows as element instances).
+        // The heuristic is deliberately disabled when direct tracking is available because
+        // it over-highlights: when a gateway's default path leads to a node that was
+        // reached via a different branch, both edges appear highlighted incorrectly.
+        let hasFlowTracking = false;
+        for (const flowId of flowIndex.keys()) {
+            if (visitedIds.has(flowId) || activeIds.has(flowId)) {
+                hasFlowTracking = true;
+                break;
             }
-            else if (visitedIds.has(targetRef)) {
-                el.classList.add("bpmnkit-token-edge-visited");
+        }
+        if (hasFlowTracking) {
+            // Direct mode: flow IDs are in visitedIds/activeIds — highlight exactly what was traversed.
+            for (const [flowId] of flowIndex) {
+                const el = edgeEl(flowId);
+                if (el === undefined)
+                    continue;
+                if (activeIds.has(flowId)) {
+                    el.classList.add("bpmnkit-token-edge-active");
+                }
+                else if (visitedIds.has(flowId)) {
+                    el.classList.add("bpmnkit-token-edge-visited");
+                }
+            }
+        }
+        else {
+            // Heuristic fallback for engines that don't return sequence-flow element instances.
+            //
+            // "Unique winner" rule: group outgoing flows by source and only highlight an
+            // edge when it is the SOLE outgoing flow from that source whose target is
+            // visited/active. If multiple candidates exist we cannot determine which path
+            // was actually taken (e.g. both branches of an exclusive gateway converge on
+            // the same downstream node), so we highlight none to avoid false positives.
+            const bySource = new Map();
+            for (const [flowId, { sourceRef, targetRef }] of flowIndex) {
+                if (!visitedIds.has(sourceRef))
+                    continue;
+                const isActive = activeIds.has(targetRef);
+                if (!isActive && !visitedIds.has(targetRef))
+                    continue;
+                const list = bySource.get(sourceRef) ?? [];
+                list.push({ flowId, isActive });
+                bySource.set(sourceRef, list);
+            }
+            for (const candidates of bySource.values()) {
+                if (candidates.length !== 1)
+                    continue;
+                for (const { flowId, isActive } of candidates) {
+                    const el = edgeEl(flowId);
+                    if (el === undefined)
+                        continue;
+                    el.classList.add(isActive ? "bpmnkit-token-edge-active" : "bpmnkit-token-edge-visited");
+                }
             }
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bpmnkit/plugins",
-  "version": "0.0.11",
+  "version": "0.0.13",
   "private": false,
   "type": "module",
   "exports": {
@@ -104,11 +104,11 @@
     "dist/**/*.d.ts"
   ],
   "dependencies": {
-    "@bpmnkit/ascii": "0.0.11",
-    "@bpmnkit/canvas": "0.0.11",
-    "@bpmnkit/core": "0.0.11",
-    "@bpmnkit/editor": "0.0.11",
-    "@bpmnkit/feel": "0.0.11"
+    "@bpmnkit/ascii": "0.0.13",
+    "@bpmnkit/canvas": "0.0.13",
+    "@bpmnkit/core": "0.0.13",
+    "@bpmnkit/editor": "0.0.13",
+    "@bpmnkit/feel": "0.0.12"
   },
   "description": "22 composable canvas plugins for BPMN editors and viewers — minimap, AI chat, process simulation, storage, and more",
   "keywords": [