@wopr-network/platform-core 1.39.0 → 1.39.1

package/dist/billing/crypto/btc/address-gen.js

@@ -56,7 +56,7 @@ function base58encode(data) {
     for (const byte of data) {
         if (byte !== 0)
             break;
-        encoded = "1" + encoded;
+        encoded = `1${encoded}`;
     }
     return encoded;
 }

package/dist/credits/ledger.js

@@ -182,8 +182,11 @@ export class DrizzleLedger {
             });
             // Phase 1: resolve all account IDs with row locks so concurrent transactions
             // are serialized before any balance check or update.
+            // Sort by accountCode to establish a consistent global lock ordering and
+            // prevent deadlocks when concurrent transactions lock overlapping accounts.
+            const sortedLines = [...input.lines].sort((a, b) => a.accountCode < b.accountCode ? -1 : a.accountCode > b.accountCode ? 1 : 0);
             const resolvedLines = [];
-            for (const line of input.lines) {
+            for (const line of sortedLines) {
                 let accountId;
                 if (line.accountCode.startsWith("2000:")) {
                     accountId = await this.ensureTenantAccountLocked(tx, line.accountCode.slice(5));

package/dist/credits/ledger.test.js

@@ -111,6 +111,27 @@ describe("DrizzleLedger", () => {
             });
             expect(entry.lines).toHaveLength(3);
         });
+        it("acquires locks in consistent order regardless of input line order", async () => {
+            // Post a 3-line entry with accounts in descending order.
+            // If lock ordering works, this should succeed without deadlock
+            // even when concurrent transactions use a different line order.
+            const entry = await ledger.post({
+                entryType: "correction",
+                tenantId: "t1",
+                lines: [
+                    { accountCode: "5070", amount: Credit.fromCents(500), side: "debit" },
+                    { accountCode: "2000:t1", amount: Credit.fromCents(200), side: "credit" },
+                    { accountCode: "1000", amount: Credit.fromCents(300), side: "credit" },
+                ],
+            });
+            // Entry should succeed and contain all 3 lines
+            expect(entry.lines).toHaveLength(3);
+            // Verify balances are correct (lock order doesn't affect correctness)
+            const cashBal = await ledger.accountBalance("1000");
+            expect(cashBal.toCentsRounded()).toBe(-300); // credit side on debit-normal = negative
+            const tb = await ledger.trialBalance();
+            expect(tb.balanced).toBe(true);
+        });
     });
     // -----------------------------------------------------------------------
     // credit() — convenience

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wopr-network/platform-core",
-  "version": "1.39.0",
+  "version": "1.39.1",
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/billing/crypto/btc/address-gen.ts

@@ -71,7 +71,7 @@ function base58encode(data: Uint8Array): string {
   }
   for (const byte of data) {
     if (byte !== 0) break;
-    encoded = "1" + encoded;
+    encoded = `1${encoded}`;
   }
   return encoded;
 }

package/src/credits/ledger.test.ts

@@ -139,6 +139,31 @@ describe("DrizzleLedger", () => {
 
       expect(entry.lines).toHaveLength(3);
     });
+
+    it("acquires locks in consistent order regardless of input line order", async () => {
+      // Post a 3-line entry with accounts in descending order.
+      // If lock ordering works, this should succeed without deadlock
+      // even when concurrent transactions use a different line order.
+      const entry = await ledger.post({
+        entryType: "correction",
+        tenantId: "t1",
+        lines: [
+          { accountCode: "5070", amount: Credit.fromCents(500), side: "debit" },
+          { accountCode: "2000:t1", amount: Credit.fromCents(200), side: "credit" },
+          { accountCode: "1000", amount: Credit.fromCents(300), side: "credit" },
+        ],
+      });
+
+      // Entry should succeed and contain all 3 lines
+      expect(entry.lines).toHaveLength(3);
+
+      // Verify balances are correct (lock order doesn't affect correctness)
+      const cashBal = await ledger.accountBalance("1000");
+      expect(cashBal.toCentsRounded()).toBe(-300); // credit side on debit-normal = negative
+
+      const tb = await ledger.trialBalance();
+      expect(tb.balanced).toBe(true);
+    });
   });
 
   // -----------------------------------------------------------------------

package/src/credits/ledger.ts

@@ -387,8 +387,13 @@ export class DrizzleLedger implements ILedger {
 
       // Phase 1: resolve all account IDs with row locks so concurrent transactions
       // are serialized before any balance check or update.
+      // Sort by accountCode to establish a consistent global lock ordering and
+      // prevent deadlocks when concurrent transactions lock overlapping accounts.
+      const sortedLines = [...input.lines].sort((a, b) =>
+        a.accountCode < b.accountCode ? -1 : a.accountCode > b.accountCode ? 1 : 0,
+      );
       const resolvedLines: Array<JournalLine & { accountId: string }> = [];
-      for (const line of input.lines) {
+      for (const line of sortedLines) {
         let accountId: string;
         if (line.accountCode.startsWith("2000:")) {
           accountId = await this.ensureTenantAccountLocked(tx, line.accountCode.slice(5));