Add sanitization to auto-generated output tool name

From #2929

We ensure that auto-generated tool output names are properly sanitized
so they conform to `[a-zA-Z0-9-_]`. All characters not in this pattern
will simply be ignored/skipped in the final name.

The error was first remarked when using generic classes as `output_type`,
where brackets in the tool name would be rejected by the provider API.

Added snapshot test to verify the sanitized tool names are generated
correctly from generic types like Result[StringData] and Result[int].

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: lionpeloux

tests/test_agent.py

@@ -665,13 +665,8 @@ class StringData(BaseModel):
     assert m.last_model_request_parameters.output_tools is not None
     assert len(m.last_model_request_parameters.output_tools) == 2
 
-    # Check that tool names don't contain brackets
     tool_names = [tool.name for tool in m.last_model_request_parameters.output_tools]
-    for tool_name in tool_names:
-        assert '[' not in tool_name, f"Tool name '{tool_name}' contains brackets"
-        assert ']' not in tool_name, f"Tool name '{tool_name}' contains brackets"
-        # Verify the name follows the pattern [a-zA-Z0-9_-]
-        assert re.match(r'^[a-zA-Z0-9_-]+$', tool_name), f"Tool name '{tool_name}' contains invalid characters"
+    assert tool_names == snapshot(['final_result_ResultStringData', 'final_result_Resultint'])
 
 
 def test_output_type_with_two_descriptions():