Skip to content

Commit 542d30c

Browse files
stainless-botstainless-bot
authored
fix(api): correct types for attachments (#1342)
1 parent 9e0ea64 commit 542d30c

File tree

8 files changed

+106
-106
lines changed

8 files changed

+106
-106
lines changed

‎src/openai/types/beta/thread_create_and_run_params.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -171,11 +171,11 @@ class ThreadCreateAndRunParamsBase(TypedDict, total=False):
171171

172172

173173
class ThreadMessageAttachment(TypedDict, total=False):
174-
add_to: List[Literal["file_search", "code_interpreter"]]
175-
176174
file_id: str
177175
"""The ID of the file to attach to the message."""
178176

177+
tools: List[Literal["file_search", "code_interpreter"]]
178+
179179

180180
class ThreadMessage(TypedDict, total=False):
181181
content: Required[str]

‎src/openai/types/beta/thread_create_params.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,11 @@ class ThreadCreateParams(TypedDict, total=False):
4141

4242

4343
class MessageAttachment(TypedDict, total=False):
44-
add_to: List[Literal["file_search", "code_interpreter"]]
45-
4644
file_id: str
4745
"""The ID of the file to attach to the message."""
4846

47+
tools: List[Literal["file_search", "code_interpreter"]]
48+
4949

5050
class Message(TypedDict, total=False):
5151
content: Required[str]

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

+2-2
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,11 @@
1010

1111

1212
class Attachment(BaseModel):
13-
add_to: Optional[List[Literal["file_search", "code_interpreter"]]] = None
14-
1513
file_id: Optional[str] = None
1614
"""The ID of the file to attach to the message."""
1715

16+
tools: Optional[List[Literal["file_search", "code_interpreter"]]] = None
17+
1818

1919
class IncompleteDetails(BaseModel):
2020
reason: Literal["content_filter", "max_tokens", "run_cancelled", "run_expired", "run_failed"]

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

+2-2
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ class MessageCreateParams(TypedDict, total=False):
3434

3535

3636
class Attachment(TypedDict, total=False):
37-
add_to: List[Literal["file_search", "code_interpreter"]]
38-
3937
file_id: str
4038
"""The ID of the file to attach to the message."""
39+
40+
tools: List[Literal["file_search", "code_interpreter"]]

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

+2-2
Original file line numberDiff line numberDiff line change
@@ -160,11 +160,11 @@ class RunCreateParamsBase(TypedDict, total=False):
160160

161161

162162
class AdditionalMessageAttachment(TypedDict, total=False):
163-
add_to: List[Literal["file_search", "code_interpreter"]]
164-
165163
file_id: str
166164
"""The ID of the file to attach to the message."""
167165

166+
tools: List[Literal["file_search", "code_interpreter"]]
167+
168168

169169
class AdditionalMessage(TypedDict, total=False):
170170
content: Required[str]

‎tests/api_resources/beta/test_threads.py

+54-54
Large diffs are not rendered by default.

‎tests/api_resources/beta/threads/test_messages.py

+6-6
Original file line numberDiff line numberDiff line change
@@ -36,15 +36,15 @@ def test_method_create_with_all_params(self, client: OpenAI) -> None:
3636
attachments=[
3737
{
3838
"file_id": "string",
39-
"add_to": ["file_search", "code_interpreter"],
39+
"tools": ["file_search", "code_interpreter"],
4040
},
4141
{
4242
"file_id": "string",
43-
"add_to": ["file_search", "code_interpreter"],
43+
"tools": ["file_search", "code_interpreter"],
4444
},
4545
{
4646
"file_id": "string",
47-
"add_to": ["file_search", "code_interpreter"],
47+
"tools": ["file_search", "code_interpreter"],
4848
},
4949
],
5050
metadata={},
@@ -265,15 +265,15 @@ async def test_method_create_with_all_params(self, async_client: AsyncOpenAI) ->
265265
attachments=[
266266
{
267267
"file_id": "string",
268-
"add_to": ["file_search", "code_interpreter"],
268+
"tools": ["file_search", "code_interpreter"],
269269
},
270270
{
271271
"file_id": "string",
272-
"add_to": ["file_search", "code_interpreter"],
272+
"tools": ["file_search", "code_interpreter"],
273273
},
274274
{
275275
"file_id": "string",
276-
"add_to": ["file_search", "code_interpreter"],
276+
"tools": ["file_search", "code_interpreter"],
277277
},
278278
],
279279
metadata={},

‎tests/api_resources/beta/threads/test_runs.py

+36-36
Original file line numberDiff line numberDiff line change
@@ -43,15 +43,15 @@ def test_method_create_with_all_params_overload_1(self, client: OpenAI) -> None:
4343
"attachments": [
4444
{
4545
"file_id": "string",
46-
"add_to": ["file_search", "code_interpreter"],
46+
"tools": ["file_search", "code_interpreter"],
4747
},
4848
{
4949
"file_id": "string",
50-
"add_to": ["file_search", "code_interpreter"],
50+
"tools": ["file_search", "code_interpreter"],
5151
},
5252
{
5353
"file_id": "string",
54-
"add_to": ["file_search", "code_interpreter"],
54+
"tools": ["file_search", "code_interpreter"],
5555
},
5656
],
5757
"metadata": {},
@@ -62,15 +62,15 @@ def test_method_create_with_all_params_overload_1(self, client: OpenAI) -> None:
6262
"attachments": [
6363
{
6464
"file_id": "string",
65-
"add_to": ["file_search", "code_interpreter"],
65+
"tools": ["file_search", "code_interpreter"],
6666
},
6767
{
6868
"file_id": "string",
69-
"add_to": ["file_search", "code_interpreter"],
69+
"tools": ["file_search", "code_interpreter"],
7070
},
7171
{
7272
"file_id": "string",
73-
"add_to": ["file_search", "code_interpreter"],
73+
"tools": ["file_search", "code_interpreter"],
7474
},
7575
],
7676
"metadata": {},
@@ -81,15 +81,15 @@ def test_method_create_with_all_params_overload_1(self, client: OpenAI) -> None:
8181
"attachments": [
8282
{
8383
"file_id": "string",
84-
"add_to": ["file_search", "code_interpreter"],
84+
"tools": ["file_search", "code_interpreter"],
8585
},
8686
{
8787
"file_id": "string",
88-
"add_to": ["file_search", "code_interpreter"],
88+
"tools": ["file_search", "code_interpreter"],
8989
},
9090
{
9191
"file_id": "string",
92-
"add_to": ["file_search", "code_interpreter"],
92+
"tools": ["file_search", "code_interpreter"],
9393
},
9494
],
9595
"metadata": {},
@@ -170,15 +170,15 @@ def test_method_create_with_all_params_overload_2(self, client: OpenAI) -> None:
170170
"attachments": [
171171
{
172172
"file_id": "string",
173-
"add_to": ["file_search", "code_interpreter"],
173+
"tools": ["file_search", "code_interpreter"],
174174
},
175175
{
176176
"file_id": "string",
177-
"add_to": ["file_search", "code_interpreter"],
177+
"tools": ["file_search", "code_interpreter"],
178178
},
179179
{
180180
"file_id": "string",
181-
"add_to": ["file_search", "code_interpreter"],
181+
"tools": ["file_search", "code_interpreter"],
182182
},
183183
],
184184
"metadata": {},
@@ -189,15 +189,15 @@ def test_method_create_with_all_params_overload_2(self, client: OpenAI) -> None:
189189
"attachments": [
190190
{
191191
"file_id": "string",
192-
"add_to": ["file_search", "code_interpreter"],
192+
"tools": ["file_search", "code_interpreter"],
193193
},
194194
{
195195
"file_id": "string",
196-
"add_to": ["file_search", "code_interpreter"],
196+
"tools": ["file_search", "code_interpreter"],
197197
},
198198
{
199199
"file_id": "string",
200-
"add_to": ["file_search", "code_interpreter"],
200+
"tools": ["file_search", "code_interpreter"],
201201
},
202202
],
203203
"metadata": {},
@@ -208,15 +208,15 @@ def test_method_create_with_all_params_overload_2(self, client: OpenAI) -> None:
208208
"attachments": [
209209
{
210210
"file_id": "string",
211-
"add_to": ["file_search", "code_interpreter"],
211+
"tools": ["file_search", "code_interpreter"],
212212
},
213213
{
214214
"file_id": "string",
215-
"add_to": ["file_search", "code_interpreter"],
215+
"tools": ["file_search", "code_interpreter"],
216216
},
217217
{
218218
"file_id": "string",
219-
"add_to": ["file_search", "code_interpreter"],
219+
"tools": ["file_search", "code_interpreter"],
220220
},
221221
],
222222
"metadata": {},
@@ -635,15 +635,15 @@ async def test_method_create_with_all_params_overload_1(self, async_client: Asyn
635635
"attachments": [
636636
{
637637
"file_id": "string",
638-
"add_to": ["file_search", "code_interpreter"],
638+
"tools": ["file_search", "code_interpreter"],
639639
},
640640
{
641641
"file_id": "string",
642-
"add_to": ["file_search", "code_interpreter"],
642+
"tools": ["file_search", "code_interpreter"],
643643
},
644644
{
645645
"file_id": "string",
646-
"add_to": ["file_search", "code_interpreter"],
646+
"tools": ["file_search", "code_interpreter"],
647647
},
648648
],
649649
"metadata": {},
@@ -654,15 +654,15 @@ async def test_method_create_with_all_params_overload_1(self, async_client: Asyn
654654
"attachments": [
655655
{
656656
"file_id": "string",
657-
"add_to": ["file_search", "code_interpreter"],
657+
"tools": ["file_search", "code_interpreter"],
658658
},
659659
{
660660
"file_id": "string",
661-
"add_to": ["file_search", "code_interpreter"],
661+
"tools": ["file_search", "code_interpreter"],
662662
},
663663
{
664664
"file_id": "string",
665-
"add_to": ["file_search", "code_interpreter"],
665+
"tools": ["file_search", "code_interpreter"],
666666
},
667667
],
668668
"metadata": {},
@@ -673,15 +673,15 @@ async def test_method_create_with_all_params_overload_1(self, async_client: Asyn
673673
"attachments": [
674674
{
675675
"file_id": "string",
676-
"add_to": ["file_search", "code_interpreter"],
676+
"tools": ["file_search", "code_interpreter"],
677677
},
678678
{
679679
"file_id": "string",
680-
"add_to": ["file_search", "code_interpreter"],
680+
"tools": ["file_search", "code_interpreter"],
681681
},
682682
{
683683
"file_id": "string",
684-
"add_to": ["file_search", "code_interpreter"],
684+
"tools": ["file_search", "code_interpreter"],
685685
},
686686
],
687687
"metadata": {},
@@ -762,15 +762,15 @@ async def test_method_create_with_all_params_overload_2(self, async_client: Asyn
762762
"attachments": [
763763
{
764764
"file_id": "string",
765-
"add_to": ["file_search", "code_interpreter"],
765+
"tools": ["file_search", "code_interpreter"],
766766
},
767767
{
768768
"file_id": "string",
769-
"add_to": ["file_search", "code_interpreter"],
769+
"tools": ["file_search", "code_interpreter"],
770770
},
771771
{
772772
"file_id": "string",
773-
"add_to": ["file_search", "code_interpreter"],
773+
"tools": ["file_search", "code_interpreter"],
774774
},
775775
],
776776
"metadata": {},
@@ -781,15 +781,15 @@ async def test_method_create_with_all_params_overload_2(self, async_client: Asyn
781781
"attachments": [
782782
{
783783
"file_id": "string",
784-
"add_to": ["file_search", "code_interpreter"],
784+
"tools": ["file_search", "code_interpreter"],
785785
},
786786
{
787787
"file_id": "string",
788-
"add_to": ["file_search", "code_interpreter"],
788+
"tools": ["file_search", "code_interpreter"],
789789
},
790790
{
791791
"file_id": "string",
792-
"add_to": ["file_search", "code_interpreter"],
792+
"tools": ["file_search", "code_interpreter"],
793793
},
794794
],
795795
"metadata": {},
@@ -800,15 +800,15 @@ async def test_method_create_with_all_params_overload_2(self, async_client: Asyn
800800
"attachments": [
801801
{
802802
"file_id": "string",
803-
"add_to": ["file_search", "code_interpreter"],
803+
"tools": ["file_search", "code_interpreter"],
804804
},
805805
{
806806
"file_id": "string",
807-
"add_to": ["file_search", "code_interpreter"],
807+
"tools": ["file_search", "code_interpreter"],
808808
},
809809
{
810810
"file_id": "string",
811-
"add_to": ["file_search", "code_interpreter"],
811+
"tools": ["file_search", "code_interpreter"],
812812
},
813813
],
814814
"metadata": {},

0 commit comments

Comments
 (0)