Cleaned up some scripts

scripts/.local/scripts/clean_screenshots

@@ -1,6 +1,5 @@
 #!/bin/sh
 
 for file in "$SCREENSHOTS"/*; do
-  echo "Removing file: $file" >&2
   rm "$file"
 done

scripts/.local/scripts/newx

@@ -6,9 +6,13 @@
 test ! -n "${SCRIPTS}" && echo "SCRIPTS not set" && exit 1;
 
 # fallback to check if the name arg ${1} exists, if not show usage text.
-test ! -n "${1}" && echo "usage: newscript <name>" && exit 1;
+test ! -n "${1}" && \
+  echo "usage: newscript <name> <optional: shell>" && \
+  echo "shell defaults to `sh` if not supplied" && \
+  exit 1;
 
 path="${SCRIPTS}/${1}"
+shell="${2:-sh}"
 
 if [ -e "${path}" ]; then
   read -p "${1} already exists. Edit? " yes
@@ -18,7 +22,7 @@ if [ -e "${path}" ]; then
   exit 0
 fi
 
-printf "#!/bin/sh\n\n" > "${path}"
+printf "#!/bin/${shell}"\n\n" > "${path}"
 chmod +x "${path}"
 exec vim +2 "${path}"
 

scripts/install_launch_agents

@@ -10,8 +10,10 @@ uid="$(id -u "$(whoami)")"
 mkdir -p "$agent_dir"
 
 for file in "${DOTFILES}"/macOS/LaunchAgents/*.plist; do
-  filename=$(basename "${file}")
+  # get just the base file name, similar to using `basename` but w/o a subshell
+  filename="${file##*/}"
   path="${agent_dir}/${filename}"
+  echo "$filename"
   if ! test -e "${path}"; then
     echo "Installing Agent: ${filename}"
     cp "${file}" "${path}"