Allow non-litellm clients to use tools (#36)

* allow non-litellm clients to use tools

* format

* added precommit to justfile to simplify life

* added precommit to justfile to simplify life

* bump patch

* Increase GitHub Actions test timeout to 10 minutes

* Update justfile to use uv run for pre-commit hook

* Add concurrency control to GitHub Actions workflow

---------

Co-authored-by: Ethan Knox <[email protected]>

Author: knowsuchagency

.github/workflows/tests.yml

@@ -1,5 +1,9 @@
 name: Tests
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 on:
   push:
     branches: [main]
@@ -30,7 +34,7 @@ jobs:
           uv sync
 
       - name: Run Tests
-        timeout-minutes: 8
+        timeout-minutes: 10
         env:
           OPENAI_API_KEY: ${{ secrets.OPENAI_API_KEY }}
           ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}

justfile

@@ -21,3 +21,6 @@ test-fn fn:
 # Run tests with coverage
 test-cov: format
     uv run pytest --cov=promptic --cov-report=term-missing tests/
+
+pre-commit:
+    uv run pre-commit

promptic.py

@@ -17,7 +17,7 @@
 from pydantic import BaseModel
 from litellm import completion as litellm_completion
 
-__version__ = "5.0.0"
+__version__ = "5.0.1"
 
 SystemPrompt = Optional[Union[str, List[str], List[Dict[str, str]]]]
 
@@ -335,7 +335,9 @@ def wrapper(*args, **kwargs):
             self.logger.debug(f"{kwargs = }")
             self.logger.debug(f"{self.cache = }")
 
-            if self.tools:
+            if (
+                self.tools and not self.openai_client
+            ):  # assume oai clients support tools
                 assert litellm.supports_function_calling(self.model), (
                     f"Model {self.model} does not support function calling"
                 )