Skip to content

fix(tool_runner): return error ToolMessage for unknown tool names - #853

Closed
Bartok9 wants to merge 1 commit into
RobotecAI:mainfrom
Bartok9:aerial/tool-runner-unknown-tool
Closed

Bartok9 wants to merge 1 commit into
RobotecAI:mainfrom
Bartok9:aerial/tool-runner-unknown-tool

Conversation

@Bartok9

@Bartok9 Bartok9 commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Unknown tool_calls raised KeyError and broke the agent loop. Map missing names to a status=error ToolMessage like other tool failures.

AI-assisted; human-reviewed.

Claim: bartok
Operator: bartok
Campaign: aerial-drone

@maciejmajek

Copy link
Copy Markdown
Member

In #872. Worth saying it wasn't actually crashing, the blanket except caught the KeyError, but the message is a lot clearer now.

@maciejmajek maciejmajek closed this Sep 7, 2026
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