Skip to content

fix: retry stalled Bun installs#1808

Merged
merryman merged 1 commit into
mainfrom
robin/bun-install-retry
Jul 20, 2026
Merged

fix: retry stalled Bun installs#1808
merryman merged 1 commit into
mainfrom
robin/bun-install-retry

Conversation

@merryman

Copy link
Copy Markdown
Member

Summary

  • detect Bun installs that produce no output for 120 seconds
  • terminate a stalled attempt and retry once using the partially warmed cache and work directory
  • force-kill an unresponsive attempt after a short grace period
  • preserve live output in verbose mode and add a regression test with a simulated stalled Bun process

Root cause

The installer waited indefinitely for the spawned Bun process while suppressing its normal output. When Bun intermittently stalled in its cache or download path, the install had no timeout or recovery path even though manually restarting the same install often succeeded.

Impact

make install and make clean-install can now recover automatically from a transient Bun stall. If both attempts stall, the existing Flatn fallback still handles the failure. Slow environments can override the default with LIVELY_BUN_INSTALL_STALL_TIMEOUT_MS.

Validation

  • git diff --check origin/main...HEAD
  • node --check flatn/bun-install.js
  • simulated first-attempt stall followed by a successful retry
  • full legacy Flatn suite was not run because mocha-es6 is unavailable in this checkout

@merryman
merryman marked this pull request as ready for review July 19, 2026 22:25
@merryman
merryman force-pushed the robin/bun-install-retry branch from b84d888 to b787d04 Compare July 20, 2026 09:02
@merryman
merryman force-pushed the robin/bun-install-retry branch from b787d04 to 2b3dcc6 Compare July 20, 2026 10:21
@merryman
merryman enabled auto-merge (rebase) July 20, 2026 10:22
@merryman
merryman disabled auto-merge July 20, 2026 11:58
@merryman
merryman merged commit 3958d8f into main Jul 20, 2026
9 checks passed
@merryman
merryman deleted the robin/bun-install-retry branch July 20, 2026 11:58
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