Skip to content

fixes#1

Open
lucianoengel wants to merge 2 commits intocanesin:mainfrom
lucianoengel:le-fixes
Open

fixes#1
lucianoengel wants to merge 2 commits intocanesin:mainfrom
lucianoengel:le-fixes

Conversation

@lucianoengel
Copy link

Summary

  • Fix nvm detection to check nvm-managed node, not system node
  • Fix git config commands to avoid permission denied errors
  • Fix verification checks for set -e compatibility
  • Add C++20 compiler auto-config for Node.js 24+ native modules
  • Add network-online.target dependency for Pi-hole (fixes boot race condition)

Test plan

  • Tested on Ubuntu 24.04 with Podman 4.9.3
  • Verified Pi-hole starts after reboot with static IP
  • Verified coder-mcp builds with Node 24 + g++-13"

lucianoengel and others added 2 commits February 18, 2026 09:23
setup-coder.sh:
- Fix nvm detection to check nvm-managed node, not system node
- Fix git config commands to cd to coder home (avoid permission denied)
- Fix verification check function to handle set -e with arithmetic
- Fix Claude Code CLI check to run as coder user
- Add C++20 compiler (g++-13) auto-config for Node.js 24+ native modules
- Skip coder-mcp verification if AITOOLS_DIR not set

setup-homelab.sh:
- Add network-online.target dependency for Pi-hole container
- Ensures Pi-hole waits for network IP before binding to port 53
- Fixes boot race condition causing DNS service failure

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

1 participant