feat: offer LazyVim starter config when Neovim is selected#83
Open
BrettKinny wants to merge 1 commit into
Open
feat: offer LazyVim starter config when Neovim is selected#83BrettKinny wants to merge 1 commit into
BrettKinny wants to merge 1 commit into
Conversation
When nvim is chosen in the editor setup, prompt to install the LazyVim starter into ~/.config/nvim (skipped if a config already exists, so a user's own config is never clobbered). Installs build-essential on demand since nvim-treesitter compiles parsers on first launch. The choice persists to /workspace/.squarebox/nvim-lazyvim and is surfaced in sqrbx-setup --list. Supersedes the "Neovim defaults from omarchy" roadmap item. https://claude.ai/code/session_016nYRxhVQCoDaie2QNzo9oi
Contributor
There was a problem hiding this comment.
Pull request overview
Adds an optional LazyVim “starter” bootstrap path to the existing first-run editor setup so users who select Neovim can easily get a ready-to-use IDE-style configuration, with the choice persisted and visible via sqrbx-setup --list.
Changes:
- Prompt for (and persist) a LazyVim starter install when
nvimis selected, and bootstrap~/.config/nvimwithout clobbering an existing config. - Install
build-essentialon demand to supportnvim-treesitterparser compilation. - Surface LazyVim enabled/disabled status in
sqrbx-setup --list, and update docs/roadmap accordingly.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
setup.sh |
Adds LazyVim selection persistence and an installer that clones the LazyVim starter when Neovim is selected. |
scripts/squarebox-setup.sh |
Extends --list output to show LazyVim status when nvim is selected. |
ROADMAP.md |
Removes the superseded “Neovim defaults from omarchy” roadmap item. |
README.md |
Documents the new optional LazyVim starter behavior for Neovim selection. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+628
to
+635
| _install_lazyvim_inner() { | ||
| # nvim-treesitter compiles parsers on first launch, which needs a C compiler | ||
| if ! command -v cc &>/dev/null && ! command -v gcc &>/dev/null; then | ||
| sudo apt-get update -qq && sudo apt-get install -y -qq build-essential >/dev/null 2>&1 || return 1 | ||
| fi | ||
| git clone --depth 1 https://github.com/LazyVim/starter ~/.config/nvim >/dev/null 2>&1 || return 1 | ||
| rm -rf ~/.config/nvim/.git | ||
| } |
Comment on lines
+572
to
+585
| # LazyVim starter — offered when Neovim is among the selected editors | ||
| LAZYVIM_CONFIG="/workspace/.squarebox/nvim-lazyvim" | ||
| lazyvim_prev="" | ||
| [ -f "$LAZYVIM_CONFIG" ] && lazyvim_prev=$(cat "$LAZYVIM_CONFIG") | ||
| lazyvim_choice=false | ||
| if [[ ",$editor_list," == *",nvim,"* ]]; then | ||
| if $INTERACTIVE; then | ||
| lv_default=true | ||
| [ "$lazyvim_prev" = "false" ] && lv_default=false | ||
| echo | ||
| echo "LazyVim turns Neovim into a preconfigured IDE (needs a Nerd Font in your terminal for icons)." | ||
| if $HAS_GUM; then | ||
| gum confirm "Install the LazyVim starter config for Neovim?" --default="$lv_default" && lazyvim_choice=true || lazyvim_choice=false | ||
| else |
Comment on lines
+647
to
+650
| # Bootstrap LazyVim starter config when chosen and Neovim is available | ||
| if [ "$lazyvim_choice" = "true" ] && command -v nvim &>/dev/null; then | ||
| install_lazyvim || echo "Warning: LazyVim starter setup failed." | ||
| fi |
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.
When nvim is chosen in the editor setup, prompt to install the LazyVim
starter into ~/.config/nvim (skipped if a config already exists, so a
user's own config is never clobbered). Installs build-essential on demand
since nvim-treesitter compiles parsers on first launch. The choice persists
to /workspace/.squarebox/nvim-lazyvim and is surfaced in sqrbx-setup --list.
Supersedes the "Neovim defaults from omarchy" roadmap item.
https://claude.ai/code/session_016nYRxhVQCoDaie2QNzo9oi