Skip to content

fix(state): use search_prs for status buffer PR info fetch#115

Merged
actionshrimp merged 1 commit intomainfrom
fix/status-pr-info-search
Feb 24, 2026
Merged

fix(state): use search_prs for status buffer PR info fetch#115
actionshrimp merged 1 commit intomainfrom
fix/status-pr-info-search

Conversation

@actionshrimp
Copy link
Owner

Summary

  • fetch_pr_info (the status buffer PR summary line) had the same bug as fix(forge): use search_prs for View current branch PR (N v) #114: it used list_prs with a limit of 50 and filtered client-side by branch name. In large repos with many open PRs, the current branch's PR could fall outside the top 50, so the PR line would never appear.
  • Now uses search_prs with a targeted head:<branch> GitHub search query to find the PR server-side, matching the pattern used by pr_list.lua.

Test plan

  • E2E test verifies fetch_pr_info calls search_prs with correct head:<branch> query and limit of 1
  • All existing PR summary tests still pass (rendering, review decision, diff stat, checks, TTL, config)
  • All 1540 unit tests pass
  • All 628 E2E tests pass
  • Lint passes

Same bug as the forge popup: fetch_pr_info used list_prs with a limit
of 50 and filtered client-side, so the status PR line silently failed
in repos with many open PRs. Now uses search_prs with a targeted
`head:<branch>` query to find the PR server-side.
@actionshrimp actionshrimp merged commit d52d280 into main Feb 24, 2026
3 checks passed
@actionshrimp actionshrimp deleted the fix/status-pr-info-search branch February 24, 2026 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant