[Agent Builder] Migrate existing routes to new UX by chrisbmar · Pull Request #258241 · elastic/kibana

chrisbmar · 2026-03-17T21:55:16Z

closes https://github.com/elastic/search-team/issues/13361

Migrates the existing /tools, /skills routes etc. to be within the /manage/tools, /manage/skills routes etc.
Fixes multiple small things related to react-query caching and agentId being a query parameter before - now it's in the path
Adds the agent-scoped conversations to the conversation sidebar

See video below for an example of what's been migrated (note: there are still many routes that do not have a UI yet, this is deliberate and they will be handled in future PRs)

Screen.Recording.2026-03-17.at.21.02.05.mov

chrisbmar · 2026-03-18T07:28:25Z

...blic/application/components/conversations/conversation_input/input_actions/input_actions.tsx

+            {isEmbeddedContext && (
+              <EuiFlexItem grow={false}>
+                <EuiTourStep {...getStepProps(TourStep.AgentSelector)}>
+                  <AgentSelector agentId={agentId} />
+                </EuiTourStep>
+              </EuiFlexItem>
+            )}


Just hides the old AgentSelector from the conversation input when in full-screen mode. In a future ticket we will remove this entirely when it moves to the header of the sidebar.

chrisbmar · 2026-03-18T07:29:56Z

...ent_builder/public/application/components/layout/unified_sidebar/views/conversation_view.tsx


 import React, { useCallback } from 'react';
 import { Link } from 'react-router-dom-v5-compat';
+import { useLocation } from 'react-router-dom';


This file just has updates because I had to add the conversations to the sidebar to make sure navigation worked. Actually fixing the UI for the sidebar will be done in a separate PR so I wouldn't pay too much attention to this one.

chrisbmar · 2026-03-18T07:30:19Z

...der/public/application/components/layout/unified_sidebar/views/sidebar_conversation_list.tsx

+  currentConversationId,
+}) => {
+  const { euiTheme } = useEuiTheme();
+  const { conversations = [], isLoading } = useConversationList({ agentId });


Only gets and lists the conversations for the agent in context.

chrisbmar · 2026-03-18T07:32:10Z

x-pack/platform/plugins/shared/agent_builder/public/application/utils/app_paths.ts

+  legacy: {
+    conversation: ({ conversationId }: { conversationId: string }) =>
+      `/conversations/${conversationId}`,
+  },
+
+  // Backward compatibility aliases pointing to new routes
+  // TODO: Migrate consumers to use appPaths.agent.* or appPaths.manage.* directly and remove these aliases
+  chat: {
+    new: '/agents/elastic-default',
+    newWithAgent: ({ agentId }: { agentId: string }) => `/agents/${agentId}`,
+  },
+  agents: {
+    list: '/manage/agents',
+    new: '/manage/agents/new',
+    edit: ({ agentId }: { agentId: string }) => `/manage/agents/${agentId}`,
+  },
+  tools: {
+    list: '/manage/tools',
+    new: '/manage/tools/new',
+    details: ({ toolId }: { toolId: string }) => `/manage/tools/${toolId}`,
+    bulkImportMcp: '/manage/tools/bulk_import_mcp',
+  },
+  skills: {
+    list: '/manage/skills',
+    new: '/manage/skills/new',
+    details: ({ skillId }: { skillId: string }) => `/manage/skills/${skillId}`,
+  },
+  plugins: {
+    list: '/manage/plugins',
+    details: ({ pluginId }: { pluginId: string }) => `/manage/plugins/${pluginId}`,
+  },


So right now, we either add these BWC legacy paths or update 43 files throughout our code to use the new paths directly - I chose the easiest for now 😄 lets see if this new UI ever lands before we consider this change.

chrisbmar · 2026-03-18T07:35:12Z

x-pack/platform/plugins/shared/agent_builder/public/application/route_config.tsx

+export const getConversationIdFromPath = (pathname: string): string | undefined => {
+  const match = pathname.match(/^\/agents\/[^/]+\/conversations\/([^/]+)/);
+  return match ? match[1] : undefined;
+};


Used only in the sidebar, as it's rendered outside the context of the conversation related hooks we cannot use useConversationId there. Sidebar is rendered at the very top as it's persisted on every route, even non-conversation ones.

elasticmachine · 2026-03-18T08:28:32Z

💔 Build Failed

Failed CI Steps

Test Failures

[job] [logs] FTR Configs #38 / Agent Builder converse Conversation Flow "after all" hook for "sends a message with tool call and receives response with thinking"
[job] [logs] FTR Configs #38 / Agent Builder converse Conversation Flow "after all" hook for "sends a message with tool call and receives response with thinking"
[job] [logs] FTR Configs #38 / Agent Builder converse Conversation Flow navigates to new conversation page and shows initial state
[job] [logs] FTR Configs #38 / Agent Builder converse Conversation Flow navigates to new conversation page and shows initial state
[job] [logs] FTR Configs #50 / lens app - group 5 lens formula should duplicate a moving average formula and be a valid table with conditional coloring
[job] [logs] Jest Tests #1 / route_config getSidebarViewForRoute agent routes returns "agentSettings" for tools route
[job] [logs] Jest Tests #1 / route_config getSidebarViewForRoute agent routes returns "agentSettings" for tools route

Metrics [docs]

‼️ ERROR: no builds found for mergeBase sha [8b2cb0a]

History

cc @chrisbmar

SiddharthMantri · 2026-03-18T09:55:02Z

x-pack/platform/plugins/shared/agent_builder/public/application/utils/app_paths.ts

+  // Backward compatibility aliases pointing to new routes
+  // TODO: Migrate consumers to use appPaths.agent.* or appPaths.manage.* directly and remove these aliases
+  chat: {
+    new: '/agents/elastic-default',


question: should this be elastic-ai-agent and not elastic-default?

these should be removed in favour of just navigating to the root now, I've removed them here.

SiddharthMantri

LGTM overall. Left a couple of questions but nothing major.

SiddharthMantri · 2026-03-18T09:59:59Z

...ic/application/components/conversations/conversations_history/conversations_history_list.tsx

      } else {
-        navigateToAgentBuilderUrl(appPaths.chat.conversation({ conversationId: conversation.id }));
+        // Use legacy path - LegacyConversationRedirect fetches conversation and redirects to correct agent-scoped URL
+        navigateToAgentBuilderUrl(


Question: could this be replced by:

navigateToAgentBuilderUrl( appPaths.agent.conversations.byId({ agentId: conversation.agent_id, conversationId: conversation.id, }) );

Since we have agent_id in the conversation response already?

yes that's exactly what we will do when we remove/recreate this component in the header of the sidebar, good spot!

chrisbmar added 5 commits March 17, 2026 12:35

update route config existing routes with the correct components

dd27519

update app paths for bwc

b450621

updates to support url structure

3ad1820

sidebar conversation list wip

c977817

conversation view wip - tbc in future pr

2ff8033

chrisbmar self-assigned this Mar 17, 2026

chrisbmar added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Mar 17, 2026

chrisbmar commented Mar 18, 2026

View reviewed changes

chrisbmar marked this pull request as ready for review March 18, 2026 07:40

chrisbmar requested a review from a team as a code owner March 18, 2026 07:40

SiddharthMantri self-requested a review March 18, 2026 09:29

SiddharthMantri reviewed Mar 18, 2026

View reviewed changes

SiddharthMantri approved these changes Mar 18, 2026

View reviewed changes

chrisbmar added 2 commits March 18, 2026 14:08

validate agent id

afdb9ed

remove unused code

5949c9c

chrisbmar merged commit 85cf346 into elastic:ab-agent-centric-ux-feature-branch Mar 18, 2026
7 of 9 checks passed

chrisbmar mentioned this pull request Mar 30, 2026

[Agent Builder] Agent-centric UX redesign #258005

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Agent Builder] Migrate existing routes to new UX#258241

[Agent Builder] Migrate existing routes to new UX#258241
chrisbmar merged 7 commits intoelastic:ab-agent-centric-ux-feature-branchfrom
chrisbmar:ab-13361-migrate-existing-routes

chrisbmar commented Mar 17, 2026 •

edited

Loading

Uh oh!

chrisbmar Mar 18, 2026

Uh oh!

chrisbmar Mar 18, 2026

Uh oh!

chrisbmar Mar 18, 2026

Uh oh!

chrisbmar Mar 18, 2026

Uh oh!

chrisbmar Mar 18, 2026

Uh oh!

elasticmachine commented Mar 18, 2026 •

edited

Loading

Uh oh!

SiddharthMantri Mar 18, 2026

Uh oh!

chrisbmar Mar 18, 2026

Uh oh!

SiddharthMantri left a comment

Uh oh!

SiddharthMantri Mar 18, 2026

Uh oh!

chrisbmar Mar 18, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

chrisbmar commented Mar 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

elasticmachine commented Mar 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

💔 Build Failed

Failed CI Steps

Test Failures

Metrics [docs]

History

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

SiddharthMantri left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

chrisbmar commented Mar 17, 2026 •

edited

Loading

elasticmachine commented Mar 18, 2026 •

edited

Loading