From db2ecca45d8a4c6d94d2c534c913e419a3efe7ce Mon Sep 17 00:00:00 2001
From: Daniel <danitex@danvics.com>
Date: Fri, 8 May 2026 08:28:54 +0200
Subject: [PATCH] remove top bar model selector

---
 public/index.html           |  5 -----
 public/js/app.js            | 19 ++-----------------
 test/model-defaults.test.js |  9 +++++++++
 3 files changed, 11 insertions(+), 22 deletions(-)

diff --git a/public/index.html b/public/index.html
index e280f42..8f8dd24 100644
--- a/public/index.html
+++ b/public/index.html
@@ -172,11 +172,6 @@
             </div>
           </div>
           <div class="header-right">
-            <div class="model-selector">
-              <label><i class="fas fa-robot"></i></label>
-              <select id="global-model-select"></select>
-              <span id="model-cost-badge" class="cost-badge" style="display:none;"></span>
-            </div>
             <div class="header-buttons">
               <button id="btn-settings" class="btn-header" title="Settings">
                 <i class="fas fa-cog"></i>
diff --git a/public/js/app.js b/public/js/app.js
index c4de0c0..fdf3ede 100644
--- a/public/js/app.js
+++ b/public/js/app.js
@@ -182,9 +182,7 @@ document.addEventListener('DOMContentLoaded', function() {
     }
   });
 
-  // --- MODEL SELECTOR ---
-  var modelSelect = document.getElementById('global-model-select');
-  var costBadge = document.getElementById('model-cost-badge');
+  // --- MODEL SELECTORS ---
   window._currentModels = [];
   window._currentProvider = 'openrouter';
   window._defaultModelId = '';
@@ -216,11 +214,6 @@ document.addEventListener('DOMContentLoaded', function() {
       var defaultModelId = data.defaultModel || (window._currentModels.length > 0 ? window._currentModels[0].id : '');
       window._defaultModelId = defaultModelId;
 
-      if (modelSelect && window._currentModels.length > 0) {
-        window._buildModelOptions(modelSelect);
-        if (costBadge) costBadge.textContent = '';
-      }
-
       // Populate all per-tab model selectors already in DOM
       document.querySelectorAll('.tab-model-select').forEach(function(sel) {
         window._buildModelOptions(sel);
@@ -228,13 +221,6 @@ document.addEventListener('DOMContentLoaded', function() {
     })
     .catch(function(err) { console.warn('Models load failed:', err); });
 
-  if (modelSelect) {
-    modelSelect.addEventListener('change', function() {
-      var m = window._currentModels.find(function(x) { return x.id === modelSelect.value; });
-      showToast('Model: ' + modelSelect.value.split('/').pop(), 'info');
-    });
-  }
-
   console.log('✅ App.js DOM ready');
 
 }); // end DOMContentLoaded
@@ -331,8 +317,7 @@ function getSelectedModel() {
     var tabSel = activeTab.querySelector('.tab-model-select');
     if (tabSel && tabSel.value) return tabSel.value;
   }
-  var sel = document.getElementById('global-model-select');
-  return sel ? sel.value : 'google/gemini-2.5-flash';
+  return undefined;
 }
 
 function showLoading(text) {
diff --git a/test/model-defaults.test.js b/test/model-defaults.test.js
index bd3ef94..329ea4f 100644
--- a/test/model-defaults.test.js
+++ b/test/model-defaults.test.js
@@ -18,11 +18,20 @@ test('api models endpoint preserves saved default in model list', () => {
 
 test('lazy-loaded tab model selectors apply admin default', () => {
   const app = read('public/js/app.js');
+  const index = read('public/index.html');
   assert.match(app, /window\._defaultModelId = ''/);
   assert.match(app, /window\._defaultModelId = defaultModelId/);
   assert.match(app, /window\._buildModelOptions\(sel\)/);
   assert.match(app, /selectEl\.value = window\._defaultModelId/);
   assert.match(app, /saved\.textContent = window\._defaultModelId \+ ' \(saved default\)'/);
+  assert.doesNotMatch(index, /global-model-select|model-cost-badge/);
+});
+
+test('top bar model selector is removed and omitted selections use backend default', () => {
+  const app = read('public/js/app.js');
+  const selector = app.match(/function getSelectedModel\(\) \{[\s\S]*?\n\}/)[0];
+  assert.doesNotMatch(selector, /global-model-select|gemini-2\.5-flash/);
+  assert.match(selector, /return undefined/);
 });
 
 test('backend AI calls use admin default when request omits model', () => {