From 38fcf69dd5aa3df516872774906ba1452bade12a Mon Sep 17 00:00:00 2001
From: Abbas Asad <168946441+Abbas-Asad@users.noreply.github.com>
Date: Thu, 14 Aug 2025 04:19:04 +0500
Subject: [PATCH] Fix: Remove redundant and unreachable type check in response
 processing

## Summary

Fixed a bug where redundant and logically impossible code was present in the function tool call processing logic, which could never be executed.

## Problem

The code contained a redundant type check that was logically impossible to reach:

```python
elif not isinstance(output, ResponseFunctionToolCall):
    logger.warning(f"Unexpected output type, ignoring: {type(output)}")
    continue

# At this point we know it's a function tool call
if not isinstance(output, ResponseFunctionToolCall):  # This check can never be reached
    continue
```

The second type check (lines 495-497) could never be executed because:
- If `output` is NOT a `ResponseFunctionToolCall`, the first check would have already caused a `continue` and skipped the rest of the loop
- If `output` is a `ResponseFunctionToolCall`, the second check would always be false

## Changes Made

Removed the redundant and unreachable type check, leaving only the necessary logic:

```python
elif not isinstance(output, ResponseFunctionToolCall):
    logger.warning(f"Unexpected output type, ignoring: {type(output)}")
    continue

# At this point we know it's a function tool call
tools_used.append(output.name)
```

This improves code clarity and eliminates dead code that could confuse developers and static analysis tools.
---
 src/agents/_run_impl.py | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/agents/_run_impl.py b/src/agents/_run_impl.py
index 6e23e9507..f291ce232 100644
--- a/src/agents/_run_impl.py
+++ b/src/agents/_run_impl.py
@@ -493,9 +493,6 @@ def process_model_response(
                 continue
 
             # At this point we know it's a function tool call
-            if not isinstance(output, ResponseFunctionToolCall):
-                continue
-
             tools_used.append(output.name)
 
             # Handoffs