@kitlangton/ghui 0.1.4 → 0.1.6

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@kitlangton/ghui",
-	"version": "0.1.4",
+	"version": "0.1.6",
 	"description": "Terminal UI for GitHub pull requests",
 	"type": "module",
 	"license": "MIT",

package/src/App.tsx

@@ -435,23 +435,104 @@ const pullRequestDiffKey = (pullRequest: PullRequestItem) => `${pullRequest.repo
 
 const diffStatText = (pullRequest: PullRequestItem) => {
 	const files = pullRequest.changedFiles === 1 ? "1 file" : `${pullRequest.changedFiles} files`
-	return `+${pullRequest.additions} -${pullRequest.deletions}  ${files}`
+	return [
+		pullRequest.additions > 0 ? `+${pullRequest.additions}` : null,
+		pullRequest.deletions > 0 ? `-${pullRequest.deletions}` : null,
+		files,
+	].filter((part): part is string => part !== null).join(" ")
 }
 
-const patchRenderableLineCount = (patch: string, view: "unified" | "split") => {
+const DiffStats = ({ pullRequest }: { pullRequest: PullRequestItem }) => {
+	const files = pullRequest.changedFiles === 1 ? "1 file" : `${pullRequest.changedFiles} files`
+	type Part = { key: string; text: string; color: string }
+	const rawParts: Array<Part | null> = [
+		pullRequest.additions > 0 ? { key: "additions", text: `+${pullRequest.additions}`, color: colors.status.passing } : null,
+		pullRequest.deletions > 0 ? { key: "deletions", text: `-${pullRequest.deletions}`, color: colors.status.failing } : null,
+		{ key: "files", text: files, color: colors.muted },
+	]
+	const parts = rawParts.filter((part): part is Part => part !== null)
+
+	return (
+		<>
+			{parts.map((part, index) => (
+				<Fragment key={part.key}>
+					{index > 0 ? <span fg={colors.muted}> </span> : null}
+					<span fg={part.color}>{part.text}</span>
+				</Fragment>
+			))}
+		</>
+	)
+}
+
+const estimatedWrappedLineCount = (text: string, width: number, wrapMode: "none" | "word") => {
+	if (wrapMode === "none") return 1
+	return Math.max(1, Math.ceil(Bun.stringWidth(text) / Math.max(1, width)))
+}
+
+const patchLineNumberGutterWidth = (lines: readonly string[]) => {
+	let maxLineNumber = 1
+	let hasSigns = false
+	let oldLine = 0
+	let newLine = 0
+
+	for (const line of lines) {
+		const hunk = line.match(/^@@ -(\d+)(?:,\d+)? \+(\d+)(?:,\d+)? @@/)
+		if (hunk) {
+			oldLine = Number(hunk[1])
+			newLine = Number(hunk[2])
+			maxLineNumber = Math.max(maxLineNumber, oldLine, newLine)
+			continue
+		}
+
+		const firstChar = line[0]
+		if (firstChar === "-") {
+			hasSigns = true
+			maxLineNumber = Math.max(maxLineNumber, oldLine)
+			oldLine++
+		} else if (firstChar === "+") {
+			hasSigns = true
+			maxLineNumber = Math.max(maxLineNumber, newLine)
+			newLine++
+		} else if (firstChar === " ") {
+			maxLineNumber = Math.max(maxLineNumber, oldLine, newLine)
+			oldLine++
+			newLine++
+		}
+	}
+
+	const digits = Math.floor(Math.log10(maxLineNumber)) + 1
+	return Math.max(3, digits + 2) + (hasSigns ? 2 : 0)
+}
+
+const patchRenderableLineCount = (patch: string, view: "unified" | "split", wrapMode: "none" | "word", width: number) => {
+	const lines = patch.split("\n")
+	const lineNumberGutterWidth = patchLineNumberGutterWidth(lines)
+	const splitPaneWidth = Math.max(1, Math.floor(width / 2) - lineNumberGutterWidth)
+	const unifiedPaneWidth = Math.max(1, width - lineNumberGutterWidth)
+	const contentWidth = view === "split" ? splitPaneWidth : unifiedPaneWidth
 	let count = 0
 	let inHunk = false
-	let deletions = 0
-	let additions = 0
+	let deletions: number[] = []
+	let additions: number[] = []
 
 	const flushChangeBlock = () => {
-		if (deletions === 0 && additions === 0) return
-		count += view === "split" ? Math.max(deletions, additions) : deletions + additions
-		deletions = 0
-		additions = 0
+		if (deletions.length === 0 && additions.length === 0) return
+		if (view === "split") {
+			const rows = Math.max(deletions.length, additions.length)
+			for (let index = 0; index < rows; index++) {
+				const deletionCount = index < deletions.length ? deletions[index]! : 1
+				const additionCount = index < additions.length ? additions[index]! : 1
+				count += Math.max(deletionCount, additionCount)
+			}
+		} else {
+			for (const deletion of deletions) count += deletion
+			for (const addition of additions) count += addition
+		}
+		deletions = []
+		additions = []
 	}
 
-	for (const line of patch.split("\n")) {
+	for (const line of lines) {
 		if (line.startsWith("@@")) {
 			flushChangeBlock()
 			inHunk = true
@@ -464,18 +545,18 @@ const patchRenderableLineCount = (patch: string, view: "unified" | "split") => {
 		if (firstChar === "\\") continue
 
 		if (firstChar === "-") {
-			deletions++
+			deletions.push(estimatedWrappedLineCount(line.slice(1), contentWidth, wrapMode))
 			continue
 		}
 
 		if (firstChar === "+") {
-			additions++
+			additions.push(estimatedWrappedLineCount(line.slice(1), contentWidth, wrapMode))
 			continue
 		}
 
 		if (firstChar === " ") {
 			flushChangeBlock()
-			count++
+			count += estimatedWrappedLineCount(line.slice(1), contentWidth, wrapMode)
 		}
 	}
 
@@ -709,10 +790,6 @@ const FooterHints = ({
 			<TextLine>
 				<span fg={colors.count}>esc</span>
 				<span fg={colors.muted}> back  </span>
-				<span fg={colors.count}>j/k</span>
-				<span fg={colors.muted}> scroll  </span>
-				<span fg={colors.count}>gg/G</span>
-				<span fg={colors.muted}> top/bot  </span>
 				<span fg={colors.count}>v</span>
 				<span fg={colors.muted}> view  </span>
 				<span fg={colors.count}>w</span>
@@ -734,12 +811,6 @@ const FooterHints = ({
 			<TextLine>
 				<span fg={colors.count}>esc</span>
 				<span fg={colors.muted}> back  </span>
-				<span fg={colors.count}>j/k</span>
-				<span fg={colors.muted}> scroll  </span>
-				<span fg={colors.count}>gg/G</span>
-				<span fg={colors.muted}> top/bot  </span>
-				<span fg={colors.count}>ctrl-d/u</span>
-				<span fg={colors.muted}> page  </span>
 				<span fg={colors.count}>o</span>
 				<span fg={colors.muted}> open  </span>
 				<span fg={colors.count}>y</span>
@@ -1046,7 +1117,7 @@ const DetailHeader = ({
 				const review = reviewLabel(pullRequest)
 				const checks = pullRequest.checkSummary?.replace(/^checks\s+/, "")
 				const statusParts = [review, checks].filter((part): part is string => Boolean(part))
-				const rightSide = statusParts.length > 0 ? `${statusParts.join(" ")}  ${opened}` : opened
+				const rightSide = statusParts.length > 0 ? `${statusParts.join(" ")} ${opened}` : opened
 				const leftWidth = 1 + number.length + 1 + repo.length
 				const gap = Math.max(2, contentWidth - leftWidth - rightSide.length)
 
@@ -1058,7 +1129,7 @@ const DetailHeader = ({
 						{review ? <span fg={statusColor(pullRequest.reviewStatus)}>{review}</span> : null}
 						{review && checks ? <span fg={colors.muted}> </span> : null}
 						{checks ? <span fg={statusColor(pullRequest.checkStatus)}>{checks}</span> : null}
-						{statusParts.length > 0 ? <span fg={colors.muted}>  </span> : null}
+						{statusParts.length > 0 ? <span fg={colors.muted}> </span> : null}
 						<span fg={colors.muted}>{opened}</span>
 					</TextLine>
 				)
@@ -1080,10 +1151,7 @@ const DetailHeader = ({
 					{showStats ? (
 						<>
 							<span fg={colors.muted}>{" ".repeat(statsGap)}</span>
-							<span fg={colors.status.passing}>+{pullRequest.additions}</span>
-							<span fg={colors.muted}> </span>
-							<span fg={colors.status.failing}>-{pullRequest.deletions}</span>
-							<span fg={colors.muted}>  {pullRequest.changedFiles === 1 ? "1 file" : `${pullRequest.changedFiles} files`}</span>
+							<DiffStats pullRequest={pullRequest} />
 						</>
 					) : null}
 				</TextLine>
@@ -1251,6 +1319,14 @@ const PullRequestDiffPane = ({
 	loadingIndicator: string
 	scrollRef: React.Ref<ScrollBoxRenderable>
 }) => {
+	const readyFiles = diffState?.status === "ready" ? diffState.files : []
+	const safeIndex = readyFiles.length > 0 ? Math.max(0, Math.min(fileIndex, readyFiles.length - 1)) : 0
+	const file = readyFiles[safeIndex] ?? null
+	const diffHeight = useMemo(
+		() => file ? patchRenderableLineCount(file.patch, view, wrapMode, paneWidth) : 1,
+		[file?.patch, view, wrapMode, paneWidth],
+	)
+
 	if (!pullRequest) {
 		return <LoadingPane content={{ title: "No pull request selected", hint: "Press esc to go back" }} width={paneWidth} height={height} />
 	}
@@ -1268,10 +1344,7 @@ const PullRequestDiffPane = ({
 						<span fg={colors.count}>#{pullRequest.number}</span>
 						<span fg={colors.muted}> {shortRepoName(pullRequest.repository)}</span>
 						<span fg={colors.muted}>{" ".repeat(headerGap)}</span>
-						<span fg={colors.status.passing}>+{pullRequest.additions}</span>
-						<span fg={colors.muted}> </span>
-						<span fg={colors.status.failing}>-{pullRequest.deletions}</span>
-						<span fg={colors.muted}>  {pullRequest.changedFiles === 1 ? "1 file" : `${pullRequest.changedFiles} files`}</span>
+						<DiffStats pullRequest={pullRequest} />
 					</TextLine>
 				</box>
 				<Divider width={paneWidth} />
@@ -1292,15 +1365,12 @@ const PullRequestDiffPane = ({
 		)
 	}
 
-	if (diffState.files.length === 0) {
+	if (readyFiles.length === 0 || !file) {
 		return <LoadingPane content={{ title: "No diff", hint: "This PR has no patch contents" }} width={paneWidth} height={height} />
 	}
 
-	const safeIndex = Math.max(0, Math.min(fileIndex, diffState.files.length - 1))
-	const file = diffState.files[safeIndex]!
-	const fileCounter = `${safeIndex + 1}/${diffState.files.length}`
+	const fileCounter = `${safeIndex + 1}/${readyFiles.length}`
 	const fileNameWidth = Math.max(8, headerWidth - fileCounter.length - 2)
-	const diffHeight = patchRenderableLineCount(file.patch, view)
 
 	return (
 		<box height={height} flexDirection="column">
@@ -1309,10 +1379,7 @@ const PullRequestDiffPane = ({
 					<span fg={colors.count}>#{pullRequest.number}</span>
 					<span fg={colors.muted}> {shortRepoName(pullRequest.repository)}</span>
 					<span fg={colors.muted}>{" ".repeat(headerGap)}</span>
-					<span fg={colors.status.passing}>+{pullRequest.additions}</span>
-					<span fg={colors.muted}> </span>
-					<span fg={colors.status.failing}>-{pullRequest.deletions}</span>
-					<span fg={colors.muted}>  {pullRequest.changedFiles === 1 ? "1 file" : `${pullRequest.changedFiles} files`}</span>
+					<DiffStats pullRequest={pullRequest} />
 				</TextLine>
 			</box>
 			<box height={1} paddingLeft={1} paddingRight={1}>
@@ -1502,6 +1569,7 @@ export const App = () => {
 	const wideBodyHeight = Math.max(8, (height ?? 24) - 4)
 	const noticeTimeoutRef = useRef<ReturnType<typeof setTimeout> | null>(null)
 	const pendingGTimeoutRef = useRef<ReturnType<typeof setTimeout> | null>(null)
+	const diffPrefetchTimeoutRef = useRef<ReturnType<typeof setTimeout> | null>(null)
 	const detailScrollRef = useRef<ScrollBoxRenderable | null>(null)
 	const diffScrollRef = useRef<ScrollBoxRenderable | null>(null)
 	const headerFooterWidth = Math.max(24, contentWidth - 2)
@@ -1523,6 +1591,9 @@ export const App = () => {
 		if (pendingGTimeoutRef.current !== null) {
 			clearTimeout(pendingGTimeoutRef.current)
 		}
+		if (diffPrefetchTimeoutRef.current !== null) {
+			clearTimeout(diffPrefetchTimeoutRef.current)
+		}
 	}, [])
 
 	const pullRequestLoad = AsyncResult.getOrElse(pullRequestResult, () => null)
@@ -1653,6 +1724,22 @@ export const App = () => {
 			})
 	}
 
+	useEffect(() => {
+		if (!selectedPullRequest || diffFullView) return
+		if (diffPrefetchTimeoutRef.current !== null) {
+			clearTimeout(diffPrefetchTimeoutRef.current)
+		}
+		diffPrefetchTimeoutRef.current = setTimeout(() => {
+			loadPullRequestDiff(selectedPullRequest)
+		}, 250)
+		return () => {
+			if (diffPrefetchTimeoutRef.current !== null) {
+				clearTimeout(diffPrefetchTimeoutRef.current)
+				diffPrefetchTimeoutRef.current = null
+			}
+		}
+	}, [selectedIndex, selectedPullRequest?.url, diffFullView])
+
 	const openDiffView = () => {
 		if (!selectedPullRequest) return
 		setDiffFullView(true)
@@ -1883,7 +1970,7 @@ export const App = () => {
 			return
 		}
 
-		// Fullscreen detail mode: scroll with j/k, Ctrl-D/U, exit with Escape/Enter
+		// Fullscreen detail mode handles its own navigation keys.
 		if (detailFullView) {
 			if (key.name === "escape" || (key.name === "return" || key.name === "enter")) {
 				setDetailFullView(false)

package/src/index.tsx

@@ -5,7 +5,13 @@ import { RegistryProvider } from "@effect/atom-react"
 import { createRoot } from "@opentui/react"
 import { App } from "./App.js"
 
-const renderer = await createCliRenderer({ exitOnCtrlC: false })
+process.env.OTUI_USE_ALTERNATE_SCREEN = "true"
+
+const renderer = await createCliRenderer({
+	exitOnCtrlC: false,
+	screenMode: "alternate-screen",
+	externalOutputMode: "passthrough",
+})
 
 createRoot(renderer).render(
 	<RegistryProvider>

package/src/services/GitHubService.ts

@@ -3,52 +3,105 @@ import { config } from "../config.js"
 import type { CheckItem, PullRequestItem } from "../domain.js"
 import { CommandRunner, type CommandError, type JsonParseError } from "./CommandRunner.js"
 
-interface GitHubListPullRequest {
+interface GitHubPullRequestNode {
 	readonly number: number
 	readonly title: string
 	readonly body: string
-	readonly labels: readonly {
-		readonly name: string
-		readonly color?: string | null
-	}[]
+	readonly labels: {
+		readonly nodes: readonly {
+			readonly name: string
+			readonly color?: string | null
+		}[]
+	}
 	readonly additions: number
 	readonly deletions: number
 	readonly changedFiles: number
 	readonly isDraft: boolean
-	readonly reviewDecision: string
-	readonly statusCheckRollup: readonly {
-		readonly name?: string | null
-		readonly context?: string | null
-		readonly status?: string | null
-		readonly conclusion?: string | null
-		readonly state?: string | null
-	}[]
+	readonly reviewDecision: string | null
+	readonly statusCheckRollup?: {
+		readonly contexts: {
+			readonly nodes: readonly GraphQLCheckContext[]
+		}
+	} | null
 	readonly state: string
 	readonly createdAt: string
 	readonly closedAt?: string | null
 	readonly url: string
-}
-
-interface GitHubSearchPullRequest {
-	readonly number: number
 	readonly repository: {
 		readonly nameWithOwner: string
 	}
 }
 
+type GraphQLCheckContext =
+	| {
+		readonly __typename: "CheckRun"
+		readonly name?: string | null
+		readonly status?: string | null
+		readonly conclusion?: string | null
+	}
+	| {
+		readonly __typename: "StatusContext"
+		readonly context?: string | null
+		readonly state?: string | null
+	}
+
+interface GraphQLSearchResponse {
+	readonly data: {
+		readonly search: {
+			readonly nodes: readonly (GitHubPullRequestNode | null)[]
+			readonly pageInfo: {
+				readonly hasNextPage: boolean
+				readonly endCursor: string | null
+			}
+		}
+	}
+}
+
 interface GitHubViewer {
 	readonly login: string
 }
 
-const searchJsonFields = "repository,number"
-const detailJsonFields = "number,title,body,labels,additions,deletions,changedFiles,isDraft,reviewDecision,statusCheckRollup,state,createdAt,closedAt,url"
+const pullRequestSearchQuery = `
+query PullRequests($searchQuery: String!, $first: Int!, $after: String) {
+  search(query: $searchQuery, type: ISSUE, first: $first, after: $after) {
+    nodes {
+      ... on PullRequest {
+        number
+        title
+        body
+        isDraft
+        reviewDecision
+        additions
+        deletions
+        changedFiles
+        state
+        createdAt
+        closedAt
+        url
+        repository { nameWithOwner }
+        labels(first: 20) { nodes { name color } }
+        statusCheckRollup {
+          contexts(first: 100) {
+            nodes {
+              __typename
+              ... on CheckRun { name status conclusion }
+              ... on StatusContext { context state }
+            }
+          }
+        }
+      }
+    }
+    pageInfo { hasNextPage endCursor }
+  }
+}
+`
 
 const normalizeDate = (value: string | null | undefined) => {
 	if (!value || value.startsWith("0001-01-01")) return null
 	return new Date(value)
 }
 
-const getReviewStatus = (item: GitHubListPullRequest): PullRequestItem["reviewStatus"] => {
+const getReviewStatus = (item: GitHubPullRequestNode): PullRequestItem["reviewStatus"] => {
 	if (item.isDraft) return "draft"
 	if (item.reviewDecision === "APPROVED") return "approved"
 	if (item.reviewDecision === "CHANGES_REQUESTED") return "changes"
@@ -73,8 +126,22 @@ const normalizeCheckConclusion = (raw?: string | null): CheckItem["conclusion"]
 	return null
 }
 
-const getCheckInfo = (item: GitHubListPullRequest): Pick<PullRequestItem, "checkStatus" | "checkSummary" | "checks"> => {
-	if (item.statusCheckRollup.length === 0) {
+const getContextStatus = (context: GraphQLCheckContext): CheckItem["status"] => {
+	if (context.__typename === "CheckRun") return normalizeCheckStatus(context.status)
+	if (context.state === "PENDING") return "in_progress"
+	return "completed"
+}
+
+const getContextConclusion = (context: GraphQLCheckContext): CheckItem["conclusion"] => {
+	if (context.__typename === "CheckRun") return normalizeCheckConclusion(context.conclusion)
+	if (context.state === "SUCCESS") return "success"
+	if (context.state === "FAILURE" || context.state === "ERROR") return "failure"
+	return null
+}
+
+const getCheckInfo = (item: GitHubPullRequestNode): Pick<PullRequestItem, "checkStatus" | "checkSummary" | "checks"> => {
+	const contexts = item.statusCheckRollup?.contexts.nodes ?? []
+	if (contexts.length === 0) {
 		return { checkStatus: "none", checkSummary: null, checks: [] }
 	}
 
@@ -84,48 +151,46 @@ const getCheckInfo = (item: GitHubListPullRequest): Pick<PullRequestItem, "check
 	let failing = false
 	const checks: CheckItem[] = []
 
-	for (const check of item.statusCheckRollup) {
-		const name = check.name ?? check.context ?? "check"
+	for (const check of contexts) {
+		const name = check.__typename === "CheckRun" ? check.name ?? "check" : check.context ?? "check"
+		const status = getContextStatus(check)
+		const conclusion = getContextConclusion(check)
 
-		checks.push({
-			name,
-			status: normalizeCheckStatus(check.status),
-			conclusion: normalizeCheckConclusion(check.conclusion),
-		})
+		checks.push({ name, status, conclusion })
 
-		if (check.status === "COMPLETED") {
+		if (status === "completed") {
 			completed += 1
 		} else {
 			pending = true
 		}
 
-		if (check.conclusion === "SUCCESS" || check.conclusion === "NEUTRAL" || check.conclusion === "SKIPPED") {
+		if (conclusion === "success" || conclusion === "neutral" || conclusion === "skipped") {
 			successful += 1
-		} else if (check.conclusion && check.conclusion !== "SUCCESS") {
+		} else if (conclusion) {
 			failing = true
 		}
 	}
 
 	if (pending) {
-		return { checkStatus: "pending", checkSummary: `checks ${completed}/${item.statusCheckRollup.length}`, checks }
+		return { checkStatus: "pending", checkSummary: `checks ${completed}/${contexts.length}`, checks }
 	}
 
 	if (failing) {
-		return { checkStatus: "failing", checkSummary: `checks ${successful}/${item.statusCheckRollup.length}`, checks }
+		return { checkStatus: "failing", checkSummary: `checks ${successful}/${contexts.length}`, checks }
 	}
 
-	return { checkStatus: "passing", checkSummary: `checks ${successful}/${item.statusCheckRollup.length}`, checks }
+	return { checkStatus: "passing", checkSummary: `checks ${successful}/${contexts.length}`, checks }
 }
 
-const parsePullRequest = (repository: string, item: GitHubListPullRequest): PullRequestItem => {
+const parsePullRequest = (item: GitHubPullRequestNode): PullRequestItem => {
 	const checkInfo = getCheckInfo(item)
 
 	return {
-		repository,
+		repository: item.repository.nameWithOwner,
 		number: item.number,
 		title: item.title,
 		body: item.body,
-		labels: item.labels.map((label) => ({
+		labels: item.labels.nodes.map((label) => ({
 			name: label.name,
 			color: label.color ? `#${label.color}` : null,
 		})),
@@ -143,22 +208,7 @@ const parsePullRequest = (repository: string, item: GitHubListPullRequest): Pull
 	}
 }
 
-const searchOpenArgs = (author: string) => [
-	"search",
-	"prs",
-	"--author",
-	author,
-	"--state",
-	"open",
-	"--limit",
-	String(config.prFetchLimit),
-	"--sort",
-	"created",
-	"--order",
-	"desc",
-	"--json",
-	searchJsonFields,
-] as const
+const searchQuery = (author: string) => `author:${author} is:pr is:open sort:created-desc`
 
 type GitHubError = CommandError | JsonParseError
 
@@ -177,18 +227,27 @@ export class GitHubService extends Context.Service<GitHubService, {
 			const command = yield* CommandRunner
 
 			const listOpenPullRequests = Effect.fn("GitHubService.listOpenPullRequests")(function*() {
-				const searchResults = yield* command.runJson<readonly GitHubSearchPullRequest[]>("gh", [...searchOpenArgs(config.author)])
-				const pullRequests = yield* Effect.forEach(
-					searchResults,
-					Effect.fn("GitHubService.loadPullRequestDetail")(function*(searchResult) {
-						const repository = searchResult.repository.nameWithOwner
-						const pullRequest = yield* command.runJson<GitHubListPullRequest>("gh", [
-							"pr", "view", String(searchResult.number), "--repo", repository, "--json", detailJsonFields,
-						])
-						return parsePullRequest(repository, pullRequest)
-					}),
-					{ concurrency: 8 },
-				)
+				const pullRequests: PullRequestItem[] = []
+				let cursor: string | null = null
+
+				while (pullRequests.length < config.prFetchLimit) {
+					const pageSize = Math.min(100, config.prFetchLimit - pullRequests.length)
+					const response: GraphQLSearchResponse = yield* command.runJson<GraphQLSearchResponse>("gh", [
+						"api", "graphql",
+						"-f", `query=${pullRequestSearchQuery}`,
+						"-F", `searchQuery=${searchQuery(config.author)}`,
+						"-F", `first=${pageSize}`,
+						...(cursor ? ["-F", `after=${cursor}`] : []),
+					])
+
+					for (const node of response.data.search.nodes) {
+						if (node) pullRequests.push(parsePullRequest(node))
+					}
+
+					if (!response.data.search.pageInfo.hasNextPage) break
+					cursor = response.data.search.pageInfo.endCursor
+					if (!cursor) break
+				}
 
 				return pullRequests.sort((left, right) => right.createdAt.getTime() - left.createdAt.getTime())
 			})