payment-kit 1.13.72 → 1.13.73

package/api/src/jobs/checkout-session.ts

@@ -2,7 +2,7 @@ import { mintNftForCheckoutSession } from '../integrations/blockchain/nft';
 import { ensurePassportIssued, ensurePassportRevoked } from '../integrations/blocklet/passport';
 import { events } from '../libs/event';
 import logger from '../libs/logger';
-import type { CheckoutSession, Subscription } from '../store/models';
+import { CheckoutSession, Price, Subscription } from '../store/models';
 
 // eslint-disable-next-line require-await
 export async function startCheckoutSessionQueue() {
@@ -13,6 +13,17 @@ export async function startCheckoutSessionQueue() {
     mintNftForCheckoutSession(checkoutSession.id).catch((err) => {
       logger.error('mintNftForCheckoutSession failed', { error: err, checkoutSession: checkoutSession.id });
     });
+
+    // lock prices used
+    Price.update(
+      { locked: true },
+      { where: { id: checkoutSession.line_items.map((x) => x.upsell_price_id || x.price_id) } }
+    ).catch((err) => {
+      logger.error('lock price on checkout session complete failed', {
+        error: err,
+        checkoutSession: checkoutSession.id,
+      });
+    });
   });
 
   events.on('customer.subscription.deleted', (subscription: Subscription) => {

package/api/src/libs/payment.ts

@@ -8,7 +8,14 @@ import { BN, fromUnitToToken } from '@ocap/util';
 import cloneDeep from 'lodash/cloneDeep';
 import type { LiteralUnion } from 'type-fest';
 
-import { Invoice, PaymentCurrency, PaymentIntent, PaymentMethod } from '../store/models';
+import {
+  CheckoutSession,
+  Invoice,
+  PaymentCurrency,
+  PaymentIntent,
+  PaymentMethod,
+  TLineItemExpanded,
+} from '../store/models';
 import type { TPaymentCurrency } from '../store/models/payment-currency';
 import { blocklet, wallet } from './auth';
 import logger from './logger';
@@ -80,7 +87,7 @@ export async function isDelegationSufficientForPayment(args: {
 
       const requested = new BN(amount);
       const allowance = new BN(tokenLimit.txAllowance);
-      if (requested.gt(allowance)) {
+      if (tokenLimit.txAllowance !== '0' && requested.gt(allowance)) {
         return { sufficient: false, reason: 'NO_ENOUGH_ALLOWANCE' };
       }
     }
@@ -181,6 +188,7 @@ export async function getPaymentDetail(userDid: string, invoice: Invoice): Promi
 }
 
 export async function getTokenLimitsForDelegation(
+  checkoutSession: CheckoutSession,
   paymentMethod: PaymentMethod,
   paymentCurrency: PaymentCurrency,
   address: string,
@@ -189,11 +197,15 @@ export async function getTokenLimitsForDelegation(
   const client = paymentMethod.getOcapClient();
   const { state } = await client.getDelegateState({ address });
 
+  const items = checkoutSession.line_items as TLineItemExpanded[];
+  const hasMetered = items.some((x) => x.price.recurring?.usage_type === 'metered');
+  const allowance = hasMetered ? '0' : amount;
+
   // @ts-ignore
   const entry: TokenLimit = {
     address: paymentCurrency.contract as string,
     to: [wallet.address], // FIXME: may broken if we have vault, migrated
-    txAllowance: amount,
+    txAllowance: allowance,
     totalAllowance: '0',
     txCount: 0,
     validUntil: 0,
@@ -204,6 +216,11 @@ export async function getTokenLimitsForDelegation(
     return [entry];
   }
 
+  // If we have metered items, we should not limit tx allowance(set to 0)
+  if (hasMetered) {
+    return [entry];
+  }
+
   const op = (state as DelegateState).ops.find((x) => x.key === OCAP_PAYMENT_TX_TYPE);
   if (op && Array.isArray(op.value.limit?.tokens) && op.value.limit.tokens.length > 0) {
     const tokenLimits = cloneDeep(op.value.limit.tokens);
@@ -212,7 +229,8 @@ export async function getTokenLimitsForDelegation(
     if (index > -1) {
       const limit = op.value.limit.tokens[index] as TokenLimit;
       // If we have a previous delegation and the txAllowance is smaller than requested amount
-      if (new BN(limit.txAllowance).lt(new BN(amount))) {
+      // If txAllowance is 0 (unlimited), we should not update it
+      if (limit.txAllowance !== '0' && new BN(limit.txAllowance).lt(new BN(amount))) {
         tokenLimits[index] = entry;
       }
     } else {

package/api/src/routes/checkout-sessions.ts

@@ -581,7 +581,7 @@ router.put('/:id/submit', user, ensureCheckoutSessionOpen, async (req, res) => {
           payment_method_id: paymentMethod.id,
           last_setup_error: null,
         });
-        logger.info('setup intent for checkout session reset', {
+        logger.info('setupIntent reset on checkout session submit', {
           session: checkoutSession.id,
           intent: setupIntent.id,
         });
@@ -599,13 +599,13 @@ router.put('/:id/submit', user, ensureCheckoutSessionOpen, async (req, res) => {
           usage: 'off_session',
           metadata: checkoutSession.metadata,
         });
+
+        // persist setup intent id
+        await checkoutSession.update({ setup_intent_id: setupIntent.id });
         logger.info('setupIntent created on checkout session submit', {
           session: checkoutSession.id,
           intent: setupIntent.id,
         });
-
-        // persist setup intent id
-        await checkoutSession.update({ setup_intent_id: setupIntent.id });
       }
     }
 
@@ -678,9 +678,6 @@ router.put('/:id/submit', user, ensureCheckoutSessionOpen, async (req, res) => {
           items: items.map((x) => x.id),
         });
 
-        // lock prices used by this subscription
-        await Price.update({ locked: true }, { where: { id: lineItems.map((x) => x.upsell_price_id || x.price_id) } });
-
         // persist subscription id
         await checkoutSession.update({ subscription_id: subscription.id });
       }

package/api/src/routes/connect/setup.ts

@@ -38,7 +38,14 @@ export default {
         checkoutSession.mode,
         paymentCurrency
       );
-      const tokenLimits = await getTokenLimitsForDelegation(paymentMethod, paymentCurrency, address, amount);
+
+      const tokenLimits = await getTokenLimitsForDelegation(
+        checkoutSession,
+        paymentMethod,
+        paymentCurrency,
+        address,
+        amount
+      );
       const tokenRequirements = await getTokenRequirements(checkoutSession, paymentMethod, paymentCurrency);
 
       return {

package/api/src/routes/connect/shared.ts

@@ -135,16 +135,16 @@ export async function ensureSetupIntent(checkoutSessionId: string, userDid?: str
   if (checkoutSession.setup_intent_id) {
     setupIntent = await SetupIntent.findByPk(checkoutSession.setup_intent_id);
     if (!setupIntent) {
-      throw new Error('Payment intent not found');
+      throw new Error('Setup intent not found');
     }
     if (setupIntent.status === 'succeeded') {
-      throw new Error('Payment intent completed');
+      throw new Error('Setup intent completed');
     }
     if (setupIntent.status === 'canceled') {
-      throw new Error('Payment intent canceled');
+      throw new Error('Setup intent canceled');
     }
     if (setupIntent.status === 'processing') {
-      throw new Error('Payment intent processing');
+      throw new Error('Setup intent processing');
     }
 
     customerId = setupIntent.customer_id;

package/api/src/routes/connect/subscribe.ts

@@ -40,7 +40,13 @@ export default {
         checkoutSession.mode,
         paymentCurrency
       );
-      const tokenLimits = await getTokenLimitsForDelegation(paymentMethod, paymentCurrency, address, amount);
+      const tokenLimits = await getTokenLimitsForDelegation(
+        checkoutSession,
+        paymentMethod,
+        paymentCurrency,
+        address,
+        amount
+      );
       const tokenRequirements = await getTokenRequirements(checkoutSession, paymentMethod, paymentCurrency);
 
       return {

package/api/src/store/migrations/20231220-setup-intent.ts

@@ -0,0 +1,22 @@
+/* eslint-disable no-await-in-loop */
+import { DataTypes } from 'sequelize';
+
+import { Migration, safeApplyColumnChanges } from '../migrate';
+
+export const up: Migration = async ({ context }) => {
+  await safeApplyColumnChanges(context, {
+    checkout_sessions: [
+      {
+        name: 'setup_intent_id',
+        field: {
+          type: DataTypes.STRING(30),
+          allowNull: true,
+        },
+      },
+    ],
+  });
+};
+
+export const down: Migration = async ({ context }) => {
+  await context.removeColumn('checkout_sessions', 'setup_intent_id');
+};

package/api/src/store/models/checkout-session.ts

@@ -413,6 +413,10 @@ export class CheckoutSession extends Model<InferAttributes<CheckoutSession>, Inf
           type: DataTypes.ENUM('auto', 'required'),
           defaultValue: 'auto',
         },
+        setup_intent_id: {
+          type: DataTypes.STRING(30),
+          allowNull: true,
+        },
       },
       {
         sequelize,

package/blocklet.yml

@@ -14,7 +14,7 @@ repository:
   type: git
   url: git+https://github.com/blocklet/payment-kit.git
 specVersion: 1.2.8
-version: 1.13.72
+version: 1.13.73
 logo: logo.png
 files:
   - dist

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "payment-kit",
-  "version": "1.13.72",
+  "version": "1.13.73",
   "scripts": {
     "dev": "COMPONENT_STORE_URL=https://test.store.blocklet.dev blocklet dev",
     "eject": "vite eject",
@@ -106,7 +106,7 @@
     "@abtnode/types": "^1.16.19",
     "@arcblock/eslint-config": "^0.2.4",
     "@arcblock/eslint-config-ts": "^0.2.4",
-    "@did-pay/types": "1.13.72",
+    "@did-pay/types": "1.13.73",
     "@types/cookie-parser": "^1.4.6",
     "@types/cors": "^2.8.17",
     "@types/dotenv-flow": "^3.3.3",
@@ -143,5 +143,5 @@
       "parser": "typescript"
     }
   },
-  "gitHead": "bbd67e67e0e6154506b12d7ce30b314e895382c0"
+  "gitHead": "544382d01feaed289ddaf2066b701a6ecf253370"
 }