Fix base_url for OpenHands provider which was broken by an upstream SDK PR (@quick-window-91705 we might need a patch release for this, this breaks new user setup flow 😞 ) https://github.com/OpenHands/OpenHands-CLI/pull/168

Thanks! Release PR: https://github.com/OpenHands/OpenHands-CLI/pull/171

Oops, but sorry, I'm not sure I understand, if it wasn't passing a

base_url

, didn't the SDK supply the

app

proxy URL?

👆 the issue is that CLI will save the settings to a local JSON and re-load it everytime The json will have

base_url=null

@calm-artist-7206 gonna hold off on release for a bit so you've had a chance to look through the change

ugh this reminds me of this bug https://github.com/OpenHands/software-agent-sdk/pull/1299#issuecomment-3607725829 😅 now i realized that is probably the PR that breaks this..

I'm on it! Xingyao Wang can track my progress here.

gonna hold off on release for a bit so you've had a chance to look through the change

Just to clarify, I think hardcoding it in the CLI should work, I'm just thinking that maybe we can also fix the condition in the SDK then...

Ah gotcha thanks Gonna proceed with patch release then

Summary of Investigation and Fix I've successfully investigated and fixed the

base_url

issue with the

openhands/

provider that broke the CLI's new user setup flow. Here's what I found and fixed: Root Cause Identified PR #1273 introduced a breaking change in

llm.py

(line 373): • Changed from:

d["base_url"] = "<https://llm-proxy.app.all-hands.dev/>"

• To:

d["base_url"] = d.get("base_url", "<https://llm-proxy.app.all-hands.dev/>")

The Problem: When CLI saves settings to JSON with

base_url=null

and reloads them, the dictionary contains

{"base_url": None}

. Python's

dict.get(key, default)

only returns the default when the key is missing, not when it's explicitly

None

. This caused LiteLLM to fail with a missing base_url error. The Workaround in 1.4.0 (commit 54d93bc1): Changed the test workflow to bypass the

openhands/

provider entirely by using

litellm_proxy/

directly with a hardcoded URL. Changes Made 1. Fixed

llm.py

(line 373): Changed to

d.get("base_url") or "<https://llm-proxy.app.all-hands.dev/>"

to handle explicit

None

values correctly 2. Reverted workflow workaround: Changed

.github/workflows/run-examples.yml

back to using

openhands/claude-haiku-4-5-20251001

with

${{ secrets.LLM_BASE_URL }}

3. Added comprehensive tests (

test_llm.py

): ◦ Test with explicit

None

base_url (reproduces CLI bug) ◦ Test with custom base_url (ensures override still works) Verification • ✅ All 467 LLM tests pass • ✅ Manual testing confirms all scenarios work correctly • ✅ Pre-commit hooks pass • ✅ Changes committed with descriptive message Would you like me to: 1. Push these changes to a new branch and create a pull request? 2. Keep the changes local for now? 3. Any other specific workflow you prefer? _{View full conversation}