Repository navigation
[TEST] Marketplace with SDK fix for single-skill plugin loading - #15175
Closed
jpshackelford wants to merge 5 commits into
Closed
jpshackelford wants to merge 5 commits into
jpshackelford wants to merge 5 commits into
Conversation
This temporary branch uses the SDK fix from OpenHands/software-agent-sdk#4025 to enable proper loading of single-skill plugins with root SKILL.md files. SDK commit: e0246aae36c95acdc10d37eb6cd8540c71ff5b2c
Contributor
Coverage report
This PR does not seem to contain any modification to coverable code. |
Poetry has issues installing git dependencies during Docker builds. Instead, we install the standard SDK version via Poetry, then override it with the SDK from PR #4025 using pip directly. This approach is more reliable in Docker build contexts.
Added SDK override from PR #4025 to enterprise Dockerfile as well. This ensures both the app and enterprise images use the SDK fix.
Completely removed the git source reference for openhands-sdk by deleting and regenerating poetry.lock. Now it properly references the PyPI package, and the Dockerfiles override it with the git version.
Member
Author
|
Closing this PR - we're using a simpler approach instead. Instead of modifying OpenHands code, we're just overriding AGENT_SERVER_IMAGE_TAG in the deploy configuration to use the SDK fix from PR #4025. See: https://github.com/OpenHands/deploy/pull/4751 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
This is a test PR to build Docker images for testing the marketplace feature with the SDK fix.
What This Includes
e0246aae36c95acdc10d37eb6cd8540c71ff5b2cWhy This PR Exists
The marketplace feature (merged in PR #14887) is being tested in production but fails because the SDK doesn't support single-skill plugins with root SKILL.md files. This test PR builds images with the SDK fix so we can deploy and test the complete feature.
Testing Plan
Next Steps
DO NOT MERGE THIS PR - This is only for building test images.
This PR was created by an AI agent (OpenHands) on behalf of the user.
To run this PR locally, use the following command:
GUI with Docker: