Merge pull request #13 from lucas-rampimdesouza_data/feature/settings-redesign-and-cache-fix

fix: cache NULL columns + Settings redesign + UI polish

Author: Rampim

backend/app/api/genie_clone_routes.py

@@ -228,6 +228,7 @@ async def _process_genie_background(
                             query_text, query_embedding, sql_query,
                             identity, gateway_id or space_id, rs,
                             original_query_text=original_query_text,
+                            genie_space_id=space_id,
                         )
                         logger.info("Background cache SAVED id=%s query=%s", cache_id, query_text[:50])
                     except Exception as e:

backend/app/services/storage_dynamic.py

@@ -217,6 +217,7 @@ async def _op():
             return backend.save_query_cache(
                 query_text, query_embedding, sql_query, identity, gateway_id,
                 original_query_text=original_query_text,
+                genie_space_id=genie_space_id,
             )
         return await self._with_reconnect(_op, runtime_settings)
 
@@ -231,7 +232,7 @@ async def _op():
             backend = await self._resolve_backend(runtime_settings)
             if hasattr(backend, 'pool'):
                 return await backend.get_all_cached_queries(identity, gateway_id=gateway_id)
-            return backend.get_all_cached_queries(identity)
+            return backend.get_all_cached_queries(identity, genie_space_id=gateway_id)
         return await self._with_reconnect(_op, runtime_settings)
 
     async def save_query_log(

backend/app/services/storage_local.py

@@ -137,8 +137,9 @@ def save_query_cache(
         query_embedding: List[float],
         sql_query: str,
         identity: str,
-        genie_space_id: str,
+        gateway_id: str,
         original_query_text: str = None,
+        genie_space_id: str = None,
     ) -> int:
         """Save a new query to the cache"""
         new_id = max([item.get('id', 0) for item in self.cache], default=0) + 1
@@ -149,6 +150,7 @@ def save_query_cache(
             'original_query_text': original_query_text,
             'sql_query': sql_query,
             'identity': identity,
+            'gateway_id': gateway_id,
             'genie_space_id': genie_space_id,
             'created_at': datetime.now().isoformat(),
             'last_used': datetime.now().isoformat(),

frontend/src/components/gateways/GatewayCacheTab.jsx

@@ -74,7 +74,7 @@ export default function GatewayCacheTab({ gateway }) {
       key: 'query_text',
       label: 'Query',
       render: (val) => (
-        <span className="block truncate max-w-[260px]" title={val}>
+        <span className="block truncate" title={val}>
           {val}
         </span>
       ),
@@ -84,10 +84,10 @@ export default function GatewayCacheTab({ gateway }) {
       label: 'SQL',
       render: (val) => (
         <code
-          className="text-[12px] px-1.5 py-0.5 rounded block truncate max-w-[200px] bg-[#F7F7F7] text-[#161616]"
+          className="text-[12px] px-1.5 py-0.5 rounded block truncate bg-[#F7F7F7] text-[#161616]"
           title={val}
         >
-          {val ? val.substring(0, 60) + (val.length > 60 ? '...' : '') : '-'}
+          {val || '-'}
         </code>
       ),
     },

frontend/src/components/playground/PlaygroundPage.jsx

@@ -583,7 +583,7 @@ export default function PlaygroundPage() {
               id="playground-run-btn"
               onClick={handleSubmit}
               disabled={running || !queryText.trim() || (!selectedGateway && gateways.length > 0)}
-              className="flex items-center justify-center gap-2 h-8 px-4 bg-[#2272B4] text-white rounded text-[13px] font-medium hover:bg-[#1b5e96] disabled:opacity-50 disabled:cursor-not-allowed transition-colors"
+              className="flex items-center justify-center gap-2 h-8 w-[100px] bg-[#2272B4] text-white rounded text-[13px] font-medium hover:bg-[#1b5e96] disabled:opacity-50 disabled:cursor-not-allowed transition-colors"
             >
               {running ? <Loader className="w-3.5 h-3.5 animate-spin" /> : <Play className="w-3.5 h-3.5" />}
               {running ? 'Running' : 'Run'}