@openagents-org/agent-launcher 0.2.131 → 0.2.133

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openagents-org/agent-launcher",
-  "version": "0.2.131",
+  "version": "0.2.133",
   "description": "OpenAgents Launcher — install, configure, and run AI coding agents from your terminal",
   "main": "src/index.js",
   "bin": {

package/src/adapters/base.js

@@ -90,6 +90,19 @@ class BaseAdapter {
       this._log(`Warning: join failed: ${e.message} \nStack: ${e.stack}`);
     }
 
+    // Sync workspace-managed skills into disabledModules
+    try {
+      const agents = await this.client.getAgents(this.workspaceId, this.token);
+      const self = agents.find((a) => a.agentName === this.agentName);
+      if (self && self.enabledSkills) {
+        const { skillsToDisabledModules } = require('../skill-catalog');
+        this.disabledModules = skillsToDisabledModules(self.enabledSkills);
+        this._log(`Synced skills from workspace: disabled=[${[...this.disabledModules].join(',')}]`);
+      }
+    } catch (e) {
+      this._log(`Warning: skill sync failed (non-fatal): ${e.message}`);
+    }
+
     // Fast-path operations (control-event cursor + heartbeat + control poll)
     // run BEFORE the message-cursor advance. Even though _skipExistingEvents
     // is fast on a healthy backend, we don't want slash commands gated on
@@ -217,9 +230,109 @@ class BaseAdapter {
       await this._postStatusReport(payload);
     } else if (action === 'routines') {
       await this._postRoutinesReport(payload);
+    } else if (action === 'skill.install') {
+      await this._handleSkillInstall(payload);
+    } else if (action === 'skill.uninstall') {
+      await this._handleSkillUninstall(payload);
+    }
+  }
+
+  /**
+   * Install a Skill Hub catalog skill into this agent's local skills
+   * directory, then report the result back to the workspace so the UI can
+   * show installing → installed / failed. Errors are logged loudly and
+   * surfaced as a `failed` status — never swallowed.
+   *
+   * payload: { action: "skill.install", skill: { id, name, source_repo, source_path } }
+   */
+  async _handleSkillInstall(payload) {
+    const installer = require('../skill-installer');
+    const skill = (payload && payload.skill) || null;
+    const skillId = skill && (skill.id || skill.skill_id);
+    if (!skillId) {
+      this._log('skill.install: missing skill metadata in payload — ignoring');
+      return;
+    }
+    this._log(`skill.install: starting install of "${skillId}" (type=${this.agentType}, dir=${this.workingDir || process.cwd()})`);
+
+    // Best-effort "installing" ping so the UI flips immediately even if the
+    // initial DB write from the request hasn't propagated to this client.
+    try {
+      await this.client.reportSkillStatus(this.workspaceId, this.agentName, this.token, {
+        skillId, state: 'installing',
+      });
+    } catch (e) {
+      this._log(`skill.install: could not report 'installing' (non-fatal): ${e && e.message ? e.message : e}`);
+    }
+
+    try {
+      const result = installer.installSkill({
+        skill,
+        agentType: this.agentType,
+        workingDir: this.workingDir,
+        log: (m) => this._log(`skill.install: ${m}`),
+      });
+      try {
+        await this.client.reportSkillStatus(this.workspaceId, this.agentName, this.token, {
+          skillId, state: 'installed', path: result.path, partial: result.partial === true,
+        });
+      } catch (e) {
+        this._log(`skill.install: installed on disk but failed to report 'installed': ${e && e.message ? e.message : e}`);
+      }
+      this._log(`skill.install: SUCCESS "${skillId}" → ${result.path}${result.partial ? ' (partial)' : ''}`);
+      await this._onSkillsChanged();
+    } catch (e) {
+      const msg = e && e.message ? e.message : String(e);
+      this._log(`skill.install: FAILED "${skillId}": ${msg}`);
+      try {
+        await this.client.reportSkillStatus(this.workspaceId, this.agentName, this.token, {
+          skillId, state: 'failed', error: msg,
+        });
+      } catch (e2) {
+        this._log(`skill.install: also failed to report 'failed': ${e2 && e2.message ? e2.message : e2}`);
+      }
     }
   }
 
+  /**
+   * Remove a previously-installed skill from disk and report `uninstalled`.
+   */
+  async _handleSkillUninstall(payload) {
+    const installer = require('../skill-installer');
+    const skill = (payload && payload.skill) || null;
+    const skillId = skill && (skill.id || skill.skill_id);
+    if (!skillId) {
+      this._log('skill.uninstall: missing skill metadata in payload — ignoring');
+      return;
+    }
+    try {
+      const result = installer.uninstallSkill({
+        skill,
+        agentType: this.agentType,
+        workingDir: this.workingDir,
+        log: (m) => this._log(`skill.uninstall: ${m}`),
+      });
+      this._log(`skill.uninstall: "${skillId}" removed=${result.removed}`);
+      try {
+        await this.client.reportSkillStatus(this.workspaceId, this.agentName, this.token, {
+          skillId, state: 'uninstalled',
+        });
+      } catch (e) {
+        this._log(`skill.uninstall: failed to report status: ${e && e.message ? e.message : e}`);
+      }
+      await this._onSkillsChanged();
+    } catch (e) {
+      const msg = e && e.message ? e.message : String(e);
+      this._log(`skill.uninstall: FAILED "${skillId}": ${msg}`);
+    }
+  }
+
+  /**
+   * Hook for subclasses to react to a change in the installed-skills set
+   * (e.g. rebuild prompt context). Default: no-op.
+   */
+  async _onSkillsChanged() {}
+
   /**
    * Post a chat message back to the requesting channel summarizing agent
    * name, type, agent-launcher version, uptime, and network. Used by the