fix: validate sync chat stream tools consistently

Author: Hughhhhcoder

src/openai/resources/chat/completions/completions.py

@@ -37,7 +37,6 @@
     ResponseFormatT,
     validate_input_tools as _validate_input_tools,
     parse_chat_completion as _parse_chat_completion,
-    materialize_input_tools as _materialize_input_tools,
     type_to_response_format_param as _type_to_response_format,
 )
 from ....lib.streaming.chat import ChatCompletionStreamManager, AsyncChatCompletionStreamManager
@@ -1613,7 +1612,7 @@ def stream(
         When the context manager exits, the response will be closed, however the `stream` instance is still available outside
         the context manager.
         """
-        chat_completion_tools = _materialize_input_tools(tools)
+        chat_completion_tools = _validate_input_tools(tools)
 
         extra_headers = {
             "X-Stainless-Helper-Method": "chat.completions.stream",

tests/lib/chat/test_single_pass_tools.py

@@ -34,6 +34,17 @@ def single_pass_tools() -> Iterator[ChatCompletionToolUnionParam]:
     }
 
 
+def non_strict_tools() -> Iterator[ChatCompletionToolUnionParam]:
+    yield {
+        "type": "function",
+        "function": {
+            "name": "get_weather",
+            "parameters": {"type": "object", "properties": {"city": {"type": "string"}}},
+            "strict": False,
+        },
+    }
+
+
 TOOL_CALL = {
     "id": "call-test",
     "type": "function",
@@ -127,6 +138,15 @@ def test_stream_preserves_single_pass_tools(client: OpenAI, respx2_mock: MockRou
     assert_request_and_parsed_tool(respx2_mock, tool_calls[0].function.parsed_arguments)
 
 
+def test_stream_rejects_non_strict_tools(client: OpenAI) -> None:
+    with pytest.raises(ValueError, match="Only `strict` function tools can be auto-parsed"):
+        client.chat.completions.stream(
+            model="gpt-test",
+            messages=[{"role": "user", "content": "weather"}],
+            tools=non_strict_tools(),
+        )
+
+
 @pytest.mark.respx2(base_url=base_url)
 @pytest.mark.asyncio
 async def test_async_stream_preserves_single_pass_tools(async_client: AsyncOpenAI, respx2_mock: MockRouter) -> None: