chore: Code tidy for Anthropic and Google Gemini client classes (#2311)

* Code tidy for Anthropic and Google Gemini

* Fix test bug

Author: marklysze

autogen/oai/anthropic.py

@@ -186,10 +186,7 @@ def supports_native_structured_outputs(model: str) -> bool:
         return True
 
     # Support future Opus 4.x versions
-    if model.startswith("claude-opus-4"):
-        return True
-
-    return False
+    return bool(model.startswith("claude-opus-4"))
 
 
 def has_beta_messages_api() -> bool:
@@ -265,9 +262,7 @@ def _is_text_block(content: Any) -> bool:
     """
     if type(content) == TextBlock:
         return True
-    if BETA_BLOCKS_AVAILABLE and type(content) == BetaTextBlock:
-        return True
-    return False
+    return BETA_BLOCKS_AVAILABLE and type(content) == BetaTextBlock
 
 
 def _is_tool_use_block(content: Any) -> bool:
@@ -288,10 +283,7 @@ def _is_tool_use_block(content: Any) -> bool:
         return True
 
     # Fallback: check by name if type comparison fails
-    if content_type_name in ("ToolUseBlock", "BetaToolUseBlock"):
-        return True
-
-    return False
+    return content_type_name in ("ToolUseBlock", "BetaToolUseBlock")
 
 
 def _is_thinking_block(content: Any) -> bool:
@@ -310,10 +302,7 @@ def _is_thinking_block(content: Any) -> bool:
         return True
 
     # Fallback: check by name if type comparison fails
-    if content_type_name == "ThinkingBlock":
-        return True
-
-    return False
+    return content_type_name == "ThinkingBlock"
 
 
 def transform_schema_for_anthropic(schema: dict[str, Any]) -> dict[str, Any]:

test/agentchat/test_multimodal_integration.py

@@ -753,7 +753,7 @@ def test_run_group_chat_multimodal(credentials_gpt_4o_mini: Credentials) -> None
 
     # Test that response objects are properly configured (if agents attribute exists)
     if hasattr(response2, "agents"):
-        assert response2.agents == [], "Initial agents list should be empty (gets populated later)"
+        assert len(response2.agents) > 0, "Agents list should be populated"
 
     # Wait for processing
     time.sleep(2)

test/oai/test_anthropic.py

@@ -1609,10 +1609,7 @@ class MathResult(BaseModel):
     content = final_response.choices[0].message.content
 
     # Parse and validate structured output
-    if isinstance(content, str):
-        result = MathResult.model_validate_json(content)
-    else:
-        result = MathResult.model_validate(content)
+    result = MathResult.model_validate_json(content) if isinstance(content, str) else MathResult.model_validate(content)
 
     # Verify structured output has required fields
     assert result.steps, "Should have steps"

test/oai/test_gemini.py

@@ -1228,7 +1228,7 @@ def test_thought_signature_retained_across_calls(self, mock_calculate_cost, mock
         ]
 
         # Second create call - thought_signature should still be available
-        response2 = gemini_client.create({
+        gemini_client.create({
             "model": "gemini-3-flash",
             "messages": messages,
             "tools": [