From e6c776d508809af5b3053f9833164541d64c2454 Mon Sep 17 00:00:00 2001
From: Yuneng Jiang <yuneng@berri.ai>
Date: Tue, 14 Jul 2026 19:52:43 -0700
Subject: [PATCH] refactor(ui): drop whole-row navigation on the skills table

Only the name cell and the overflow menu act on a row, matching the unified
table pattern; the previous table navigated on any row click
---
 .../app/(dashboard)/skills/_components/PluginTable.test.tsx   | 4 ++--
 .../src/app/(dashboard)/skills/_components/PluginTable.tsx    | 1 -
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/ui/litellm-dashboard/src/app/(dashboard)/skills/_components/PluginTable.test.tsx b/ui/litellm-dashboard/src/app/(dashboard)/skills/_components/PluginTable.test.tsx
index 3c9d46e26fd..dd81a96d80a 100644
--- a/ui/litellm-dashboard/src/app/(dashboard)/skills/_components/PluginTable.test.tsx
+++ b/ui/litellm-dashboard/src/app/(dashboard)/skills/_components/PluginTable.test.tsx
@@ -69,11 +69,11 @@ describe("PluginTable", () => {
     expect(mockOnPluginClick).toHaveBeenCalledWith("plugin-id-newer");
   });
 
-  it("should call onPluginClick with the plugin ID when the row is clicked", async () => {
+  it("should not navigate when clicking elsewhere in the row", async () => {
     const user = userEvent.setup();
     render(<PluginTable {...defaultProps} />);
     await user.click(screen.getByText("A skill for testing"));
-    expect(mockOnPluginClick).toHaveBeenCalledWith("plugin-id-newer");
+    expect(mockOnPluginClick).not.toHaveBeenCalled();
   });
 
   it("should badge the category and fall back to Uncategorized", () => {
diff --git a/ui/litellm-dashboard/src/app/(dashboard)/skills/_components/PluginTable.tsx b/ui/litellm-dashboard/src/app/(dashboard)/skills/_components/PluginTable.tsx
index 9078124c7f5..7cbe53e609c 100644
--- a/ui/litellm-dashboard/src/app/(dashboard)/skills/_components/PluginTable.tsx
+++ b/ui/litellm-dashboard/src/app/(dashboard)/skills/_components/PluginTable.tsx
@@ -48,7 +48,6 @@ const PluginTable: React.FC<PluginTableProps> = ({ pluginsList, isLoading, onDel
       sortingMode="client"
       sorting={sorting}
       onSortingChange={setSorting}
-      onRowClick={(plugin) => onPluginClick(plugin.id)}
       isLoading={isLoading}
       loadingMessage="Loading skills…"
       noDataMessage={<EmptyState />}