Use helper functions from tools

Author: lumaxis

e2e/e2e_test.go

@@ -1807,41 +1807,41 @@ func TestFindClosingPullRequestsGraphQLParameters(t *testing.T) {
 				owner:            "microsoft",
 				repo:             "vscode",
 				issueNumbers:     []int{1},
-				includeClosedPrs: boolPtr(true),
+				includeClosedPrs: github.ToBoolPtr(true),
 				description:      "Test includeClosedPrs parameter with popular repository",
 			},
 			{
 				name:             "includeClosedPrs=false - should exclude closed/merged PRs",
 				owner:            "microsoft",
 				repo:             "vscode",
 				issueNumbers:     []int{2},
-				includeClosedPrs: boolPtr(false),
+				includeClosedPrs: github.ToBoolPtr(false),
 				description:      "Test includeClosedPrs=false parameter",
 			},
 			{
 				name:         "orderByState=true - should order results by PR state",
 				owner:        "microsoft",
 				repo:         "vscode",
 				issueNumbers: []int{1, 2}, // Use low numbers for older issues
-				orderByState: boolPtr(true),
+				orderByState: github.ToBoolPtr(true),
 				description:  "Test orderByState parameter with larger repository",
 			},
 			{
 				name:           "userLinkedOnly=true - should return only manually linked PRs",
 				owner:          "facebook",
 				repo:           "react",
 				issueNumbers:   []int{1}, // First issue in React repo
-				userLinkedOnly: boolPtr(true),
+				userLinkedOnly: github.ToBoolPtr(true),
 				description:    "Test userLinkedOnly parameter",
 			},
 			{
 				name:             "Combined boolean parameters",
 				owner:            "facebook",
 				repo:             "react",
 				issueNumbers:     []int{1, 2},
-				includeClosedPrs: boolPtr(true),
-				orderByState:     boolPtr(true),
-				userLinkedOnly:   boolPtr(false),
+				includeClosedPrs: github.ToBoolPtr(true),
+				orderByState:     github.ToBoolPtr(true),
+				userLinkedOnly:   github.ToBoolPtr(false),
 				description:      "Test multiple boolean parameters together",
 			},
 		}
@@ -1975,23 +1975,23 @@ func TestFindClosingPullRequestsGraphQLParameters(t *testing.T) {
 				owner:        "microsoft",
 				repo:         "vscode",
 				issueNumbers: []int{1, 2},
-				first:        intPtr(1),
+				first:        github.ToIntPtr(1),
 				description:  "Test forward pagination using first parameter",
 			},
 			{
 				name:         "Backward pagination with last parameter",
 				owner:        "microsoft",
 				repo:         "vscode",
 				issueNumbers: []int{1, 2},
-				last:         intPtr(1),
+				last:         github.ToIntPtr(1),
 				description:  "Test backward pagination using last parameter",
 			},
 			{
 				name:         "Large repository pagination test",
 				owner:        "microsoft",
 				repo:         "vscode",
 				issueNumbers: []int{1, 2, 3},
-				first:        intPtr(1), // Small page size
+				first:        github.ToIntPtr(1), // Small page size
 				description:  "Test pagination with larger repository",
 			},
 		}
@@ -2235,16 +2235,3 @@ func TestFindClosingPullRequestsGraphQLParameters(t *testing.T) {
 		}
 	})
 }
-
-// Helper functions for pointer creation
-func boolPtr(b bool) *bool {
-	return &b
-}
-
-func intPtr(i int) *int {
-	return &i
-}
-
-func stringPtr(s string) *string {
-	return &s
-}

pkg/github/tools.go

@@ -213,6 +213,11 @@ func ToBoolPtr(b bool) *bool {
 	return &b
 }
 
+// ToIntPtr converts an int to a *int pointer.
+func ToIntPtr(i int) *int {
+	return &i
+}
+
 // ToStringPtr converts a string to a *string pointer.
 // Returns nil if the string is empty.
 func ToStringPtr(s string) *string {