@kookapp/web-bridge 0.0.2 → 0.0.3

package/README.md

@@ -9,7 +9,8 @@ A PostMessage-based bridge for iframe communication, providing symmetric callHan
 - **Origin Validation**: Strict domain whitelist verification
 - **Type-Safe**: Full TypeScript support with comprehensive type definitions
 - **Promise & Callback**: Supports both async/await and callback patterns
-- **Error Handling**: Comprehensive error codes and timeout management
+- **No Exceptions**: Errors returned via result values, never throws exceptions (like globalBridge)
+- **Error Handling**: Comprehensive error reporting through return values
 
 ## Installation
 
@@ -39,10 +40,22 @@ bridge.registerHandler('getUserInfo', (data, callback) => {
 const iframeElement = document.getElementById('myIframe') as HTMLIFrameElement
 const channel = bridge.createChannel(iframeElement)
 
-// Call a handler inside iframe
-channel.callHandler('getIframeData', { type: 'user' })
-  .then(result => console.log('Result:', result))
-  .catch(error => console.error('Error:', error))
+// Call a handler inside iframe (using callback)
+channel.callHandler('getIframeData', { type: 'user' }, (result) => {
+  if (result.error) {
+    console.error('Error:', result.error)
+  } else {
+    console.log('Result:', result)
+  }
+})
+
+// Or using async/await (without callback)
+const result = await channel.callHandler('getIframeData', { type: 'user' })
+if (result.error) {
+  console.error('Error:', result.error)
+} else {
+  console.log('Result:', result)
+}
 
 // Clean up
 // bridge.destroy()
@@ -64,10 +77,22 @@ bridge.registerHandler('getIframeData', (data, callback) => {
   callback(response)
 })
 
-// Call a handler on parent
-bridge.callHandler('getUserInfo', {})
-  .then(userInfo => console.log('User:', userInfo))
-  .catch(error => console.error('Error:', error))
+// Call a handler on parent (using callback)
+bridge.callHandler('getUserInfo', {}, (userInfo) => {
+  if (userInfo.error) {
+    console.error('Error:', userInfo.error)
+  } else {
+    console.log('User:', userInfo)
+  }
+})
+
+// Or using async/await
+const userInfo = await bridge.callHandler('getUserInfo', {})
+if (userInfo.error) {
+  console.error('Error:', userInfo.error)
+} else {
+  console.log('User:', userInfo)
+}
 ```
 
 ## API Reference
@@ -121,13 +146,40 @@ interface BridgeConfig {
 
 ## Error Handling
 
-The bridge throws `BridgeError` with specific error codes:
+**WebBridge never throws exceptions** - all errors are returned through the result value, keeping behavior consistent with globalBridge.
+
+All errors are returned as objects with an `error` property:
+
+```typescript
+// Using callback
+bridge.callHandler('someHandler', {}, (result) => {
+  if (result.error) {
+    // Handle error cases:
+    // - Handler not found
+    // - Request timeout
+    // - Invalid message
+    // - Connection issues
+    console.error('Error:', result.error)
+  } else {
+    console.log('Success:', result)
+  }
+})
+
+// Using Promise (never rejects!)
+const result = await bridge.callHandler('someHandler', {})
+if (result.error) {
+  console.error('Error:', result.error)
+} else {
+  console.log('Success:', result)
+}
+```
+
+**Common error types:**
+- Handler not found: `{ error: 'Handler "handlerName" not found' }`
+- Timeout: `{ error: 'Timeout: no response after 5000ms' }`
+- Connection issues: `{ error: 'iframe contentWindow is not available' }`
 
-- `HANDLER_NOT_FOUND`: Handler not found on the other side
-- `ORIGIN_NOT_ALLOWED`: Message origin not in whitelist
-- `TIMEOUT`: Request exceeded timeout
-- `INVALID_MESSAGE`: Malformed message received
-- `CONNECTION_LOST`: Connection to other side lost
+See [ERROR_HANDLING.md](./ERROR_HANDLING.md) for complete documentation.
 
 ## Security
 

package/dist/child/ChildBridge.d.ts

@@ -18,6 +18,8 @@ export declare class ChildBridge extends MessageHandler {
     private notifyReady;
     /**
      * Call a handler on parent
+     * If callback is provided: calls callback with result (data or error object), returns Promise<undefined>
+     * If callback is not provided: returns Promise<result|error>
      */
     callHandler(handlerName: string, data?: any, callback?: (result: any) => void): Promise<any>;
     /**

package/dist/child/ChildBridge.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ChildBridge.d.ts","sourceRoot":"","sources":["../../src/child/ChildBridge.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,iBAAiB,EAAE,aAAa,EAAE,MAAM,SAAS,CAAA;AAC1D,OAAO,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAA;AAGxD,qBAAa,WAAY,SAAQ,cAAc;IAC7C,OAAO,CAAC,YAAY,CAAQ;IAC5B,OAAO,CAAC,YAAY,CAAQ;IAC5B,OAAO,CAAC,SAAS,CAAiB;gBAEtB,MAAM,EAAE,iBAAiB;IA+BrC;;OAEG;IACH,OAAO,CAAC,eAAe;IAyBvB;;OAEG;IACH,OAAO,CAAC,WAAW;IAgBnB;;OAEG;IACG,WAAW,CACf,WAAW,EAAE,MAAM,EACnB,IAAI,CAAC,EAAE,GAAG,EACV,QAAQ,CAAC,EAAE,CAAC,MAAM,EAAE,GAAG,KAAK,IAAI,GAC/B,OAAO,CAAC,GAAG,CAAC;IAUf;;OAEG;IACH,SAAS,CAAC,WAAW,CAAC,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,aAAa,GAAG,IAAI;IAezE;;OAEG;IACH,SAAS,CAAC,eAAe,IAAI,MAAM;IAInC;;OAEG;IACH,WAAW,IAAI,OAAO;IAItB;;OAEG;IACH,OAAO,IAAI,IAAI;CAQhB"}
+{"version":3,"file":"ChildBridge.d.ts","sourceRoot":"","sources":["../../src/child/ChildBridge.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,iBAAiB,EAAE,aAAa,EAAE,MAAM,SAAS,CAAA;AAC1D,OAAO,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAA;AAGxD,qBAAa,WAAY,SAAQ,cAAc;IAC7C,OAAO,CAAC,YAAY,CAAQ;IAC5B,OAAO,CAAC,YAAY,CAAQ;IAC5B,OAAO,CAAC,SAAS,CAAiB;gBAEtB,MAAM,EAAE,iBAAiB;IA+BrC;;OAEG;IACH,OAAO,CAAC,eAAe;IAyBvB;;OAEG;IACH,OAAO,CAAC,WAAW;IAgBnB;;;;OAIG;IACG,WAAW,CACf,WAAW,EAAE,MAAM,EACnB,IAAI,CAAC,EAAE,GAAG,EACV,QAAQ,CAAC,EAAE,CAAC,MAAM,EAAE,GAAG,KAAK,IAAI,GAC/B,OAAO,CAAC,GAAG,CAAC;IAaf;;OAEG;IACH,SAAS,CAAC,WAAW,CAAC,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,aAAa,GAAG,IAAI;IAezE;;OAEG;IACH,SAAS,CAAC,eAAe,IAAI,MAAM;IAInC;;OAEG;IACH,WAAW,IAAI,OAAO;IAItB;;OAEG;IACH,OAAO,IAAI,IAAI;CAQhB"}

package/dist/child/ChildBridge.js

@@ -74,12 +74,17 @@ export class ChildBridge extends MessageHandler {
     }
     /**
      * Call a handler on parent
+     * If callback is provided: calls callback with result (data or error object), returns Promise<undefined>
+     * If callback is not provided: returns Promise<result|error>
      */
     async callHandler(handlerName, data, callback) {
         const promise = this.callHandlerWithTimeout(this.parentOrigin, handlerName, data);
         if (callback) {
-            promise.then(callback).catch(error => console.error(error));
+            // With callback: pass result to callback, return undefined
+            promise.then(callback);
+            return undefined;
         }
+        // Without callback: return the promise with result or error object
         return promise;
     }
     /**

package/dist/parent/IframeChannel.d.ts

@@ -14,6 +14,8 @@ export declare class IframeChannel extends MessageHandler implements IframeChann
     private extractOrigin;
     /**
      * Call a handler in the iframe
+     * If callback is provided: calls callback with result (data or error object), returns Promise<undefined>
+     * If callback is not provided: returns Promise<result|error>
      */
     callHandler(handlerName: string, data?: any, callback?: (result: any) => void): Promise<any>;
     /**

package/dist/parent/IframeChannel.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"IframeChannel.d.ts","sourceRoot":"","sources":["../../src/parent/IframeChannel.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,aAAa,EAAE,sBAAsB,EAAE,MAAM,SAAS,CAAA;AAC/D,OAAO,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAA;AAExD,qBAAa,aAAc,SAAQ,cAAe,YAAW,sBAAsB;IACjF,OAAO,CAAC,MAAM,CAAmB;IACjC,OAAO,CAAC,YAAY,CAAQ;IAC5B,OAAO,CAAC,SAAS,CAAiB;gBAGhC,MAAM,EAAE,iBAAiB,EACzB,cAAc,EAAE,MAAM,EAAE,EACxB,OAAO,GAAE,MAAa,EACtB,KAAK,GAAE,OAAe;IAmBxB;;OAEG;IACH,OAAO,CAAC,aAAa;IASrB;;OAEG;IACG,WAAW,CACf,WAAW,EAAE,MAAM,EACnB,IAAI,CAAC,EAAE,GAAG,EACV,QAAQ,CAAC,EAAE,CAAC,MAAM,EAAE,GAAG,KAAK,IAAI,GAC/B,OAAO,CAAC,GAAG,CAAC;IAcf;;OAEG;IACH,SAAS,CAAC,WAAW,CAAC,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,aAAa,GAAG,IAAI;IAkBzE;;OAEG;IACH,SAAS,CAAC,eAAe,IAAI,MAAM;IAInC;;OAEG;IACH,SAAS,IAAI,iBAAiB;IAI9B;;OAEG;IACH,WAAW,IAAI,OAAO;IAItB;;OAEG;IACH,YAAY,CAAC,SAAS,EAAE,OAAO,GAAG,IAAI;IAItC;;OAEG;IACH,OAAO,IAAI,IAAI;CAIhB"}
+{"version":3,"file":"IframeChannel.d.ts","sourceRoot":"","sources":["../../src/parent/IframeChannel.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,aAAa,EAAE,sBAAsB,EAAE,MAAM,SAAS,CAAA;AAC/D,OAAO,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAA;AAExD,qBAAa,aAAc,SAAQ,cAAe,YAAW,sBAAsB;IACjF,OAAO,CAAC,MAAM,CAAmB;IACjC,OAAO,CAAC,YAAY,CAAQ;IAC5B,OAAO,CAAC,SAAS,CAAiB;gBAGhC,MAAM,EAAE,iBAAiB,EACzB,cAAc,EAAE,MAAM,EAAE,EACxB,OAAO,GAAE,MAAa,EACtB,KAAK,GAAE,OAAe;IAmBxB;;OAEG;IACH,OAAO,CAAC,aAAa;IASrB;;;;OAIG;IACG,WAAW,CACf,WAAW,EAAE,MAAM,EACnB,IAAI,CAAC,EAAE,GAAG,EACV,QAAQ,CAAC,EAAE,CAAC,MAAM,EAAE,GAAG,KAAK,IAAI,GAC/B,OAAO,CAAC,GAAG,CAAC;IAsBf;;OAEG;IACH,SAAS,CAAC,WAAW,CAAC,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,aAAa,GAAG,IAAI;IAkBzE;;OAEG;IACH,SAAS,CAAC,eAAe,IAAI,MAAM;IAInC;;OAEG;IACH,SAAS,IAAI,iBAAiB;IAI9B;;OAEG;IACH,WAAW,IAAI,OAAO;IAItB;;OAEG;IACH,YAAY,CAAC,SAAS,EAAE,OAAO,GAAG,IAAI;IAItC;;OAEG;IACH,OAAO,IAAI,IAAI;CAIhB"}

package/dist/parent/IframeChannel.js

@@ -34,15 +34,25 @@ export class IframeChannel extends MessageHandler {
     }
     /**
      * Call a handler in the iframe
+     * If callback is provided: calls callback with result (data or error object), returns Promise<undefined>
+     * If callback is not provided: returns Promise<result|error>
      */
     async callHandler(handlerName, data, callback) {
         if (!this.iframe.contentWindow) {
-            throw new Error('iframe contentWindow is not available');
+            const error = { error: 'iframe contentWindow is not available' };
+            if (callback) {
+                callback(error);
+                return undefined;
+            }
+            return error;
         }
         const promise = this.callHandlerWithTimeout(this.iframeOrigin, handlerName, data);
         if (callback) {
-            promise.then(callback).catch(error => console.error(error));
+            // With callback: pass result to callback, return undefined
+            promise.then(callback);
+            return undefined;
         }
+        // Without callback: return the promise with result or error object
         return promise;
     }
     /**

package/dist/shared/MessageHandler.d.ts

@@ -25,6 +25,7 @@ export declare abstract class MessageHandler {
     unregisterHandler(handlerName: string): void;
     /**
      * Call a handler on the other side with timeout
+     * Never rejects - errors are resolved as {error: string}
      */
     protected callHandlerWithTimeout(targetOrigin: string, handlerName: string, data?: any): Promise<any>;
     /**
@@ -45,6 +46,7 @@ export declare abstract class MessageHandler {
     private handleCall;
     /**
      * Handle incoming response message
+     * Never rejects - always resolves with data or error object
      */
     private handleResponse;
     /**

package/dist/shared/MessageHandler.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"MessageHandler.d.ts","sourceRoot":"","sources":["../../src/shared/MessageHandler.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,aAAa,EAAE,eAAe,EAAE,cAAc,EAAE,MAAM,SAAS,CAAA;AACxE,OAAO,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAA;AACpD,OAAO,EAAE,eAAe,EAAE,MAAM,0BAA0B,CAAA;AAC1D,OAAO,EAAE,WAAW,EAAE,MAAM,sBAAsB,CAAA;AAIlD,8BAAsB,cAAc;IAClC,SAAS,CAAC,QAAQ,EAAE,eAAe,CAAY;IAC/C,SAAS,CAAC,eAAe,EAAE,GAAG,CAAC,MAAM,EAAE,cAAc,CAAC,CAAY;IAClE,SAAS,CAAC,YAAY,EAAE,YAAY,CAAqB;IACzD,SAAS,CAAC,eAAe,EAAE,eAAe,CAAA;IAC1C,SAAS,CAAC,WAAW,EAAE,WAAW,CAAoB;IACtD,SAAS,CAAC,OAAO,EAAE,MAAM,CAAA;IACzB,SAAS,CAAC,KAAK,EAAE,OAAO,CAAA;IACxB,SAAS,CAAC,eAAe,EAAE,CAAC,CAAC,KAAK,EAAE,YAAY,KAAK,IAAI,CAAC,GAAG,IAAI,CAAO;gBAE5D,cAAc,EAAE,MAAM,EAAE,EAAE,OAAO,GAAE,MAAwB,EAAE,KAAK,GAAE,OAAe;IAM/F;;OAEG;IACH,eAAe,CAAC,WAAW,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC,IAAI,CAAC,EAAE,GAAG,EAAE,QAAQ,CAAC,EAAE,QAAQ,KAAK,IAAI,GAAG,IAAI;IAO9F;;OAEG;IACH,iBAAiB,CAAC,WAAW,EAAE,MAAM,GAAG,IAAI;IAO5C;;OAEG;IACH,SAAS,CAAC,sBAAsB,CAC9B,YAAY,EAAE,MAAM,EACpB,WAAW,EAAE,MAAM,EACnB,IAAI,CAAC,EAAE,GAAG,GACT,OAAO,CAAC,GAAG,CAAC;IA2Bf;;OAEG;IACH,SAAS,CAAC,QAAQ,CAAC,WAAW,CAAC,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,aAAa,GAAG,IAAI;IAElF;;OAEG;IACH,SAAS,CAAC,QAAQ,CAAC,eAAe,IAAI,MAAM;IAE5C;;OAEG;IACH,SAAS,CAAC,aAAa,CAAC,OAAO,EAAE,aAAa,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI;IAgCrE;;OAEG;YACW,UAAU;IAyCxB;;OAEG;IACH,OAAO,CAAC,cAAc;IAqBtB;;OAEG;IACH,SAAS,CAAC,oBAAoB,CAAC,OAAO,EAAE,CAAC,KAAK,EAAE,YAAY,KAAK,IAAI,GAAG,IAAI;IAK5E;;OAEG;IACH,SAAS,CAAC,qBAAqB,IAAI,IAAI;IAOvC;;OAEG;IACH,QAAQ,CAAC,WAAW,IAAI,OAAO;IAE/B;;OAEG;IACH,EAAE,CAAC,IAAI,EAAE,OAAO,GAAG,YAAY,GAAG,OAAO,EAAE,QAAQ,EAAE,CAAC,IAAI,CAAC,EAAE,GAAG,KAAK,IAAI,GAAG,IAAI;IAIhF;;OAEG;IACH,GAAG,CAAC,IAAI,EAAE,OAAO,GAAG,YAAY,GAAG,OAAO,EAAE,QAAQ,EAAE,CAAC,IAAI,CAAC,EAAE,GAAG,KAAK,IAAI,GAAG,IAAI;IAIjF;;OAEG;IACH,OAAO,IAAI,IAAI;CAOhB"}
+{"version":3,"file":"MessageHandler.d.ts","sourceRoot":"","sources":["../../src/shared/MessageHandler.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,aAAa,EAAE,eAAe,EAAE,cAAc,EAAE,MAAM,SAAS,CAAA;AACxE,OAAO,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAA;AACpD,OAAO,EAAE,eAAe,EAAE,MAAM,0BAA0B,CAAA;AAC1D,OAAO,EAAE,WAAW,EAAE,MAAM,sBAAsB,CAAA;AAGlD,8BAAsB,cAAc;IAClC,SAAS,CAAC,QAAQ,EAAE,eAAe,CAAY;IAC/C,SAAS,CAAC,eAAe,EAAE,GAAG,CAAC,MAAM,EAAE,cAAc,CAAC,CAAY;IAClE,SAAS,CAAC,YAAY,EAAE,YAAY,CAAqB;IACzD,SAAS,CAAC,eAAe,EAAE,eAAe,CAAA;IAC1C,SAAS,CAAC,WAAW,EAAE,WAAW,CAAoB;IACtD,SAAS,CAAC,OAAO,EAAE,MAAM,CAAA;IACzB,SAAS,CAAC,KAAK,EAAE,OAAO,CAAA;IACxB,SAAS,CAAC,eAAe,EAAE,CAAC,CAAC,KAAK,EAAE,YAAY,KAAK,IAAI,CAAC,GAAG,IAAI,CAAO;gBAE5D,cAAc,EAAE,MAAM,EAAE,EAAE,OAAO,GAAE,MAAwB,EAAE,KAAK,GAAE,OAAe;IAM/F;;OAEG;IACH,eAAe,CAAC,WAAW,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC,IAAI,CAAC,EAAE,GAAG,EAAE,QAAQ,CAAC,EAAE,QAAQ,KAAK,IAAI,GAAG,IAAI;IAO9F;;OAEG;IACH,iBAAiB,CAAC,WAAW,EAAE,MAAM,GAAG,IAAI;IAO5C;;;OAGG;IACH,SAAS,CAAC,sBAAsB,CAC9B,YAAY,EAAE,MAAM,EACpB,WAAW,EAAE,MAAM,EACnB,IAAI,CAAC,EAAE,GAAG,GACT,OAAO,CAAC,GAAG,CAAC;IAsCf;;OAEG;IACH,SAAS,CAAC,QAAQ,CAAC,WAAW,CAAC,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,aAAa,GAAG,IAAI;IAElF;;OAEG;IACH,SAAS,CAAC,QAAQ,CAAC,eAAe,IAAI,MAAM;IAE5C;;OAEG;IACH,SAAS,CAAC,aAAa,CAAC,OAAO,EAAE,aAAa,EAAE,MAAM,EAAE,MAAM,GAAG,IAAI;IAgCrE;;OAEG;YACW,UAAU;IAgDxB;;;OAGG;IACH,OAAO,CAAC,cAAc;IAsBtB;;OAEG;IACH,SAAS,CAAC,oBAAoB,CAAC,OAAO,EAAE,CAAC,KAAK,EAAE,YAAY,KAAK,IAAI,GAAG,IAAI;IAK5E;;OAEG;IACH,SAAS,CAAC,qBAAqB,IAAI,IAAI;IAOvC;;OAEG;IACH,QAAQ,CAAC,WAAW,IAAI,OAAO;IAE/B;;OAEG;IACH,EAAE,CAAC,IAAI,EAAE,OAAO,GAAG,YAAY,GAAG,OAAO,EAAE,QAAQ,EAAE,CAAC,IAAI,CAAC,EAAE,GAAG,KAAK,IAAI,GAAG,IAAI;IAIhF;;OAEG;IACH,GAAG,CAAC,IAAI,EAAE,OAAO,GAAG,YAAY,GAAG,OAAO,EAAE,QAAQ,EAAE,CAAC,IAAI,CAAC,EAAE,GAAG,KAAK,IAAI,GAAG,IAAI;IAIjF;;OAEG;IACH,OAAO,IAAI,IAAI;CAOhB"}

package/dist/shared/MessageHandler.js

@@ -4,7 +4,6 @@
 import { EventEmitter } from '../shared/EventEmitter';
 import { OriginValidator } from '../shared/OriginValidator';
 import { IdGenerator } from '../shared/IdGenerator';
-import { errorFactory } from '../errors/errorFactory';
 import { DEFAULT_TIMEOUT } from '../constants';
 export class MessageHandler {
     constructor(allowedOrigins, timeout = DEFAULT_TIMEOUT, debug = false) {
@@ -37,19 +36,32 @@ export class MessageHandler {
     }
     /**
      * Call a handler on the other side with timeout
+     * Never rejects - errors are resolved as {error: string}
      */
     callHandlerWithTimeout(targetOrigin, handlerName, data) {
-        return new Promise((resolve, reject) => {
+        return new Promise((resolve) => {
             const messageId = this.idGenerator.generate();
             const timeoutHandle = setTimeout(() => {
                 this.pendingRequests.delete(messageId);
-                reject(errorFactory.timeout(this.timeout));
+                resolve({ error: `Timeout: no response after ${this.timeout}ms` });
             }, this.timeout);
             this.pendingRequests.set(messageId, {
                 id: messageId,
                 timeout: timeoutHandle,
-                resolve,
-                reject
+                resolve: (data) => {
+                    // Wrap successful responses
+                    if (data !== undefined && !data.error) {
+                        resolve(data);
+                    }
+                    else {
+                        resolve(data);
+                    }
+                },
+                reject: (error) => {
+                    // Never reject - resolve errors instead
+                    const errorMessage = error instanceof Error ? error.message : String(error);
+                    resolve({ error: errorMessage });
+                }
             });
             const message = {
                 type: 'call',
@@ -109,16 +121,20 @@ export class MessageHandler {
             return;
         }
         try {
-            // Execute handler
+            let callbackCalled = false;
+            // Execute handler with callback function
             const result = await Promise.resolve(handler(data, (response) => {
-                this.sendMessage(origin, {
-                    type: 'response',
-                    id,
-                    data: response
-                });
+                if (!callbackCalled) {
+                    callbackCalled = true;
+                    this.sendMessage(origin, {
+                        type: 'response',
+                        id,
+                        data: response
+                    });
+                }
             }));
-            // If handler returns a value directly (not using callback)
-            if (result !== undefined) {
+            // If handler returns a value and callback wasn't called, send the return value
+            if (!callbackCalled && result !== undefined) {
                 this.sendMessage(origin, {
                     type: 'response',
                     id,
@@ -137,6 +153,7 @@ export class MessageHandler {
     }
     /**
      * Handle incoming response message
+     * Never rejects - always resolves with data or error object
      */
     handleResponse(message) {
         const { id, data, error } = message;
@@ -150,7 +167,8 @@ export class MessageHandler {
         clearTimeout(pending.timeout);
         this.pendingRequests.delete(id);
         if (error) {
-            pending.reject(new Error(error));
+            // Resolve with error object instead of rejecting
+            pending.resolve({ error });
         }
         else {
             pending.resolve(data);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kookapp/web-bridge",
-  "version": "0.0.2",
+  "version": "0.0.3",
   "description": "PostMessage-based bridge for iframe communication",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",