create-brainerce-store 1.12.2 → 1.12.4

package/dist/index.js

@@ -31,7 +31,7 @@ var require_package = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "create-brainerce-store",
-      version: "1.12.2",
+      version: "1.12.4",
       description: "Scaffold a production-ready e-commerce storefront connected to Brainerce",
       bin: {
         "create-brainerce-store": "dist/index.js"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-brainerce-store",
-  "version": "1.12.2",
+  "version": "1.12.4",
   "description": "Scaffold a production-ready e-commerce storefront connected to Brainerce",
   "bin": {
     "create-brainerce-store": "dist/index.js"

package/templates/nextjs/base/src/app/order-confirmation/page.tsx

@@ -119,7 +119,9 @@ function OrderConfirmationContent() {
 
         <p className="text-muted-foreground mt-2">{t('confirmationEmail')}</p>
 
-        {result.status.orderId && <ConfirmationDownloads orderId={result.status.orderId} />}
+        {result.status.orderId && (
+          <ConfirmationDownloads orderId={result.status.orderId} checkoutId={checkoutId!} />
+        )}
 
         <div className="mt-8 flex flex-col items-center justify-center gap-3 sm:flex-row">
           <Link
@@ -180,26 +182,35 @@ function OrderConfirmationContent() {
   );
 }
 
-function ConfirmationDownloads({ orderId }: { orderId: string }) {
+function ConfirmationDownloads({ orderId, checkoutId }: { orderId: string; checkoutId: string }) {
   const t = useTranslations('orderConfirmation');
   const [downloads, setDownloads] = useState<OrderDownloadLink[] | null>(null);
 
   useEffect(() => {
     let cancelled = false;
     async function fetchDownloads() {
-      try {
-        const client = getClient();
-        const links = await client.getOrderDownloads(orderId);
-        if (!cancelled && links.length > 0) setDownloads(links);
-      } catch {
-        // Not all orders have downloads - silently ignore
+      const client = getClient();
+      // Retry a few times — the worker may still be writing downloadMeta
+      for (let attempt = 0; attempt < 3; attempt++) {
+        try {
+          const links = await client.getOrderDownloads(orderId, { checkoutId });
+          if (!cancelled && links.length > 0) {
+            setDownloads(links);
+            return;
+          }
+        } catch {
+          // Not all orders have downloads
+        }
+        if (attempt < 2 && !cancelled) {
+          await new Promise((r) => setTimeout(r, 1500));
+        }
       }
     }
     fetchDownloads();
     return () => {
       cancelled = true;
     };
-  }, [orderId]);
+  }, [orderId, checkoutId]);
 
   if (!downloads || downloads.length === 0) return null;
 

package/templates/nextjs/base/src/components/checkout/payment-step.tsx

@@ -82,6 +82,7 @@ export function PaymentStep({ checkoutId, className }: PaymentStepProps) {
     onError: (_r: unknown) => {},
     onTimeout: () => {},
     onWalletChange: (_s: string) => {},
+    retryRender: () => {},
   });
 
   const handleSuccess = useCallback(
@@ -115,8 +116,9 @@ export function PaymentStep({ checkoutId, className }: PaymentStepProps) {
       ];
       const msg = extractMessage(response);
       if (TRANSIENT.some((e) => msg.includes(e))) {
-        console.info('Payment SDK: transient startup error (will retry):', msg);
-        return; // Don't show error — renderWhenReady will retry
+        console.info('Payment SDK: transient error, retrying render in 1s:', msg);
+        setTimeout(() => cbRef.current.retryRender(), 1000);
+        return;
       }
       console.error('Payment SDK error:', response);
       setError(msg || t('paymentError'));
@@ -133,6 +135,7 @@ export function PaymentStep({ checkoutId, className }: PaymentStepProps) {
       }
       if (state === 'close') setSdkReady(false);
     },
+    retryRender: () => {},
   };
 
   // =========================================================================
@@ -220,6 +223,8 @@ export function PaymentStep({ checkoutId, className }: PaymentStepProps) {
 
     // --- Init: called in s.onload (as Grow docs require) ---
     function initSdk(sdk: PaymentClientSdk) {
+      if (sdkInitDone) return; // Guard against double init
+
       const global = (window as any)[sdk.globalName!];
       if (!global) {
         setError(t('failedToLoadPaymentSdk'));
@@ -245,37 +250,47 @@ export function PaymentStep({ checkoutId, className }: PaymentStepProps) {
       sdkInitDone = true;
     }
 
-    // --- Render: retry until wallet actually opens (onWalletChange) ---
-    // No artificial timeout — the SDK fires onTimeout when the session expires.
-    function renderWhenReady(sdk: PaymentClientSdk, intent: PaymentIntent) {
+    // --- Render: call once, then safety-net retries if wallet doesn't open ---
+    // Grow SDK sometimes silently swallows renderPaymentOptions when its
+    // internal resources (mp.min.js etc.) aren't fully loaded yet.
+    // Strategy: render once, then retry up to 3 times with increasing delays
+    // (2s, 3s, 4s) if onWalletChange("open") hasn't fired.
+    let pendingRender: { sdk: PaymentClientSdk; intent: PaymentIntent } | null = null;
+    let renderAttempts = 0;
+    const MAX_RENDER_ATTEMPTS = 4;
+
+    function renderPayment(sdk: PaymentClientSdk, intent: PaymentIntent) {
       const global = (window as any)[sdk.globalName!];
-      if (!global) return;
+      if (!global || walletOpenRef.current) return;
 
       const renderMethod = sdk.renderMethod || 'renderPaymentOptions';
       const renderArg = sdk.renderArg || intent.clientSecret;
-      let renderCalled = false;
-
-      function attempt() {
-        if (walletOpenRef.current) return;
-        try {
-          global[renderMethod](renderArg);
-          renderCalled = true;
-          console.info('Payment SDK: render call accepted');
-        } catch (err) {
-          console.info('Payment SDK: render not ready yet, retrying...');
-        }
+      renderAttempts++;
+
+      try {
+        global[renderMethod](renderArg);
+        console.info(`Payment SDK: renderPaymentOptions called (attempt ${renderAttempts})`);
+      } catch (err) {
+        console.info('Payment SDK: render threw, will retry in 1s');
       }
 
-      // Try immediately, then keep retrying until onWalletChange('open') fires
-      attempt();
-      const id = setInterval(() => {
-        if (walletOpenRef.current) {
-          clearInterval(id);
-          return;
-        }
-        if (!renderCalled) attempt();
-      }, 500);
-      cleanups.push(() => clearInterval(id));
+      // Safety net: if wallet doesn't open within a delay, retry
+      if (renderAttempts < MAX_RENDER_ATTEMPTS) {
+        const delay = 1000 + renderAttempts * 1000; // 2s, 3s, 4s
+        const retryId = setTimeout(() => {
+          if (!walletOpenRef.current) {
+            console.info(`Payment SDK: wallet not open after ${delay}ms, retrying render...`);
+            renderPayment(sdk, intent);
+          }
+        }, delay);
+        cleanups.push(() => clearTimeout(retryId));
+      }
+    }
+
+    function retryRender() {
+      if (pendingRender && !walletOpenRef.current) {
+        renderPayment(pendingRender.sdk, pendingRender.intent);
+      }
     }
 
     // =============================================
@@ -326,14 +341,18 @@ export function PaymentStep({ checkoutId, className }: PaymentStepProps) {
       }
       if (sdk.renderType !== 'sdk-widget' || !sdk.globalName) return;
 
+      // Store for retryRender from onError callback
+      pendingRender = { sdk, intent };
+      cbRef.current.retryRender = retryRender;
+
       // If SDK wasn't loaded from providers, load + init now
       if (!sdkInitDone) {
         loadScript(sdk);
-        // Wait for init to complete, then render
+        // Wait for init to complete, then render once
         const id = setInterval(() => {
           if (sdkInitDone) {
             clearInterval(id);
-            renderWhenReady(sdk, intent);
+            renderPayment(sdk, intent);
           }
         }, 100);
         cleanups.push(() => clearInterval(id));
@@ -358,8 +377,8 @@ export function PaymentStep({ checkoutId, className }: PaymentStepProps) {
         }
       }
 
-      // SDK ready — render
-      renderWhenReady(sdk, intent);
+      // SDK ready — render once
+      renderPayment(sdk, intent);
     });
 
     return () => cleanups.forEach((fn) => fn());