loopengine 0.1.36 → 0.1.38

package/dist/bin/ability-manager.d.ts

@@ -12,6 +12,11 @@ export interface AbilityEnvDecl {
     name: string;
     description?: string;
     secret?: boolean;
+    /** A closed set of valid values, e.g. `["openai", "google"]` for a
+     * provider switch — the Admin UI renders these as a dropdown instead
+     * of a free-text field. Omit for anything that isn't genuinely a fixed
+     * enum (an API key, a free-form model name, a path, ...). */
+    options?: string[];
 }
 /** `name`/`version` deliberately aren't declared here — they're read off
  * the ability's own sibling `package.json` instead (see readManifest),

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

@@ -1138,41 +1138,60 @@ export const agentsConfigPageHtml = `<!doctype html>
     var sharedNote = v.abilityNames.length > 1
       ? ' <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>'
       : '';
+    // A secret's value is never sent to the browser at all (see
+    // env-admin.ts) — "set" is the most this tab can ever say about one.
+    // A non-secret var's actual value comes through, so show it instead
+    // of a bare "set", since the whole point of looking here is usually
+    // to check *what* something is currently configured to, not just
+    // whether it has a value.
+    var statusHtml = !v.set
+      ? '<span class="error">not set</span>'
+      : v.secret
+        ? '<span class="hint">set</span>'
+        : '<code>' + escapeHtml(v.value || '') + '</code>';
+    // A var with a closed set of valid values (AbilityEnvDecl.options,
+    // e.g. a provider switch) gets a <select> instead of a free-text
+    // field, so there's nothing to mistype — the same "value" form field
+    // name either element uses means wireEnvHandlers's own submit
+    // handler doesn't need to know which one it's looking at.
+    var fieldHtml = v.options
+      ? '<select name="value">' + v.options.map(function (opt) {
+          return '<option value="' + escapeHtml(opt) + '"' + (v.value === opt ? ' selected' : '') + '>' + escapeHtml(opt) + '</option>';
+        }).join('') + '</select>'
+      : '<input type="' + (v.secret ? 'password' : 'text') + '" name="value" placeholder="' + (v.set ? 'unchanged unless you type a new value' : 'value') + '" required>';
     return '<tr>' +
       '<td><code>' + escapeHtml(v.name) + '</code>' + sharedNote + '</td>' +
       '<td style="max-width:320px">' + escapeHtml(v.description || '') + '</td>' +
-      '<td>' + (v.set ? '<span class="hint">set</span>' : '<span class="error">not set</span>') + '</td>' +
+      '<td>' + statusHtml + '</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>' +
+        fieldHtml +
         '<button type="submit">Save</button>' +
       '</form></td>' +
       '</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.
+  // One card per ability, matching the same .source/.source-head look
+  // Gateway Tools sources already use — so an operator scanning the tab
+  // sees, per ability, which env vars it declared, instead of hunting
+  // through one long flat table for rows that belong to whichever
+  // ability they're actually trying to set up right now. Deliberately no
+  // "not set" count here (or in renderEnvConfigHtml's own summary) — a
+  // declared var not being set doesn't mean anything is broken, since
+  // plenty of them have working defaults (see each var's own
+  // description) and are only ever declared so an operator *can*
+  // override them, not because every single one is required.
   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 class="source-head"><h4>' + escapeHtml(abilityName) + '</h4><span class="hint">' + vars.length + ' var' + (vars.length === 1 ? '' : 's') + '</span></div>' +
+      '<table><thead><tr><th>Name</th><th style="max-width:320px">Description</th><th>Current value</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 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>';
+    var summaryHtml = '<p class="hint">' + vars.length + ' environment variable' + (vars.length === 1 ? '' : 's') + ' across every installed ability.</p>';
 
     // Grouped by declaring ability, not a single flat list — a var
     // declared by more than one ability (rare, but see renderEnvRow's

package/dist/web/env-admin.d.ts

@@ -9,6 +9,15 @@ export interface DeclaredEnvVar {
      * an array, not a single name. */
     abilityNames: string[];
     set: boolean;
+    /** The live value, but only when `secret` is false — a secret is never
+     * echoed back once set, same rule this file's own top comment and
+     * web/http-tool-admin.ts's `{{ENV_VAR}}` header handling already
+     * follow. Undefined whenever `secret` is true or `set` is false. */
+    value?: string;
+    /** A closed set of valid values (see AbilityEnvDecl.options) — present
+     * only when the declaring ability actually named one; the Admin UI
+     * renders a dropdown instead of a free-text field when this is set. */
+    options?: string[];
 }
 /** Every env var any ability installed for `agentName` declared it
  * needs, merged by name. There's one .env per *project*, not per agent

package/dist/web/env-admin.js

@@ -1,9 +1,11 @@
 // Backs the Admin UI's "Environment" section — every env var an ability
 // (see ABILITIES.md, bin/ability-manager.ts) declared as required,
 // across every ability installed for an agent, with set/not-set status
-// only. A value marked `secret` is never echoed back once set — same
-// never-echo-a-secret rule web/http-tool-admin.ts's own `{{ENV_VAR}}`
-// header handling already establishes for a tool's own secrets.
+// and, for a non-secret var, its live value (so an operator can actually
+// tell what's configured, not just that something is). A value marked
+// `secret` is never echoed back once set — same never-echo-a-secret rule
+// web/http-tool-admin.ts's own `{{ENV_VAR}}` header handling already
+// establishes for a tool's own secrets.
 import { existsSync, readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
 import { agentDir } from '../core/gateway-tools.js';
@@ -51,14 +53,19 @@ export function listDeclaredEnvVars(agentName) {
             if (existing) {
                 existing.abilityNames.push(abilityName);
                 existing.secret = existing.secret || decl.secret === true;
+                if (existing.secret)
+                    existing.value = undefined;
                 continue;
             }
+            const rawValue = process.env[decl.name];
             byName.set(decl.name, {
                 name: decl.name,
                 description: decl.description,
                 secret: decl.secret === true,
                 abilityNames: [abilityName],
-                set: process.env[decl.name] !== undefined,
+                set: rawValue !== undefined,
+                value: decl.secret === true ? undefined : rawValue,
+                options: decl.options,
             });
         }
     }

package/package.json

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