deepbase-redis 3.10.0 → 3.11.0

package/README.md

@@ -118,6 +118,22 @@ const allUsers = await db.get('users');
 // Scans all keys matching prefix
 ```
 
+### Querying
+
+`db.query(...path)` is evaluated in memory: the driver reads the whole
+document already stored in Redis and walks it locally. v1 does not push
+filters to the server and does not use indexes.
+
+Every result has the `{ id, value }` shape, where `id` is the property name
+inside the queried object:
+
+```javascript
+await db.set('users', 'alice', { name: 'Alice', age: 30 });
+
+const records = await db.query('users').where('age', '>=', 18).toArray();
+// [{ id: 'alice', value: { name: 'Alice', age: 30 } }]
+```
+
 ## Connection String Formats
 
 ```javascript

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "deepbase-redis",
-  "version": "3.10.0",
+  "version": "3.11.0",
   "description": "⚡ DeepBase Redis (vanilla) - driver",
   "type": "module",
   "main": "src/index.cjs",
@@ -17,10 +17,7 @@
     "redis": "^4.7.0"
   },
   "peerDependencies": {
-    "deepbase": "^3.10.0"
-  },
-  "scripts": {
-    "test": "mocha test/test.js"
+    "deepbase": "^3.11.0"
   },
   "devDependencies": {
     "mocha": "^10.8.2"
@@ -44,5 +41,8 @@
   "bugs": {
     "url": "https://github.com/clasen/DeepBase/issues"
   },
-  "homepage": "https://github.com/clasen/DeepBase/tree/main/packages/driver-redis"
-}
+  "homepage": "https://github.com/clasen/DeepBase/tree/main/packages/driver-redis",
+  "scripts": {
+    "test": "mocha test/test.js"
+  }
+}

package/src/RedisDriver.js

@@ -1,4 +1,4 @@
-import { DeepBaseDriver } from 'deepbase';
+import { DeepBaseDriver, evaluateQuery, removeKey } from 'deepbase';
 import { createClient } from 'redis';
 
 export class RedisDriver extends DeepBaseDriver {
@@ -50,6 +50,18 @@ export class RedisDriver extends DeepBaseDriver {
       return null;
     }
   }
+
+  /**
+   * Run a query descriptor over the object stored at ...path.
+   * The value is read from Redis and evaluated in memory; filters are not
+   * pushed to the server and no indexes are used.
+   * @param {Array<string|number>} path - Path to the queried object
+   * @param {object[]} steps - Query descriptor steps
+   * @returns {Promise<Array<{id: string, value: any}>>} Matching records
+   */
+  async query(path, steps) {
+    return evaluateQuery(await this.get(...path), steps, { path });
+  }
   
   async _acquireLock(key) {
     // Wait for any pending operation on this key to complete
@@ -176,9 +188,11 @@ export class RedisDriver extends DeepBaseDriver {
       }
       
       const lastKey = args[args.length - 1];
-      delete current[lastKey];
-      
-      await this.client.set(redisKey, JSON.stringify(data));
+      // Array indices are spliced so the array shrinks instead of keeping a
+      // hole that JSON.stringify renders as null.
+      if (removeKey(current, lastKey)) {
+        await this.client.set(redisKey, JSON.stringify(data));
+      }
     }
     
     return [key, ...args];

package/test/test.js

@@ -1,5 +1,7 @@
 import assert from 'assert';
 import { DeepBase } from '../../core/src/index.js';
+import { arrayScenarios } from '../../core/test/array-scenario.js';
+import { queryScenarios, seedQueryFixture } from '../../core/test/query-scenario.js';
 import { RedisDriver } from '../src/RedisDriver.js';
 
 describe('RedisDriver', function() {
@@ -434,5 +436,24 @@ describe('RedisDriver', function() {
       assert.strictEqual(keys.length, operations);
     });
   });
-});
 
+  describe('query()', function() {
+    beforeEach(async function() {
+      await seedQueryFixture(db);
+    });
+
+    for (const scenario of queryScenarios) {
+      it(scenario.title, async function() {
+        await scenario.run(db);
+      });
+    }
+  });
+
+  describe('Array Operations', function() {
+    for (const scenario of arrayScenarios) {
+      it(scenario.title, async function() {
+        await scenario.run(db);
+      });
+    }
+  });
+});