lint

Author: jaskfla

scripts/bash/backendStartDev.sh

@@ -9,11 +9,10 @@ set -e
 DIR=$(cd -- "$(dirname -- "${BASH_SOURCE[0]}")" &>/dev/null && pwd)
 . "$DIR/ansiControlSequences.sh"
 
-USAGE="Usage: ${BOLD}backendStartDev babel_port_inspector${RESET} [${BOLD}-i${RESET}|${BOLD}--include-internal${RESET}] [${BOLD}-ts${RESET}|${BOLD}--typescript${RESET}]"
 watch_flags=''
 include_internal=false
 type_script=false
-inspect_port=${1}
+inspect_port=$1
 
 # Start server command for JS
 start_server="nodemon -w src --exec \"babel-node src --inspect=${inspect_port} --config-file '../../babel.config.json'\""
@@ -28,29 +27,26 @@ while [[ $2 != '' ]]; do
         include_internal=true
         shift
         ;;
-    -s | --skip-internal)
-        echo -e "Skipping internal dependencies is now done by default. Remove the ${BOLD}--skip-internal${RESET} (${BOLD}-s${RESET}) flag, and if you want to include internal dependencies, add a ${BOLD}-i${RESET}. (Do try it - it’s a lot faster than it used to be, because it only builds those relevant to the current package$EXCLAMATION_MARK)"
-        exit 1
-        ;;
     *)
-        echo -e "$USAGE"
+        echo -e "Usage: ${BOLD}backendStartDev babel_port_inspector${RESET} [${BOLD}-i${RESET}|${BOLD}--include-internal${RESET}] [${BOLD}-ts${RESET}|${BOLD}--typescript${RESET}]"
         exit 1
         ;;
     esac
 done
 
 # Start server command for TS
 if [[ $type_script = true ]]; then
-    start_server="nodemon --watch src -e ts,json,js --exec node --inspect=${inspect_port} -r ts-node/register src/index.ts"
+    start_server="nodemon --watch src -e ts,json,js --exec node --inspect=${inspect_port@Q} -r ts-node/register src/index.ts"
 fi
 
 echo -e "${BOLD}Starting server...${RESET}"
 
 # If internal dependencies are included, add them to the watch list. This will watch for changes to the dist folder of each internal dependency. If the internal dependency then gets rebuilt, the server will restart.
 if [[ $include_internal = true ]]; then
     echo 'Internal dependencies are under watch for hot reload'
-    for PACKAGE in $("$DIR/getInternalDependencies.sh"); do
-        watch_flags+=" --watch ../$PACKAGE/dist"
+    for package in $("$DIR"/getInternalDependencies.sh); do
+        dist_path=../$package/dist
+        watch_flags+=" --watch ${dist_path@Q}"
     done
     # add the watch flags to the server start process, as well as a 1 second delay to debounce the
     # many restarts that otherwise happen during the initial build of internal dependencies