fix: remove --ignore-scripts for crush install and add crush --version step

pkg/workflow/crush_engine.go

@@ -59,13 +59,34 @@ func (e *CrushEngine) GetInstallationSteps(workflowData *WorkflowData) []GitHubA
 		return []GitHubActionStep{}
 	}
 
-	npmSteps := BuildStandardNpmEngineInstallSteps(
+	// Use version from engine config if provided, otherwise default to pinned version
+	version := string(constants.DefaultCrushVersion)
+	if workflowData.EngineConfig != nil && workflowData.EngineConfig.Version != "" {
+		version = workflowData.EngineConfig.Version
+	}
+
+	// Crush requires post-install scripts (native binaries) so --ignore-scripts must
+	// NOT be passed. This is intentionally different from other engine installs.
+	npmSteps := GenerateNpmInstallSteps(
 		"@charmland/crush",
-		string(constants.DefaultCrushVersion),
+		version,
 		"Install Crush CLI",
 		"crush",
-		workflowData,
+		true, // Include Node.js setup
+		true, // Crush requires post-install scripts for native binaries
 	)
+
+	// Run crush --version to verify the installation and force any deferred binary downloads
+	commandName := "crush"
+	if workflowData.EngineConfig != nil && workflowData.EngineConfig.Command != "" {
+		commandName = workflowData.EngineConfig.Command
+	}
+	versionStep := GitHubActionStep{
+		"      - name: Verify Crush CLI installation",
+		"        run: " + commandName + " --version",
+	}
+	npmSteps = append(npmSteps, versionStep)
+
 	return BuildNpmEngineInstallStepsWithAWF(npmSteps, workflowData)
 }
 

pkg/workflow/crush_engine_test.go

@@ -161,8 +161,32 @@ func TestCrushEngineInstallation(t *testing.T) {
 		steps := engine.GetInstallationSteps(workflowData)
 		require.NotEmpty(t, steps, "Should generate installation steps")
 
-		// Should have at least: Node.js setup + Install Crush
-		assert.GreaterOrEqual(t, len(steps), 2, "Should have at least 2 installation steps")
+		// Should have at least: Node.js setup + Install Crush + Verify Crush CLI installation
+		assert.GreaterOrEqual(t, len(steps), 3, "Should have at least 3 installation steps")
+
+		// Find install step and verify --ignore-scripts is NOT present (Crush needs post-install scripts for native binaries)
+		var installStep string
+		for _, step := range steps {
+			content := strings.Join(step, "\n")
+			if strings.Contains(content, "@charmland/crush@") {
+				installStep = content
+				break
+			}
+		}
+		require.NotEmpty(t, installStep, "Should find a step installing @charmland/crush")
+		assert.NotContains(t, installStep, "--ignore-scripts", "Should not use --ignore-scripts for Crush (requires post-install scripts for native binaries)")
+
+		// Find crush --version step to confirm binary download is forced
+		var versionStep string
+		for _, step := range steps {
+			content := strings.Join(step, "\n")
+			if strings.Contains(content, "crush --version") {
+				versionStep = content
+				break
+			}
+		}
+		require.NotEmpty(t, versionStep, "Should find crush --version step to force binary download")
+		assert.Contains(t, versionStep, "Verify Crush CLI installation", "Should have a descriptive step name")
 	})
 
 	t.Run("custom command skips installation", func(t *testing.T) {