Skip to content

Improve parameter descriptions in sequential-thinking for better LLM type safety#2611

Merged
olaservo merged 1 commit intomainfrom
claude/issue-2598-20250824-0325
Aug 26, 2025
Merged

Improve parameter descriptions in sequential-thinking for better LLM type safety#2611
olaservo merged 1 commit intomainfrom
claude/issue-2598-20250824-0325

Conversation

@olaservo
Copy link
Member

Update thoughtNumber and totalThoughts parameter descriptions to use positive-only examples that guide LLMs toward correct numeric type usage. This addresses the issue where LLMs sometimes generate string values (e.g. "/1") instead of numeric values (e.g. 1) for integer parameters.

Fixes #2598

Generated with Claude Code

…type safety

Update thoughtNumber and totalThoughts parameter descriptions to use positive-only examples that guide LLMs toward correct numeric type usage. This addresses the issue where LLMs sometimes generate string values (e.g. "1") instead of numeric values (e.g. 1) for integer parameters.

Co-authored-by: Ola Hungerford <olaservo@users.noreply.github.com>
Copy link
Member

@domdomegg domdomegg left a comment

Choose a reason for hiding this comment

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

lgtm

@olaservo olaservo merged commit e968c6c into main Aug 26, 2025
23 checks passed
@olaservo olaservo deleted the claude/issue-2598-20250824-0325 branch August 26, 2025 15:04
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.

Improve parameter descriptions in sequential-thinking for better LLM type safety

2 participants