Nectenda/nectenda

Watch

Fork 0

Give the 20,000-run text-merge property test 30 s #256

Merged

cruelacid merged 1 commit from worktree-text-merge-timeout into main

2026-10-02 14:38:17 +01:00

cruelacid commented

2026-10-02 14:28:33 +01:00

Owner

text-merge.test.ts, "keeps every typed run and every collaborator's character, whatever the timing", timed out at vitest's default 5 s in the build job of main run 1205. That run was on the 0.4.1 release commit (adb365e), which touched no code.

The test runs 20,000 seeded cases. It takes about 1.3 s on a Mac and took 5.6 s on the runner. Because it is seeded, its result cannot vary; only its time can. It now has 30 s, with a comment saying why. The change is test-only.

Verification

pnpm test, pnpm -r typecheck, pnpm lint and build-mirror --check all pass.
The publish dry run for 0.4.1 on adb365e passed meanwhile: full container e2e 160 passed, 2 skipped; the previous-release rehearsal passed 24/24.

Changelog

NONE

🤖 Generated with Claude Code

https://claude.ai/code/session_01AsUFqoi4GHKYjFtPbYCgRp

`text-merge.test.ts`, "keeps every typed run and every collaborator's character, whatever the timing", timed out at vitest's default 5 s in the `build` job of main run 1205. That run was on the 0.4.1 release commit (`adb365e`), which touched no code. The test runs 20,000 seeded cases. It takes about 1.3 s on a Mac and took 5.6 s on the runner. Because it is seeded, its result cannot vary; only its time can. It now has 30 s, with a comment saying why. The change is test-only. ## Verification - `pnpm test`, `pnpm -r typecheck`, `pnpm lint` and `build-mirror --check` all pass. - The publish dry run for 0.4.1 on `adb365e` passed meanwhile: full container e2e 160 passed, 2 skipped; the previous-release rehearsal passed 24/24. ## Changelog NONE 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01AsUFqoi4GHKYjFtPbYCgRp

cruelacid added the

area:ops

label

2026-10-02 14:28:33 +01:00

cruelacid added 1 commit

2026-10-02 14:28:33 +01:00

Give the 20,000-run text-merge property test 30 s

Release note / release-note (pull_request) Successful in 20s

Details

CI / build (pull_request) Successful in 7m34s

Details

CI / e2e (pull_request) Successful in 9m31s

Details

CI / promote (pull_request) Has been skipped

Details

CI / build (push) Successful in 7m27s

Details

CI / e2e (push) Successful in 9m26s

Details

CI / promote (push) Successful in 50s

Details

985ca0c3ae

It takes about 1.3 s on a Mac and passed the default 5 s on a busy CI runner,
failing main run 1205 for a release commit that touched no code. The test is
seeded, so its result cannot vary; only its time can.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AsUFqoi4GHKYjFtPbYCgRp

cruelacid merged commit 985ca0c3ae into main

2026-10-02 14:38:17 +01:00

No reviewers

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

Nectenda/nectenda!256

No description provided.

Rows
Columns