Added explanation to empty risks

cmd/comment.md

@@ -52,12 +52,15 @@ No expected changes found.
 
 [Open in Overmind]({{ .ChangeUrl }})
 
-{{ if .Risks }}
 # <img width="24" alt="warning" src="{{ .AssetPath }}/risks.svg"> Risks
+{{ if not .Risks }}
+Overmind has not identified any risks associated with this change.
 
+This could be due to the change being low risk with no impact on other parts of the system, or involving resources that Overmind currently does not support.
+{{ else -}}
 {{ range .Risks }}
 ## <img width="18" alt="{{ .SeverityAlt }}" src="{{ $top.AssetPath }}/{{ .SeverityIcon }}"> {{ .Title }} [{{ .SeverityText }}]
 
 {{ .Description }}
 {{ end }}
-{{ end }}
+{{ end -}}

cmd/root.go

@@ -215,14 +215,16 @@ func Execute() {
 			case loggedError:
 				log.WithContext(ctx).WithError(err.err).WithFields(err.fields).Error(err.message)
 			}
-			// if printing the error was not requested by the appropriate
-			// wrapper, only record the data to honeycomb and sentry, the
-			// command already has handled logging
-			cmdSpan.SetAttributes(
-				attribute.Bool("ovm.cli.fatalError", true),
-				attribute.String("ovm.cli.fatalError.msg", err.Error()),
-			)
-			cmdSpan.RecordError(err)
+			if cmdSpan != nil {
+				// if printing the error was not requested by the appropriate
+				// wrapper, only record the data to honeycomb and sentry, the
+				// command already has handled logging
+				cmdSpan.SetAttributes(
+					attribute.Bool("ovm.cli.fatalError", true),
+					attribute.String("ovm.cli.fatalError.msg", err.Error()),
+				)
+				cmdSpan.RecordError(err)
+			}
 			sentry.CaptureException(err)
 		}
 

cmd/tea_submitplan.go

@@ -795,7 +795,13 @@ func (m submitPlanModel) FinalReport() string {
 		bits = append(bits, styleH1().Render("Blast Radius"))
 		bits = append(bits, fmt.Sprintf("\nItems: %v\nEdges: %v\n", m.blastRadiusItems, m.blastRadiusEdges))
 	}
-	if m.changeUrl != "" && len(m.risks) > 0 {
+	if len(m.risks) == 0 {
+		bits = append(bits, styleH1().Render("Potential Risks"))
+		bits = append(bits, "")
+		bits = append(bits, "Overmind has not identified any risks associated with this change.")
+		bits = append(bits, "")
+		bits = append(bits, "This could be due to the change being low risk with no impact on other parts of the system, or involving resources that Overmind currently does not support.")
+	} else if m.changeUrl != "" {
 		bits = append(bits, styleH1().Render("Potential Risks"))
 		bits = append(bits, "")
 		for _, r := range m.risks {
@@ -836,5 +842,6 @@ func (m submitPlanModel) FinalReport() string {
 		}
 		bits = append(bits, fmt.Sprintf("\nCheck the blast radius graph and risks at:\n%v\n\n", m.changeUrl))
 	}
+
 	return strings.Join(bits, "\n") + "\n"
 }