@data-client/core 0.9.7 → 0.10.0

package/src/controller/Controller.ts

@@ -103,9 +103,8 @@ export default class Controller<
   >(
     endpoint: E,
     ...args: readonly [...Parameters<E>]
-  ): E['schema'] extends undefined | null
-    ? ReturnType<E>
-    : Promise<Denormalize<E['schema']>> => {
+  ): E['schema'] extends undefined | null ? ReturnType<E>
+  : Promise<Denormalize<E['schema']>> => {
     const action = createFetch(endpoint, {
       args,
     });
@@ -128,9 +127,8 @@ export default class Controller<
   >(
     endpoint: E,
     ...args: readonly [...Parameters<E>]
-  ): E['schema'] extends undefined | null
-    ? ReturnType<E> | ResolveType<E>
-    : Promise<Denormalize<E['schema']>> | Denormalize<E['schema']> => {
+  ): E['schema'] extends undefined | null ? ReturnType<E> | ResolveType<E>
+  : Promise<Denormalize<E['schema']>> | Denormalize<E['schema']> => {
     const { data, expiresAt, expiryStatus } = this.getResponse(
       endpoint,
       ...args,
@@ -149,13 +147,13 @@ export default class Controller<
     endpoint: E,
     ...args: readonly [...Parameters<E>] | readonly [null]
   ): Promise<void> =>
-    args[0] !== null
-      ? this.dispatch(
-          createInvalidate(endpoint, {
-            args: args as readonly [...Parameters<E>],
-          }),
-        )
-      : Promise.resolve();
+    args[0] !== null ?
+      this.dispatch(
+        createInvalidate(endpoint, {
+          args: args as readonly [...Parameters<E>],
+        }),
+      )
+    : Promise.resolve();
 
   /**
    * Forces refetching and suspense on useSuspense on all matching endpoint result keys.
@@ -261,13 +259,13 @@ export default class Controller<
     endpoint: E,
     ...args: readonly [...Parameters<E>] | readonly [null]
   ): Promise<void> =>
-    args[0] !== null
-      ? this.dispatch(
-          createSubscription(endpoint, {
-            args: args as readonly [...Parameters<E>],
-          }),
-        )
-      : Promise.resolve();
+    args[0] !== null ?
+      this.dispatch(
+        createSubscription(endpoint, {
+          args: args as readonly [...Parameters<E>],
+        }),
+      )
+    : Promise.resolve();
 
   /**
    * Marks completion of subscription to a given Endpoint.
@@ -283,13 +281,13 @@ export default class Controller<
     endpoint: E,
     ...args: readonly [...Parameters<E>] | readonly [null]
   ): Promise<void> =>
-    args[0] !== null
-      ? this.dispatch(
-          createUnsubscription(endpoint, {
-            args: args as readonly [...Parameters<E>],
-          }),
-        )
-      : Promise.resolve();
+    args[0] !== null ?
+      this.dispatch(
+        createUnsubscription(endpoint, {
+          args: args as readonly [...Parameters<E>],
+        }),
+      )
+    : Promise.resolve();
 
   /*************** More ***************/
 
@@ -386,10 +384,9 @@ export default class Controller<
     if (!endpoint.schema || !schemaHasEntity(endpoint.schema)) {
       return {
         data: results,
-        expiryStatus: meta?.invalidated
-          ? ExpiryStatus.Invalid
-          : cacheResults && !endpoint.invalidIfStale
-          ? ExpiryStatus.Valid
+        expiryStatus:
+          meta?.invalidated ? ExpiryStatus.Invalid
+          : cacheResults && !endpoint.invalidIfStale ? ExpiryStatus.Valid
           : ExpiryStatus.InvalidIfStale,
         expiresAt: expiresAt || 0,
       } as {
@@ -423,11 +420,11 @@ export default class Controller<
     // we don't track the difference between stale or fresh because that is tied to triggering
     // conditions
     const expiryStatus =
-      meta?.invalidated || (invalidDenormalize && !meta?.error)
-        ? ExpiryStatus.Invalid
-        : invalidDenormalize || endpoint.invalidIfStale || invalidResults
-        ? ExpiryStatus.InvalidIfStale
-        : ExpiryStatus.Valid;
+      meta?.invalidated || (invalidDenormalize && !meta?.error) ?
+        ExpiryStatus.Invalid
+      : invalidDenormalize || endpoint.invalidIfStale || invalidResults ?
+        ExpiryStatus.InvalidIfStale
+      : ExpiryStatus.Valid;
 
     return { data, expiryStatus, expiresAt };
   };

package/src/controller/createSet.ts

@@ -51,8 +51,9 @@ export default function createSet<
     error?: boolean;
   },
 ): SetAction<E> {
-  const expiryLength: number = error
-    ? endpoint.errorExpiryLength ?? 1000
+  const expiryLength: number =
+    error ?
+      endpoint.errorExpiryLength ?? 1000
     : endpoint.dataExpiryLength ?? 60000;
   /* istanbul ignore next */
   if (process.env.NODE_ENV === 'development' && expiryLength < 0) {

package/src/controller/ensurePojo.ts

@@ -1,11 +1,10 @@
 export const ensurePojo =
   // FormData doesn't exist in node
   /* istanbul ignore else we don't run coverage when we test node*/
-  typeof FormData !== 'undefined'
-    ? (body: any) =>
-        body instanceof FormData
-          ? Object.fromEntries((body as any).entries())
-          : body
-    : /* istanbul ignore next */
-      (body: any) => body;
+  typeof FormData !== 'undefined' ?
+    (body: any) =>
+      body instanceof FormData ?
+        Object.fromEntries((body as any).entries())
+      : body
+  : /* istanbul ignore next */ (body: any) => body;
 export default ensurePojo;

package/src/controller/types.ts

@@ -4,10 +4,8 @@ import type {
   ResolveType,
 } from '@data-client/normalizr';
 
-export type ResultEntry<E extends EndpointInterface> = E['schema'] extends
-  | undefined
-  | null
-  ? ResolveType<E>
+export type ResultEntry<E extends EndpointInterface> =
+  E['schema'] extends undefined | null ? ResolveType<E>
   : Normalize<E['schema']>;
 
 export type EndpointUpdateFunction<

package/src/manager/DevtoolsManager.ts

@@ -47,8 +47,9 @@ if (process.env.NODE_ENV !== 'production') {
     serialize: {
       options: undefined,
       /* istanbul ignore next */
-      replacer: HASINTL
-        ? (key: string | number | symbol, value: unknown) => {
+      replacer:
+        HASINTL ?
+          (key: string | number | symbol, value: unknown) => {
             if (
               typeof value === 'number' &&
               typeof key === 'string' &&
@@ -81,6 +82,7 @@ export default class DevToolsManager implements Manager {
   protected started = false;
   protected actions: [ActionTypes, State<unknown>][] = [];
   protected declare controller: Controller;
+  maxBufferLength = 100;
 
   constructor(
     config?: DevToolsConfig,
@@ -150,6 +152,10 @@ export default class DevToolsManager implements Manager {
     if (this.started) {
       this.devTools.send(action, state, undefined, 'RDC');
     } else {
+      // avoid this getting too big in case this is long running
+      // we cut in half so we aren't constantly reallocating
+      if (this.actions.length > this.maxBufferLength)
+        this.actions = this.actions.slice(this.maxBufferLength / 2);
       // queue actions
       this.actions.push([action, state]);
     }

package/src/manager/__tests__/subscriptionManager.ts

@@ -51,8 +51,9 @@ describe('SubscriptionManager', () => {
       payload: Record<string, any>,
       reject = false,
     ): SubscribeAction {
-      const fetch = reject
-        ? () => Promise.reject(new Error('Failed'))
+      const fetch =
+        reject ?
+          () => Promise.reject(new Error('Failed'))
         : () => Promise.resolve(payload);
       return {
         type: SUBSCRIBE_TYPE,

package/src/middlewareTypes.ts

@@ -25,15 +25,7 @@ export type Dispatch<R extends Reducer<any, any>> = (
 ) => Promise<void>;
 
 export type Reducer<S, A> = (prevState: S, action: A) => S;
-export type ReducerState<R extends Reducer<any, any>> = R extends Reducer<
-  infer S,
-  any
->
-  ? S
-  : never;
-export type ReducerAction<R extends Reducer<any, any>> = R extends Reducer<
-  any,
-  infer A
->
-  ? A
-  : never;
+export type ReducerState<R extends Reducer<any, any>> =
+  R extends Reducer<infer S, any> ? S : never;
+export type ReducerAction<R extends Reducer<any, any>> =
+  R extends Reducer<any, infer A> ? A : never;

package/src/state/RIC.ts

@@ -1,6 +1,5 @@
 const RIC: (cb: (...args: any[]) => void, options: any) => void = (
-  typeof requestIdleCallback === 'function'
-    ? requestIdleCallback
-    : (cb: any) => setTimeout(cb, 0)
-) as any;
+  typeof requestIdleCallback === 'function' ? requestIdleCallback : (
+    (cb: any) => setTimeout(cb, 0)
+  )) as any;
 export default RIC;

package/src/state/__tests__/reducer.ts

@@ -159,17 +159,15 @@ describe('reducer', () => {
           existing: any,
           incoming: any,
         ) {
-          return this.shouldReorder(
-            existingMeta,
-            incomingMeta,
-            existing,
-            incoming,
-          )
-            ? existingMeta
+          return (
+              this.shouldReorder(existingMeta, incomingMeta, existing, incoming)
+            ) ?
+              existingMeta
             : {
                 ...incomingMeta,
-                expiresAt: incoming.content
-                  ? incomingMeta.expiresAt
+                expiresAt:
+                  incoming.content ?
+                    incomingMeta.expiresAt
                   : existingMeta.expiresAt,
               };
         }

package/src/state/reducer/setReducer.ts

@@ -138,8 +138,8 @@ function filterOptimistic(
   return state.optimistic.filter(
     optimisticAction =>
       optimisticAction.meta.key !== resolvingAction.meta.key ||
-      (optimisticAction.type === OPTIMISTIC_TYPE
-        ? optimisticAction.meta.fetchedAt !== resolvingAction.meta.fetchedAt
-        : optimisticAction.meta.date > resolvingAction.meta.date),
+      (optimisticAction.type === OPTIMISTIC_TYPE ?
+        optimisticAction.meta.fetchedAt !== resolvingAction.meta.fetchedAt
+      : optimisticAction.meta.date > resolvingAction.meta.date),
   );
 }

package/ts3.4/manager/DevtoolsManager.d.ts

@@ -18,6 +18,7 @@ export default class DevToolsManager implements Manager {
         State<unknown>
     ][];
     protected controller: Controller;
+    maxBufferLength: number;
     constructor(config?: DevToolsConfig, skipLogging?: (action: ActionTypes) => boolean);
     handleAction(action: any, state: any): void;
     /** Called when initial state is ready */