Skip to content

Update agent builder docs #3206

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 26 commits into from
Sep 29, 2025
Merged

Update agent builder docs #3206

merged 26 commits into from
Sep 29, 2025

Conversation

leemthompo
Copy link
Contributor

@leemthompo leemthompo commented Sep 29, 2025 •
edited
Loading

@leemthompo
Update API examples
a3f1bbb
@leemthompo
populate chat content, cleanup
85ad4c8 
warning headers: replaced "wip" with standardized warning text and landing page links across all agent builder docs
content updates: enhanced agent descriptions, added chat ui screenshots and detailed gui documentation
api examples: added "example:" labels to all api code blocks for clarity
formatting: improved code block formatting, removed extra whitespace, added proper environment variable documentation
structure: reorganized kibana api page with better intro text and moved api reference link to bottom
commented out: a2a execute task section pending completion
cross-references: added proper internal doc links between related pages
@leemthompo leemthompo self-assigned this Sep 29, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 29, 2025 •
edited
Loading

🔍 Preview links for changed docs

@leemthompo leemthompo requested a review from a team September 29, 2025 12:30
@leemthompo leemthompo marked this pull request as ready for review September 29, 2025 12:30
@leemthompo leemthompo requested a review from a team as a code owner September 29, 2025 12:30
kosabogi
kosabogi approved these changes Sep 29, 2025
View reviewed changes
Copy link
Contributor

@kosabogi kosabogi left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 with two tiny nits. I’ve suggested one of them. The other I can’t suggest in this PR because it’s not part of this change:

In the agent-builder-agents file, the sentence “Add a Display description to explain the agent's purpose” is missing a period.

marciw
marciw approved these changes Sep 29, 2025
View reviewed changes
Copy link
Contributor

@marciw marciw left a comment

Choose a reason for hiding this comment

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

LGTM - just some very optional nits.

leemthompo
leemthompo commented Sep 29, 2025
View reviewed changes
@leemthompo leemthompo merged commit 7ba2d42 into main Sep 29, 2025
8 checks passed
@leemthompo leemthompo deleted the leemthompo/agent-builder-chat-etc branch September 29, 2025 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marciw marciw marciw approved these changes

@kosabogi kosabogi kosabogi approved these changes

Assignees

@leemthompo leemthompo

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@leemthompo @marciw @kosabogi