Skip to content

Conversation

@xingyaoww
Copy link
Collaborator

@xingyaoww xingyaoww commented Jan 13, 2026

Adds a backward-compatible, file-based repo hook .openhands/agent_finish.sh.

Implementation: HookConfig.load() detects the file in the project working_dir and automatically registers it as a Stop hook (command: bash .openhands/agent_finish.sh, timeout: 600s).

Includes unit test coverage in tests/sdk/hooks/test_config.py.


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.12-nodejs22 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:e1c5b1d-python

Run

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

All tags pushed for this build

ghcr.io/openhands/agent-server:e1c5b1d-golang-amd64
ghcr.io/openhands/agent-server:e1c5b1d-golang_tag_1.21-bookworm-amd64
ghcr.io/openhands/agent-server:e1c5b1d-golang-arm64
ghcr.io/openhands/agent-server:e1c5b1d-golang_tag_1.21-bookworm-arm64
ghcr.io/openhands/agent-server:e1c5b1d-java-amd64
ghcr.io/openhands/agent-server:e1c5b1d-eclipse-temurin_tag_17-jdk-amd64
ghcr.io/openhands/agent-server:e1c5b1d-java-arm64
ghcr.io/openhands/agent-server:e1c5b1d-eclipse-temurin_tag_17-jdk-arm64
ghcr.io/openhands/agent-server:e1c5b1d-python-amd64
ghcr.io/openhands/agent-server:e1c5b1d-nikolaik_s_python-nodejs_tag_python3.12-nodejs22-amd64
ghcr.io/openhands/agent-server:e1c5b1d-python-arm64
ghcr.io/openhands/agent-server:e1c5b1d-nikolaik_s_python-nodejs_tag_python3.12-nodejs22-arm64
ghcr.io/openhands/agent-server:e1c5b1d-golang
ghcr.io/openhands/agent-server:e1c5b1d-java
ghcr.io/openhands/agent-server:e1c5b1d-python

About Multi-Architecture Support

  • Each variant tag (e.g., e1c5b1d-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., e1c5b1d-python-amd64) are also available if needed

Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

Coverage

Coverage Report •
FileStmtsMissCoverMissing
openhands-sdk/openhands/sdk/hooks
   config.py991584%52, 64–65, 71, 73, 116, 125, 127, 156, 196–197, 201–202, 204–205
TOTAL15121443670% 

@openhands-ai
Copy link

openhands-ai bot commented Jan 13, 2026

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • Pre-commit checks

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #1719 at branch `feature/agent-finish-sh-hook`

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

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.

3 participants