Fix execution log order and move waiting states by philippfromme · Pull Request #91 · camunda/task-testing

philippfromme · 2026-03-26T09:30:23Z

Proposed Changes

fix execution log order by using causal ordering for state entries and related entries with same timestamp
move waiting states out of log so they are always visible when existing

Example: Execution log order

deployed and instance started always come first
element instance created comes before any of the work (e.g. job) is created
work completed comes before element instance completed
start execution listener job comes before regular job and so on
finished state always comes last

Example: Waiting state always visible and includes identifier (`Form`)

Related to #81 (comment)

Checklist

Ensure you provide everything we need to review your contribution:

Your contribution meets the definition of done
Any new additions or modifications are consistent with the existing UI and UX patterns
Pull request description establishes context:
- Link to related issue(s), i.e. Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}
- Brief textual description of the changes
- Screenshots or short videos showing UI/UX changes
- Steps to try out, i.e. using the @bpmn-io/sr tool

Sort log entries not only by timestamp but also by causal priority when timestamps are equal. This ensures entries appear in a logically correct sequence even when the engine produces identical timestamps for related lifecycle events. Causal ordering is always applied for global status entries and only between related entries (same elementId or identity key) for runtime entries, preserving insertion order for unrelated entries.

Reverts #88 which was a fix for only one of the ordering issues.

Adds call activity to example

Introduce an `ENTRY_ORDER` enum to replace bare numeric literals (0–10) in `getEntryOrder`, making the causal ordering of execution log entries self-documenting.

Related to camunda/camunda-modeler#5805

philippfromme · 2026-03-30T11:49:34Z

Ready for final review.

Buckwich · 2026-03-30T13:24:10Z

+    // order.
+    this._entries = entries
+      .map((entry, index) => /** @type {[number, ExecutionLogEntry]} */ ([ index, entry ]))
+      .sort(([ indexA, entryA ], [ indexB, entryB ]) => {


could be a bit simpler. not sure whats better readable:

Suggested change

.sort(([ indexA, entryA ], [ indexB, entryB ]) => {

.sort(([ indexA, entryA ], [ indexB, entryB ]) => {

if (isStatusEntry(entryA) || isStatusEntry(entryB)) {

return compareEntryOrders(entryA, entryB);

}

return compareEntryTimestamps(entryA, entryB)

|| (areEntriesRelated(entryA, entryB) ? compareEntryOrders(entryA, entryB) : 0)

|| indexA - indexB;

})

bpmn-io-tasks Bot added the in progress Currently worked on label Mar 26, 2026

bpmn-io-tasks Bot assigned philippfromme Mar 26, 2026

philippfromme force-pushed the order branch 3 times, most recently from eccae4a to 2b87ebc Compare March 26, 2026 10:43

philippfromme marked this pull request as ready for review March 26, 2026 12:55

bpmn-io-tasks Bot added needs review Review pending and removed in progress Currently worked on labels Mar 26, 2026

philippfromme requested review from barmac, jarekdanielak and nikku March 26, 2026 12:58

Buckwich mentioned this pull request Mar 27, 2026

Release camunda-modeler v5.46.0 camunda/camunda-modeler#5726

Closed

52 tasks

nikku reviewed Mar 27, 2026

View reviewed changes

Comment thread lib/ExecutionLog.js Outdated

philippfromme force-pushed the order branch 3 times, most recently from 0110486 to 13e79e7 Compare March 28, 2026 13:08

philippfromme marked this pull request as draft March 30, 2026 08:15

bpmn-io-tasks Bot added in progress Currently worked on and removed needs review Review pending labels Mar 30, 2026

philippfromme added 5 commits March 30, 2026 10:36

fix(log): do not show active entries for completed items

7a99de3

chore(log): remove timestamp override in favor of causal sorting

c5528c7

Reverts #88 which was a fix for only one of the ordering issues.

chore: update example

2eb9bd1

Adds call activity to example

refactor(log): replace magic numbers with named constants

b3448c0

Introduce an `ENTRY_ORDER` enum to replace bare numeric literals (0–10) in `getEntryOrder`, making the causal ordering of execution log entries self-documenting.

philippfromme force-pushed the order branch from 7b538c5 to 837c825 Compare March 30, 2026 08:38

philippfromme changed the title ~~Improve ordering of log entries~~ Fix execution log order and always show waiting states Mar 30, 2026

philippfromme force-pushed the order branch from 837c825 to ccaf80d Compare March 30, 2026 10:59

philippfromme requested a review from Copilot March 30, 2026 10:59

Copilot started reviewing on behalf of philippfromme March 30, 2026 10:59 View session

This comment was marked as outdated.

Sign in to view

philippfromme added 2 commits March 30, 2026 13:19

fix(log): show waiting states outside of log

d3dd021

Related to camunda/camunda-modeler#5805

fix: clarify user task instance

f54f2a3

philippfromme force-pushed the order branch from ccaf80d to f54f2a3 Compare March 30, 2026 11:19

philippfromme requested a review from Copilot March 30, 2026 11:19

Copilot started reviewing on behalf of philippfromme March 30, 2026 11:20 View session

philippfromme changed the title ~~Fix execution log order and always show waiting states~~ Fix execution log order and move waiting states Mar 30, 2026

This comment was marked as outdated.

Sign in to view

philippfromme added 2 commits March 30, 2026 13:25

chore(CHANGELOG): update

beab61a

fix: use insertion index as final tiebreaker

064a16b

philippfromme marked this pull request as ready for review March 30, 2026 11:35

bpmn-io-tasks Bot added needs review Review pending and removed in progress Currently worked on labels Mar 30, 2026

philippfromme requested review from Buckwich and nikku March 30, 2026 11:49

Buckwich reviewed Mar 30, 2026

View reviewed changes

Buckwich approved these changes Mar 30, 2026

View reviewed changes

philippfromme added 2 commits March 30, 2026 17:41

fix(log): consider containment order

10e3887

chore: simplify

f74fd56

philippfromme merged commit ec64b38 into main Mar 30, 2026
5 checks passed

philippfromme deleted the order branch March 30, 2026 17:31

bpmn-io-tasks Bot removed the needs review Review pending label Mar 30, 2026

philippfromme mentioned this pull request Mar 31, 2026

feat: update to @camunda/task-testing@4 camunda/camunda-modeler#5821

Merged

6 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix execution log order and move waiting states#91

Fix execution log order and move waiting states#91
philippfromme merged 11 commits intomainfrom
order

philippfromme commented Mar 26, 2026 •

edited

Loading

Uh oh!

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

philippfromme commented Mar 30, 2026

Uh oh!

Buckwich Mar 30, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

-      .sort(([ indexA, entryA ], [ indexB, entryB ]) => {
+      .sort(([ indexA, entryA ], [ indexB, entryB ]) => {
+        if (isStatusEntry(entryA) || isStatusEntry(entryB)) {
+          return compareEntryOrders(entryA, entryB);
+        }
+        return compareEntryTimestamps(entryA, entryB)
+          || (areEntriesRelated(entryA, entryB) ? compareEntryOrders(entryA, entryB) : 0)
+          || indexA - indexB;
+      })

Conversation

philippfromme commented Mar 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Proposed Changes

Example: Execution log order

Example: Waiting state always visible and includes identifier (Form)

Checklist

Uh oh!

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

philippfromme commented Mar 30, 2026

Uh oh!

Buckwich Mar 30, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

philippfromme commented Mar 26, 2026 •

edited

Loading

Example: Waiting state always visible and includes identifier (`Form`)