Skip to content

Conversation

@enyst
Copy link
Collaborator

@enyst enyst commented Nov 7, 2025

Summary

Add a dedicated guide for provider-agnostic SDK exceptions and client error-handling patterns. This documents the new typed exceptions introduced in OpenHands/software-agent-sdk PR #980 and shows practical examples for both direct LLM usage and higher-level agent/conversation flows.

What’s in this PR

  • New guide: sdk/guides/error-handling.mdx
  • Navigation: add under SDK → Guides → LLM Features
  • Cross-links to Context Condenser to avoid context-window interruptions

Why
SDK PR #980 normalizes common provider errors (auth, rate limit, timeout, service unavailable, bad request, context-window exceeded) into stable types. Client developers need a clear reference and examples to handle these consistently across providers.

Related

Co-authored-by: openhands [email protected]

@enyst can click here to continue refining the PR

- New guide: sdk/guides/error-handling.mdx
- Navigation: add under SDK > Guides > LLM Features

Refs OpenHands/software-agent-sdk#980

Co-authored-by: openhands <[email protected]>
- Show try/except around LLM.responses()

Refs OpenHands/software-agent-sdk#980

Co-authored-by: openhands <[email protected]>
- Rename file to align with other LLM features
- Update navigation to point at new path

Refs OpenHands/software-agent-sdk#980

Co-authored-by: openhands <[email protected]>
- Lead with Agent/Conversation examples and bubbling behavior
- Move LLM examples (completion/responses) into secondary section

Refs OpenHands/software-agent-sdk#980

Co-authored-by: openhands <[email protected]>
…uide

- Content is redundant and covered elsewhere

Refs OpenHands/software-agent-sdk#980

Co-authored-by: openhands <[email protected]>
Copy link
Contributor

@malhotra5 malhotra5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Collaborator Author

enyst commented Nov 7, 2025

Restructured guide to be conversation-first, kept LLM examples as secondary, and removed the final 'Notes for advanced users' section as requested. Ready for re-review.

Refs: OpenHands/software-agent-sdk#980

@enyst enyst merged commit b7b84f7 into main Nov 7, 2025
2 checks passed
@enyst enyst deleted the sdk/error-normalization-context-auth branch November 7, 2025 19:16
enyst pushed a commit that referenced this pull request Nov 8, 2025
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.

4 participants