From 966709547f6c8bd5838e8058869da6170ea71811 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Wed, 24 Dec 2025 18:48:38 +0000
Subject: [PATCH] Fix upgrade: ensure dir exists, use sudo tee directly

- Create kiosk directory if it doesn't exist before extraction
- Use 'sudo tee' instead of 'sudo -u kiosk tee' for reliable writes
- Add line numbers to output for debugging
- chown at end fixes permissions
---
 install_kiosk_v0.9.9.1.sh | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/install_kiosk_v0.9.9.1.sh b/install_kiosk_v0.9.9.1.sh
index f41223e..c3dbcbd 100644
--- a/install_kiosk_v0.9.9.1.sh
+++ b/install_kiosk_v0.9.9.1.sh
@@ -10549,6 +10549,13 @@ upgrade_kiosk() {
 
     echo "[4/6] Extracting new app files from script..."
 
+    # Ensure kiosk directory exists with correct ownership
+    if [[ ! -d "$KIOSK_DIR" ]]; then
+        echo "  Creating $KIOSK_DIR..."
+        sudo mkdir -p "$KIOSK_DIR"
+    fi
+    sudo chown "$KIOSK_USER:$KIOSK_USER" "$KIOSK_DIR"
+
     # Helper function to extract heredoc content using line numbers
     extract_file() {
         local start_pattern="$1"
@@ -10579,13 +10586,14 @@ upgrade_kiosk() {
         local content_start=$((start_line + 1))
         local content_end=$((start_line + end_offset - 2))
 
-        sed -n "${content_start},${content_end}p" "$script_path" | sudo -u "$KIOSK_USER" tee "$output_file" > /dev/null
+        # Extract content and write to file (use sudo tee for reliability)
+        sed -n "${content_start},${content_end}p" "$script_path" | sudo tee "$output_file" > /dev/null
 
         if [[ -s "$output_file" ]]; then
-            echo "  ✓ $fname"
+            echo "  ✓ $fname (lines $content_start-$content_end)"
             return 0
         else
-            echo "  ✗ $fname (empty)"
+            echo "  ✗ $fname (extracted 0 lines from $content_start-$content_end)"
             return 1
         fi
     }