Commit b05f6d3
authored
fix(groq): check supports_response_schema before using json_tool_call workaround (#17438)
Fixes issue where LiteLLM used the json_tool_call workaround for all Groq
models, even those that support native json_schema (e.g., gpt-oss-120b,
llama-4, kimi-k2). This caused errors when users passed their own tools
alongside structured outputs.
Changes:
- Check `supports_response_schema()` before using the workaround
- Only use json_tool_call workaround for models without native support
- Add clear error message when using workaround with user-provided tools
- Update model config to set `supports_response_schema: false` for models
that don't support native json_schema
- Add unit tests for structured output handling1 parent 6d831ff commit b05f6d3
File tree
4 files changed
+209
-50
lines changed- litellm
- llms/groq/chat
- tests/llm_translation
4 files changed
+209
-50
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
218 | 218 | | |
219 | 219 | | |
220 | 220 | | |
221 | | - | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
222 | 227 | | |
223 | 228 | | |
224 | | - | |
225 | | - | |
226 | | - | |
227 | | - | |
228 | | - | |
229 | | - | |
230 | | - | |
231 | | - | |
232 | | - | |
233 | | - | |
234 | | - | |
235 | | - | |
236 | | - | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
| 251 | + | |
| 252 | + | |
| 253 | + | |
| 254 | + | |
| 255 | + | |
| 256 | + | |
| 257 | + | |
| 258 | + | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
237 | 262 | | |
238 | 263 | | |
239 | 264 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
16881 | 16881 | | |
16882 | 16882 | | |
16883 | 16883 | | |
16884 | | - | |
| 16884 | + | |
16885 | 16885 | | |
16886 | 16886 | | |
16887 | 16887 | | |
| |||
16900 | 16900 | | |
16901 | 16901 | | |
16902 | 16902 | | |
16903 | | - | |
| 16903 | + | |
16904 | 16904 | | |
16905 | 16905 | | |
16906 | 16906 | | |
| |||
16912 | 16912 | | |
16913 | 16913 | | |
16914 | 16914 | | |
16915 | | - | |
| 16915 | + | |
16916 | 16916 | | |
16917 | 16917 | | |
16918 | 16918 | | |
| |||
16924 | 16924 | | |
16925 | 16925 | | |
16926 | 16926 | | |
16927 | | - | |
| 16927 | + | |
16928 | 16928 | | |
16929 | 16929 | | |
16930 | 16930 | | |
| |||
16937 | 16937 | | |
16938 | 16938 | | |
16939 | 16939 | | |
16940 | | - | |
| 16940 | + | |
16941 | 16941 | | |
16942 | 16942 | | |
16943 | 16943 | | |
| |||
16949 | 16949 | | |
16950 | 16950 | | |
16951 | 16951 | | |
16952 | | - | |
| 16952 | + | |
16953 | 16953 | | |
16954 | 16954 | | |
16955 | 16955 | | |
| |||
16962 | 16962 | | |
16963 | 16963 | | |
16964 | 16964 | | |
16965 | | - | |
| 16965 | + | |
16966 | 16966 | | |
16967 | 16967 | | |
16968 | 16968 | | |
| |||
16975 | 16975 | | |
16976 | 16976 | | |
16977 | 16977 | | |
16978 | | - | |
| 16978 | + | |
16979 | 16979 | | |
16980 | 16980 | | |
16981 | 16981 | | |
| |||
16989 | 16989 | | |
16990 | 16990 | | |
16991 | 16991 | | |
16992 | | - | |
| 16992 | + | |
16993 | 16993 | | |
16994 | 16994 | | |
16995 | 16995 | | |
| |||
17002 | 17002 | | |
17003 | 17003 | | |
17004 | 17004 | | |
17005 | | - | |
| 17005 | + | |
17006 | 17006 | | |
17007 | 17007 | | |
17008 | 17008 | | |
| |||
17015 | 17015 | | |
17016 | 17016 | | |
17017 | 17017 | | |
17018 | | - | |
| 17018 | + | |
17019 | 17019 | | |
17020 | 17020 | | |
17021 | 17021 | | |
| |||
17028 | 17028 | | |
17029 | 17029 | | |
17030 | 17030 | | |
17031 | | - | |
| 17031 | + | |
17032 | 17032 | | |
17033 | 17033 | | |
17034 | 17034 | | |
| |||
17052 | 17052 | | |
17053 | 17053 | | |
17054 | 17054 | | |
17055 | | - | |
| 17055 | + | |
17056 | 17056 | | |
17057 | 17057 | | |
17058 | 17058 | | |
| |||
17073 | 17073 | | |
17074 | 17074 | | |
17075 | 17075 | | |
17076 | | - | |
| 17076 | + | |
17077 | 17077 | | |
17078 | 17078 | | |
17079 | 17079 | | |
| |||
17086 | 17086 | | |
17087 | 17087 | | |
17088 | 17088 | | |
17089 | | - | |
| 17089 | + | |
17090 | 17090 | | |
17091 | 17091 | | |
17092 | 17092 | | |
| |||
17099 | 17099 | | |
17100 | 17100 | | |
17101 | 17101 | | |
17102 | | - | |
| 17102 | + | |
17103 | 17103 | | |
17104 | 17104 | | |
17105 | 17105 | | |
| |||
17145 | 17145 | | |
17146 | 17146 | | |
17147 | 17147 | | |
17148 | | - | |
| 17148 | + | |
17149 | 17149 | | |
17150 | 17150 | | |
17151 | 17151 | | |
| |||
17221 | 17221 | | |
17222 | 17222 | | |
17223 | 17223 | | |
17224 | | - | |
| 17224 | + | |
17225 | 17225 | | |
17226 | 17226 | | |
17227 | 17227 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
16881 | 16881 | | |
16882 | 16882 | | |
16883 | 16883 | | |
16884 | | - | |
| 16884 | + | |
16885 | 16885 | | |
16886 | 16886 | | |
16887 | 16887 | | |
| |||
16900 | 16900 | | |
16901 | 16901 | | |
16902 | 16902 | | |
16903 | | - | |
| 16903 | + | |
16904 | 16904 | | |
16905 | 16905 | | |
16906 | 16906 | | |
| |||
16912 | 16912 | | |
16913 | 16913 | | |
16914 | 16914 | | |
16915 | | - | |
| 16915 | + | |
16916 | 16916 | | |
16917 | 16917 | | |
16918 | 16918 | | |
| |||
16924 | 16924 | | |
16925 | 16925 | | |
16926 | 16926 | | |
16927 | | - | |
| 16927 | + | |
16928 | 16928 | | |
16929 | 16929 | | |
16930 | 16930 | | |
| |||
16937 | 16937 | | |
16938 | 16938 | | |
16939 | 16939 | | |
16940 | | - | |
| 16940 | + | |
16941 | 16941 | | |
16942 | 16942 | | |
16943 | 16943 | | |
| |||
16949 | 16949 | | |
16950 | 16950 | | |
16951 | 16951 | | |
16952 | | - | |
| 16952 | + | |
16953 | 16953 | | |
16954 | 16954 | | |
16955 | 16955 | | |
| |||
16962 | 16962 | | |
16963 | 16963 | | |
16964 | 16964 | | |
16965 | | - | |
| 16965 | + | |
16966 | 16966 | | |
16967 | 16967 | | |
16968 | 16968 | | |
| |||
16975 | 16975 | | |
16976 | 16976 | | |
16977 | 16977 | | |
16978 | | - | |
| 16978 | + | |
16979 | 16979 | | |
16980 | 16980 | | |
16981 | 16981 | | |
| |||
16989 | 16989 | | |
16990 | 16990 | | |
16991 | 16991 | | |
16992 | | - | |
| 16992 | + | |
16993 | 16993 | | |
16994 | 16994 | | |
16995 | 16995 | | |
| |||
17002 | 17002 | | |
17003 | 17003 | | |
17004 | 17004 | | |
17005 | | - | |
| 17005 | + | |
17006 | 17006 | | |
17007 | 17007 | | |
17008 | 17008 | | |
| |||
17015 | 17015 | | |
17016 | 17016 | | |
17017 | 17017 | | |
17018 | | - | |
| 17018 | + | |
17019 | 17019 | | |
17020 | 17020 | | |
17021 | 17021 | | |
| |||
17028 | 17028 | | |
17029 | 17029 | | |
17030 | 17030 | | |
17031 | | - | |
| 17031 | + | |
17032 | 17032 | | |
17033 | 17033 | | |
17034 | 17034 | | |
| |||
17052 | 17052 | | |
17053 | 17053 | | |
17054 | 17054 | | |
17055 | | - | |
| 17055 | + | |
17056 | 17056 | | |
17057 | 17057 | | |
17058 | 17058 | | |
| |||
17073 | 17073 | | |
17074 | 17074 | | |
17075 | 17075 | | |
17076 | | - | |
| 17076 | + | |
17077 | 17077 | | |
17078 | 17078 | | |
17079 | 17079 | | |
| |||
17086 | 17086 | | |
17087 | 17087 | | |
17088 | 17088 | | |
17089 | | - | |
| 17089 | + | |
17090 | 17090 | | |
17091 | 17091 | | |
17092 | 17092 | | |
| |||
17099 | 17099 | | |
17100 | 17100 | | |
17101 | 17101 | | |
17102 | | - | |
| 17102 | + | |
17103 | 17103 | | |
17104 | 17104 | | |
17105 | 17105 | | |
| |||
17145 | 17145 | | |
17146 | 17146 | | |
17147 | 17147 | | |
17148 | | - | |
| 17148 | + | |
17149 | 17149 | | |
17150 | 17150 | | |
17151 | 17151 | | |
| |||
17221 | 17221 | | |
17222 | 17222 | | |
17223 | 17223 | | |
17224 | | - | |
| 17224 | + | |
17225 | 17225 | | |
17226 | 17226 | | |
17227 | 17227 | | |
| |||
0 commit comments