Check if fnm is installed before shell setup (#937)

* Make install.sh tee instead of echoing twice

Also, fish bit was the only one that didn't print a newline to
$CONF_FILE, now they're all the same.

* Check if fnm is installed before shell setup

Change $CONF_FILE append strings to check whether $INSTALL_DIR exists
before doing any setup. This way whenever a user syncs their shell
config across machines, the machines without fnm installed won't throw
errors.

---------

Co-authored-by: Gal Schlezinger <gal@spitfire.co.il>

.ci/install.sh

@@ -168,29 +168,29 @@ setup_shell() {
     CONF_FILE=${ZDOTDIR:-$HOME}/.zshrc
     ensure_containing_dir_exists "$CONF_FILE"
     echo "Installing for Zsh. Appending the following to $CONF_FILE:"
-    echo ""
+    {
-    echo '  # fnm'
+      echo ''
-    echo '  export PATH="'"$INSTALL_DIR"':$PATH"'
+      echo '# fnm'
-    echo '  eval "`fnm env`"'
+      echo 'FNM_PATH="'"$INSTALL_DIR"'"'
-
+      echo 'if [ -d "$FNM_PATH" ]; then'
-    echo '' >>$CONF_FILE
+      echo '  export PATH="'$INSTALL_DIR':$PATH"'
-    echo '# fnm' >>$CONF_FILE
+      echo '  eval "`fnm env`"'
-    echo 'export PATH="'$INSTALL_DIR':$PATH"' >>$CONF_FILE
+      echo 'fi'
-    echo 'eval "`fnm env`"' >>$CONF_FILE
+    } | tee -a "$CONF_FILE"
 
   elif [ "$CURRENT_SHELL" = "fish" ]; then
     CONF_FILE=$HOME/.config/fish/conf.d/fnm.fish
     ensure_containing_dir_exists "$CONF_FILE"
     echo "Installing for Fish. Appending the following to $CONF_FILE:"
-    echo ""
+    {
-    echo '  # fnm'
+      echo ''
-    echo '  set PATH "'"$INSTALL_DIR"'" $PATH'
+      echo '# fnm'
-    echo '  fnm env | source'
+      echo 'set FNM_PATH "'"$INSTALL_DIR"'"'
-
+      echo 'if [ -d "$FNM_PATH" ]'
-    echo '' >>$CONF_FILE
+      echo '  set PATH "$FNM_PATH" $PATH'
-    echo '# fnm' >>$CONF_FILE
+      echo '  fnm env | source'
-    echo 'set PATH "'"$INSTALL_DIR"'" $PATH' >>$CONF_FILE
+      echo 'end'
-    echo 'fnm env | source' >>$CONF_FILE
+    } | tee -a "$CONF_FILE"
 
   elif [ "$CURRENT_SHELL" = "bash" ]; then
     if [ "$OS" = "Darwin" ]; then
@@ -200,15 +200,15 @@ setup_shell() {
     fi
     ensure_containing_dir_exists "$CONF_FILE"
     echo "Installing for Bash. Appending the following to $CONF_FILE:"
-    echo ""
+    {
-    echo '  # fnm'
+      echo ''
-    echo '  export PATH="'"$INSTALL_DIR"':$PATH"'
+      echo '# fnm'
-    echo '  eval "`fnm env`"'
+      echo 'FNM_PATH="'"$INSTALL_DIR"'"'
-
+      echo 'if [ -d "$FNM_PATH" ]; then'
-    echo '' >>$CONF_FILE
+      echo '  export PATH="$FNM_PATH:$PATH"'
-    echo '# fnm' >>$CONF_FILE
+      echo '  eval "`fnm env`"'
-    echo 'export PATH="'"$INSTALL_DIR"':$PATH"' >>$CONF_FILE
+      echo 'fi'
-    echo 'eval "`fnm env`"' >>$CONF_FILE
+    } | tee -a "$CONF_FILE"
 
   else
     echo "Could not infer shell type. Please set up manually."