Greptile Summary

Fixes intermittent "Tool not found" errors by trimming whitespace from LLM-returned tool names before lookup. Some models return tool names like " read " or " exec" with leading/trailing whitespace, which causes exact-match failures in pi-agent-core's tools.find(t => t.name === toolCall.name).

Key changes:

• src/agents/pi-embedded-runner/run/attempt.ts: Added wrapStreamFnTrimToolNames() stream wrapper that intercepts tool call events and trims whitespace from tool names in-place
• Applied wrapper at line 881 (after anthropic payload logger wrapper if present)
• src/agents/pi-tool-definition-adapter.ts: Cosmetic refactor extracting map result to variable (likely for debugging/future enhancement)

Implementation notes:

• Wrapper correctly implements async iterator protocol with next(), return(), and throw() methods
• Mutates tool call name blocks in-place; partial message objects are same references used by final result, so trimming propagates automatically
• Only trims when trimmed !== original (zero overhead for well-formed names)
• Follows same pattern as existing stream wrappers (anthropic-payload-log, cache-trace)

Safety:

• Fully backward-compatible
• Type guards prevent errors on unexpected block structures
• Minimal performance overhead

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• Score reflects targeted bug fix with minimal code surface area, full backward compatibility, established code patterns, passing tests, and no security implications. The fix directly addresses a real issue ([Bug]: Tool Names Not Trimmed, Causing "Tool Not Found" Errors #27045) with a simple, well-documented solution.
• No files require special attention

_{Last reviewed commit: 5bc6e02}

Superseded by #27094 (merged on main).

Thanks @Sid-Qin for the excellent fix direction here.

We incorporated this live stream-wrapper approach into #27094 and merged the combined solution to main in 6b317b1.

Closing as superseded by #27094.