Skip to content

fix(app): offer PR actions only when they can work, and say what GitHub and the terminal mean - #587

Merged
iamnbutler merged 5 commits into
mainfrom
fix-pr-actions
Oct 22, 2026
Merged

iamnbutler merged 5 commits into
mainfrom
fix-pr-actions

Conversation

@iamnbutler

Copy link
Copy Markdown
Contributor

Fix-train task "PR actions, React loop and error text". Map ids: ^720 ^719 ^724 ^725 ^743 ^633 ^770 ^811.

  • ^720 Merge PR waits for nothing left to commit or push, not just nothing unpushed (git.ts).
  • ^719 ^724 Ready for review and Fix checks appear only on pull requests the channel owner's GitHub account wrote; the agent can't push to or ready someone else's PR.
  • ^725 A dismissed failed-checks notice stays dismissed across reloads (per host and channel, keyed by the failed runs so a new failure shows again).
  • ^743 GitHub's GraphQL: Could not resolve to a Repository … becomes a sentence about the origin and the gh account's access; other GraphQL errors say "GitHub refused the request:".
  • ^770 A terminal tab whose shell the host no longer has says it's a new shell and the old output is gone.
  • ^811 On a phone, choosing a project closes the drawer over Issues and PRs.
  • ^633 React fix(app): add per-project channel creation in All Projects #185 was not reproduced (two attempts with a development React build during multi-step runs with the Diff tab and details open), so nothing changed for it.

Smoked on a loopback host from this lane: Merge offered on clean githubnext/ace-testbed#2 and hidden with a dirty file; Ready for review shown on my own draft and absent (with Fix) on honojs/hono#5535; notice dismissal survived a reload; a project with origin githubnext/no-such-repo-fixb showed the new sentence on Issues; restarting the host showed the new-shell line in an open terminal; at 390px the drawer closed after picking a project and PRs was reachable.

Not covered: the PRs list for a missing repo shows "No open pull requests" because gh pr list --search returns nothing instead of an error. The session-details checks list warns about a duplicate React key when GitHub lists the same check URL twice (codecov on hono#5535).

Built in Ace

No. Built by Claude Code as a fix-train worker outside Ace, driving a loopback Ace host with Playwright for smoke tests; the fix train runs headless agents in parallel, which Ace doesn't orchestrate yet. See #5.

…ub and the terminal mean

Merge waits for nothing left to commit or push; Ready for review and Fix checks appear only on
pull requests the channel owner's GitHub account wrote. A dismissed checks notice stays dismissed
across reloads. GitHub's "Could not resolve to a Repository" becomes a sentence about the origin
and access. A terminal whose shell the host lost says so. Choosing a project on a phone closes the
drawer over Issues and PRs.

Why: apps/app/src/git.ts#function useGit | the agent pushes as the channel owner's GitHub account, so Fix and Ready for review only work on that account's pull requests
@iamnbutler
iamnbutler merged commit b7c7f8d into main Oct 22, 2026
1 check passed
@iamnbutler
iamnbutler deleted the fix-pr-actions branch October 22, 2026 15:25
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