-
Notifications
You must be signed in to change notification settings - Fork 6
Add guide: app-server + custom agent integration #194
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
… integrating custom agents Co-authored-by: openhands <openhands@all-hands.dev>
Co-authored-by: openhands <openhands@all-hands.dev>
|
I really don't know how to review this. Can you add someone that you think would be better to review this please? |
|
@OpenHands This is a markdown doc!
|
|
I'm on it! enyst can track my progress at all-hands.dev |
… readability\n\n- Add openhands/usage/developers/v1-local-agent-server to Web tab under Developers\n- Improve headings, styled notes, and code block labels in the guide\n\nCo-authored-by: openhands <openhands@all-hands.dev>
|
Summary of changes pushed to branch docs/v1-local-agent-server-guide (updates PR #194) What I changed
Checklist against request
Conciseness
Commit
Notes
|
…de> expressions in inline literals\n\n- Switch to backticked inline code for {agent_server_url} examples\n\nCo-authored-by: openhands <openhands@all-hands.dev>
…ration and remove new Developers group\n\n- Place openhands/usage/developers/v1-local-agent-server at end of Advanced Configuration pages\n\nCo-authored-by: openhands <openhands@all-hands.dev>
|
Looks like there are a few issues preventing this PR from being merged!
If you'd like me to help, just leave a comment, like Feel free to include any additional details that might help me get this PR into a better state. You can manage your notification settings |
|
@OpenHands read this PR. It's actually attempting to talk about 2 different things. Separate them:
Split them. Look around and decide if any of them is redundant, then drop it. If both are useful, figure out where they fit, maybe the second belongs in the SDK, and fix the json for them. When you are done, push and wait for CI, see if it's green. We have the habit to break mintlify :P |
|
I'm on it! enyst can track my progress at all-hands.dev |
…1-local-agent-server.mdx to fully local app-server + local SDK agent-server flow\n- Add new SDK guide: sdk/guides/agent-server/app-integration (how GUI composes/loads SDK Agents)\n- Wire new page into Mintlify nav (docs.json)\n\nCo-authored-by: openhands <openhands@all-hands.dev>
This PR adds a new guide that explains how to:
The guide is grounded in source code with inline links to the relevant modules and functions.
Key references include:
python -m openhands.agent_serverAsyncRemoteWorkspaceused by the app-server to talk to the agent-serverStartConversationRequestin the V1 serviceCo-authored-by: openhands openhands@all-hands.dev
@enyst can click here to continue refining the PR