Process review comments:

- Prevent crash when user has empty functions directory
- Prevent parallel function calling from the LLM as we don't handle that correctly
- The option to disable parallel function calling is added in a later release of the openai library, so upgraded it

pyproject.toml

@@ -28,7 +28,7 @@ classifiers = [
     "Programming Language :: Python :: 3.11",
 ]
 dependencies = [
-    "openai >= 1.6.1, < 2.0.0",
+    "openai >= 1.34.0, < 2.0.0",
     "typer >= 0.7.0, < 1.0.0",
     "click >= 7.1.1, < 9.0.0",
     "rich >= 13.1.0, < 14.0.0",

sgpt/handlers/handler.py

@@ -98,13 +98,16 @@ def get_completion(
         if is_shell_role or is_code_role or is_dsc_shell_role:
             functions = None
 
+        if functions:
+            additional_kwargs["tool_choice"] = "auto"
+            additional_kwargs["tools"] = functions
+            additional_kwargs["parallel_tool_calls"] = False
+
         response = completion(
             model=model,
             temperature=temperature,
             top_p=top_p,
             messages=messages,
-            tools=functions,
-            tool_choice="auto",
             stream=True,
             **additional_kwargs,
         )