simplify if condition

Jack Yuan · Jack Yuan · commit 2ddefa2a2d68 · 2025-08-29T16:15:52.000-04:00
diff --git a/src/strands/tools/registry.py b/src/strands/tools/registry.py
@@ -191,8 +191,7 @@ def register_tool(self, tool: AgentTool) -> None:
         )
 
         # Check duplicate tool name, throw on duplicate tool names except if hot_reloading is enabled
-        if tool.tool_name in self.registry:
-            if not tool.supports_hot_reload:
+        if tool.tool_name in self.registry and not tool.supports_hot_reload:
                 raise ValueError(
                     f"Tool name '{tool.tool_name}' already exists. Cannot register tools with exact same name."
                 )

Original file line number	Diff line number	Diff line change
`@@ -191,8 +191,7 @@ def register_tool(self, tool: AgentTool) -> None:`
`191`	`191`	`)`
`192`	`192`
`193`	`193`	`# Check duplicate tool name, throw on duplicate tool names except if hot_reloading is enabled`
`194`		`- if tool.tool_name in self.registry:`
`195`		`- if not tool.supports_hot_reload:`
	`194`	`+ if tool.tool_name in self.registry and not tool.supports_hot_reload:`
`196`	`195`	`raise ValueError(`
`197`	`196`	`f"Tool name '{tool.tool_name}' already exists. Cannot register tools with exact same name."`
`198`	`197`	`)`