chore: point pr-review workflow to extensions@main by xingyaoww · Pull Request #2928 · OpenHands/software-agent-sdk

xingyaoww · 2026-04-23T15:33:48Z

Why

OpenHands/extensions#164 (sub-agent delegation for file-level reviews) has been merged to main. The pr-review workflow was temporarily pinned to the feature branch — this PR reverts it to @main.

Summary

Update uses: from @feat/pr-review-sub-agent-delegation → @main
Remove the extensions-version input (no longer needed — the feature branch pin was only required while feat: Add GPT-5 model support and reasoning debug example #164 was unmerged)
Remove the deprecated review-style input (was already ignored per extensions#164)
Keep use-sub-agents: 'true' — the sub-agent delegation code is now on main

Notes

This resolves the TODO comment that was left in the workflow file by PR #2839:

# TODO: Revert to @main after OpenHands/extensions#164 is merged

This PR was created by an AI assistant (OpenHands) on behalf of a user.

Agent Server images for this PR

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

Variants & Base Images

Variant	Architectures	Base Image	Docs / Tags
java	amd64, arm64	`eclipse-temurin:17-jdk`	Link
python	amd64, arm64	`nikolaik/python-nodejs:python3.13-nodejs22-slim`	Link
golang	amd64, arm64	`golang:1.21-bookworm`	Link

Pull (multi-arch manifest)

# Each variant is a multi-arch manifest supporting both amd64 and arm64
docker pull ghcr.io/openhands/agent-server:4f5abe9-python

Run

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

All tags pushed for this build

ghcr.io/openhands/agent-server:4f5abe9-golang-amd64
ghcr.io/openhands/agent-server:4f5abe9-golang_tag_1.21-bookworm-amd64
ghcr.io/openhands/agent-server:4f5abe9-golang-arm64
ghcr.io/openhands/agent-server:4f5abe9-golang_tag_1.21-bookworm-arm64
ghcr.io/openhands/agent-server:4f5abe9-java-amd64
ghcr.io/openhands/agent-server:4f5abe9-eclipse-temurin_tag_17-jdk-amd64
ghcr.io/openhands/agent-server:4f5abe9-java-arm64
ghcr.io/openhands/agent-server:4f5abe9-eclipse-temurin_tag_17-jdk-arm64
ghcr.io/openhands/agent-server:4f5abe9-python-amd64
ghcr.io/openhands/agent-server:4f5abe9-nikolaik_s_python-nodejs_tag_python3.13-nodejs22-slim-amd64
ghcr.io/openhands/agent-server:4f5abe9-python-arm64
ghcr.io/openhands/agent-server:4f5abe9-nikolaik_s_python-nodejs_tag_python3.13-nodejs22-slim-arm64
ghcr.io/openhands/agent-server:4f5abe9-golang
ghcr.io/openhands/agent-server:4f5abe9-java
ghcr.io/openhands/agent-server:4f5abe9-python

About Multi-Architecture Support

Each variant tag (e.g., 4f5abe9-python) is a multi-arch manifest supporting both amd64 and arm64
Docker automatically pulls the correct architecture for your platform
Individual architecture tags (e.g., 4f5abe9-python-amd64) are also available if needed

@main

OpenHands/extensions#164 (sub-agent delegation for file-level reviews) has been merged to main. Update the workflow to: - Reference @main instead of the feature branch - Remove the now-unnecessary extensions-version pin - Remove the deprecated review-style input - Keep use-sub-agents: 'true' (now available on main) Co-authored-by: openhands <openhands@all-hands.dev>

github-actions · 2026-04-23T15:34:18Z

Python API breakage checks — ✅ PASSED

Result: ✅ PASSED

Action log

github-actions · 2026-04-23T15:34:27Z

REST API breakage checks (OpenAPI) — ✅ PASSED

Result: ✅ PASSED

Action log

all-hands-bot

✅ QA Report: PASS

This PR correctly reverts the pr-review workflow to use extensions@main now that the sub-agent delegation feature has been merged upstream.

Does this PR achieve its stated goal?

Yes. The PR successfully reverts the workflow configuration as intended. The prerequisite (OpenHands/extensions#164 merged to main at 2026-04-23T15:32:28Z) is met, and all four claimed changes are correctly implemented: (1) uses: updated from @feat/pr-review-sub-agent-delegation to @main, (2) deprecated review-style input removed, (3) extensions-version input removed (defaults to main), and (4) use-sub-agents: 'true' preserved. The TODO comment from PR #2839 is resolved.

Phase	Result
Environment Setup	✅ Checked out PR branch, validated YAML syntax
CI & Tests	⚠️ 18/20 pass, PR Review workflow fails (pre-existing Python version issue in extensions@main, not caused by this PR)
Functional Verification	✅ All changes verified: workflow reference updated, deprecated inputs removed, action.yml schema validated

Functional Verification

Test 1: Verify prerequisite (extensions#164 merged)

Baseline check:

gh pr view 164 --repo OpenHands/extensions --json state,mergedAt,title

Output:

{
  "mergedAt": "2026-04-23T15:32:28Z",
  "state": "MERGED",
  "title": "feat(pr-review): add sub-agent delegation for file-level reviews"
}

This confirms the prerequisite is met — extensions#164 was merged to main 7 minutes before this QA run.

Test 2: Validate workflow YAML syntax

Ran:

python -c "import yaml; yaml.safe_load(open('.github/workflows/pr-review-by-openhands.yml')); print('✓ Workflow YAML syntax is valid')"

Output:

✓ Workflow YAML syntax is valid

The workflow file parses correctly as valid YAML.

Test 3: Verify action exists at extensions@main

Ran:

curl -H "Authorization: token $GITHUB_TOKEN" \
  "https://api.github.com/repos/OpenHands/extensions/contents/plugins/pr-review/action.yml?ref=main" \
  | jq -r '.content' | base64 -d | head -50

The action.yml exists at OpenHands/extensions/plugins/pr-review@main and shows:

review-style input: "[DEPRECATED] Previously chose between 'standard' and 'roasted' review styles... Will be removed in a future version."
extensions-version input: defaults to main
use-sub-agents input: valid, defaults to 'false'

This confirms the removed inputs (review-style, extensions-version) are either deprecated or have sensible defaults, and the kept input (use-sub-agents) is valid.

Test 4: Compare before/after workflow configuration

Before (main branch, lines 42-55):

- name: Run PR Review
  # TODO: Revert to @main after OpenHands/extensions#164 is merged
  uses: OpenHands/extensions/plugins/pr-review@feat/pr-review-sub-agent-delegation
  with:
      llm-model: litellm_proxy/claude-sonnet-4-5-20250929
      llm-base-url: https://llm-proxy.app.all-hands.dev
      # Review style: roasted (other option: standard)
      review-style: roasted
      # Enable experimental sub-agent delegation for file-level reviews
      use-sub-agents: 'true'
      # Must match the branch in `uses:` so the script checkout
      # picks up the sub-agent code (extensions#164 not yet on main)
      extensions-version: feat/pr-review-sub-agent-delegation
      llm-api-key: ${{ secrets.LLM_API_KEY }}

After (PR branch, lines 42-52):

- name: Run PR Review
  uses: OpenHands/extensions/plugins/pr-review@main
  with:
      llm-model: litellm_proxy/claude-sonnet-4-5-20250929
      llm-base-url: https://llm-proxy.app.all-hands.dev
      # Enable experimental sub-agent delegation for file-level reviews
      use-sub-agents: 'true'
      llm-api-key: ${{ secrets.LLM_API_KEY }}
      github-token: ${{ secrets.OPENHANDS_BOT_GITHUB_PAT_PUBLIC }}
      lmnr-api-key: ${{ secrets.LMNR_SKILLS_API_KEY }}

Delta:

✅ uses: changed from @feat/pr-review-sub-agent-delegation → @main
✅ Removed review-style: roasted (deprecated per action.yml)
✅ Removed extensions-version: feat/pr-review-sub-agent-delegation (defaults to main)
✅ Kept use-sub-agents: 'true' (as stated in PR description)
✅ Removed TODO comment
✅ Removed explanatory comments that are no longer relevant

All changes match the PR description exactly.

Test 5: Validate all workflow inputs against action.yml schema

Inputs provided in updated workflow:

llm-model → ✅ Valid (optional, has default)
llm-base-url → ✅ Valid (optional)
use-sub-agents → ✅ Valid (optional, defaults to 'false')
llm-api-key → ✅ Valid (required)
github-token → ✅ Valid (required)
lmnr-api-key → ✅ Valid (optional)

All inputs are recognized by the action and satisfy the schema requirements.

CI Status Note

PR Review workflow failure: The "PR Review by OpenHands" check failed with:

error: No interpreter found for Python 3.13 in virtual environments or search path

This is a Python version mismatch in the extensions@main action (sets up Python 3.12 but uv tries to use 3.13). This is a pre-existing issue in the extensions repository, not caused by this PR:

Timeline shows extensions#164 merged at 15:32:28Z
Last successful PR review run was at 12:28:26Z (using the old @feat/pr-review-sub-agent-delegation branch)
This PR's run at 15:39:05Z is the first run using @main after the merge
The Python version issue exists in the newly merged extensions@main code

The issue should be fixed in the extensions repo separately (likely need to update Python version constraints in the agent script or action.yml).

All other CI checks pass: 18/20 checks successful, including pre-commit (which validates YAML syntax).

Issues Found

None. The workflow change is correct and achieves its stated goal.

all-hands-bot

Clean revert to @main after upstream merge. All changes are correct: feature branch removed, deprecated inputs cleaned up, sub-agent flag preserved.

xingyaoww marked this pull request as ready for review April 23, 2026 15:39

xingyaoww enabled auto-merge (squash) April 23, 2026 15:39

all-hands-bot reviewed Apr 23, 2026

View reviewed changes

xingyaoww mentioned this pull request Apr 23, 2026

fix(pr-review): use UV_PYTHON instead of UV_PYTHON_PREFERENCE OpenHands/extensions#203

Merged

all-hands-bot approved these changes Apr 23, 2026

View reviewed changes

xingyaoww merged commit 2336e9e into main Apr 23, 2026
38 of 40 checks passed

xingyaoww deleted the chore/pr-review-point-to-extensions-main branch April 23, 2026 16:08

VascoSch92 mentioned this pull request Apr 26, 2026

pr-review action hangs ~15m when sub-agent review delegation is enabled OpenHands/extensions#208

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: point pr-review workflow to extensions@main#2928

chore: point pr-review workflow to extensions@main#2928
xingyaoww merged 1 commit into
mainfrom
chore/pr-review-point-to-extensions-main

xingyaoww commented Apr 23, 2026 •

edited by github-actions Bot

Loading

Uh oh!

github-actions Bot commented Apr 23, 2026

Uh oh!

github-actions Bot commented Apr 23, 2026

Uh oh!

all-hands-bot left a comment

Uh oh!

all-hands-bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

xingyaoww commented Apr 23, 2026 • edited by github-actions Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Why

Summary

Notes

Uh oh!

github-actions Bot commented Apr 23, 2026

Python API breakage checks — ✅ PASSED

Uh oh!

github-actions Bot commented Apr 23, 2026

REST API breakage checks (OpenAPI) — ✅ PASSED

Uh oh!

all-hands-bot left a comment

Choose a reason for hiding this comment

✅ QA Report: PASS

Does this PR achieve its stated goal?

Test 1: Verify prerequisite (extensions#164 merged)

Test 2: Validate workflow YAML syntax

Test 3: Verify action exists at extensions@main

Test 4: Compare before/after workflow configuration

Test 5: Validate all workflow inputs against action.yml schema

Issues Found

Uh oh!

all-hands-bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

xingyaoww commented Apr 23, 2026 •

edited by github-actions Bot

Loading