Potential fix for pull request finding

Not-Dhananjay-Mishra · Copilot · web-flow · commit 851bdef644db · 2026-04-21T18:41:23.000+05:30
Co-authored-by: Copilot Autofix powered by AI &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/pkg/github/search.go b/pkg/github/search.go
@@ -517,8 +517,7 @@ func SearchCommits(t translations.TranslationHelperFunc) inventory.ServerTool {
 				return ghErrors.NewGitHubAPIStatusErrorResponse(ctx, "failed to search commits", resp, body), nil, nil
 			}
 
-			minimalCommits := make([]MinimalCommit, 0, len(result.Commits))
-			for _, commit := range result.Commits {
+			convertCommitResultToMinimalCommit := func(commit *github.CommitResult) MinimalCommit {
 				minimalCommit := MinimalCommit{
 					SHA:     commit.GetSHA(),
 					HTMLURL: commit.GetHTMLURL(),
@@ -568,7 +567,12 @@ func SearchCommits(t translations.TranslationHelperFunc) inventory.ServerTool {
 					}
 				}
 
-				minimalCommits = append(minimalCommits, minimalCommit)
+				return minimalCommit
+			}
+
+			minimalCommits := make([]MinimalCommit, 0, len(result.Commits))
+			for _, commit := range result.Commits {
+				minimalCommits = append(minimalCommits, convertCommitResultToMinimalCommit(commit))
 			}
 
 			minimalResult := &MinimalSearchCommitsResult{

Original file line number	Diff line number	Diff line change
`@@ -517,8 +517,7 @@ func SearchCommits(t translations.TranslationHelperFunc) inventory.ServerTool {`
`517`	`517`	`return ghErrors.NewGitHubAPIStatusErrorResponse(ctx, "failed to search commits", resp, body), nil, nil`
`518`	`518`	`}`
`519`	`519`
`520`		`- minimalCommits := make([]MinimalCommit, 0, len(result.Commits))`
`521`		`- for _, commit := range result.Commits {`
	`520`	`+ convertCommitResultToMinimalCommit := func(commit *github.CommitResult) MinimalCommit {`
`522`	`521`	`minimalCommit := MinimalCommit{`
`523`	`522`	`SHA: commit.GetSHA(),`
`524`	`523`	`HTMLURL: commit.GetHTMLURL(),`
`@@ -568,7 +567,12 @@ func SearchCommits(t translations.TranslationHelperFunc) inventory.ServerTool {`
`568`	`567`	`}`
`569`	`568`	`}`
`570`	`569`
`571`		`- minimalCommits = append(minimalCommits, minimalCommit)`
	`570`	`+ return minimalCommit`
	`571`	`+ }`
	`572`	`+`
	`573`	`+ minimalCommits := make([]MinimalCommit, 0, len(result.Commits))`
	`574`	`+ for _, commit := range result.Commits {`
	`575`	`+ minimalCommits = append(minimalCommits, convertCommitResultToMinimalCommit(commit))`
`572`	`576`	`}`
`573`	`577`
`574`	`578`	`minimalResult := &MinimalSearchCommitsResult{`