Skip to content

Update agent settings guide for canonical class#502

Merged
neubig merged 1 commit into
mainfrom
update-agent-settings-removal-docs
May 14, 2026
Merged

Update agent settings guide for canonical class#502
neubig merged 1 commit into
mainfrom
update-agent-settings-removal-docs

Conversation

@neubig
Copy link
Copy Markdown
Contributor

@neubig neubig commented May 11, 2026

Summary

This prepares the docs for the SDK PR that removes the deprecated AgentSettings public alias.

Testing

  • Not run; documentation-only change.

This PR was created by an AI agent (OpenHands) on behalf of the user.

Co-authored-by: openhands <openhands@all-hands.dev>
@neubig neubig requested review from enyst and xingyaoww as code owners May 11, 2026 17:15
Copy link
Copy Markdown
Contributor

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

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

🟢 Good taste - Clean, complete refactoring.

[RISK ASSESSMENT]
⚠️ Risk: 🟢 LOW - Documentation-only change. All references to AgentSettings updated consistently to OpenHandsAgentSettings, imports correctly moved from openhands.sdk to openhands.sdk.settings. Verified no other files need updating.

VERDICT: ✅ Worth merging (coordinate with SDK PR #3208 timing)

KEY INSIGHT: Mechanical refactoring executed correctly—demonstrates good coordination between SDK deprecation removal and docs updates.

@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 11, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
all-hands-ai 🟢 Ready View Preview May 11, 2026, 5:19 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@enyst enyst removed their request for review May 11, 2026 18:54
@neubig neubig merged commit dd6c9e9 into main May 14, 2026
8 checks passed
@neubig neubig deleted the update-agent-settings-removal-docs branch May 14, 2026 12:43
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.

3 participants