From 0ad8c2d09ca613c6b7b8ab71e9ad44df6dcf3f8e Mon Sep 17 00:00:00 2001
From: Derek Bouius <derek.bouius@gmail.com>
Date: Thu, 5 Mar 2026 17:14:54 -0500
Subject: [PATCH] fix: refresh bank list when dropdown is opened (#504)

Previously, the bank selector dropdown only loaded banks on initial page
load, requiring a full page refresh to see newly created banks. Now calls
loadBanks() each time the popover opens.
---
 hindsight-control-plane/src/components/bank-selector.tsx | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/hindsight-control-plane/src/components/bank-selector.tsx b/hindsight-control-plane/src/components/bank-selector.tsx
index 0c3e7446..34c7173a 100644
--- a/hindsight-control-plane/src/components/bank-selector.tsx
+++ b/hindsight-control-plane/src/components/bank-selector.tsx
@@ -367,7 +367,13 @@ function BankSelectorInner() {
         <div className="h-8 w-px bg-border" />
 
         {/* Memory Bank Selector */}
-        <Popover open={open} onOpenChange={setOpen}>
+        <Popover
+          open={open}
+          onOpenChange={(isOpen) => {
+            setOpen(isOpen);
+            if (isOpen) loadBanks();
+          }}
+        >
           <PopoverTrigger asChild>
             <Button
               variant="outline"