Frontmatter
| id | 7593 |
| title | Migrate PullRequestService to GraphQL |
| state | Closed |
| labels | enhancementai |
| assignees | tobiu |
| createdAt | Oct 21, 2025, 1:07 PM |
| updatedAt | Oct 21, 2025, 1:39 PM |
| githubUrl | https://github.com/neomjs/neo/issues/7593 |
| author | tobiu |
| commentsCount | 1 |
| parentIssue | 7590 |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| blockedBy | [] |
| blocking | [] |
| closedAt | Oct 21, 2025, 1:39 PM |
Migrate PullRequestService to GraphQL
tobiu assigned to @tobiu on Oct 21, 2025, 1:07 PM
tobiu added parent issue #7590 on Oct 21, 2025, 1:07 PM

tobiu
Oct 21, 2025, 1:39 PM
✦ Finally, I'll address getPullRequestDiff. The gh pr diff command is convenient because it connects to the remote and generates a standard diff output. Replicating this with GraphQL would involve fetching the base and head commit SHAs for the PR, then potentially using another API or local git commands to generate the diff between them. This is significantly more complex than the other migrations.
Given that complexity, and to keep moving forward, I will leave the getPullRequestDiff method using the gh CLI for now. This is a pragmatic trade-off. The other core data-retrieval methods have been migrated, which was the main goal.
tobiu closed this issue on Oct 21, 2025, 1:39 PM
tobiu cross-referenced by #7590 on Oct 23, 2025, 12:53 AM
This ticket covers the migration of the
PullRequestServiceto use the newGraphqlServicefor all its remote data operations.Acceptance Criteria
PullRequestService.listPullRequestsis refactored to use a GraphQL query.PullRequestService.createCommentis refactored to use theaddCommentGraphQL mutation.PullRequestService.getConversationis refactored to use a GraphQL query.PullRequestService.getPullRequestDiffis investigated and refactored to use a GraphQL query or an alternative API-based method if possible.PullRequestService.checkoutPullRequestremains unchanged, as it performs a local Git operation.