ci: Add bash script syntax check to CI

This commit introduces a new step to the CI workflow (`ci.yml`) that performs a syntax check on all bash scripts within the repository.

The new CI job:
- Identifies all `.sh` files and files with a bash shebang (`#!/bin/bash` or `#!/usr/bin/env bash`).
- Excludes common directories like `node_modules`, `.git`, and `.augment`.
- Uses `bash -n` to perform a non-destructive syntax validation for each identified script.
- Reports any syntax errors found, providing detailed output to aid in debugging.

This addition aims to proactively catch and prevent syntax errors in shell scripts before they are merged, improving code quality and reliability.

Author: miklschmidt

.github/workflows/ci.yml

@@ -96,6 +96,60 @@ jobs:
           repository: Arksine/moonraker
           path: "moonraker"
 
+      - name: Bash Syntax Check
+        run: |
+          echo "🔍 Finding bash scripts in the repository..."
+
+          # Find all .sh files and files with bash shebangs
+          bash_files=()
+
+          # Add all .sh files
+          while IFS= read -r -d '' file; do
+            bash_files+=("$file")
+          done < <(find . -name "*.sh" -type f -not -path "./src/node_modules/*" -not -path "./.git/*" -not -path "./.augment/*" -print0)
+
+          # Add files with bash shebangs (excluding .sh files already found)
+          while IFS= read -r -d '' file; do
+            # Skip if it's already a .sh file
+            if [[ "$file" != *.sh ]]; then
+              bash_files+=("$file")
+            fi
+          done < <(find . -type f -not -path "./src/node_modules/*" -not -path "./.git/*" -not -path "./.augment/*" -exec grep -l "^#!/bin/bash\|^#!/usr/bin/env bash" {} \; -print0 2>/dev/null)
+
+          if [ ${#bash_files[@]} -eq 0 ]; then
+            echo "❌ No bash scripts found in the repository"
+            exit 1
+          fi
+
+          echo "📋 Found ${#bash_files[@]} bash script(s) to validate:"
+          printf '  - %s\n' "${bash_files[@]}"
+          echo
+
+          # Validate syntax of each script
+          failed_scripts=()
+          for script in "${bash_files[@]}"; do
+            echo "🔍 Checking syntax of: $script"
+            if bash -n "$script" 2>/dev/null; then
+              echo "✅ $script - syntax OK"
+            else
+              echo "❌ $script - syntax ERROR:"
+              bash -n "$script" 2>&1 | sed 's/^/    /'
+              failed_scripts+=("$script")
+            fi
+            echo
+          done
+
+          # Report results
+          if [ ${#failed_scripts[@]} -eq 0 ]; then
+            echo "🎉 All bash scripts passed syntax validation!"
+          else
+            echo "💥 ${#failed_scripts[@]} script(s) failed syntax validation:"
+            printf '  - %s\n' "${failed_scripts[@]}"
+            echo
+            echo "Please fix the syntax errors in the above scripts before merging."
+            exit 1
+          fi
+
       - name: Lint
         working-directory: ratos-configurator/src
         run: pnpm run lint:ci