johnlindquist / johnlindquist/script-kit-next
Support for Alternative Package Managers (mise, asdf) and Custom Bun Paths
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 28
- Forks
- 6
- PR merge metrics
- No merged PRs in 30d
Description
Description
Currently, Script Kit discovery for the Bun executable relies on a hardcoded list of common global paths (Homebrew, /usr/local/bin, etc.) and the system PATH in file src/executor/runner.rs.
While this works for standard installations, it fails for users managing their environments via tools like mise/asdf or managers not hardcoded, and even if the user defined the path in the config.ts file it was not being respected because the path was not one of the hardcoded.
Problem
- The "Bun not installed" warning banner persists even if Bun is managed by mise/asdf, unless the user manually launches the app from a terminal with a fully initialized environment.
- Discovery logic was fragmented across preflight.rs, app_run_setup.rs, and runtime_watchers_scheduler.rs, making it difficult to keep path resolution consistent.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing Bun executable discovery in src/executor/runner.rs and compare the related logic in preflight.rs, app_run_setup.rs, and runtime_watchers_scheduler.rs. Check how config.ts paths and initialized environments are handled; done means discovery is consistent across these locations and the warning no longer persists for supported custom or managed Bun paths.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- bun, rust
- Domain
- tooling
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100