Skip to content

Conversation

@aconchillo
Copy link
Contributor

No description provided.

@markbackman
Copy link
Contributor

FYI: @jptaylor just proposed this to moving where body appears:
pipecat-ai/pipecat#2890

We should consider this holistically before making either change.

@aconchillo
Copy link
Contributor Author

FYI: @jptaylor just proposed this to moving where body appears: pipecat-ai/pipecat#2890

We should consider this holistically before making either change.

I'm totally fine with @jptaylor changes. 🙌

@markbackman
Copy link
Contributor

FYI: @jptaylor just proposed this to moving where body appears: pipecat-ai/pipecat#2890
We should consider this holistically before making either change.

I'm totally fine with @jptaylor changes. 🙌

If we move body to the base RunnerArguments, then I think this change isn't needed. But, we need another pipecatcloud change for the update to move body to the RunnerArguments.

@aconchillo aconchillo force-pushed the aleix/use-pipecat-runner-arguments branch from 56ed233 to 44acafb Compare October 21, 2025 18:36
@aconchillo aconchillo changed the title agent: use PipecatRunnerArguments agent: add body to RunnerArguments Oct 21, 2025
Copy link
Contributor

@markbackman markbackman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@aconchillo aconchillo force-pushed the aleix/use-pipecat-runner-arguments branch from 44acafb to a40eaec Compare October 21, 2025 20:51
@aconchillo aconchillo force-pushed the aleix/use-pipecat-runner-arguments branch from a40eaec to 08d5ca6 Compare October 21, 2025 23:26
@aconchillo aconchillo merged commit 0468e02 into main Oct 21, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants