Clean alias completion plugin (#1872)

* Sort plugins/available

* Clean alias-completion plugin

* Fix shellcheck warning. SC1090

* Change shellcheck source to /dev/null for explicity

* Disable shellcheck warning SC2162

clean_files.txt

@@ -50,15 +50,16 @@ themes/purity
 
 # plugins
 #
+plugins/available/alias-completion.plugin.bash
 plugins/available/basher.plugin.bash
 plugins/available/cmd-returned-notify.plugin.bash
 plugins/available/docker-machine.plugin.bash
 plugins/available/git.plugin.bash
 plugins/available/go.plugin.bash
 plugins/available/goenv.plugin.bash
-plugins/available/history.plugin.bash
 plugins/available/history-search.plugin.bash
 plugins/available/history-substring-search.plugin.bash
+plugins/available/history.plugin.bash
 plugins/available/xterm.plugin.bash
 
 # completions
@@ -105,6 +106,7 @@ aliases/available/vim.aliases.bash
 aliases/available/git.aliases.bash
 
 # tests
+test/plugins/alias-completion.plugin.bats
 test/test_helper.bash
 
 # vendor init files

plugins/available/alias-completion.plugin.bash

@@ -1,3 +1,4 @@
+# shellcheck shell=bash
 # Load after the other completions to understand what needs to be completed
 # BASH_IT_LOAD_PRIORITY: 365
 
@@ -26,43 +27,50 @@ function alias_completion {
 
 	# create array of function completion triggers, keeping multi-word triggers together
 	eval "local completions=($(complete -p | sed -Ene "/$compl_regex/s//'\3'/p"))"
-    (( ${#completions[@]} == 0 )) && return 0
+	((${#completions[@]} == 0)) && return 0
 
 	# create temporary file for wrapper functions and completions
-    local tmp_file; tmp_file="$(mktemp -t "${namespace}-${RANDOM}XXXXXX")" || return 1
+	local tmp_file
+	tmp_file="$(mktemp -t "${namespace}-${RANDOM}XXXXXX")" || return 1
 
-    local completion_loader; completion_loader="$(complete -p -D 2>/dev/null | sed -Ene 's/.* -F ([^ ]*).*/\1/p')"
+	local completion_loader
+	completion_loader="$(complete -p -D 2> /dev/null | sed -Ene 's/.* -F ([^ ]*).*/\1/p')"
 
 	# read in "<alias> '<aliased command>' '<command args>'" lines from defined aliases
-    local line; while read line; do
-        eval "local alias_tokens; alias_tokens=($line)" 2>/dev/null || continue # some alias arg patterns cause an eval parse error
+	local line
+
+	# shellcheck disable=SC2162
+	# some aliases do have backslashes that needs to be interpreted
+	while read line; do
+		eval "local alias_tokens; alias_tokens=($line)" 2> /dev/null || continue # some alias arg patterns cause an eval parse error
 		local alias_name="${alias_tokens[0]}" alias_cmd="${alias_tokens[1]}" alias_args="${alias_tokens[2]# }"
 
 		# skip aliases to pipes, boolean control structures and other command lists
 		# (leveraging that eval errs out if $alias_args contains unquoted shell metacharacters)
-        eval "local alias_arg_words; alias_arg_words=($alias_args)" 2>/dev/null || continue
+		eval "local alias_arg_words; alias_arg_words=($alias_args)" 2> /dev/null || continue
 		# avoid expanding wildcards
-        read -a alias_arg_words <<< "$alias_args"
+		read -ra alias_arg_words <<< "$alias_args"
 
 		# skip alias if there is no completion function triggered by the aliased command
-        if [[ ! " ${completions[*]} " =~ " $alias_cmd " ]]; then
+		if [[ ! " ${completions[*]} " =~ $alias_cmd ]]; then
 			if [[ -n "$completion_loader" ]]; then
 				# force loading of completions for the aliased command
 				eval "$completion_loader $alias_cmd"
 				# 124 means completion loader was successful
 				[[ $? -eq 124 ]] || continue
-                completions+=($alias_cmd)
+				completions+=("$alias_cmd")
 			else
 				continue
 			fi
 		fi
-        local new_completion="$(complete -p "$alias_cmd" 2>/dev/null)"
+		local new_completion="$(complete -p "$alias_cmd" 2> /dev/null)"
 
 		# create a wrapper inserting the alias arguments if any
 		if [[ -n $alias_args ]]; then
-            local compl_func="${new_completion/#* -F /}"; compl_func="${compl_func%% *}"
+			local compl_func="${new_completion/#* -F /}"
+			compl_func="${compl_func%% *}"
 			# avoid recursive call loops by ignoring our own functions
-            if [[ "${compl_func#_$namespace::}" == $compl_func ]]; then
+			if [[ "${compl_func#_$namespace::}" == "$compl_func" ]]; then
 				local compl_wrapper="_${namespace}::${alias_name}"
 				echo "function $compl_wrapper {
                         local compl_word=\$2
@@ -91,5 +99,8 @@ function alias_completion {
 			echo "$new_completion" >> "$tmp_file"
 		fi
 	done < <(alias -p | sed -Ene "s/$alias_regex/\2 '\3' '\4'/p")
+	# shellcheck source=/dev/null
 	source "$tmp_file" && command rm -f "$tmp_file"
-}; alias_completion
+}
+
+alias_completion