refactor: improve error handling and fix critical bugs

Enhance code robustness and fix several critical issues identified
during code review:

**Error Handling:**
- Add set -euo pipefail to all modules for consistent error detection
- Add sleep delays after partition refresh to prevent race conditions

**Bug Fixes:**
- Fix package verification flag from -R to -r (xbps-query)
- Fix swap file creation for btrfs (use dd instead of fallocate)
- Add atomic fstab updates with backup mechanism

**Optimizations:**
- Remove @swap btrfs subvolume, use regular directory instead
- Direct dd usage for btrfs swap files (COW-compatible)
- Optimize swap creation logic per filesystem type

**Security:**
- Add GRUB/LUKS2 compatibility check with version warning
- Create fstab backup before modifications

Changes affect 14 files across all installer phases. All changes
improve reliability, error detection, and filesystem compatibility.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

README.md

@@ -78,7 +78,8 @@ Bei Auswahl von btrfs werden automatisch folgende Subvolumes angelegt:
 - `@var` → `/var` (Variable Daten)
 - `@log` → `/var/log` (System-Logs)
 - `@snapshots` → `/.snapshots` (Snapshot-Speicher)
-- `@swap` → `/swap` (Swap-File Container)
+
+**Hinweis:** Das Swap-File wird direkt im Root-Subvolume unter `/swap/swapfile` erstellt (kein separates Subvolume), da dies bei btrfs performanter ist.
 
 ## Package-Liste
 

src/config.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Centralize user-provided inputs to avoid scattered magic values.

src/encryption.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Protect user data with full-disk encryption while keeping boot reliable.

src/filesystems.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Provide a stable filesystem base aligned with user intent.
@@ -83,7 +84,6 @@ format_filesystems() {
     btrfs subvolume create "$temp_mount/@var"
     btrfs subvolume create "$temp_mount/@log"
     btrfs subvolume create "$temp_mount/@snapshots"
-    btrfs subvolume create "$temp_mount/@swap"
     umount "$temp_mount"
     rmdir "$temp_mount"
     trap - EXIT

src/locale.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Configure system locale, timezone, and keyboard layout for international users.

src/logging.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Provide transparency and auditability during destructive operations.

src/mounts.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Present a clean mount tree to the installer.

src/packages.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Install all required packages for a bootable Void Linux system.
@@ -88,7 +89,7 @@ packages_install() {
   log_info "Verifying package installation..."
   local critical_packages=("base-system" "linux" "grub" "cryptsetup")
   for pkg in "${critical_packages[@]}"; do
-    if ! xbps-query -R "$MOUNT_ROOT" "$pkg" >/dev/null 2>&1; then
+    if ! xbps-query -r "$MOUNT_ROOT" "$pkg" >/dev/null 2>&1; then
       die "Critical package not found: $pkg"
     fi
   done

src/partitioning.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Establish a predictable disk layout that supports encrypted root.
@@ -53,12 +54,16 @@ refresh_partition_table() {
 
   if command -v partprobe >/dev/null 2>&1; then
     partprobe "$disk"
+    sleep 1
   elif command -v partx >/dev/null 2>&1; then
     partx -u "$disk"
+    sleep 1
   elif command -v udevadm >/dev/null 2>&1; then
     udevadm settle
+    sleep 1
   else
     log_warn "No partition refresh tool available; proceeding without refresh."
+    sleep 2
   fi
 }
 

src/postinstall.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Ensure the system can boot with encrypted root after installation.
@@ -121,20 +122,26 @@ update_fstab_entry() {
   fi
   local line="UUID=$uuid $mount_point $fs_type $options 0 0"
 
+  # Create backup before first modification
+  if [[ -f /etc/fstab ]] && [[ ! -f /etc/fstab.backup ]]; then
+    cp /etc/fstab /etc/fstab.backup
+    log_info "Created fstab backup at /etc/fstab.backup"
+  fi
+
   if awk -v mp="$mount_point" '$2==mp {found=1} END {exit found?0:1}' /etc/fstab; then
+    # Update existing entry
     awk -v mp="$mount_point" -v line="$line" 'BEGIN{OFS=" "} $2==mp {$0=line} {print}' /etc/fstab > /etc/fstab.tmp
+    # Atomic move with sync
+    sync /etc/fstab.tmp
     mv /etc/fstab.tmp /etc/fstab
   else
+    # Append new entry
     printf '%s\n' "$line" >> /etc/fstab
   fi
 }
 
 create_swapfile() {
   local swap_size="$1"
-
-  # Try fallocate first, fall back to dd if it fails
-  if ! fallocate -l "$swap_size" /swap/swapfile 2>/dev/null; then
-    echo "fallocate failed, using dd as fallback..."
   local size_num
   local size_unit
   local bs="1M"
@@ -149,7 +156,16 @@ create_swapfile() {
     *) echo "Warning: unknown size unit, assuming MiB"; count=$(awk "BEGIN {print int($size_num)}") ;;
   esac
 
+  # Use dd for btrfs, fallocate for ext4
+  if [[ "$FS_TYPE" == "btrfs" ]]; then
+    log_info "Creating swap file with dd (btrfs requires this)..."
     dd if=/dev/zero of=/swap/swapfile bs="$bs" count="$count" status=progress
+  else
+    log_info "Creating swap file with fallocate..."
+    if ! fallocate -l "$swap_size" /swap/swapfile 2>/dev/null; then
+      log_warn "fallocate failed, falling back to dd..."
+      dd if=/dev/zero of=/swap/swapfile bs="$bs" count="$count" status=progress
+    fi
   fi
 }
 
@@ -162,7 +178,6 @@ if [[ "$FS_TYPE" == "btrfs" ]]; then
   update_fstab_entry /var "btrfs" "defaults,subvol=@var" "$ROOT_UUID"
   update_fstab_entry /var/log "btrfs" "defaults,subvol=@log" "$ROOT_UUID"
   update_fstab_entry /.snapshots "btrfs" "defaults,subvol=@snapshots" "$ROOT_UUID"
-  update_fstab_entry /swap "btrfs" "defaults,subvol=@swap" "$ROOT_UUID"
 else
   update_fstab_entry / "ext4" "defaults" "$ROOT_UUID"
 fi
@@ -187,6 +202,23 @@ if [[ "$SWAP_SIZE" != "0" ]]; then
   fi
 fi
 
+# Check GRUB version for LUKS2 compatibility
+if [[ "${LUKS_VERSION:-2}" == "2" ]]; then
+  log_info "Checking GRUB version for LUKS2 compatibility..."
+  if command -v grub-install >/dev/null 2>&1; then
+    grub_version=$(grub-install --version 2>/dev/null | grep -oP '\d+\.\d+' | head -n1)
+    if [[ -n "$grub_version" ]]; then
+      # Compare version (2.06 is minimum for reliable LUKS2 support)
+      if awk "BEGIN {exit !($grub_version < 2.06)}"; then
+        log_warn "GRUB version $grub_version detected. LUKS2 requires GRUB >= 2.06 for reliable boot support."
+        log_warn "Consider using LUKS version 1 if you experience boot issues."
+      else
+        log_info "GRUB version $grub_version - LUKS2 support OK"
+      fi
+    fi
+  fi
+fi
+
 # Ensure GRUB unlocks the encrypted root and passes the LUKS UUID to initramfs.
 mkdir -p /etc/default
 touch /etc/default/grub

src/rollback.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Reduce fallout if a phase fails mid-way.

src/sanity.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Prevent catastrophic mistakes before any disk operations.

src/services.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Enable essential system services for network connectivity and basic functionality.

src/users.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+set -euo pipefail
 
 # === Motivation ===
 # Create user accounts with proper permissions for system access.