feat: Update handling of tools_strict to ensure compatibility with vLLM (#150)

Emil-io · julian-risch · web-flow · commit 7ade6a28f166 · 2024-12-10T18:58:21.000+01:00
* update handling of tools_strict

* Update haystack_experimental/components/generators/chat/openai.py

Co-authored-by: Julian Risch &lt;julianrisch@gmx.de&gt;

---------

Co-authored-by: Julian Risch &lt;julianrisch@gmx.de&gt;
diff --git a/haystack_experimental/components/generators/chat/openai.py b/haystack_experimental/components/generators/chat/openai.py
@@ -447,7 +447,10 @@ def _prepare_api_call(  # noqa: PLR0913
             openai_tools = [
                 {
                     "type": "function",
-                    "function": {**t.tool_spec, "strict": tools_strict},
+                    "function": {
+                        **t.tool_spec,
+                        **({"strict": tools_strict} if tools_strict is not None else {}),
+                    },
                 }
                 for t in tools
             ]

Original file line number	Diff line number	Diff line change
`@@ -447,7 +447,10 @@ def _prepare_api_call( # noqa: PLR0913`
`447`	`447`	`openai_tools = [`
`448`	`448`	`{`
`449`	`449`	`"type": "function",`
`450`		`- "function": {**t.tool_spec, "strict": tools_strict},`
	`450`	`+ "function": {`
	`451`	`+ **t.tool_spec,`
	`452`	`+ **({"strict": tools_strict} if tools_strict is not None else {}),`
	`453`	`+ },`
`451`	`454`	`}`
`452`	`455`	`for t in tools`
`453`	`456`	`]`