loopengine 0.1.33 → 0.1.35

package/dist/web/agents-config-page.js

@@ -1136,7 +1136,7 @@ export const agentsConfigPageHtml = `<!doctype html>
       : '';
     return '<tr>' +
       '<td><code>' + escapeHtml(v.name) + '</code></td>' +
-      '<td>' + escapeHtml(v.description || '') + '</td>' +
+      '<td style="max-width:320px">' + escapeHtml(v.description || '') + '</td>' +
       '<td class="hint">' + escapeHtml(v.abilityNames.join(', ')) + sharedNote + '</td>' +
       '<td>' + (v.set ? '<span class="hint">set</span>' : '<span class="error">not set</span>') + '</td>' +
       '<td><form class="add-source env-var-form" data-name="' + escapeHtml(v.name) + '">' +
@@ -1149,7 +1149,7 @@ export const agentsConfigPageHtml = `<!doctype html>
   function renderEnvConfigHtml(vars) {
     if (!vars.length) return '<p class="hint">No installed ability has declared any environment variables for this agent yet.</p>';
     var rows = vars.map(renderEnvRow).join('');
-    return '<table><thead><tr><th>Name</th><th>Description</th><th>Required by</th><th>Status</th><th>Set value</th></tr></thead><tbody>' + rows + '</tbody></table>';
+    return '<table><thead><tr><th>Name</th><th style="max-width:320px">Description</th><th>Required by</th><th>Status</th><th>Set value</th></tr></thead><tbody>' + rows + '</tbody></table>';
   }
 
   function envContentEl() {
@@ -1246,7 +1246,7 @@ export const agentsConfigPageHtml = `<!doctype html>
       : '<p class="hint">No abilities installed for this agent yet.</p>';
 
     return '<h3>Installed</h3>' + installedHtml +
-      '<h3>Search public abilities</h3>' +
+      '<h3 style="margin-top:24px">Search public abilities</h3>' +
       '<form id="abilitySearchForm" class="add-source">' +
         '<input type="text" name="q" placeholder="Search by name or keyword (optional)">' +
         '<button type="submit">Search</button>' +
@@ -2187,9 +2187,14 @@ export const agentsConfigPageHtml = `<!doctype html>
   // so only Overview is left needing a re-fetch.
   function refreshOverviewPanel(name) {
     fetch('/agents/' + encodeURIComponent(name) + '/config')
-      .then(function (r) { return r.json(); })
-      .then(function (cfg) {
+      .then(function (r) { return r.json().then(function (j) { return { ok: r.ok, body: j }; }); })
+      .then(function (result) {
         if (currentName !== name) return;
+        // A non-ok body is {error: "..."}, not a real config — bail out
+        // to the catch below rather than rendering it as one (same risk
+        // selectAgent's own /config fetch has — see its own comment).
+        if (!result.ok) throw new Error(result.body.error || 'request failed');
+        var cfg = result.body;
         currentCfg = cfg;
         var overviewPanel = detail.querySelector('[data-tab-panel="overview"]');
         if (overviewPanel) {
@@ -2225,9 +2230,13 @@ export const agentsConfigPageHtml = `<!doctype html>
   // re-rendering both it and Overview from a fresh /config fetch.
   function refreshSkillsDependentPanels(name) {
     fetch('/agents/' + encodeURIComponent(name) + '/config')
-      .then(function (r) { return r.json(); })
-      .then(function (cfg) {
+      .then(function (r) { return r.json().then(function (j) { return { ok: r.ok, body: j }; }); })
+      .then(function (result) {
         if (currentName !== name) return;
+        // Same "don't render an {error: ...} body as if it were a real
+        // config" guard as refreshOverviewPanel/selectAgent above.
+        if (!result.ok) throw new Error(result.body.error || 'request failed');
+        var cfg = result.body;
         currentCfg = cfg;
         var overviewPanel = detail.querySelector('[data-tab-panel="overview"]');
         var skillsPanel = skillsPanelEl();
@@ -2295,10 +2304,18 @@ export const agentsConfigPageHtml = `<!doctype html>
     empty.style.display = 'none';
     detail.style.display = 'block';
     fetch('/agents/' + encodeURIComponent(name) + '/config')
-      .then(function (r) { return r.json(); })
-      .then(function (cfg) {
+      .then(function (r) { return r.json().then(function (j) { return { ok: r.ok, body: j }; }); })
+      .then(function (result) {
         if (currentName !== name) return;
-        renderDetail(cfg);
+        // A non-ok response's body is {error: "..."}, not a real config
+        // — rendering it as one would crash somewhere deep in
+        // renderDetail (e.g. reading .model.provider off an object with
+        // no .model at all) with a confusing, unrelated-looking message
+        // instead of the actual server-side error this route already
+        // went out of its way to surface clearly (see handleAgentConfig
+        // Get's own doc comment).
+        if (!result.ok) throw new Error(result.body.error || 'request failed');
+        renderDetail(result.body);
       })
       .catch(function (err) {
         if (currentName !== name) return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "loopengine",
-  "version": "0.1.33",
+  "version": "0.1.35",
   "type": "module",
   "description": "Runtime for tool-using AI agents with a transparent ReAct loop.",
   "license": "MIT",