History

fix(ssh): fix support for remotes that use fnm (#2641)

Soorria Saruva committed 4 months ago

Commit 804d44c1 parent 0651af92 files · +5 −2Browse files

Unified · Split

2 files
  1. packages/ssh/src/tunnel.test.ts +3 −1
  2. packages/ssh/src/tunnel.ts +2 −1
packages/ssh/src/tunnel.test.ts+3−1
@@ -105,7 +105,9 @@ describe("ssh tunnel scripts", () => { · 104 unmodified lines
105105 assert.include(script, 'prepend_path_if_dir "$VOLTA_HOME/bin"');
106106 assert.include(script, 'prepend_path_if_dir "$HOME/.asdf/shims"');
107107 assert.include(script, 'prepend_path_if_dir "$HOME/.local/share/mise/shims"');
108− assert.include(script, 'eval "$(fnm env --use-on-cd --shell sh)"');
108+ assert.include(script, 'eval "$(fnm env --shell bash)"');
109+ assert.include(script, "fnm use --silent-if-unchanged");
110+ assert.include(script, "fnm use default");
109111 assert.include(script, 'prepend_path_if_dir "$HOME/.nodenv/shims"');
110112 assert.include(script, 'NVM_DIR="$HOME/.nvm"');
111113 assert.include(script, "nvm use --silent default");
packages/ssh/src/tunnel.ts+2−1
@@ -398,7 +398,8 @@ ensure_remote_node_path() { · 397 unmodified lines
398398 prepend_path_if_dir "$FNM_DIR"
399399 prepend_path_if_dir "$HOME/.fnm"
400400 if ! command -v node >/dev/null 2>&1 && command -v fnm >/dev/null 2>&1; then
401− eval "$(fnm env --use-on-cd --shell sh)" >/dev/null 2>&1 || eval "$(fnm env --shell sh)" >/dev/null 2>&1 || true
401+ eval "$(fnm env --shell bash)" >/dev/null 2>&1 || true
402+ fnm use --silent-if-unchanged >/dev/null 2>&1 || fnm use default >/dev/null 2>&1 || true
402403 fi
403404
404405 prepend_path_if_dir "$HOME/.nodenv/bin"