-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Open
Labels
P3Nice to haves, rare edge casesNice to haves, rare edge casesenhancementRequest for a new feature that's not currently supportedRequest for a new feature that's not currently supportedready for workEnough information for someone to start working onEnough information for someone to start working onv2Ideas, requests and plans for v2 of the SDK which will incorporate major changes and fixesIdeas, requests and plans for v2 of the SDK which will incorporate major changes and fixes
Milestone
Description
Describe the bug
The input schema property is not following the snake case convention. Currently the property is defined as "inputSchema" instead of "input_schema". Ideally we can follow the convention as it is widely adopted. Additionally, not following the convention requires more lines of code on the client when passing the available tools to the API request.
Line 660 in aaf32b5
| inputSchema: dict[str, Any] |
Expected behavior
inputSchema get renamed to input_schema
Screenshots
N/A
soohoonc
Metadata
Metadata
Assignees
Labels
P3Nice to haves, rare edge casesNice to haves, rare edge casesenhancementRequest for a new feature that's not currently supportedRequest for a new feature that's not currently supportedready for workEnough information for someone to start working onEnough information for someone to start working onv2Ideas, requests and plans for v2 of the SDK which will incorporate major changes and fixesIdeas, requests and plans for v2 of the SDK which will incorporate major changes and fixes