refactor: don't use deprecated .Completed property

Author: yjp20

pkg/cmd/build.go

@@ -523,7 +523,7 @@ func (c buildCompletionModel) IsCompleted() bool {
 
 		// Check if download is completed (if applicable)
 		downloadIsCompleted := true
-		if buildTarget.IsCommitCompleted() && stainlessutils.IsGoodCommitConclusion(buildTarget.Commit.Completed.Conclusion) {
+		if buildTarget.IsCommitCompleted() && buildTarget.IsGoodCommitConclusion() {
 			if download, ok := c.Build.Downloads[target]; ok {
 				downloadIsCompleted = download.Status == "completed"
 			}

pkg/stainlessutils/stainlessutils.go

@@ -188,18 +188,18 @@ func (bt *BuildTarget) StepInfo(step string) (status, url, conclusion string) {
 	if u, ok := stepUnion.(stainless.BuildTargetCommitUnion); ok {
 		status = u.Status
 		if u.Status == "completed" {
-			conclusion = u.Completed.Conclusion
+			conclusion = u.Conclusion
 			// Use merge conflict PR URL if available, otherwise use commit URL
-			if u.Completed.JSON.MergeConflictPr.Valid() {
+			if u.JSON.MergeConflictPr.Valid() {
 				url = fmt.Sprintf("https://github.com/%s/%s/pull/%.0f",
-					u.Completed.MergeConflictPr.Repo.Owner,
-					u.Completed.MergeConflictPr.Repo.Name,
-					u.Completed.MergeConflictPr.Number)
-			} else if u.Completed.JSON.Commit.Valid() {
+					u.MergeConflictPr.Repo.Owner,
+					u.MergeConflictPr.Repo.Name,
+					u.MergeConflictPr.Number)
+			} else if u.JSON.Commit.Valid() {
 				url = fmt.Sprintf("https://github.com/%s/%s/commit/%s",
-					u.Completed.Commit.Repo.Owner,
-					u.Completed.Commit.Repo.Name,
-					u.Completed.Commit.Sha)
+					u.Commit.Repo.Owner,
+					u.Commit.Repo.Name,
+					u.Commit.Sha)
 			}
 		}
 	}