fix: add missing PATH entries in Hetzner and DigitalOcean exec functions#2450
Merged
fix: add missing PATH entries in Hetzner and DigitalOcean exec functions#2450
Conversation
louisgv
approved these changes
Mar 10, 2026
Member
louisgv
left a comment
louisgv
left a comment
There was a problem hiding this comment.
Security Review
Verdict: APPROVED
Commit: 78416d8
Findings
No security issues found. This PR adds PATH entries for npm-global and claude/local bins to Hetzner and DigitalOcean exec functions, bringing them into alignment with AWS and GCP which already have these entries.
Analysis:
- PATH entries use
$HOMEshell variable (not user-controllable) - safe - No change to command interpolation behavior (matches existing AWS/GCP pattern)
- Version correctly bumped (0.15.38 → 0.15.39)
- Improves consistency across cloud drivers
Tests
- bash -n: N/A (no shell scripts modified)
- bun test: PASS (1497/1497 tests passing)
- lint: PASS (biome clean)
- curl|bash: N/A (no shell scripts modified)
- macOS compat: N/A (no shell scripts modified)
-- security/pr-reviewer
…nteractiveSession AWS and GCP both include $HOME/.npm-global/bin and $HOME/.claude/local/bin in the PATH exported before running remote commands. Hetzner and DO were missing these two entries, causing "command not found" errors for Claude Code and npm-global packages on those clouds. Agent: code-health Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
louisgv
force-pushed
the
fix/hetzner-do-path-entries
branch
from
78416d8 to
26a0163
Compare
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
2 participants
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.
Why: Hetzner and DigitalOcean `runServer()` and `interactiveSession()` were missing `$HOME/.npm-global/bin` and `$HOME/.claude/local/bin` from their PATH exports, causing "command not found" errors for Claude Code (`
/.claude/local/bin`) and npm-global packages (`/.npm-global/bin`) on those two clouds. AWS and GCP already had the correct PATH.Changes
4 one-line fixes to match AWS/GCP's PATH:
Before (Hetzner/DO):
```
export PATH="$HOME/.local/bin:$HOME/.bun/bin:$PATH"
```
After (matching AWS/GCP):
```
export PATH="$HOME/.npm-global/bin:$HOME/.claude/local/bin:$HOME/.local/bin:$HOME/.bun/bin:$PATH"
```
Biome: 0 errors. Tests: 1497 pass, 0 fail.
-- refactor/code-health