fix: only retry ref updates on fast-forward conflicts

Avoid retrying unrelated 422 errors during push_files and delete_file
ref updates. Match hyphenated fast-forward messages from the GitHub API.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: root

pkg/github/repositories_helper.go

@@ -407,20 +407,15 @@ func closeGitHubResponse(resp *github.Response) {
 }
 
 func isGitRefUpdateConflict(err error, resp *github.Response) bool {
-	statusCode := 0
-	if resp != nil && resp.Response != nil {
-		statusCode = resp.StatusCode
-	}
 	if ghErr, ok := err.(*github.ErrorResponse); ok {
-		if ghErr.Response != nil {
-			statusCode = ghErr.Response.StatusCode
-		}
 		msg := strings.ToLower(ghErr.Message)
-		if strings.Contains(msg, "fast forward") || strings.Contains(msg, "not a fast forward") {
+		if strings.Contains(msg, "fast forward") ||
+			strings.Contains(msg, "fast-forward") ||
+			strings.Contains(msg, "not a fast forward") {
 			return true
 		}
 	}
-	return statusCode == http.StatusUnprocessableEntity
+	return false
 }
 
 type gitPushResult struct {

pkg/github/repositories_helper_test.go

@@ -82,6 +82,15 @@ func Test_isGitRefUpdateConflict(t *testing.T) {
 		Response: resp422.Response,
 		Message:  "Update is not a fast forward",
 	}, resp422))
+	assert.True(t, isGitRefUpdateConflict(&github.ErrorResponse{
+		Response: resp422.Response,
+		Message:  "Cannot fast-forward",
+	}, resp422))
+
+	assert.False(t, isGitRefUpdateConflict(&github.ErrorResponse{
+		Response: resp422.Response,
+		Message:  "Reference does not exist",
+	}, resp422))
 
 	resp200 := &github.Response{Response: &http.Response{StatusCode: http.StatusOK}}
 	assert.False(t, isGitRefUpdateConflict(nil, resp200))