loopengine 0.1.35 → 0.1.36

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

@@ -1128,16 +1128,19 @@ export const agentsConfigPageHtml = `<!doctype html>
     // More than one ability declaring the same name doesn't get resolved
     // here — there's one .env per project, so both read whatever single
     // value ends up set, whether that's actually correct or a coincidence
-    // (see env-admin.ts's own doc comment). Listing every ability instead
-    // of just the first one seen at least makes that visible to whoever's
-    // looking, instead of silently hiding all but one.
+    // (see env-admin.ts's own doc comment). Naming every declaring
+    // ability instead of just the first one seen makes that visible to
+    // whoever's looking, instead of silently hiding all but one — shown
+    // here (not a separate "Required by" column, now that rows are
+    // already grouped under one ability's own card) only when there
+    // actually is more than one, since inside its own card which ability
+    // this is is already obvious from context.
     var sharedNote = v.abilityNames.length > 1
-      ? ' <span class="hint" title="More than one installed ability declares this name — there is only one value for it project-wide, so check whether they actually need the same one.">(shared)</span>'
+      ? ' <span class="hint" title="Also declared by: ' + escapeHtml(v.abilityNames.join(', ')) + '. There is only one value for it project-wide, so check whether they actually need the same one.">(shared)</span>'
       : '';
     return '<tr>' +
-      '<td><code>' + escapeHtml(v.name) + '</code></td>' +
+      '<td><code>' + escapeHtml(v.name) + '</code>' + sharedNote + '</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) + '">' +
         '<input type="' + (v.secret ? 'password' : 'text') + '" name="value" placeholder="' + (v.set ? 'unchanged unless you type a new value' : 'value') + '" required>' +
@@ -1146,10 +1149,52 @@ export const agentsConfigPageHtml = `<!doctype html>
       '</tr>';
   }
 
+  // One card per ability, matching the same .source/.source-head/
+  // status-ok/status-error look Gateway Tools sources already use — so
+  // an operator scanning the tab sees, per ability, whether it's fully
+  // configured or not, instead of hunting through one long flat table
+  // for rows that belong to whichever ability they're actually trying to
+  // set up right now.
+  function renderEnvAbilityCard(abilityName, vars) {
+    var missing = vars.filter(function (v) { return !v.set; }).length;
+    var statusHtml = missing === 0
+      ? '<span class="status-ok">all set</span>'
+      : '<span class="status-error">' + missing + ' of ' + vars.length + ' not set</span>';
+    var rows = vars.map(renderEnvRow).join('');
+    return '<div class="source">' +
+      '<div class="source-head"><h4>' + escapeHtml(abilityName) + '</h4>' + statusHtml + '</div>' +
+      '<table><thead><tr><th>Name</th><th style="max-width:320px">Description</th><th>Status</th><th>Set value</th></tr></thead><tbody>' + rows + '</tbody></table>' +
+      '</div>';
+  }
+
   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 style="max-width:320px">Description</th><th>Required by</th><th>Status</th><th>Set value</th></tr></thead><tbody>' + rows + '</tbody></table>';
+
+    var missingCount = vars.filter(function (v) { return !v.set; }).length;
+    var summaryHtml = '<p class="hint">' + vars.length + ' environment variable' + (vars.length === 1 ? '' : 's') + ' across every installed ability' +
+      (missingCount ? ' — <span class="error">' + missingCount + ' still not set</span>.' : ' — all set.') + '</p>';
+
+    // Grouped by declaring ability, not a single flat list — a var
+    // declared by more than one ability (rare, but see renderEnvRow's
+    // own comment) appears once under each, since either ability's own
+    // card is a place an operator setting that ability up would
+    // reasonably look for it.
+    var byAbility = {};
+    var abilityNames = [];
+    vars.forEach(function (v) {
+      v.abilityNames.forEach(function (abilityName) {
+        if (!byAbility[abilityName]) {
+          byAbility[abilityName] = [];
+          abilityNames.push(abilityName);
+        }
+        byAbility[abilityName].push(v);
+      });
+    });
+    abilityNames.sort();
+
+    return summaryHtml + abilityNames.map(function (abilityName) {
+      return renderEnvAbilityCard(abilityName, byAbility[abilityName]);
+    }).join('');
   }
 
   function envContentEl() {

package/package.json

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