fix(bootstrap): provision viddy (watch replacement) via go install#52
Merged
Conversation
Core ships a watch->viddy alias (HAVE_VIDDY-guarded in zsh/aliases.zsh) but no distro installed viddy, so watch silently stayed classic watch (dotfiles-core doc-audit, issue #213 / S2). Add viddy to the existing best-effort go-install pass alongside doggo/carapace/sesh/gron. Inert without the binary; go is already a pinned mise runtime. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018hzGkfEPWFHcyGrYMaYt6F
There was a problem hiding this comment.
Pull request overview
This PR updates the Fedora bootstrap provisioning so the viddy binary is installed via the existing best-effort go install pass. This aligns the environment with dotfiles-core’s watch→viddy alias (guarded by HAVE_VIDDY) so watch doesn’t silently fall back to classic watch due to a missing binary.
Changes:
- Add
viddyto thego install where absenttool provisioning list inbootstrap.sh. - Update the corresponding
blib_saystatus message to includeviddy. - Document why
viddyis installed from source (not available in Fedora repos).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
viddy was rewritten from Go to Rust upstream, so `go install github.com/sachaos/viddy` reliably fails and never provisions the binary (flagged in PR review). Install it via cargo instead, matching the other Rust CLIs in this script (Arch ships no rust toolchain, so it prints a `paru -S viddy` AUR hint instead of auto-installing). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018hzGkfEPWFHcyGrYMaYt6F
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.
Core ships a
watch→viddyalias (HAVE_VIDDY-guarded inzsh/aliases.zsh) but viddy was never installed on Fedora, sowatchsilently stayed classicwatch. This closes that gap, flagged by the dotfiles-core doc-audit routine (dotgibson/dotfiles-core#213, item S2).Change
Add
viddyto the existing best-effort go-install pass inbootstrap.sh, alongside doggo/carapace/sesh/gron:Inert without the binary (the alias is guarded), and
gois already a pinned mise runtime so the install works everywhere. Matches theviddyrow now documented incore/PORTING-MATRIX.md.Verification
bash -n bootstrap.sh→ syntax OK🤖 Generated with Claude Code
https://claude.ai/code/session_018hzGkfEPWFHcyGrYMaYt6F
Generated by Claude Code