littlefox/neokom

Watch

Flaky: commands.spec "skip a comment tree", the comment read by being seen #259

New issue

Closed

opened 2026-10-22 00:23:39 +00:00 by littlefox · 1 comment

littlefox commented

2026-10-22 00:23:39 +00:00

Owner

CI run #413 (PR #255): "Hoppade över svaren på 89 (0 markerade som lästa)", expected 1. On a slow runner the comment's card was on screen for over a second before "Hoppa över svaren", so it was read by being seen (#180) and the skip had nothing to mark. Same race as #249. Fix: turn read-by-seen off in this test (window.__neokomSeenOff).

CI run #413 (PR #255): "Hoppade över svaren på 89 (0 markerade som lästa)", expected 1. On a slow runner the comment's card was on screen for over a second before "Hoppa över svaren", so it was read by being seen (#180) and the skip had nothing to mark. Same race as #249. Fix: turn read-by-seen off in this test (`window.__neokomSeenOff`).

littlefox added this to the 0.6: Performance milestone

2026-10-22 00:23:39 +00:00

littlefox added the

labels

2026-10-22 00:23:39 +00:00

littlefox referenced this issue from a commit

2026-10-22 00:23:44 +00:00

commands.spec: no read-by-seen while skipping a comment tree

littlefox referenced this issue from a pull request that will close it,

2026-10-22 00:23:53 +00:00

Read on in a thread from where reading is, not from its root #255

littlefox closed this issue

2026-10-22 01:50:51 +00:00

littlefox commented

2026-10-22 02:26:34 +00:00

Author

Owner

Fixed in #255 (main #429 green): the skip-tree test turns read-by-seen off.

Fixed in #255 (main #429 green): the skip-tree test turns read-by-seen off.

Sign in to join this conversation.

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

littlefox/neokom#259

No description provided.