Address review comments: UMD exports for Workers, ECONNABORTED on abort, setCookie returns header string

Agent-Logs-Url: https://github.com/Crowdhandler/crowdhandler-javascript-sdk/sessions/c6b7fcba-80a4-4ac9-af1a-0db689f102cd

Co-authored-by: luke-owen-crowdhandler <60788788+luke-owen-crowdhandler@users.noreply.github.com>

Author: Copilot

package-lock.json

@@ -17,8 +17,8 @@
   "exports": {
     ".": {
       "types": "./dist/types/index.d.ts",
-      "workerd": "./dist/crowdhandler.esm.js",
-      "worker": "./dist/crowdhandler.esm.js",
+      "workerd": "./dist/crowdhandler.umd.js",
+      "worker": "./dist/crowdhandler.umd.js",
       "browser": "./dist/crowdhandler.umd.js",
       "import": "./dist/crowdhandler.esm.js",
       "require": "./dist/crowdhandler.cjs.js",

package.json

@@ -108,6 +108,9 @@ export class BaseClient {
       // Mirror axios's "no response received" error shape so errorHandler's
       // `else if (error.request)` branch fires.
       const wrapped: any = new Error(err?.message || "Network request failed");
+      if (controller.signal.aborted || err?.name === "AbortError") {
+        wrapped.code = "ECONNABORTED";
+      }
       wrapped.request = { url: finalUrl, method };
       wrapped.config = { url: finalUrl, method };
       throw wrapped;

src/client/base_client.ts

@@ -962,14 +962,25 @@ export class Gatekeeper {
    * 
    * @param {string} value - The cookie value to set (from result.cookieValue)
    * @param {string} domain - Optional domain pattern to determine cookie domain scope
-   * @returns {boolean} True if the cookie was successfully set, false otherwise
+   * @returns {boolean | string} In Node.js/Lambda/browser environments returns true on success
+   *   or false on failure. In Cloudflare Workers returns the Set-Cookie header string that
+   *   must be applied to the outgoing Response by the caller.
    * 
    * @example
+   * // Node.js / Lambda
    * if (result.setCookie) {
    *   gatekeeper.setCookie(result.cookieValue, result.domain);
    * }
+   * 
+   * @example
+   * // Cloudflare Workers
+   * if (result.setCookie) {
+   *   const setCookieHeader = gatekeeper.setCookie(result.cookieValue, result.domain);
+   *   // setCookieHeader is the Set-Cookie header value — apply it to the Response:
+   *   // response.headers.append('Set-Cookie', setCookieHeader as string);
+   * }
    */
-  public setCookie(value: string, domain?: string): boolean {
+  public setCookie(value: string, domain?: string): boolean | string {
     try {
       // Determine cookie domain if domain pattern is provided
       let cookieDomain: string | undefined;
@@ -981,9 +992,12 @@ export class Gatekeeper {
         }
       }
 
-      // Set the cookie with the provided value and options
-      this.REQUEST.setCookie(value, this.STORAGE_NAME, cookieDomain);
-      return true;
+      // Set the cookie with the provided value and options.
+      // CloudflareWorkersHandler returns the Set-Cookie header string because
+      // Workers are response-out and the caller must apply the header manually.
+      // All other handlers set the cookie as a side-effect and return void.
+      const result = this.REQUEST.setCookie(value, this.STORAGE_NAME, cookieDomain);
+      return typeof result === 'string' ? result : true;
     } catch (error: any) {
       logger(this.options.debug, "error", error);
       return false;