open-agents-ai 0.187.221 → 0.187.223

package/dist/index.js

@@ -317580,6 +317580,10 @@ const conv = document.getElementById('conversation');
 const input = document.getElementById('input-area');
 const sendBtn = document.getElementById('send-btn');
 const modelSelect = document.getElementById('model-select');
+// Persist the selected model on every change so a browser refresh /
+// daemon restart can recall it. The change handler is added immediately
+// because modelSelect already exists in the DOM at this point.
+modelSelect.addEventListener('change', () => persistSelectedModel());
 const statusEl = document.getElementById('status');
 let apiKey = localStorage.getItem('oa-api-key') || '';
 let streaming = false;
@@ -317619,25 +317623,66 @@ async function checkHealth() {
   }
 }
 
-// Load models
+// Load models. Recall the previously-selected model in this priority:
+//   1. Daemon's persisted config.model (server-side, survives daemon restart)
+//   2. localStorage 'oa.selectedModel' (client-side, survives browser refresh
+//      even before the daemon answers /v1/config)
+//   3. First model whose name contains "9b" (sensible default)
+//   4. First model overall
 async function loadModels() {
   try {
-    const r = await fetch('/v1/models', { headers: headers() });
-    const d = await r.json();
+    const [modelsResp, cfgResp] = await Promise.all([
+      fetch('/v1/models', { headers: headers() }).then(r => r.json()).catch(() => ({})),
+      fetch('/v1/config', { headers: headers() }).then(r => r.json()).catch(() => ({})),
+    ]);
     modelSelect.innerHTML = '';
-    for (const m of (d.data || [])) {
+    for (const m of (modelsResp.data || [])) {
       const opt = document.createElement('option');
       // Strip "local/" prefix for cleaner display
       opt.value = m.id.replace(/^local\\//, '');
       opt.textContent = m.id.replace(/^local\\//, '');
       modelSelect.appendChild(opt);
     }
-    // Default to first model with "9b" in name, or first overall
-    const preferred = Array.from(modelSelect.options).find(o => /9b/i.test(o.value));
-    if (preferred) modelSelect.value = preferred.value;
+    // Resolve the recalled model in priority order
+    const daemonModel = (cfgResp && cfgResp.config && cfgResp.config.model) || '';
+    let cachedModel = '';
+    try { cachedModel = localStorage.getItem('oa.selectedModel') || ''; } catch {}
+    const candidates = [daemonModel, cachedModel].filter(Boolean);
+    let recalled = null;
+    for (const c of candidates) {
+      const stripped = c.replace(/^local\\//, '');
+      const hit = Array.from(modelSelect.options).find(o => o.value === stripped);
+      if (hit) { recalled = hit.value; break; }
+    }
+    if (recalled) {
+      modelSelect.value = recalled;
+    } else {
+      // Fall back to first 9b model, then first overall
+      const preferred = Array.from(modelSelect.options).find(o => /9b/i.test(o.value));
+      if (preferred) modelSelect.value = preferred.value;
+    }
+    // Persist the resolved value back to localStorage so a subsequent
+    // refresh (before /v1/config has answered) can recall it instantly.
+    try { localStorage.setItem('oa.selectedModel', modelSelect.value); } catch {}
   } catch { modelSelect.innerHTML = '<option>error loading models</option>'; }
 }
 
+// Persist user changes immediately and push to the daemon so a daemon
+// restart also recalls the same model. The change handler is wired
+// after DOMContentLoaded — see the addEventListener call further down.
+function persistSelectedModel() {
+  const v = modelSelect.value;
+  if (!v) return;
+  try { localStorage.setItem('oa.selectedModel', v); } catch {}
+  // Best-effort: push to /v1/config/model so the daemon also persists it.
+  // Don't await — we don't want to block the UI on a slow daemon write.
+  fetch('/v1/config/model', {
+    method: 'PUT',
+    headers: headers(),
+    body: JSON.stringify({ model: v }),
+  }).catch(() => { /* offline / read-only — localStorage still saved */ });
+}
+
 function addMessage(role, content) {
   const div = document.createElement('div');
   div.className = 'msg ' + role;
@@ -317759,6 +317804,45 @@ function escHtml(s) {
   return String(s == null ? '' : s).replace(/&/g,'&amp;').replace(/</g,'&lt;').replace(/>/g,'&gt;').replace(/"/g,'&quot;').replace(/'/g,'&#39;');
 }
 
+// Append text content to a parent element with a "Show more" / "Hide"
+// toggle when the text exceeds truncateAt characters. The truncated
+// preview is shown by default; the toggle button sits UNDERNEATH the
+// content (per user request) and swaps between expand/collapse states.
+// Returns the wrapper div so callers can chain styles.
+function appendExpandableContent(parent, fullText, opts) {
+  opts = opts || {};
+  const truncateAt = typeof opts.truncateAt === 'number' ? opts.truncateAt : 500;
+  const monospace = opts.monospace !== false; // default true
+  const baseStyle = (opts.baseStyle || '') +
+    (monospace ? 'font-family:monospace;white-space:pre-wrap;word-break:break-word;' : 'white-space:pre-wrap;');
+  const wrapper = document.createElement('div');
+  wrapper.style.cssText = 'display:flex;flex-direction:column;gap:2px;';
+  const text = String(fullText == null ? '' : fullText);
+  const isLong = text.length > truncateAt;
+
+  const contentEl = document.createElement('div');
+  contentEl.style.cssText = baseStyle;
+  contentEl.textContent = isLong ? text.slice(0, truncateAt) + '\\u2026' : text;
+  wrapper.appendChild(contentEl);
+
+  if (isLong) {
+    const btn = document.createElement('button');
+    btn.type = 'button';
+    btn.style.cssText = 'align-self:flex-start;margin-top:2px;padding:2px 8px;background:#2a2a30;border:1px solid #3a3a42;color:#b2920a;font-size:0.6rem;border-radius:2px;cursor:pointer;font-family:inherit;';
+    btn.textContent = 'Show more (' + text.length + ' chars)';
+    let expanded = false;
+    btn.addEventListener('click', (e) => {
+      e.stopPropagation();
+      expanded = !expanded;
+      contentEl.textContent = expanded ? text : (text.slice(0, truncateAt) + '\\u2026');
+      btn.textContent = expanded ? 'Hide' : 'Show more (' + text.length + ' chars)';
+    });
+    wrapper.appendChild(btn);
+  }
+  parent.appendChild(wrapper);
+  return wrapper;
+}
+
 async function sendMessage() {
   const text = input.value.trim();
   if (!text || streaming) return;
@@ -317855,6 +317939,27 @@ async function sendMessage() {
         try {
           const chunk = JSON.parse(data);
 
+          // task_complete promotion: surface its summary as the final
+          // assistant message instead of leaving it stuck inside the
+          // tool dropdown. The agent uses task_complete to deliver its
+          // final answer; without this hoist the user sees an empty
+          // assistant bubble and a collapsed details element they have
+          // to expand to read the result.
+          if (chunk.type === 'tool_call' && chunk.tool === 'task_complete') {
+            const ta = (chunk.args && typeof chunk.args === 'object') ? chunk.args : {};
+            const summaryText = (typeof ta.summary === 'string' && ta.summary)
+              || (typeof ta.message === 'string' && ta.message)
+              || (typeof ta.result === 'string' && ta.result)
+              || '';
+            if (summaryText) {
+              if (fullContent && !fullContent.endsWith('\\n')) fullContent += '\\n\\n';
+              fullContent += summaryText;
+              contentDiv.innerHTML = renderMarkdown(fullContent);
+              conv.scrollTop = conv.scrollHeight;
+            }
+            // fall through so the dropdown still renders for inspection
+          }
+
           // Tool call event — show live as expandable section
           if (chunk.type === 'tool_call') {
             chatTools.push(chunk);
@@ -317936,14 +318041,26 @@ async function sendMessage() {
               } else {
                 for (const [k, v] of Object.entries(chunk.args)) {
                   const row = document.createElement('div');
-                  row.style.cssText = 'padding:2px 0;display:flex;gap:8px';
+                  row.style.cssText = 'padding:2px 0;display:flex;gap:8px;align-items:flex-start';
+                  // Convert value to a displayable string with proper typing
                   let vs;
                   if (v === null || v === undefined) vs = String(v);
                   else if (typeof v === 'string') vs = v;
                   else if (typeof v === 'number' || typeof v === 'boolean') vs = String(v);
                   else { try { vs = JSON.stringify(v, null, 2); } catch { vs = '[object]'; } }
-                  if (vs.length > 500) vs = vs.slice(0, 497) + '…';
-                  row.innerHTML = '<span style="color:#b2920a;min-width:60px">' + k + '</span><span style="color:#b0b0b0;word-break:break-all;white-space:pre-wrap">' + escHtml(vs) + '</span>';
+
+                  const keyEl = document.createElement('span');
+                  keyEl.style.cssText = 'color:#b2920a;min-width:60px;flex-shrink:0';
+                  keyEl.textContent = k;
+                  row.appendChild(keyEl);
+
+                  const valWrap = document.createElement('div');
+                  valWrap.style.cssText = 'flex:1;min-width:0;color:#b0b0b0';
+                  // Use the show-more helper so long values are collapsed
+                  // by default and the user can expand inline.
+                  appendExpandableContent(valWrap, vs, { truncateAt: 500, baseStyle: 'color:#b0b0b0;' });
+                  row.appendChild(valWrap);
+
                   argsDiv.appendChild(row);
                 }
               }
@@ -317954,11 +318071,13 @@ async function sendMessage() {
             continue;
           }
 
-          // Tool result
+          // Tool result — render with show-more/hide so the user can
+          // expand to see the FULL output instead of being capped at
+          // 150 chars. The button sits underneath the result block.
           if (chunk.type === 'tool_result') {
             const resultEl = document.createElement('div');
-            resultEl.style.cssText = 'background:#1e1e22;padding:2px 8px 2px 18px;margin:0 0 2px 0;color:#555;font-size:0.65rem;max-height:60px;overflow:hidden';
-            resultEl.textContent = (chunk.output || '').slice(0, 150);
+            resultEl.style.cssText = 'background:#1e1e22;padding:4px 8px 4px 18px;margin:0 0 2px 0;color:#888;font-size:0.65rem';
+            appendExpandableContent(resultEl, chunk.output || '', { truncateAt: 150, baseStyle: 'color:#888;' });
             toolsContainer.appendChild(resultEl);
             continue;
           }
@@ -322452,6 +322571,15 @@ ${task}` : task;
               done: false,
               _oa: { type: "tool_call", tool: evt.tool, args: evt.args }
             }) + "\n");
+            try {
+              const evtType = evt.tool === "sub_agent" || evt.tool === "full_sub_agent" ? "run.started" : "tool.called";
+              publishEvent(evtType, {
+                run_id: `ollama-${evt.tool}-${Date.now()}`,
+                tool: evt.tool,
+                parent: "chat"
+              });
+            } catch {
+            }
           } else {
             finalLines.push(line);
           }
@@ -323757,6 +323885,16 @@ ${historyLines}
                   tool: evt.tool,
                   args: evt.args
                 }) + "\n\n");
+                try {
+                  const evtType = evt.tool === "sub_agent" || evt.tool === "full_sub_agent" ? "run.started" : "tool.called";
+                  publishEvent(evtType, {
+                    run_id: `${session.id.slice(0, 12)}-${evt.tool}-${Date.now()}`,
+                    tool: evt.tool,
+                    session_id: session.id,
+                    parent: "chat"
+                  }, { subject: session.id });
+                } catch {
+                }
               } else {
                 finalLines.push(line);
               }
@@ -324768,7 +324906,7 @@ function createTaskCompleteTool(modelTier) {
   const summaryDesc = modelTier === "small" || modelTier === "medium" ? "Your complete response to the user. For questions/chat: put your FULL answer here (this is what the user will see). For coding tasks: brief summary of what was accomplished." : "Brief summary of what was accomplished";
   return {
     name: "task_complete",
-    description: "Signal that the task is complete.",
+    description: "Signal that the task is complete. GUARDED: cannot fire while the active todo list (todo_write) has pending, in_progress, or blocked items. If you're truly done, first call todo_write to mark every remaining item completed. If you're not done, continue working down the list and call this only after the last item flips to completed.",
     parameters: {
       type: "object",
       properties: {
@@ -324777,6 +324915,35 @@ function createTaskCompleteTool(modelTier) {
       required: ["summary"]
     },
     async execute(args) {
+      try {
+        const sessionId = getTodoSessionId();
+        const todos = readTodos(sessionId);
+        if (todos.length > 0) {
+          const incomplete = todos.filter(
+            (t2) => t2.status === "pending" || t2.status === "in_progress" || t2.status === "blocked"
+          );
+          if (incomplete.length > 0) {
+            const incompleteList = incomplete.slice(0, 10).map((t2) => `  - [${t2.status}] ${t2.content}${t2.blocker ? ` (blocked: ${t2.blocker})` : ""}`).join("\n");
+            const more = incomplete.length > 10 ? `
+  ... +${incomplete.length - 10} more` : "";
+            return {
+              success: false,
+              output: "",
+              error: `task_complete BLOCKED — ${incomplete.length} todo item(s) still incomplete. You must either:
+  1. Continue working on the remaining items, OR
+  2. If they're actually done, call todo_write with status='completed' for each one, THEN call task_complete again.
+
+Incomplete items:
+${incompleteList}${more}`
+            };
+          }
+          try {
+            writeTodos(sessionId, []);
+          } catch {
+          }
+        }
+      } catch {
+      }
       return { success: true, output: args["summary"] || "Task completed." };
     }
   };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "open-agents-ai",
-  "version": "0.187.221",
+  "version": "0.187.223",
   "description": "AI coding agent powered by open-source models (Ollama/vLLM) — interactive TUI with agentic tool-calling loop",
   "type": "module",
   "main": "./dist/index.js",