@bjnstnkvc/db 2.0.1 → 2.0.2

package/README.md

@@ -835,6 +835,7 @@ await DB.table<User>('users').where('id', 1).increment('visits');
 await DB.table<User>('users').where('id', 1).decrement('credits', 5);
 
 await DB.table<User>('users').where('role', 'guest').delete();
+await DB.table<User>('users').oldest('created_at').limit(100).delete();
 await DB.table<User>('users').truncate();
 ```
 
@@ -851,9 +852,10 @@ enforce anything else. Any other column throws `SchemaException`.
 
 The key path may not be updated, so `update`, `upsert` and `increment` all refuse it.
 
-`update` and `delete` honour `limit` and `offset` in the order the plan scans, which is index order
-when an index drives the query and key order otherwise. Pair them with an indexed `orderBy` if you
-need a defined order.
+`update`, `delete`, `increment` and `decrement` honor `orderBy` together with `limit` and
+`offset`, so they touch the same records a read of the query would return, whether or not an index
+serves the order. Without an `orderBy`, they follow the order the plan scans, which is index order
+when an index drives the query and key order otherwise.
 
 > Modeled on Laravel's [Database: Query Builder](https://laravel.com/docs/12.x/queries). The method
 > names and their semantics match, and every terminal is asynchronous because IndexedDB is.

package/dist/main.cjs

@@ -2910,7 +2910,7 @@ var Builder = class _Builder {
     }
   }
   /**
-   * Apply a change to every record matching the query, in the order the plan scans them.
+   * Apply a change to every record matching the query, in the order the query asks for.
    */
   async #modify(apply) {
     const schema = await this.#connection.schema(this.#table);
@@ -2919,8 +2919,20 @@ var Builder = class _Builder {
     const started = performance.now();
     const matches = Predicate.compile(plan.residual);
     const ceiling = this.#limit === null ? null : this.#offset + this.#limit;
+    const collects = !plan.ordered && this.#orders.length > 0 && (this.#limit !== null || this.#offset > 0);
     let seen = 0;
     let affected = 0;
+    if (collects) {
+      const collected = plan.values === null ? await this.#cursored(store, plan, matches) : await this.#points(store, plan, matches);
+      for (const entry of this.#paged(this.#sorted(collected))) {
+        await Request.walk(store.openCursor(IDBKeyRange.only(entry.key)), (cursor) => {
+          apply(cursor);
+          affected++;
+        });
+      }
+      this.#emit(Planner.describe(plan), started, affected);
+      return affected;
+    }
     const visit = (cursor) => {
       if (!matches(cursor.value)) {
         return true;
@@ -2937,6 +2949,9 @@ var Builder = class _Builder {
       await Request.walk(source.openCursor(plan.range, plan.direction), visit);
     } else {
       for (const value of plan.values) {
+        if (ceiling !== null && seen >= ceiling) {
+          break;
+        }
         const source = plan.index === null ? store : store.index(plan.index);
         await Request.walk(source.openCursor(IDBKeyRange.only(value)), visit);
       }

package/dist/main.js

@@ -2846,7 +2846,7 @@ var Builder = class _Builder {
     }
   }
   /**
-   * Apply a change to every record matching the query, in the order the plan scans them.
+   * Apply a change to every record matching the query, in the order the query asks for.
    */
   async #modify(apply) {
     const schema = await this.#connection.schema(this.#table);
@@ -2855,8 +2855,20 @@ var Builder = class _Builder {
     const started = performance.now();
     const matches = Predicate.compile(plan.residual);
     const ceiling = this.#limit === null ? null : this.#offset + this.#limit;
+    const collects = !plan.ordered && this.#orders.length > 0 && (this.#limit !== null || this.#offset > 0);
     let seen = 0;
     let affected = 0;
+    if (collects) {
+      const collected = plan.values === null ? await this.#cursored(store, plan, matches) : await this.#points(store, plan, matches);
+      for (const entry of this.#paged(this.#sorted(collected))) {
+        await Request.walk(store.openCursor(IDBKeyRange.only(entry.key)), (cursor) => {
+          apply(cursor);
+          affected++;
+        });
+      }
+      this.#emit(Planner.describe(plan), started, affected);
+      return affected;
+    }
     const visit = (cursor) => {
       if (!matches(cursor.value)) {
         return true;
@@ -2873,6 +2885,9 @@ var Builder = class _Builder {
       await Request.walk(source.openCursor(plan.range, plan.direction), visit);
     } else {
       for (const value of plan.values) {
+        if (ceiling !== null && seen >= ceiling) {
+          break;
+        }
         const source = plan.index === null ? store : store.index(plan.index);
         await Request.walk(source.openCursor(IDBKeyRange.only(value)), visit);
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bjnstnkvc/db",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "description": "TypeScript database layer for IndexedDB with an API modeled on Laravel.",
   "type": "module",
   "main": "./dist/main.cjs",