feat: show GitHub pull request status across desktop, remote, and iOS sessions - #116
Open
jsgrrchg wants to merge 18 commits into
Open
feat: show GitHub pull request status across desktop, remote, and iOS sessions#116jsgrrchg wants to merge 18 commits into
jsgrrchg wants to merge 18 commits into
Conversation
jsgrrchg
marked this pull request as ready for review
August 15, 2026 23:02
wingleeio
requested changes
Aug 16, 2026
wingleeio
left a comment
Contributor
There was a problem hiding this comment.
The host-resolved PR path and desktop UI behave correctly in a live fork/upstream run, but the new engine test suite has one blocking portability failure documented inline.
Validation performed:
cargo fmt --all -- --check— passedcargo test -p zeron-engine— failed atgithub_cli_on_login_shell_path_only_resolves_pull_request- Full engine suite with only that test skipped, plus
zeron-rpc,zeron-ui,zeron-proto, andzeron-harness— passed - Live desktop run — resolved cross-repository PR #116 and rendered the sidebar/composer badges and tooltip
- iOS tests were not runnable on this Linux host
Live desktop badges:
Tooltip state/title:
Contributor
Author
|
Fixed in The test now uses an isolated direct Validated with: Ui wise looks good? Tried to be minimal for now. |
jsgrrchg
added a commit
to jsgrrchg/comet
that referenced
this pull request
Aug 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Summary
Add host-resolved GitHub pull request status to Zeron sessions across local desktop, remote desktop, and iOS surfaces.
The checkout host uses its existing authenticated
ghinstallation to resolve the pull request. Remote desktop and iOS clients only receive the sanitized change-request summary through the existing RPC/device relay infrastructure.What changed
zeron-proto.gh pr list.gh.WatchCheckoutChangeRequestas a local and forwardable stream RPC.UnknownMethod.Compatibility and security
ghruns only on the device that owns the checkout.Future PR views
This work establishes the foundation for a future pull-request view.
The provider boundary, Git remote discovery, host-side
ghexecution, neutral change-request models, error classification, caching, and device-aware RPC transport are intentionally reusable for listing pull requests, loading details, checks, reviews, diffs, and future PR actions.This PR only resolves the PR associated with the active checkout branch; a complete PR browser remains out of scope.
Validation
Passing:
cargo fmt --all -- --checkcargo test -p zeron-enginecargo test -p zeron-rpccargo test -p zeron-uighinstallations, and default-branch historical PR suppression.The iOS simulator suite could not be completed in the current environment: the documented
iPhone 16destination is unavailable, and the availableZeron iPhone 17 Prosimulator rejected the test runner launch with an externalApplication failed preflight checks: Busyerror.Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.