@zapier/zapier-sdk 0.18.0 → 0.18.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @zapier/zapier-sdk
 
+## 0.18.1
+
+### Patch Changes
+
+- ee5beb6: Refactor transport classes to factory functions
+
 ## 0.18.0
 
 ### Minor Changes

package/dist/index.cjs

@@ -4916,70 +4916,73 @@ var listInputFieldChoicesPlugin = ({ context, sdk }) => {
 // src/plugins/eventEmission/transport.ts
 var DEFAULT_RETRY_ATTEMPTS = 2;
 var DEFAULT_RETRY_DELAY_MS = 300;
-var HttpTransport = class {
-  constructor(config) {
-    this.config = config;
-  }
-  async emit(subject, event) {
-    try {
-      await this.emitWithRetry(
-        subject,
-        event,
-        this.config.retryAttempts || DEFAULT_RETRY_ATTEMPTS
-      );
-    } catch {
-    }
-  }
-  async emitWithRetry(subject, event, attemptsLeft) {
+function createHttpTransport(config) {
+  const delay = async (ms) => {
+    return new Promise((resolve2) => {
+      const timer = setTimeout(resolve2, ms);
+      if (typeof timer.unref === "function") {
+        timer.unref();
+      }
+    });
+  };
+  const emitWithRetry = async (subject, event, attemptsLeft) => {
     try {
       const payload = {
         subject,
         properties: event
       };
-      const response = await fetch(this.config.endpoint, {
+      const response = await fetch(config.endpoint, {
         method: "POST",
         headers: {
           "Content-Type": "application/json",
-          ...this.config.headers
+          ...config.headers
         },
         body: JSON.stringify(payload)
       });
       if (!response.ok && attemptsLeft > 1) {
-        await this.delay(this.config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
-        return this.emitWithRetry(subject, event, attemptsLeft - 1);
+        await delay(config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
+        return emitWithRetry(subject, event, attemptsLeft - 1);
       }
     } catch (error) {
       if (attemptsLeft > 1) {
-        await this.delay(this.config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
-        return this.emitWithRetry(subject, event, attemptsLeft - 1);
+        await delay(config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
+        return emitWithRetry(subject, event, attemptsLeft - 1);
       }
       throw error;
     }
-  }
-  async delay(ms) {
-    return new Promise((resolve2) => {
-      const timer = setTimeout(resolve2, ms);
-      if (typeof timer.unref === "function") {
-        timer.unref();
+  };
+  return {
+    async emit(subject, event) {
+      try {
+        await emitWithRetry(
+          subject,
+          event,
+          config.retryAttempts || DEFAULT_RETRY_ATTEMPTS
+        );
+      } catch {
       }
-    });
-  }
-};
-var ConsoleTransport = class {
-  async emit(subject, event) {
-    try {
-      console.log(
-        "[SDK Telemetry]",
-        JSON.stringify({ subject, properties: event }, null, 2)
-      );
-    } catch {
     }
-  }
-};
-var NoopTransport = class {
-  async emit(_subject, _event) {
-  }
-};
+  };
+}
+function createConsoleTransport() {
+  return {
+    async emit(subject, event) {
+      try {
+        console.log(
+          "[SDK Telemetry]",
+          JSON.stringify({ subject, properties: event }, null, 2)
+        );
+      } catch {
+      }
+    }
+  };
+}
+function createNoopTransport() {
+  return {
+    async emit(_subject, _event) {
+    }
+  };
+}
 function createTransport(config) {
   try {
     switch (config.type) {
@@ -4987,20 +4990,20 @@ function createTransport(config) {
         if (!config.endpoint) {
           throw new Error("HTTP transport requires endpoint");
         }
-        return new HttpTransport({
+        return createHttpTransport({
           endpoint: config.endpoint,
           headers: config.headers,
           retryAttempts: config.retryAttempts,
           retryDelayMs: config.retryDelayMs
         });
       case "console":
-        return new ConsoleTransport();
+        return createConsoleTransport();
       case "noop":
       default:
-        return new NoopTransport();
+        return createNoopTransport();
     }
   } catch {
-    return new NoopTransport();
+    return createNoopTransport();
   }
 }
 function generateEventId() {
@@ -5069,7 +5072,7 @@ function getCpuTime() {
 
 // package.json
 var package_default = {
-  version: "0.18.0"};
+  version: "0.18.1"};
 
 // src/plugins/eventEmission/builders.ts
 function createBaseEvent(context = {}) {

package/dist/index.mjs

@@ -4894,70 +4894,73 @@ var listInputFieldChoicesPlugin = ({ context, sdk }) => {
 // src/plugins/eventEmission/transport.ts
 var DEFAULT_RETRY_ATTEMPTS = 2;
 var DEFAULT_RETRY_DELAY_MS = 300;
-var HttpTransport = class {
-  constructor(config) {
-    this.config = config;
-  }
-  async emit(subject, event) {
-    try {
-      await this.emitWithRetry(
-        subject,
-        event,
-        this.config.retryAttempts || DEFAULT_RETRY_ATTEMPTS
-      );
-    } catch {
-    }
-  }
-  async emitWithRetry(subject, event, attemptsLeft) {
+function createHttpTransport(config) {
+  const delay = async (ms) => {
+    return new Promise((resolve2) => {
+      const timer = setTimeout(resolve2, ms);
+      if (typeof timer.unref === "function") {
+        timer.unref();
+      }
+    });
+  };
+  const emitWithRetry = async (subject, event, attemptsLeft) => {
     try {
       const payload = {
         subject,
         properties: event
       };
-      const response = await fetch(this.config.endpoint, {
+      const response = await fetch(config.endpoint, {
         method: "POST",
         headers: {
           "Content-Type": "application/json",
-          ...this.config.headers
+          ...config.headers
         },
         body: JSON.stringify(payload)
       });
       if (!response.ok && attemptsLeft > 1) {
-        await this.delay(this.config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
-        return this.emitWithRetry(subject, event, attemptsLeft - 1);
+        await delay(config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
+        return emitWithRetry(subject, event, attemptsLeft - 1);
       }
     } catch (error) {
       if (attemptsLeft > 1) {
-        await this.delay(this.config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
-        return this.emitWithRetry(subject, event, attemptsLeft - 1);
+        await delay(config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
+        return emitWithRetry(subject, event, attemptsLeft - 1);
       }
       throw error;
     }
-  }
-  async delay(ms) {
-    return new Promise((resolve2) => {
-      const timer = setTimeout(resolve2, ms);
-      if (typeof timer.unref === "function") {
-        timer.unref();
+  };
+  return {
+    async emit(subject, event) {
+      try {
+        await emitWithRetry(
+          subject,
+          event,
+          config.retryAttempts || DEFAULT_RETRY_ATTEMPTS
+        );
+      } catch {
       }
-    });
-  }
-};
-var ConsoleTransport = class {
-  async emit(subject, event) {
-    try {
-      console.log(
-        "[SDK Telemetry]",
-        JSON.stringify({ subject, properties: event }, null, 2)
-      );
-    } catch {
     }
-  }
-};
-var NoopTransport = class {
-  async emit(_subject, _event) {
-  }
-};
+  };
+}
+function createConsoleTransport() {
+  return {
+    async emit(subject, event) {
+      try {
+        console.log(
+          "[SDK Telemetry]",
+          JSON.stringify({ subject, properties: event }, null, 2)
+        );
+      } catch {
+      }
+    }
+  };
+}
+function createNoopTransport() {
+  return {
+    async emit(_subject, _event) {
+    }
+  };
+}
 function createTransport(config) {
   try {
     switch (config.type) {
@@ -4965,20 +4968,20 @@ function createTransport(config) {
         if (!config.endpoint) {
           throw new Error("HTTP transport requires endpoint");
         }
-        return new HttpTransport({
+        return createHttpTransport({
           endpoint: config.endpoint,
           headers: config.headers,
           retryAttempts: config.retryAttempts,
           retryDelayMs: config.retryDelayMs
         });
       case "console":
-        return new ConsoleTransport();
+        return createConsoleTransport();
       case "noop":
       default:
-        return new NoopTransport();
+        return createNoopTransport();
     }
   } catch {
-    return new NoopTransport();
+    return createNoopTransport();
   }
 }
 function generateEventId() {
@@ -5047,7 +5050,7 @@ function getCpuTime() {
 
 // package.json
 var package_default = {
-  version: "0.18.0"};
+  version: "0.18.1"};
 
 // src/plugins/eventEmission/builders.ts
 function createBaseEvent(context = {}) {

package/dist/plugins/eventEmission/transport.d.ts

@@ -15,23 +15,13 @@ export interface TransportConfig {
     retryAttempts?: number;
     retryDelayMs?: number;
 }
-export declare class HttpTransport implements EventTransport {
-    private config;
-    constructor(config: {
-        endpoint: string;
-        headers?: Record<string, string>;
-        retryAttempts?: number;
-        retryDelayMs?: number;
-    });
-    emit<T extends any>(subject: string, event: T): Promise<void>;
-    private emitWithRetry;
-    private delay;
-}
-export declare class ConsoleTransport implements EventTransport {
-    emit<T extends any>(subject: string, event: T): Promise<void>;
-}
-export declare class NoopTransport implements EventTransport {
-    emit<T extends any>(_subject: string, _event: T): Promise<void>;
-}
+export declare function createHttpTransport(config: {
+    endpoint: string;
+    headers?: Record<string, string>;
+    retryAttempts?: number;
+    retryDelayMs?: number;
+}): EventTransport;
+export declare function createConsoleTransport(): EventTransport;
+export declare function createNoopTransport(): EventTransport;
 export declare function createTransport(config: TransportConfig): EventTransport;
 //# sourceMappingURL=transport.d.ts.map

package/dist/plugins/eventEmission/transport.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"transport.d.ts","sourceRoot":"","sources":["../../../src/plugins/eventEmission/transport.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAOH,MAAM,WAAW,cAAc;IAC7B,IAAI,CAAC,CAAC,SAAS,GAAG,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAC9D,KAAK,CAAC,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;CACzB;AAGD,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,CAAC;IAClC,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACjC,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAGD,qBAAa,aAAc,YAAW,cAAc;IAEhD,OAAO,CAAC,MAAM;gBAAN,MAAM,EAAE;QACd,QAAQ,EAAE,MAAM,CAAC;QACjB,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;QACjC,aAAa,CAAC,EAAE,MAAM,CAAC;QACvB,YAAY,CAAC,EAAE,MAAM,CAAC;KACvB;IAGG,IAAI,CAAC,CAAC,SAAS,GAAG,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;YAYrD,aAAa;YAiCb,KAAK;CAQpB;AAGD,qBAAa,gBAAiB,YAAW,cAAc;IAC/C,IAAI,CAAC,CAAC,SAAS,GAAG,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;CAUpE;AAGD,qBAAa,aAAc,YAAW,cAAc;IAC5C,IAAI,CAAC,CAAC,SAAS,GAAG,EAAE,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;CAGtE;AAGD,wBAAgB,eAAe,CAAC,MAAM,EAAE,eAAe,GAAG,cAAc,CAyBvE"}
+{"version":3,"file":"transport.d.ts","sourceRoot":"","sources":["../../../src/plugins/eventEmission/transport.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAOH,MAAM,WAAW,cAAc;IAC7B,IAAI,CAAC,CAAC,SAAS,GAAG,EAAE,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAC9D,KAAK,CAAC,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;CACzB;AAGD,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,GAAG,SAAS,GAAG,MAAM,CAAC;IAClC,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACjC,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAGD,wBAAgB,mBAAmB,CAAC,MAAM,EAAE;IAC1C,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACjC,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB,GAAG,cAAc,CAwDjB;AAGD,wBAAgB,sBAAsB,IAAI,cAAc,CAavD;AAGD,wBAAgB,mBAAmB,IAAI,cAAc,CAMpD;AAGD,wBAAgB,eAAe,CAAC,MAAM,EAAE,eAAe,GAAG,cAAc,CAyBvE"}

package/dist/plugins/eventEmission/transport.js

@@ -8,70 +8,73 @@
 const DEFAULT_RETRY_ATTEMPTS = 2;
 const DEFAULT_RETRY_DELAY_MS = 300;
 // HTTP Transport - sends events to remote endpoint
-export class HttpTransport {
-    constructor(config) {
-        this.config = config;
-    }
-    async emit(subject, event) {
-        try {
-            await this.emitWithRetry(subject, event, this.config.retryAttempts || DEFAULT_RETRY_ATTEMPTS);
-        }
-        catch {
-            // Silent failure - never throw
-        }
-    }
-    async emitWithRetry(subject, event, attemptsLeft) {
+export function createHttpTransport(config) {
+    const delay = async (ms) => {
+        return new Promise((resolve) => {
+            const timer = setTimeout(resolve, ms);
+            if (typeof timer.unref === "function") {
+                timer.unref();
+            }
+        });
+    };
+    const emitWithRetry = async (subject, event, attemptsLeft) => {
         try {
             const payload = {
                 subject,
                 properties: event,
             };
-            const response = await fetch(this.config.endpoint, {
+            const response = await fetch(config.endpoint, {
                 method: "POST",
                 headers: {
                     "Content-Type": "application/json",
-                    ...this.config.headers,
+                    ...config.headers,
                 },
                 body: JSON.stringify(payload),
             });
             if (!response.ok && attemptsLeft > 1) {
-                await this.delay(this.config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
-                return this.emitWithRetry(subject, event, attemptsLeft - 1);
+                await delay(config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
+                return emitWithRetry(subject, event, attemptsLeft - 1);
             }
         }
         catch (error) {
             if (attemptsLeft > 1) {
-                await this.delay(this.config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
-                return this.emitWithRetry(subject, event, attemptsLeft - 1);
+                await delay(config.retryDelayMs || DEFAULT_RETRY_DELAY_MS);
+                return emitWithRetry(subject, event, attemptsLeft - 1);
             }
             throw error;
         }
-    }
-    async delay(ms) {
-        return new Promise((resolve) => {
-            const timer = setTimeout(resolve, ms);
-            if (typeof timer.unref === "function") {
-                timer.unref();
+    };
+    return {
+        async emit(subject, event) {
+            try {
+                await emitWithRetry(subject, event, config.retryAttempts || DEFAULT_RETRY_ATTEMPTS);
             }
-        });
-    }
+            catch {
+                // Silent failure - never throw
+            }
+        },
+    };
 }
 // Console Transport - logs events to console (for development)
-export class ConsoleTransport {
-    async emit(subject, event) {
-        try {
-            console.log("[SDK Telemetry]", JSON.stringify({ subject, properties: event }, null, 2));
-        }
-        catch {
-            // Silent failure - never throw
-        }
-    }
+export function createConsoleTransport() {
+    return {
+        async emit(subject, event) {
+            try {
+                console.log("[SDK Telemetry]", JSON.stringify({ subject, properties: event }, null, 2));
+            }
+            catch {
+                // Silent failure - never throw
+            }
+        },
+    };
 }
 // No-op Transport - discards all events (for testing/disabled state)
-export class NoopTransport {
-    async emit(_subject, _event) {
-        // Intentionally do nothing
-    }
+export function createNoopTransport() {
+    return {
+        async emit(_subject, _event) {
+            // Intentionally do nothing
+        },
+    };
 }
 // Transport factory
 export function createTransport(config) {
@@ -81,21 +84,21 @@ export function createTransport(config) {
                 if (!config.endpoint) {
                     throw new Error("HTTP transport requires endpoint");
                 }
-                return new HttpTransport({
+                return createHttpTransport({
                     endpoint: config.endpoint,
                     headers: config.headers,
                     retryAttempts: config.retryAttempts,
                     retryDelayMs: config.retryDelayMs,
                 });
             case "console":
-                return new ConsoleTransport();
+                return createConsoleTransport();
             case "noop":
             default:
-                return new NoopTransport();
+                return createNoopTransport();
         }
     }
     catch {
         // If transport creation fails, return noop to maintain silent operation
-        return new NoopTransport();
+        return createNoopTransport();
     }
 }

package/dist/plugins/eventEmission/transport.test.js

@@ -2,7 +2,7 @@
  * Tests for Event Transport Layer
  */
 import { describe, it, expect, vi, beforeEach } from "vitest";
-import { HttpTransport, ConsoleTransport, NoopTransport, createTransport, } from "./transport";
+import { createHttpTransport, createConsoleTransport, createNoopTransport, createTransport, } from "./transport";
 // Mock fetch globally
 const mockFetch = vi.fn();
 global.fetch = mockFetch;
@@ -20,13 +20,13 @@ describe("Transport Layer", () => {
     beforeEach(() => {
         vi.clearAllMocks();
     });
-    describe("HttpTransport", () => {
+    describe("createHttpTransport", () => {
         it("should emit events via HTTP successfully", async () => {
             mockFetch.mockResolvedValueOnce({
                 ok: true,
                 status: 200,
             });
-            const transport = new HttpTransport({
+            const transport = createHttpTransport({
                 endpoint: "https://telemetry.example.com/events",
                 headers: { "x-api-key": "test-key" },
             });
@@ -49,7 +49,7 @@ describe("Transport Layer", () => {
                 .mockResolvedValueOnce({ ok: false, status: 500 })
                 .mockResolvedValueOnce({ ok: false, status: 500 })
                 .mockResolvedValueOnce({ ok: true, status: 200 });
-            const transport = new HttpTransport({
+            const transport = createHttpTransport({
                 endpoint: "https://telemetry.example.com/events",
                 retryAttempts: 3,
                 retryDelayMs: 10, // Short delay for testing
@@ -59,7 +59,7 @@ describe("Transport Layer", () => {
         });
         it("should handle network errors silently", async () => {
             mockFetch.mockRejectedValue(new Error("Network error"));
-            const transport = new HttpTransport({
+            const transport = createHttpTransport({
                 endpoint: "https://telemetry.example.com/events",
                 retryAttempts: 1,
                 retryDelayMs: 1,
@@ -69,7 +69,7 @@ describe("Transport Layer", () => {
         });
         it("should stop retrying after max attempts", async () => {
             mockFetch.mockResolvedValue({ ok: false, status: 500 });
-            const transport = new HttpTransport({
+            const transport = createHttpTransport({
                 endpoint: "https://telemetry.example.com/events",
                 retryAttempts: 2,
                 retryDelayMs: 1,
@@ -78,9 +78,9 @@ describe("Transport Layer", () => {
             expect(mockFetch).toHaveBeenCalledTimes(2);
         });
     });
-    describe("ConsoleTransport", () => {
+    describe("createConsoleTransport", () => {
         it("should log events to console", async () => {
-            const transport = new ConsoleTransport();
+            const transport = createConsoleTransport();
             await transport.emit(sampleSubject, sampleEvent);
             expect(mockConsoleLog).toHaveBeenCalledWith("[SDK Telemetry]", JSON.stringify({ subject: sampleSubject, properties: sampleEvent }, null, 2));
         });
@@ -88,14 +88,14 @@ describe("Transport Layer", () => {
             mockConsoleLog.mockImplementation(() => {
                 throw new Error("Console error");
             });
-            const transport = new ConsoleTransport();
+            const transport = createConsoleTransport();
             // Should not throw despite console error
             await expect(transport.emit(sampleSubject, sampleEvent)).resolves.toBeUndefined();
         });
     });
-    describe("NoopTransport", () => {
+    describe("createNoopTransport", () => {
         it("should do nothing when emitting events", async () => {
-            const transport = new NoopTransport();
+            const transport = createNoopTransport();
             await transport.emit(sampleSubject, sampleEvent);
             // Verify no side effects
             expect(mockFetch).not.toHaveBeenCalled();
@@ -103,7 +103,8 @@ describe("Transport Layer", () => {
         });
     });
     describe("createTransport", () => {
-        it("should create HTTP transport with valid config", () => {
+        it("should create HTTP transport with valid config", async () => {
+            mockFetch.mockResolvedValueOnce({ ok: true, status: 200 });
             const transport = createTransport({
                 type: "http",
                 endpoint: "https://example.com",
@@ -111,42 +112,52 @@ describe("Transport Layer", () => {
                 retryAttempts: 5,
                 retryDelayMs: 2000,
             });
-            expect(transport).toBeInstanceOf(HttpTransport);
+            expect(transport.emit).toBeTypeOf("function");
+            await transport.emit(sampleSubject, sampleEvent);
+            expect(mockFetch).toHaveBeenCalledWith("https://example.com", expect.any(Object));
         });
-        it("should create console transport", () => {
+        it("should create console transport", async () => {
             const transport = createTransport({
                 type: "console",
             });
-            expect(transport).toBeInstanceOf(ConsoleTransport);
+            expect(transport.emit).toBeTypeOf("function");
+            await transport.emit(sampleSubject, sampleEvent);
+            expect(mockConsoleLog).toHaveBeenCalled();
         });
-        it("should create noop transport by default", () => {
+        it("should create noop transport by default", async () => {
             const transport = createTransport({
                 type: "noop",
             });
-            expect(transport).toBeInstanceOf(NoopTransport);
+            expect(transport.emit).toBeTypeOf("function");
+            await transport.emit(sampleSubject, sampleEvent);
+            expect(mockFetch).not.toHaveBeenCalled();
+            expect(mockConsoleLog).not.toHaveBeenCalled();
         });
-        it("should fallback to noop transport on invalid HTTP config", () => {
+        it("should fallback to noop transport on invalid HTTP config", async () => {
             const transport = createTransport({
                 type: "http",
                 // Missing endpoint
             });
-            expect(transport).toBeInstanceOf(NoopTransport);
+            expect(transport.emit).toBeTypeOf("function");
+            await transport.emit(sampleSubject, sampleEvent);
+            expect(mockFetch).not.toHaveBeenCalled();
         });
-        it("should fallback to noop transport on unknown type", () => {
+        it("should fallback to noop transport on unknown type", async () => {
             const transport = createTransport({
                 type: "unknown",
             });
-            expect(transport).toBeInstanceOf(NoopTransport);
+            expect(transport.emit).toBeTypeOf("function");
+            await transport.emit(sampleSubject, sampleEvent);
+            expect(mockFetch).not.toHaveBeenCalled();
+            expect(mockConsoleLog).not.toHaveBeenCalled();
         });
-        it("should handle transport creation errors gracefully", () => {
-            // Since createTransport already handles errors internally,
-            // this test verifies that invalid configs don't throw
+        it("should handle transport creation errors gracefully", async () => {
             expect(() => {
                 const transport = createTransport({
                     type: "http",
                     // Missing required endpoint - should fallback to noop
                 });
-                expect(transport).toBeInstanceOf(NoopTransport);
+                expect(transport.emit).toBeTypeOf("function");
             }).not.toThrow();
         });
     });

package/package.json

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