@sparkvault/sdk 1.23.8 → 1.23.9

package/dist/sparkvault.esm.js

@@ -639,7 +639,10 @@ class IdentityApi {
     constructor(config, timeoutMs = DEFAULT_TIMEOUT_MS$1) {
         /** Cached config promise - allows preloading and deduplication */
         this.configCache = null;
-        /** Abort controller for cancelling all pending requests on close */
+        /**
+         * Abort controller for cancelling all pending requests on close.
+         * Replaced by resetSession() at the start of every user-facing session.
+         */
         this.closeController = new AbortController();
         this.config = config;
         this.timeoutMs = timeoutMs;
@@ -657,6 +660,16 @@ class IdentityApi {
     isAborted() {
         return this.closeController.signal.aborted;
     }
+    /**
+     * Reset the session lifecycle by swapping in a fresh AbortController.
+     * The Module calls this at the start of every user-facing session so a
+     * previously-closed session can never leak its aborted state into the
+     * next one. The config cache is intentionally preserved across sessions
+     * so preloadConfig() keeps making modal opens instant.
+     */
+    resetSession() {
+        this.closeController = new AbortController();
+    }
     get baseUrl() {
         return `${this.config.identityBaseUrl}/${this.config.accountId}`;
     }
@@ -6306,6 +6319,10 @@ class IdentityModule {
         if (this.renderer) {
             this.renderer.close();
         }
+        // Reset session lifecycle before every modal open. Without this, the
+        // AbortController aborted on close() would short-circuit the next
+        // session's first request as "Request cancelled".
+        this.api.resetSession();
         const isInline = !!options.target;
         const container = isInline
             ? this.createInlineContainer(options.target)
@@ -6545,7 +6562,10 @@ function isInitiateUploadResponse(data) {
 }
 class UploadApi {
     constructor(config) {
-        /** Abort controller for cancelling all pending requests on close */
+        /**
+         * Abort controller for cancelling all pending requests on close.
+         * Replaced by resetSession() at the start of every user-facing session.
+         */
         this.closeController = new AbortController();
         this.config = config;
         this.timeoutMs = config.timeout ?? DEFAULT_TIMEOUT_MS;
@@ -6569,6 +6589,15 @@ class UploadApi {
     getAbortSignal() {
         return this.closeController.signal;
     }
+    /**
+     * Reset the session lifecycle by swapping in a fresh AbortController.
+     * The Module calls this at the start of every user-facing session so a
+     * previously-closed session can never leak its aborted state into the
+     * next one. (Long-lived caches on `this` are intentionally preserved.)
+     */
+    resetSession() {
+        this.closeController = new AbortController();
+    }
     /**
      * Get vault upload info (public endpoint).
      */
@@ -9370,6 +9399,7 @@ class VaultUploadModule {
         this.renderer = null;
         this.attachedElements = new Map();
         this.config = config;
+        this.api = new UploadApi(config);
     }
     /**
      * Upload a file to a vault.
@@ -9404,12 +9434,12 @@ class VaultUploadModule {
             ...options,
             backdropBlur: options.backdropBlur ?? this.config.backdropBlur,
         };
-        // Fresh API instance per upload session: the AbortController inside the
-        // api is single-use, so reusing it across modal opens would make every
-        // request after the first close throw "Request cancelled".
-        const api = new UploadApi(this.config);
+        // Reset session lifecycle before every modal open. Without this, the
+        // AbortController aborted on close() would short-circuit the next
+        // session's first request as "Request cancelled".
+        this.api.resetSession();
         return new Promise((resolve, reject) => {
-            this.renderer = new UploadRenderer(container, api, mergedOptions, {
+            this.renderer = new UploadRenderer(container, this.api, mergedOptions, {
                 onSuccess: (result) => {
                     options.onSuccess?.(result);
                     resolve(result);