improve summary error reporting

Author: motatoes

backend/controllers/projects.go

@@ -666,19 +666,28 @@ func CreateTerraformOutputsSummary(gh utils.GithubClientProvider, batch *models.
 		prService, err := GetPrServiceFromBatch(batch, gh)
 		if err != nil {
 			log.Printf("Error getting github service: %v", err)
+			prService.EditComment(batch.PrNumber, batch.AiSummaryCommentId, ":x: could not generate AI summary \n\n could not communicate with github")
 			return fmt.Errorf("error getting github service: %v", err)
 		}
 
+		if batch.AiSummaryCommentId == "" {
+			log.Printf("could not post summary comment, initial comment not found")
+			prService.PublishComment(batch.PrNumber, ":x: could not generate AI summary \n\n could not communicate with github")
+			return fmt.Errorf("could not post summary comment, initial comment not found")
+		}
+
 		summaryEndpoint := os.Getenv("DIGGER_AI_SUMMARY_ENDPOINT")
 		if summaryEndpoint == "" {
 			log.Printf("could not generate AI summary, ai summary endpoint missing")
+			prService.EditComment(batch.PrNumber, batch.AiSummaryCommentId, ":x: could not generate AI summary \n\n comment ID is missing ")
 			return fmt.Errorf("could not generate AI summary, ai summary endpoint missing")
 		}
 		apiToken := os.Getenv("DIGGER_AI_SUMMARY_API_TOKEN")
 
 		jobs, err := models.DB.GetDiggerJobsForBatch(batch.ID)
 		if err != nil {
 			log.Printf("could not get jobs for batch: %v", err)
+			prService.EditComment(batch.PrNumber, batch.AiSummaryCommentId, ":x: could not generate AI summary \n\n error fetching jobs ")
 			return fmt.Errorf("could not get jobs for batch: %v", err)
 		}
 
@@ -688,6 +697,7 @@ func CreateTerraformOutputsSummary(gh utils.GithubClientProvider, batch *models.
 			err := json.Unmarshal(job.SerializedJobSpec, &jobSpec)
 			if err != nil {
 				log.Printf("could not summarise plans due to unmarshalling error: %v", err)
+				prService.EditComment(batch.PrNumber, batch.AiSummaryCommentId, ":x: could not generate AI summary \n\n error fetching job spec")
 				return fmt.Errorf("could not summarise plans due to unmarshalling error: %v", err)
 			}
 			projectName := jobSpec.ProjectName
@@ -696,14 +706,10 @@ func CreateTerraformOutputsSummary(gh utils.GithubClientProvider, batch *models.
 		summary, err := utils.GetAiSummaryFromTerraformPlans(terraformOutputs, summaryEndpoint, apiToken)
 		if err != nil {
 			log.Printf("could not summarise terraform outputs: %v", err)
+			prService.EditComment(batch.PrNumber, batch.AiSummaryCommentId, ":x: could not generate AI summary \n\n error generating summary from plans")
 			return fmt.Errorf("could not summarise terraform outputs: %v", err)
 		}
 
-		if batch.AiSummaryCommentId == "" {
-			log.Printf("could not post summary comment, initial comment not found")
-			return fmt.Errorf("could not post summary comment, initial comment not found")
-		}
-
 		prService.EditComment(batch.PrNumber, batch.AiSummaryCommentId, summary)
 	}
 	return nil

libs/digger_config/digger_config.go

@@ -36,6 +36,9 @@ func ReadDiggerYmlFileContents(dir string) (string, error) {
 	if err != nil {
 		// if file doesn't exist look for digger.yaml instead
 		diggerYmlBytes, err = os.ReadFile(path.Join(dir, "digger.yaml"))
+		if err != nil {
+			return "", fmt.Errorf("could not read the file both digger.yml and digger.yaml are missing: %v", err)
+		}
 	}
 	diggerYmlStr := string(diggerYmlBytes)
 	return diggerYmlStr, nil