@zapier/zapier-sdk 0.16.1 → 0.16.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @zapier/zapier-sdk
 
+## 0.16.2
+
+### Patch Changes
+
+- 93cbb3c: Fix search to prefer public app versions over private when user has access to both
+
 ## 0.16.1
 
 ### Patch Changes

package/dist/index.cjs

@@ -3897,8 +3897,17 @@ async function augmentWithSearchResults({
       searchParams: { term: searchTerm }
     }
   );
-  const searchResults = searchResponse.results.map(
-    transformImplementationMetaToAppItem
+  const byImplementationName = /* @__PURE__ */ new Map();
+  for (const result of searchResponse.results) {
+    const [implementationName] = splitVersionedKey2(result.id);
+    const isPublic = result.visibility === "public";
+    const existing = byImplementationName.get(implementationName);
+    if (!existing || isPublic && !existing.isPublic) {
+      byImplementationName.set(implementationName, { result, isPublic });
+    }
+  }
+  const searchResults = Array.from(byImplementationName.values()).map(
+    (entry) => transformImplementationMetaToAppItem(entry.result)
   );
   const implementationNameSet = new Set(
     implementationIds.map((id) => {
@@ -5018,7 +5027,7 @@ function getCpuTime() {
 
 // package.json
 var package_default = {
-  version: "0.16.1"};
+  version: "0.16.2"};
 
 // src/plugins/eventEmission/builders.ts
 function createBaseEvent(context = {}) {

package/dist/index.mjs

@@ -3875,8 +3875,17 @@ async function augmentWithSearchResults({
       searchParams: { term: searchTerm }
     }
   );
-  const searchResults = searchResponse.results.map(
-    transformImplementationMetaToAppItem
+  const byImplementationName = /* @__PURE__ */ new Map();
+  for (const result of searchResponse.results) {
+    const [implementationName] = splitVersionedKey2(result.id);
+    const isPublic = result.visibility === "public";
+    const existing = byImplementationName.get(implementationName);
+    if (!existing || isPublic && !existing.isPublic) {
+      byImplementationName.set(implementationName, { result, isPublic });
+    }
+  }
+  const searchResults = Array.from(byImplementationName.values()).map(
+    (entry) => transformImplementationMetaToAppItem(entry.result)
   );
   const implementationNameSet = new Set(
     implementationIds.map((id) => {
@@ -4996,7 +5005,7 @@ function getCpuTime() {
 
 // package.json
 var package_default = {
-  version: "0.16.1"};
+  version: "0.16.2"};
 
 // src/plugins/eventEmission/builders.ts
 function createBaseEvent(context = {}) {

package/dist/temporary-internal-core/handlers/listApps.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"listApps.d.ts","sourceRoot":"","sources":["../../../src/temporary-internal-core/handlers/listApps.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,WAAW,EAAE,MAAM,UAAU,CAAC;AACrD,OAAO,EAEL,KAAK,sBAAsB,EAC3B,KAAK,gBAAgB,EACtB,MAAM,iBAAiB,CAAC;AAkBzB;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,WAAW,UAAU;IACzB,GAAG,CAAC,CAAC,GAAG,OAAO,EACb,IAAI,EAAE,MAAM,EACZ,OAAO,CAAC,EAAE;QAAE,YAAY,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;KAAE,GAClD,OAAO,CAAC,CAAC,CAAC,CAAC;CACf;AAED;;;;;;;;;;GAUG;AACH,MAAM,WAAW,mBAAoB,SAAQ,WAAW;IACtD,UAAU,EAAE,UAAU,CAAC;CACxB;AAwDD;;;;;;;;;;;;;GAaG;AACH,eAAO,MAAM,cAAc,EAAE,OAAO,CAClC,sBAAsB,EACtB,gBAAgB,EAChB,mBAAmB,CA+EpB,CAAC"}
+{"version":3,"file":"listApps.d.ts","sourceRoot":"","sources":["../../../src/temporary-internal-core/handlers/listApps.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,KAAK,EAAE,OAAO,EAAE,WAAW,EAAE,MAAM,UAAU,CAAC;AACrD,OAAO,EAEL,KAAK,sBAAsB,EAC3B,KAAK,gBAAgB,EACtB,MAAM,iBAAiB,CAAC;AAkBzB;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,WAAW,UAAU;IACzB,GAAG,CAAC,CAAC,GAAG,OAAO,EACb,IAAI,EAAE,MAAM,EACZ,OAAO,CAAC,EAAE;QAAE,YAAY,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;KAAE,GAClD,OAAO,CAAC,CAAC,CAAC,CAAC;CACf;AAED;;;;;;;;;;GAUG;AACH,MAAM,WAAW,mBAAoB,SAAQ,WAAW;IACtD,UAAU,EAAE,UAAU,CAAC;CACxB;AA2ED;;;;;;;;;;;;;GAaG;AACH,eAAO,MAAM,cAAc,EAAE,OAAO,CAClC,sBAAsB,EACtB,gBAAgB,EAChB,mBAAmB,CA+EpB,CAAC"}

package/dist/temporary-internal-core/handlers/listApps.js

@@ -25,13 +25,26 @@ const DEFAULT_PAGE_SIZE = 20;
  * Augments implementation IDs with search results
  *
  * Calls the search endpoint and merges results with existing implementation IDs,
- * deduplicating by implementation name (without version).
+ * deduplicating by implementation name (without version) and preferring public versions.
  */
 async function augmentWithSearchResults({ searchTerm, implementationIds, httpClient, }) {
     const searchResponse = await httpClient.get("/zapier/api/v4/implementations-meta/search/", {
         searchParams: { term: searchTerm },
     });
-    const searchResults = searchResponse.results.map(transformImplementationMetaToAppItem);
+    // Deduplicate by implementation name, preferring public versions
+    // This ensures that when a user has access to both a private version (e.g., 1.0.17)
+    // and a public version (e.g., 1.1.0), we use the public one for search results
+    const byImplementationName = new Map();
+    for (const result of searchResponse.results) {
+        const [implementationName] = splitVersionedKey(result.id);
+        const isPublic = result.visibility === "public";
+        const existing = byImplementationName.get(implementationName);
+        // Take this result if we don't have one yet, or if this one is public and existing is not
+        if (!existing || (isPublic && !existing.isPublic)) {
+            byImplementationName.set(implementationName, { result, isPublic });
+        }
+    }
+    const searchResults = Array.from(byImplementationName.values()).map((entry) => transformImplementationMetaToAppItem(entry.result));
     const implementationNameSet = new Set(implementationIds.map((id) => {
         const [name] = splitVersionedKey(id);
         return name;

package/dist/temporary-internal-core/handlers/listApps.test.js

@@ -2,13 +2,14 @@ import { describe, it, expect, vi } from "vitest";
 import { handleListApps, } from "./listApps";
 import { ListAppsResponseSchema, } from "../schemas/apps";
 describe("handleListApps", () => {
-    const mockImplementation = (id, name) => ({
+    const mockImplementation = (id, name, visibility = "public") => ({
         id,
         name,
         slug: name.toLowerCase().replace(/\s+/g, "-"),
         description: `${name} description`,
         primary_color: "#000000",
         categories: [{ id: 1, name: "test", slug: "test" }],
+        visibility,
     });
     describe("pageSize handling", () => {
         it("should use default pageSize of 20 when not provided", async () => {
@@ -186,6 +187,44 @@ describe("handleListApps", () => {
                 }),
             }));
         });
+        it("should prefer public version when search returns multiple versions of same app", async () => {
+            // Simulates a user having access to both private (1.0.17) and public (1.1.0) versions
+            const mockSearchResponse = {
+                count: 2,
+                results: [
+                    mockImplementation("LumAppsCLIAPI@1.0.17", "LumApps (1.0.17)", "private"),
+                    mockImplementation("LumAppsCLIAPI@1.1.0", "LumApps", "public"),
+                ],
+            };
+            const mockLookupResponse = {
+                count: 1,
+                results: [
+                    mockImplementation("LumAppsCLIAPI@1.1.0", "LumApps", "public"),
+                ],
+                next: null,
+            };
+            const mockHttpClient = {
+                get: vi
+                    .fn()
+                    .mockResolvedValueOnce(mockSearchResponse)
+                    .mockResolvedValueOnce(mockLookupResponse),
+            };
+            await handleListApps({
+                request: {
+                    implementationIds: [],
+                    search: "lumapps",
+                    pageSize: undefined,
+                    cursor: undefined,
+                },
+                deps: { httpClient: mockHttpClient },
+            });
+            // Should request the public version (1.1.0), not the private one (1.0.17)
+            expect(mockHttpClient.get).toHaveBeenNthCalledWith(2, "/zapier/api/v4/implementations-meta/lookup/", expect.objectContaining({
+                searchParams: expect.objectContaining({
+                    selected_apis: "LumAppsCLIAPI@1.1.0",
+                }),
+            }));
+        });
     });
     describe("specific implementation IDs", () => {
         it("should fetch specific apps when implementationIds provided", async () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zapier/zapier-sdk",
-  "version": "0.16.1",
+  "version": "0.16.2",
   "description": "Complete Zapier SDK - combines all Zapier SDK packages",
   "main": "dist/index.cjs",
   "module": "dist/index.mjs",