Skip to content

fix maxTokens schema#6

Merged
j4ys0n merged 1 commit into
mainfrom
tools-fix
Mar 14, 2026
Merged

fix maxTokens schema#6
j4ys0n merged 1 commit into
mainfrom
tools-fix

Conversation

@j4ys0n
Copy link
Copy Markdown
Contributor

@j4ys0n j4ys0n commented Mar 14, 2026

No description provided.

@j4ys0n
Copy link
Copy Markdown
Contributor Author

j4ys0n commented Mar 14, 2026

Automated review 🤖

Summary of Changes
Bumps package version from 0.2.3 to 0.2.4 and updates descriptions for temperature and maxTokens fields in UpdateAgentSchema to clarify their behavior and storage location (modelOptions). The maxTokens description now explicitly states that -1 or omission means unlimited/model default, resolving ambiguity in prior documentation.

Key Changes & Positives

  • Clarified maxTokens behavior: explicitly defines -1 or omission as “unlimited/model default” (src/schemas.ts, line 135) 🟢
  • Improved temperature description to specify usage of -1 or omission for model defaults (src/schemas.ts, line 134) 🟢

Language/Framework Checks

  • Schema descriptions updated to align with zod best practices for clarity and runtime documentation (src/schemas.ts, lines 134–135).
  • No type changes—only .describe() text modifications, preserving backward compatibility.

Approval Recommendation
Approve

@j4ys0n j4ys0n merged commit d072f19 into main Mar 14, 2026
1 check passed
@j4ys0n j4ys0n deleted the tools-fix branch March 14, 2026 05: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.

1 participant