Skip to content

Frontend fixes: interim Reports pane never rendered, elapsed timer frozen, Ollama URL field restore #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
cdwashi wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Frontend fixes: interim Reports pane never rendered, elapsed timer frozen, Ollama URL field restore #4

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8f08575
Fix Ollama backend-URL field: env-aware default, working text-input r…
Jul 12, 2026
324b3da
Fix interim Reports panel: guard removed report-preview-card wrapper
Jul 14, 2026
fe0b823
Fix elapsed timer frozen at 00:00: publish window.startTime
Jul 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 21 additions & 5 deletions gui/static/js/app.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down Expand Up @@ -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>
Expand All @@ -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>
Expand Down Expand Up @@ -654,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";
Expand Down Expand Up @@ -694,7 +707,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 };

Expand Down Expand Up @@ -862,7 +878,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 = "";
Expand Down