refactor: Use msg_* helpers consistently

Replace echo with emoji (❌, ⚠️, ✓) with msg_* functions.
Added msg_* to omarchy-looking-glass-install.

Author: slawomir-andreasik

bin/omarchy-gpu-passthrough-bind

@@ -78,7 +78,7 @@ check_gpu_processes() {
           echo ""
           sudo fuser -v /dev/nvidia* 2>&1 | grep -iE "Hyprland|USER" | head -10
           echo ""
-          msg_warn "Solution: Run from TTY or SSH"
+          msg_warning "Solution: Run from TTY or SSH"
           echo "  1. Switch to TTY (Ctrl+Alt+F2) or SSH from another machine"
           echo "  2. Run: omarchy-gpu-passthrough mode vm"
           echo "  3. Start VM from there"
@@ -157,7 +157,7 @@ check_gpu_processes() {
       echo ""
     fi
 
-    msg_warn "GPU is bound to vfio-pci and in use (likely by running VM)"
+    msg_warning "GPU is bound to vfio-pci and in use (likely by running VM)"
     echo ""
     echo "To unbind GPU:"
     echo "  1. Stop your VM: omarchy-windows-vm stop"
@@ -253,7 +253,7 @@ check_display_safety() {
           echo ""
           msg_error "Detected $connected_displays monitor(s) on $gpu_drm_card - unbind will cause BLACKSCREEN!"
           echo ""
-          msg_warn "Solution:"
+          msg_warning "Solution:"
           echo "  1. Move monitor cable to iGPU port (motherboard HDMI/DP)"
           echo "  2. Set BIOS: Primary Display = IGD"
           echo "  3. Reboot and run: omarchy-gpu-passthrough setup"
@@ -651,7 +651,7 @@ cmd_unbind() {
       log_warn "Failed to update state marker (continuing anyway)"
     fi
   else
-    msg_warn "Unbound but driver not auto-loaded (current: ${current_driver:-none})"
+    msg_warning "Unbound but driver not auto-loaded (current: ${current_driver:-none})"
     msg_info "Manual reload: sudo modprobe -i $GPU_DRIVER_ORIGINAL"
     log_warn "=== UNBIND PARTIAL: driver not loaded (${current_driver:-none}) ==="
 

bin/omarchy-gpu-passthrough-info

@@ -42,7 +42,7 @@ verify_hardware_requirements() {
     fi
     [[ "$quiet" != "true" ]] && msg_success "  $GPU_COUNT GPUs detected$iommu_status"
   elif [[ "$GPU_COUNT" -eq 1 ]]; then
-    [[ "$quiet" != "true" ]] && msg_warn "  Only 1 GPU detected (need 2+ for passthrough)"
+    [[ "$quiet" != "true" ]] && msg_warning "  Only 1 GPU detected (need 2+ for passthrough)"
     if [[ "$is_configured" == "true" ]]; then
       ((errors++))
     fi
@@ -125,7 +125,7 @@ verify_kernel_config() {
     if echo "$limine_params" | grep -qE "amd_iommu=on|intel_iommu=on"; then
       msg_success "    IOMMU parameters present ✓"
     else
-      msg_warn "    IOMMU parameters NOT in config - run setup to reconfigure"
+      msg_warning "    IOMMU parameters NOT in config - run setup to reconfigure"
     fi
   fi
 
@@ -237,7 +237,7 @@ verify_setup() {
         msg_success "$status_msg (ready for VM) ✓"
         ((vfio_gpus++))
       elif [[ "$driver" == "none" ]]; then
-        msg_warn "$status_msg (no driver)"
+        msg_warning "$status_msg (no driver)"
       else
         if [[ "$type" == "integrated" ]]; then
           msg_info "$status_msg (host display)"
@@ -263,7 +263,7 @@ verify_setup() {
       [[ -n "$renderer" ]] && msg_success "  Renderer: $renderer"
       [[ -n "$version" ]] && echo "  OpenGL:   $version"
     else
-      msg_warn "  Unable to query OpenGL info"
+      msg_warning "  Unable to query OpenGL info"
     fi
   fi
 
@@ -282,9 +282,9 @@ verify_setup() {
       msg_success "Status: Ready for GPU passthrough! ✓"
       echo ""
       echo "GPU passthrough is properly configured:"
-      echo "  ✓ $vfio_gpus GPU(s) bound to vfio-pci"
-      echo "  ✓ IOMMU enabled and configured"
-      echo "  ✓ Kernel parameters active"
+      msg_success "$vfio_gpus GPU(s) bound to vfio-pci"
+      msg_success "IOMMU enabled and configured"
+      msg_success "Kernel parameters active"
     elif [[ "$total_errors" -eq 0 ]] && [[ "$vfio_gpus" -eq 0 ]]; then
       # Detect current GPU mode (dynamic binding design supports 3 states)
       local gpu_mode="unknown"
@@ -298,8 +298,8 @@ verify_setup() {
         msg_success "Status: GPU in boot state (dynamic binding) ✓"
         echo ""
         echo "GPU is blacklisted and inactive (by design)."
-        echo "  ✓ IOMMU enabled and configured"
-        echo "  ✓ Kernel parameters active"
+        msg_success "IOMMU enabled and configured"
+        msg_success "Kernel parameters active"
         echo ""
         echo "To use GPU:"
         echo "  • For VM: omarchy-gpu-passthrough mode vm"
@@ -309,15 +309,15 @@ verify_setup() {
         msg_success "Status: GPU bound to native driver ✓"
         echo ""
         echo "GPU is available for host use (CUDA/compute)."
-        echo "  ✓ IOMMU enabled and configured"
-        echo "  ✓ GPU bound to native driver"
+        msg_success "IOMMU enabled and configured"
+        msg_success "GPU bound to native driver"
         echo ""
         echo "To use GPU in VM:"
         echo "  • Run: omarchy-gpu-passthrough mode vm"
         ;;
       *)
         # Unknown state (shouldn't happen, but keep old message as fallback)
-        msg_warn "Status: Configured (GPU mode: $gpu_mode)"
+        msg_warning "Status: Configured (GPU mode: $gpu_mode)"
         echo ""
         echo "Configuration exists. GPU not bound to vfio-pci."
         echo "  Hint: Use 'omarchy-gpu-passthrough mode vm' to bind for VM"
@@ -715,7 +715,7 @@ cmd_detect() {
   if check_iommu_support; then
     msg_success "IOMMU: Enabled"
   else
-    msg_warn "IOMMU: Not detected (enable in BIOS)"
+    msg_warning "IOMMU: Not detected (enable in BIOS)"
   fi
 }
 
@@ -727,7 +727,7 @@ cmd_status() {
   if ! load_gpu_config 2>/dev/null; then
     # Not configured - show helpful message
     echo ""
-    msg_warn "GPU passthrough not configured"
+    msg_warning "GPU passthrough not configured"
     echo ""
     echo "Next steps:"
     echo "  1. omarchy-gpu-passthrough setup"

bin/omarchy-gpu-passthrough-setup

@@ -48,7 +48,7 @@ create_backup_snapshot() {
 
   # Validate user_home before using in path
   if [[ -z "$user_home" ]]; then
-    msg_warn "Cannot determine home directory for user $real_user"
+    msg_warning "Cannot determine home directory for user $real_user"
     log_warn "Skipping Hyprland config backup - user home not found"
   elif [[ -f "$user_home/.config/hypr/hyprland.conf" ]]; then
     sudo cp -a "$user_home/.config/hypr/hyprland.conf" "$snapshot_dir/hyprland.conf" 2>/dev/null && files_backed_up+=("hyprland.conf")
@@ -148,7 +148,7 @@ configure_omarchy_limine() {
     echo ""
     msg_error "Conflicting IOMMU parameter detected (iommu=off)"
     msg_info "Found IOMMU explicitly disabled in kernel parameters"
-    msg_warn "Please edit $LIMINE_DEFAULT and remove iommu=off parameter"
+    msg_warning "Please edit $LIMINE_DEFAULT and remove iommu=off parameter"
     echo ""
     log_error "Setup aborted: Conflicting IOMMU parameter (iommu=off) found in Limine config"
     return 1
@@ -191,7 +191,7 @@ configure_gpu_blacklist() {
   1002) gpu_vendor="amdgpu" ;;
   8086) gpu_vendor="i915" ;;
   *)
-    msg_warn "Unknown GPU vendor: $vendor_id"
+    msg_warning "Unknown GPU vendor: $vendor_id"
     return 1
     ;;
   esac
@@ -287,7 +287,7 @@ EOF
     msg_info "Adding user to kvm group..."
     if sudo usermod -aG kvm "$USER"; then
       msg_success "User added to kvm group"
-      msg_warning "You must LOG OUT and LOG BACK IN for group changes to take effect"
+      msg_warninging "You must LOG OUT and LOG BACK IN for group changes to take effect"
     else
       msg_error "Failed to add user to kvm group"
       msg_info "  Run manually: sudo usermod -aG kvm $USER"
@@ -301,7 +301,7 @@ EOF
 
 configure_vfio_modules_mkinitcpio() {
   if [[ ! -f "$MKINITCPIO_CONF" ]]; then
-    msg_warn "$MKINITCPIO_CONF not found"
+    msg_warning "$MKINITCPIO_CONF not found"
     return 1
   fi
 
@@ -364,7 +364,7 @@ configure_system_environment() {
 
     if [[ -z "$igpu_card_path" ]]; then
       # DRM device not yet available - skip configuration but don't fail
-      msg_warn "Cannot resolve DRM device for iGPU ($igpu_pci) yet"
+      msg_warning "Cannot resolve DRM device for iGPU ($igpu_pci) yet"
       msg_info "  This is normal on first setup (iGPU driver may not be loaded)"
       msg_info "  Configuration will be applied automatically after reboot"
       msg_info "  If black screen persists, run setup again after reboot"
@@ -419,7 +419,7 @@ configure_hyprland_igpu() {
 
   local hypr_conf="$user_home/.config/hypr/hyprland.conf"
   if [[ ! -f "$hypr_conf" ]]; then
-    msg_warn "Hyprland config not found - skipping Hyprland configuration"
+    msg_warning "Hyprland config not found - skipping Hyprland configuration"
     log_info "Hyprland config not found at $hypr_conf - user may not use Hyprland"
     return 0
   fi
@@ -454,7 +454,7 @@ configure_hyprland_igpu() {
 
     if [[ -z "$igpu_card_path" ]]; then
       # DRM device not yet available - skip configuration but don't fail
-      msg_warn "Cannot resolve DRM device for iGPU ($igpu_pci) yet"
+      msg_warning "Cannot resolve DRM device for iGPU ($igpu_pci) yet"
       msg_info "  This is normal on first setup (iGPU driver may not be loaded)"
       msg_info "  Hyprland config will be updated automatically after reboot"
       msg_info "  If black screen persists, run setup again after reboot"
@@ -637,10 +637,10 @@ check_passthrough_feasibility() {
     echo ""
     echo "Detected virtualization: $VIRTUALIZATION_TYPE"
     echo ""
-    msg_warn "GPU passthrough requires bare metal (physical hardware)"
+    msg_warning "GPU passthrough requires bare metal (physical hardware)"
     echo ""
     echo "This tool is for:"
-    echo "  ✓ Physical Omarchy installations (bare metal)"
+    msg_success "Physical Omarchy installations (bare metal)"
     echo "  ✗ NOT for Omarchy running inside a VM"
     echo ""
     echo "If you want to use GPU inside a VM:"
@@ -775,7 +775,7 @@ validate_gpu_selection() {
   # Warn if selecting iGPU
   if [[ "$type" == "integrated" ]]; then
     msg_error "WARNING: iGPU selected - NOT recommended!"
-    msg_warn "Linux will have no display after passthrough"
+    msg_warning "Linux will have no display after passthrough"
     echo ""
 
     gum confirm "Continue anyway?" || return 1
@@ -808,7 +808,7 @@ validate_gpu_selection() {
 
   # Display monitor connection instructions
   echo ""
-  msg_warn "After reboot, dedicated GPU ($selected_gpu_name) will be INACTIVE!"
+  msg_warning "After reboot, dedicated GPU ($selected_gpu_name) will be INACTIVE!"
   echo ""
   echo "For host display:"
   echo "  • Connect monitor(s) to MOTHERBOARD ports (HDMI/DP on back panel)"
@@ -868,8 +868,8 @@ validate_gpu_selection() {
   if [[ "$igpu_monitor_count" -eq 0 ]] && [[ "$dgpu_monitor_count" -gt 0 ]]; then
     msg_error "CRITICAL: No monitors on iGPU, but monitors detected on dedicated GPU!"
     echo ""
-    msg_warn "After reboot, ALL your monitors will go BLANK (dedicated GPU blacklisted)."
-    msg_warn "You MUST connect at least one monitor to motherboard ports!"
+    msg_warning "After reboot, ALL your monitors will go BLANK (dedicated GPU blacklisted)."
+    msg_warning "You MUST connect at least one monitor to motherboard ports!"
     echo ""
     log_warn "Critical: No iGPU monitors, $dgpu_monitor_count monitor(s) on dGPU - user at risk of black screen"
 
@@ -879,7 +879,7 @@ validate_gpu_selection() {
     }
   elif [[ "$dgpu_monitor_count" -gt 0 ]] && [[ "$igpu_monitor_count" -gt 0 ]]; then
     msg_success "Good: $igpu_monitor_count monitor(s) on iGPU (will work after reboot)"
-    msg_warn "Note: $dgpu_monitor_count monitor(s) on dedicated GPU will go blank after reboot"
+    msg_warning "Note: $dgpu_monitor_count monitor(s) on dedicated GPU will go blank after reboot"
     echo ""
     log_info "User has $igpu_monitor_count iGPU monitors + $dgpu_monitor_count dGPU monitors"
 
@@ -896,7 +896,7 @@ validate_gpu_selection() {
       return 1
     }
   else
-    msg_warn "Cannot detect monitor connections - asking user to confirm"
+    msg_warning "Cannot detect monitor connections - asking user to confirm"
     log_warn "Cannot detect monitor connections"
 
     gum confirm "At least one monitor is connected to motherboard ports?" || {
@@ -929,15 +929,15 @@ cmd_setup() {
   echo "  • BIOS: Set Primary Display = IGD or Auto (recommended)"
   echo "  • Monitor: Connect to motherboard ports (iGPU), not GPU ports"
   echo ""
-  msg_warn "Important: Dedicated GPU will be blacklisted (mode=none at boot)"
+  msg_warning "Important: Dedicated GPU will be blacklisted (mode=none at boot)"
   echo "           Use 'omarchy-gpu-passthrough mode vm' to bind for VM"
   echo "           Use 'omarchy-gpu-passthrough mode host' to bind for host use (optional)"
   echo ""
 
   gum confirm "Continue with setup?" || exit 0
 
   echo ""
-  msg_warn "WARNING: This will modify critical system configuration!"
+  msg_warning "WARNING: This will modify critical system configuration!"
   echo ""
   echo "What will be changed:"
   echo "  • Kernel boot parameters (/etc/default/limine)"
@@ -980,11 +980,11 @@ cmd_setup() {
     ;;
   esac
   echo ""
-  msg_warn "If not enabled, GPU passthrough will NOT work!"
+  msg_warning "If not enabled, GPU passthrough will NOT work!"
   echo ""
 
   gum confirm "Have you enabled these settings in BIOS?" || {
-    msg_warn "Please enable IOMMU in BIOS first, then run setup again"
+    msg_warning "Please enable IOMMU in BIOS first, then run setup again"
     exit 0
   }
   echo ""
@@ -1042,7 +1042,7 @@ cmd_setup() {
     case "$vendor_id" in
     10de) # NVIDIA
       if ! command -v nvidia-smi &>/dev/null; then
-        msg_warn "  NVIDIA GPU detected but nvidia-smi not found"
+        msg_warning "  NVIDIA GPU detected but nvidia-smi not found"
         msg_info "    GPU: $name"
         msg_info "    This is OK if you haven't installed NVIDIA drivers yet"
         driver_warnings=$((driver_warnings + 1))
@@ -1054,7 +1054,7 @@ cmd_setup() {
       if [[ "$driver" == "amdgpu" ]]; then
         msg_success "  AMD GPU using amdgpu driver"
       elif [[ "$driver" == "none" ]]; then
-        msg_warn "  AMD GPU detected but no driver loaded"
+        msg_warning "  AMD GPU detected but no driver loaded"
         msg_info "    GPU: $name"
         msg_info "    This is OK if you haven't installed AMD drivers yet"
         driver_warnings=$((driver_warnings + 1))
@@ -1084,7 +1084,7 @@ cmd_setup() {
   # IOMMU not detected - warn but allow to continue
   if [[ "$feasibility_errors" -gt 0 ]]; then
     echo ""
-    msg_warn "IOMMU not currently detected - you will need to:"
+    msg_warning "IOMMU not currently detected - you will need to:"
     msg_info "  1. Complete this setup to configure kernel parameters"
     msg_info "  2. Enable IOMMU in BIOS (AMD-Vi or Intel VT-d)"
     msg_info "  3. Reboot"
@@ -1206,8 +1206,8 @@ cmd_setup() {
       exit 1
     }
   else
-    msg_warn "No iGPU detected, skipping Hyprland/environment configuration"
-    msg_warn "You may need to manually configure your compositor"
+    msg_warning "No iGPU detected, skipping Hyprland/environment configuration"
+    msg_warning "You may need to manually configure your compositor"
   fi
 
   # Rebuild initramfs with new configuration (Omarchy uses limine-mkinitcpio)
@@ -1248,7 +1248,7 @@ cmd_setup() {
   log_info "GPU configured: $name ($pci)"
   log_info "Next step: REBOOT REQUIRED"
 
-  msg_warn "SYSTEM REBOOT REQUIRED"
+  msg_warning "SYSTEM REBOOT REQUIRED"
   echo ""
   msg_info "Kernel parameters require reboot to take effect"
   echo ""
@@ -1262,7 +1262,7 @@ cmd_setup() {
   msg_info "No driver will be loaded until you manually bind it."
   echo ""
   echo ""
-  msg_warn "⚠️  IF YOU SEE BLACK SCREEN AFTER REBOOT:"
+  msg_warning "⚠️  IF YOU SEE BLACK SCREEN AFTER REBOOT:"
   echo ""
   echo "This can happen on first setup if iGPU driver wasn't loaded yet."
   echo ""
@@ -1299,7 +1299,7 @@ cmd_uninstall() {
   fi
 
   echo ""
-  msg_warn "This will remove ALL GPU passthrough configuration!"
+  msg_warning "This will remove ALL GPU passthrough configuration!"
   echo ""
   echo "What will be removed:"
   echo "  • Kernel parameters (amd_iommu, iommu=pt)"
@@ -1352,12 +1352,12 @@ cmd_uninstall() {
   echo "  • /etc/mkinitcpio.conf.backup.uninstall.*"
   echo "  • ~/.config/hypr/hyprland.conf.backup.uninstall.*"
   echo ""
-  msg_warn "REBOOT REQUIRED"
+  msg_warning "REBOOT REQUIRED"
   echo ""
   echo "After reboot:"
-  echo "  ✓ Your GPU will be available to Linux"
-  echo "  ✓ GPU drivers (nvidia/amdgpu) will load normally"
-  echo "  ✓ You can use your GPU for gaming, rendering, etc."
+  msg_success "Your GPU will be available to Linux"
+  msg_success "GPU drivers (nvidia/amdgpu) will load normally"
+  msg_success "You can use your GPU for gaming, rendering, etc."
   echo ""
   msg_info "To reboot now: sudo reboot"
 }