Skip to content

Fix two flaky CI test failures - #990

Merged
theoephraim merged 2 commits into
mainfrom
fix-flaky-test-timeouts
Aug 8, 2026
Merged

Fix two flaky CI test failures#990
theoephraim merged 2 commits into
mainfrom
fix-flaky-test-timeouts

Conversation

@theoephraim

Copy link
Copy Markdown
Member

Two unrelated tests flaked in CI on 2026-08-07, both test-only fixes:

  • nextjs fifo-sources: "never reads or watches FIFO sources..." hit vitest's default 5s timeout (5083ms) on a slow runner; the first import of next-env-compat pays the transform cost. Both tests in the suite now have an explicit 30s timeout.
  • varlock cache-lock-crossproc: the SIGINT test asserted the lock dir was gone immediately after observing the child's exit. bun run can interpose a wrapper process, so the observed exit can race the actual holder's cleanup. The assertion now polls for up to 5s before failing.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

bumpy-frog

The changes in this PR will be included in the next version bump.

Bump files in this PR

Click here if you want to add another bump file to this PR


This comment is maintained by bumpy.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

📦 Bundle size

no change to bundle size

Metric main This PR Δ
Total dist 5010.6 KB 5010.6 KB
JS 1717.9 KB 1717.9 KB
Sourcemaps 3216.2 KB 3216.2 KB
Type defs 76.6 KB 76.6 KB

dist/ only; native binaries are versioned separately and not counted here.

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes covering the test stabilization updates for the Next.js FIFO suite and Varlock cross-process cache-lock suite.

  • FIFO test timeouts: Gives both FIFO source tests a 30 second budget for slow initial transforms while preserving their existing behavioral assertions.
  • SIGINT cleanup polling: Waits for the process holder's lock cleanup after wrapper exit while retaining the exact final assertion that the lock directory is gone.
  • Release metadata: Records test-only changes with no package version bumps.

Pullfrog  | View workflow run | Using azure/gpt-5.6-sol𝕏

@theoephraim
theoephraim merged commit 4f02019 into main Aug 8, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant