create-nextblock 0.16.3 → 0.17.0

package/templates/nextblock-template/lib/cms/payments-reminder.test.ts

@@ -1,135 +1,135 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-
-type ProductRow = {
-  product_type: 'physical' | 'digital';
-  status: 'active' | 'draft';
-};
-
-const mocks = vi.hoisted(() => ({
-  getStoreReadiness: vi.fn(),
-  verifyPackageOnline: vi.fn(),
-  products: [] as ProductRow[],
-}));
-
-vi.mock('server-only', () => ({}));
-
-vi.mock('@nextblock-cms/ecommerce/server', () => ({
-  getStoreReadiness: mocks.getStoreReadiness,
-}));
-
-vi.mock('@nextblock-cms/db/server', () => ({
-  verifyPackageOnline: mocks.verifyPackageOnline,
-  createClient: () => ({
-    auth: {
-      getUser: async () => ({ data: { user: { id: 'admin-1' } } }),
-    },
-    from: (table: string) => {
-      if (table === 'profiles') {
-        return {
-          select: () => ({
-            eq: () => ({
-              maybeSingle: async () => ({ data: { role: 'ADMIN' } }),
-            }),
-          }),
-        };
-      }
-
-      if (table === 'products') {
-        return {
-          select: () => {
-            const filters: Array<(product: ProductRow) => boolean> = [];
-            const query = {
-              eq(column: keyof ProductRow, value: string) {
-                filters.push((product) => product[column] === value);
-                return query;
-              },
-              then(onFulfilled: (value: { count: number }) => unknown) {
-                const count = mocks.products.filter((product) =>
-                  filters.every((filter) => filter(product)),
-                ).length;
-                return Promise.resolve({ count }).then(onFulfilled);
-              },
-            };
-            return query;
-          },
-        };
-      }
-
-      throw new Error(`Unexpected table in test: ${table}`);
-    },
-  }),
-}));
-
-import { getPaymentsReminder } from './payments-reminder';
-
-function providerReadiness({
-  stripeReady,
-  freemiusReady,
-}: {
-  stripeReady: boolean;
-  freemiusReady: boolean;
-}) {
-  return {
-    stripe: {
-      provider: 'stripe',
-      label: 'Stripe',
-      enabled: true,
-      hasKeys: stripeReady,
-      ready: stripeReady,
-      missing: stripeReady ? [] : ['Secret key'],
-    },
-    freemius: {
-      provider: 'freemius',
-      label: 'Freemius',
-      enabled: true,
-      hasKeys: freemiusReady,
-      ready: freemiusReady,
-      missing: freemiusReady ? [] : ['Secret key'],
-    },
-  };
-}
-
-describe('CMS payments reminder', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-    mocks.products.length = 0;
-    mocks.verifyPackageOnline.mockResolvedValue(true);
-  });
-
-  it('ignores an unconfigured Stripe toggle when the store has only digital products', async () => {
-    mocks.products.push({ product_type: 'digital', status: 'active' });
-    mocks.getStoreReadiness.mockResolvedValue(
-      providerReadiness({ stripeReady: false, freemiusReady: true }),
-    );
-
-    await expect(getPaymentsReminder()).resolves.toBeNull();
-  });
-
-  it('warns about Stripe when any physical product needs it', async () => {
-    mocks.products.push({ product_type: 'physical', status: 'draft' });
-    mocks.getStoreReadiness.mockResolvedValue(
-      providerReadiness({ stripeReady: false, freemiusReady: true }),
-    );
-
-    await expect(getPaymentsReminder()).resolves.toEqual({
-      blocked: [
-        { provider: 'stripe', label: 'Stripe', missing: ['Secret key'] },
-      ],
-      affectedProducts: 0,
-    });
-  });
-
-  it('warns about Freemius when a digital product needs it', async () => {
-    mocks.products.push({ product_type: 'digital', status: 'active' });
-    mocks.getStoreReadiness.mockResolvedValue(
-      providerReadiness({ stripeReady: true, freemiusReady: false }),
-    );
-
-    await expect(getPaymentsReminder()).resolves.toEqual({
-      blocked: [
-        { provider: 'freemius', label: 'Freemius', missing: ['Secret key'] },
-      ],
-      affectedProducts: 1,
-    });
-  });
-});
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
+type ProductRow = {
+  product_type: 'physical' | 'digital';
+  status: 'active' | 'draft';
+};
+
+const mocks = vi.hoisted(() => ({
+  getStoreReadiness: vi.fn(),
+  verifyPackageOnline: vi.fn(),
+  products: [] as ProductRow[],
+}));
+
+vi.mock('server-only', () => ({}));
+
+vi.mock('@nextblock-cms/ecommerce/server', () => ({
+  getStoreReadiness: mocks.getStoreReadiness,
+}));
+
+vi.mock('@nextblock-cms/db/server', () => ({
+  verifyPackageOnline: mocks.verifyPackageOnline,
+  createClient: () => ({
+    auth: {
+      getUser: async () => ({ data: { user: { id: 'admin-1' } } }),
+    },
+    from: (table: string) => {
+      if (table === 'profiles') {
+        return {
+          select: () => ({
+            eq: () => ({
+              maybeSingle: async () => ({ data: { role: 'ADMIN' } }),
+            }),
+          }),
+        };
+      }
+
+      if (table === 'products') {
+        return {
+          select: () => {
+            const filters: Array<(product: ProductRow) => boolean> = [];
+            const query = {
+              eq(column: keyof ProductRow, value: string) {
+                filters.push((product) => product[column] === value);
+                return query;
+              },
+              then(onFulfilled: (value: { count: number }) => unknown) {
+                const count = mocks.products.filter((product) =>
+                  filters.every((filter) => filter(product)),
+                ).length;
+                return Promise.resolve({ count }).then(onFulfilled);
+              },
+            };
+            return query;
+          },
+        };
+      }
+
+      throw new Error(`Unexpected table in test: ${table}`);
+    },
+  }),
+}));
+
+import { getPaymentsReminder } from './payments-reminder';
+
+function providerReadiness({
+  stripeReady,
+  freemiusReady,
+}: {
+  stripeReady: boolean;
+  freemiusReady: boolean;
+}) {
+  return {
+    stripe: {
+      provider: 'stripe',
+      label: 'Stripe',
+      enabled: true,
+      hasKeys: stripeReady,
+      ready: stripeReady,
+      missing: stripeReady ? [] : ['Secret key'],
+    },
+    freemius: {
+      provider: 'freemius',
+      label: 'Freemius',
+      enabled: true,
+      hasKeys: freemiusReady,
+      ready: freemiusReady,
+      missing: freemiusReady ? [] : ['Secret key'],
+    },
+  };
+}
+
+describe('CMS payments reminder', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    mocks.products.length = 0;
+    mocks.verifyPackageOnline.mockResolvedValue(true);
+  });
+
+  it('ignores an unconfigured Stripe toggle when the store has only digital products', async () => {
+    mocks.products.push({ product_type: 'digital', status: 'active' });
+    mocks.getStoreReadiness.mockResolvedValue(
+      providerReadiness({ stripeReady: false, freemiusReady: true }),
+    );
+
+    await expect(getPaymentsReminder()).resolves.toBeNull();
+  });
+
+  it('warns about Stripe when any physical product needs it', async () => {
+    mocks.products.push({ product_type: 'physical', status: 'draft' });
+    mocks.getStoreReadiness.mockResolvedValue(
+      providerReadiness({ stripeReady: false, freemiusReady: true }),
+    );
+
+    await expect(getPaymentsReminder()).resolves.toEqual({
+      blocked: [
+        { provider: 'stripe', label: 'Stripe', missing: ['Secret key'] },
+      ],
+      affectedProducts: 0,
+    });
+  });
+
+  it('warns about Freemius when a digital product needs it', async () => {
+    mocks.products.push({ product_type: 'digital', status: 'active' });
+    mocks.getStoreReadiness.mockResolvedValue(
+      providerReadiness({ stripeReady: true, freemiusReady: false }),
+    );
+
+    await expect(getPaymentsReminder()).resolves.toEqual({
+      blocked: [
+        { provider: 'freemius', label: 'Freemius', missing: ['Secret key'] },
+      ],
+      affectedProducts: 1,
+    });
+  });
+});

package/templates/nextblock-template/lib/cms/payments-reminder.ts

@@ -1,105 +1,105 @@
-import 'server-only';
-
-import { createClient, verifyPackageOnline } from '@nextblock-cms/db/server';
-import { getStoreReadiness } from '@nextblock-cms/ecommerce/server';
-
-/**
- * A store that has commerce switched on but cannot actually take money. This is easy to
- * end up in — the products, prices and pages all look finished — and the only symptom
- * used to be a shopper hitting a payment error at checkout. The banner makes the gap
- * visible in the CMS instead of on the storefront.
- */
-export interface PaymentsReminder {
-  /** Providers that are not ready, with what each is missing. */
-  blocked: Array<{
-    provider: 'stripe' | 'freemius';
-    label: string;
-    missing: string[];
-  }>;
-  /** Published products that currently cannot be bought because of the above. */
-  affectedProducts: number;
-}
-
-/**
- * Best-effort reminder state for the CMS layout. Returns null (no banner) whenever
- * nagging would be wrong or the answer can't be determined — an unmigrated database,
- * an instance without commerce, or the sandbox.
- */
-export async function getPaymentsReminder(): Promise<PaymentsReminder | null> {
-  // The sandbox refuses to store live credentials at all (savePaymentCredentials throws),
-  // so a "finish setting up payments" nag there is noise the operator cannot act on.
-  if (process.env.NEXT_PUBLIC_IS_SANDBOX === 'true') return null;
-
-  try {
-    if (!(await verifyPackageOnline('ecommerce'))) return null;
-
-    const supabase = createClient();
-
-    // Only ADMINs can act on this (the /cms/payments route is admin-only), so don't do
-    // the work for anyone else.
-    const {
-      data: { user },
-    } = await supabase.auth.getUser();
-    if (!user) return null;
-    const { data: profile } = await supabase
-      .from('profiles')
-      .select('role')
-      .eq('id', user.id)
-      .maybeSingle();
-    if (profile?.role !== 'ADMIN') return null;
-
-    const readiness = await getStoreReadiness();
-    const unready = (['stripe', 'freemius'] as const)
-      .map((provider) => readiness[provider])
-      .filter((entry) => !entry.ready);
-
-    if (unready.length === 0) return null;
-
-    // A provider matters to this reminder only when the catalogue contains products
-    // of the type routed through it. Do not treat a stale/accidental provider toggle as
-    // usage: that produced a Stripe warning in digital-only stores with no Stripe keys.
-    // Count across ALL statuses so a shop building a draft catalogue is warned before
-    // it publishes rather than after.
-    const usage = await Promise.all(
-      unready.map(async (entry) => {
-        const productType =
-          entry.provider === 'stripe' ? 'physical' : 'digital';
-        const [{ count: totalCount }, { count: activeCount }] =
-          await Promise.all([
-            supabase
-              .from('products')
-              .select('id', { count: 'exact', head: true })
-              .eq('product_type', productType),
-            supabase
-              .from('products')
-              .select('id', { count: 'exact', head: true })
-              .eq('status', 'active')
-              .eq('product_type', productType),
-          ]);
-        return {
-          entry,
-          inUse: (totalCount ?? 0) > 0,
-          activeProducts: activeCount ?? 0,
-        };
-      }),
-    );
-
-    const relevant = usage.filter((item) => item.inUse);
-    if (relevant.length === 0) return null;
-
-    return {
-      blocked: relevant.map(({ entry }) => ({
-        provider: entry.provider,
-        label: entry.label,
-        missing: entry.missing,
-      })),
-      affectedProducts: relevant.reduce(
-        (total, item) => total + item.activeProducts,
-        0,
-      ),
-    };
-  } catch {
-    // Never let a reminder break the CMS shell.
-    return null;
-  }
-}
+import 'server-only';
+
+import { createClient, verifyPackageOnline } from '@nextblock-cms/db/server';
+import { getStoreReadiness } from '@nextblock-cms/ecommerce/server';
+
+/**
+ * A store that has commerce switched on but cannot actually take money. This is easy to
+ * end up in — the products, prices and pages all look finished — and the only symptom
+ * used to be a shopper hitting a payment error at checkout. The banner makes the gap
+ * visible in the CMS instead of on the storefront.
+ */
+export interface PaymentsReminder {
+  /** Providers that are not ready, with what each is missing. */
+  blocked: Array<{
+    provider: 'stripe' | 'freemius';
+    label: string;
+    missing: string[];
+  }>;
+  /** Published products that currently cannot be bought because of the above. */
+  affectedProducts: number;
+}
+
+/**
+ * Best-effort reminder state for the CMS layout. Returns null (no banner) whenever
+ * nagging would be wrong or the answer can't be determined — an unmigrated database,
+ * an instance without commerce, or the sandbox.
+ */
+export async function getPaymentsReminder(): Promise<PaymentsReminder | null> {
+  // The sandbox refuses to store live credentials at all (savePaymentCredentials throws),
+  // so a "finish setting up payments" nag there is noise the operator cannot act on.
+  if (process.env.NEXT_PUBLIC_IS_SANDBOX === 'true') return null;
+
+  try {
+    if (!(await verifyPackageOnline('ecommerce'))) return null;
+
+    const supabase = createClient();
+
+    // Only ADMINs can act on this (the /cms/payments route is admin-only), so don't do
+    // the work for anyone else.
+    const {
+      data: { user },
+    } = await supabase.auth.getUser();
+    if (!user) return null;
+    const { data: profile } = await supabase
+      .from('profiles')
+      .select('role')
+      .eq('id', user.id)
+      .maybeSingle();
+    if (profile?.role !== 'ADMIN') return null;
+
+    const readiness = await getStoreReadiness();
+    const unready = (['stripe', 'freemius'] as const)
+      .map((provider) => readiness[provider])
+      .filter((entry) => !entry.ready);
+
+    if (unready.length === 0) return null;
+
+    // A provider matters to this reminder only when the catalogue contains products
+    // of the type routed through it. Do not treat a stale/accidental provider toggle as
+    // usage: that produced a Stripe warning in digital-only stores with no Stripe keys.
+    // Count across ALL statuses so a shop building a draft catalogue is warned before
+    // it publishes rather than after.
+    const usage = await Promise.all(
+      unready.map(async (entry) => {
+        const productType =
+          entry.provider === 'stripe' ? 'physical' : 'digital';
+        const [{ count: totalCount }, { count: activeCount }] =
+          await Promise.all([
+            supabase
+              .from('products')
+              .select('id', { count: 'exact', head: true })
+              .eq('product_type', productType),
+            supabase
+              .from('products')
+              .select('id', { count: 'exact', head: true })
+              .eq('status', 'active')
+              .eq('product_type', productType),
+          ]);
+        return {
+          entry,
+          inUse: (totalCount ?? 0) > 0,
+          activeProducts: activeCount ?? 0,
+        };
+      }),
+    );
+
+    const relevant = usage.filter((item) => item.inUse);
+    if (relevant.length === 0) return null;
+
+    return {
+      blocked: relevant.map(({ entry }) => ({
+        provider: entry.provider,
+        label: entry.label,
+        missing: entry.missing,
+      })),
+      affectedProducts: relevant.reduce(
+        (total, item) => total + item.activeProducts,
+        0,
+      ),
+    };
+  } catch {
+    // Never let a reminder break the CMS shell.
+    return null;
+  }
+}