Investigate bug root cause (#2005)

* Fix nil pointer dereference in comment updater for terragrunt projects

Co-authored-by: mo <mo@digger.dev>

* Fix nil pointer dereference in comment updater workflow URL handling

Co-authored-by: mo <mo@digger.dev>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: mo <mo@digger.dev>

Author: 3 people

ee/cli/pkg/comment_updater/updater.go

@@ -38,8 +38,14 @@ func (a AdvancedCommentUpdater) UpdateComment(jobs []scheduler.SerializedJob, pr
 		}
 		isPlan := jobSpec.IsPlan()
 
+		// Safe handling of WorkflowRunUrl pointer
+		workflowUrl := "#"
+		if job.WorkflowRunUrl != nil {
+			workflowUrl = *job.WorkflowRunUrl
+		}
+
 		message = message + fmt.Sprintf("<!-- PROJECTHOLDER %v -->\n", job.ProjectName)
-		message = message + fmt.Sprintf("%v **%v** <a href='%v'>%v</a>%v %v\n", job.Status.ToEmoji(), jobSpec.ProjectName, *job.WorkflowRunUrl, job.Status.ToString(), job.ResourcesSummaryString(isPlan), DriftSummaryString(job.ProjectName, issuesMap))
+		message = message + fmt.Sprintf("%v **%v** <a href='%v'>%v</a>%v %v\n", job.Status.ToEmoji(), jobSpec.ProjectName, workflowUrl, job.Status.ToString(), job.ResourcesSummaryString(isPlan), DriftSummaryString(job.ProjectName, issuesMap))
 		message = message + fmt.Sprintf("<!-- PROJECTHOLDEREND %v -->\n", job.ProjectName)
 	}
 

libs/comment_utils/summary/updater.go

@@ -41,10 +41,17 @@ func (b BasicCommentUpdater) UpdateComment(jobs []scheduler.SerializedJob, prNum
 	for i, job := range jobs {
 		jobSpec := jobSpecs[i]
 		prCommentUrl := job.PRCommentUrl
+		
+		// Safe handling of WorkflowRunUrl pointer
+		workflowUrl := "#"
+		if job.WorkflowRunUrl != nil {
+			workflowUrl = *job.WorkflowRunUrl
+		}
+		
 		message = message + fmt.Sprintf("|%v **%v** |<a href='%v'>%v</a> | <a href='%v'>%v</a> | %v | %v | %v|\n",
 			job.Status.ToEmoji(),
 			jobSpec.ProjectName,
-			*job.WorkflowRunUrl,
+			workflowUrl,
 			job.Status.ToString(),
 			prCommentUrl,
 			jobTypeTitle,