From 430f3ddc3492a59d0014f07c3be268bdc5318ec9 Mon Sep 17 00:00:00 2001
From: mbecker20 <becker.maxh@gmail.com>
Date: Wed, 29 Oct 2025 00:02:32 -0700
Subject: [PATCH] fix omni search container double select on same name

---
 frontend/src/components/omnibar.tsx | 72 ++++++++++++++++-------------
 1 file changed, 39 insertions(+), 33 deletions(-)

diff --git a/frontend/src/components/omnibar.tsx b/frontend/src/components/omnibar.tsx
index bf2569dbe..dff97f931 100644
--- a/frontend/src/components/omnibar.tsx
+++ b/frontend/src/components/omnibar.tsx
@@ -272,25 +272,28 @@ const OmniContainers = ({
     <>
       <CommandSeparator />
       <CommandGroup heading="Containers">
-        {containers?.map((container) => (
-          <CommandItem
-            key={container.id}
-            value={container.name}
-            className="flex items-center gap-2 cursor-pointer"
-            onSelect={() => {
-              closeSearch();
-              navigate(
-                `/servers/${container.server_id!}/container/${container.name}`
-              );
-            }}
-          >
-            <DOCKER_LINK_ICONS.container
-              server_id={container.server_id!}
-              name={container.name}
-            />
-            {container.name}
-          </CommandItem>
-        ))}
+        {containers?.map((container) => {
+          const key = container.server_id + container.name;
+          return (
+            <CommandItem
+              key={key}
+              value={key}
+              className="flex items-center gap-2 cursor-pointer"
+              onSelect={() => {
+                closeSearch();
+                navigate(
+                  `/servers/${container.server_id!}/container/${container.name}`
+                );
+              }}
+            >
+              <DOCKER_LINK_ICONS.container
+                server_id={container.server_id!}
+                name={container.name}
+              />
+              {container.name}
+            </CommandItem>
+          );
+        })}
       </CommandGroup>
     </>
   );
@@ -323,20 +326,23 @@ const OmniTerminals = ({
     <>
       <CommandSeparator />
       <CommandGroup heading="Terminals">
-        {terminals?.map((terminal) => (
-          <CommandItem
-            key={JSON.stringify(terminal.target) + terminal.name}
-            value={JSON.stringify(terminal.target) + terminal.name}
-            className="flex items-center gap-2 cursor-pointer"
-            onSelect={() => {
-              closeSearch();
-              navigate(terminalLink(terminal));
-            }}
-          >
-            <Terminal className="w-4 h-4" />
-            {terminal.name}
-          </CommandItem>
-        ))}
+        {terminals?.map((terminal) => {
+          const key = JSON.stringify(terminal.target) + terminal.name;
+          return (
+            <CommandItem
+              key={key}
+              value={key}
+              className="flex items-center gap-2 cursor-pointer"
+              onSelect={() => {
+                closeSearch();
+                navigate(terminalLink(terminal));
+              }}
+            >
+              <Terminal className="w-4 h-4" />
+              {terminal.name}
+            </CommandItem>
+          );
+        })}
       </CommandGroup>
     </>
   );