You've already forked adk-python
mirror of
https://github.com/encounter/adk-python.git
synced 2026-03-30 10:57:20 -07:00
b8a2b6c570
Previously we only do a simple prefix string matching, thus `agent_00` will match with `agent_0` With this new change, we either check directly equality, or must expect seeing `agent_0.`. See added test for branches we now match / skip. TBF `.` is also not a perfect delimiter (I would imagine users might put dot in agent names). We might consider a follow up that bans such agent names. Tested with script in the linked issue (I updated prompt so we see which agent they see from): Before: ``` [agent_8]: 73 [agent_0]: 97 [agent_1]: 73 [agent_5]: 97 [agent_4]: 73 [agent_2]: 73 [agent_3]: 73 [agent_9]: 93 [agent_6]: 73 [agent_7]: 1 [agent_70]: 1 (agent_7) [agent_20]: 73 (agent_2) [agent_30]: 73 (agent_3) [agent_00]: 97 (agent_0) [agent_40]: 73 (agent_4) [agent_80]: 73 (agent_8) [agent_50]: 97 (agent_5) [agent_90]: 93 (agent_9) [agent_10]: 73 (agent_1) [agent_60]: 73 (agent_6) ``` After: ``` [agent_9]: 73 [agent_6]: 73 [agent_2]: 73 [agent_7]: 93 [agent_4]: 73 [agent_1]: 73 [agent_3]: 73 [agent_5]: 97 [agent_0]: 73 [agent_8]: 87 [agent_50]: 0 [agent_80]: 0 [agent_10]: 0 [agent_90]: 0 [agent_30]: 0 [agent_20]: 0 [agent_60]: 0 [agent_00]: 0 [agent_40]: 0 [agent_70]: 0 ``` Closes #2948 Co-authored-by: Kevin Qian <kqian@google.com> PiperOrigin-RevId: 826187198