@stackframe/stack-shared 2.7.3 → 2.7.5

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @stackframe/stack-shared
 
+## 2.7.5
+
+### Patch Changes
+
+- Various changes
+  - @stackframe/stack-sc@2.7.5
+
+## 2.7.4
+
+### Patch Changes
+
+- @stackframe/stack-sc@2.7.4
+
 ## 2.7.3
 
 ### Patch Changes

package/dist/utils/caches.d.ts

@@ -50,6 +50,7 @@ declare class AsyncValueCache<T> {
     private readonly _rateLimitOptions;
     private _subscriptionsCount;
     private _unsubscribers;
+    private _mostRecentRefreshPromiseIndex;
     constructor(fetcher: () => Promise<T>, _options?: {
         onSubscribe?: (refresh: () => void) => (() => void);
         rateLimiter?: Omit<RateLimitOptions, "batchCalls">;

package/dist/utils/caches.js

@@ -63,6 +63,7 @@ class AsyncValueCache {
         this._options = _options;
         this._subscriptionsCount = 0;
         this._unsubscribers = [];
+        this._mostRecentRefreshPromiseIndex = 0;
         this._store = new AsyncStore();
         this._rateLimitOptions = {
             concurrency: 1,
@@ -132,17 +133,8 @@ class AsyncValueCache {
         const storeObj = this._store.onChange(callback);
         runAsynchronously(this.getOrWait("read-write"));
         if (this._subscriptionsCount++ === 0 && this._options.onSubscribe) {
-            let mostRecentRefreshPromiseIndex = 0;
             const unsubscribe = this._options.onSubscribe(() => {
-                const currentRefreshPromiseIndex = mostRecentRefreshPromiseIndex++;
-                runAsynchronously(async () => {
-                    // wait a few seconds; if anything changes during that time, we don't want to refresh
-                    // else we do unnecessary requests if we unsubscribe and then subscribe again immediately
-                    await wait(5000);
-                    if (this._subscriptionsCount === 0 && currentRefreshPromiseIndex === mostRecentRefreshPromiseIndex) {
-                        this.invalidate();
-                    }
-                });
+                runAsynchronously(this.refresh());
             });
             this._unsubscribers.push(unsubscribe);
         }
@@ -154,6 +146,15 @@ class AsyncValueCache {
                 hasUnsubscribed = true;
                 storeObj.unsubscribe();
                 if (--this._subscriptionsCount === 0) {
+                    const currentRefreshPromiseIndex = ++this._mostRecentRefreshPromiseIndex;
+                    runAsynchronously(async () => {
+                        // wait a few seconds; if anything changes during that time, we don't want to refresh
+                        // else we do unnecessary requests if we unsubscribe and then subscribe again immediately
+                        await wait(5000);
+                        if (this._subscriptionsCount === 0 && currentRefreshPromiseIndex === this._mostRecentRefreshPromiseIndex) {
+                            this.invalidate();
+                        }
+                    });
                     for (const unsubscribe of this._unsubscribers) {
                         unsubscribe();
                     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stackframe/stack-shared",
-  "version": "2.7.3",
+  "version": "2.7.5",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "files": [
@@ -51,7 +51,7 @@
     "oauth4webapi": "^2.10.3",
     "semver": "^7.6.3",
     "uuid": "^9.0.1",
-    "@stackframe/stack-sc": "2.7.3"
+    "@stackframe/stack-sc": "2.7.5"
   },
   "devDependencies": {
     "@sentry/nextjs": "^8.40.0",