alias: smarter determination of editor

Author: weikinhuang

bashrc.sh

@@ -99,6 +99,8 @@ unset hook
 
 # modify path to include useful scripts
 # shellcheck source=/dev/null
+source "${DOTFILES__ROOT}/.dotfiles/dotenv/lib/utils.sh"
+# shellcheck source=/dev/null
 source "${DOTFILES__ROOT}/.dotfiles/dotenv/lib/path.sh"
 __dot_path_setup
 

dotenv/aliases.sh

@@ -22,8 +22,14 @@ alias mv="mv -i"
 # run this after plugins are loaded in case gnu grep and gnu ls is added to path in plugins
 function __grep_ls_colors() {
   local LS_COLOR_FLAG LS_BIN GREP_BIN
-  LS_BIN="$(unalias ls &>/dev/null; command -v ls)"
-  GREP_BIN="$(unalias grep &>/dev/null; command -v grep)"
+  LS_BIN="$(
+    unalias ls &>/dev/null
+    command -v ls
+  )"
+  GREP_BIN="$(
+    unalias grep &>/dev/null
+    command -v grep
+  )"
 
   # Colorize grep matches
   # test if color is supported, if it is, always add color
@@ -88,3 +94,7 @@ alias extip="curl -s https://api.ipify.org/?format=text"
 # Reload the current shell
 # shellcheck disable=SC2139
 alias reload="exec ${SHELL} -l"
+
+# Alias vi to a found editor
+# shellcheck disable=SC2139
+alias vi="$(__find_editor)"

dotenv/darwin/aliases.sh

@@ -1,11 +1,6 @@
 # shellcheck shell=bash
 # Shortcuts
 
-# Alias vi to vim
-if command -v vim &>/dev/null; then
-  alias vi="vim"
-fi
-
 # Networking shortcuts
 function ips() {
   ifconfig | grep 'inet\>' | perl -nle'/(\d+\.\d+\.\d+\.\d+)/ && print $1'

dotenv/exports.sh

@@ -18,5 +18,6 @@ export COMP_TAR_INTERNAL_PATHS
 # Make = a wordbreak character
 COMP_WORDBREAKS=${COMP_WORDBREAKS/=/}
 
-# set $EDITOR to vi(m) if not already set
-export EDITOR="${EDITOR:-$(command -v vim &>/dev/null && echo vim || echo vi)}"
+# set $EDITOR to a if not already set
+EDITOR="${EDITOR:-$(__find_editor)}"
+export EDITOR

dotenv/functions.sh

@@ -1,23 +1,5 @@
 # shellcheck shell=bash
 
-# push a command to the prompt command
-function __push_prompt_command() {
-  local command="${1/%;/}"
-  PROMPT_COMMAND="$(echo "$(echo "${PROMPT_COMMAND/%;/}" | tr ';' '\n' | grep -v -F "${command}" | grep -v '^ *$' | tr '\n' ';')${command};" | sed 's/;;/;/' | sed 's/^;//')"
-}
-
-# internal prompt command stack to simplify the PROMPT_COMMAND variable
-declare -a __prompt_actions
-function __push_internal_prompt_command() {
-  local command="${1/%;/}"
-  __prompt_actions+=("${command}")
-}
-function __run_prompt_command() {
-  for l in "${__prompt_actions[@]}"; do
-    eval "$l"
-  done
-}
-
 # update the dotfiles repo
 function dotfiles-update() (
   # shellcheck disable=SC2164

dotenv/lib/utils.sh

@@ -0,0 +1,40 @@
+# shellcheck shell=bash
+
+# push a command to the prompt command
+function __push_prompt_command() {
+  local command="${1/%;/}"
+  PROMPT_COMMAND="$(echo "$(echo "${PROMPT_COMMAND/%;/}" | tr ';' '\n' | grep -v -F "${command}" | grep -v '^ *$' | tr '\n' ';')${command};" | sed 's/;;/;/' | sed 's/^;//')"
+}
+
+# internal prompt command stack to simplify the PROMPT_COMMAND variable
+declare -a __prompt_actions
+function __push_internal_prompt_command() {
+  local command="${1/%;/}"
+  __prompt_actions+=("${command}")
+}
+function __run_prompt_command() {
+  for l in "${__prompt_actions[@]}"; do
+    eval "$l"
+  done
+}
+
+# helper function get the closest base editor
+function __find_editor() {
+  local editor
+  # shellcheck disable=SC2209
+  editor="$(which vi nano | head -1)"
+
+  if command -v code-insiders &>/dev/null && echo "${PATH}" | grep -q "/.vscode-server-insiders/bin/" && ! (echo "${PATH}" | grep -q "/.vscode-server/bin/"); then
+    editor="code-insiders --wait"
+  elif command -v code &>/dev/null && echo "${PATH}" | grep -q "/.vscode-server/bin/"; then
+    editor="code --wait"
+  elif [[ -n "${DOT___IS_WSL}" ]] && command -v npp &>/dev/null; then
+    editor=npp
+  elif command -v nvim &>/dev/null; then
+    editor=nvim
+  elif command -v vim &>/dev/null; then
+    editor=vim
+  fi
+
+  echo "${editor}"
+}

dotenv/linux/aliases.sh

@@ -1,10 +1,5 @@
 # shellcheck shell=bash
 
-# Alias vi to vim
-if command -v vim &>/dev/null; then
-  alias vi="vim"
-fi
-
 # Networking shortcuts
 function ips() {
   ip addr | grep 'inet\>' | perl -nle'/(\d+\.\d+\.\d+\.\d+)/ && print $1'

dotenv/wsl/aliases.sh

@@ -4,12 +4,3 @@
 # test by running `net.exe sessions` vs winsudo net.exe sessions
 alias wudo="winsudo"
 alias wsl-sudo="winsudo"
-
-# Alias vi to vscode or notepad++
-if command -v code-insiders &>/dev/null && echo "${PATH}" | grep -q "/.vscode-server-insiders/bin/" && ! (echo "${PATH}" | grep -q "/.vscode-server/bin/"); then
-  alias vi="code-insiders --wait"
-elif command -v code &>/dev/null && echo "${PATH}" | grep -q "/.vscode-server/bin/"; then
-  alias vi="code --wait"
-elif command -v npp &>/dev/null; then
-  alias vi="npp"
-fi

dotenv/wsl/bin/npp

@@ -7,6 +7,10 @@ if [[ -d "$(wslpath -ua "c:/")Program Files (x86)/Notepad++/" ]]; then
 fi
 
 PROGRAM="$(wslpath -ua "c:/${PGM_PATH}/Notepad++/notepad++.exe")"
+if [[ ! -e "${PROGRAM}" ]]; then
+  # fallback to a vi program
+  exec "$(which nvim vim vi | head -1)" "$@"
+fi
 
 # if there are any baked in arguments they are defined here
 args=()