Make NoArtifactError.Error() nil-safe (#67)

Copilot · bdehamer · web-flow · commit 3195680f0346 · 2026-03-20T14:43:45.000-07:00
* Initial plan * Make NoArtifactError.Error() nil-safe Co-authored-by: bdehamer <398027+bdehamer@users.noreply.github.com> Agent-Logs-Url: https://github.com/github/deployment-tracker/sessions/3eb7150d-f2e1-4ef9-8452-686a5ddb22f0 --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: bdehamer <398027+bdehamer@users.noreply.github.com>
diff --git a/pkg/deploymentrecord/client.go b/pkg/deploymentrecord/client.go
@@ -170,6 +170,9 @@ type NoArtifactError struct {
 }
 
 func (n *NoArtifactError) Error() string {
+	if n == nil || n.err == nil {
+		return "no artifact found"
+	}
 	return fmt.Sprintf("no artifact found: %s", n.err.Error())
 }
 

Original file line number	Diff line number	Diff line change
`@@ -170,6 +170,9 @@ type NoArtifactError struct {`
`170`	`170`	`}`
`171`	`171`
`172`	`172`	`func (n *NoArtifactError) Error() string {`
	`173`	`+ if n == nil \|\| n.err == nil {`
	`174`	`+ return "no artifact found"`
	`175`	`+ }`
`173`	`176`	`return fmt.Sprintf("no artifact found: %s", n.err.Error())`
`174`	`177`	`}`
`175`	`178`