From 8f085751ae6b4a18c804c2d17ba5d1f95978f010 Mon Sep 17 00:00:00 2001
From: cdogwash72 <cdwashi@gmail.com>
Date: Sun, 12 Jul 2026 20:43:31 +0000
Subject: [PATCH 1/3] Fix Ollama backend-URL field: env-aware default, working
 text-input restore

- Field template no longer hardcodes localhost; uses the server-provided
  provider URL (env-aware since d611295)
- setIfPresent threw on text inputs (Array.from(undefined)), silently
  aborting the whole late-restore batch - now handles input vs select

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 gui/static/js/app.js | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/gui/static/js/app.js b/gui/static/js/app.js
index bf0b316559f..5fc0e15b1f9 100644
--- a/gui/static/js/app.js
+++ b/gui/static/js/app.js
@@ -274,8 +274,16 @@ async function loadProviders() {
     const setIfPresent = (id, val) => {
       const el = document.getElementById(id);
       if (!el || !val) return;
-      const has = Array.from(el.options).some(o => o.value === val);
-      if (has) el.value = val;
+      // <select>: only restore values that exist in the option list.
+      // Text inputs have no .options — restore unconditionally (the old
+      // Array.from(el.options) threw here and silently aborted the rest
+      // of the restore batch).
+      if (el.options) {
+        const has = Array.from(el.options).some(o => o.value === val);
+        if (has) el.value = val;
+      } else {
+        el.value = val;
+      }
     };
     setIfPresent("quick-model-select", saved.quick_model);
     setIfPresent("deep-model-select",  saved.deep_model);
@@ -420,6 +428,10 @@ function renderProviderSettings(providerKey) {
     `;
     document.getElementById("google-thinking").addEventListener("change", markConfigUnsaved);
   } else if (providerKey === "openrouter" || providerKey === "azure" || providerKey === "ollama") {
+    // Server-provided default — env-aware in containerized deployments.
+    // Never hardcode localhost here: runs would silently target the wrong host.
+    const ollamaUrl = (availableProviders.find(p => p.key === "ollama") || {}).url
+                      || "http://localhost:11434/v1";
     container.innerHTML = `
       <div class="provider-option-group">
         <h4>Custom Model Strings</h4>
@@ -436,7 +448,7 @@ function renderProviderSettings(providerKey) {
         ${providerKey === "ollama" ? `
         <div class="form-group" style="margin-top:0.75rem">
           <label>Ollama Server URL</label>
-          <input type="text" id="custom-backend-url" value="http://localhost:11434/v1" placeholder="http://localhost:11434/v1">
+          <input type="text" id="custom-backend-url" value="${ollamaUrl}" placeholder="${ollamaUrl}">
           <span class="form-hint">URL of your local Ollama instance</span>
         </div>` : ""}
       </div>

From 324b3da93ff61aa5965ebebb746fb6cd1bc412cf Mon Sep 17 00:00:00 2001
From: cdogwash72 <cdwashi@gmail.com>
Date: Tue, 14 Jul 2026 01:34:36 +0000
Subject: [PATCH 2/3] Fix interim Reports panel: guard removed
 report-preview-card wrapper

The v2 template redesign removed #report-preview-card but the legacy
renderer still dereferenced it unconditionally in two places:
- renderReportPreview() crashed before drawing any tab, so interim
  analyst reports never appeared during a run
- the run-start reset crashed before clearing currentReportSections,
  leaking previous-run sections into the next run

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 gui/static/js/app.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/gui/static/js/app.js b/gui/static/js/app.js
index 5fc0e15b1f9..0916849ceab 100644
--- a/gui/static/js/app.js
+++ b/gui/static/js/app.js
@@ -706,7 +706,10 @@ function resetUiForNewRun() {
   document.getElementById("stat-llm").textContent = "LLM 0";
   document.getElementById("stat-tools").textContent = "TOOLS 0";
   document.getElementById("stat-tokens").textContent = "TOKENS 0\u2191 0\u2193";
-  document.getElementById("report-preview-card").style.display = "none";
+  // v2 template removed the preview-card wrapper; guard so the reset
+  // below still runs (unguarded, this line killed the whole function).
+  const _rpc = document.getElementById("report-preview-card");
+  if (_rpc) _rpc.style.display = "none";
   currentReportSections = {};
   analysisStats = { llm_calls: 0, tool_calls: 0, tokens_in: 0, tokens_out: 0 };
   
@@ -874,7 +877,7 @@ function handleChunkData(state) {
 
 function renderReportPreview() {
   const card = document.getElementById("report-preview-card");
-  card.style.display = "block";
+  if (card) card.style.display = "block";
   
   const tabsContainer = document.getElementById("report-tabs");
   tabsContainer.innerHTML = "";

From fe0b823ddff7aa94083c7352d2058b202a6cd20c Mon Sep 17 00:00:00 2001
From: cdogwash72 <cdwashi@gmail.com>
Date: Tue, 14 Jul 2026 01:34:36 +0000
Subject: [PATCH 3/3] Fix elapsed timer frozen at 00:00: publish
 window.startTime

Legacy 'let startTime' is lexically scoped, but the v2 timer override
reads window.startTime - always undefined, so the timer never ticked.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 gui/static/js/app.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gui/static/js/app.js b/gui/static/js/app.js
index 0916849ceab..331d61e0180 100644
--- a/gui/static/js/app.js
+++ b/gui/static/js/app.js
@@ -666,6 +666,7 @@ function setUiRunning(isRunning) {
     dot.className = "status-dot running";
     txt.textContent = "RUNNING";
     startTime = Date.now();
+    window.startTime = startTime;  // v2 timer override reads window.startTime
     timerInterval = setInterval(updateTimer, 1000);
   } else {
     runBtn.style.display = "inline-flex";