From 85bd25c67e62291bb6908e7e3093b18536feba6e Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 17 Feb 2026 14:20:49 +0000
Subject: [PATCH 1/2] Initial plan


From 4e297156cff46cd84f63aad8403e59a4d65c879f Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 17 Feb 2026 14:30:03 +0000
Subject: [PATCH 2/2] Add GraphQL support for fetching pull requests

- Add PullRequestNode and related GraphQL types
- Implement GetPullRequestsAsync in GitHubGraphQLClient
- Create comprehensive tests for GetPullRequestsAsync
- Add PullRequestInfo internal type to replace Octokit.PullRequest
- Update GitHubRepoConnector to use GraphQL for pull requests
- Replace Octokit PullRequest.GetAllForRepository with GraphQL

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
---
 .../GitHub/GitHubGraphQLClient.cs             |  83 +++
 .../GitHub/GitHubGraphQLTypes.cs              |  63 ++
 .../RepoConnectors/GitHubRepoConnector.cs     |  85 ++-
 ...GitHubGraphQLClientGetPullRequestsTests.cs | 546 ++++++++++++++++++
 4 files changed, 769 insertions(+), 8 deletions(-)
 create mode 100644 test/DemaConsulting.BuildMark.Tests/RepoConnectors/GitHub/GitHubGraphQLClientGetPullRequestsTests.cs

diff --git a/src/DemaConsulting.BuildMark/RepoConnectors/GitHub/GitHubGraphQLClient.cs b/src/DemaConsulting.BuildMark/RepoConnectors/GitHub/GitHubGraphQLClient.cs
index be75d4e..1f45ff1 100644
--- a/src/DemaConsulting.BuildMark/RepoConnectors/GitHub/GitHubGraphQLClient.cs
+++ b/src/DemaConsulting.BuildMark/RepoConnectors/GitHub/GitHubGraphQLClient.cs
@@ -394,6 +394,89 @@ public async Task<List<TagNode>> GetAllTagsAsync(
         }
     }
 
+    /// <summary>
+    ///     Gets all pull requests for a repository using GraphQL with pagination.
+    /// </summary>
+    /// <param name="owner">Repository owner.</param>
+    /// <param name="repo">Repository name.</param>
+    /// <returns>List of pull request nodes.</returns>
+    public async Task<List<PullRequestNode>> GetPullRequestsAsync(
+        string owner,
+        string repo)
+    {
+        try
+        {
+            var allPullRequestNodes = new List<PullRequestNode>();
+            string? afterCursor = null;
+            bool hasNextPage;
+
+            // Paginate through all pull requests
+            do
+            {
+                // Create GraphQL request to get pull requests for a repository with pagination support
+                var request = new GraphQLRequest
+                {
+                    Query = @"
+                        query($owner: String!, $repo: String!, $after: String) {
+                            repository(owner: $owner, name: $repo) {
+                                pullRequests(first: 100, after: $after) {
+                                    nodes {
+                                        number
+                                        title
+                                        url
+                                        merged
+                                        mergeCommit {
+                                            oid
+                                        }
+                                        headRefOid
+                                        labels(first: 100) {
+                                            nodes {
+                                                name
+                                            }
+                                        }
+                                    }
+                                    pageInfo {
+                                        hasNextPage
+                                        endCursor
+                                    }
+                                }
+                            }
+                        }",
+                    Variables = new
+                    {
+                        owner,
+                        repo,
+                        after = afterCursor
+                    }
+                };
+
+                // Execute GraphQL query
+                var response = await _graphqlClient.SendQueryAsync<GetPullRequestsResponse>(request);
+
+                // Extract pull request nodes from the GraphQL response, filtering out null or invalid values
+                var pagePullRequestNodes = response.Data?.Repository?.PullRequests?.Nodes?
+                    .Where(n => n.Number.HasValue && !string.IsNullOrEmpty(n.Title))
+                    .ToList() ?? [];
+
+                allPullRequestNodes.AddRange(pagePullRequestNodes);
+
+                // Check if there are more pages
+                var pageInfo = response.Data?.Repository?.PullRequests?.PageInfo;
+                hasNextPage = pageInfo?.HasNextPage ?? false;
+                afterCursor = pageInfo?.EndCursor;
+            }
+            while (hasNextPage);
+
+            // Return list of all pull request nodes
+            return allPullRequestNodes;
+        }
+        catch
+        {
+            // If GraphQL query fails, return empty list
+            return [];
+        }
+    }
+
     /// <summary>
     ///     Disposes the GraphQL client if owned by this instance.
     /// </summary>
diff --git a/src/DemaConsulting.BuildMark/RepoConnectors/GitHub/GitHubGraphQLTypes.cs b/src/DemaConsulting.BuildMark/RepoConnectors/GitHub/GitHubGraphQLTypes.cs
index e7c5d24..3db903d 100644
--- a/src/DemaConsulting.BuildMark/RepoConnectors/GitHub/GitHubGraphQLTypes.cs
+++ b/src/DemaConsulting.BuildMark/RepoConnectors/GitHub/GitHubGraphQLTypes.cs
@@ -178,3 +178,66 @@ internal record TagNode(
 /// <param name="Oid">Git object ID (SHA) of the commit.</param>
 internal record TagTargetData(
     string? Oid);
+
+/// <summary>
+///     Response for getting pull requests from a repository.
+/// </summary>
+/// <param name="Repository">Repository data containing pull requests information.</param>
+internal record GetPullRequestsResponse(
+    PullRequestRepositoryData? Repository);
+
+/// <summary>
+///     Repository data containing pull requests information.
+/// </summary>
+/// <param name="PullRequests">Pull requests connection data.</param>
+internal record PullRequestRepositoryData(
+    PullRequestsConnectionData? PullRequests);
+
+/// <summary>
+///     Pull requests connection data containing nodes and page info.
+/// </summary>
+/// <param name="Nodes">Pull request nodes.</param>
+/// <param name="PageInfo">Pagination information.</param>
+internal record PullRequestsConnectionData(
+    List<PullRequestNode>? Nodes,
+    PageInfo? PageInfo);
+
+/// <summary>
+///     Pull request node containing pull request information.
+/// </summary>
+/// <param name="Number">Pull request number.</param>
+/// <param name="Title">Pull request title.</param>
+/// <param name="Url">Pull request HTML URL.</param>
+/// <param name="Merged">Whether the pull request is merged.</param>
+/// <param name="MergeCommit">Merge commit information.</param>
+/// <param name="HeadRefOid">Head reference commit SHA.</param>
+/// <param name="Labels">Labels assigned to the pull request.</param>
+internal record PullRequestNode(
+    int? Number,
+    string? Title,
+    string? Url,
+    bool Merged,
+    PullRequestMergeCommit? MergeCommit,
+    string? HeadRefOid,
+    PullRequestLabelsConnection? Labels);
+
+/// <summary>
+///     Pull request merge commit information.
+/// </summary>
+/// <param name="Oid">Commit SHA of the merge commit.</param>
+internal record PullRequestMergeCommit(
+    string? Oid);
+
+/// <summary>
+///     Pull request labels connection containing nodes.
+/// </summary>
+/// <param name="Nodes">Label nodes.</param>
+internal record PullRequestLabelsConnection(
+    List<PullRequestLabel>? Nodes);
+
+/// <summary>
+///     Pull request label information.
+/// </summary>
+/// <param name="Name">Label name.</param>
+internal record PullRequestLabel(
+    string? Name);
diff --git a/src/DemaConsulting.BuildMark/RepoConnectors/GitHubRepoConnector.cs b/src/DemaConsulting.BuildMark/RepoConnectors/GitHubRepoConnector.cs
index 8c481f6..a7eeb80 100644
--- a/src/DemaConsulting.BuildMark/RepoConnectors/GitHubRepoConnector.cs
+++ b/src/DemaConsulting.BuildMark/RepoConnectors/GitHubRepoConnector.cs
@@ -139,6 +139,24 @@ internal sealed record Tag(
     internal sealed record TagCommit(
         string Sha);
 
+    /// <summary>
+    ///     Pull request information.
+    /// </summary>
+    internal sealed record PullRequestInfo(
+        int Number,
+        string Title,
+        string HtmlUrl,
+        bool Merged,
+        string? MergeCommitSha,
+        string? HeadSha,
+        IReadOnlyList<PullRequestLabelInfo> Labels);
+
+    /// <summary>
+    ///     Pull request label information.
+    /// </summary>
+    internal sealed record PullRequestLabelInfo(
+        string Name);
+
     /// <summary>
     ///     Container for GitHub data fetched from the API.
     /// </summary>
@@ -146,7 +164,7 @@ internal sealed record GitHubData(
         IReadOnlyList<Commit> Commits,
         IReadOnlyList<ReleaseNode> Releases,
         IReadOnlyList<Tag> Tags,
-        IReadOnlyList<PullRequest> PullRequests,
+        IReadOnlyList<PullRequestInfo> PullRequests,
         IReadOnlyList<Issue> Issues);
 
     /// <summary>
@@ -154,7 +172,7 @@ internal sealed record GitHubData(
     /// </summary>
     internal sealed record LookupData(
         Dictionary<int, Issue> IssueById,
-        Dictionary<string, PullRequest> CommitHashToPr,
+        Dictionary<string, PullRequestInfo> CommitHashToPr,
         List<ReleaseNode> BranchReleases,
         Dictionary<string, Tag> TagsByName,
         Dictionary<string, ReleaseNode> TagToRelease,
@@ -181,7 +199,7 @@ private static async Task<GitHubData> FetchGitHubDataAsync(
         var commitsTask = GetAllCommitsAsync(graphqlClient, owner, repo, branch);
         var releasesTask = GetAllReleasesAsync(graphqlClient, owner, repo);
         var tagsTask = GetAllTagsAsync(graphqlClient, owner, repo);
-        var pullRequestsTask = client.PullRequest.GetAllForRepository(owner, repo, new PullRequestRequest { State = ItemStateFilter.All });
+        var pullRequestsTask = GetAllPullRequestsAsync(graphqlClient, owner, repo);
         var issuesTask = client.Issue.GetAllForRepository(owner, repo, new RepositoryIssueRequest { State = ItemStateFilter.All });
 
         // Wait for all parallel fetches to complete
@@ -211,8 +229,8 @@ internal static LookupData BuildLookupData(GitHubData data)
         // This is used to associate commits with their pull requests for change tracking.
         // For merged PRs, use MergeCommitSha; for open PRs, use head SHA.
         var commitHashToPr = data.PullRequests
-            .Where(p => (p.Merged && p.MergeCommitSha != null) || (!p.Merged && p.Head?.Sha != null))
-            .ToDictionary(p => p.Merged ? p.MergeCommitSha! : p.Head.Sha, p => p);
+            .Where(p => (p.Merged && p.MergeCommitSha != null) || (!p.Merged && p.HeadSha != null))
+            .ToDictionary(p => p.Merged ? p.MergeCommitSha! : p.HeadSha!, p => p);
 
         // Build a set of commit SHAs in the current branch.
         // This is used for efficient filtering of branch-related tags.
@@ -480,7 +498,7 @@ private static int DetermineSearchStartIndex(int toIndex, int releaseCount)
     private static void ProcessLinkedIssues(
         IReadOnlyList<int> linkedIssueIds,
         Dictionary<int, Issue> issueById,
-        PullRequest pr,
+        PullRequestInfo pr,
         HashSet<string> allChangeIds,
         List<ItemInfo> bugs,
         List<ItemInfo> nonBugChanges)
@@ -523,7 +541,7 @@ private static void ProcessLinkedIssues(
     /// <param name="bugs">List of bug changes (modified in place).</param>
     /// <param name="nonBugChanges">List of non-bug changes (modified in place).</param>
     private static void ProcessPullRequestWithoutIssues(
-        PullRequest pr,
+        PullRequestInfo pr,
         HashSet<string> allChangeIds,
         List<ItemInfo> bugs,
         List<ItemInfo> nonBugChanges)
@@ -623,6 +641,38 @@ private static async Task<IReadOnlyList<Tag>> GetAllTagsAsync(
             .ToList();
     }
 
+    /// <summary>
+    ///     Gets all pull requests for a repository using GraphQL pagination.
+    /// </summary>
+    /// <param name="graphqlClient">GitHub GraphQL client.</param>
+    /// <param name="owner">Repository owner.</param>
+    /// <param name="repo">Repository name.</param>
+    /// <returns>List of all pull requests.</returns>
+    private static async Task<IReadOnlyList<PullRequestInfo>> GetAllPullRequestsAsync(
+        GitHubGraphQLClient graphqlClient,
+        string owner,
+        string repo)
+    {
+        // Fetch all pull requests for the repository using GraphQL
+        var prNodes = await graphqlClient.GetPullRequestsAsync(owner, repo);
+
+        // Convert PullRequestNode objects to PullRequestInfo objects
+        return prNodes
+            .Where(pr => pr.Number.HasValue && !string.IsNullOrEmpty(pr.Title))
+            .Select(pr => new PullRequestInfo(
+                pr.Number!.Value,
+                pr.Title!,
+                pr.Url ?? string.Empty,
+                pr.Merged,
+                pr.MergeCommit?.Oid,
+                pr.HeadRefOid,
+                pr.Labels?.Nodes?
+                    .Where(l => !string.IsNullOrEmpty(l.Name))
+                    .Select(l => new PullRequestLabelInfo(l.Name!))
+                    .ToList() ?? []))
+            .ToList();
+    }
+
     /// <summary>
     ///     Gets commits in the range from fromHash (exclusive) to toHash (inclusive).
     /// </summary>
@@ -687,7 +737,7 @@ internal static ItemInfo CreateItemInfoFromIssue(Issue issue, int index)
     /// </summary>
     /// <param name="pr">GitHub pull request.</param>
     /// <returns>ItemInfo instance.</returns>
-    internal static ItemInfo CreateItemInfoFromPullRequest(PullRequest pr)
+    internal static ItemInfo CreateItemInfoFromPullRequest(PullRequestInfo pr)
     {
         // Determine item type from PR labels
         var type = GetTypeFromLabels(pr.Labels);
@@ -720,6 +770,25 @@ internal static string GetTypeFromLabels(IReadOnlyList<Label> labels)
         return matchingType ?? "other";
     }
 
+    /// <summary>
+    ///     Determines item type from pull request labels.
+    /// </summary>
+    /// <param name="labels">List of pull request labels.</param>
+    /// <returns>Item type string.</returns>
+    internal static string GetTypeFromLabels(IReadOnlyList<PullRequestLabelInfo> labels)
+    {
+        // Find first matching label type by checking label names against the type map
+        var matchingType = labels
+            .Select(label => label.Name.ToLowerInvariant())
+            .SelectMany(lowerLabel => LabelTypeMap
+                .Where(kvp => lowerLabel.Contains(kvp.Key))
+                .Select(kvp => kvp.Value))
+            .FirstOrDefault();
+
+        // Return matched type or default to "other"
+        return matchingType ?? "other";
+    }
+
     /// <summary>
     ///     Gets GitHub token from environment or gh CLI.
     /// </summary>
diff --git a/test/DemaConsulting.BuildMark.Tests/RepoConnectors/GitHub/GitHubGraphQLClientGetPullRequestsTests.cs b/test/DemaConsulting.BuildMark.Tests/RepoConnectors/GitHub/GitHubGraphQLClientGetPullRequestsTests.cs
new file mode 100644
index 0000000..6fdeb29
--- /dev/null
+++ b/test/DemaConsulting.BuildMark.Tests/RepoConnectors/GitHub/GitHubGraphQLClientGetPullRequestsTests.cs
@@ -0,0 +1,546 @@
+// Copyright (c) 2026 DEMA Consulting
+// 
+// Permission is hereby granted, free of charge, to any person obtaining a copy
+// of this software and associated documentation files (the "Software"), to deal
+// in the Software without restriction, including without limitation the rights
+// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+// copies of the Software, and to permit persons to whom the Software is
+// furnished to do so, subject to the following conditions:
+// 
+// The above copyright notice and this permission notice shall be included in all
+// copies or substantial portions of the Software.
+// 
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+// SOFTWARE.
+
+using System.Net;
+using System.Text;
+using DemaConsulting.BuildMark.RepoConnectors.GitHub;
+
+namespace DemaConsulting.BuildMark.Tests;
+
+/// <summary>
+///     Tests for the GitHubGraphQLClient GetPullRequestsAsync method.
+/// </summary>
+[TestClass]
+public class GitHubGraphQLClientGetPullRequestsTests
+{
+    /// <summary>
+    ///     Test that GetPullRequestsAsync returns expected pull requests with valid response.
+    /// </summary>
+    [TestMethod]
+    public async Task GitHubGraphQLClient_GetPullRequestsAsync_ValidResponse_ReturnsPullRequests()
+    {
+        // Arrange
+        var mockResponse = @"{
+            ""data"": {
+                ""repository"": {
+                    ""pullRequests"": {
+                        ""nodes"": [
+                            {
+                                ""number"": 1,
+                                ""title"": ""Add feature A"",
+                                ""url"": ""https://github.com/owner/repo/pull/1"",
+                                ""merged"": true,
+                                ""mergeCommit"": {
+                                    ""oid"": ""abc123""
+                                },
+                                ""headRefOid"": ""def456"",
+                                ""labels"": {
+                                    ""nodes"": [
+                                        { ""name"": ""feature"" }
+                                    ]
+                                }
+                            },
+                            {
+                                ""number"": 2,
+                                ""title"": ""Fix bug B"",
+                                ""url"": ""https://github.com/owner/repo/pull/2"",
+                                ""merged"": false,
+                                ""mergeCommit"": null,
+                                ""headRefOid"": ""ghi789"",
+                                ""labels"": {
+                                    ""nodes"": [
+                                        { ""name"": ""bug"" }
+                                    ]
+                                }
+                            }
+                        ],
+                        ""pageInfo"": {
+                            ""hasNextPage"": false,
+                            ""endCursor"": null
+                        }
+                    }
+                }
+            }
+        }";
+
+        using var httpClient = CreateMockHttpClient(mockResponse, HttpStatusCode.OK);
+        using var client = new GitHubGraphQLClient(httpClient);
+
+        // Act
+        var pullRequests = await client.GetPullRequestsAsync("owner", "repo");
+
+        // Assert
+        Assert.IsNotNull(pullRequests);
+        Assert.HasCount(2, pullRequests);
+        
+        Assert.AreEqual(1, pullRequests[0].Number);
+        Assert.AreEqual("Add feature A", pullRequests[0].Title);
+        Assert.AreEqual("https://github.com/owner/repo/pull/1", pullRequests[0].Url);
+        Assert.IsTrue(pullRequests[0].Merged);
+        Assert.AreEqual("abc123", pullRequests[0].MergeCommit?.Oid);
+        Assert.AreEqual("def456", pullRequests[0].HeadRefOid);
+        Assert.IsNotNull(pullRequests[0].Labels?.Nodes);
+        Assert.HasCount(1, pullRequests[0].Labels!.Nodes!);
+        Assert.AreEqual("feature", pullRequests[0].Labels!.Nodes![0].Name);
+
+        Assert.AreEqual(2, pullRequests[1].Number);
+        Assert.AreEqual("Fix bug B", pullRequests[1].Title);
+        Assert.IsFalse(pullRequests[1].Merged);
+        Assert.IsNull(pullRequests[1].MergeCommit);
+    }
+
+    /// <summary>
+    ///     Test that GetPullRequestsAsync returns empty list when no pull requests are found.
+    /// </summary>
+    [TestMethod]
+    public async Task GitHubGraphQLClient_GetPullRequestsAsync_NoPullRequests_ReturnsEmptyList()
+    {
+        // Arrange
+        var mockResponse = @"{
+            ""data"": {
+                ""repository"": {
+                    ""pullRequests"": {
+                        ""nodes"": [],
+                        ""pageInfo"": {
+                            ""hasNextPage"": false,
+                            ""endCursor"": null
+                        }
+                    }
+                }
+            }
+        }";
+
+        using var httpClient = CreateMockHttpClient(mockResponse, HttpStatusCode.OK);
+        using var client = new GitHubGraphQLClient(httpClient);
+
+        // Act
+        var pullRequests = await client.GetPullRequestsAsync("owner", "repo");
+
+        // Assert
+        Assert.IsNotNull(pullRequests);
+        Assert.IsEmpty(pullRequests);
+    }
+
+    /// <summary>
+    ///     Test that GetPullRequestsAsync returns empty list when response has missing data.
+    /// </summary>
+    [TestMethod]
+    public async Task GitHubGraphQLClient_GetPullRequestsAsync_MissingData_ReturnsEmptyList()
+    {
+        // Arrange
+        var mockResponse = @"{
+            ""data"": {
+                ""repository"": null
+            }
+        }";
+
+        using var httpClient = CreateMockHttpClient(mockResponse, HttpStatusCode.OK);
+        using var client = new GitHubGraphQLClient(httpClient);
+
+        // Act
+        var pullRequests = await client.GetPullRequestsAsync("owner", "repo");
+
+        // Assert
+        Assert.IsNotNull(pullRequests);
+        Assert.IsEmpty(pullRequests);
+    }
+
+    /// <summary>
+    ///     Test that GetPullRequestsAsync returns empty list on HTTP error.
+    /// </summary>
+    [TestMethod]
+    public async Task GitHubGraphQLClient_GetPullRequestsAsync_HttpError_ReturnsEmptyList()
+    {
+        // Arrange
+        var mockResponse = @"{ ""message"": ""Not Found"" }";
+
+        using var httpClient = CreateMockHttpClient(mockResponse, HttpStatusCode.NotFound);
+        using var client = new GitHubGraphQLClient(httpClient);
+
+        // Act
+        var pullRequests = await client.GetPullRequestsAsync("owner", "repo");
+
+        // Assert
+        Assert.IsNotNull(pullRequests);
+        Assert.IsEmpty(pullRequests);
+    }
+
+    /// <summary>
+    ///     Test that GetPullRequestsAsync returns empty list on invalid JSON.
+    /// </summary>
+    [TestMethod]
+    public async Task GitHubGraphQLClient_GetPullRequestsAsync_InvalidJson_ReturnsEmptyList()
+    {
+        // Arrange
+        var mockResponse = "This is not valid JSON";
+
+        using var httpClient = CreateMockHttpClient(mockResponse, HttpStatusCode.OK);
+        using var client = new GitHubGraphQLClient(httpClient);
+
+        // Act
+        var pullRequests = await client.GetPullRequestsAsync("owner", "repo");
+
+        // Assert
+        Assert.IsNotNull(pullRequests);
+        Assert.IsEmpty(pullRequests);
+    }
+
+    /// <summary>
+    ///     Test that GetPullRequestsAsync returns single pull request correctly.
+    /// </summary>
+    [TestMethod]
+    public async Task GitHubGraphQLClient_GetPullRequestsAsync_SinglePullRequest_ReturnsOnePullRequest()
+    {
+        // Arrange
+        var mockResponse = @"{
+            ""data"": {
+                ""repository"": {
+                    ""pullRequests"": {
+                        ""nodes"": [
+                            {
+                                ""number"": 42,
+                                ""title"": ""Important PR"",
+                                ""url"": ""https://github.com/owner/repo/pull/42"",
+                                ""merged"": true,
+                                ""mergeCommit"": {
+                                    ""oid"": ""merge123""
+                                },
+                                ""headRefOid"": ""head456"",
+                                ""labels"": {
+                                    ""nodes"": []
+                                }
+                            }
+                        ],
+                        ""pageInfo"": {
+                            ""hasNextPage"": false,
+                            ""endCursor"": null
+                        }
+                    }
+                }
+            }
+        }";
+
+        using var httpClient = CreateMockHttpClient(mockResponse, HttpStatusCode.OK);
+        using var client = new GitHubGraphQLClient(httpClient);
+
+        // Act
+        var pullRequests = await client.GetPullRequestsAsync("owner", "repo");
+
+        // Assert
+        Assert.IsNotNull(pullRequests);
+        Assert.HasCount(1, pullRequests);
+        Assert.AreEqual(42, pullRequests[0].Number);
+        Assert.AreEqual("Important PR", pullRequests[0].Title);
+    }
+
+    /// <summary>
+    ///     Test that GetPullRequestsAsync handles nodes with missing number or title property.
+    /// </summary>
+    [TestMethod]
+    public async Task GitHubGraphQLClient_GetPullRequestsAsync_MissingNumberOrTitle_SkipsInvalidNodes()
+    {
+        // Arrange
+        var mockResponse = @"{
+            ""data"": {
+                ""repository"": {
+                    ""pullRequests"": {
+                        ""nodes"": [
+                            {
+                                ""number"": 1,
+                                ""title"": ""Valid PR"",
+                                ""url"": ""https://github.com/owner/repo/pull/1"",
+                                ""merged"": false,
+                                ""mergeCommit"": null,
+                                ""headRefOid"": ""abc123"",
+                                ""labels"": {
+                                    ""nodes"": []
+                                }
+                            },
+                            {
+                                ""title"": ""Missing number"",
+                                ""url"": ""https://github.com/owner/repo/pull/2"",
+                                ""merged"": false,
+                                ""mergeCommit"": null,
+                                ""headRefOid"": ""def456"",
+                                ""labels"": {
+                                    ""nodes"": []
+                                }
+                            },
+                            {
+                                ""number"": 3,
+                                ""url"": ""https://github.com/owner/repo/pull/3"",
+                                ""merged"": false,
+                                ""mergeCommit"": null,
+                                ""headRefOid"": ""ghi789"",
+                                ""labels"": {
+                                    ""nodes"": []
+                                }
+                            },
+                            {
+                                ""number"": 4,
+                                ""title"": ""Another valid PR"",
+                                ""url"": ""https://github.com/owner/repo/pull/4"",
+                                ""merged"": false,
+                                ""mergeCommit"": null,
+                                ""headRefOid"": ""jkl012"",
+                                ""labels"": {
+                                    ""nodes"": []
+                                }
+                            }
+                        ],
+                        ""pageInfo"": {
+                            ""hasNextPage"": false,
+                            ""endCursor"": null
+                        }
+                    }
+                }
+            }
+        }";
+
+        using var httpClient = CreateMockHttpClient(mockResponse, HttpStatusCode.OK);
+        using var client = new GitHubGraphQLClient(httpClient);
+
+        // Act
+        var pullRequests = await client.GetPullRequestsAsync("owner", "repo");
+
+        // Assert
+        Assert.IsNotNull(pullRequests);
+        Assert.HasCount(2, pullRequests);
+        Assert.AreEqual(1, pullRequests[0].Number);
+        Assert.AreEqual("Valid PR", pullRequests[0].Title);
+        Assert.AreEqual(4, pullRequests[1].Number);
+        Assert.AreEqual("Another valid PR", pullRequests[1].Title);
+    }
+
+    /// <summary>
+    ///     Test that GetPullRequestsAsync handles pagination correctly.
+    /// </summary>
+    [TestMethod]
+    public async Task GitHubGraphQLClient_GetPullRequestsAsync_WithPagination_ReturnsAllPullRequests()
+    {
+        // Arrange - Create mock handler that returns different responses for different pages
+        var mockHandler = new PullRequestPaginationMockHttpMessageHandler();
+        using var httpClient = new HttpClient(mockHandler);
+        using var client = new GitHubGraphQLClient(httpClient);
+
+        // Act
+        var pullRequests = await client.GetPullRequestsAsync("owner", "repo");
+
+        // Assert
+        Assert.IsNotNull(pullRequests);
+        Assert.HasCount(3, pullRequests);
+        Assert.AreEqual(1, pullRequests[0].Number);
+        Assert.AreEqual("PR from page 1", pullRequests[0].Title);
+        Assert.AreEqual(2, pullRequests[1].Number);
+        Assert.AreEqual("PR from page 2", pullRequests[1].Title);
+        Assert.AreEqual(3, pullRequests[2].Number);
+        Assert.AreEqual("PR from page 3", pullRequests[2].Title);
+    }
+
+    /// <summary>
+    ///     Mock HTTP message handler for testing pull request pagination.
+    /// </summary>
+    private sealed class PullRequestPaginationMockHttpMessageHandler : HttpMessageHandler
+    {
+        /// <summary>
+        ///     Request count to track pagination.
+        /// </summary>
+        private int _requestCount;
+
+        /// <summary>
+        ///     Sends a mock HTTP response with pagination.
+        /// </summary>
+        /// <param name="request">HTTP request message.</param>
+        /// <param name="cancellationToken">Cancellation token.</param>
+        /// <returns>Mock HTTP response.</returns>
+        protected override async Task<HttpResponseMessage> SendAsync(
+            HttpRequestMessage request,
+            CancellationToken cancellationToken)
+        {
+            // Read request body to determine which page to return
+            var requestBody = request.Content != null
+                ? await request.Content.ReadAsStringAsync(cancellationToken)
+                : string.Empty;
+
+            string responseContent;
+            if (_requestCount == 0 || !requestBody.Contains("\"after\""))
+            {
+                // First page
+                responseContent = @"{
+                    ""data"": {
+                        ""repository"": {
+                            ""pullRequests"": {
+                                ""nodes"": [
+                                    {
+                                        ""number"": 1,
+                                        ""title"": ""PR from page 1"",
+                                        ""url"": ""https://github.com/owner/repo/pull/1"",
+                                        ""merged"": false,
+                                        ""mergeCommit"": null,
+                                        ""headRefOid"": ""page1"",
+                                        ""labels"": {
+                                            ""nodes"": []
+                                        }
+                                    }
+                                ],
+                                ""pageInfo"": {
+                                    ""hasNextPage"": true,
+                                    ""endCursor"": ""cursor1""
+                                }
+                            }
+                        }
+                    }
+                }";
+            }
+            else if (requestBody.Contains("\"cursor1\""))
+            {
+                // Second page
+                responseContent = @"{
+                    ""data"": {
+                        ""repository"": {
+                            ""pullRequests"": {
+                                ""nodes"": [
+                                    {
+                                        ""number"": 2,
+                                        ""title"": ""PR from page 2"",
+                                        ""url"": ""https://github.com/owner/repo/pull/2"",
+                                        ""merged"": false,
+                                        ""mergeCommit"": null,
+                                        ""headRefOid"": ""page2"",
+                                        ""labels"": {
+                                            ""nodes"": []
+                                        }
+                                    }
+                                ],
+                                ""pageInfo"": {
+                                    ""hasNextPage"": true,
+                                    ""endCursor"": ""cursor2""
+                                }
+                            }
+                        }
+                    }
+                }";
+            }
+            else
+            {
+                // Third (last) page
+                responseContent = @"{
+                    ""data"": {
+                        ""repository"": {
+                            ""pullRequests"": {
+                                ""nodes"": [
+                                    {
+                                        ""number"": 3,
+                                        ""title"": ""PR from page 3"",
+                                        ""url"": ""https://github.com/owner/repo/pull/3"",
+                                        ""merged"": false,
+                                        ""mergeCommit"": null,
+                                        ""headRefOid"": ""page3"",
+                                        ""labels"": {
+                                            ""nodes"": []
+                                        }
+                                    }
+                                ],
+                                ""pageInfo"": {
+                                    ""hasNextPage"": false,
+                                    ""endCursor"": null
+                                }
+                            }
+                        }
+                    }
+                }";
+            }
+
+            _requestCount++;
+
+            // Create response with content
+            // Note: The returned HttpResponseMessage will be disposed by HttpClient,
+            // which also disposes the Content. This is the expected pattern for HttpMessageHandler.
+            var content = new StringContent(responseContent, Encoding.UTF8, "application/json");
+            var response = new HttpResponseMessage(HttpStatusCode.OK)
+            {
+                Content = content
+            };
+
+            return response;
+        }
+    }
+
+    /// <summary>
+    ///     Creates a mock HttpClient with pre-canned response.
+    /// </summary>
+    /// <param name="responseContent">Response content to return.</param>
+    /// <param name="statusCode">HTTP status code to return.</param>
+    /// <returns>HttpClient configured with mock handler.</returns>
+    private static HttpClient CreateMockHttpClient(string responseContent, HttpStatusCode statusCode)
+    {
+        var handler = new MockHttpMessageHandler(responseContent, statusCode);
+        return new HttpClient(handler);
+    }
+
+    /// <summary>
+    ///     Mock HTTP message handler for testing.
+    /// </summary>
+    private sealed class MockHttpMessageHandler : HttpMessageHandler
+    {
+        /// <summary>
+        ///     Response content to return.
+        /// </summary>
+        private readonly string _responseContent;
+
+        /// <summary>
+        ///     HTTP status code to return.
+        /// </summary>
+        private readonly HttpStatusCode _statusCode;
+
+        /// <summary>
+        ///     Initializes a new instance of the <see cref="MockHttpMessageHandler"/> class.
+        /// </summary>
+        /// <param name="responseContent">Response content to return.</param>
+        /// <param name="statusCode">HTTP status code to return.</param>
+        public MockHttpMessageHandler(string responseContent, HttpStatusCode statusCode)
+        {
+            _responseContent = responseContent;
+            _statusCode = statusCode;
+        }
+
+        /// <summary>
+        ///     Sends a mock HTTP response.
+        /// </summary>
+        /// <param name="request">HTTP request message.</param>
+        /// <param name="cancellationToken">Cancellation token.</param>
+        /// <returns>Mock HTTP response.</returns>
+        protected override Task<HttpResponseMessage> SendAsync(
+            HttpRequestMessage request,
+            CancellationToken cancellationToken)
+        {
+            // Create response with content
+            // Note: The returned HttpResponseMessage will be disposed by HttpClient,
+            // which also disposes the Content. This is the expected pattern for HttpMessageHandler.
+            var content = new StringContent(_responseContent, Encoding.UTF8, "application/json");
+            var response = new HttpResponseMessage(_statusCode)
+            {
+                Content = content
+            };
+
+            return Task.FromResult(response);
+        }
+    }
+}