Replace TypeScript Record types with index signatures.

Author: denisp22

Cache.mjs

@@ -6,7 +6,7 @@
  */
 export default class Cache extends EventTarget {
   /**
-   * @param {CacheStore} [store] Initial {@link Cache.store cache store} record.
+   * @param {CacheStore} [store] Initial {@link Cache.store cache store}.
    *   Defaults to `{}`. Useful for hydrating cache data from a server side
    *   render prior to the initial client side render.
    */
@@ -65,6 +65,6 @@ export default class Cache extends EventTarget {
  */
 
 /**
- * Cache store record.
- * @typedef {Record<CacheKey, CacheValue>} CacheStore
+ * Cache store.
+ * @typedef {{ [cacheKey: CacheKey]: CacheValue }} CacheStore
  */

Loading.mjs

@@ -16,7 +16,7 @@ export default class Loading extends EventTarget {
      * Store of loading {@link CacheKey cache keys} and associated
      * {@link LoadingCacheValue loading cache values}. Multiple for the same key
      * are set in the order loading started.
-     * @type {Record<CacheKey, Set<LoadingCacheValue>>}
+     * @type {{ [cacheKey: CacheKey]: Set<LoadingCacheValue> }}
      */
     this.store = {};
   }

LoadingCacheValue.test.mjs

@@ -123,7 +123,7 @@ export default (tests) => {
     loading.addEventListener(`${cacheKey}/end`, loadingListener);
 
     const { promise: loadingResult, resolve: loadingResultResolve } =
-      /** @type {Deferred<Readonly<Record<string, unknown>>>} */
+      /** @type {Deferred<Readonly<{ [key: string]: unknown }>>} */
       (new Deferred());
     const abortController = new AbortController();
     const loadingCacheValue = new LoadingCacheValue(
@@ -206,7 +206,7 @@ export default (tests) => {
         promise: firstLoadingResult,
         resolve: firstLoadingResultResolve,
       } =
-        /** @type {Deferred<Readonly<Record<string, unknown>>>} */
+        /** @type {Deferred<Readonly<{ [key: string]: unknown }>>} */
         (new Deferred());
       const firstAbortController = new AbortController();
       const firstLoadingCacheValue = new LoadingCacheValue(
@@ -246,7 +246,7 @@ export default (tests) => {
         promise: secondLoadingResult,
         resolve: secondLoadingResultResolve,
       } =
-        /** @type {Deferred<Readonly<Record<string, unknown>>>} */
+        /** @type {Deferred<Readonly<{ [key: string]: unknown }>>} */
         (new Deferred());
       const secondAbortController = new AbortController();
       const secondLoadingCacheValue = new LoadingCacheValue(
@@ -375,7 +375,7 @@ export default (tests) => {
         promise: firstLoadingResult,
         resolve: firstLoadingResultResolve,
       } =
-        /** @type {Deferred<Readonly<Record<string, unknown>>>} */
+        /** @type {Deferred<Readonly<{ [key: string]: unknown }>>} */
         (new Deferred());
       const firstAbortController = new AbortController();
       const firstLoadingCacheValue = new LoadingCacheValue(
@@ -415,7 +415,7 @@ export default (tests) => {
         promise: secondLoadingResult,
         resolve: secondLoadingResultResolve,
       } =
-        /** @type {Deferred<Readonly<Record<string, unknown>>>} */
+        /** @type {Deferred<Readonly<{ [key: string]: unknown }>>} */
         (new Deferred());
       const secondAbortController = new AbortController();
       const secondLoadingCacheValue = new LoadingCacheValue(

changelog.md

@@ -12,6 +12,7 @@
 - Updated dev dependencies.
 - Use the `node:` URL scheme for Node.js builtin module imports in tests.
 - Updated ESLint config.
+- Replaced TypeScript `Record` types with index signatures.
 - Added missing readme “Installation” section import map instructions for [`is-plain-obj`](https://npm.im/is-plain-obj).
 - Added [Browserslist](https://browsersl.ist) links to the readme.
 

fetchOptionsGraphQL.mjs

@@ -39,7 +39,7 @@ export default function fetchOptionsGraphQL(operation) {
 
     form.set("operations", operationJSON);
 
-    /** @type {Record<string, Array<string>>} */
+    /** @type {{ [formFieldName: string]: Array<string> }} */
     const map = {};
 
     let i = 0;
@@ -60,7 +60,7 @@ export default function fetchOptionsGraphQL(operation) {
 
     fetchOptions.body = form;
   } else {
-    /** @type {Record<string, string>} */ (fetchOptions.headers)[
+    /** @type {{ [headerName: string]: string }} */ (fetchOptions.headers)[
       "Content-Type"
     ] = "application/json";
     fetchOptions.body = operationJSON;

types.mjs

@@ -21,8 +21,8 @@ export {};
  * GraphQL server.
  * @typedef {object} GraphQLOperation
  * @prop {string} query GraphQL queries or mutations.
- * @prop {Record<string, unknown>} [variables] Variables used in the GraphQL
- *   queries or mutations.
+ * @prop {{ [variableName: string ]: unknown}} [variables] Variables used in the
+ *   GraphQL queries or mutations.
  */
 
 /**
@@ -35,14 +35,16 @@ export {};
  *   for hydration.
  * @prop {Array<ErrorTypes>} [errors] GraphQL errors from the server, along with
  *   any loading errors added on the client.
- * @prop {Record<string, unknown> | null} [data] GraphQL data.
+ * @prop {{ [key: string]: unknown } | null} [data] GraphQL data.
  */
 
 /**
  * {@link GraphQLResult.errors GraphQL result error}.
  * @see [GraphQL spec for response errors](https://spec.graphql.org/October2021/#sec-Errors).
- * @template {Record<string, unknown>} [Extensions=Record<string, unknown>]
- *   Extensions to a standard GraphQL error.
+ * @template {{
+ *   [key: string]: unknown
+ * }} [Extensions={ [key: string]: unknown }] Extensions to a standard GraphQL
+ *   error.
  * @typedef {object} GraphQLResultError
  * @prop {string} message Error message.
  * @prop {Array<{ line: number; column: number }>} [locations] GraphQL query
@@ -56,7 +58,8 @@ export {};
  * {@link GraphQLResult GraphQL result} loading error generated on the client,
  * not the GraphQL server.
  * @template {string} Code Error code.
- * @template {Record<string, unknown>} [Extensions={}] Error specific details.
+ * @template {{ [key: string]: unknown }} [Extensions={}] Error specific
+ *   details.
  * @typedef {object} GraphQLResultErrorLoading
  * @prop {string} message Error message.
  * @prop {GraphQLResultErrorLoadingMeta<Code> & Extensions} extensions Error

useLoadingEntry.test.mjs

@@ -109,7 +109,7 @@ export default (tests) => {
       strictEqual(results[0].returned, undefined);
 
       const { promise: loadingA1Result, resolve: loadingA1ResultResolve } =
-        /** @type {Deferred<Readonly<Record<string, unknown>>>} */
+        /** @type {Deferred<Readonly<{ [key: string]: unknown }>>} */
         (new Deferred());
 
       /** @type {LoadingCacheValue | undefined} */
@@ -158,7 +158,7 @@ export default (tests) => {
       strictEqual(results[3].returned, undefined);
 
       const { promise: loadingB1Result, resolve: loadingB1ResultResolve } =
-        /** @type {Deferred<Readonly<Record<string, unknown>>>} */
+        /** @type {Deferred<Readonly<{ [key: string]: unknown }>>} */
         (new Deferred());
 
       /** @type {LoadingCacheValue | undefined} */
@@ -196,7 +196,7 @@ export default (tests) => {
       const cache = new Cache();
       const cacheKeyA = "a";
       const { promise: loadingA1Result, resolve: loadingA1ResultResolve } =
-        /** @type {Deferred<Readonly<Record<string, unknown>>>} */
+        /** @type {Deferred<Readonly<{ [key: string]: unknown }>>} */
         (new Deferred());
       const loadingA1CacheValue = new LoadingCacheValue(
         loading,
@@ -225,7 +225,7 @@ export default (tests) => {
       deepStrictEqual(results[0].returned, new Set([loadingA1CacheValue]));
 
       const { promise: loadingA2Result, resolve: loadingA2ResultResolve } =
-        /** @type {Deferred<Readonly<Record<string, unknown>>>} */
+        /** @type {Deferred<Readonly<{ [key: string]: unknown }>>} */
         (new Deferred());
 
       /** @type {LoadingCacheValue | undefined} */
@@ -268,7 +268,7 @@ export default (tests) => {
 
       const cacheKeyB = "b";
       const { promise: loadingB1Result, resolve: loadingB1ResultResolve } =
-        /** @type {Deferred<Readonly<Record<string, unknown>>>} */
+        /** @type {Deferred<Readonly<{ [key: string]: unknown }>>} */
         (new Deferred());
 
       /** @type {LoadingCacheValue | undefined} */