fix(api): correct types for attachments

examples/demo.py

@@ -36,3 +36,18 @@
 
     print(chunk.choices[0].delta.content, end="")
 print()
+
+# Response headers:
+print("----- custom response headers test -----")
+response = client.chat.completions.with_raw_response.create(
+    model="gpt-4",
+    messages=[
+        {
+            "role": "user",
+            "content": "Say this is a test",
+        }
+    ],
+)
+completion = response.parse()
+print(response.request_id)
+print(completion.choices[0].message.content)

helpers.md

@@ -33,11 +33,13 @@ class EventHandler(AssistantEventHandler):
   def on_text_delta(self, delta: TextDelta, snapshot: Text):
     print(delta.value, end="", flush=True)
 
+  @override
   def on_tool_call_created(self, tool_call: ToolCall):
     print(f"\nassistant > {tool_call.type}\n", flush=True)
 
+  @override
   def on_tool_call_delta(self, delta: ToolCallDelta, snapshot: ToolCall):
-    if delta.type == 'code_interpreter':
+    if delta.type == "code_interpreter" and delta.code_interpreter:
       if delta.code_interpreter.input:
         print(delta.code_interpreter.input, end="", flush=True)
       if delta.code_interpreter.outputs:
@@ -69,7 +71,7 @@ with client.beta.threads.runs.stream(
 ) as stream:
     for event in stream:
         # Print the text from text delta events
-        if event.type == "thread.message.delta" and event.data.delta.content:
+        if event.event == "thread.message.delta" and event.data.delta.content:
             print(event.data.delta.content[0].text)
 ```
 

src/openai/_legacy_response.py

@@ -71,6 +71,10 @@ def __init__(
         self._options = options
         self.http_response = raw
 
+    @property
+    def request_id(self) -> str | None:
+        return self.http_response.headers.get("x-request-id")  # type: ignore[no-any-return]
+
     @overload
     def parse(self, *, to: type[_T]) -> _T:
         ...

src/openai/_response.py

@@ -258,6 +258,10 @@ def _parse(self, *, to: type[_T] | None = None) -> R | _T:
 
 
 class APIResponse(BaseAPIResponse[R]):
+    @property
+    def request_id(self) -> str | None:
+        return self.http_response.headers.get("x-request-id")  # type: ignore[no-any-return]
+
     @overload
     def parse(self, *, to: type[_T]) -> _T:
         ...
@@ -362,6 +366,10 @@ def iter_lines(self) -> Iterator[str]:
 
 
 class AsyncAPIResponse(BaseAPIResponse[R]):
+    @property
+    def request_id(self) -> str | None:
+        return self.http_response.headers.get("x-request-id")  # type: ignore[no-any-return]
+
     @overload
     async def parse(self, *, to: type[_T]) -> _T:
         ...

src/openai/types/beta/thread_create_and_run_params.py

@@ -171,11 +171,11 @@ class ThreadCreateAndRunParamsBase(TypedDict, total=False):
 
 
 class ThreadMessageAttachment(TypedDict, total=False):
-    add_to: List[Literal["file_search", "code_interpreter"]]
-
     file_id: str
     """The ID of the file to attach to the message."""
 
+    tools: List[Literal["file_search", "code_interpreter"]]
+
 
 class ThreadMessage(TypedDict, total=False):
     content: Required[str]

src/openai/types/beta/thread_create_params.py

@@ -41,11 +41,11 @@ class ThreadCreateParams(TypedDict, total=False):
 
 
 class MessageAttachment(TypedDict, total=False):
-    add_to: List[Literal["file_search", "code_interpreter"]]
-
     file_id: str
     """The ID of the file to attach to the message."""
 
+    tools: List[Literal["file_search", "code_interpreter"]]
+
 
 class Message(TypedDict, total=False):
     content: Required[str]

src/openai/types/beta/threads/message.py

@@ -10,11 +10,11 @@
 
 
 class Attachment(BaseModel):
-    add_to: Optional[List[Literal["file_search", "code_interpreter"]]] = None
-
     file_id: Optional[str] = None
     """The ID of the file to attach to the message."""
 
+    tools: Optional[List[Literal["file_search", "code_interpreter"]]] = None
+
 
 class IncompleteDetails(BaseModel):
     reason: Literal["content_filter", "max_tokens", "run_cancelled", "run_expired", "run_failed"]

src/openai/types/beta/threads/message_create_params.py

@@ -34,7 +34,7 @@ class MessageCreateParams(TypedDict, total=False):
 
 
 class Attachment(TypedDict, total=False):
-    add_to: List[Literal["file_search", "code_interpreter"]]
-
     file_id: str
     """The ID of the file to attach to the message."""
+
+    tools: List[Literal["file_search", "code_interpreter"]]

src/openai/types/beta/threads/run_create_params.py

@@ -160,11 +160,11 @@ class RunCreateParamsBase(TypedDict, total=False):
 
 
 class AdditionalMessageAttachment(TypedDict, total=False):
-    add_to: List[Literal["file_search", "code_interpreter"]]
-
     file_id: str
     """The ID of the file to attach to the message."""
 
+    tools: List[Literal["file_search", "code_interpreter"]]
+
 
 class AdditionalMessage(TypedDict, total=False):
     content: Required[str]