Skip to content

Fix OpenHands PR review workflow auth#131

Merged
neubig merged 1 commit into
mainfrom
fix/pr-review-workflow
Apr 24, 2026
Merged

Fix OpenHands PR review workflow auth#131
neubig merged 1 commit into
mainfrom
fix/pr-review-workflow

Conversation

@neubig
Copy link
Copy Markdown
Member

@neubig neubig commented Apr 24, 2026

Summary

  • switch the PR review workflow to the org-standard LiteLLM proxy model/base URL used by other OpenHands repos
  • use the shared OPENHANDS_BOT_GITHUB_PAT_PUBLIC secret for review comments instead of the default GITHUB_TOKEN
  • add the same trigger/security comments used in other working OpenHands repositories for consistency

Why

The current workflow is configured differently from other OpenHands repos and fails with litellm.AuthenticationError during the OpenHands review step.

Verification

  • compared .github/workflows/pr-review-by-openhands.yml against working configurations in OpenHands/extensions, OpenHands/docs, OpenHands/benchmarks, and OpenHands/software-agent-sdk
  • confirmed the failing run for OpenHands/typescript-client#130 errors with litellm.AuthenticationError

This PR was created by an AI agent (OpenHands) on behalf of the user.

Co-authored-by: openhands <openhands@all-hands.dev>
@neubig neubig merged commit e43aad2 into main Apr 24, 2026
6 of 7 checks passed
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.

2 participants