Skip to content

Find user-local Unix binaries during dependency lookup - #92

Merged
vanloctech merged 1 commit into
vanloctech:mainfrom
huuhungn:pr/user-local-binaries
Jul 3, 2026
Merged

vanloctech merged 1 commit into
vanloctech:mainfrom
huuhungn:pr/user-local-binaries

Conversation

@anhtahaylove

Copy link
Copy Markdown
Contributor

Summary

  • Include ~/.local/bin in Unix fallback dependency lookup paths.
  • Keep Deno's extra ~/.deno/bin lookup while reusing the shared Unix fallback list.
  • Silence a Windows-only trait import warning by importing it as _ where needed.

Why

Many Linux and macOS users install CLI tools into ~/.local/bin. GUI apps can miss that path depending on launch environment, so the shared fallback list should cover it before system locations.

Verification

  • cargo fmt --check
  • cargo test utils::path::tests --lib
  • cargo check

Notes

This deliberately avoids larger dependency/packaging refactors and only changes the shared fallback search list.

Unix users often install helper tools into ~/.local/bin, but the fallback dependency search only checked Homebrew and system directories. Adding the user-local bin directory to the shared Unix fallback keeps GUI-launched dependency detection aligned with common CLI install locations.

Constraint: Keep this PR limited to dependency path resolution.
Rejected: Port the larger path-resolution branch | it mixes unrelated packaging and UI changes.
Confidence: high
Scope-risk: narrow
Directive: Keep dependency resolution changes inside the shared path helpers unless a tool needs a truly special install location.
Tested: cargo fmt --check
Tested: cargo test utils::path::tests --lib
Tested: cargo check
Not-tested: Manual Linux/macOS install with deno in ~/.local/bin.
@vanloctech
vanloctech merged commit 2358ce2 into vanloctech:main Jul 3, 2026
3 checks passed
@anhtahaylove
anhtahaylove deleted the pr/user-local-binaries branch July 4, 2026 16:50
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.

2 participants