Fix PHP workflow execution issues

- Add automatic Docker image pulling in run_container_inner
- Implement smart container image selection for GitHub actions
- Fix shell command parsing to use bash -c for proper quote/pipe handling
- Map shivammathur/setup-php to composer:latest container
- Support complex commands with quotes, pipes, and substitutions

Fixes issues where:
1. Docker images required manual pulling
2. PHP actions used wrong Node.js containers
3. Commands like 'echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT' failed

.github/workflows/release.yml

@@ -42,7 +42,30 @@ jobs:
           cargo install git-cliff --force
       
       - name: Generate Changelog
-        run: git-cliff --latest --output CHANGELOG.md
+        run: |
+          # Debug: Show current state
+          echo "Current ref: ${{ github.ref_name }}"
+          echo "Input version: ${{ github.event.inputs.version }}"
+          echo "All tags:"
+          git tag --sort=-version:refname | head -10
+          
+          # Generate changelog from the current tag to the previous version tag
+          CURRENT_TAG="${{ github.event.inputs.version || github.ref_name }}"
+          PREVIOUS_TAG=$(git tag --sort=-version:refname | grep "^v" | head -2 | tail -1)
+          
+          echo "Current tag: $CURRENT_TAG"
+          echo "Previous tag: $PREVIOUS_TAG"
+          
+          if [ -n "$PREVIOUS_TAG" ] && [ "$PREVIOUS_TAG" != "$CURRENT_TAG" ]; then
+            echo "Generating changelog for range: $PREVIOUS_TAG..$CURRENT_TAG"
+            git-cliff --tag "$CURRENT_TAG" "$PREVIOUS_TAG..$CURRENT_TAG" --output CHANGELOG.md
+          else
+            echo "Generating latest changelog for tag: $CURRENT_TAG"
+            git-cliff --tag "$CURRENT_TAG" --latest --output CHANGELOG.md
+          fi
+          
+          echo "Generated changelog:"
+          cat CHANGELOG.md
       
       - name: Create Release
         id: create_release

crates/executor/src/docker.rs

@@ -840,6 +840,14 @@ impl DockerRuntime {
         working_dir: &Path,
         volumes: &[(&Path, &Path)],
     ) -> Result<ContainerOutput, ContainerError> {
+        // First, try to pull the image if it's not available locally
+        if let Err(e) = self.pull_image_inner(image).await {
+            wrkflw_logging::warning(&format!(
+                "Failed to pull image {}: {}. Attempting to continue with existing image.",
+                image, e
+            ));
+        }
+
         // Collect environment variables
         let mut env: Vec<String> = env_vars
             .iter()

crates/executor/src/engine.rs

@@ -526,9 +526,64 @@ async fn prepare_action(
         }
     }
 
-    // GitHub action: use standard runner image
-    // In a real implementation, you'd need to clone the repo at the specified version
-    Ok("node:16-buster-slim".to_string())
+    // GitHub action: determine appropriate image based on action type
+    let image = determine_action_image(&action.repository);
+    Ok(image)
+}
+
+/// Determine the appropriate Docker image for a GitHub action
+fn determine_action_image(repository: &str) -> String {
+    // Handle specific well-known actions
+    match repository {
+        // PHP setup actions
+        repo if repo.starts_with("shivammathur/setup-php") => {
+            "composer:latest".to_string() // Use composer image which includes PHP and composer
+        }
+
+        // Python setup actions
+        repo if repo.starts_with("actions/setup-python") => "python:3.11-slim".to_string(),
+
+        // Node.js setup actions
+        repo if repo.starts_with("actions/setup-node") => "node:20-slim".to_string(),
+
+        // Java setup actions
+        repo if repo.starts_with("actions/setup-java") => "eclipse-temurin:17-jdk".to_string(),
+
+        // Go setup actions
+        repo if repo.starts_with("actions/setup-go") => "golang:1.21-slim".to_string(),
+
+        // .NET setup actions
+        repo if repo.starts_with("actions/setup-dotnet") => {
+            "mcr.microsoft.com/dotnet/sdk:7.0".to_string()
+        }
+
+        // Rust setup actions
+        repo if repo.starts_with("actions-rs/toolchain")
+            || repo.starts_with("dtolnay/rust-toolchain") =>
+        {
+            "rust:latest".to_string()
+        }
+
+        // Docker/container actions
+        repo if repo.starts_with("docker/") => "docker:latest".to_string(),
+
+        // AWS actions
+        repo if repo.starts_with("aws-actions/") => "amazon/aws-cli:latest".to_string(),
+
+        // Default to Node.js for most GitHub actions (checkout, upload-artifact, etc.)
+        _ => {
+            // Check if it's a common core GitHub action that should use a more complete environment
+            if repository.starts_with("actions/checkout")
+                || repository.starts_with("actions/upload-artifact")
+                || repository.starts_with("actions/download-artifact")
+                || repository.starts_with("actions/cache")
+            {
+                "catthehacker/ubuntu:act-latest".to_string() // Use act runner image for core actions
+            } else {
+                "node:16-buster-slim".to_string() // Default for other actions
+            }
+        }
+    }
 }
 
 async fn execute_job_batch(
@@ -1536,8 +1591,9 @@ async fn execute_step(ctx: StepExecutionContext<'_>) -> Result<StepResult, Execu
         // Check if this is a cargo command
         let is_cargo_cmd = run.trim().starts_with("cargo");
 
-        // Convert command string to array of string slices
-        let cmd_parts: Vec<&str> = run.split_whitespace().collect();
+        // For complex shell commands, use bash to execute them properly
+        // This handles quotes, pipes, redirections, and command substitutions correctly
+        let cmd_parts = vec!["bash", "-c", run];
 
         // Convert environment variables to the required format
         let env_vars: Vec<(&str, &str)> = step_env