Skip to content

fix(e2e): the keyboard-scroll test notes the row in view after the end row moves it - #1007

Merged
V3RON merged 2 commits into
mainfrom
bug/1006
Oct 25, 2026
Merged

V3RON merged 2 commits into
mainfrom
bug/1006

Conversation

@V3RON

@V3RON V3RON commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Closes #1006

Status

Implement: done (1/1 green) Review: round 1 passed, 0 findings Hardware: n/a Gate: running
Next: gate

Done when

  • The keyboard-scroll test passes on every run of an unchanged commit. — the test holds the older-page answer until the feed is at its end, so the late end row happens on every run; 5/5 repeats in audit and --repeat-each 8 in review passed; noting the row before the release fails with the issue's -51 vs -37

Assumptions

none

Review

Spec review: 0 blocking, 0 fixed, 0 notes. Code review: 0 blocking, 0 fixed, 0 notes. Claims review: 0 blocking, 0 fixed, 0 notes.
Rounds: 1. Tokens: 4.6M.
Runs: 6. Cost USD: 0.9474. Agent seconds: 623.

Assessment: complete

Written by an agent.

@V3RON

V3RON commented Oct 25, 2026

Copy link
Copy Markdown
Contributor Author

Review ledger

Round 1: 584719d

Runs: 6. Cost USD: 0.9474. Agent seconds: 623.
Tokens input 102, output 19301, cache read 1236155, cache write 180190.
- build implementer: claude-sonnet-5-5 173 s $0.1269
- audit implementer: claude-sonnet-5-5 76 s $0.1175
- review 1 reviewer: claude-sonnet-5-5 214 s $0.1986
- review 1 spec: claude-sonnet-5-5 16 s $0.0854
- review 1 code tests: claude-opus-5-5 131 s $0.3603
- review 1 claims: claude-sonnet-5-5 13 s $0.0587
Finding outcomes: {}. Classes: {}.
{"findings":[]}

Written by an agent.

@V3RON
V3RON marked this pull request as ready for review October 25, 2026 14:02
@V3RON
V3RON merged commit 5a4969d into main Oct 25, 2026
65 checks passed
@V3RON
V3RON deleted the bug/1006 branch October 25, 2026 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flaky: the Activity feed keyboard-scroll test reads the wrong first row in view

1 participant