Skip to content

Comments

fix: use async exec for lazy install so spinner animates#660

Merged
arjunkmrm merged 1 commit intomainfrom
fix/lazy-install-async-spinner
Feb 24, 2026
Merged

fix: use async exec for lazy install so spinner animates#660
arjunkmrm merged 1 commit intomainfrom
fix/lazy-install-async-spinner

Conversation

@arjunkmrm
Copy link
Collaborator

Summary

  • execSync blocks the event loop, causing the yocto-spinner animation to freeze while npm runs
  • Switch to promisified execFile so the spinner can tick during installation

Test plan

  • All 338 existing tests pass
  • Manually verify spinner animates during smithery mcp build lazy esbuild install

🤖 Generated with Claude Code

execSync blocks the event loop, causing the yocto-spinner animation to
freeze while npm runs. Switch to promisified execFile so the spinner
can tick during installation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@arjunkmrm arjunkmrm enabled auto-merge (squash) February 24, 2026 09:42
@arjunkmrm arjunkmrm merged commit 82617d4 into main Feb 24, 2026
4 checks passed
@arjunkmrm arjunkmrm deleted the fix/lazy-install-async-spinner branch February 24, 2026 09:42
@smithery-chore-bot smithery-chore-bot bot mentioned this pull request Feb 24, 2026
arjunkmrm pushed a commit that referenced this pull request Feb 24, 2026
Automated Release PR
---


## [4.1.8](v4.1.7...v4.1.8)
(2026-02-24)


### Bug Fixes

* use async exec for lazy install so spinner animates
([#660](#660))
([82617d4](82617d4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: smithery-chore-bot[bot] <259537197+smithery-chore-bot[bot]@users.noreply.github.com>
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