chore: add logs for active subscriptions that are missing payment methods (#6863)

Author: jdolle

packages/services/api/src/modules/commerce/resolvers/Organization.ts

@@ -60,7 +60,10 @@ export const Organization: Pick<
       organizationId: billingRecord.organizationId,
     });
 
+    const logger = injector.get(Logger);
+
     if (!subscriptionInfo) {
+      logger.info('No active subscription for organization (id=%s)', org.id);
       return {
         hasActiveSubscription: false,
         canUpdateSubscription: true,
@@ -85,6 +88,10 @@ export const Organization: Pick<
       i => i.charge !== null && typeof i.charge === 'object' && i.charge?.failure_code !== null,
     );
 
+    if (!subscriptionInfo.paymentMethod) {
+      logger.warn('Active subscription found but is missing a payment method (id=%s)', org.id);
+    }
+
     return {
       hasActiveSubscription: subscriptionInfo.subscription !== null,
       canUpdateSubscription: subscriptionInfo.subscription !== null,

packages/services/commerce/src/stripe-billing/index.ts

@@ -92,10 +92,13 @@ export const stripeBillingRouter = router({
 
       const actualSubscription = subscriptions[0] || null;
 
-      const paymentMethod = await ctx.stripeBilling.stripe.paymentMethods.list({
-        customer: customer.id,
-        type: 'card',
-      });
+      const paymentMethod = await ctx.stripeBilling.stripe.customers.listPaymentMethods(
+        customer.id,
+        {
+          type: 'card',
+          limit: 1,
+        },
+      );
 
       return {
         paymentMethod: paymentMethod.data[0] || null,