modified: CHANGELOG.md

	modified:   simpletool/models.py
	modified:   tests/conftest.py
	modified:   tests/test_errors.py

Author: azdolinski

CHANGELOG.md

@@ -5,6 +5,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/),
 and this project adheres to [SimpleTool](https://github.com/nchekwa/simpletool-python/tree/master).
 
+# [0.0.16] - 2025-01-11 Milestone Alpha2
+
+## Fixed
+- incorrect `SimpleToolResponseModel` Pydantic model configuration + from_attributes allow easy serialize/deserialize
+
 # [0.0.15] - 2025-01-09 Milestone Alpha2
 
 ## Fixed

simpletool/models.py

@@ -46,15 +46,18 @@ class SimpleToolResponseModel(BaseModel):
 
     class Config:
         """Pydantic model configuration."""
-        schema_extra = {
-            "example": {
-                "name": "example_tool",
-                "description": "An example tool for demonstration",
-                "input_schema": {
-                    "type": "object",
-                    "properties": {
-                        "input": {"type": "string"}
+        model_config = {
+            "json_schema_extra": {
+                "example": {
+                    "name": "example_tool",
+                    "description": "An example tool for demonstration",
+                    "input_schema": {
+                        "type": "object",
+                        "properties": {
+                            "input": {"type": "string"}
+                        }
                     }
                 }
-            }
+            },
+            "from_attributes": True  # Enables serialization from other types
         }

tests/conftest.py

@@ -20,5 +20,5 @@ def pytest_collection_modifyitems(config, items):
         warnings.filterwarnings(
             "ignore",
             category=pytest.PytestCollectionWarning,
-            message=f"{item} cannot collect test class .* because it has a __init__ constructor"
+            message=r".*cannot collect test class .* because it has a __init__ constructor"
         )

tests/test_errors.py

@@ -5,34 +5,32 @@
 from simpletool.errors import SimpleToolError, ValidationError
 
 
-def test_simple_tool_error():
-    """Test SimpleToolError initialization and attributes."""
-    error = SimpleToolError(
-        message="Test error", 
-        code=501, 
-        details={"context": "test"}
-    )
-    
-    assert str(error) == "Test error"
-    assert error.message == "Test error"
-    assert error.code == 501
-    assert error.details == {"context": "test"}
+@pytest.mark.parametrize("error_class,input_data,expected", [
+    (
+        SimpleToolError,
+        {"message": "Test error", "code": 501, "details": {"context": "test"}},
+        {"str": "Test error", "code": 501, "details": {"context": "test"}}
+    ),
+    (
+        SimpleToolError,
+        {"message": "Default error"},
+        {"str": "Default error", "code": 500, "details": {}}
+    ),
+])
+def test_error_initialization(error_class, input_data, expected):
+    """Test error class initialization with various inputs."""
+    error = error_class(**input_data)
 
-
-def test_simple_tool_error_default_values():
-    """Test SimpleToolError with default values."""
-    error = SimpleToolError("Default error")
-    
-    assert str(error) == "Default error"
-    assert error.message == "Default error"
-    assert error.code == 500
-    assert error.details == {}
+    assert str(error) == expected["str"]
+    assert error.message == expected["str"]
+    assert error.code == expected["code"]
+    assert error.details == expected["details"]
 
 
 def test_validation_error():
     """Test ValidationError initialization."""
     error = ValidationError(field="test_field", reason="Invalid input")
-    
+
     assert str(error) == "Validation failed for field 'test_field': Invalid input"
     assert error.message == "Validation failed for field 'test_field': Invalid input"
     assert error.code == 400