s3db.js 11.0.0 → 11.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "s3db.js",
-  "version": "11.0.0",
+  "version": "11.0.2",
   "description": "Use AWS S3, the world's most reliable document storage, as a database with this ORM.",
   "main": "dist/s3db.cjs.js",
   "module": "dist/s3db.es.js",
@@ -133,6 +133,7 @@
     "dev": "rollup -c -w",
     "test": "pnpm run test:js && pnpm run test:ts",
     "test:js": "node --no-warnings --experimental-vm-modules node_modules/jest/bin/jest.js",
+    "test:quick": "node --no-warnings --experimental-vm-modules node_modules/jest/bin/jest.js --maxWorkers=2 --bail",
     "test:ts": "tsc --noEmit --project tests/typescript/tsconfig.json",
     "test:coverage": "node --no-warnings --experimental-vm-modules node_modules/jest/bin/jest.js --coverage --maxWorkers=1",
     "test:serial": "node --no-warnings --experimental-vm-modules node_modules/jest/bin/jest.js --runInBand",

package/src/plugins/eventual-consistency/analytics.js

@@ -17,6 +17,18 @@ import { groupByCohort } from "./utils.js";
 export async function updateAnalytics(transactions, analyticsResource, config) {
   if (!analyticsResource || transactions.length === 0) return;
 
+  // CRITICAL VALIDATION: Ensure field is set in config
+  // This can be undefined due to race conditions when multiple handlers share config
+  if (!config.field) {
+    throw new Error(
+      `[EventualConsistency] CRITICAL BUG: config.field is undefined in updateAnalytics()!\n` +
+      `This indicates a race condition in the plugin where multiple handlers are sharing the same config object.\n` +
+      `Config: ${JSON.stringify({ resource: config.resource, field: config.field })}\n` +
+      `Transactions count: ${transactions.length}\n` +
+      `AnalyticsResource: ${analyticsResource?.name || 'unknown'}`
+    );
+  }
+
   if (config.verbose) {
     console.log(
       `[EventualConsistency] ${config.resource}.${config.field} - ` +

package/src/plugins/eventual-consistency/consolidation.js

@@ -413,17 +413,71 @@ export async function consolidateRecord(
       );
     }
 
+    // 🔥 DEBUG: Log BEFORE update
+    if (config.verbose) {
+      console.log(
+        `🔥 [DEBUG] BEFORE targetResource.update() {` +
+        `\n  originalId: '${originalId}',` +
+        `\n  field: '${config.field}',` +
+        `\n  consolidatedValue: ${consolidatedValue},` +
+        `\n  currentValue: ${currentValue}` +
+        `\n}`
+      );
+    }
+
     // Update the original record
     // NOTE: We do NOT attempt to insert non-existent records because:
     // 1. Target resources typically have required fields we don't know about
     // 2. Record creation should be the application's responsibility
     // 3. Transactions will remain pending until the record is created
-    const [updateOk, updateErr] = await tryFn(() =>
+    const [updateOk, updateErr, updateResult] = await tryFn(() =>
       targetResource.update(originalId, {
         [config.field]: consolidatedValue
       })
     );
 
+    // 🔥 DEBUG: Log AFTER update
+    if (config.verbose) {
+      console.log(
+        `🔥 [DEBUG] AFTER targetResource.update() {` +
+        `\n  updateOk: ${updateOk},` +
+        `\n  updateErr: ${updateErr?.message || 'undefined'},` +
+        `\n  updateResult: ${JSON.stringify(updateResult, null, 2)},` +
+        `\n  hasField: ${updateResult?.[config.field]}` +
+        `\n}`
+      );
+    }
+
+    // 🔥 VERIFY: Check if update actually persisted
+    if (updateOk && config.verbose) {
+      // Bypass cache to get fresh data
+      const [verifyOk, verifyErr, verifiedRecord] = await tryFn(() =>
+        targetResource.get(originalId, { skipCache: true })
+      );
+
+      console.log(
+        `🔥 [DEBUG] VERIFICATION (fresh from S3, no cache) {` +
+        `\n  verifyOk: ${verifyOk},` +
+        `\n  verifiedRecord[${config.field}]: ${verifiedRecord?.[config.field]},` +
+        `\n  expectedValue: ${consolidatedValue},` +
+        `\n  ✅ MATCH: ${verifiedRecord?.[config.field] === consolidatedValue}` +
+        `\n}`
+      );
+
+      // If verification fails, this is a critical bug
+      if (verifyOk && verifiedRecord?.[config.field] !== consolidatedValue) {
+        console.error(
+          `❌ [CRITICAL BUG] Update reported success but value not persisted!` +
+          `\n  Resource: ${config.resource}` +
+          `\n  Field: ${config.field}` +
+          `\n  Record ID: ${originalId}` +
+          `\n  Expected: ${consolidatedValue}` +
+          `\n  Actually got: ${verifiedRecord?.[config.field]}` +
+          `\n  This indicates a bug in s3db.js resource.update()`
+        );
+      }
+    }
+
     if (!updateOk) {
       // Check if record doesn't exist
       if (updateErr?.message?.includes('does not exist')) {