Skip to content

docs(sdk): Remove max_turns from TaskToolSet docs#418

Merged
xingyaoww merged 1 commit intomainfrom
vasco/max_turns
Mar 26, 2026
Merged

docs(sdk): Remove max_turns from TaskToolSet docs#418
xingyaoww merged 1 commit intomainfrom
vasco/max_turns

Conversation

@VascoSch92
Copy link
Copy Markdown
Contributor

@VascoSch92 VascoSch92 commented Mar 25, 2026

  • 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
Remove the max_turns parameter from the TaskToolSet tool parameters table, matching the SDK change in OpenHands/software-agent-sdk#2491

@mintlify
Copy link
Copy Markdown

mintlify bot commented Mar 25, 2026

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

Project Status Preview Updated (UTC)
all-hands-ai 🟢 Ready View Preview Mar 25, 2026, 10:27 PM

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 documentation fix.

Verified against SDK source: The max_turns parameter is marked SkipJsonSchema (excluded from user-facing schema), deprecated, and ignored in the SDK (see definition.py:54-61). Removing it from docs matches the implementation.

Verdict: ✅ Worth merging

@xingyaoww xingyaoww merged commit 66ab2db into main Mar 26, 2026
8 checks passed
@xingyaoww xingyaoww deleted the vasco/max_turns branch March 26, 2026 13:49
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