lamix 4.2.11 → 4.2.12

package/lib/index.js

@@ -232,31 +232,51 @@ class DB {
 
       try {
         const result = await executor();
+
         this._emit('query', {
           sql,
           bindings,
           time: performance.now() - start,
           driver: this.driver
         });
+
         return result;
       } catch (err) {
-        this._emit('error', { err, sql, bindings, attempt });
+        const isConnectionError =
+          err?.code === 'PROTOCOL_CONNECTION_LOST' ||
+          err?.code === 'ECONNRESET' ||
+          err?.code === 'ECONNREFUSED' ||
+          err?.code === 'ETIMEDOUT' ||
+          /dead|lost|timeout|reset|closed|connect/i.test(err.message);
+
+        this._emit('error', {
+          err,
+          sql,
+          bindings,
+          attempt,
+          connectionError: isConnectionError
+        });
 
         const canRetry =
-          !isWrite ||
-          this._als.getStore()?.inTransaction;
-
-        if (
-          canRetry &&
-          attempt < this.retryAttempts &&
-          /dead|lost|timeout|reset|closed/i.test(err.message)
-        ) {
+          (!isWrite || this._als.getStore()?.inTransaction) &&
+          isConnectionError;
+
+        if (canRetry && attempt < this.retryAttempts) {
           await this.reconnect();
           await sleep(100 * attempt);
           continue;
         }
 
-        throw new DBError('DB query failed', { sql, bindings, err });
+        throw new DBError(
+          isConnectionError ? 'DB connection failed' : 'DB query failed',
+          {
+            sql,
+            bindings,
+            attempt,
+            connectionError: isConnectionError,
+            err
+          }
+        );
       }
     }
   }
@@ -1146,9 +1166,13 @@ class QueryBuilder {
 
   // Helper to normalize operator
   _normalizeOperator(operator) {
-    const op = operator ? operator.toUpperCase() : '=';
+    const op = operator ? operator.toUpperCase() : '='
     if (!VALID_OPERATORS.includes(op)) {
-      throw new Error(`Invalid SQL operator: ${operator}`);
+      throw new DBError('Invalid SQL operator', {
+        operator,
+        normalized: op,
+        method: '_normalizeOperator'
+      });
     }
 
     // Convert ILIKE to proper operator for MySQL
@@ -1348,7 +1372,13 @@ class QueryBuilder {
   }
 
   whereIn(column, values = []) {
-    if (!Array.isArray(values)) throw new Error('whereIn expects array');
+    if (!Array.isArray(values)) {
+      throw new DBError('whereIn expects an array', {
+        method: 'whereIn',
+        column,
+        values
+      });
+    }
     if (!values.length) {
       return this._pushWhere({ type: 'raw', raw: '0 = 1', bindings: [] });
     }
@@ -1391,7 +1421,13 @@ class QueryBuilder {
   }
 
   whereNotIn(column, values = []) {
-    if (!Array.isArray(values)) throw new Error('whereNotIn expects array');
+    if (!Array.isArray(values)) {
+      throw new DBError('whereNotIn expects an array', {
+        method: 'whereNotIn',
+        column,
+        values
+      });
+    }
     if (!values.length) {
       return this._pushWhere({ type: 'raw', raw: '1 = 1', bindings: [] });
     }
@@ -1576,7 +1612,14 @@ class QueryBuilder {
   whereHas(relationName, callback, boolean = 'AND') {
     const relation = this.modelClass.relations()?.[relationName];
     if (!relation) {
-      throw new Error(`Relation '${relationName}' is not defined on ${this.modelClass.name}`);
+      throw new DBError(
+        `Relation '${relationName}' is not defined`,
+        {
+          model: this.modelClass?.name,
+          relation: relationName,
+          method: 'whereHas'
+        }
+      );
     }
 
     const RelatedModel = relation.model();
@@ -1641,6 +1684,16 @@ class QueryBuilder {
     parts.push('SELECT');
     if (this._distinct) parts.push('DISTINCT');
 
+    // Normalize SELECT * to table.* when a model is attached
+    if (
+      this.modelClass &&
+      this._select.length === 1 &&
+      this._select[0] === '*'
+    ) {
+      const table = this.tableAlias || this.table;
+      this._select = [`${escapeId(table)}.*`];
+    }
+
     parts.push(this._select.length ? this._select.join(', ') : '*');
 
     if (this._fromRaw) {
@@ -1785,7 +1838,10 @@ class QueryBuilder {
           break;
 
         default:
-          throw new Error('Unknown where type: ' + w.type);
+          throw new DBError('Unknown where clause type', {
+            type: w.type,
+            where: w
+          });
       }
     });
 
@@ -1823,20 +1879,29 @@ class QueryBuilder {
     const sql = this._compileSelect();
     const binds = this._gatherBindings();
 
-    const rows = await DB.raw(sql, binds);
+    try {
+      const rows = await DB.raw(sql, binds);
 
-    if (this.modelClass) {
-      const models = rows.map(r => new this.modelClass(r, true));
+      if (this.modelClass) {
+        const models = rows.map(r => new this.modelClass(r, true));
 
-      if (this._with.length) {
-        const loaded = await this._eagerLoad(models);
-        return new Collection(loaded);
+        if (this._with.length) {
+          const loaded = await this._eagerLoad(models);
+          return new Collection(loaded);
+        }
+
+        return new Collection(models);
       }
 
-      return new Collection(models);
+      return new Collection(rows);
+    } catch (err) {
+      throw new DBError('Select query failed', {
+        table: this.table,
+        sql,
+        bindings: binds,
+        err
+      });
     }
-
-    return new Collection(rows);
   }
 
   async first() {
@@ -1849,7 +1914,13 @@ class QueryBuilder {
 
   async firstOrFail() {
     const r = await this.first();
-    if (!r) throw new Error('Record not found');
+    if (!r) {
+      throw new DBError('Record not found', {
+        method: 'firstOrFail',
+        table: this.table,
+        model: this.modelClass?.name
+      });
+    }
     return r;
   }
 
@@ -1860,10 +1931,19 @@ class QueryBuilder {
     c.limit(1);
 
     const sql = c._compileSelect();
-    const b = c._gatherBindings();
+    const bindings = c._gatherBindings();
 
-    const rows = await DB.raw(sql, b);
-    return rows.length > 0;
+    try {
+      const rows = await DB.raw(sql, bindings);
+      return rows.length > 0;
+    } catch (err) {
+      throw new DBError('Exists query failed', {
+        table: this.table,
+        sql,
+        bindings,
+        err
+      });
+    }
   }
 
   async doesntExist() {
@@ -1878,11 +1958,19 @@ class QueryBuilder {
     c._offset = null;
 
     const sql = c._compileSelect();
-    const b = c._gatherBindings();
+    const bindings = c._gatherBindings();
 
-    const rows = await DB.raw(sql, b);
-
-    return rows[0] ? Number(rows[0].aggregate) : 0;
+    try {
+      const rows = await DB.raw(sql, bindings);
+      return rows[0] ? Number(rows[0].aggregate) : 0;
+    } catch (err) {
+      throw new DBError('Count query failed', {
+        table: this.table,
+        sql,
+        bindings,
+        err
+      });
+    }
   }
 
   async _aggregate(expr) {
@@ -1893,11 +1981,20 @@ class QueryBuilder {
     c._offset = null;
 
     const sql = c._compileSelect();
-    const b = c._gatherBindings();
+    const bindings = c._gatherBindings();
 
-    const rows = await DB.raw(sql, b);
-
-    return rows[0] ? Number(rows[0].aggregate) : 0;
+    try {
+      const rows = await DB.raw(sql, bindings);
+      return rows[0] ? Number(rows[0].aggregate) : 0;
+    } catch (err) {
+      throw new DBError('Aggregate query failed', {
+        table: this.table,
+        expression: expr,
+        sql,
+        bindings,
+        err
+      });
+    }
   }
 
   sum(c) { return this._aggregate(`SUM(${escapeId(c)})`); }
@@ -1911,36 +2008,51 @@ class QueryBuilder {
     c._select = [escapeId(col)];
 
     const sql = c._compileSelect();
-    const b = c._gatherBindings();
-
-    const rows = await DB.raw(sql, b);
+    const bindings = c._gatherBindings();
 
-    return rows.map(r => r[col]);
+    try {
+      const rows = await DB.raw(sql, bindings);
+      return rows.map(r => r[col]);
+    } catch (err) {
+      throw new DBError('Pluck query failed', {
+        table: this.table,
+        column: col,
+        sql,
+        bindings,
+        err
+      });
+    }
   }
 
   async paginate(page = 1, perPage = 15) {
     page = Math.max(1, Number(page));
     perPage = Math.max(1, Number(perPage));
 
-    // Build a clone to compute total (safe — uses your existing _clone and count)
-    const countClone = this._clone();
-    countClone._select = [`COUNT(*) AS aggregate`];
-    countClone._orders = [];
-    countClone._limit = null;
-    countClone._offset = null;
-
-    const total = await countClone.count('*');
-
-    const offset = (page - 1) * perPage;
-
-    // Use a clone to fetch rows so we don't mutate caller's builder
-    const rows = await this._clone()
-      .limit(perPage)
-      .offset(offset)
-      .get();
-
-    // Return Paginator instance with .toJSON()
-    return new Paginator(rows, total, page, perPage);
+    try {
+      const countClone = this._clone();
+      countClone._select = [`COUNT(*) AS aggregate`];
+      countClone._orders = [];
+      countClone._limit = null;
+      countClone._offset = null;
+
+      const total = await countClone.count('*');
+
+      const offset = (page - 1) * perPage;
+
+      const rows = await this._clone()
+        .limit(perPage)
+        .offset(offset)
+        .get();
+
+      return new Paginator(rows, total, page, perPage);
+    } catch (err) {
+      throw new DBError('Pagination failed', {
+        table: this.table,
+        page,
+        perPage,
+        err
+      });
+    }
   }
 
   /**************************************************************************
@@ -1955,10 +2067,18 @@ class QueryBuilder {
       `) VALUES (${placeholders})`;
 
     const bindings = Object.values(values);
-
-    const result = await DB.raw(sql, bindings);
-
-    return result.affectedRows || 0;
+    try {
+      const result = await DB.raw(sql, bindings);
+
+      return result.affectedRows || 0;
+    } catch (err) {
+      throw new DBError('Insert failed', {
+        table: this.table,
+        sql,
+        bindings,
+        err
+      });
+    }
   }
 
   async insertGetId(values) {
@@ -1970,15 +2090,22 @@ class QueryBuilder {
       `) VALUES (${placeholders})`;
 
     const bindings = Object.values(values);
-
-    const result = await DB.raw(sql, bindings);
-
-    return result.insertId ?? null;
+    try {
+      const result = await DB.raw(sql, bindings);
+
+      return result.insertId ?? null;
+    } catch (err) {
+      throw new DBError('Insert (get ID) failed', {
+        table: this.table,
+        sql,
+        bindings,
+        err
+      });
+    }
   }
 
   async update(values) {
     if (!Object.keys(values).length) return 0;
-
     const setClause = Object.keys(values)
       .map(k => `${escapeId(k)} = ?`)
       .join(', ');
@@ -1988,10 +2115,18 @@ class QueryBuilder {
       `UPDATE ${escapeId(this.table)} SET ${setClause} ${whereSql}`;
 
     const bindings = [...Object.values(values), ...this._gatherBindings()];
-
-    const result = await DB.raw(sql, bindings);
-
-    return result.affectedRows || 0;
+    try {
+      const result = await DB.raw(sql, bindings);
+
+      return result.affectedRows || 0;
+    } catch (err) {
+      throw new DBError('Update failed', {
+        table: this.table,
+        sql,
+        bindings,
+        err
+      });
+    }
   }
 
   async increment(col, by = 1) {
@@ -2000,11 +2135,20 @@ class QueryBuilder {
       `SET ${escapeId(col)} = ${escapeId(col)} + ? ` +
       this._compileWhereOnly();
 
-    const b = [by, ...this._gatherBindings()];
-
-    const res = await DB.raw(sql, b);
+    const bindings = [by, ...this._gatherBindings()];
 
-    return res.affectedRows || 0;
+    try {
+      const res = await DB.raw(sql, bindings);
+      return res.affectedRows || 0;
+    } catch (err) {
+      throw new DBError('Increment failed', {
+        table: this.table,
+        column: col,
+        sql,
+        bindings,
+        err
+      });
+    }
   }
 
   async decrement(col, by = 1) {
@@ -2013,9 +2157,20 @@ class QueryBuilder {
       `SET ${escapeId(col)} = ${escapeId(col)} - ? ` +
       this._compileWhereOnly();
 
-    const b = [by, ...this._gatherBindings()];
-    const res = await DB.raw(sql, b);
-    return res.affectedRows || 0;
+    const bindings = [by, ...this._gatherBindings()];
+
+    try {
+      const res = await DB.raw(sql, bindings);
+      return res.affectedRows || 0;
+    } catch (err) {
+      throw new DBError('Decrement failed', {
+        table: this.table,
+        column: col,
+        sql,
+        bindings,
+        err
+      });
+    }
   }
 
   async delete() {
@@ -2023,16 +2178,34 @@ class QueryBuilder {
       `DELETE FROM ${escapeId(this.table)} ` +
       this._compileWhereOnly();
 
-    const b = this._gatherBindings();
+    const bindings = this._gatherBindings();
 
-    const res = await DB.raw(sql, b);
-    return res.affectedRows || 0;
+    try {
+      const res = await DB.raw(sql, bindings);
+      return res.affectedRows || 0;
+    } catch (err) {
+      throw new DBError('Delete failed', {
+        table: this.table,
+        sql,
+        bindings,
+        err
+      });
+    }
   }
 
   async truncate() {
     const sql = `TRUNCATE TABLE ${escapeId(this.table)}`;
-    await DB.raw(sql);
-    return true;
+
+    try {
+      await DB.raw(sql);
+      return true;
+    } catch (err) {
+      throw new DBError('Truncate failed', {
+        table: this.table,
+        sql,
+        err
+      });
+    }
   }
 
   _compileWhereOnly() {
@@ -2044,19 +2217,23 @@ class QueryBuilder {
    * EAGER LOAD (unchanged except robust checks)
    **************************************************************************/
   async _eagerLoad(models) {
-    for (const relName of this._with) {
-      const sample = models[0];
-      if (!sample) return models;
+    if (!models.length) return models;
 
-      const relationMethod = sample[relName];
-      if (typeof relationMethod !== 'function') {
-        throw new Error(`Relation "${relName}" is not a method on ${sample.constructor.name}`);
-      }
+    for (const relName of this._with) {
+      const sample = models.find(m => typeof m[relName] === 'function');
+      if (!sample) continue;
 
-      const relation = relationMethod.call(sample);
+      const relation = sample[relName]();
 
       if (!relation || typeof relation.eagerLoad !== 'function') {
-        throw new Error(`Relation "${relName}" does not have a valid eagerLoad method`);
+        throw new DBError(
+          `Relation "${relName}" is not eager-loadable`,
+          {
+            model: sample.constructor.name,
+            relation: relName,
+            method: 'eagerLoad'
+          }
+        );
       }
 
       await relation.eagerLoad(models, relName);
@@ -2802,28 +2979,57 @@ class Model {
     this._relations = {};
     this._exists = !!fresh;
 
-    // Only store keys with defined values
+    // ──────────────────────────────
+    // 1️⃣ hydrate attributes safely
+    // ──────────────────────────────
     for (const [k, v] of Object.entries(attributes)) {
-      if (v !== undefined) this._attributes[k] = v;
+      if (v !== undefined) {
+        this._attributes[k] = v;
+      }
+    }
+
+    // ──────────────────────────────
+    // 2️⃣ ensure timestamps ALWAYS exist
+    // ──────────────────────────────
+    if (this.constructor.timestamps) {
+      if ('created_at' in attributes)
+        this._attributes.created_at = attributes.created_at;
+
+      if ('updated_at' in attributes)
+        this._attributes.updated_at = attributes.updated_at;
+    }
+
+    // ──────────────────────────────
+    // 3️⃣ always keep primary key
+    // ──────────────────────────────
+    const pk = this.constructor.primaryKey;
+    if (pk && pk in attributes) {
+      this._attributes[pk] = attributes[pk];
     }
 
     this._original = { ...this._attributes, ...data };
 
-    // Define getters for attributes
+    // ──────────────────────────────
+    // 4️⃣ define getters/setters FOR ALL ATTRIBUTES
+    // ──────────────────────────────
     for (const k of Object.keys(this._attributes)) {
       if (!(k in this)) {
         Object.defineProperty(this, k, {
-          get: function() {
-            return this._attributes[k];
-          },
-          enumerable: true
+          get: () => this._attributes[k],
+          set: v => { this._attributes[k] = v; },
+          enumerable: true,
+          configurable: true
         });
       }
     }
   }
 
   static async validate(data, id, ignoreId = null) {
-    if (!Validator) throw new Error('Validator not found.');
+    if (!Validator) {
+      throw new DBError('Validator not found', {
+        model: this.name
+      });
+    }
 
     const rules = this.rules || {};
 
@@ -2897,20 +3103,6 @@ class Model {
     }
   }
 
-  // ──────────────────────────────
-  // Query builder accessors
-  // ──────────────────────────────
-  // static query({ withTrashed = false } = {}) {
-  //   // use tableName getter (throws if missing)
-  //   const qb = new QueryBuilder(this.tableName, this);
-  //   if (this.softDeletes && !withTrashed) {
-  //     // avoid mutating shared _wheres reference
-  //     qb._wheres = Array.isArray(qb._wheres) ? qb._wheres.slice() : [];
-  //     qb._wheres.push({ raw: `${DB.escapeId(this.deletedAt)} IS NULL` });
-  //   }
-  //   return qb;
-  // }
-
   static query({ withTrashed = false } = {}) {
     const qb = new QueryBuilder(this.tableName, this);
 
@@ -3000,14 +3192,26 @@ class Model {
   }
 
   static async findManyBy(col, values = []) {
-    if (!Array.isArray(values)) throw new Error('findManyBy expects an array of values');
+    if (!Array.isArray(values)) {
+      throw new DBError('findManyBy expects an array of values', {
+        method: 'findManyBy',
+        column: col
+      });
+    }
     if (!values.length) return [];
     return await this.query().whereIn(col, values).get();
   }
 
   // additional common accessors
   static async findMany(ids = []) {
-    if (!Array.isArray(ids)) ids = [ids];
+    // if (!Array.isArray(ids)) ids = [ids];
+    if (!Array.isArray(ids)) {
+      throw new DBError('findMany expects an array of IDs', {
+        method: 'findMany',
+        model: this.name,
+        ids
+      });
+    }
     if (!ids.length) return [];
     return await this.query().whereIn(this.primaryKey, ids).get();
   }
@@ -3077,7 +3281,10 @@ class Model {
 
     // 3. Block empty payload
     if (!Object.keys(payload).length) {
-      throw new DBError('Attempted to create with empty payload');
+      throw new DBError('Attempted to create with empty payload', {
+        model: this.name,
+        attrs
+      });
     }
 
     // 4. Create + save
@@ -3164,9 +3371,9 @@ class Model {
   // 🛡 SANITIZATION UTIL
   // ──────────────────────────────
   static isBadValue(value) {
-    if (value === null || value === undefined || value === '') return true;
-    if (typeof value === 'string' && !value.trim()) return true;
-    return false;
+    if (value === null || value === undefined) return true;
+    if (typeof value === 'string' && !value.trim()) return true; // empty string
+    return false; // allow 0, false, etc.
   }
 
   sanitize(attrs = {}) {
@@ -3177,7 +3384,7 @@ class Model {
       const val = attrs[key];
 
       if (!this.constructor.isBadValue(val)) {
-        clean[key] = val;
+        clean[key] = val;   // keep 0, false, etc.
       } else if (keepCols.includes(key)) {
         clean[key] = null;
       }
@@ -3191,13 +3398,24 @@ class Model {
   // ──────────────────────────────
   async fill(attrs = {}) {
     const allowed = this.constructor.fillable || Object.keys(attrs);
+    let filled = false;
 
     for (const key of Object.keys(attrs)) {
       const val = attrs[key];
-      if (allowed.includes(key) && !this.constructor.isBadValue(val)) {
-        this._attributes[key] = val;
+      if (allowed.includes(key) && val !== undefined) {
+        this._attributes[key] = val; // 0, false, null preserved
+        filled = true;
       }
     }
+
+    if (!filled && Object.keys(attrs).length > 0) {
+      throw new DBError('No fillable attributes provided', {
+        model: this.constructor.name,
+        attrs,
+        fillable: allowed
+      });
+    }
+
     return this;
   }
 
@@ -3209,17 +3427,23 @@ class Model {
   // INSERT – validation first
   // ──────────────────────────────
   async saveNew(attrs) {
+    if (!this._exists && !Object.keys(this._attributes).length) {
+      throw new DBError('Cannot save empty model', {
+        model: this.constructor.name
+      });
+    }
+
     const payload = this.sanitize(attrs || this._attributes);
 
-    // Validate BEFORE hooks/db
+    // validate BEFORE hooks/db
     await this.constructor.validate(payload);
     await this.trigger('creating');
 
     // timestamps
     if (this.constructor.timestamps) {
       const now = new Date();
-      payload.created_at = payload.created_at || now;
-      payload.updated_at = payload.updated_at || now;
+      if (payload.created_at === undefined) payload.created_at = now;
+      if (payload.updated_at === undefined) payload.updated_at = now;
     }
 
     // soft deletes
@@ -3231,13 +3455,11 @@ class Model {
     }
 
     const qb = this.constructor.query();
-
     const result = await qb.insert(payload);
 
     // handle pk
     const pk = this.constructor.primaryKey;
     const insertId = Array.isArray(result) ? result[0] : result;
-
     if (!(pk in payload) && insertId !== undefined) {
       payload[pk] = insertId;
     }
@@ -3248,12 +3470,16 @@ class Model {
 
     return this;
   }
-
   // ──────────────────────────────
   // UPDATE – only dirty fields
   // ──────────────────────────────
   async save() {
-    if (!this._exists) return this.saveNew(this._attributes);
+    // if (!this._exists) return this.saveNew(this._attributes);
+    if (!this._exists && !Object.keys(this._attributes).length) {
+      throw new DBError('Cannot save empty model', {
+        model: this.constructor.name
+      });
+    }
 
     await this.trigger('updating');
 
@@ -3264,11 +3490,8 @@ class Model {
     for (const key of Object.keys(attrs)) {
       const val = attrs[key];
 
-      if (val !== orig[key] && !this.constructor.isBadValue(val)) {
-        if (this.constructor.softDeletes &&
-            key === this.constructor.deletedAt) {
-          continue;
-        }
+      if (val !== orig[key] && val !== undefined) { // only treat undefined as "no change"
+        if (this.constructor.softDeletes && key === this.constructor.deletedAt) continue;
         dirty[key] = val;
       }
     }
@@ -3285,14 +3508,10 @@ class Model {
 
     // validate BEFORE db write
     const pk = this.constructor.primaryKey;
-    await this.constructor.validate(
-      { ...this._original, ...payload },
-      this._attributes[pk]
-    );
+    await this.constructor.validate({ ...this._original, ...payload }, this._attributes[pk]);
 
     const id = this._attributes[pk];
     const qb = this.constructor.query();
-
     await qb.where(pk, id).update(payload);
 
     this._original = { ...this.sanitize(this._attributes) };
@@ -3326,8 +3545,13 @@ class Model {
       let rel;
       try {
         rel = fn.call(dummy);
-      } catch {
-        continue;
+      } catch (err) {
+        // continue;
+        throw new DBError('Failed to resolve relation', {
+          model: this.name,
+          relation: name,
+          err
+        });
       }
 
       if (!rel) continue;
@@ -3395,8 +3619,13 @@ class Model {
         // RESTRICT — block delete
         // ──────────────────────────────────
         case 'restrict':
-          throw new Error(
-            `Cannot delete ${this.constructor.name}: related ${relName} exists`
+          throw new DBError(
+            `Cannot delete ${this.constructor.name}: related ${relName} exists`,
+            {
+              model: this.constructor.name,
+              relation: relName,
+              behavior: 'restrict'
+            }
           );
 
         // ──────────────────────────────────
@@ -3458,7 +3687,14 @@ class Model {
 
 
   static async destroy(ids) {
-    if (!Array.isArray(ids)) ids = [ids];
+    if (!Array.isArray(ids)) {
+      throw new DBError('destroy expects an array of IDs', {
+        model: this.name,
+        ids
+      });
+    }
+
+    // if (!Array.isArray(ids)) ids = [ids];
     const pk = this.primaryKey;
 
     // --- Load models so cascade works ---
@@ -3618,7 +3854,12 @@ class Model {
 
   async refresh() {
     const pk = this.constructor.primaryKey;
-    if (!this._attributes[pk]) return this;
+    // if (!this._attributes[pk]) return this;
+    if (!this._attributes[pk]) {
+      throw new DBError('Cannot refresh model without primary key', {
+        model: this.constructor.name
+      });
+    }
     const fresh = await this.constructor.find(this._attributes[pk]);
     if (fresh) {
       this._attributes = { ...fresh._attributes };