shelving 1.57.1 → 1.58.0

package/db/{Pagination.d.ts → PaginationState.d.ts}

@@ -1,14 +1,17 @@
 import { Results, Entry, Entries, Data } from "../util/index.js";
-import { State } from "../stream/index.js";
+import { BooleanState, State } from "../stream/index.js";
 import { DatabaseQuery } from "./Database.js";
 /**
  * State that wraps a `Documents` reference to enable pagination.
  * - If you pass in initial values, it will use that as the first page.
  * - If you don't pass in initial values, it will autoload the first page.
  */
-export declare class Pagination<T extends Data> extends State<Results<T>> implements Iterable<Entry<T>> {
-    protected _pending: boolean;
+export declare class PaginationState<T extends Data> extends State<Results<T>> implements Iterable<Entry<T>> {
+    /** Whether this pagination is currently loading results. */
+    readonly busy: BooleanState;
+    /** Reference to the query this pagination is paginating. */
     readonly ref: DatabaseQuery<T>;
+    /** Limit set on this pagination's query. */
     readonly limit: number;
     constructor(ref: DatabaseQuery<T>);
     /**

package/db/{Pagination.js → PaginationState.js}

@@ -1,15 +1,16 @@
 import { getLastItem, assertNumber, yieldMerged, getMap, LOADING, assertMax } from "../util/index.js";
-import { State } from "../stream/index.js";
+import { BooleanState, State } from "../stream/index.js";
 import { ConditionError } from "../index.js";
 /**
  * State that wraps a `Documents` reference to enable pagination.
  * - If you pass in initial values, it will use that as the first page.
  * - If you don't pass in initial values, it will autoload the first page.
  */
-export class Pagination extends State {
+export class PaginationState extends State {
     constructor(ref) {
         super();
-        this._pending = false; // Prevents double-loading.
+        /** Whether this pagination is currently loading results. */
+        this.busy = new BooleanState();
         /**
          * Load more results after the end.
          * - Promise that needs to be handled.
@@ -17,25 +18,29 @@ export class Pagination extends State {
         this.more = async () => {
             if (this.closed)
                 throw new ConditionError("Pagination is closed");
-            if (!this._pending) {
-                this._pending = true;
-                if (!this.loading) {
+            if (!this.busy.value) {
+                this.busy.next(true);
+                if (!this.exists) {
+                    // First set of results.
+                    this._value === LOADING;
+                    const next = await this.ref.results;
+                    this.next(next);
+                    if (next.size < this.limit)
+                        this.complete();
+                    this.busy.next(false);
+                }
+                else {
+                    // Additional set of results.
                     const lastItem = getLastItem(this.value);
                     if (lastItem) {
                         const next = await this.ref.after(lastItem[0], lastItem[1]).results;
                         this.merge(next);
                         if (next.size < this.limit)
                             this.complete();
-                        this._pending = false;
+                        this.busy.next(false);
                         return;
                     }
                 }
-                this._value === LOADING;
-                const next = await this.ref.results;
-                this.next(next);
-                if (next.size < this.limit)
-                    this.complete();
-                this._pending = false;
             }
         };
         this.ref = ref;
@@ -48,7 +53,7 @@ export class Pagination extends State {
      * @return The change in the number of results.
      */
     merge(more) {
-        this.next(getMap(this.ref.sorts.transform(yieldMerged(more, this.value))));
+        this.next(this.exists ? getMap(this.ref.sorts.transform(yieldMerged(more, this.value))) : getMap(more));
     }
     /** Iterate over the entries of the values currently in the pagination. */
     [Symbol.iterator]() {

package/db/index.d.ts

@@ -1,4 +1,4 @@
 export * from "./Database.js";
 export * from "./Operation.js";
-export * from "./Pagination.js";
+export * from "./PaginationState.js";
 export * from "./errors.js";

package/db/index.js

@@ -1,4 +1,4 @@
 export * from "./Database.js";
 export * from "./Operation.js";
-export * from "./Pagination.js";
+export * from "./PaginationState.js";
 export * from "./errors.js";

package/package.json

@@ -11,7 +11,7 @@
 		"state-management",
 		"query-builder"
 	],
-	"version": "1.57.1",
+	"version": "1.58.0",
 	"repository": "https://github.com/dhoulb/shelving",
 	"author": "Dave Houlbrooke <dave@shax.com>",
 	"license": "0BSD",
@@ -62,21 +62,21 @@
 		"@google-cloud/firestore": "^5.0.2",
 		"@types/jest": "^27.4.1",
 		"@types/react": "^17.0.44",
-		"@types/react-dom": "^17.0.15",
-		"@typescript-eslint/eslint-plugin": "^5.20.0",
-		"@typescript-eslint/parser": "^5.20.0",
+		"@types/react-dom": "^17.0.16",
+		"@typescript-eslint/eslint-plugin": "^5.21.0",
+		"@typescript-eslint/parser": "^5.21.0",
 		"eslint": "^8.14.0",
 		"eslint-config-prettier": "^8.5.0",
 		"eslint-plugin-import": "^2.26.0",
 		"eslint-plugin-prettier": "^4.0.0",
-		"firebase": "^9.6.11",
+		"firebase": "^9.7.0",
 		"jest": "^27.5.1",
 		"jest-ts-webcompat-resolver": "^1.0.0",
 		"prettier": "^2.6.2",
 		"react": "^17.0.2",
 		"react-dom": "^17.0.2",
 		"ts-jest": "^27.1.4",
-		"typescript": "^4.6.3"
+		"typescript": "^4.6.4"
 	},
 	"peerDependencies": {
 		"@google-cloud/firestore": ">=4.0.0",

package/react/usePagination.d.ts

@@ -1,6 +1,6 @@
-import { DatabaseQuery, Results, Pagination, Data } from "../index.js";
+import { DatabaseQuery, Results, PaginationState, Data } from "../index.js";
 /**
  * Use a `Pagination` for a collection.
  * - Doesn't persist the state, so if the component or anything beneath it throws the currently paginated results will be lost.
  */
-export declare function usePagination<T extends Data>(ref: DatabaseQuery<T>, initial?: Results<T>): Pagination<T>;
+export declare function usePagination<T extends Data>(ref: DatabaseQuery<T>, initial?: Results<T>): PaginationState<T>;

package/react/usePagination.js

@@ -1,4 +1,4 @@
-import { Pagination } from "../index.js";
+import { PaginationState } from "../index.js";
 import { useSubscribe } from "./useSubscribe.js";
 import { useLazy } from "./useLazy.js";
 /**
@@ -11,7 +11,7 @@ export function usePagination(ref, initial) {
     return pagination;
 }
 function _createPagination(ref, initial) {
-    const pagination = new Pagination(ref);
+    const pagination = new PaginationState(ref);
     if (initial)
         pagination.next(initial);
     return pagination;

package/stream/State.d.ts

@@ -32,7 +32,7 @@ export declare class State<T> extends Stream<T> {
     get value(): T;
     protected _value: T | typeof LOADING;
     /** Is there a current value, or is it still loading. */
-    get loading(): boolean;
+    get exists(): boolean;
     /** Apply a transformer to this state. */
     apply(transformer: Transformer<T, T>): void;
     error(reason: Error | unknown): void;

package/stream/State.js

@@ -23,8 +23,8 @@ export class State extends Stream {
         return this._value;
     }
     /** Is there a current value, or is it still loading. */
-    get loading() {
-        return this._value === LOADING;
+    get exists() {
+        return this._value !== LOADING;
     }
     /** Apply a transformer to this state. */
     apply(transformer) {