Skip to content

Conversation

@cpsievert
Copy link
Collaborator

@cpsievert cpsievert commented Sep 8, 2025

Before this PR, this code would produce a (pydantic) runtime error (now it works)

import chatlas as ctl

chat = ctl.ChatOpenAI()

def add(_x: int, _y: int) -> int:
    return _x + _y

chat.register_tool(add)

@cpsievert cpsievert requested a review from Copilot September 8, 2025 16:11

This comment was marked as resolved.

@cpsievert cpsievert changed the title feat: tool function parameters may now include a leading underscore feat: tool function parameters can now include a leading underscore Sep 8, 2025
@cpsievert
Copy link
Collaborator Author

Errors are due to temporary overloaded server errors -- merging

@cpsievert cpsievert merged commit 53ef636 into main Sep 8, 2025
6 of 12 checks passed
@cpsievert cpsievert deleted the feat/tool-param-leading-underscore branch November 6, 2025 15:33
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.

2 participants