fix: allow ConversationalAgentBuilder to be built using a dynamic LLM…

[samuelint]

It's currently not possible to create a ConversationalAgent using the ConversationalAgentBuilder if the LLM instance is not static. This Pull-Request fix that.

Usage example:

let llm: Box<dyn LLM> = self.llm_factory.create(CreateLLMParameters {
    model: parameters.model, // openai:gpt-4o-mini or anthropic:claude-xyz
})?;

let agent = ConversationalAgentBuilder::new().build(llm).unwrap();

[samuelint]

@Abraxas-365 can this be merged?

[samuelint]

Any update @prabirshrestha, @Abraxas-365 ? This is a very small change / easy to review. And the pipeline passed ;)

[prabirshrestha]

Was back from vacation and have been catching up at work. I did see the PR but wanted to see if we can avoid Box if possible.

[samuelint]

Was back from vacation and have been catching up at work. I did see the PR but wanted to see if we can avoid Box if possible.

I'm curious, why Box should be avoided?

[prabirshrestha]

LGTM. Was thinking if there are alternatives.

[samuelint]

@prabirshrestha when should I expect a new release? (So my change is on the crate registry)

[Abraxas-365]

I will push a new release later today 💪🏾