Skip to content

Fix pull request lookup for fork branches - #1150

Merged
ymichael merged 3 commits into
mainfrom
bb/diagnose-localhost-thread-url-issue-thr_djyeg2py3p
Aug 7, 2026
Merged

Fix pull request lookup for fork branches#1150
ymichael merged 3 commits into
mainfrom
bb/diagnose-localhost-thread-url-issue-thr_djyeg2py3p

Conversation

@ymichael

@ymichael ymichael commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • let GitHub CLI resolve pull requests from the checked-out branch and its configured upstream
  • fix prompt-banner PR detection for branches tracking contributor forks
  • apply the same tracking-aware resolution to ready, draft, and merge actions
  • preserve detached-head and lookup-failure behavior

Reproduction

PR #1076 checked out a branch that tracked Diffuzmetall/bb. Passing its bare branch name to gh pr view searched the base repository and returned no pull request. Running gh pr view from the worktree without a positional target followed the fork upstream and found the PR.

Validation

  • 185 host-workspace tests passed
  • host-workspace and host-daemon typechecks passed
  • verified the current-branch helper resolves same-repository and fork-backed pull requests and still reports no PR for main

Commands

  • pnpm exec turbo run test --filter=@bb/host-workspace --force
  • pnpm exec turbo run typecheck --filter=@bb/host-workspace --filter=@bb/host-daemon

@ymichael
ymichael merged commit 787d717 into main Aug 7, 2026
10 checks passed
@ymichael
ymichael deleted the bb/diagnose-localhost-thread-url-issue-thr_djyeg2py3p branch August 7, 2026 18:28
amadad pushed a commit to amadad/bb that referenced this pull request Aug 14, 2026
## Summary

- let GitHub CLI resolve pull requests from the checked-out branch and
its configured upstream
- fix prompt-banner PR detection for branches tracking contributor forks
- apply the same tracking-aware resolution to ready, draft, and merge
actions
- preserve detached-head and lookup-failure behavior

## Reproduction

PR get-bb#1076 checked out a branch that tracked Diffuzmetall/bb. Passing its
bare branch name to gh pr view searched the base repository and returned
no pull request. Running gh pr view from the worktree without a
positional target followed the fork upstream and found the PR.

## Validation

- 185 host-workspace tests passed
- host-workspace and host-daemon typechecks passed
- verified the current-branch helper resolves same-repository and
fork-backed pull requests and still reports no PR for main

## Commands

- pnpm exec turbo run test --filter=@bb/host-workspace --force
- pnpm exec turbo run typecheck --filter=@bb/host-workspace
--filter=@bb/host-daemon
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