@moneypot/hub 1.22.1 → 1.22.3

package/dist/src/liability/public/db.js

@@ -1,5 +1,9 @@
 import { exactlyOneRow } from "../../db/util.js";
 export async function dbCreateLiability(pgClient, { lockedBankrollId, casinoId, experienceId, userId, currencyKey, refId, refType, maxPayout, deadline, }) {
+    await pgClient.query({
+        text: `SELECT 1 FROM hub.bankroll WHERE id = $1 FOR UPDATE`,
+        values: [lockedBankrollId],
+    });
     const row = await pgClient
         .query({
         text: `
@@ -8,9 +12,16 @@ export async function dbCreateLiability(pgClient, { lockedBankrollId, casinoId,
           VALUES ($1, $2, $3, $4, $5, $6, $7, $8, $9)
           RETURNING *
         ),
+        -- Sub-statements share one snapshot, so the sum can't see the
+        -- inserted row; its payout is added separately.
         bankroll_update AS (
           UPDATE hub.bankroll
-          SET liability = liability + $8
+          SET liability = $8 + (
+            SELECT COALESCE(SUM(max_payout), 0)
+            FROM hub.liability
+            WHERE bankroll_id = $1
+              AND resolved_at IS NULL
+          )
           WHERE id = $1
         )
         SELECT * FROM inserted
@@ -32,6 +43,17 @@ export async function dbCreateLiability(pgClient, { lockedBankrollId, casinoId,
 }
 export async function dbResolveLiabilityByRef(pgClient, { refType, refId, lockedBankrollId, }) {
     const _proofOfGoodBehavior = lockedBankrollId;
+    await pgClient.query({
+        text: `
+      SELECT 1 FROM hub.bankroll
+      WHERE id = (
+        SELECT bankroll_id FROM hub.liability
+        WHERE ref_type = $1 AND ref_id = $2
+      )
+      FOR UPDATE
+    `,
+        values: [refType, refId],
+    });
     const result = await pgClient.query({
         text: `
       WITH resolved AS (
@@ -40,10 +62,18 @@ export async function dbResolveLiabilityByRef(pgClient, { refType, refId, locked
         WHERE ref_type = $1
           AND ref_id = $2
           AND resolved_at IS NULL
-        RETURNING bankroll_id, max_payout
+        RETURNING id, bankroll_id
       )
+      -- The sum reads the statement's snapshot, where the resolved row is
+      -- still active, so it is excluded by id.
       UPDATE hub.bankroll
-      SET liability = liability - resolved.max_payout
+      SET liability = (
+        SELECT COALESCE(SUM(l.max_payout), 0)
+        FROM hub.liability l
+        WHERE l.bankroll_id = resolved.bankroll_id
+          AND l.resolved_at IS NULL
+          AND l.id <> resolved.id
+      )
       FROM resolved
       WHERE hub.bankroll.id = resolved.bankroll_id
     `,

package/dist/src/take-request/process-take-request.js

@@ -178,13 +178,18 @@ export async function processTakeRequests({ abortSignal, controllerId, casinoId,
         if (abortSignal.aborted) {
             break;
         }
-        await processSingleTakeRequest({
-            mpTakeRequestId: takeRequest.id,
-            mpTakeRequest: takeRequest,
-            casinoId,
-            graphqlClient,
-            pool,
-        });
+        try {
+            await processSingleTakeRequest({
+                mpTakeRequestId: takeRequest.id,
+                mpTakeRequest: takeRequest,
+                casinoId,
+                graphqlClient,
+                pool,
+            });
+        }
+        catch (error) {
+            logger.error(error, `[processTakeRequests] Failed to process take request ${takeRequest.id}`);
+        }
     }
     await processPendingTransferCompletions({
         casinoId,
@@ -252,7 +257,11 @@ async function createAndProcessNewTakeRequest({ pgClient, mpTakeRequest, casinoI
         currencyKey: mpTakeRequest.currencyKey,
         casinoId,
     });
-    assert(dbUser && dbExperience && dbCurrency && dbBalance, "Required entities not found");
+    if (!dbUser || !dbExperience || !dbCurrency || !dbBalance) {
+        logger.warn(`[createAndProcessNewTakeRequest] Rejecting take request ${mpTakeRequest.id}: no hub balance`);
+        await rejectMpTakeRequest(pgClient, graphqlClient, mpTakeRequest.id);
+        return null;
+    }
     const amountToTransfer = Math.floor(typeof mpTakeRequest.amount === "number"
         ? Math.min(mpTakeRequest.amount, dbBalance.amount)
         : dbBalance.amount);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moneypot/hub",
-  "version": "1.22.1",
+  "version": "1.22.3",
   "author": "moneypot.com",
   "homepage": "https://moneypot.com/hub",
   "keywords": [