Skip to content

Align test automation SDK version with service#190

Open
enyst wants to merge 1 commit into
mainfrom
cleanup-openhands-provider-boundary
Open

Align test automation SDK version with service#190
enyst wants to merge 1 commit into
mainfrom
cleanup-openhands-provider-boundary

Conversation

@enyst

@enyst enyst commented Jun 12, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove a stale tracked E2E log artifact that showed pre-OpenHands/software-agent-sdk#3548 client-visible litellm_proxy/* behavior.
  • Update scripts/test_tarball/setup.sh to fetch the SDK version from ${AUTOMATION_API_URL}/sdk-version instead of pinning openhands-sdk==1.22.0.
  • Pass AUTOMATION_API_URL from scripts/test_automation.py so the test tarball uses the same dynamic SDK-version path as production presets.
  • Add regression coverage to prevent the test tarball from reintroducing a hardcoded stale SDK pin.

Validation

  • uv run ruff format scripts/test_automation.py tests/test_test_tarball_setup.py
  • uv run pytest tests/test_test_tarball_setup.py
  • uv run ruff check scripts/test_automation.py tests/test_test_tarball_setup.py
  • uv run python -m py_compile scripts/test_automation.py tests/test_test_tarball_setup.py
  • Live-like scripts/test_tarball/setup.sh run against a local /sdk-version HTTP endpoint with uv stubbed: confirmed it fetched 1.28.1 and used that version for openhands-sdk, openhands-tools, and openhands-workspace installs.
  • Live-like SDK check with openhands-sdk==1.28.1 and the app LiteLLM proxy key: confirmed SDK stores model='openhands/minimax-m2.7', base_url=None, and rewrites only at the LiteLLM call boundary to litellm_proxy/minimax-m2.7 with https://llm-proxy.app.all-hands.dev.

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

@enyst can click here to continue refining the PR

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

Copy link
Copy Markdown

Coverage

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.

1 participant