make install script more robust

Author: mkagenius

README.md

@@ -30,7 +30,7 @@ CodeRunner is an MCP (Model Context Protocol) server that executes AI-generated
 git clone https://github.com/instavm/coderunner.git
 cd coderunner
 chmod +x install.sh
-sudo ./install.sh
+./install.sh
 ```
 
 MCP server will be available at: http://coderunner.local:8222/mcp

install.sh

@@ -36,6 +36,7 @@ then
     fi
 
     echo "Stopping any running Apple 'container' processes..."
+    container system stop 2>/dev/null || true
 else
     echo "Apple 'container' tool not detected. Proceeding with installation..."
 
@@ -47,40 +48,69 @@ else
     sudo installer -pkg container-installer.pkg -target /
 fi
 
-echo "Starting the Sandbox Container..."
-container system start
+# Stop any existing container system to clean up stale connections
+echo "Stopping any existing container system..."
+container system stop 2>/dev/null || true
+
+# Wait a moment for cleanup
+sleep 2
+
+# Start the container system (this is blocking and will wait for kernel download if needed)
+echo "Starting the Sandbox Container system (this may take a few minutes if downloading kernel)..."
+if ! container system start; then
+    echo "❌ Failed to start container system."
+    exit 1
+fi
+
+# Quick verification that system is ready
+echo "Verifying container system is ready..."
+if container system status &>/dev/null; then
+    echo "✅ Container system is ready."
+else
+    echo "❌ Container system started but status check failed."
+    echo "Try running: container system stop && container system start"
+    exit 1
+fi
 
 echo "Setting up local network domain..."
 
 # Run the commands for setting up the local network
 echo "Running: sudo container system dns create local"
-sudo container system dns create local
+sudo container system dns create local 2>/dev/null || echo "DNS domain 'local' already exists (this is fine)"
 
-echo "Running: container system dns default set local"
+echo "Running: container system property set dns.domain local"
 container system property set dns.domain local
 
-echo "Starting the Sandbox Container..."
-container system start
-
 
 echo "Pulling the latest image: instavm/coderunner"
-container image pull instavm/coderunner
+if ! container image pull instavm/coderunner; then
+    echo "❌ Failed to pull image. Please check your internet connection and try again."
+    exit 1
+fi
 
 echo "→ Ensuring coderunner assets directories…"
 ASSETS_SRC="$HOME/.coderunner/assets"
 mkdir -p "$ASSETS_SRC/skills/user"
 mkdir -p "$ASSETS_SRC/outputs"
 
+# Stop any existing coderunner container
+echo "Stopping any existing coderunner container..."
+container stop coderunner 2>/dev/null || true
+sleep 2
+
 # Run the command to start the sandbox container
 echo "Running: container run --name coderunner --detach --rm --cpus 8 --memory 4g instavm/coderunner"
-container run \
+if container run \
   --volume "$ASSETS_SRC/skills/user:/app/uploads/skills/user" \
   --volume "$ASSETS_SRC/outputs:/app/uploads/outputs" \
   --name coderunner \
   --detach \
   --rm \
   --cpus 8 \
   --memory 4g \
-  instavm/coderunner
-
-echo "✅ Setup complete. MCP server is available at http://coderunner.local:8222/mcp"
+  instavm/coderunner; then
+    echo "✅ Setup complete. MCP server is available at http://coderunner.local:8222/mcp"
+else
+    echo "❌ Failed to start coderunner container. Please check the logs with: container logs coderunner"
+    exit 1
+fi