feat(packages): install starship/yazi/tree-sitter-cli/viddy from community (#71) - #74
Merged
Merged
Conversation
…unity (#71) These four landed in Alpine community (native musl builds) since the list was written. apk now installs them first; bootstrap.sh keeps the presence-guarded cargo/curl source builds as fallbacks, so the change is purely additive. Also drops the stale sd MED-confidence hedge. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WYbU5Xm57LrLg7RrNJoMxu
There was a problem hiding this comment.
Pull request overview
Updates the Alpine OS-layer package list to prefer native apk installs for tools that are now available as musl builds in Alpine’s community repository, while keeping the existing source-install blocks in bootstrap.sh as presence-guarded fallbacks.
Changes:
- Add
starship,yazi,tree-sitter-cli, andviddytoinstall/packages.txtnow that they’re packaged incommunity. - Refresh and expand the
install/packages.txtheader/comments to reflect the new “apk first, fallback second” approach. - Reword the related sections in
bootstrap.shto clarify that the curl/cargo installs are fallbacks whenapkdoesn’t provide the binaries.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| install/packages.txt | Adds newly-available community packages and updates the file’s explanatory header/comments. |
| bootstrap.sh | Updates comments to describe source installs as presence-guarded fallbacks after apk package installation. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # presence-guarded fallbacks that skip when apk already provided the binary. | ||
| starship # prompt (musl build in community; official installer kept as fallback) | ||
| yazi # file manager (community; cargo build kept as fallback) | ||
| tree-sitter-cli # nvim parser CLI — the community build IS the musl build (>=0.26.1 floor met) |
Comment on lines
+3
to
+7
| # Tools still built from source on musl (tealdeer — testing-only; duf/glow/sesh via | ||
| # go install) are handled by bootstrap.sh — keep them OUT of here. Tools that recently | ||
| # landed in `community` (starship, yazi, tree-sitter-cli, viddy) are now apk-installed | ||
| # below; bootstrap.sh keeps a presence-guarded source build as a fallback for boxes | ||
| # where the community package is missing. |
…eview) Header no longer claims duf/glow are kept out (they're listed as best-effort); fold the tree-sitter-cli detail into the block comment to keep the column aligned. Addresses Copilot review on #74. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WYbU5Xm57LrLg7RrNJoMxu
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Resolves the availability drift flagged in the
os-package-availabilityrun (#71).starship,yazi,tree-sitter-cliandviddyhave all landed in Alpine'scommunityrepo (native musl builds) since this list was written — the ideal case for the musl outlier.Changes
install/packages.txt— add the four to the apk list; refresh the header; drop the stalesdMED-confidence hedge (confirmed incommunity).bootstrap.sh— reword thestarship/yazi/tree-sitter-cli/viddyblocks to note apk installs the community build first; the existing presence-guarded source builds now act as fallbacks.Why this is safe
Every source-build block is guarded by
if ! command -v X, andapk_installis per-package resilient (a name missing on a box is skipped, never fatal). So the change is purely additive: where the community package resolves, apk installs the native musl build and the cargo/curl fallback no-ops; where it doesn't, the fallback builds it exactly as before. Zero regression regardless.Deliberately excluded
jujutsu— now in Alpinecommunitytoo, but it's an opt-in tool not carried in any OS repo'spackages.txt; adding it only here would create single-repo fleet drift. The matrix's stale "not in Alpine" claim is corrected in the companion Core PR without wiring it in.The matching
PORTING-MATRIX.mdupdate is in a companiondotfiles-corePR.🤖 Generated with Claude Code
https://claude.ai/code/session_01WYbU5Xm57LrLg7RrNJoMxu
Generated by Claude Code