fixed result formating and return if result exist

Author: yashsinghcodes

shared.go

@@ -23451,20 +23451,20 @@ func ActionSkip(ctx context.Context, foundAction Action, exec *WorkflowExecution
 	_, actionResult := GetActionResult(ctx, *exec, foundAction.ID)
 	if actionResult.Action.ID == foundAction.ID {
 		log.Printf("[DEBUG][%s] Result already exist for the action %s (%s)", exec.ExecutionId, foundAction.Label, foundAction.ID)
+		return nil
 	}
 
 	newResult := ActionResult{
 		Action:        foundAction,
 		ExecutionId:   exec.ExecutionId,
 		Authorization: exec.Authorization,
-		Result:        fmt.Sprintf(`{"success": false, "reason": "Skipped because of previous node - %d" - %v}`, len(parent), parent),
+		Result:        fmt.Sprintf(`{"success": false, "reason": "Skipped because of previous node - %d - %v"}`, len(parent), parent),
 		StartedAt:     0,
 		CompletedAt:   0,
 		Status:        "SKIPPED",
 	}
 	resultData, err := json.Marshal(newResult)
 	if err != nil {
-		log.Printf("[ERROR] Failed skipping action")
 		return err
 	}