[BUG] Tools loaded by passing a file path only load the last defined tool in file

[jmorgan-restrata]

Checks

• I have updated to the lastest minor and patch version of Strands
• I have checked the documentation and this is not expected behavior
• I have searched ./issues and there are no duplicates of my issue

Strands Version

1.3.0

Python Version

3.11.13

Operating System

Linux

Installation Method

pip

Steps to Reproduce

Create a file called multiple_tools.py with multiple unique @tool definitions

Pass tool as file path to Agent:

agent = Agent(
    tools=['/tools/multiple_tools.py']
)

Expected Behavior

The agent should have access to all the tools defined in multiple_tools.py

Actual Behavior

The agent only has access to the last defined @tool in the file - the rest are ignored.

Additional Context

If you create a file called multiple_tools.py with multiple @tool definitions and use load_tools_from_directory=True

agent = Agent(load_tools_from_directory=True)

All tools defined by @tool are loaded into the agent.

If you instead pass the file path only the last tool is loaded.

Possible Solution

N/A

Related Issues

#508

[zastrowm]

I think this is because it's using this code path when passing a string as a tool:

sdk-python/src/strands/tools/loader.py

Line 93 in ebcd4c0

return cast(AgentTool, attr)

This in contrast to if you pass it in as a module:

sdk-python/src/strands/tools/registry.py

Line 88 in e597e07

function_tools = self._scan_module_for_tools(tool)

So the workaround until this is fixed is to load the tool(s) via module instead:

import multiple_tools from .tools

agent = Agent(
    tools=[multiple_tools]
)

[Ratish1]

I'd like to contribute to this.

[zastrowm]

Implemented via #742; thanks for the help @Ratish1

[Ratish1]

Thank you for helping me @zastrowm