loopengine 0.1.37 → 0.1.39

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/index.d.ts

@@ -57,8 +57,8 @@ export { listDeclaredEnvVars, setEnvVar, EnvVarNameError } from './web/env-admin
 export type { DeclaredEnvVar } from './web/env-admin.js';
 export { readActauthConfig, addActauthRule, updateActauthRule, removeActauthRule, setDefaultDecision, ActauthRuleExistsError, ActauthRuleNotFoundError, } from './web/actauth-admin.js';
 export type { ActauthRuleInput, ActauthConfigView } from './web/actauth-admin.js';
-export { installAbility, listInstalledAbilities, AbilityManifestError, AbilityVersionError, AbilityCollisionError, AbilityAlreadyInstalledError, } from './bin/ability-manager.js';
-export type { InstalledAbilityRecord, AbilityEnvDecl } from './bin/ability-manager.js';
+export { installAbility, upgradeAbility, listInstalledAbilities, AbilityManifestError, AbilityVersionError, AbilityCollisionError, AbilityAlreadyInstalledError, AbilityNotInstalledError, } from './bin/ability-manager.js';
+export type { InstalledAbilityRecord, AbilityEnvDecl, UpgradeFileResult } from './bin/ability-manager.js';
 export { searchPublicAbilities } from './web/abilities-admin.js';
 export type { PublicAbilitySearchResult } from './web/abilities-admin.js';
 export { describeModelProviders, describeGateways } from './web/global-config.js';

package/dist/index.js

@@ -51,7 +51,7 @@ export { playgroundHtml } from './web/playground.js';
 export { readSkill, writeSkill, deleteSkill, SkillInvalidIdError, SkillNotFoundError } from './web/skills-admin.js';
 export { listDeclaredEnvVars, setEnvVar, EnvVarNameError } from './web/env-admin.js';
 export { readActauthConfig, addActauthRule, updateActauthRule, removeActauthRule, setDefaultDecision, ActauthRuleExistsError, ActauthRuleNotFoundError, } from './web/actauth-admin.js';
-export { installAbility, listInstalledAbilities, AbilityManifestError, AbilityVersionError, AbilityCollisionError, AbilityAlreadyInstalledError, } from './bin/ability-manager.js';
+export { installAbility, upgradeAbility, listInstalledAbilities, AbilityManifestError, AbilityVersionError, AbilityCollisionError, AbilityAlreadyInstalledError, AbilityNotInstalledError, } from './bin/ability-manager.js';
 export { searchPublicAbilities } from './web/abilities-admin.js';
 export { describeModelProviders, describeGateways } from './web/global-config.js';
 export { createTrackedApprover, listApprovals, decideApproval, findApproval } from './web/web-approver.js';

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

@@ -1149,31 +1149,41 @@ export const agentsConfigPageHtml = `<!doctype html>
       : 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>' + 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>' +
+      '<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>';
   }
@@ -1181,9 +1191,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 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
@@ -1284,6 +1292,7 @@ export const agentsConfigPageHtml = `<!doctype html>
       '<td><code>' + escapeHtml(a.name) + '</code></td>' +
       '<td>' + escapeHtml(a.version) + '</td>' +
       '<td class="hint">' + escapeHtml((a.tools || []).join(', ')) + '</td>' +
+      '<td><button type="button" class="ability-upgrade-btn" data-name="' + escapeHtml(a.name) + '">Upgrade</button></td>' +
       '</tr>';
   }
 
@@ -1298,7 +1307,8 @@ export const agentsConfigPageHtml = `<!doctype html>
   function renderAbilitiesConfigHtml(data) {
     var installed = data.abilities || [];
     var installedHtml = installed.length
-      ? '<table><thead><tr><th>Name</th><th>Version</th><th>Tools</th></tr></thead><tbody>' + installed.map(renderInstalledAbilityRow).join('') + '</tbody></table>'
+      ? '<table><thead><tr><th>Name</th><th>Version</th><th>Tools</th><th></th></tr></thead><tbody>' + installed.map(renderInstalledAbilityRow).join('') + '</tbody></table>' +
+        '<p id="abilityUpgradeStatus" class="hint"></p>'
       : '<p class="hint">No abilities installed for this agent yet.</p>';
 
     return '<h3>Installed</h3>' + installedHtml +
@@ -1363,6 +1373,48 @@ export const agentsConfigPageHtml = `<!doctype html>
       });
   }
 
+  // Upgrades an already-installed ability to whatever it currently
+  // resolves to — same endpoint upgradeAbility (bin/ability-manager.ts)
+  // backs, which three-way merges tool/skill files instead of blindly
+  // overwriting a hand-edited one, so a merge conflict is a real,
+  // expected outcome here, not an error — surfaced distinctly from a
+  // request actually failing.
+  function upgradeAbility(name, abilityName, statusEl, btn) {
+    if (btn) { btn.disabled = true; }
+    statusEl.className = 'hint';
+    statusEl.textContent = 'Upgrading ' + abilityName + '…';
+    fetch('/agents/' + encodeURIComponent(name) + '/abilities/' + encodeURIComponent(abilityName) + '/upgrade', {
+      method: 'POST',
+      headers: { 'content-type': 'application/json' },
+      body: JSON.stringify({}),
+    })
+      .then(function (r) { return r.json().then(function (j) { return { ok: r.ok, body: j }; }); })
+      .then(function (result) {
+        if (!result.ok) throw new Error(result.body.error || 'request failed');
+        var files = result.body.files || [];
+        var updated = files.filter(function (f) { return f.status === 'updated'; });
+        var conflicts = files.filter(function (f) { return f.status === 'conflict'; });
+        var message;
+        if (!files.length || (!updated.length && !conflicts.length)) {
+          message = escapeHtml(abilityName) + ' is already up to date.';
+        } else {
+          message = 'Upgraded ' + escapeHtml(abilityName) + ': ' + escapeHtml(updated.map(function (f) { return f.path; }).join(', ') || 'no files changed') +
+            '. Already running under npx loopengine dev? It becomes active automatically. Running under serve (or nothing yet)? Restart the server to pick it up.';
+          if (conflicts.length) {
+            message += ' <strong>' + conflicts.length + ' file(s) need resolving by hand: ' + escapeHtml(conflicts.map(function (f) { return f.path; }).join(', ')) + '</strong> — a hand-edited file with upstream changes too, left alone rather than overwritten.';
+          }
+        }
+        statusEl.className = conflicts.length ? 'error' : 'hint';
+        statusEl.innerHTML = message;
+        loadAbilitiesTab(name);
+      })
+      .catch(function (err) {
+        statusEl.className = 'error';
+        statusEl.textContent = 'Could not upgrade: ' + err.message;
+        if (btn) { btn.disabled = false; }
+      });
+  }
+
   // Shared by the initial (empty-query) load right after the tab opens
   // and the search form's own submit — the empty-query case is just
   // "list every published ability" (the keyword filter alone, no extra
@@ -1395,6 +1447,15 @@ export const agentsConfigPageHtml = `<!doctype html>
     var content = abilitiesContentEl();
     if (!content) return;
 
+    var upgradeStatusEl = content.querySelector('#abilityUpgradeStatus');
+    var upgradeBtns = content.querySelectorAll('.ability-upgrade-btn');
+    for (var u = 0; u < upgradeBtns.length; u++) {
+      upgradeBtns[u].addEventListener('click', function (ev) {
+        var btn = ev.currentTarget;
+        upgradeAbility(name, btn.dataset.name, upgradeStatusEl, btn);
+      });
+    }
+
     var searchForm = content.querySelector('#abilitySearchForm');
     var resultsEl = content.querySelector('#abilitySearchResults');
     if (searchForm && resultsEl) {

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

@@ -14,6 +14,10 @@ export interface DeclaredEnvVar {
      * 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

@@ -65,6 +65,7 @@ export function listDeclaredEnvVars(agentName) {
                 abilityNames: [abilityName],
                 set: rawValue !== undefined,
                 value: decl.secret === true ? undefined : rawValue,
+                options: decl.options,
             });
         }
     }

package/package.json

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