Skip to content

fix(web): open a fork PR against origin, not upstream - #412

Merged
patroza merged 1 commit into
fork/devfrom
fix/pr-panel-fork-repo
Aug 15, 2026
Merged

fix(web): open a fork PR against origin, not upstream#412
patroza merged 1 commit into
fork/devfrom
fix/pr-panel-fork-repo

Conversation

@patroza

@patroza patroza commented Aug 15, 2026

Copy link
Copy Markdown
Owner

Summary

The thread PR panel used the project's primary repository identity. On a fork that prefers upstream, so #410 was loaded as pingdotgg/t3code#410.

That number exists only on the fork:

Repo #410
patroza/t3code open — docs(agents): drop obsolete fork stack workflow
pingdotgg/t3code GraphQL: could not resolve

The agent on the left found the fork PR (gh pr view 410 with GH_REPO / origin). The panel asked GitHub for the upstream number and showed Pull request not found.

This is not the earlier App-install / wrapper issue. Ops #81 and product #408 already landed; t3vm is minting public-read fallback tokens.

Fix: open the panel from the change-request URL (patroza/t3code), and accept any of the checkout's remotes on the server so origin is still this project.

Test plan

  • vp test run apps/server/src/pullRequest/PullRequestService.test.ts apps/web/src/components/pullRequest/pullRequestDetail.logic.test.ts
  • After deploy: open the #410 thread panel and confirm it loads patroza/t3code#410
  • Confirm an upstream PR on the same checkout still opens against pingdotgg/t3code

The thread PR panel used the project's primary repository identity, which
prefers the upstream remote on a fork. #410 exists on patroza/t3code and
not on pingdotgg/t3code, so detail failed as "Pull request not found".

Open the panel from the change-request URL, and accept any of the
checkout's remotes on the server so origin is still this project.
@patroza
patroza marked this pull request as ready for review August 15, 2026 14:30
@patroza
patroza enabled auto-merge (squash) August 15, 2026 14:30
@patroza
patroza disabled auto-merge August 15, 2026 14:32
@patroza
patroza enabled auto-merge (squash) August 15, 2026 14:32
@patroza
patroza disabled auto-merge August 15, 2026 14:32
@patroza
patroza merged commit e8d6913 into fork/dev Aug 15, 2026
3 of 8 checks passed
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