Skip to content

Conversation

fjuma
Copy link
Collaborator

@fjuma fjuma commented Sep 10, 2025

No description provided.

@fjuma
Copy link
Collaborator Author

fjuma commented Sep 10, 2025

@cescoffier Please review, thanks!

(Btw, it seems I'm not able to add Reviewers to PRs in this repo)

Copy link

github-actions bot commented Sep 10, 2025

🙈 The PR is closed and the preview is expired.

@fjuma fjuma force-pushed the a2a_content_creation branch 2 times, most recently from 048f4a0 to 295592b Compare September 15, 2025 13:58
@fjuma fjuma requested a review from cescoffier September 15, 2025 15:29

We're going to do a deep dive into the https://github.com/a2aproject/a2a-samples/tree/main/samples/python/hosts/content_creation[Content Creation] sample from the https://github.com/a2aproject/a2a-samples[a2a-samples] project.

This sample showcases a content creation pipeline with a Host agent that acts as the central orchestrator,
Copy link
Member

Choose a reason for hiding this comment

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

Why Host with an uppercase?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

"Host" is the agent name that I'm using for the central orchestrator, it's referenced in the table and in the diagrams. I'll update this to use Host instead to hopefully make it more clear.


=== Handling a Content Creation Request

Upon receiving a content creation request from the user, the Host agent breaks down the content creation task
Copy link
Member

Choose a reason for hiding this comment

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

Can you give names to the agents and reuse these names here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The table and diagrams use the names "Host", "Content Planner", "Content Writer", and "Content Editor". I'll update the references to use Host, Content Planner, Content Writer, and Content Editor. Hopefully that's more clear.

@fjuma fjuma force-pushed the a2a_content_creation branch from 295592b to dd0d917 Compare September 16, 2025 16:21
@fjuma fjuma requested a review from cescoffier September 16, 2025 16:28
@fjuma fjuma force-pushed the a2a_content_creation branch from dd0d917 to ce6c23b Compare September 16, 2025 17:52
@cescoffier cescoffier merged commit 89b0bcc into quarkusio:main Sep 17, 2025
1 check passed
@cescoffier
Copy link
Member

Thanks!

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