Remove ArrayCursor#some/every

Author: pluma

CHANGELOG.md

@@ -62,6 +62,15 @@ This is a major release and breaks backwards compatibility.
   The underlying collection can still be accessed from the `collection`
   property.
 
+- Removed `cursor.some` and `cursor.every`
+
+  These methods encouraged overfetching and should be replaced with more
+  efficient AQL queries.
+
+  The behavior can still be implemented by using the `next` method directly
+  or iterating over the cursor using the `forEach` method or the `for await`
+  syntax.
+
 ### Deprecated
 
 - Deprecated `db.useDatabase`

src/cursor.ts

@@ -232,41 +232,6 @@ export class ArrayCursor<T = any> {
     return true;
   }
 
-  async every(
-    fn: (value: T, index: number, self: ArrayCursor<T>) => boolean
-  ): Promise<boolean> {
-    let index = 0;
-    while (this._result.length || this.hasMore) {
-      let result;
-      while (this._result.length) {
-        result = fn(this._result.shift()!, index, this);
-        index++;
-        if (!result) return false;
-      }
-      if (this.hasMore) await this._more();
-    }
-    return true;
-  }
-
-  /**
-   * TODO
-   */
-  async some(
-    fn: (value: T, index: number, self: ArrayCursor<T>) => boolean
-  ): Promise<boolean> {
-    let index = 0;
-    while (this._result.length || this.hasMore) {
-      let result;
-      while (this._result.length) {
-        result = fn(this._result.shift()!, index, this);
-        index++;
-        if (result) return true;
-      }
-      if (this.hasMore) await this._more();
-    }
-    return false;
-  }
-
   /**
    * TODO
    */