Refactor converters

griptape/artifacts/audio_artifact.py

@@ -2,19 +2,17 @@
 
 from attrs import define, field
 
-from griptape.artifacts import BaseArtifact
+from griptape.artifacts import BlobArtifact
 
 
 @define
-class AudioArtifact(BaseArtifact):
+class AudioArtifact(BlobArtifact):
     """Stores audio data.
 
     Attributes:
-        value: The audio data.
         format: The audio format, e.g. "wav" or "mp3".
     """
 
-    value: bytes = field(metadata={"serializable": True})
     format: str = field(kw_only=True, metadata={"serializable": True})
 
     @property

griptape/artifacts/base_artifact.py

@@ -24,7 +24,8 @@ class BaseArtifact(SerializableMixin, ABC):
         meta: The metadata associated with the Artifact. Defaults to an empty dictionary.
         name: The name of the Artifact. Defaults to the id.
         value: The value of the Artifact.
-        encoding: The encoding of the Artifact when converting to bytes.
+        encoding: The encoding to use when encoding/decoding the value.
+        encoding_error_handler: The error handler to use when encoding/decoding the value.
     """
 
     id: str = field(default=Factory(lambda: uuid.uuid4().hex), kw_only=True, metadata={"serializable": True})
@@ -36,6 +37,7 @@ class BaseArtifact(SerializableMixin, ABC):
         metadata={"serializable": True},
     )
     value: Any = field()
+    encoding_error_handler: str = field(default="strict", kw_only=True)
     encoding: str = field(default="utf-8", kw_only=True)
 
     def __str__(self) -> str:
@@ -48,7 +50,7 @@ def __len__(self) -> int:
         return len(self.value)
 
     def to_bytes(self) -> bytes:
-        return self.to_text().encode(self.encoding)
+        return self.to_text().encode(encoding=self.encoding, errors=self.encoding_error_handler)
 
     @abstractmethod
     def to_text(self) -> str: ...

griptape/artifacts/blob_artifact.py

@@ -13,13 +13,9 @@ class BlobArtifact(BaseArtifact):
 
     Attributes:
         value: The binary data.
-        encoding: The encoding to use when converting the binary data to text.
-        encoding_error_handler: The error handler to use when converting the binary data to text.
     """
 
     value: bytes = field(converter=lambda value: BlobArtifact.value_to_bytes(value), metadata={"serializable": True})
-    encoding: str = field(default="utf-8", kw_only=True)
-    encoding_error_handler: str = field(default="strict", kw_only=True)
 
     @property
     def mime_type(self) -> str:

griptape/artifacts/image_artifact.py

@@ -12,13 +12,11 @@ class ImageArtifact(BlobArtifact):
     """Stores image data.
 
     Attributes:
-        value: The image data.
         format: The format of the image data.
         width: The width of the image.
         height: The height of the image
     """
 
-    value: bytes = field(metadata={"serializable": True})
     format: str = field(kw_only=True, metadata={"serializable": True})
     width: int = field(kw_only=True, metadata={"serializable": True})
     height: int = field(kw_only=True, metadata={"serializable": True})

griptape/artifacts/text_artifact.py

@@ -14,8 +14,6 @@
 @define
 class TextArtifact(BaseArtifact):
     value: str = field(converter=str, metadata={"serializable": True})
-    encoding: str = field(default="utf-8", kw_only=True)
-    encoding_error_handler: str = field(default="strict", kw_only=True)
     embedding: Optional[list[float]] = field(default=None, kw_only=True)
 
     def __add__(self, other: BaseArtifact) -> TextArtifact: