fix(registry): include server title in search filter so visible labels are searchable (#2289) (#2291)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

Author: toolhive-studio-ci[bot]

renderer/src/routes/(registry)/-registry.route.tsx

@@ -66,7 +66,11 @@ export default function RegistryRouteComponent() {
 
   const { filter, setFilter, filteredData } = useFilterSort({
     data: items,
-    filterFields: (item) => [item.name || '', item.description || ''],
+    filterFields: (item) => [
+      item.name || '',
+      ('title' in item && item.title) || '',
+      item.description || '',
+    ],
     sortBy: (item) => item.name || '',
   })
 

renderer/src/routes/__tests__/registry.test.tsx

@@ -170,3 +170,67 @@ describe('Refresh button (custom registry)', () => {
     expect(screen.getByRole('button', { name: /refresh/i })).toBeVisible()
   })
 })
+
+describe('Bug: search for "github" should match GitHub MCP servers', () => {
+  it('matches an MCP whose user-visible title contains the term ("github") even when name does not', async () => {
+    // The Registry tab renders an MCP's `title` (falling back to `name`) as
+    // the row label — see `table-registry.tsx` (`server.title ?? server.name`)
+    // and `card-registry-server.tsx`. So a user reasonably expects that the
+    // text they see in the UI is what the search input matches against.
+    //
+    // The bug report says searching "github" returns no MCP results. With the
+    // production registry, the GitHub MCP server is displayed with the title
+    // "GitHub" but is technically registered under a non-"github" name (a
+    // packaged variant), so the current filter — which only checks
+    // `name + description` — never sees the visible title and drops the row.
+    mockedGetApiV1BetaRegistryByName.override(
+      (data) =>
+        ({
+          ...data,
+          registry: {
+            ...(data as V1GetRegistryResponse).registry,
+            groups: [],
+          },
+        }) as unknown as V1GetRegistryResponse
+    )
+    mockedGetApiV1BetaRegistryByNameServers.override(() => ({
+      servers: [
+        {
+          name: 'gh-mcp-server',
+          title: 'GitHub',
+          image: 'mcp/gh-server:latest',
+          description: 'Official MCP server for accessing repos and PRs.',
+          repository_url: 'https://github.com/github/github-mcp-server',
+        },
+        {
+          name: 'time',
+          image: 'mcp/time:latest',
+          description: 'Time server.',
+        },
+      ],
+      remote_servers: [],
+    }))
+
+    renderRoute(router)
+
+    await waitFor(() => {
+      expect(screen.queryByText('GitHub')).toBeVisible()
+    })
+
+    const searchInput = screen.getByPlaceholderText('Search...')
+    await userEvent.type(searchInput, 'github')
+
+    // The visible title "GitHub" must remain in the document after typing
+    // "github" — this is the user-facing label of the row the user is looking
+    // for. Currently fails because the filter only inspects name+description.
+    await waitFor(() => {
+      expect(
+        screen.queryByText('GitHub'),
+        'MCP row labelled "GitHub" should remain visible when searching "github"'
+      ).toBeVisible()
+    })
+
+    // The unrelated MCP should be filtered out.
+    expect(screen.queryByText('time')).not.toBeInTheDocument()
+  })
+})