wave-agent-sdk 0.17.2 → 0.17.3

package/dist/services/MarketplaceService.d.ts

@@ -49,7 +49,7 @@ export declare class MarketplaceService {
      */
     getCacheRegistry(): Promise<KnownMarketplacesRegistry | null>;
     /**
-     * Legacy method: loads known marketplaces with builtin injection.
+     * Legacy method: loads known marketplaces from cache.
      * @deprecated Use listMarketplaces() instead, which combines scoped settings.
      */
     getKnownMarketplaces(): Promise<KnownMarketplacesRegistry>;
@@ -90,7 +90,7 @@ export declare class MarketplaceService {
      */
     addMarketplace(input: string, scope?: Scope): Promise<KnownMarketplace>;
     /**
-     * Lists all registered marketplaces by combining scoped settings + built-in
+     * Lists all registered marketplaces by combining scoped settings + cache
      */
     listMarketplaces(): Promise<KnownMarketplace[]>;
     /**

package/dist/services/MarketplaceService.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"MarketplaceService.d.ts","sourceRoot":"","sources":["../../src/services/MarketplaceService.ts"],"names":[],"mappings":"AAIA,OAAO,EACL,gBAAgB,EAChB,yBAAyB,EACzB,eAAe,EACf,wBAAwB,EACxB,mBAAmB,EACnB,iBAAiB,EAClB,MAAM,yBAAyB,CAAC;AAEjC,OAAO,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAC;AACjE,OAAO,KAAK,EAAqB,KAAK,EAAE,MAAM,2BAA2B,CAAC;AAG1E;;;;;;;;GAQG;AACH,qBAAa,kBAAkB;IAC7B,OAAO,CAAC,MAAM,CAAC,iBAAiB,CAAS;IACzC,OAAO,CAAC,UAAU,CAAS;IAC3B,OAAO,CAAC,qBAAqB,CAAS;IACtC,OAAO,CAAC,oBAAoB,CAAS;IACrC,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,MAAM,CAAS;IACvB,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,eAAe,CAAS;IAChC,OAAO,CAAC,UAAU,CAAa;IAC/B,OAAO,CAAC,oBAAoB,CAAuB;IACnD,OAAO,CAAC,OAAO,CAAS;IACxB,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,mBAAmB,CAOzC;gBAGA,OAAO,GAAE,MAAsB,EAC/B,oBAAoB,GAAE,oBAAiD;IAuBzE;;OAEG;IACH,OAAO,CAAC,wBAAwB;IAUhC;;;OAGG;YACW,YAAY;IAkC1B;;;OAGG;YACW,WAAW;IAgBzB;;;OAGG;YACW,QAAQ;IAoDtB;;;OAGG;IACG,gBAAgB,IAAI,OAAO,CAAC,yBAAyB,GAAG,IAAI,CAAC;IAenE;;;OAGG;IACG,oBAAoB,IAAI,OAAO,CAAC,yBAAyB,CAAC;IA4BhE;;OAEG;YACW,sBAAsB;IAwBpC;;OAEG;YACW,eAAe;IAY7B;;OAEG;IACG,mBAAmB,IAAI,OAAO,CAAC,wBAAwB,CAAC;IAgB9D;;OAEG;IACG,oBAAoB,CACxB,QAAQ,EAAE,wBAAwB,GACjC,OAAO,CAAC,IAAI,CAAC;IAMhB;;OAEG;IACG,uBAAuB,CAC3B,eAAe,EAAE,MAAM,GACtB,OAAO,CAAC,mBAAmB,CAAC;IAuB/B;;OAEG;IACI,kBAAkB,CAAC,MAAM,EAAE,iBAAiB,GAAG,MAAM;IAW5D;;OAEG;YACW,qBAAqB;IAgBnC;;OAEG;IACH,6BAA6B,CAAC,IAAI,EAAE,MAAM,GAAG,KAAK,GAAG,SAAS,GAAG,IAAI;IAcrE;;OAEG;IACG,cAAc,CAClB,KAAK,EAAE,MAAM,EACb,KAAK,GAAE,KAAc,GACpB,OAAO,CAAC,gBAAgB,CAAC;IAkG5B;;OAEG;IACG,gBAAgB,IAAI,OAAO,CAAC,gBAAgB,EAAE,CAAC;IA6CrD;;OAEG;IACG,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,KAAK,GAAG,OAAO,CAAC,IAAI,CAAC;IAoBnE;;OAEG;IACG,iBAAiB,CACrB,IAAI,CAAC,EAAE,MAAM,EACb,OAAO,CAAC,EAAE;QAAE,aAAa,CAAC,EAAE,OAAO,CAAA;KAAE,GACpC,OAAO,CAAC,IAAI,CAAC;IA2GhB;;OAEG;IACG,aAAa,IAAI,OAAO,CAAC,IAAI,CAAC;IAuBpC;;OAEG;IACG,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;IA6BrE;;OAEG;IACG,aAAa,CACjB,mBAAmB,EAAE,MAAM,EAC3B,WAAW,CAAC,EAAE,MAAM,GACnB,OAAO,CAAC,eAAe,CAAC;IA8H3B;;OAEG;IACG,eAAe,CACnB,mBAAmB,EAAE,MAAM,EAC3B,WAAW,CAAC,EAAE,MAAM,GACnB,OAAO,CAAC,IAAI,CAAC;IAoChB;;OAEG;IACG,YAAY,CAAC,mBAAmB,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,CAAC;CAM1E"}
+{"version":3,"file":"MarketplaceService.d.ts","sourceRoot":"","sources":["../../src/services/MarketplaceService.ts"],"names":[],"mappings":"AAIA,OAAO,EACL,gBAAgB,EAChB,yBAAyB,EACzB,eAAe,EACf,wBAAwB,EACxB,mBAAmB,EACnB,iBAAiB,EAClB,MAAM,yBAAyB,CAAC;AAEjC,OAAO,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAC;AACjE,OAAO,KAAK,EAAqB,KAAK,EAAE,MAAM,2BAA2B,CAAC;AAG1E;;;;;;;;GAQG;AACH,qBAAa,kBAAkB;IAC7B,OAAO,CAAC,MAAM,CAAC,iBAAiB,CAAS;IACzC,OAAO,CAAC,UAAU,CAAS;IAC3B,OAAO,CAAC,qBAAqB,CAAS;IACtC,OAAO,CAAC,oBAAoB,CAAS;IACrC,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,MAAM,CAAS;IACvB,OAAO,CAAC,QAAQ,CAAS;IACzB,OAAO,CAAC,eAAe,CAAS;IAChC,OAAO,CAAC,UAAU,CAAa;IAC/B,OAAO,CAAC,oBAAoB,CAAuB;IACnD,OAAO,CAAC,OAAO,CAAS;IACxB,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,mBAAmB,CAOzC;gBAGA,OAAO,GAAE,MAAsB,EAC/B,oBAAoB,GAAE,oBAAiD;IAuBzE;;OAEG;IACH,OAAO,CAAC,wBAAwB;IAUhC;;;OAGG;YACW,YAAY;IAkC1B;;;OAGG;YACW,WAAW;IAgBzB;;;OAGG;YACW,QAAQ;IAoDtB;;;OAGG;IACG,gBAAgB,IAAI,OAAO,CAAC,yBAAyB,GAAG,IAAI,CAAC;IAenE;;;OAGG;IACG,oBAAoB,IAAI,OAAO,CAAC,yBAAyB,CAAC;IAehE;;OAEG;YACW,sBAAsB;IAwBpC;;OAEG;YACW,eAAe;IAY7B;;OAEG;IACG,mBAAmB,IAAI,OAAO,CAAC,wBAAwB,CAAC;IAgB9D;;OAEG;IACG,oBAAoB,CACxB,QAAQ,EAAE,wBAAwB,GACjC,OAAO,CAAC,IAAI,CAAC;IAMhB;;OAEG;IACG,uBAAuB,CAC3B,eAAe,EAAE,MAAM,GACtB,OAAO,CAAC,mBAAmB,CAAC;IAuB/B;;OAEG;IACI,kBAAkB,CAAC,MAAM,EAAE,iBAAiB,GAAG,MAAM;IAW5D;;OAEG;YACW,qBAAqB;IAgBnC;;OAEG;IACH,6BAA6B,CAAC,IAAI,EAAE,MAAM,GAAG,KAAK,GAAG,SAAS,GAAG,IAAI;IAarE;;OAEG;IACG,cAAc,CAClB,KAAK,EAAE,MAAM,EACb,KAAK,GAAE,KAAc,GACpB,OAAO,CAAC,gBAAgB,CAAC;IAkG5B;;OAEG;IACG,gBAAgB,IAAI,OAAO,CAAC,gBAAgB,EAAE,CAAC;IAqCrD;;OAEG;IACG,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,KAAK,GAAG,OAAO,CAAC,IAAI,CAAC;IAkBnE;;OAEG;IACG,iBAAiB,CACrB,IAAI,CAAC,EAAE,MAAM,EACb,OAAO,CAAC,EAAE;QAAE,aAAa,CAAC,EAAE,OAAO,CAAA;KAAE,GACpC,OAAO,CAAC,IAAI,CAAC;IA2GhB;;OAEG;IACG,aAAa,IAAI,OAAO,CAAC,IAAI,CAAC;IAuBpC;;OAEG;IACG,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;IA6BrE;;OAEG;IACG,aAAa,CACjB,mBAAmB,EAAE,MAAM,EAC3B,WAAW,CAAC,EAAE,MAAM,GACnB,OAAO,CAAC,eAAe,CAAC;IA8H3B;;OAEG;IACG,eAAe,CACnB,mBAAmB,EAAE,MAAM,EAC3B,WAAW,CAAC,EAAE,MAAM,GACnB,OAAO,CAAC,IAAI,CAAC;IAoChB;;OAEG;IACG,YAAY,CAAC,mBAAmB,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,CAAC;CAM1E"}

package/dist/services/MarketplaceService.js

@@ -155,30 +155,19 @@ export class MarketplaceService {
         }
     }
     /**
-     * Legacy method: loads known marketplaces with builtin injection.
+     * Legacy method: loads known marketplaces from cache.
      * @deprecated Use listMarketplaces() instead, which combines scoped settings.
      */
     async getKnownMarketplaces() {
         const cache = await this.getCacheRegistry();
-        // If cache is null (file doesn't exist or has no parseable content), inject builtin
         if (cache === null) {
-            return {
-                marketplaces: [
-                    {
-                        ...MarketplaceService.BUILTIN_MARKETPLACE,
-                        isBuiltin: true,
-                        declaredScope: "builtin",
-                    },
-                ],
-            };
+            return { marketplaces: [] };
         }
-        // File has valid JSON - respect user's explicit choice even if empty
-        const hasBuiltin = cache.marketplaces.some((m) => m.name === MarketplaceService.BUILTIN_MARKETPLACE.name);
         return {
             marketplaces: cache.marketplaces.map((m) => ({
                 ...m,
                 isBuiltin: m.isBuiltin || m.name === MarketplaceService.BUILTIN_MARKETPLACE.name,
-                declaredScope: m.declaredScope ?? (hasBuiltin ? "builtin" : "user"),
+                declaredScope: m.declaredScope ?? "user",
             })),
         };
     }
@@ -293,14 +282,14 @@ export class MarketplaceService {
      * Finds which scope declared a marketplace (user, project, local, or builtin)
      */
     getMarketplaceDeclaringSource(name) {
-        if (name === MarketplaceService.BUILTIN_MARKETPLACE.name)
-            return "builtin";
         const scopes = ["local", "project", "user"];
         for (const scope of scopes) {
             const scoped = this.configurationService.getScopedMarketplaces(this.workdir, scope);
             if (scoped[name])
                 return scope;
         }
+        if (name === MarketplaceService.BUILTIN_MARKETPLACE.name)
+            return "builtin";
         return null;
     }
     /**
@@ -381,37 +370,28 @@ export class MarketplaceService {
         });
     }
     /**
-     * Lists all registered marketplaces by combining scoped settings + built-in
+     * Lists all registered marketplaces by combining scoped settings + cache
      */
     async listMarketplaces() {
         const scopedMarketplaces = this.configurationService.getMergedMarketplaces(this.workdir);
         const cacheRegistry = await this.getCacheRegistry();
         const cacheMap = new Map((cacheRegistry?.marketplaces ?? []).map((m) => [m.name, m]));
         const result = [];
-        // Add built-in marketplace
-        result.push({
-            ...MarketplaceService.BUILTIN_MARKETPLACE,
-            isBuiltin: true,
-            declaredScope: "builtin",
-        });
         // Add all scoped marketplaces (local overrides project overrides user)
         for (const [name, config] of Object.entries(scopedMarketplaces)) {
-            if (name === MarketplaceService.BUILTIN_MARKETPLACE.name)
-                continue;
             const cache = cacheMap.get(name);
             const declaredScope = this.getMarketplaceDeclaringSource(name);
             result.push(await this.buildMarketplaceEntry(name, config, cache, declaredScope));
         }
         // Add cache entries not yet in scoped settings (backwards compatibility)
         for (const [name, cache] of cacheMap.entries()) {
-            if (name === MarketplaceService.BUILTIN_MARKETPLACE.name)
-                continue;
             if (scopedMarketplaces[name])
                 continue;
+            const isBuiltin = name === MarketplaceService.BUILTIN_MARKETPLACE.name;
             result.push({
                 ...cache,
-                isBuiltin: false,
-                declaredScope: cache.declaredScope ?? "user",
+                isBuiltin,
+                declaredScope: cache.declaredScope ?? (isBuiltin ? "builtin" : "user"),
             });
         }
         return result;
@@ -422,10 +402,9 @@ export class MarketplaceService {
     async removeMarketplace(name, scope) {
         return this.withLock(async () => {
             const targetScope = scope || this.getMarketplaceDeclaringSource(name) || "user";
-            if (targetScope === "builtin") {
-                throw new Error("Cannot remove built-in marketplace");
+            if (targetScope !== "builtin") {
+                await this.configurationService.removeMarketplaceFromScope(this.workdir, targetScope, name);
             }
-            await this.configurationService.removeMarketplaceFromScope(this.workdir, targetScope, name);
             // Also remove from cache
             await this.removeFromCache(name);
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wave-agent-sdk",
-  "version": "0.17.2",
+  "version": "0.17.3",
   "description": "SDK for building AI-powered development tools and agents",
   "keywords": [
     "ai",

package/src/services/MarketplaceService.ts

@@ -217,33 +217,20 @@ export class MarketplaceService {
   }
 
   /**
-   * Legacy method: loads known marketplaces with builtin injection.
+   * Legacy method: loads known marketplaces from cache.
    * @deprecated Use listMarketplaces() instead, which combines scoped settings.
    */
   async getKnownMarketplaces(): Promise<KnownMarketplacesRegistry> {
     const cache = await this.getCacheRegistry();
-    // If cache is null (file doesn't exist or has no parseable content), inject builtin
     if (cache === null) {
-      return {
-        marketplaces: [
-          {
-            ...MarketplaceService.BUILTIN_MARKETPLACE,
-            isBuiltin: true,
-            declaredScope: "builtin",
-          },
-        ],
-      };
+      return { marketplaces: [] };
     }
-    // File has valid JSON - respect user's explicit choice even if empty
-    const hasBuiltin = cache.marketplaces.some(
-      (m) => m.name === MarketplaceService.BUILTIN_MARKETPLACE.name,
-    );
     return {
       marketplaces: cache.marketplaces.map((m) => ({
         ...m,
         isBuiltin:
           m.isBuiltin || m.name === MarketplaceService.BUILTIN_MARKETPLACE.name,
-        declaredScope: m.declaredScope ?? (hasBuiltin ? "builtin" : "user"),
+        declaredScope: m.declaredScope ?? "user",
       })),
     };
   }
@@ -385,8 +372,6 @@ export class MarketplaceService {
    * Finds which scope declared a marketplace (user, project, local, or builtin)
    */
   getMarketplaceDeclaringSource(name: string): Scope | "builtin" | null {
-    if (name === MarketplaceService.BUILTIN_MARKETPLACE.name) return "builtin";
-
     const scopes: Scope[] = ["local", "project", "user"];
     for (const scope of scopes) {
       const scoped = this.configurationService.getScopedMarketplaces(
@@ -395,6 +380,7 @@ export class MarketplaceService {
       );
       if (scoped[name]) return scope;
     }
+    if (name === MarketplaceService.BUILTIN_MARKETPLACE.name) return "builtin";
     return null;
   }
 
@@ -503,7 +489,7 @@ export class MarketplaceService {
   }
 
   /**
-   * Lists all registered marketplaces by combining scoped settings + built-in
+   * Lists all registered marketplaces by combining scoped settings + cache
    */
   async listMarketplaces(): Promise<KnownMarketplace[]> {
     const scopedMarketplaces = this.configurationService.getMergedMarketplaces(
@@ -516,16 +502,8 @@ export class MarketplaceService {
 
     const result: KnownMarketplace[] = [];
 
-    // Add built-in marketplace
-    result.push({
-      ...MarketplaceService.BUILTIN_MARKETPLACE,
-      isBuiltin: true,
-      declaredScope: "builtin",
-    });
-
     // Add all scoped marketplaces (local overrides project overrides user)
     for (const [name, config] of Object.entries(scopedMarketplaces)) {
-      if (name === MarketplaceService.BUILTIN_MARKETPLACE.name) continue;
       const cache = cacheMap.get(name);
       const declaredScope = this.getMarketplaceDeclaringSource(name) as
         | "user"
@@ -538,12 +516,12 @@ export class MarketplaceService {
 
     // Add cache entries not yet in scoped settings (backwards compatibility)
     for (const [name, cache] of cacheMap.entries()) {
-      if (name === MarketplaceService.BUILTIN_MARKETPLACE.name) continue;
       if (scopedMarketplaces[name]) continue;
+      const isBuiltin = name === MarketplaceService.BUILTIN_MARKETPLACE.name;
       result.push({
         ...cache,
-        isBuiltin: false,
-        declaredScope: cache.declaredScope ?? "user",
+        isBuiltin,
+        declaredScope: cache.declaredScope ?? (isBuiltin ? "builtin" : "user"),
       });
     }
 
@@ -558,16 +536,14 @@ export class MarketplaceService {
       const targetScope =
         scope || this.getMarketplaceDeclaringSource(name) || "user";
 
-      if (targetScope === "builtin") {
-        throw new Error("Cannot remove built-in marketplace");
+      if (targetScope !== "builtin") {
+        await this.configurationService.removeMarketplaceFromScope(
+          this.workdir,
+          targetScope,
+          name,
+        );
       }
 
-      await this.configurationService.removeMarketplaceFromScope(
-        this.workdir,
-        targetScope,
-        name,
-      );
-
       // Also remove from cache
       await this.removeFromCache(name);
     });