Update src/api/admin.py

src/api/admin.py

@@ -21,16 +21,38 @@ from src.config import get_settings
 from src.services.runtime_store import JsonRuntimeStore
 
 router = APIRouter()
-admin_sessions: Dict[str, float] = {}  # in-memory cache
+admin_sessions: Dict[str, float] = {}
+
+_START_TIME = time.time()
+
+_CONFIG_DEFAULTS: Dict[str, Any] = {
+    "max_results": 5,
+    "maintenance_mode": False,
+    "welcome_message": "Hi! I'm LibBee, the Khalifa University Library AI Assistant.",
+    "custom_instructions": "",
+    "announcement": "",
+    "maintenance_message": "LibBee is currently under maintenance. Please try again shortly.",
+}
+
+
+# ── Session helpers ────────────────────────────────────────────────────────────
+
+def _cleanup_sessions() -> None:
+    now = time.time()
+    expired = [t for t, exp in admin_sessions.items() if exp <= now]
+    for t in expired:
+        admin_sessions.pop(t, None)
+    try:
+        _load_sessions()
+    except Exception:
+        pass
 
 
-# ── Persistent session store (plain JSON — no external dependencies) ──────────
 
 import json as _json
 import os as _os
 
 def _sessions_file() -> str:
-    """Return path for persistent sessions file."""
     cfg_path = _os.environ.get("CONFIG_STORE_PATH", "")
     if cfg_path and "/data" in cfg_path:
         return _os.path.join(_os.path.dirname(cfg_path), "admin_sessions.json")
@@ -38,7 +60,6 @@ def _sessions_file() -> str:
 
 
 def _load_sessions() -> None:
-    """Load persisted sessions into in-memory cache."""
     global admin_sessions
     try:
         with open(_sessions_file(), "r", encoding="utf-8") as _f:
@@ -48,11 +69,10 @@ def _load_sessions() -> None:
                  if isinstance(exp, (int, float)) and exp > now}
         admin_sessions.update(valid)
     except Exception:
-        pass  # file missing or corrupt — start fresh
+        pass
 
 
 def _save_sessions() -> None:
-    """Persist valid sessions to disk."""
     try:
         now = time.time()
         valid = {t: exp for t, exp in admin_sessions.items() if exp > now}
@@ -62,19 +82,6 @@ def _save_sessions() -> None:
         pass
 
 
-# ── Session helpers ────────────────────────────────────────────────────────────
-
-def _cleanup_sessions() -> None:
-    now = time.time()
-    expired = [t for t, exp in list(admin_sessions.items()) if exp <= now]
-    for t in expired:
-        admin_sessions.pop(t, None)
-    try:
-        _load_sessions()
-    except Exception:
-        pass
-
-
 def _require_session(request: Request) -> None:
     _cleanup_sessions()
     token = request.cookies.get("admin_session")
@@ -294,10 +301,11 @@ tr:last-child td{{border-bottom:none}}
   <div id="metricsTable" style="overflow-x:auto"><div style="color:#6b7280;font-size:.84rem">Loading…</div></div>
 </div>
 
+
 <div class="card">
   <h2>📈 Cloudflare Analytics (Persistent)</h2>
-  <div class="stat-grid" id="cfStatsGrid"><div style="color:#6b7280;font-size:.84rem">Loading from Cloudflare D1…</div></div>
-  <div style="margin-top:14px" id="cfToolsTable"><div style="color:#6b7280;font-size:.84rem">Loading tool breakdown…</div></div>
+  <div class="stat-grid" id="cfStatsGrid"><div style="color:#6b7280;font-size:.84rem">Loading…</div></div>
+  <div style="margin-top:14px" id="cfToolsTable"></div>
 </div>
 
 <div class="card">
@@ -305,7 +313,6 @@ tr:last-child td{{border-bottom:none}}
   <div id="recentTable" style="overflow-x:auto"><div style="color:#6b7280;font-size:.84rem">Loading…</div></div>
 </div>
 
-
 <div class="card">
   <h2>🔌 System Status</h2>
   <div id="sysStatus"><div style="color:#6b7280;font-size:.84rem">Loading…</div></div>
@@ -325,7 +332,7 @@ async function loadStats(){{
       <div class="stat"><div class="val">${{d.follow_up_hits||0}}</div><div class="lbl">Follow-ups (session)</div></div>
       <div class="stat"><div class="val">${{d.feedback_total||0}}</div><div class="lbl">Feedback (session)</div></div>
     `;
-  }}catch(e){{document.getElementById('statsGrid').innerHTML='<span style="color:#dc2626">Local metrics unavailable</span>';}}
+  }}catch(e){{document.getElementById('statsGrid').innerHTML='<span style="color:#dc2626">Unavailable</span>';}}
 }}
 
 async function loadCFAnalytics(){{
@@ -343,10 +350,9 @@ async function loadCFAnalytics(){{
     const tools=(d.tools||[]).slice(0,15);
     let rows=tools.map(t=>`<tr><td>${{t.tool_used||"unknown"}}</td><td>${{t.c}}</td></tr>`).join('');
     document.getElementById('cfToolsTable').innerHTML=rows
-      ?`<table><thead><tr><th>Intent:Tool</th><th>Count</th></tr></thead><tbody>${{rows}}</tbody></table>`
-      :'<span style="color:#6b7280;font-size:.84rem">No data yet.</span>';
+      ?`<table><thead><tr><th>Intent:Tool</th><th>Count</th></tr></thead><tbody>${{rows}}</tbody></table>`:'';
   }}catch(e){{
-    document.getElementById('cfStatsGrid').innerHTML='<span style="color:#dc2626">Cloudflare D1 unavailable: '+e.message+'</span>';
+    document.getElementById('cfStatsGrid').innerHTML='<span style="color:#dc2626">CF D1 unavailable: '+e.message+'</span>';
   }}
 }}
 
@@ -357,17 +363,17 @@ async function loadRecentQueries(){{
     if(d.error) throw new Error(d.error);
     const rows=(d.results||[]).map(q=>`<tr>
       <td style="white-space:nowrap;font-size:.74rem">${{q.timestamp||""}}</td>
-      <td style="max-width:280px;word-break:break-word">${{(q.question||"").substring(0,100)}}</td>
+      <td style="max-width:260px;word-break:break-word">${{(q.question||"").substring(0,100)}}</td>
       <td>${{q.tool_used||""}}</td>
       <td>${{q.model||""}}</td>
-      <td style="white-space:nowrap">${{(q.response_time||0).toFixed(2)}}s</td>
+      <td>${{(q.response_time||0).toFixed(2)}}s</td>
       <td>${{q.result_count||0}}</td>
     </tr>`).join('');
     document.getElementById('recentTable').innerHTML=rows
-      ?`<table><thead><tr><th>Time</th><th>Question</th><th>Tool</th><th>Model</th><th>Time</th><th>Results</th></tr></thead><tbody>${{rows}}</tbody></table>`
-      :'<span style="color:#6b7280;font-size:.84rem">No queries logged yet.</span>';
+      ?`<table><thead><tr><th>Time</th><th>Question</th><th>Tool</th><th>Model</th><th>Time</th><th>#</th></tr></thead><tbody>${{rows}}</tbody></table>`
+      :'<span style="color:#6b7280;font-size:.84rem">No queries yet.</span>';
   }}catch(e){{
-    document.getElementById('recentTable').innerHTML='<span style="color:#dc2626">Could not load: '+e.message+'</span>';
+    document.getElementById('recentTable').innerHTML='<span style="color:#dc2626">'+e.message+'</span>';
   }}
 }}
 
@@ -383,8 +389,8 @@ async function loadMetrics(){{
       .map(([k,v])=>`<tr><td style="color:#dc2626">${{k}}</td><td>${{v}}</td><td style="color:#6b7280">error</td></tr>`).join('');
     document.getElementById('metricsTable').innerHTML=rows
       ?`<table><thead><tr><th>Key</th><th>Count</th><th>Type</th></tr></thead><tbody>${{rows}}</tbody></table>`
-      :'<span style="color:#6b7280;font-size:.84rem">No session data yet — resets on deploy.</span>';
-  }}catch(e){{document.getElementById('metricsTable').innerHTML='<span style="color:#dc2626">Could not load metrics</span>';}}
+      :'<span style="color:#6b7280;font-size:.84rem">No session data yet.</span>';
+  }}catch(e){{document.getElementById('metricsTable').innerHTML='<span style="color:#dc2626">Could not load</span>';}}
 }}
 
 async function loadRAGInfo(){{
@@ -444,9 +450,9 @@ loadRecentQueries();
 loadMetrics();
 loadRAGInfo();
 loadStatus();
-setInterval(loadStatus, 30000);
-setInterval(loadCFAnalytics, 60000);
-setInterval(loadRecentQueries, 60000);
+setInterval(loadStatus,30000);
+setInterval(loadCFAnalytics,60000);
+setInterval(loadRecentQueries,60000);
 </script>
 </body></html>""")
 
@@ -455,7 +461,7 @@ setInterval(loadRecentQueries, 60000);
 
 @router.get("/public-config")
 async def public_config():
-    """Public endpoint — no auth required. Returns safe fields for the frontend."""
+    """No auth — returns public fields for the chat frontend."""
     try:
         cfg = _load_config()
     except Exception:
@@ -468,7 +474,6 @@ async def public_config():
     }
 
 
-
 @router.get("/config")
 async def get_config(request: Request):
     _require_session(request)