saico 2.9.1 → 2.9.2

package/README.md

@@ -267,20 +267,24 @@ const restored = await Saico.restore(agent.id, { store: 'sessions' });
 
 ## Database Access
 
-Saico provides backend-agnostic DB methods. Configure via `Saico.registerBackend('dynamodb', config)` (library-level), `opt.dynamodb` (instance-level auto-creates adapter), or `opt.db` (any adapter). Table name is required on every call. Child Saico instances without their own DB inherit the parent's adapter automatically via `_getDb()`, which also falls back to the registered backend.
+Saico provides backend-agnostic DB methods. Configure via `Saico.registerBackend('dynamodb', config)` (library-level), `opt.dynamodb` (instance-level auto-creates adapter), or `opt.db` (any adapter). Key, key value, and table default to `'id'`, `this.id`, and `this._storeName` — so operating on own record is a one-liner. Child Saico instances inherit the parent's adapter via `_getDb()`, which also falls back to the registered backend.
 
 ```js
-// CRUD — table name required on every call
+// CRUD — shorthand (defaults: key='id', value=this.id, table=this._storeName)
+const me = await agent.dbGetItem();                   // get own record
+await agent.dbDeleteItem();                           // delete own record
+// Explicit
 await agent.dbPutItem({ id: '123', name: 'test' }, 'my-table');
 const item = await agent.dbGetItem('id', '123', 'my-table');
 await agent.dbDeleteItem('id', '123', 'my-table');
 const items = await agent.dbQuery('email-index', 'email', 'user@test.com', 'my-table');
 const all = await agent.dbGetAll('my-table');
 
-// Updates
-await agent.dbUpdate('id', '123', 'status', 'active', 'my-table');
-await agent.dbUpdatePath('id', '123', [{ key: 'nested' }], 'field', 'value', 'my-table');
-await agent.dbListAppend('id', '123', 'tags', 'new-tag', 'my-table');
+// Updates — setKey, item first; key, keyValue, table last (with defaults)
+await agent.dbUpdate('status', 'active');             // update own record
+await agent.dbUpdate('status', 'active', 'id', '123', 'my-table');
+await agent.dbUpdatePath([{ key: 'nested' }], 'field', 'value');
+await agent.dbListAppend('tags', 'new-tag');
 
 // Counters
 const nextId = await agent.dbNextCounterId('OrderId', 'counters');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "saico",
-  "version": "2.9.1",
+  "version": "2.9.2",
   "main": "index.js",
   "type": "commonjs",
   "description": "Hierarchical AI Conversation Orchestrator - Task hierarchy with conversation contexts",

package/saico.js

@@ -38,7 +38,8 @@ class Saico {
      * @param {boolean} [opt.redis=true] - Set false to skip Redis proxy
      * @param {boolean} [opt.createQ] - Create message Q on activate()
      * @param {boolean} [opt.isolate] - Isolate: don't aggregate from ancestors
-     * @param {Object} [opt.dynamodb] - DynamoDB config { region, credentials: { accessKeyId, secretAccessKey }, client }
+     * @param {Object} [opt.dynamodb] - DynamoDB config { region, credentials: { accessKeyId, secretAccessKey },
+     *   client }
      * @param {Object} [opt.db] - Pluggable DB backend
      * @param {string} [opt.store] - Table name for instance persistence
      * @param {Object} [opt.userData] - Initial user data
@@ -46,7 +47,8 @@ class Saico {
      */
     constructor(opt = {}) {
         // Internal properties (underscore-prefixed, not persisted to Redis)
-        this.id = opt.id || crypto.randomBytes(8).toString('hex');
+        this.name = opt.name || this.constructor.name || 'saico';
+        this.id = opt.id || this._genId();
         this._task = null;
         this._storeName = (typeof opt.store === 'string') ? opt.store : null;
         this._opt = opt;
@@ -57,7 +59,6 @@ class Saico {
         this.msgs_id = null;
 
         // Public configuration
-        this.name = opt.name || this.constructor.name || 'saico';
         this.prompt = opt.prompt || '';
         this.functions = opt.functions || null;
         this.createQ = opt.createQ || false;
@@ -500,6 +501,14 @@ class Saico {
         };
     }
 
+    /**
+     * Generate an id. Called by constructor when opt.id is not provided.
+     * Overridable by subclasses for custom ID schemes.
+     */
+    _genId() {
+        return crypto.randomBytes(8).toString('hex');
+    }
+
     /**
      * Save instance state to registered backend under _storeName.
      */
@@ -537,23 +546,23 @@ class Saico {
         throw new Error('No DB backend configured. Call Saico.registerBackend() or set opt.db.');
     }
 
-    async dbPutItem(item, table) {
+    async dbPutItem(item, table = this._storeName) {
         const db = this._getDb();
         return db.put(item, table);
     }
 
-    async dbGetItem(key, value, table) {
+    async dbGetItem(key = 'id', value = this.id, table = this._storeName) {
         const db = this._getDb();
         const result = await db.get(key, value, table);
         return result ? this._deserializeRecord(result) : result;
     }
 
-    async dbDeleteItem(key, value, table) {
+    async dbDeleteItem(key = 'id', value = this.id, table = this._storeName) {
         const db = this._getDb();
         return db.delete(key, value, table);
     }
 
-    async dbQuery(index, key, value, table) {
+    async dbQuery(index, key, value, table = this._storeName) {
         const db = this._getDb();
         const results = await db.query(index, key, value, table);
         return Array.isArray(results)
@@ -561,7 +570,7 @@ class Saico {
             : results;
     }
 
-    async dbGetAll(table) {
+    async dbGetAll(table = this._storeName) {
         const db = this._getDb();
         const results = await db.getAll(table);
         return Array.isArray(results)
@@ -569,42 +578,42 @@ class Saico {
             : results;
     }
 
-    async dbUpdate(key, keyValue, setKey, item, table) {
+    async dbUpdate(setKey, item, key = 'id', keyValue = this.id, table = this._storeName) {
         const db = this._getDb();
         return db.update(key, keyValue, setKey, item, table);
     }
 
-    async dbUpdatePath(key, keyValue, path, setKey, item, table) {
+    async dbUpdatePath(path, setKey, item, key = 'id', keyValue = this.id, table = this._storeName) {
         const db = this._getDb();
         return db.updatePath(key, keyValue, path, setKey, item, table);
     }
 
-    async dbListAppend(key, keyValue, setKey, item, table) {
+    async dbListAppend(setKey, item, key = 'id', keyValue = this.id, table = this._storeName) {
         const db = this._getDb();
         return db.listAppend(key, keyValue, setKey, item, table);
     }
 
-    async dbListAppendPath(key, keyValue, path, setKey, item, table) {
+    async dbListAppendPath(path, setKey, item, key = 'id', keyValue = this.id, table = this._storeName) {
         const db = this._getDb();
         return db.listAppendPath(key, keyValue, path, setKey, item, table);
     }
 
-    async dbNextCounterId(counter, table) {
+    async dbNextCounterId(counter, table = this._storeName) {
         const db = this._getDb();
         return db.nextCounterId(counter, table);
     }
 
-    async dbGetCounterValue(counter, table) {
+    async dbGetCounterValue(counter, table = this._storeName) {
         const db = this._getDb();
         return db.getCounterValue(counter, table);
     }
 
-    async dbSetCounterValue(counter, value, table) {
+    async dbSetCounterValue(counter, value, table = this._storeName) {
         const db = this._getDb();
         return db.setCounterValue(counter, value, table);
     }
 
-    async dbCountItems(table) {
+    async dbCountItems(table = this._storeName) {
         const db = this._getDb();
         return db.countItems(table);
     }