Improve change file detection

Author: JohT

scripts/detectChangedFiles.sh

@@ -21,11 +21,17 @@ ARTIFACTS_CHANGE_DETECTION_HASH_FILE=${ARTIFACTS_CHANGE_DETECTION_HASH_FILE:-"ar
 CHANGE_DETECTION_HASH_FILE=${CHANGE_DETECTION_HASH_FILE:-"${ARTIFACTS_CHANGE_DETECTION_HASH_FILE}"} # Name of the file that contains the hash code of the file list for change detection
 CHANGE_DETECTION_HASH_FILE_PATH=${CHANGE_DETECTION_HASH_FILE_PATH:-"./${ARTIFACTS_DIRECTORY}/${CHANGE_DETECTION_HASH_FILE}"} # Default path of the file that contains the hash code of the file list for change detection. Can be overridden by a command line option.
 
+COLOR_INFO='\033[0;30m' # dark grey
+COLOR_ERROR='\033[0;31m' # red
+COLOR_DEFAULT='\033[0m'    
+
 # Function to display script usage
 usage() {
-  echo "Usage: $0 [--readonly]"
-  echo "          [--paths <comma separated list of file and directory names> (default=artifacts)]"
-  echo "          [--hashfile <path to the file that contains the hash for change detection> (default=env var CHANGE_DETECTION_HASH_FILE_PATH)]"
+  echo -e "${COLOR_ERROR}" >&2
+  echo "Usage: $0 [--readonly]" >&2
+  echo "          [--paths <comma separated list of file and directory names> (default=artifacts)]" >&2
+  echo "          [--hashfile <path to the file that contains the hash for change detection> (default=env var CHANGE_DETECTION_HASH_FILE_PATH)]" >&2
+  echo -e "${COLOR_DEFAULT}" >&2
   exit 1
 }
 
@@ -52,17 +58,17 @@ while [[ $# -gt 0 ]]; do
       shift
       ;;
     *)
-      echo "detectChangedFiles: Error: Unknown option: ${key}"
+      echo -e "${COLOR_ERROR}detectChangedFiles: Error: Unknown option: ${key}${COLOR_DEFAULT}" >&2
       usage
       ;;
   esac
   shift || true # ignore error when there are no more arguments
 done
 
 if ${readonlyMode}; then
-  echo "detectChangedFiles: Readonly mode activated. Change detection file won't be created." >&2
+  echo -e "${COLOR_INFO}detectChangedFiles: Readonly mode activated. Change detection file won't be created.${COLOR_DEFAULT}" >&2
 else
-  echo "detectChangedFiles: ${hashFilePath} will be used as change detection file." >&2
+  echo -e "${COLOR_INFO}detectChangedFiles: ${hashFilePath} will be used as change detection file.${COLOR_DEFAULT}" >&2
 fi
 
 # Check if the paths parameter exist
@@ -71,6 +77,19 @@ if [ -z "${paths}" ] ; then
     exit 0
 fi
 
+# Check all paths if they are valid files or valid directories
+for path in ${paths//,/ }; do
+    if [ -f "${path}" ] ; then
+      continue # Valid file
+    fi
+    if [ -d "${path}" ] ; then
+      continue # Valid directory
+    fi
+    # Neither a valid directory and file
+    echo -e "${COLOR_ERROR}detectChangedFiles: Error: Invalid path: ${path}${COLOR_DEFAULT}" >&2
+    exit 1
+done
+
 # Function to get file size
 get_file_size() {
     if [ -f "$1" ]; then
@@ -113,7 +132,7 @@ get_md5_checksum_of_all_file_names_and_sizes() {
       local files_and_their_size; files_and_their_size=$(file_names_and_sizes "${path}")
       all_files_and_sizes="${all_files_and_sizes}${files_and_their_size}"
       processed_paths=$((processed_paths + 1))
-      echo -ne "detectChangedFiles: Calculate checksum progress: ($processed_paths/$total_paths)\r" >&2
+      echo -ne "${COLOR_INFO}detectChangedFiles: Calculate checksum progress: ($processed_paths/$total_paths)\r${COLOR_DEFAULT}" >&2
   done
   echo "" >&2
   echo "${all_files_and_sizes}" | openssl md5 | awk '{print $2}'
@@ -133,9 +152,9 @@ if [ ! -f "${hashFilePath}" ] ; then
         mkdir -p "${hash_file_directory}"
         # Create the file containing the hash of the files list to a new file for the next call
         echo "${CURRENT_FILES_HASH}" > "${hashFilePath}"
-        echo "detectChangedFiles: Change detection file created" >&2
+        echo -e "${COLOR_INFO}detectChangedFiles: Change detection file created.${COLOR_DEFAULT}" >&2
     else
-        echo "detectChangedFiles: Skipping file creation with content (=hash) ${CURRENT_FILES_HASH}" >&2
+        echo -e "${COLOR_INFO}detectChangedFiles: Skipping file creation with content (=hash) ${CURRENT_FILES_HASH}${COLOR_DEFAULT}" >&2
     fi
     echo 1 # 1=Change detected and change detection file created
     exit 0
@@ -149,9 +168,9 @@ else
     if ! ${readonlyMode}; then
         # Write the updated hash into the file containing the hash of the files list for the next call
         echo "${CURRENT_FILES_HASH}" > "${hashFilePath}"
-        echo "detectChangedFiles: Change detection file updated" >&2
+        echo -e "${COLOR_INFO}detectChangedFiles: Change detection file updated.${COLOR_DEFAULT}" >&2
     else
-        echo "detectChangedFiles: Skipping file update with content (=hash) ${CURRENT_FILES_HASH}" >&2
+        echo -e "${COLOR_INFO}detectChangedFiles: Skipping file update with content (=hash) ${CURRENT_FILES_HASH}.${COLOR_DEFAULT}" >&2
     fi
     echo 2 # 2=Change detected and change detection file updated
 fi

scripts/scanTypescript.sh

@@ -115,14 +115,10 @@ is_valid_scan_result() {
 }
 
 is_change_detected() {
-    local COLOR_DARK_GREY='\033[0;30m'
-    local COLOR_DEFAULT='\033[0m'    
     local source_directory_name; source_directory_name=$(basename "${source_directory}");
 
-    echo -e "${COLOR_DARK_GREY}"
     changeDetectionHashFilePath="./${SOURCE_DIRECTORY}/typescriptScanChangeDetection-${source_directory_name}.sha"
     changeDetectionReturnCode=$( source "${SCRIPTS_DIR}/detectChangedFiles.sh" --readonly --hashfile "${changeDetectionHashFilePath}" --paths "${source_directory}")
-    echo -e "${COLOR_DEFAULT}"
 
     if [ "${changeDetectionReturnCode}" == "0" ] && [ "${TYPESCRIPT_SCAN_CHANGE_DETECTION}" = true ]; then
         true