Skip to content

Enhance ManagedServer and ServerManager to support tool call parsing.…#366

Open
teknium1 wants to merge 3 commits intomainfrom
tool_call_support
Open

Enhance ManagedServer and ServerManager to support tool call parsing.…#366
teknium1 wants to merge 3 commits intomainfrom
tool_call_support

Conversation

@teknium1
Copy link
Collaborator

@teknium1 teknium1 commented Feb 7, 2026

Added optional tool_call_parser parameter to ManagedServer and managed_server context manager, enabling structured extraction of tool calls from raw completion text. Updated _convert_messages_to_prompt to forward additional template kwargs for improved prompt construction.

teknium and others added 2 commits February 7, 2026 09:12
… Added optional tool_call_parser parameter to ManagedServer and managed_server context manager, enabling structured extraction of tool calls from raw completion text. Updated _convert_messages_to_prompt to forward additional template kwargs for improved prompt construction.
…s for downstream processing. Removed reasoning content extraction and associated attributes to streamline choice construction.
@teknium1 teknium1 requested a review from dmahan93 February 19, 2026 17:59
Copy link
Collaborator

@dmahan93 dmahan93 left a comment

Choose a reason for hiding this comment

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

@teknium1 i'll approve it, but do you have tests you can run to ensure it works?

@dmahan93
Copy link
Collaborator

would be good for future updates to ensure we don't break it in the future 🙂

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