Aliasing (#2113)

* ensure alias in user facing areas

* support alias in comment ops

* handle one line comments

* remove unintended change

* add alias to instructions, allow parsing

* remove changes - not target

* Update spec.go

remove space

* address comments

* remove further

* forgot to refactor

Author: breardon2011

backend/services/spec.go

@@ -100,11 +100,16 @@ func GetRunNameFromJob(job models.DiggerJob) (*string, error) {
 	batch := job.Batch
 	batchIdShort := batch.ID.String()[:8]
 	diggerCommand := fmt.Sprintf("digger %v", batch.BatchType)
+	// Use alias for display, keep original name for logging
 	projectName := jobSpec.ProjectName
+	projectDisplayName := jobSpec.ProjectName
+	if jobSpec.ProjectAlias != "" {
+		projectDisplayName = jobSpec.ProjectAlias
+	}
 	requestedBy := jobSpec.RequestedBy
 	prNumber := *jobSpec.PullRequestNumber
 
-	runName := fmt.Sprintf("[%v] %v %v By: %v PR: %v", batchIdShort, diggerCommand, projectName, requestedBy, prNumber)
+	runName := fmt.Sprintf("[%v] %v %v By: %v PR: %v", batchIdShort, diggerCommand, projectDisplayName, requestedBy, prNumber)
 	slog.Debug("Generated run name",
 		"jobId", job.DiggerJobID,
 		"runName", runName,

backend/utils/comment_utils.go

@@ -140,11 +140,16 @@ func GenerateRealtimeCommentMessage(jobs []models.DiggerJob, batchType orchestra
 
 		// Get project name from job spec
 		var jobSpec orchestrator_scheduler.JobJson
-		projectName := "Unknown"
+		projectDisplayName := "Unknown"
 		if job.SerializedJobSpec != nil {
 			err := json.Unmarshal(job.SerializedJobSpec, &jobSpec)
 			if err == nil {
-				projectName = jobSpec.ProjectName
+				// Use alias if available, fallback to project name
+				if jobSpec.ProjectAlias != "" {
+					projectDisplayName = jobSpec.ProjectAlias
+				} else {
+					projectDisplayName = jobSpec.ProjectName
+				}
 			} else {
 				slog.Warn("Failed to unmarshal job spec for project name", 
 					"jobId", job.DiggerJobID, 
@@ -167,7 +172,7 @@ func GenerateRealtimeCommentMessage(jobs []models.DiggerJob, batchType orchestra
 		// Match exact CLI format: |emoji **project** |<a href='workflow'>status</a> | <a href='comment'>jobType</a> | + | ~ | - |
 		message += fmt.Sprintf("|%s **%s** |<a href='%s'>%s</a> | <a href='%s'>%s</a> | %d | %d | %d|\n",
 			job.Status.ToEmoji(),
-			projectName,
+			projectDisplayName,
 			workflowUrl,
 			job.Status.ToString(),
 			prCommentUrl,

cli/pkg/spec/spec.go

@@ -99,7 +99,7 @@ func RunSpec(
 		reportError(spec, backendApi, message, err)
 	}
 
-	reporter, err := reporterProvider.GetReporter(fmt.Sprintf("%v for %v", spec.Job.JobType, job.ProjectName), spec.Reporter, prService, *spec.Job.PullRequestNumber, spec.VCS.VcsType)
+	reporter, err := reporterProvider.GetReporter(fmt.Sprintf("%v for %v", spec.Job.JobType, job.GetProjectAlias()), spec.Reporter, prService, *spec.Job.PullRequestNumber, spec.VCS.VcsType)
 	if err != nil {
 		message := fmt.Sprintf("could not get reporter: %v", err)
 		reportError(spec, backendApi, message, err)

libs/comment_utils/reporting/source_grouping.go

@@ -93,7 +93,9 @@ func (r SourceGroupingReporter) UpdateComment(sourceDetails []SourceDetails, loc
 			continue
 		}
 		expanded := i == 0 || !allSimilarInGroup
-		commenter := GetTerraformOutputAsCollapsibleComment(fmt.Sprintf("Plan for %v", project), expanded)
+		// Use alias for display with fallback to project name
+		displayName := scheduler.GetProjectAlias(job)
+		commenter := GetTerraformOutputAsCollapsibleComment(fmt.Sprintf("Plan for %v", displayName), expanded)
 		message = message + commenter(terraformOutputs[project]) + "\n"
 	}