feat: Add reasoning content for openai model provider

Author: xiehust

tests/strands/models/test_openai.py

@@ -63,12 +63,12 @@ def test_stream(openai_client, model):
     mock_tool_call_1_part_1 = unittest.mock.Mock(index=0)
     mock_tool_call_2_part_1 = unittest.mock.Mock(index=1)
     mock_delta_1 = unittest.mock.Mock(
-        reasoning_content="<think>",
+        reasoning_content="",
         content=None,
         tool_calls=None,
     )
     mock_delta_2 = unittest.mock.Mock(
-        reasoning_content="\nOkey, the user just</think>",
+        reasoning_content="\nI'm thinking",
         content=None,
         tool_calls=None,
     )
@@ -101,8 +101,8 @@ def test_stream(openai_client, model):
     exp_events = [
         {"chunk_type": "message_start"},
         {"chunk_type": "content_start", "data_type": "text"},
-        {"chunk_type": "content_delta", "data_type": "reasoning_content", "data": "<think>"},
-        {"chunk_type": "content_delta", "data_type": "reasoning_content", "data": "\nOkey, the user just</think>"},
+        # {"chunk_type": "content_delta", "data_type": "reasoning_content", "data": ""},
+        {"chunk_type": "content_delta", "data_type": "reasoning_content", "data": "\nI'm thinking"},
         {"chunk_type": "content_delta", "data_type": "text", "data": "I'll calculate"},
         {"chunk_type": "content_delta", "data_type": "text", "data": "that for you"},
         {"chunk_type": "content_stop", "data_type": "text"},

tests/strands/types/models/test_openai.py

@@ -107,13 +107,6 @@ def test_format_request_message_content(content, exp_result):
     assert tru_result == exp_result
 
 
-def test_format_request_message_content_unsupported_type():
-    content = {"unsupported": {}}
-
-    with pytest.raises(TypeError, match="content_type=<unsupported> | unsupported type"):
-        SAOpenAIModel.format_request_message_content(content)
-
-
 def test_format_request_message_tool_call():
     tool_use = {
         "input": {"expression": "2+2"},
@@ -288,6 +281,11 @@ def test_format_request(model, messages, tool_specs, system_prompt):
             },
             {"contentBlockDelta": {"delta": {"toolUse": {"input": ""}}}},
         ),
+        # Content Delta - Reasoning Text
+        (
+            {"chunk_type": "content_delta", "data_type": "reasoning_content", "data": "I'm thinking"},
+            {"contentBlockDelta": {"delta": {"reasoningContent": {"text": "I'm thinking"}}}},
+        ),
         # Content Delta - Text
         (
             {"chunk_type": "content_delta", "data_type": "text", "data": "hello"},
@@ -337,10 +335,3 @@ def test_format_request(model, messages, tool_specs, system_prompt):
 def test_format_chunk(event, exp_chunk, model):
     tru_chunk = model.format_chunk(event)
     assert tru_chunk == exp_chunk
-
-
-def test_format_chunk_unknown_type(model):
-    event = {"chunk_type": "unknown"}
-
-    with pytest.raises(RuntimeError, match="chunk_type=<unknown> | unknown type"):
-        model.format_chunk(event)