Skip to content

feat: --pr names the pull request a detached checkout cannot, and the inbox passes it - #87

Merged
fiddur merged 2 commits into
developfrom
inbox-open-pr
Sep 3, 2026
Merged

feat: --pr names the pull request a detached checkout cannot, and the inbox passes it#87
fiddur merged 2 commits into
developfrom
inbox-open-pr

Conversation

@fiddur

@fiddur fiddur commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

A session opened from the inbox showed no GitHub controls. Its worktree is detached at the PR head, so the server's gh pr view for the current branch found nothing and /api/github/details answered null; the page rendered an empty bar where the description, reviews and submit dialog belong.

  • diffity --pr <number> <base>: names the pull request when the checkout cannot. Needs exactly one ref (the base the PR is against), refuses a working-tree ref, and cannot be combined with a PR URL. Like the URL flow, it pins /diff to that base.
  • The inbox spawns its sessions with --pr <store row number>.
  • Verified on the live NCBackend3#14502 worktree with this machine's gh 2.46: the details endpoint returns the PR and a bare /diff redirects to the pinned base.

An inbox session started by the previous build keeps being reused by /open until it is stopped once; the daemon also needs a restart to pass the flag.

Bumps to 0.10.13.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Bwp5QefjsjMFeq6CK6cT6w

fiddur and others added 2 commits September 3, 2026 14:33
… inbox passes it

A session the inbox opened had no pull request: the worktree is detached at the PR head, so
`gh pr view` for the current branch found nothing, the details endpoint answered null, and
the page showed an empty bar where the description, reviews and submit dialog belong.
`diffity --pr <number> <base>` hands the server the number directly and pins /diff to the
base, as the URL flow does; the inbox spawns its sessions that way, with the number from
the store row.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Bwp5QefjsjMFeq6CK6cT6w
… and its wiring is tested

The check ran before --base/--compare were folded into the refs, so `--pr 5 --base X` was
refused with a message asking for what --base had supplied. A repository with no GitHub
remote is refused up front, as the URL flow does, instead of accepting a number that could
show nothing. The tests now start the entry with --pr and check the pinned redirect.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Bwp5QefjsjMFeq6CK6cT6w
@fiddur
fiddur marked this pull request as ready for review September 3, 2026 12:43
@fiddur
fiddur merged commit 03073c6 into develop Sep 3, 2026
2 checks passed
@fiddur
fiddur deleted the inbox-open-pr branch September 3, 2026 12:43
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