Skip to content

Fix OpenHands skill loading with current SDK#53

Open
Rohithmatham12 wants to merge 1 commit into
sentient-agi:mainfrom
Rohithmatham12:fix-openhands-skill-loader
Open

Fix OpenHands skill loading with current SDK#53
Rohithmatham12 wants to merge 1 commit into
sentient-agi:mainfrom
Rohithmatham12:fix-openhands-skill-loader

Conversation

@Rohithmatham12

@Rohithmatham12 Rohithmatham12 commented Jul 18, 2026

Copy link
Copy Markdown

Summary

  • load OpenHands skills from the current openhands.sdk.skills module
  • keep a fallback to openhands.sdk.context.skills for older SDKs
  • add regression coverage for the current SDK module path so evoskill run / evoskill eval no longer fail before agent execution

Fixes #44.

Testing

  • .venv/bin/python -m pytest tests/test_openhands_runtime.py -q
  • .venv/bin/python -m pytest tests/test_openhands_runtime.py tests/test_openhands_auth.py tests/test_openhands_contract.py -q
  • git diff --check

Signed-off-by: Rohithmatham12 <rohithmatham@gmail.com>
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.

openhands harness: 'module openhands.sdk.context.skills has no attribute load_skills_from_dir' (breaks with openhands-sdk 1.24.0)

1 participant