Skip to content

fix(pr): show the project fork instead of upstream pull requests #128

Open

Description

@PsychedelicShayna

Note

GPT-6 Astra on behalf of PsychedelicShayna

Observed behavior

The owner selects NeoPi but sees can1357/oh-my-pi PRs rather than PsychedelicShayna/neopi; Authored is empty and filters take too long to apply.

At nightly 3c5789f, RepositoryIdentityResolver.ts:58-71 explicitly prefers upstream over origin. The GitHub CLI already scopes requests to the resolved repository and verified account, so the wrong identity directs those queries to the wrong repository. Cached rows can retain the same projectId after identity correction.

Acceptance

  • A project with origin and upstream uses its origin fork; existing fallback behavior works when origin is absent.
  • Authored queries use the verified account against that project repository, not its upstream parent.
  • Previously cached upstream rows cannot survive the corrected project/environment repository filter.
  • Selected-project requests avoid redundant host-wide discovery before an already-known explicit repository query; All Projects remains correct.
  • Record focused repository/filter regressions and isolated PR-page evidence without mutating upstream repositories.

Claimed by Mercury/ForkPullRequestsRepair. No new author identity mechanism or whole-page redesign.

Owner's words (verbatim)

2026-10-22 owner dictation — contiguous excerpt

There's a PR button, pull requests. Okay. And that's relative to what? If I select my fork. it takes way too long to apply a filter. You see, I do new apply, and it's showing me upstream's PRs instead of my PRs, simply because it's a fork of upstream. I think that has to change. That behavior's weird. Unless I'm, like, misunderstanding something. Yeah, pull requests, all, all, yeah. But the other options are open, close marriage, yeah, I wanna see all of them, and, you know, reviewing an authored. Authored? Well, nothing shows up. This is like, I haven't authored anything, even though I have. I do all projects, and then I do involvement authored. Nope, nothing. And it takes way too long to refresh, too. Yeah, so that page is broken, in my opinion. The way that it functions is not useful to me. I cannot see the, the PRs of my projects. and I don't know what it's relative to. I would assume that whatever project I'm currently in, I click the PR button, it shows me the PRs for that project, right? I would much rather I see why. It's showing upstream instead of remote. It's showing, yeah, it's not showing psychedelic Shane slash Neo NeoPy. This is showing, okay, hold on. It's classifying oh my pi under, yeah, it's calling repos neo pi can one three five seven slash oh my pi. It's not recognizing the fact that it's a fork because it's listening to it being it's listening to upstream being present, and it's not listening to remote. It doesn't care about remote, it cares about upstream. In other words, that makes it just useless for me. It's showing me PRs of a repo that isn't mine. That's a behavior that needs to change. You need to change that behavior.

Complete unedited message: #125 . The owner's words take precedence over this summary.

Activity

  1. added
    bugSomething isn't working
    priority: p1High value or prerequisite for several issues
    agent: startedAn agent has claimed this; do not duplicate the work
    on Oct 22, 2026
  2. PsychedelicShayna commented on Oct 22, 2026

    @PsychedelicShayna
    OwnerAuthor

    Note

    GPT-6 Astra on behalf of PsychedelicShayna

    Installed nightly source 3ca07c37659338f9fde4142baf50d20f09e4c5d8, version 0.0.42-nightly.20261022.39742893669.1. The NeoCode launcher now selects this versioned AppRun; the previous artifact and launcher backup remain available. AppImage SHA256: bde7d6dce6a2928cedf8104ffcf912d8dafb75d59b65f79a052aa65b671bc113.

    Proof used the actual installed Electron artifact, normal shell hydration and private state on Xvfb :10. The owner's live window, database and wallet were not used or restarted.

    PR #130 is delivered. The same private profile/project showed can1357/oh-my-pi before the repair and psychedelicshayna/neopi after it. The explicit Authored filter now shows the owner's real fork PRs. Explicit project selection with all PR states remains on the fork; upstream rows are absent. GitHub requests in this proof were read-only.

    The four focused repository/query/filter files passed 431 tests. The corrected cross-environment fixture then passed 128 tests in its affected web file; web types passed. Targeted lint/format passed. Server types retain only the disclosed serverSettings.test.ts:235 baseline diagnostic.

    Before · After · Authored · Explicit project/all states · Structured proof.

    Exact tested Linux AppImage. Marking agent work finished; this issue remains open until promotion to main.

  3. added
    agent: finishedThe agent's part is done; not necessarily mergeable
    and removed
    agent: startedAn agent has claimed this; do not duplicate the work
    on Oct 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    agent: finishedThe agent's part is done; not necessarily mergeablebugSomething isn't workingpriority: p1High value or prerequisite for several issues

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions