@zapier/zapier-sdk 0.15.1 → 0.15.3

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

@@ -0,0 +1,328 @@
+import { describe, it, expect, vi } from "vitest";
+import { handleListApps, } from "./listApps";
+import { ListAppsResponseSchema, } from "../schemas/apps";
+describe("handleListApps", () => {
+    const mockImplementation = (id, name) => ({
+        id,
+        name,
+        slug: name.toLowerCase().replace(/\s+/g, "-"),
+        description: `${name} description`,
+        primary_color: "#000000",
+        categories: [{ id: 1, name: "test", slug: "test" }],
+    });
+    describe("pageSize handling", () => {
+        it("should use default pageSize of 20 when not provided", async () => {
+            const mockHttpClient = {
+                get: vi.fn().mockResolvedValue({
+                    count: 0,
+                    results: [],
+                }),
+            };
+            await handleListApps({
+                request: {
+                    implementationIds: [],
+                    search: undefined,
+                    pageSize: undefined,
+                    cursor: undefined,
+                },
+                deps: { httpClient: mockHttpClient },
+            });
+            expect(mockHttpClient.get).toHaveBeenCalledWith(expect.any(String), expect.objectContaining({
+                searchParams: expect.objectContaining({
+                    limit: "20",
+                }),
+            }));
+        });
+        it("should use custom pageSize when provided", async () => {
+            const mockHttpClient = {
+                get: vi.fn().mockResolvedValue({
+                    count: 0,
+                    results: [],
+                }),
+            };
+            await handleListApps({
+                request: {
+                    implementationIds: [],
+                    search: undefined,
+                    pageSize: 50,
+                    cursor: undefined,
+                },
+                deps: { httpClient: mockHttpClient },
+            });
+            expect(mockHttpClient.get).toHaveBeenCalledWith(expect.any(String), expect.objectContaining({
+                searchParams: expect.objectContaining({
+                    limit: "50",
+                }),
+            }));
+        });
+    });
+    describe("empty implementation IDs handling", () => {
+        it("should fetch all apps when no implementationIds and no search", async () => {
+            const mockResponse = {
+                count: 2,
+                results: [
+                    mockImplementation("SlackCLIAPI@1.0.0", "Slack"),
+                    mockImplementation("GitHubCLIAPI@1.0.0", "GitHub"),
+                ],
+            };
+            const mockHttpClient = {
+                get: vi.fn().mockResolvedValue(mockResponse),
+            };
+            const result = await handleListApps({
+                request: {
+                    implementationIds: [],
+                    search: undefined,
+                    pageSize: undefined,
+                    cursor: undefined,
+                },
+                deps: { httpClient: mockHttpClient },
+            });
+            expect(result.data).toHaveLength(2);
+            expect(mockHttpClient.get).toHaveBeenCalledWith("/zapier/api/v4/implementations-meta/lookup/", expect.objectContaining({
+                searchParams: expect.objectContaining({
+                    latest_only: "true",
+                    selected_apis: "",
+                }),
+            }));
+        });
+        it("should return empty when search finds nothing (bug fix)", async () => {
+            const mockSearchResponse = {
+                count: 0,
+                results: [],
+            };
+            const mockHttpClient = {
+                get: vi.fn().mockResolvedValue(mockSearchResponse),
+            };
+            const result = await handleListApps({
+                request: {
+                    implementationIds: [],
+                    search: "NonexistentApp",
+                    pageSize: undefined,
+                    cursor: undefined,
+                },
+                deps: { httpClient: mockHttpClient },
+            });
+            // Should only call search endpoint
+            expect(mockHttpClient.get).toHaveBeenCalledTimes(1);
+            expect(mockHttpClient.get).toHaveBeenCalledWith("/zapier/api/v4/implementations-meta/search/", expect.objectContaining({
+                searchParams: { term: "NonexistentApp" },
+            }));
+            // Should return empty (bug fix - not all apps)
+            expect(result.data).toHaveLength(0);
+            expect(result.nextCursor).toBeUndefined();
+        });
+    });
+    describe("search augmentation", () => {
+        it("should augment results with search when search term provided", async () => {
+            const mockSearchResponse = {
+                count: 1,
+                results: [mockImplementation("SlackCLIAPI@1.0.0", "Slack")],
+            };
+            const mockLookupResponse = {
+                count: 2,
+                results: [
+                    mockImplementation("GitHubCLIAPI@1.0.0", "GitHub"),
+                    mockImplementation("SlackCLIAPI@1.0.0", "Slack"),
+                ],
+                next: null,
+            };
+            const mockHttpClient = {
+                get: vi
+                    .fn()
+                    .mockResolvedValueOnce(mockSearchResponse) // First call: search
+                    .mockResolvedValueOnce(mockLookupResponse), // Second call: lookup
+            };
+            const result = await handleListApps({
+                request: {
+                    implementationIds: ["GitHubCLIAPI@1.0.0"],
+                    search: "Slack",
+                    pageSize: undefined,
+                    cursor: undefined,
+                },
+                deps: { httpClient: mockHttpClient },
+            });
+            // Should call both search and lookup
+            expect(mockHttpClient.get).toHaveBeenCalledTimes(2);
+            expect(mockHttpClient.get).toHaveBeenNthCalledWith(1, "/zapier/api/v4/implementations-meta/search/", expect.objectContaining({
+                searchParams: { term: "Slack" },
+            }));
+            expect(mockHttpClient.get).toHaveBeenNthCalledWith(2, "/zapier/api/v4/implementations-meta/lookup/", expect.objectContaining({
+                searchParams: expect.objectContaining({
+                    selected_apis: "GitHubCLIAPI@1.0.0,SlackCLIAPI@1.0.0",
+                }),
+            }));
+            expect(result.data).toHaveLength(2);
+        });
+        it("should not duplicate apps when search returns apps already in implementationIds", async () => {
+            const mockSearchResponse = {
+                count: 1,
+                results: [mockImplementation("SlackCLIAPI@1.0.0", "Slack")],
+            };
+            const mockLookupResponse = {
+                count: 1,
+                results: [mockImplementation("SlackCLIAPI@1.0.0", "Slack")],
+                next: null,
+            };
+            const mockHttpClient = {
+                get: vi
+                    .fn()
+                    .mockResolvedValueOnce(mockSearchResponse)
+                    .mockResolvedValueOnce(mockLookupResponse),
+            };
+            await handleListApps({
+                request: {
+                    implementationIds: ["SlackCLIAPI@1.0.0"],
+                    search: "Slack",
+                    pageSize: undefined,
+                    cursor: undefined,
+                },
+                deps: { httpClient: mockHttpClient },
+            });
+            // Should call search and lookup but deduplicate IDs
+            expect(mockHttpClient.get).toHaveBeenCalledTimes(2);
+            expect(mockHttpClient.get).toHaveBeenNthCalledWith(2, "/zapier/api/v4/implementations-meta/lookup/", expect.objectContaining({
+                searchParams: expect.objectContaining({
+                    selected_apis: "SlackCLIAPI@1.0.0",
+                }),
+            }));
+        });
+    });
+    describe("specific implementation IDs", () => {
+        it("should fetch specific apps when implementationIds provided", async () => {
+            const mockResponse = {
+                count: 2,
+                results: [
+                    mockImplementation("SlackCLIAPI@1.0.0", "Slack"),
+                    mockImplementation("GitHubCLIAPI@1.0.0", "GitHub"),
+                ],
+            };
+            const mockHttpClient = {
+                get: vi.fn().mockResolvedValue(mockResponse),
+            };
+            const result = await handleListApps({
+                request: {
+                    implementationIds: ["SlackCLIAPI@1.0.0", "GitHubCLIAPI@1.0.0"],
+                    search: undefined,
+                    pageSize: undefined,
+                    cursor: undefined,
+                },
+                deps: { httpClient: mockHttpClient },
+            });
+            expect(result.data).toHaveLength(2);
+            expect(mockHttpClient.get).toHaveBeenCalledWith("/zapier/api/v4/implementations-meta/lookup/", expect.objectContaining({
+                searchParams: expect.objectContaining({
+                    selected_apis: "SlackCLIAPI@1.0.0,GitHubCLIAPI@1.0.0",
+                }),
+            }));
+        });
+    });
+    describe("pagination", () => {
+        it("should pass cursor to API when provided", async () => {
+            const mockResponse = {
+                count: 0,
+                results: [],
+            };
+            const mockHttpClient = {
+                get: vi.fn().mockResolvedValue(mockResponse),
+            };
+            await handleListApps({
+                request: {
+                    implementationIds: ["SlackCLIAPI@1.0.0"],
+                    search: undefined,
+                    pageSize: undefined,
+                    cursor: "page-2-cursor",
+                },
+                deps: { httpClient: mockHttpClient },
+            });
+            expect(mockHttpClient.get).toHaveBeenCalledWith(expect.any(String), expect.objectContaining({
+                searchParams: expect.objectContaining({
+                    offset: "page-2-cursor",
+                }),
+            }));
+        });
+        it("should extract nextCursor from response", async () => {
+            const mockResponse = {
+                count: 1,
+                results: [mockImplementation("SlackCLIAPI@1.0.0", "Slack")],
+                next: "https://api.zapier.com/v4/implementations-meta/lookup/?offset=next-page",
+            };
+            const mockHttpClient = {
+                get: vi.fn().mockResolvedValue(mockResponse),
+            };
+            const result = await handleListApps({
+                request: {
+                    implementationIds: [],
+                    search: undefined,
+                    pageSize: undefined,
+                    cursor: undefined,
+                },
+                deps: { httpClient: mockHttpClient },
+            });
+            expect(result.nextCursor).toBe("next-page");
+        });
+    });
+    describe("request validation", () => {
+        it("should reject request with invalid implementationIds type", async () => {
+            const mockHttpClient = {
+                get: vi.fn(),
+            };
+            const invalidRequest = {
+                implementationIds: "not-an-array",
+                search: undefined,
+                pageSize: undefined,
+                cursor: undefined,
+            };
+            const deps = {
+                httpClient: mockHttpClient,
+            };
+            await expect(handleListApps({ request: invalidRequest, deps })).rejects.toThrow();
+        });
+        it("should reject request with negative pageSize", async () => {
+            const mockHttpClient = {
+                get: vi.fn(),
+            };
+            const invalidRequest = {
+                implementationIds: [],
+                search: undefined,
+                pageSize: -1,
+                cursor: undefined,
+            };
+            const deps = {
+                httpClient: mockHttpClient,
+            };
+            await expect(handleListApps({ request: invalidRequest, deps })).rejects.toThrow();
+        });
+    });
+    describe("response validation", () => {
+        it("should return response matching ListAppsResponseSchema", async () => {
+            const mockHttpClient = {
+                get: vi.fn().mockResolvedValue({
+                    count: 1,
+                    results: [
+                        {
+                            id: "SlackCLIAPI@1.0.0",
+                            name: "SlackCLIAPI",
+                            slug: "slack",
+                            description: "Slack app",
+                            primary_color: "#000000",
+                        },
+                    ],
+                }),
+            };
+            const request = {
+                implementationIds: [],
+                search: undefined,
+                pageSize: 20,
+                cursor: undefined,
+            };
+            const deps = {
+                httpClient: mockHttpClient,
+            };
+            const result = await handleListApps({ request, deps });
+            expect(() => ListAppsResponseSchema.parse(result)).not.toThrow();
+            expect(result).toHaveProperty("data");
+            expect(Array.isArray(result.data)).toBe(true);
+            expect(result).toHaveProperty("nextCursor");
+        });
+    });
+});

package/dist/temporary-internal-core/index.d.ts

@@ -11,4 +11,8 @@
  * - Lives inside zapier-sdk during extraction/migration (bundles when published)
  * - Will be extracted to separate package
  */
+export type { HandlerDeps, Handler, ValidatedHandler } from "./types";
+export * from "./schemas/apps";
+export * from "./schemas/implementations";
+export { handleListApps, type ListAppsHandlerDeps } from "./handlers/listApps";
 //# sourceMappingURL=index.d.ts.map

package/dist/temporary-internal-core/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/temporary-internal-core/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/temporary-internal-core/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAGH,YAAY,EAAE,WAAW,EAAE,OAAO,EAAE,gBAAgB,EAAE,MAAM,SAAS,CAAC;AAGtE,cAAc,gBAAgB,CAAC;AAC/B,cAAc,2BAA2B,CAAC;AAG1C,OAAO,EAAE,cAAc,EAAE,KAAK,mBAAmB,EAAE,MAAM,qBAAqB,CAAC"}

package/dist/temporary-internal-core/index.js

@@ -1,4 +1,3 @@
-"use strict";
 /**
  * temporary-internal-core
  *
@@ -12,3 +11,8 @@
  * - Lives inside zapier-sdk during extraction/migration (bundles when published)
  * - Will be extracted to separate package
  */
+// Schemas
+export * from "./schemas/apps";
+export * from "./schemas/implementations";
+// Handlers
+export { handleListApps } from "./handlers/listApps";