From fd7b20b8d711c682b0e731a21d87593dc6333df2 Mon Sep 17 00:00:00 2001
From: John D Pell <John+git@gaelicWizard.net>
Date: Tue, 12 Oct 2021 22:02:09 -0700
Subject: [PATCH] reloader: `shellcheck` && `shfmt`

Rewrite globbing per `shellcheck`'s SC2013 recommendations, and standardize whitespace.
---
 clean_files.txt       |  1 +
 scripts/reloader.bash | 79 ++++++++++++++++++++++++-------------------
 2 files changed, 45 insertions(+), 35 deletions(-)
 mode change 100755 => 100644 scripts/reloader.bash

diff --git a/clean_files.txt b/clean_files.txt
index 06b19f5d..ced8ec58 100644
--- a/clean_files.txt
+++ b/clean_files.txt
@@ -18,6 +18,7 @@
 #
 docs/
 hooks/
+scripts/
 
 # root files
 #
diff --git a/scripts/reloader.bash b/scripts/reloader.bash
old mode 100755
new mode 100644
index 5871b7cf..9299a4e4
--- a/scripts/reloader.bash
+++ b/scripts/reloader.bash
@@ -1,44 +1,53 @@
-#!/bin/bash
+# shellcheck shell=bash
+#
+# The core component loader.
+
+# shellcheck disable=SC2034
 BASH_IT_LOG_PREFIX="core: reloader: "
 
-if [[ "$1" != "skip" ]] && [[ -d "$BASH_IT/enabled" ]]; then
-  _bash_it_config_type=""
+if [[ "${1:-}" != "skip" ]] && [[ -d "${BASH_IT?}/enabled" ]]; then
+	_bash_it_config_type=""
 
-  case $1 in
-    alias|completion|plugin)
-      _bash_it_config_type=$1
-      _log_debug "Loading enabled $1 components..." ;;
-    ''|*)
-      _log_debug "Loading all enabled components..." ;;
-  esac
+	case $1 in
+		alias | completion | plugin)
+			_bash_it_config_type=$1
+			_log_debug "Loading enabled $1 components..."
+			;;
+		'' | *)
+			_log_debug "Loading all enabled components..."
+			;;
+	esac
 
-  for _bash_it_config_file in $(sort <(compgen -G "$BASH_IT/enabled/*${_bash_it_config_type}.bash")); do
-    if [ -e "${_bash_it_config_file}" ]; then
-      _bash-it-log-prefix-by-path "${_bash_it_config_file}"
-      _log_debug "Loading component..."
-      # shellcheck source=/dev/null
-      source $_bash_it_config_file
-    else
-      echo "Unable to read ${_bash_it_config_file}" > /dev/stderr
-    fi
-  done
+	for _bash_it_config_file in "$BASH_IT/enabled"/*"${_bash_it_config_type}.bash"; do
+		if [[ -e "${_bash_it_config_file}" ]]; then
+			_bash-it-log-prefix-by-path "${_bash_it_config_file}"
+			_log_debug "Loading component..."
+			# shellcheck source=/dev/null
+			source "$_bash_it_config_file"
+			_log_debug "Loaded."
+		else
+			_log_error "Unable to read ${_bash_it_config_file}"
+		fi
+	done
 fi
 
-if [[ -n "${2}" ]] && [[ -d "$BASH_IT/${2}/enabled" ]]; then
-  case $2 in
-    aliases|completion|plugins)
-      _log_warning "Using legacy enabling for $2, please update your bash-it version and migrate"
-      for _bash_it_config_file in $(sort <(compgen -G "$BASH_IT/${2}/enabled/*.bash")); do
-        if [[ -e "$_bash_it_config_file" ]]; then
-          _bash-it-log-prefix-by-path "${_bash_it_config_file}"
-          _log_debug "Loading component..."
-          # shellcheck source=/dev/null
-          source "$_bash_it_config_file"
-        else
-          echo "Unable to locate ${_bash_it_config_file}" > /dev/stderr
-        fi
-      done ;;
-  esac
+if [[ -n "${2:-}" ]] && [[ -d "$BASH_IT/${2}/enabled" ]]; then
+	case $2 in
+		aliases | completion | plugins)
+			_log_warning "Using legacy enabling for $2, please update your bash-it version and migrate"
+			for _bash_it_config_file in "$BASH_IT/${2}/enabled"/*.bash; do
+				if [[ -e "$_bash_it_config_file" ]]; then
+					_bash-it-log-prefix-by-path "${_bash_it_config_file}"
+					_log_debug "Loading component..."
+					# shellcheck source=/dev/null
+					source "$_bash_it_config_file"
+					_log_debug "Loaded."
+				else
+					_log_error "Unable to locate ${_bash_it_config_file}"
+				fi
+			done
+			;;
+	esac
 fi
 
 unset _bash_it_config_file