Skip to content
This repository was archived by the owner on Aug 27, 2025. It is now read-only.

Conversation

@dbennerIBM
Copy link
Contributor

missing role parameter in the client.py example in README.md, this fixes an error where the current code cannot talk to agent.py server.

missing role parameter in the client.py example in README.md, this fixes an error where the current code cannot talk to agent.py server. 

Signed-off-by: Daniel Benner <dbenner@us.ibm.com>
@dbennerIBM dbennerIBM requested a review from a team as a code owner August 9, 2025 16:29
@pilartomas
Copy link
Contributor

pilartomas commented Aug 11, 2025

Hello, are you sure this is the culprit? Pydantic should apply user as the default role when it is not specified:

role: Literal["user"] | Literal["agent"] | str = Field("user", pattern=r"^(user|agent(\/[a-zA-Z0-9_\-]+)?)$")

Regardless, we should merge this to make the example more explicit 👍

@pilartomas pilartomas merged commit 28b5fcd into i-am-bee:main Aug 11, 2025
8 checks passed
@dbennerIBM dbennerIBM deleted the patch-1 branch August 11, 2025 12:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants