Update start.sh

start.sh

@@ -1,78 +1,70 @@
 #!/bin/bash
 
-# Define the log file path using the environment variable from the Dockerfile
-# This ensures consistency with the Python application's logging display.
+# Define the log file path
 PLAYIT_LOG_FILE="${MINECRAFT_SERVER_DIR}/playit_error.log"
 
-# --- CRITICAL TROUBLESHOOTING STEP: Test Agent Execution ---
-# Forces the agent to run '--help' to ensure the binary is executable and writes to the log.
+# --- CRITICAL TROUBLESHOOTING: Binary Check & Immediate Failure Capture ---
+
 echo "==================================================" > "$PLAYIT_LOG_FILE"
-echo "Starting PlayIt Agent Execution Test..." >> "$PLAYIT_LOG_FILE"
+echo "STARTING PLAYIT AGENT DIAGNOSTICS" >> "$PLAYIT_LOG_FILE"
 echo "--------------------------------------------------" >> "$PLAYIT_LOG_FILE"
-# Execute playit with '--help' and capture all output (stdout and stderr)
-/usr/local/bin/playit --help >> "$PLAYIT_LOG_FILE" 2>&1
 
-# Check if the execution test was successful by looking for the 'Usage' string
-if grep -q "Usage: playit" "$PLAYIT_LOG_FILE"; then
-    echo "--------------------------------------------------" >> "$PLAYIT_LOG_FILE"
-    echo "PlayIt agent execution test SUCCESSFUL (Binary is running)." >> "$PLAYIT_LOG_FILE"
-    echo "--------------------------------------------------" >> "$PLAYIT_LOG_FILE"
-else
-    # If the test fails, print the error to the log and stop the tunnel process
+# 1. Use 'file' command to check if the binary is valid
+echo "1. Checking binary file type..." >> "$PLAYIT_LOG_FILE"
+file /usr/local/bin/playit >> "$PLAYIT_LOG_FILE" 2>&1
+
+# 2. Check execution permissions
+echo "2. Checking permissions and ownership..." >> "$PLAYIT_LOG_FILE"
+ls -l /usr/local/bin/playit >> "$PLAYIT_LOG_FILE" 2>&1
+
+# 3. CRITICAL: Try running a minimal command and capture the exit code
+echo "3. Testing basic execution and exit code..." >> "$PLAYIT_LOG_FILE"
+# Run with exec to capture any immediate low-level crash (e.g., missing library)
+/usr/local/bin/playit --version > /dev/null 2>&1
+EXEC_STATUS=$?
+echo "Exit Status of 'playit --version': $EXEC_STATUS" >> "$PLAYIT_LOG_FILE"
+
+if [ "$EXEC_STATUS" -ne 0 ]; then
     echo "--------------------------------------------------" >> "$PLAYIT_LOG_FILE"
-    echo "PlayIt agent execution test FAILED. Check binary permissions or integrity." >> "$PLAYIT_LOG_FILE"
+    echo "FATAL: PlayIt binary failed to execute (Exit Code: $EXEC_STATUS)." >> "$PLAYIT_LOG_FILE"
+    echo "This indicates a corrupted file, missing shared library, or wrong architecture." >> "$PLAYIT_LOG_FILE"
+    echo "PlayIt setup aborted." >> "$PLAYIT_LOG_FILE"
     echo "--------------------------------------------------" >> "$PLAYIT_LOG_FILE"
-    # Skip running the agent since the binary is broken
 fi
-# --- END TROUBLESHOOTING STEP ---
 
+# --- End Troubleshooting ---
 
 # --- 1. Start PlayIt.gg Agent in the background ---
 if [ -z "$PLAYIT_SECRET_KEY" ]; then
     echo "WARNING: PLAYIT_SECRET_KEY is not set. PlayIt tunnel will not start." >> "$PLAYIT_LOG_FILE"
-else
-    # Only run if the execution test was successful (i.e., we found the 'Usage' string)
-    if grep -q "SUCCESSFUL" "$PLAYIT_LOG_FILE"; then
-        echo "Starting PlayIt.gg Agent with secret key..." >> "$PLAYIT_LOG_FILE"
-        # Run the agent in the background, appending all output to the log file.
-        /usr/local/bin/playit run --secret-key "$PLAYIT_SECRET_KEY" --api-address 127.0.0.1:9000 >> "$PLAYIT_LOG_FILE" 2>&1 &
-        
-        # Wait up to 30 seconds for the tunnel to establish
-        for i in {1..30}; do
-            if grep -q ".playit.gg" "$PLAYIT_LOG_FILE"; then
-                echo "PlayIt.gg tunnel address found! Logs are now complete." >> "$PLAYIT_LOG_FILE"
-                break
-            fi
-            sleep 1
-        done
-    fi
+elif [ "$EXEC_STATUS" -eq 0 ]; then
+    # Only proceed if the basic execution test passed
+    echo "Starting PlayIt.gg Agent with 'run' command..." >> "$PLAYIT_LOG_FILE"
+    /usr/local/bin/playit run --secret-key "$PLAYIT_SECRET_KEY" --api-address 127.0.0.1:9000 >> "$PLAYIT_LOG_FILE" 2>&1 &
+    
+    # Wait for tunnel establishment
+    for i in {1..30}; do
+        if grep -q ".playit.gg" "$PLAYIT_LOG_FILE"; then
+            echo "PlayIt.gg tunnel address found! Logs are now complete." >> "$PLAYIT_LOG_FILE"
+            break
+        fi
+        sleep 1
+    done
 fi
 
 
-# --- 2. Display the PlayIt Tunnel Address (if found) ---
-# This part is for the console output when the app starts
+# --- 2. Display the PlayIt Tunnel Address (in the main console) ---
 echo "=================================================="
 echo "PLAYIT.GG LOGS (Tunnel Address):"
-
 if [ -f "$PLAYIT_LOG_FILE" ]; then
-    # Print the last few lines for immediate diagnosis in the console log
-    tail -n 10 "$PLAYIT_LOG_FILE"
-    
-    # Filter for the tunnel URL or relevant addresses in the console output
-    TUNNEL_ADDRESS=$(grep -E "available_addresses" "$PLAYIT_LOG_FILE" | head -n 1)
-    if [ -n "$TUNNEL_ADDRESS" ]; then
-        echo "---"
-        echo "Found Tunnel: $TUNNEL_ADDRESS"
-    else
-        echo "Tunnel address not yet available or failed to start."
-    fi
+    # Print the full log contents for immediate diagnosis
+    cat "$PLAYIT_LOG_FILE"
 else
-    echo "PlayIt.gg Agent log file not found."
+    echo "PlayIt.gg Agent log file was never created."
 fi
 echo "=================================================="
 
 
 # --- 3. Start the main Python Application (Flask + Minecraft) ---
 echo "Starting main Python application (app.py)..."
-# The 'exec' command replaces the current shell process with the python process
 exec python app.py