Skip to content

Stabilize move pill CI timing - #4

Merged
jiangsir-tech merged 1 commit into
mainfrom
codex/stabilize-move-pill-ci
Aug 12, 2026
Merged

Stabilize move pill CI timing#4
jiangsir-tech merged 1 commit into
mainfrom
codex/stabilize-move-pill-ci

Conversation

@jiangsir-tech

Copy link
Copy Markdown
Owner

What changed

  • Keeps the strict assertion that the move pill must not appear during the first 50 ms of its 140 ms debounce.
  • Replaces the fixed additional 150 ms sleep with an eventual assertion capped at 1 second.

Why

The exact 1.4.71 merge commit passed both PR workflows, but the repeated main workflow exposed this pre-existing fixed-timing test as flaky under runner load. It failed twice on Xcode 15.4 despite the product behavior being unchanged, while every cursor-specific test and all other jobs passed.

This change affects tests only; no app source or release version changes.

Validation

  • Repeated targeted test: 10/10 passed
  • Full local suite: 599 passed, 0 failed
  • Product release cursor tests remain unchanged

@jiangsir-tech
jiangsir-tech marked this pull request as ready for review August 12, 2026 03:05
@jiangsir-tech
jiangsir-tech merged commit e243678 into main Aug 12, 2026
8 checks passed
@jiangsir-tech
jiangsir-tech deleted the codex/stabilize-move-pill-ci branch August 12, 2026 03: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.

1 participant