docs: add OpenAI-compatible agent-server gateway guide#554
Conversation
Co-authored-by: openhands <openhands@all-hands.dev>
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
Co-authored-by: openhands <openhands@all-hands.dev>
|
@OpenHands Clone yourself agent-sdk from the same org. Understand the workflow there that verifies companion docs PRs for examples/ files. Check the PR to which this PR is companion and make sure we update this PR if in the meantime, the other PR changed in. a way that affects this. If you think this PR is in good shape, mark it open for review and /iterate . |
|
I'm on it! enyst can track my progress at all-hands.dev |
Co-authored-by: openhands <openhands@all-hands.dev>
|
/iterate Companion SDK PR #3545 was checked against this docs branch. The new docs code block matches This comment was created by an AI agent (OpenHands) on behalf of the user. |
This comment was marked as duplicate.
This comment was marked as duplicate.
|
✅ Review complete. This review was performed through OpenHands Cloud Automation. You can log in and view the conversation here. |
all-hands-bot
left a comment
There was a problem hiding this comment.
Code Review: PR #554
🟢 Good taste - Well-structured documentation
Summary
This PR adds documentation for the OpenAI-compatible agent-server gateway feature. The documentation is:
- Clear and well-organized: The "How it works" section breaks down the protocol into digestible steps
- Accurate: Correctly describes the model naming pattern (
openhands_<profile_name>) and authentication headers - Complete: Includes the important caveat about streaming not being supported
- Educational: The code example covers the full lifecycle (profile creation → chat → conversation continuation → cleanup)
No Issues Found
- [docs.json] Navigation entry is correctly placed alphabetically
- [openai-gateway.mdx] Frontmatter is properly structured
- [openai-gateway.mdx] Example links to actual source code for long-term maintainability
- [openai-gateway.mdx] Resource cleanup in example follows best practices
Minor Note
The streaming limitation note is important to keep. Make sure it's updated if/when streaming support is added.
[RISK ASSESSMENT]
- [Overall PR]
⚠️ Risk Assessment: 🟢 LOW
Pure documentation addition with no code changes. No breaking changes, no security concerns, no dependency changes.
VERDICT:
✅ Worth merging: Documentation is accurate, well-structured, and provides good coverage of the feature.
KEY INSIGHT:
The documentation correctly emphasizes the protocol flow and correctly calls out the streaming limitation as a known constraint.
This review was generated by an AI agent (OpenHands) on behalf of the user through OpenHands Automation. View conversation
Summary of changes
/v1/chat/completionsgateway.openhands_<profile_name>), compatible auth headers, and conversation ID reuse viaX-OpenHands-ServerConversation-ID.examples/02_remote_agent_server/14_openai_compatible_gateway.pycode block so the agent-sdk documented-examples check passes.Related SDK PR: OpenHands/software-agent-sdk#3545
This PR was created by an AI agent (OpenHands) on behalf of the user.
@enyst can click here to continue refining the PR