fix(core): pass tool_call_id to on_tool_start and on_tool_end callbacks

[thromel]

Summary

• Pass tool_call_id to on_tool_start and on_tool_end callback handlers in BaseTool.run() and BaseTool.arun()
• Add regression tests for both sync and async tool invocation

Why

When invoking a tool with a ToolCall input (which contains an id field), the tool_call_id was correctly extracted but never forwarded to callback handlers. This made it impossible for callbacks to correlate tool execution with the original LLM tool call.

Test plan

• Added test_tool_call_id_passed_to_callbacks for sync invocation
• Added test_tool_call_id_passed_to_async_callbacks for async invocation
• All 148 existing tool tests pass

Fixes #34168

[codspeed-hq]

CodSpeed Performance Report

Merging #34201 will not alter performance

_{Comparing thromel:fix/tool-call-id-callbacks (d6106e3) with master (1867521)¹}

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

✅ 13 untouched
⏩ 21 skipped²

Footnotes

1. No successful run was found on master (b95cb77) during the generation of this report, so 1867521 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

2. 21 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩