Skip to content

Add a wrapper handler for our tools#989

Merged
dgageot merged 10 commits intodocker:mainfrom
rumpl:cleanup-tools
Nov 29, 2025
Merged

Add a wrapper handler for our tools#989
dgageot merged 10 commits intodocker:mainfrom
rumpl:cleanup-tools

Conversation

@rumpl
Copy link
Copy Markdown
Member

@rumpl rumpl commented Nov 29, 2025

Add a generic tool call handler wrapper that parses the tool call arguments and passes them down to the toolset handler. This cleans up the code and the tests quite a bit.

rumpl added 10 commits November 29, 2025 16:09
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
@rumpl rumpl marked this pull request as ready for review November 29, 2025 15:45
@rumpl rumpl requested a review from a team as a code owner November 29, 2025 15:45
@dgageot dgageot merged commit c6b9a58 into docker:main Nov 29, 2025
5 checks passed
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