@atproto-labs/simple-store 0.2.0 → 0.4.0-next.0

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # @atproto-labs/simple-store
 
+## 0.4.0-next.0
+
+### Minor Changes
+
+- [#4929](https://github.com/bluesky-social/atproto/pull/4929) [`bb7491c`](https://github.com/bluesky-social/atproto/commit/bb7491c29e06181e1d2f8cf6eb454f9bb8ab961b) Thanks [@devinivy](https://github.com/devinivy)! - **BREAKING:** Drop support for Node.js 18 and 20. Node.js 22 is now the minimum supported version. Docker images now use Node.js 24.
+
+- [#4943](https://github.com/bluesky-social/atproto/pull/4943) [`07ae5d4`](https://github.com/bluesky-social/atproto/commit/07ae5d4452df51e045e0239da7a04cf0bc154028) Thanks [@devinivy](https://github.com/devinivy)! - **BREAKING:** Convert to pure ESM. All packages now ship `"type": "module"` with ES module output and Node16 module resolution.
+
+  Node.js 22's `require()` compatibility layer can still load these packages in CommonJS code.
+
+- [#4930](https://github.com/bluesky-social/atproto/pull/4930) [`042df15`](https://github.com/bluesky-social/atproto/commit/042df15087c0e62cd1e715fcbf58852fab875af9) Thanks [@devinivy](https://github.com/devinivy)! - Build with TypeScript 6.0. Emitted `.d.ts` files now use TypeScript 6's stricter `Uint8Array<ArrayBuffer>` typing in places where Web/Node APIs require buffer-backed (not shared-memory) byte arrays. Consumers compiling against these types on older TypeScript should see no runtime impact, but may need to widen or cast in spots that previously relied on `Uint8Array` defaulting to `<ArrayBufferLike>`.
+
+  Internal: tsconfig `moduleResolution: "node"` is silenced via `ignoreDeprecations: "6.0"` for now; the proper migration to `node16`/`bundler` resolution is deferred.
+
+## 0.3.0
+
+### Minor Changes
+
+- [#4108](https://github.com/bluesky-social/atproto/pull/4108) [`f9dc9aa4c`](https://github.com/bluesky-social/atproto/commit/f9dc9aa4c9eaf2f82d140fbf011a9015e7f1a00d) Thanks [@matthieusieben](https://github.com/matthieusieben)! - Allow `Key` type to be anything (except nullable values)
+
+- [#4108](https://github.com/bluesky-social/atproto/pull/4108) [`f9dc9aa4c`](https://github.com/bluesky-social/atproto/commit/f9dc9aa4c9eaf2f82d140fbf011a9015e7f1a00d) Thanks [@matthieusieben](https://github.com/matthieusieben)! - Update `Getter` function's option argument to `GetterOptions`
+
 ## 0.2.0
 
 ### Minor Changes

package/LICENSE.txt

@@ -1,6 +1,6 @@
 Dual MIT/Apache-2.0 License
 
-Copyright (c) 2022-2025 Bluesky PBC, and Contributors
+Copyright (c) 2022-2026 Bluesky Social PBC, and Contributors
 
 Except as otherwise noted in individual files, this software is licensed under the MIT license (<http://opensource.org/licenses/MIT>), or the Apache License, Version 2.0 (<http://www.apache.org/licenses/LICENSE-2.0>).
 

package/dist/cached-getter.d.ts

@@ -1,6 +1,7 @@
-import { Awaitable, GetOptions as GetStoredOptions, Key, SimpleStore, Value } from './simple-store.js';
-export type { GetStoredOptions };
-export type GetCachedOptions<C = void> = {
+import { GetOptions, Key, SimpleStore, Value } from './simple-store.js';
+import { Awaitable, ContextOptions } from './util.js';
+export type { GetOptions };
+export type GetCachedOptions<C = void> = ContextOptions<C> & {
     signal?: AbortSignal;
     /**
      * Do not use the cache to get the value. Always get a new value from the
@@ -19,12 +20,13 @@ export type GetCachedOptions<C = void> = {
      * @default false // If no isStale option was provided to the CachedGetter
      */
     allowStale?: boolean;
-} & (C extends void ? {
-    context?: C;
-} : {
-    context: C;
-});
-export type Getter<K extends Key, V extends Value, C = void> = (key: K, options: GetCachedOptions<C>, storedValue: undefined | V) => Awaitable<V>;
+};
+export type GetterOptions<C = void> = {
+    context: C extends void ? undefined : C;
+    noCache: boolean;
+    signal?: AbortSignal;
+};
+export type Getter<K extends Key, V extends Value, C = void> = (key: K, options: GetterOptions<C>, storedValue: undefined | V) => Awaitable<V>;
 export type CachedGetterOptions<K extends Key, V extends Value> = {
     isStale?: (key: K, value: V) => boolean | PromiseLike<boolean>;
     onStoreError?: (err: unknown, key: K, value: V) => void | PromiseLike<void>;
@@ -37,12 +39,12 @@ export type CachedGetterOptions<K extends Key, V extends Value> = {
 export declare class CachedGetter<K extends Key = string, V extends Value = Value, C = void> {
     readonly getter: Getter<K, V, C>;
     readonly store: SimpleStore<K, V>;
-    readonly options?: CachedGetterOptions<K, V> | undefined;
-    private pending;
-    constructor(getter: Getter<K, V, C>, store: SimpleStore<K, V>, options?: CachedGetterOptions<K, V> | undefined);
+    readonly options: CachedGetterOptions<K, V>;
+    private readonly pending;
+    constructor(getter: Getter<K, V, C>, store: SimpleStore<K, V>, options?: CachedGetterOptions<K, V>);
     get(key: C extends void ? K : never, options?: GetCachedOptions<C>): Promise<V>;
     get(key: C extends void ? never : K, options: GetCachedOptions<C>): Promise<V>;
-    getStored(key: K, options?: GetStoredOptions): Promise<V | undefined>;
+    getStored(key: K, options?: GetOptions): Promise<V | undefined>;
     setStored(key: K, value: V): Promise<void>;
     delStored(key: K, _cause?: unknown): Promise<void>;
 }

package/dist/cached-getter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cached-getter.d.ts","sourceRoot":"","sources":["../src/cached-getter.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,SAAS,EACT,UAAU,IAAI,gBAAgB,EAC9B,GAAG,EACH,WAAW,EACX,KAAK,EACN,MAAM,mBAAmB,CAAA;AAE1B,YAAY,EAAE,gBAAgB,EAAE,CAAA;AAChC,MAAM,MAAM,gBAAgB,CAAC,CAAC,GAAG,IAAI,IAAI;IACvC,MAAM,CAAC,EAAE,WAAW,CAAA;IAEpB;;;;;OAKG;IACH,OAAO,CAAC,EAAE,OAAO,CAAA;IAEjB;;;;;;;;OAQG;IACH,UAAU,CAAC,EAAE,OAAO,CAAA;CACrB,GAAG,CAAC,CAAC,SAAS,IAAI,GAAG;IAAE,OAAO,CAAC,EAAE,CAAC,CAAA;CAAE,GAAG;IAAE,OAAO,EAAE,CAAC,CAAA;CAAE,CAAC,CAAA;AAEvD,MAAM,MAAM,MAAM,CAAC,CAAC,SAAS,GAAG,EAAE,CAAC,SAAS,KAAK,EAAE,CAAC,GAAG,IAAI,IAAI,CAC7D,GAAG,EAAE,CAAC,EACN,OAAO,EAAE,gBAAgB,CAAC,CAAC,CAAC,EAC5B,WAAW,EAAE,SAAS,GAAG,CAAC,KACvB,SAAS,CAAC,CAAC,CAAC,CAAA;AAEjB,MAAM,MAAM,mBAAmB,CAAC,CAAC,SAAS,GAAG,EAAE,CAAC,SAAS,KAAK,IAAI;IAChE,OAAO,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,KAAK,OAAO,GAAG,WAAW,CAAC,OAAO,CAAC,CAAA;IAC9D,YAAY,CAAC,EAAE,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,KAAK,IAAI,GAAG,WAAW,CAAC,IAAI,CAAC,CAAA;IAC3E,aAAa,CAAC,EAAE,CACd,GAAG,EAAE,OAAO,EACZ,GAAG,EAAE,CAAC,EACN,KAAK,EAAE,CAAC,KACL,OAAO,GAAG,WAAW,CAAC,OAAO,CAAC,CAAA;CACpC,CAAA;AAOD;;;GAGG;AACH,qBAAa,YAAY,CACvB,CAAC,SAAS,GAAG,GAAG,MAAM,EACtB,CAAC,SAAS,KAAK,GAAG,KAAK,EACvB,CAAC,GAAG,IAAI;IAKN,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;IAChC,QAAQ,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC,EAAE,CAAC,CAAC;IACjC,QAAQ,CAAC,OAAO,CAAC,EAAE,mBAAmB,CAAC,CAAC,EAAE,CAAC,CAAC;IAL9C,OAAO,CAAC,OAAO,CAA+B;gBAGnC,MAAM,EAAE,MAAM,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,EACvB,KAAK,EAAE,WAAW,CAAC,CAAC,EAAE,CAAC,CAAC,EACxB,OAAO,CAAC,EAAE,mBAAmB,CAAC,CAAC,EAAE,CAAC,CAAC,YAAA;IAGxC,GAAG,CACP,GAAG,EAAE,CAAC,SAAS,IAAI,GAAG,CAAC,GAAG,KAAK,EAC/B,OAAO,CAAC,EAAE,gBAAgB,CAAC,CAAC,CAAC,GAC5B,OAAO,CAAC,CAAC,CAAC;IACP,GAAG,CACP,GAAG,EAAE,CAAC,SAAS,IAAI,GAAG,KAAK,GAAG,CAAC,EAC/B,OAAO,EAAE,gBAAgB,CAAC,CAAC,CAAC,GAC3B,OAAO,CAAC,CAAC,CAAC;IAsFP,SAAS,CAAC,GAAG,EAAE,CAAC,EAAE,OAAO,CAAC,EAAE,gBAAgB,GAAG,OAAO,CAAC,CAAC,GAAG,SAAS,CAAC;IAQrE,SAAS,CAAC,GAAG,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;IAS1C,SAAS,CAAC,GAAG,EAAE,CAAC,EAAE,MAAM,CAAC,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;CAGzD"}
+{"version":3,"file":"cached-getter.d.ts","sourceRoot":"","sources":["../src/cached-getter.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,GAAG,EAAE,WAAW,EAAE,KAAK,EAAE,MAAM,mBAAmB,CAAA;AACvE,OAAO,EAAE,SAAS,EAAE,cAAc,EAAE,MAAM,WAAW,CAAA;AAErD,YAAY,EAAE,UAAU,EAAE,CAAA;AAC1B,MAAM,MAAM,gBAAgB,CAAC,CAAC,GAAG,IAAI,IAAI,cAAc,CAAC,CAAC,CAAC,GAAG;IAC3D,MAAM,CAAC,EAAE,WAAW,CAAA;IAEpB;;;;;OAKG;IACH,OAAO,CAAC,EAAE,OAAO,CAAA;IAEjB;;;;;;;;OAQG;IACH,UAAU,CAAC,EAAE,OAAO,CAAA;CACrB,CAAA;AAED,MAAM,MAAM,aAAa,CAAC,CAAC,GAAG,IAAI,IAAI;IACpC,OAAO,EAAE,CAAC,SAAS,IAAI,GAAG,SAAS,GAAG,CAAC,CAAA;IACvC,OAAO,EAAE,OAAO,CAAA;IAChB,MAAM,CAAC,EAAE,WAAW,CAAA;CACrB,CAAA;AAED,MAAM,MAAM,MAAM,CAAC,CAAC,SAAS,GAAG,EAAE,CAAC,SAAS,KAAK,EAAE,CAAC,GAAG,IAAI,IAAI,CAC7D,GAAG,EAAE,CAAC,EACN,OAAO,EAAE,aAAa,CAAC,CAAC,CAAC,EACzB,WAAW,EAAE,SAAS,GAAG,CAAC,KACvB,SAAS,CAAC,CAAC,CAAC,CAAA;AAEjB,MAAM,MAAM,mBAAmB,CAAC,CAAC,SAAS,GAAG,EAAE,CAAC,SAAS,KAAK,IAAI;IAChE,OAAO,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,KAAK,OAAO,GAAG,WAAW,CAAC,OAAO,CAAC,CAAA;IAC9D,YAAY,CAAC,EAAE,CAAC,GAAG,EAAE,OAAO,EAAE,GAAG,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,KAAK,IAAI,GAAG,WAAW,CAAC,IAAI,CAAC,CAAA;IAC3E,aAAa,CAAC,EAAE,CACd,GAAG,EAAE,OAAO,EACZ,GAAG,EAAE,CAAC,EACN,KAAK,EAAE,CAAC,KACL,OAAO,GAAG,WAAW,CAAC,OAAO,CAAC,CAAA;CACpC,CAAA;AAOD;;;GAGG;AACH,qBAAa,YAAY,CACvB,CAAC,SAAS,GAAG,GAAG,MAAM,EACtB,CAAC,SAAS,KAAK,GAAG,KAAK,EACvB,CAAC,GAAG,IAAI;IAKN,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;IAChC,QAAQ,CAAC,KAAK,EAAE,WAAW,CAAC,CAAC,EAAE,CAAC,CAAC;IACjC,QAAQ,CAAC,OAAO,EAAE,mBAAmB,CAAC,CAAC,EAAE,CAAC,CAAC;IAL7C,OAAO,CAAC,QAAQ,CAAC,OAAO,CAA+B;gBAG5C,MAAM,EAAE,MAAM,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,EACvB,KAAK,EAAE,WAAW,CAAC,CAAC,EAAE,CAAC,CAAC,EACxB,OAAO,GAAE,mBAAmB,CAAC,CAAC,EAAE,CAAC,CAAM;IAG5C,GAAG,CACP,GAAG,EAAE,CAAC,SAAS,IAAI,GAAG,CAAC,GAAG,KAAK,EAC/B,OAAO,CAAC,EAAE,gBAAgB,CAAC,CAAC,CAAC,GAC5B,OAAO,CAAC,CAAC,CAAC;IACP,GAAG,CACP,GAAG,EAAE,CAAC,SAAS,IAAI,GAAG,KAAK,GAAG,CAAC,EAC/B,OAAO,EAAE,gBAAgB,CAAC,CAAC,CAAC,GAC3B,OAAO,CAAC,CAAC,CAAC;IAsGP,SAAS,CAAC,GAAG,EAAE,CAAC,EAAE,OAAO,CAAC,EAAE,UAAU,GAAG,OAAO,CAAC,CAAC,GAAG,SAAS,CAAC;IAQ/D,SAAS,CAAC,GAAG,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;IAS1C,SAAS,CAAC,GAAG,EAAE,CAAC,EAAE,MAAM,CAAC,EAAE,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC;CAGzD"}

package/dist/cached-getter.js

@@ -1,59 +1,40 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.CachedGetter = void 0;
 const returnTrue = () => true;
 const returnFalse = () => false;
 /**
  * Wrapper utility that uses a store to speed up the retrieval of values from an
  * (expensive) getter function.
  */
-class CachedGetter {
-    constructor(getter, store, options) {
-        Object.defineProperty(this, "getter", {
-            enumerable: true,
-            configurable: true,
-            writable: true,
-            value: getter
-        });
-        Object.defineProperty(this, "store", {
-            enumerable: true,
-            configurable: true,
-            writable: true,
-            value: store
-        });
-        Object.defineProperty(this, "options", {
-            enumerable: true,
-            configurable: true,
-            writable: true,
-            value: options
-        });
-        Object.defineProperty(this, "pending", {
-            enumerable: true,
-            configurable: true,
-            writable: true,
-            value: new Map()
-        });
+export class CachedGetter {
+    constructor(getter, store, options = {}) {
+        this.getter = getter;
+        this.store = store;
+        this.options = options;
+        this.pending = new Map();
     }
-    async get(key, options = {}) {
-        options.signal?.throwIfAborted();
-        const isStale = this.options?.isStale;
-        const allowStored = options.noCache
+    async get(key, { signal, context, allowStale = false, noCache = false, } = {}) {
+        signal?.throwIfAborted();
+        const { isStale, deleteOnError } = this.options;
+        const allowStored = noCache
             ? returnFalse // Never allow stored values to be returned
-            : options.allowStale || isStale == null
+            : allowStale || isStale == null
                 ? returnTrue // Always allow stored values to be returned
                 : async (value) => !(await isStale(key, value));
         // As long as concurrent requests are made for the same key, only one
-        // request will be made to the cache & getter function at a time. This works
-        // because there is no async operation between the while() loop and the
-        // pending.set() call. Because of the "single threaded" nature of
+        // request will be made to the getStored & getter functions at a time. This
+        // works because there is no async operation between the while() loop and
+        // the pending.set() call below. Because of the single threaded nature of
         // JavaScript, the pending item will be set before the next iteration of the
-        // while loop.
+        // while loop of any concurrent request.
         let previousExecutionFlow;
         while ((previousExecutionFlow = this.pending.get(key))) {
             try {
+                // If a concurrent request is already in progress, wait for it to finish
                 const { isFresh, value } = await previousExecutionFlow;
+                // Use the concurrent request's result if it is fresh
                 if (isFresh)
                     return value;
+                // Use the concurrent request's result if not fresh (loaded from the
+                // store), and matches the conditions for using a stored value.
                 if (await allowStored(value))
                     return value;
             }
@@ -61,11 +42,12 @@ class CachedGetter {
                 // Ignore errors from previous execution flows (they will have been
                 // propagated by that flow).
             }
-            options.signal?.throwIfAborted();
+            // Break the loop if the signal was aborted
+            signal?.throwIfAborted();
         }
         const currentExecutionFlow = Promise.resolve()
             .then(async () => {
-            const storedValue = await this.getStored(key, options);
+            const storedValue = await this.getStored(key, { signal });
             if (storedValue !== undefined && (await allowStored(storedValue))) {
                 // Use the stored value as return value for the current execution
                 // flow. Notify other concurrent execution flows (that should be
@@ -74,11 +56,13 @@ class CachedGetter {
                 return { isFresh: false, value: storedValue };
             }
             return Promise.resolve()
-                .then(async () => (0, this.getter)(key, options, storedValue))
+                .then(async () => {
+                const options = { signal, noCache, context };
+                return this.getter.call(null, key, options, storedValue);
+            })
                 .catch(async (err) => {
                 if (storedValue !== undefined) {
                     try {
-                        const deleteOnError = this.options?.deleteOnError;
                         if (await deleteOnError?.(err, key, storedValue)) {
                             await this.delStored(key, err);
                         }
@@ -130,5 +114,4 @@ class CachedGetter {
         await this.store.del(key);
     }
 }
-exports.CachedGetter = CachedGetter;
 //# sourceMappingURL=cached-getter.js.map

package/dist/cached-getter.js.map

@@ -1 +1 @@
-{"version":3,"file":"cached-getter.js","sourceRoot":"","sources":["../src/cached-getter.ts"],"names":[],"mappings":";;;AAkDA,MAAM,UAAU,GAAG,GAAG,EAAE,CAAC,IAAI,CAAA;AAC7B,MAAM,WAAW,GAAG,GAAG,EAAE,CAAC,KAAK,CAAA;AAE/B;;;GAGG;AACH,MAAa,YAAY;IAOvB,YACW,MAAuB,EACvB,KAAwB,EACxB,OAAmC;QAF5C;;;;mBAAS,MAAM;WAAiB;QAChC;;;;mBAAS,KAAK;WAAmB;QACjC;;;;mBAAS,OAAO;WAA4B;QALtC;;;;mBAAU,IAAI,GAAG,EAAqB;WAAA;IAM3C,CAAC;IAUJ,KAAK,CAAC,GAAG,CAAC,GAAM,EAAE,UAAU,EAAyB;QACnD,OAAO,CAAC,MAAM,EAAE,cAAc,EAAE,CAAA;QAEhC,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,EAAE,OAAO,CAAA;QAErC,MAAM,WAAW,GAAqC,OAAO,CAAC,OAAO;YACnE,CAAC,CAAC,WAAW,CAAC,2CAA2C;YACzD,CAAC,CAAC,OAAO,CAAC,UAAU,IAAI,OAAO,IAAI,IAAI;gBACrC,CAAC,CAAC,UAAU,CAAC,4CAA4C;gBACzD,CAAC,CAAC,KAAK,EAAE,KAAQ,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,OAAO,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC,CAAA;QAEtD,qEAAqE;QACrE,4EAA4E;QAC5E,uEAAuE;QACvE,iEAAiE;QACjE,4EAA4E;QAC5E,cAAc;QACd,IAAI,qBAAiD,CAAA;QACrD,OAAO,CAAC,qBAAqB,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,EAAE,CAAC;YACvD,IAAI,CAAC;gBACH,MAAM,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,MAAM,qBAAqB,CAAA;gBAEtD,IAAI,OAAO;oBAAE,OAAO,KAAK,CAAA;gBACzB,IAAI,MAAM,WAAW,CAAC,KAAK,CAAC;oBAAE,OAAO,KAAK,CAAA;YAC5C,CAAC;YAAC,MAAM,CAAC;gBACP,mEAAmE;gBACnE,4BAA4B;YAC9B,CAAC;YAED,OAAO,CAAC,MAAM,EAAE,cAAc,EAAE,CAAA;QAClC,CAAC;QAED,MAAM,oBAAoB,GAAmB,OAAO,CAAC,OAAO,EAAE;aAC3D,IAAI,CAAC,KAAK,IAAI,EAAE;YACf,MAAM,WAAW,GAAG,MAAM,IAAI,CAAC,SAAS,CAAC,GAAG,EAAE,OAAO,CAAC,CAAA;YACtD,IAAI,WAAW,KAAK,SAAS,IAAI,CAAC,MAAM,WAAW,CAAC,WAAW,CAAC,CAAC,EAAE,CAAC;gBAClE,iEAAiE;gBACjE,gEAAgE;gBAChE,sEAAsE;gBACtE,8DAA8D;gBAC9D,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,WAAW,EAAE,CAAA;YAC/C,CAAC;YAED,OAAO,OAAO,CAAC,OAAO,EAAE;iBACrB,IAAI,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,OAAO,EAAE,WAAW,CAAC,CAAC;iBAC7D,KAAK,CAAC,KAAK,EAAE,GAAG,EAAE,EAAE;gBACnB,IAAI,WAAW,KAAK,SAAS,EAAE,CAAC;oBAC9B,IAAI,CAAC;wBACH,MAAM,aAAa,GAAG,IAAI,CAAC,OAAO,EAAE,aAAa,CAAA;wBACjD,IAAI,MAAM,aAAa,EAAE,CAAC,GAAG,EAAE,GAAG,EAAE,WAAW,CAAC,EAAE,CAAC;4BACjD,MAAM,IAAI,CAAC,SAAS,CAAC,GAAG,EAAE,GAAG,CAAC,CAAA;wBAChC,CAAC;oBACH,CAAC;oBAAC,OAAO,KAAK,EAAE,CAAC;wBACf,MAAM,IAAI,cAAc,CACtB,CAAC,GAAG,EAAE,KAAK,CAAC,EACZ,mCAAmC,CACpC,CAAA;oBACH,CAAC;gBACH,CAAC;gBACD,MAAM,GAAG,CAAA;YACX,CAAC,CAAC;iBACD,IAAI,CAAC,KAAK,EAAE,KAAK,EAAE,EAAE;gBACpB,6DAA6D;gBAC7D,MAAM,IAAI,CAAC,SAAS,CAAC,GAAG,EAAE,KAAK,CAAC,CAAA;gBAChC,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,KAAK,EAAE,CAAA;YACjC,CAAC,CAAC,CAAA;QACN,CAAC,CAAC;aACD,OAAO,CAAC,GAAG,EAAE;YACZ,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAA;QAC1B,CAAC,CAAC,CAAA;QAEJ,IAAI,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC;YAC1B,kEAAkE;YAClE,mEAAmE;YACnE,sEAAsE;YACtE,6CAA6C;YAC7C,MAAM,IAAI,KAAK,CAAC,qCAAqC,CAAC,CAAA;QACxD,CAAC;QAED,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,EAAE,oBAAoB,CAAC,CAAA;QAE3C,MAAM,EAAE,KAAK,EAAE,GAAG,MAAM,oBAAoB,CAAA;QAC5C,OAAO,KAAK,CAAA;IACd,CAAC;IAED,KAAK,CAAC,SAAS,CAAC,GAAM,EAAE,OAA0B;QAChD,IAAI,CAAC;YACH,OAAO,MAAM,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,GAAG,EAAE,OAAO,CAAC,CAAA;QAC3C,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,OAAO,SAAS,CAAA;QAClB,CAAC;IACH,CAAC;IAED,KAAK,CAAC,SAAS,CAAC,GAAM,EAAE,KAAQ;QAC9B,IAAI,CAAC;YACH,MAAM,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,GAAG,EAAE,KAAK,CAAC,CAAA;QAClC,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,MAAM,YAAY,GAAG,IAAI,CAAC,OAAO,EAAE,YAAY,CAAA;YAC/C,MAAM,YAAY,EAAE,CAAC,GAAG,EAAE,GAAG,EAAE,KAAK,CAAC,CAAA;QACvC,CAAC;IACH,CAAC;IAED,KAAK,CAAC,SAAS,CAAC,GAAM,EAAE,MAAgB;QACtC,MAAM,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,GAAG,CAAC,CAAA;IAC3B,CAAC;CACF;AA9HD,oCA8HC"}
+{"version":3,"file":"cached-getter.js","sourceRoot":"","sources":["../src/cached-getter.ts"],"names":[],"mappings":"AAmDA,MAAM,UAAU,GAAG,GAAG,EAAE,CAAC,IAAI,CAAA;AAC7B,MAAM,WAAW,GAAG,GAAG,EAAE,CAAC,KAAK,CAAA;AAE/B;;;GAGG;AACH,MAAM,OAAO,YAAY;IAOvB,YACW,MAAuB,EACvB,KAAwB,EACxB,UAAqC,EAAE;QAFvC,WAAM,GAAN,MAAM,CAAiB;QACvB,UAAK,GAAL,KAAK,CAAmB;QACxB,YAAO,GAAP,OAAO,CAAgC;QALjC,YAAO,GAAG,IAAI,GAAG,EAAqB,CAAA;IAMpD,CAAC;IAUJ,KAAK,CAAC,GAAG,CACP,GAAM,EACN,EACE,MAAM,EACN,OAAO,EACP,UAAU,GAAG,KAAK,EAClB,OAAO,GAAG,KAAK,MACb,EAAyB;QAE7B,MAAM,EAAE,cAAc,EAAE,CAAA;QAExB,MAAM,EAAE,OAAO,EAAE,aAAa,EAAE,GAAG,IAAI,CAAC,OAAO,CAAA;QAE/C,MAAM,WAAW,GAAqC,OAAO;YAC3D,CAAC,CAAC,WAAW,CAAC,2CAA2C;YACzD,CAAC,CAAC,UAAU,IAAI,OAAO,IAAI,IAAI;gBAC7B,CAAC,CAAC,UAAU,CAAC,4CAA4C;gBACzD,CAAC,CAAC,KAAK,EAAE,KAAQ,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,OAAO,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC,CAAA;QAEtD,qEAAqE;QACrE,2EAA2E;QAC3E,yEAAyE;QACzE,yEAAyE;QACzE,4EAA4E;QAC5E,wCAAwC;QACxC,IAAI,qBAAiD,CAAA;QACrD,OAAO,CAAC,qBAAqB,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,EAAE,CAAC;YACvD,IAAI,CAAC;gBACH,wEAAwE;gBACxE,MAAM,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,MAAM,qBAAqB,CAAA;gBAEtD,qDAAqD;gBACrD,IAAI,OAAO;oBAAE,OAAO,KAAK,CAAA;gBACzB,oEAAoE;gBACpE,+DAA+D;gBAC/D,IAAI,MAAM,WAAW,CAAC,KAAK,CAAC;oBAAE,OAAO,KAAK,CAAA;YAC5C,CAAC;YAAC,MAAM,CAAC;gBACP,mEAAmE;gBACnE,4BAA4B;YAC9B,CAAC;YAED,2CAA2C;YAC3C,MAAM,EAAE,cAAc,EAAE,CAAA;QAC1B,CAAC;QAED,MAAM,oBAAoB,GAAmB,OAAO,CAAC,OAAO,EAAE;aAC3D,IAAI,CAAC,KAAK,IAAI,EAAE;YACf,MAAM,WAAW,GAAG,MAAM,IAAI,CAAC,SAAS,CAAC,GAAG,EAAE,EAAE,MAAM,EAAE,CAAC,CAAA;YAEzD,IAAI,WAAW,KAAK,SAAS,IAAI,CAAC,MAAM,WAAW,CAAC,WAAW,CAAC,CAAC,EAAE,CAAC;gBAClE,iEAAiE;gBACjE,gEAAgE;gBAChE,sEAAsE;gBACtE,8DAA8D;gBAC9D,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,WAAW,EAAE,CAAA;YAC/C,CAAC;YAED,OAAO,OAAO,CAAC,OAAO,EAAE;iBACrB,IAAI,CAAC,KAAK,IAAI,EAAE;gBACf,MAAM,OAAO,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,EAAsB,CAAA;gBAChE,OAAO,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,IAAI,EAAE,GAAG,EAAE,OAAO,EAAE,WAAW,CAAC,CAAA;YAC1D,CAAC,CAAC;iBACD,KAAK,CAAC,KAAK,EAAE,GAAG,EAAE,EAAE;gBACnB,IAAI,WAAW,KAAK,SAAS,EAAE,CAAC;oBAC9B,IAAI,CAAC;wBACH,IAAI,MAAM,aAAa,EAAE,CAAC,GAAG,EAAE,GAAG,EAAE,WAAW,CAAC,EAAE,CAAC;4BACjD,MAAM,IAAI,CAAC,SAAS,CAAC,GAAG,EAAE,GAAG,CAAC,CAAA;wBAChC,CAAC;oBACH,CAAC;oBAAC,OAAO,KAAK,EAAE,CAAC;wBACf,MAAM,IAAI,cAAc,CACtB,CAAC,GAAG,EAAE,KAAK,CAAC,EACZ,mCAAmC,CACpC,CAAA;oBACH,CAAC;gBACH,CAAC;gBACD,MAAM,GAAG,CAAA;YACX,CAAC,CAAC;iBACD,IAAI,CAAC,KAAK,EAAE,KAAK,EAAE,EAAE;gBACpB,6DAA6D;gBAC7D,MAAM,IAAI,CAAC,SAAS,CAAC,GAAG,EAAE,KAAK,CAAC,CAAA;gBAChC,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,KAAK,EAAE,CAAA;YACjC,CAAC,CAAC,CAAA;QACN,CAAC,CAAC;aACD,OAAO,CAAC,GAAG,EAAE;YACZ,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,GAAG,CAAC,CAAA;QAC1B,CAAC,CAAC,CAAA;QAEJ,IAAI,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC;YAC1B,kEAAkE;YAClE,mEAAmE;YACnE,sEAAsE;YACtE,6CAA6C;YAC7C,MAAM,IAAI,KAAK,CAAC,qCAAqC,CAAC,CAAA;QACxD,CAAC;QAED,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,EAAE,oBAAoB,CAAC,CAAA;QAE3C,MAAM,EAAE,KAAK,EAAE,GAAG,MAAM,oBAAoB,CAAA;QAC5C,OAAO,KAAK,CAAA;IACd,CAAC;IAED,KAAK,CAAC,SAAS,CAAC,GAAM,EAAE,OAAoB;QAC1C,IAAI,CAAC;YACH,OAAO,MAAM,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,GAAG,EAAE,OAAO,CAAC,CAAA;QAC3C,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,OAAO,SAAS,CAAA;QAClB,CAAC;IACH,CAAC;IAED,KAAK,CAAC,SAAS,CAAC,GAAM,EAAE,KAAQ;QAC9B,IAAI,CAAC;YACH,MAAM,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,GAAG,EAAE,KAAK,CAAC,CAAA;QAClC,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,MAAM,YAAY,GAAG,IAAI,CAAC,OAAO,EAAE,YAAY,CAAA;YAC/C,MAAM,YAAY,EAAE,CAAC,GAAG,EAAE,GAAG,EAAE,KAAK,CAAC,CAAA;QACvC,CAAC;IACH,CAAC;IAED,KAAK,CAAC,SAAS,CAAC,GAAM,EAAE,MAAgB;QACtC,MAAM,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,GAAG,CAAC,CAAA;IAC3B,CAAC;CACF","sourcesContent":["import { GetOptions, Key, SimpleStore, Value } from './simple-store.js'\nimport { Awaitable, ContextOptions } from './util.js'\n\nexport type { GetOptions }\nexport type GetCachedOptions<C = void> = ContextOptions<C> & {\n  signal?: AbortSignal\n\n  /**\n   * Do not use the cache to get the value. Always get a new value from the\n   * getter function.\n   *\n   * @default false\n   */\n  noCache?: boolean\n\n  /**\n   * When getting a value from the cache, allow the value to be returned even if\n   * it is stale.\n   *\n   * Has no effect if the `isStale` option was not provided to the CachedGetter.\n   *\n   * @default true // If the CachedGetter has an isStale option\n   * @default false // If no isStale option was provided to the CachedGetter\n   */\n  allowStale?: boolean\n}\n\nexport type GetterOptions<C = void> = {\n  context: C extends void ? undefined : C\n  noCache: boolean\n  signal?: AbortSignal\n}\n\nexport type Getter<K extends Key, V extends Value, C = void> = (\n  key: K,\n  options: GetterOptions<C>,\n  storedValue: undefined | V,\n) => Awaitable<V>\n\nexport type CachedGetterOptions<K extends Key, V extends Value> = {\n  isStale?: (key: K, value: V) => boolean | PromiseLike<boolean>\n  onStoreError?: (err: unknown, key: K, value: V) => void | PromiseLike<void>\n  deleteOnError?: (\n    err: unknown,\n    key: K,\n    value: V,\n  ) => boolean | PromiseLike<boolean>\n}\n\ntype PendingItem<V> = Promise<{ value: V; isFresh: boolean }>\n\nconst returnTrue = () => true\nconst returnFalse = () => false\n\n/**\n * Wrapper utility that uses a store to speed up the retrieval of values from an\n * (expensive) getter function.\n */\nexport class CachedGetter<\n  K extends Key = string,\n  V extends Value = Value,\n  C = void,\n> {\n  private readonly pending = new Map<K, PendingItem<V>>()\n\n  constructor(\n    readonly getter: Getter<K, V, C>,\n    readonly store: SimpleStore<K, V>,\n    readonly options: CachedGetterOptions<K, V> = {},\n  ) {}\n\n  async get(\n    key: C extends void ? K : never,\n    options?: GetCachedOptions<C>,\n  ): Promise<V>\n  async get(\n    key: C extends void ? never : K,\n    options: GetCachedOptions<C>,\n  ): Promise<V>\n  async get(\n    key: K,\n    {\n      signal,\n      context,\n      allowStale = false,\n      noCache = false,\n    } = {} as GetCachedOptions<C>,\n  ): Promise<V> {\n    signal?.throwIfAborted()\n\n    const { isStale, deleteOnError } = this.options\n\n    const allowStored: (value: V) => Awaitable<boolean> = noCache\n      ? returnFalse // Never allow stored values to be returned\n      : allowStale || isStale == null\n        ? returnTrue // Always allow stored values to be returned\n        : async (value: V) => !(await isStale(key, value))\n\n    // As long as concurrent requests are made for the same key, only one\n    // request will be made to the getStored & getter functions at a time. This\n    // works because there is no async operation between the while() loop and\n    // the pending.set() call below. Because of the single threaded nature of\n    // JavaScript, the pending item will be set before the next iteration of the\n    // while loop of any concurrent request.\n    let previousExecutionFlow: undefined | PendingItem<V>\n    while ((previousExecutionFlow = this.pending.get(key))) {\n      try {\n        // If a concurrent request is already in progress, wait for it to finish\n        const { isFresh, value } = await previousExecutionFlow\n\n        // Use the concurrent request's result if it is fresh\n        if (isFresh) return value\n        // Use the concurrent request's result if not fresh (loaded from the\n        // store), and matches the conditions for using a stored value.\n        if (await allowStored(value)) return value\n      } catch {\n        // Ignore errors from previous execution flows (they will have been\n        // propagated by that flow).\n      }\n\n      // Break the loop if the signal was aborted\n      signal?.throwIfAborted()\n    }\n\n    const currentExecutionFlow: PendingItem<V> = Promise.resolve()\n      .then(async () => {\n        const storedValue = await this.getStored(key, { signal })\n\n        if (storedValue !== undefined && (await allowStored(storedValue))) {\n          // Use the stored value as return value for the current execution\n          // flow. Notify other concurrent execution flows (that should be\n          // \"stuck\" in the loop before until this promise resolves) that we got\n          // a value, but that it came from the store (isFresh = false).\n          return { isFresh: false, value: storedValue }\n        }\n\n        return Promise.resolve()\n          .then(async () => {\n            const options = { signal, noCache, context } as GetterOptions<C>\n            return this.getter.call(null, key, options, storedValue)\n          })\n          .catch(async (err) => {\n            if (storedValue !== undefined) {\n              try {\n                if (await deleteOnError?.(err, key, storedValue)) {\n                  await this.delStored(key, err)\n                }\n              } catch (error) {\n                throw new AggregateError(\n                  [err, error],\n                  'Error while deleting stored value',\n                )\n              }\n            }\n            throw err\n          })\n          .then(async (value) => {\n            // The value should be stored even is the signal was aborted.\n            await this.setStored(key, value)\n            return { isFresh: true, value }\n          })\n      })\n      .finally(() => {\n        this.pending.delete(key)\n      })\n\n    if (this.pending.has(key)) {\n      // This should never happen. Indeed, there must not be any 'await'\n      // statement between this and the loop iteration check meaning that\n      // this.pending.get returned undefined. It is there to catch bugs that\n      // would occur in future changes to the code.\n      throw new Error('Concurrent request for the same key')\n    }\n\n    this.pending.set(key, currentExecutionFlow)\n\n    const { value } = await currentExecutionFlow\n    return value\n  }\n\n  async getStored(key: K, options?: GetOptions): Promise<V | undefined> {\n    try {\n      return await this.store.get(key, options)\n    } catch (err) {\n      return undefined\n    }\n  }\n\n  async setStored(key: K, value: V): Promise<void> {\n    try {\n      await this.store.set(key, value)\n    } catch (err) {\n      const onStoreError = this.options?.onStoreError\n      await onStoreError?.(err, key, value)\n    }\n  }\n\n  async delStored(key: K, _cause?: unknown): Promise<void> {\n    await this.store.del(key)\n  }\n}\n"]}

package/dist/index.d.ts

@@ -1,3 +1,4 @@
 export * from './cached-getter.js';
 export * from './simple-store.js';
+export * from './util.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,oBAAoB,CAAA;AAClC,cAAc,mBAAmB,CAAA"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,oBAAoB,CAAA;AAClC,cAAc,mBAAmB,CAAA;AACjC,cAAc,WAAW,CAAA"}

package/dist/index.js

@@ -1,19 +1,4 @@
-"use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __exportStar = (this && this.__exportStar) || function(m, exports) {
-    for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-__exportStar(require("./cached-getter.js"), exports);
-__exportStar(require("./simple-store.js"), exports);
+export * from './cached-getter.js';
+export * from './simple-store.js';
+export * from './util.js';
 //# sourceMappingURL=index.js.map

package/dist/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;AAAA,qDAAkC;AAClC,oDAAiC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA,cAAc,oBAAoB,CAAA;AAClC,cAAc,mBAAmB,CAAA;AACjC,cAAc,WAAW,CAAA","sourcesContent":["export * from './cached-getter.js'\nexport * from './simple-store.js'\nexport * from './util.js'\n"]}

package/dist/simple-store.d.ts

@@ -1,10 +1,10 @@
-export type Awaitable<V> = V | PromiseLike<V>;
-export type Key = string | number;
+import { Awaitable } from './util.js';
+export type Key = NonNullable<unknown>;
 export type Value = NonNullable<unknown> | null;
 export type GetOptions = {
     signal?: AbortSignal;
 };
-export interface SimpleStore<K extends Key = string, V extends Value = Value> {
+export interface SimpleStore<K extends Key, V extends Value> {
     /**
      * @return undefined if the key is not in the store (which is why Value cannot contain "undefined").
      */

package/dist/simple-store.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"simple-store.d.ts","sourceRoot":"","sources":["../src/simple-store.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,SAAS,CAAC,CAAC,IAAI,CAAC,GAAG,WAAW,CAAC,CAAC,CAAC,CAAA;AAE7C,MAAM,MAAM,GAAG,GAAG,MAAM,GAAG,MAAM,CAAA;AACjC,MAAM,MAAM,KAAK,GAAG,WAAW,CAAC,OAAO,CAAC,GAAG,IAAI,CAAA;AAE/C,MAAM,MAAM,UAAU,GAAG;IAAE,MAAM,CAAC,EAAE,WAAW,CAAA;CAAE,CAAA;AAEjD,MAAM,WAAW,WAAW,CAAC,CAAC,SAAS,GAAG,GAAG,MAAM,EAAE,CAAC,SAAS,KAAK,GAAG,KAAK;IAC1E;;OAEG;IACH,GAAG,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,OAAO,CAAC,EAAE,UAAU,KAAK,SAAS,CAAC,SAAS,GAAG,CAAC,CAAC,CAAA;IAC/D,GAAG,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,KAAK,SAAS,CAAC,IAAI,CAAC,CAAA;IAC1C,GAAG,EAAE,CAAC,GAAG,EAAE,CAAC,KAAK,SAAS,CAAC,IAAI,CAAC,CAAA;IAChC,KAAK,CAAC,EAAE,MAAM,SAAS,CAAC,IAAI,CAAC,CAAA;CAC9B"}
+{"version":3,"file":"simple-store.d.ts","sourceRoot":"","sources":["../src/simple-store.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,WAAW,CAAA;AAErC,MAAM,MAAM,GAAG,GAAG,WAAW,CAAC,OAAO,CAAC,CAAA;AACtC,MAAM,MAAM,KAAK,GAAG,WAAW,CAAC,OAAO,CAAC,GAAG,IAAI,CAAA;AAE/C,MAAM,MAAM,UAAU,GAAG;IAAE,MAAM,CAAC,EAAE,WAAW,CAAA;CAAE,CAAA;AAEjD,MAAM,WAAW,WAAW,CAAC,CAAC,SAAS,GAAG,EAAE,CAAC,SAAS,KAAK;IACzD;;OAEG;IACH,GAAG,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,OAAO,CAAC,EAAE,UAAU,KAAK,SAAS,CAAC,SAAS,GAAG,CAAC,CAAC,CAAA;IAC/D,GAAG,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,KAAK,SAAS,CAAC,IAAI,CAAC,CAAA;IAC1C,GAAG,EAAE,CAAC,GAAG,EAAE,CAAC,KAAK,SAAS,CAAC,IAAI,CAAC,CAAA;IAChC,KAAK,CAAC,EAAE,MAAM,SAAS,CAAC,IAAI,CAAC,CAAA;CAC9B"}

package/dist/simple-store.js

@@ -1,3 +1,2 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
+export {};
 //# sourceMappingURL=simple-store.js.map

package/dist/simple-store.js.map

@@ -1 +1 @@
-{"version":3,"file":"simple-store.js","sourceRoot":"","sources":["../src/simple-store.ts"],"names":[],"mappings":""}
+{"version":3,"file":"simple-store.js","sourceRoot":"","sources":["../src/simple-store.ts"],"names":[],"mappings":"","sourcesContent":["import { Awaitable } from './util.js'\n\nexport type Key = NonNullable<unknown>\nexport type Value = NonNullable<unknown> | null\n\nexport type GetOptions = { signal?: AbortSignal }\n\nexport interface SimpleStore<K extends Key, V extends Value> {\n  /**\n   * @return undefined if the key is not in the store (which is why Value cannot contain \"undefined\").\n   */\n  get: (key: K, options?: GetOptions) => Awaitable<undefined | V>\n  set: (key: K, value: V) => Awaitable<void>\n  del: (key: K) => Awaitable<void>\n  clear?: () => Awaitable<void>\n}\n"]}

package/dist/util.d.ts

@@ -0,0 +1,7 @@
+export type Awaitable<V> = V | PromiseLike<V>;
+export type ContextOptions<C> = C extends void | undefined ? {
+    context?: undefined;
+} : {
+    context: C;
+};
+//# sourceMappingURL=util.d.ts.map

package/dist/util.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"util.d.ts","sourceRoot":"","sources":["../src/util.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,SAAS,CAAC,CAAC,IAAI,CAAC,GAAG,WAAW,CAAC,CAAC,CAAC,CAAA;AAE7C,MAAM,MAAM,cAAc,CAAC,CAAC,IAAI,CAAC,SAAS,IAAI,GAAG,SAAS,GACtD;IAAE,OAAO,CAAC,EAAE,SAAS,CAAA;CAAE,GACvB;IAAE,OAAO,EAAE,CAAC,CAAA;CAAE,CAAA"}

package/dist/util.js

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=util.js.map

package/dist/util.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"util.js","sourceRoot":"","sources":["../src/util.ts"],"names":[],"mappings":"","sourcesContent":["export type Awaitable<V> = V | PromiseLike<V>\n\nexport type ContextOptions<C> = C extends void | undefined\n  ? { context?: undefined }\n  : { context: C }\n"]}

package/package.json

@@ -1,6 +1,9 @@
 {
   "name": "@atproto-labs/simple-store",
-  "version": "0.2.0",
+  "version": "0.4.0-next.0",
+  "engines": {
+    "node": ">=22"
+  },
   "license": "MIT",
   "description": "Simple store interfaces & utilities",
   "keywords": [
@@ -13,9 +16,7 @@
     "url": "https://github.com/bluesky-social/atproto",
     "directory": "packages/internal/simple-store"
   },
-  "type": "commonjs",
-  "main": "dist/index.js",
-  "types": "dist/index.d.ts",
+  "type": "module",
   "exports": {
     ".": {
       "types": "./dist/index.d.ts",
@@ -23,7 +24,7 @@
     }
   },
   "devDependencies": {
-    "typescript": "^5.6.3"
+    "typescript": "^6.0.3"
   },
   "scripts": {
     "build": "tsc --build tsconfig.build.json"

package/src/cached-getter.ts

@@ -1,13 +1,8 @@
-import {
-  Awaitable,
-  GetOptions as GetStoredOptions,
-  Key,
-  SimpleStore,
-  Value,
-} from './simple-store.js'
-
-export type { GetStoredOptions }
-export type GetCachedOptions<C = void> = {
+import { GetOptions, Key, SimpleStore, Value } from './simple-store.js'
+import { Awaitable, ContextOptions } from './util.js'
+
+export type { GetOptions }
+export type GetCachedOptions<C = void> = ContextOptions<C> & {
   signal?: AbortSignal
 
   /**
@@ -28,11 +23,17 @@ export type GetCachedOptions<C = void> = {
    * @default false // If no isStale option was provided to the CachedGetter
    */
   allowStale?: boolean
-} & (C extends void ? { context?: C } : { context: C })
+}
+
+export type GetterOptions<C = void> = {
+  context: C extends void ? undefined : C
+  noCache: boolean
+  signal?: AbortSignal
+}
 
 export type Getter<K extends Key, V extends Value, C = void> = (
   key: K,
-  options: GetCachedOptions<C>,
+  options: GetterOptions<C>,
   storedValue: undefined | V,
 ) => Awaitable<V>
 
@@ -60,12 +61,12 @@ export class CachedGetter<
   V extends Value = Value,
   C = void,
 > {
-  private pending = new Map<K, PendingItem<V>>()
+  private readonly pending = new Map<K, PendingItem<V>>()
 
   constructor(
     readonly getter: Getter<K, V, C>,
     readonly store: SimpleStore<K, V>,
-    readonly options?: CachedGetterOptions<K, V>,
+    readonly options: CachedGetterOptions<K, V> = {},
   ) {}
 
   async get(
@@ -76,41 +77,55 @@ export class CachedGetter<
     key: C extends void ? never : K,
     options: GetCachedOptions<C>,
   ): Promise<V>
-  async get(key: K, options = {} as GetCachedOptions<C>): Promise<V> {
-    options.signal?.throwIfAborted()
-
-    const isStale = this.options?.isStale
-
-    const allowStored: (value: V) => Awaitable<boolean> = options.noCache
+  async get(
+    key: K,
+    {
+      signal,
+      context,
+      allowStale = false,
+      noCache = false,
+    } = {} as GetCachedOptions<C>,
+  ): Promise<V> {
+    signal?.throwIfAborted()
+
+    const { isStale, deleteOnError } = this.options
+
+    const allowStored: (value: V) => Awaitable<boolean> = noCache
       ? returnFalse // Never allow stored values to be returned
-      : options.allowStale || isStale == null
+      : allowStale || isStale == null
         ? returnTrue // Always allow stored values to be returned
         : async (value: V) => !(await isStale(key, value))
 
     // As long as concurrent requests are made for the same key, only one
-    // request will be made to the cache & getter function at a time. This works
-    // because there is no async operation between the while() loop and the
-    // pending.set() call. Because of the "single threaded" nature of
+    // request will be made to the getStored & getter functions at a time. This
+    // works because there is no async operation between the while() loop and
+    // the pending.set() call below. Because of the single threaded nature of
     // JavaScript, the pending item will be set before the next iteration of the
-    // while loop.
+    // while loop of any concurrent request.
     let previousExecutionFlow: undefined | PendingItem<V>
     while ((previousExecutionFlow = this.pending.get(key))) {
       try {
+        // If a concurrent request is already in progress, wait for it to finish
         const { isFresh, value } = await previousExecutionFlow
 
+        // Use the concurrent request's result if it is fresh
         if (isFresh) return value
+        // Use the concurrent request's result if not fresh (loaded from the
+        // store), and matches the conditions for using a stored value.
         if (await allowStored(value)) return value
       } catch {
         // Ignore errors from previous execution flows (they will have been
         // propagated by that flow).
       }
 
-      options.signal?.throwIfAborted()
+      // Break the loop if the signal was aborted
+      signal?.throwIfAborted()
     }
 
     const currentExecutionFlow: PendingItem<V> = Promise.resolve()
       .then(async () => {
-        const storedValue = await this.getStored(key, options)
+        const storedValue = await this.getStored(key, { signal })
+
         if (storedValue !== undefined && (await allowStored(storedValue))) {
           // Use the stored value as return value for the current execution
           // flow. Notify other concurrent execution flows (that should be
@@ -120,11 +135,13 @@ export class CachedGetter<
         }
 
         return Promise.resolve()
-          .then(async () => (0, this.getter)(key, options, storedValue))
+          .then(async () => {
+            const options = { signal, noCache, context } as GetterOptions<C>
+            return this.getter.call(null, key, options, storedValue)
+          })
           .catch(async (err) => {
             if (storedValue !== undefined) {
               try {
-                const deleteOnError = this.options?.deleteOnError
                 if (await deleteOnError?.(err, key, storedValue)) {
                   await this.delStored(key, err)
                 }
@@ -161,7 +178,7 @@ export class CachedGetter<
     return value
   }
 
-  async getStored(key: K, options?: GetStoredOptions): Promise<V | undefined> {
+  async getStored(key: K, options?: GetOptions): Promise<V | undefined> {
     try {
       return await this.store.get(key, options)
     } catch (err) {

package/src/index.ts

@@ -1,2 +1,3 @@
 export * from './cached-getter.js'
 export * from './simple-store.js'
+export * from './util.js'

package/src/simple-store.ts

@@ -1,11 +1,11 @@
-export type Awaitable<V> = V | PromiseLike<V>
+import { Awaitable } from './util.js'
 
-export type Key = string | number
+export type Key = NonNullable<unknown>
 export type Value = NonNullable<unknown> | null
 
 export type GetOptions = { signal?: AbortSignal }
 
-export interface SimpleStore<K extends Key = string, V extends Value = Value> {
+export interface SimpleStore<K extends Key, V extends Value> {
   /**
    * @return undefined if the key is not in the store (which is why Value cannot contain "undefined").
    */

package/src/util.ts

@@ -0,0 +1,5 @@
+export type Awaitable<V> = V | PromiseLike<V>
+
+export type ContextOptions<C> = C extends void | undefined
+  ? { context?: undefined }
+  : { context: C }

package/tsconfig.build.tsbuildinfo

@@ -1 +1 @@
-{"root":["./src/cached-getter.ts","./src/index.ts","./src/simple-store.ts"],"version":"5.8.2"}
+{"root":["./src/cached-getter.ts","./src/index.ts","./src/simple-store.ts","./src/util.ts"],"version":"6.0.3"}