SDK Docs for local "guides"

[xingyaoww]

• I have read and reviewed the documentation changes to the best of my ability.
• If the change is significant, I have run the documentation site locally and confirmed it renders as expected.

Summary of changes

Add SDK documentation for local conversation part of "sdk/guides".

Will need to merge OpenHands/software-agent-sdk#859 first to update SDK examples before merging this PR.

[neubig]

@OpenHands Do a code review of this PR.

[openhands-ai]

I'm on it! neubig can track my progress at all-hands.dev

[openhands-ai]

I've completed a comprehensive code review of PR #41 "SDK Docs for local 'guides'" in the OpenHands/docs repository.

Review Summary

This PR adds 20 new SDK documentation files covering conversation features, LLM features, agent features, and security. I analyzed all files for:

• Documentation structure and formatting
• Code block syntax compliance with SDK guidelines
• GitHub link accuracy and consistency
• Navigation configuration in docs.json
• Grammar, typos, and style issues
• Internal link integrity

Findings

🟢 Minor Issues (2 total)

1. Grammar error in security.mdx (line 10)

   • "Confirmation policy control" should be "Confirmation policies control" (subject-verb agreement)

2. Capitalization inconsistency in mcp.mdx (lines 106, 219)

   • Two bash blocks use "Running the example" instead of "Running the Example"

✅ What's Working Well

• All 20 files have proper frontmatter with titles and descriptions
• Code block syntax correctly uses icon="python" and expandable attributes
• All synced code blocks have corresponding Note components with GitHub links
• Navigation structure properly organized with logical groupings
• All files correctly listed in docs.json
• No broken internal links detected
• No typos found
• Consistent GitHub repository references
• OpenAPI updates appear legitimate

Conclusion

This is a high-quality documentation PR that follows the established SDK documentation guidelines. The only issues found are very minor formatting/grammar items that don't affect functionality. The documentation is comprehensive, well-structured, and ready for use once the dependent agent-sdk PR #859 is merged (as noted in the PR description).

No changes were made to the code as this was a review-only request. The minor issues identified above can be addressed by the PR author if desired, but they are not blocking.

_{View full conversation}

[xingyaoww]

@enyst here's where we will add the API reference for the SDK - this feels more in-place to sit near the description of agent-server