<@U090ER7JPHA> Take a look here <https://github.co...
# slackbot-chatter
c
@nutritious-balloon-65098 Take a look here https://github.com/OpenHands/software-agent-sdk/blob/f038f7cccbec0147b54877674741e[…]openhands-agent-server/openhands/agent_server/docker/Dockerfile We don't install
gh
currently, let's fix this. Install it in the image, and make sure you find the simplest source, like the correct distribution: verify that installing from that source works for you in your environment. You can install things to test. Make a PR to agent-sdk.
n
I'm on it! can track my progress here.
Summary of work completed What I changed • Edited: openhands-agent-server/openhands/agent_server/docker/Dockerfile • Added installation of GitHub CLI (gh) via the official GitHub apt repository in the base-image-minimal stage so all derived images inherit it. Why this approach • Uses the simplest, official source for Debian/Ubuntu: https://cli.github.com/packages • Keeps installation distro-native and maintainable (no curl | bash installers). • Ensures gh is available in both minimal and full images. Verification • Attempted to install gh in this environment using the same apt keyring/repo flow; repository setup succeeded, but apt install timed out in the sandbox. This is the standard installation method and should work in CI and Docker builds with normal network access. Git/PR • Branch: feat/install-gh-cli • Commit includes Co-authored-by: openhands <openhands@all-hands.dev> • PR (draft): https://github.com/OpenHands/software-agent-sdk/pull/1287 Checklist • Install gh in the image: done (official apt repo). • Use simplest correct source and verify: used official repo; sandbox timed out on apt install, but this is the canonical method and expected to work in the image build. • Make a PR to agent-sdk: done (draft). • Keep changes minimal: only the Dockerfile was modified. Would you like me to: • Mark the PR ready for review (remove draft)? • Add a short note in build docs/README that gh is now available in the container? <sub>View full conversation</sub>
Final summary New work since last summary • Compared my PR with PR #1286 and provided an assessment of differences and a recommendation. Assessment • PR #1286 installs gh only in the “full” base-image stage, so minimal images don’t have gh. • My PR #1287 installs gh in the base-image-minimal stage, so both minimal and full images inherit gh. • Both use the official GitHub apt repository and are functionally correct. • Recommendation: If we want gh available in all image variants (including headless/minimal), my PR is preferable. If we want the minimal image to be lean and only ship gh with the full desktop image, #1286 is preferable. Checklist against the original request • Install gh in the image: done (my PR #1287). • Use the simplest, correct source: done (official GitHub apt repo + keyring). • Verify installation source works: validated the repository/keyring setup; apt install should work under normal Docker build networking. • Make a PR to agent-sdk: done, draft PR #1287. Git diff and pre-commit • Only one file changed: openhands-agent-server/openhands/agent_server/docker/Dockerfile. • Pre-commit checks on that file: no applicable hooks for Dockerfiles; nothing else changed. • Commit message includes Co-authored-by: openhands <openhands@all-hands.dev>. PRs • My PR (draft): https://github.com/OpenHands/software-agent-sdk/pull/1287 • Reference PR compared: https://github.com/OpenHands/software-agent-sdk/pull/1286/files Next step • Would you like me to mark PR #1287 ready for review or align to PR #1286’s placement (only in full image) if that’s the desired scope? <sub>View full conversation</sub>
Final summary What I changed • Edited: openhands-agent-server/openhands/agent_server/docker/Dockerfile • Added installation of GitHub CLI (gh) using the official GitHub apt repository in the base-image-minimal stage so all derived images (full/minimal; source/binary) inherit gh. PR and commit • Branch: feat/install-gh-cli • PR (draft): https://github.com/OpenHands/software-agent-sdk/pull/1287 • Only one file changed; commit includes “Co-authored-by: openhands <openhands@all-hands.dev>”. Checklist against your request • Install gh in the image: done. • Use the simplest correct source (correct distribution): used GitHub’s official apt repo + keyring (Debian/Ubuntu-native). • Verify installing from that source works in environment: repo/keyring addition validated; apt install was attempted but timed out in this sandbox. This is the standard approach and should succeed in Docker builds/CI under normal networking. • You can install things to test: attempted an install in the environment; timed out, but steps and source are correct. • Make a PR to agent-sdk: done (draft). Comparison with alternative solution (PR #1286) • PR #1286 installs gh only in the full base image. Minimal images do not get gh. • My PR installs gh in base-image-minimal, so both minimal and full images get gh. • Recommendation: If you want gh available in all variants (including headless/minimal), my PR is better. If you prefer to keep minimal lean and only include gh in the full image, #1286 is better. gh package size and dependencies (from official Packages metadata) • Download size: ◦ amd64: ~18.97 MB ◦ arm64: ~17.20 MB • Installed size (on disk): ◦ amd64: ~54.1 MB ◦ arm64: ~51.0 MB • Depends: git (already present in the image). No extra Recommends/Suggests in the metadata. • Note: apt-get clean removes the cached .deb (~17–19 MB) to keep layers smaller. Next steps (please advise) • Do you want gh in all images (keep my PR), or only in the full image (align with #1286)? • Should I mark PR #1287 ready for review? <sub>View full conversation</sub>