Merge pull request #249 from kyle-mccarthy/feat/object-safe-executor

feat: make Executor trait object safe

crates/llm-chain/src/traits.rs

@@ -50,17 +50,22 @@ pub enum ExecutorError {
 #[async_trait]
 /// The `Executor` trait represents an executor that performs a single step in a chain. It takes a
 /// step, executes it, and returns the output.
-pub trait Executor: Sized {
+pub trait Executor {
     type StepTokenizer<'a>: Tokenizer
     where
         Self: 'a;
 
     /// Create a new executor with the given options. If you don't need to set any options, you can use the `new` method instead.
     /// # Parameters
     /// * `options`: The options to set.
-    fn new_with_options(options: Options) -> Result<Self, ExecutorCreationError>;
+    fn new_with_options(options: Options) -> Result<Self, ExecutorCreationError>
+    where
+        Self: Sized;
 
-    fn new() -> Result<Self, ExecutorCreationError> {
+    fn new() -> Result<Self, ExecutorCreationError>
+    where
+        Self: Sized,
+    {
         Self::new_with_options(Options::empty().clone())
     }