Skip to content

feat(agent-builder): agent specific skill listing page#258121

Merged
SiddharthMantri merged 14 commits intoelastic:ab-agent-centric-ux-feature-branchfrom
SiddharthMantri:ab/13362-list-agent-skills
Mar 19, 2026
Merged

feat(agent-builder): agent specific skill listing page#258121
SiddharthMantri merged 14 commits intoelastic:ab-agent-centric-ux-feature-branchfrom
SiddharthMantri:ab/13362-list-agent-skills

Conversation

@SiddharthMantri
Copy link
Copy Markdown
Contributor

@SiddharthMantri SiddharthMantri commented Mar 17, 2026 •
edited
Loading

Closes https://github.com/elastic/search-team/issues/13362

Summary

This PR adds a new Skills page for the chosen agent.

Notes:

  • In built capabilties are toggled at an agent level and inject/remove skills to a specific agent
  • If in built skills are enabled, then they are displayed and non-editable for that specific agent
  • If disabled, in built skilsl can be added manually to the agent
  • In built skills are not editable from agent specific UX
  • When new skill is created it is auto added to the global config but also enabled for the current selected agent.
Screen.Recording.2026-03-19.at.11.29.48.mov

@SiddharthMantri
Copy link
Copy Markdown
Contributor Author

SiddharthMantri commented Mar 19, 2026

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Mar 19, 2026

There are no new commits on the base branch.

@SiddharthMantri SiddharthMantri added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting feature:agent-builder Identify agent builder functionalities to be grouped together for release notes labels Mar 19, 2026
@SiddharthMantri SiddharthMantri changed the title feat(agent-builder): listing page for agent centric ux feat(agent-builder): agent specific skill listing page Mar 19, 2026
@SiddharthMantri SiddharthMantri marked this pull request as ready for review March 19, 2026 12:10
@SiddharthMantri SiddharthMantri requested a review from a team as a code owner March 19, 2026 12:10
pgayvallet
pgayvallet approved these changes Mar 19, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@pgayvallet pgayvallet left a comment

Choose a reason for hiding this comment

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

(feature branch, no review)

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Mar 19, 2026 •
edited
Loading

⏳ Build in-progress, with failures

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #66 / Agent Builder converse Conversation Flow "after all" hook for "sends a message with tool call and receives response with thinking"
  • [job] [logs] FTR Configs #66 / Agent Builder converse Conversation Flow "after all" hook for "sends a message with tool call and receives response with thinking"
  • [job] [logs] FTR Configs #66 / Agent Builder converse Conversation Flow navigates to new conversation page and shows initial state
  • [job] [logs] FTR Configs #66 / Agent Builder converse Conversation Flow navigates to new conversation page and shows initial state
  • [job] [logs] Jest Tests #10 / route_config getSidebarViewForRoute agent routes returns "agentSettings" for tools route
  • [job] [logs] Jest Tests #10 / route_config getSidebarViewForRoute agent routes returns "agentSettings" for tools route

History

@SiddharthMantri SiddharthMantri merged commit f786a08 into elastic:ab-agent-centric-ux-feature-branch Mar 19, 2026
13 of 14 checks passed
chrisbmar
chrisbmar reviewed Mar 19, 2026
View reviewed changes
x-pack/platform/plugins/shared/agent_builder/public/application/hooks/skills/use_skills.ts
const { data, isLoading, error, isError } = useQuery({
queryKey: queryKeys.skills.all,
queryFn: () => skillsService.list(),
keepPreviousData: true,
Copy link
Copy Markdown
Contributor

@chrisbmar chrisbmar Mar 19, 2026

Choose a reason for hiding this comment

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

why do we need this?

Copy link
Copy Markdown
Contributor Author

@SiddharthMantri SiddharthMantri Mar 19, 2026

Choose a reason for hiding this comment

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

the refetch was causing the page to flash into loading briefly - so this keeps previous data while the new one is refetched

@chrisbmar chrisbmar mentioned this pull request Mar 30, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chrisbmar chrisbmar chrisbmar left review comments

@pgayvallet pgayvallet pgayvallet approved these changes

Assignees

No one assigned

Labels

backport:skip This PR does not require backporting feature:agent-builder Identify agent builder functionalities to be grouped together for release notes release_note:skip Skip the PR/issue when compiling release notes v9.4.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@SiddharthMantri @elasticmachine @pgayvallet @chrisbmar @kibanamachine