Skip to content

Conversation

@simonrosenberg
Copy link
Collaborator

@simonrosenberg simonrosenberg commented Oct 21, 2025

This PR demonstrates that the fork fix from #841 works correctly by applying it to the feature/sre-debugger branch from the fork tejas-goyal/agent-sdk.

Changes Made

  • Applied the fork fix to both workflow files:
    • .github/workflows/pr-review-by-openhands.yml
    • examples/03_github_workflows/02_pr_review/workflow.yml
  • Added repository: ${{ github.event.pull_request.head.repo.full_name }} parameter to the checkout action

Testing the Fix

This PR can be used to test that the PR review workflow now works correctly with forks:

  1. Add the review-this label to this PR
  2. The workflow should successfully checkout the code from the fork repository
  3. The workflow should complete without the "branch could not be found" error

Related

Note

This is a test PR to validate the fork fix. Once confirmed working, this branch can be deleted as the main fix is already in #841.

@simonrosenberg can click here to continue refining the PR


Agent Server images for this PR

GHCR package: https://github.com/OpenHands/agent-sdk/pkgs/container/agent-server

Variants & Base Images

Variant Base Image Docs / Tags
golang golang:1.21-bookworm Link
java eclipse-temurin:17-jdk Link
python nikolaik/python-nodejs:python3.12-nodejs22 Link

Pull (multi-arch manifest)

docker pull ghcr.io/openhands/agent-server:64f7461-python

Run

docker run -it --rm \
  -p 8000:8000 \
  --name agent-server-64f7461-python \
  ghcr.io/openhands/agent-server:64f7461-python

All tags pushed for this build

ghcr.io/openhands/agent-server:64f7461-golang
ghcr.io/openhands/agent-server:v1.0.0a3_golang_tag_1.21-bookworm_binary
ghcr.io/openhands/agent-server:64f7461-java
ghcr.io/openhands/agent-server:v1.0.0a3_eclipse-temurin_tag_17-jdk_binary
ghcr.io/openhands/agent-server:64f7461-python
ghcr.io/openhands/agent-server:v1.0.0a3_nikolaik_s_python-nodejs_tag_python3.12-nodejs22_binary

The 64f7461 tag is a multi-arch manifest (amd64/arm64); your client pulls the right arch automatically.

Tejas Goyal and others added 5 commits October 17, 2025 15:53
Add repository parameter to checkout action to properly handle PRs from forks.
This fixes the issue where the workflow fails with 'branch could not be found'
when trying to checkout from fork repositories.

Fixes #809

Co-authored-by: openhands <[email protected]>
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.

OpenHands PR review workflow is failing on forks

4 participants