Avoid mutating task steps, extract hasName slice instead

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: silverwind

routers/web/repo/actions/view.go

@@ -318,22 +318,27 @@ func convertToViewModel(ctx *context_module.Context, cursors []LogCursor, task *
 	var viewJobs []*ViewJobStep
 	var logs []*ViewStepLog
 
-	// Add "Run" prefix for unnamed steps
+	steps := actions.FullSteps(task)
+
+	// Parse workflow to identify unnamed steps that need a "Run" prefix.
+	// FullSteps prepends a "Set up job" step, so workflow step index is i-1.
+	var hasName []bool
 	if task.Job != nil {
 		if wj, err := task.Job.ParseJob(); err == nil {
-			for i, step := range task.Steps {
-				if i < len(wj.Steps) && wj.Steps[i].Name == "" {
-					step.Name = ctx.Locale.TrString("actions.runs.run", step.Name)
-				}
+			hasName = make([]bool, len(wj.Steps))
+			for i, ws := range wj.Steps {
+				hasName[i] = ws.Name != ""
 			}
 		}
 	}
 
-	steps := actions.FullSteps(task)
-
-	for _, v := range steps {
+	for i, v := range steps {
+		summary := v.Name
+		if wi := i - 1; wi >= 0 && wi < len(hasName) && !hasName[wi] {
+			summary = ctx.Locale.TrString("actions.runs.run", summary)
+		}
 		viewJobs = append(viewJobs, &ViewJobStep{
-			Summary:  v.Name,
+			Summary:  summary,
 			Duration: v.Duration().String(),
 			Status:   v.Status.String(),
 		})