@mohasinac/appkit 3.2.11 → 3.2.12

package/dist/_internal/server/features/checkout/actions.js

@@ -1,4 +1,5 @@
 import { normalizeError } from "../../../../errors/normalize";
+import { safeFireAndForget } from "../../../../utils/safe-fire-forget";
 /**
  * Checkout server actions (appkit).
  *
@@ -29,6 +30,7 @@ import { formatShippingAddress } from "./data";
 import { enforceMaxPerUserForCart, computePrizeRevealDeadline, } from "./prize-bundle-gates";
 import { getListingRule, runSyncPreflight, } from "../../../shared/checkout/rules";
 import { cartIsDigitalOnly } from "../../../shared/listing-types/cart-shipping";
+const AUDIT_LOG_FAIL_MSG = "checkout: audit log failed (non-critical)";
 /**
  * SB-UNI-N — Atomically claim the next available digital code for a confirmed
  * order. Pre-fetches a candidate code outside the transaction then locks and
@@ -264,7 +266,7 @@ export async function createCheckoutOrderAction(input) {
         if (!addressId) {
             failedCheckoutRepository
                 .logCheckout(uid, "address_not_found", "Address required for physical cart", { addressId: "", paymentMethod })
-                .catch(console.error);
+                .catch((err) => { void normalizeError(err); serverLogger.warn(AUDIT_LOG_FAIL_MSG, { error: err instanceof Error ? err.message : String(err) }); });
             throw new NotFoundError(ERROR_MESSAGES.CHECKOUT.ADDRESS_REQUIRED);
         }
         const addressDoc = await unitOfWork.addresses.findById(addressId);
@@ -275,7 +277,7 @@ export async function createCheckoutOrderAction(input) {
         if (!resolvedAddress) {
             failedCheckoutRepository
                 .logCheckout(uid, "address_not_found", "Address not found", { addressId, paymentMethod })
-                .catch(console.error);
+                .catch((err) => { void normalizeError(err); serverLogger.warn(AUDIT_LOG_FAIL_MSG, { error: err instanceof Error ? err.message : String(err) }); });
             throw new NotFoundError(ERROR_MESSAGES.CHECKOUT.ADDRESS_REQUIRED);
         }
         shippingAddress = formatShippingAddress(resolvedAddress);
@@ -414,7 +416,7 @@ export async function createCheckoutOrderAction(input) {
             addressId,
             paymentMethod,
         })
-            .catch(console.error);
+            .catch((logErr) => { void normalizeError(logErr); serverLogger.warn(AUDIT_LOG_FAIL_MSG, { error: logErr instanceof Error ? logErr.message : String(logErr) }); });
         throw err;
     }
     const { available, unavailable, emailOtpUsed } = stockResult;
@@ -425,7 +427,7 @@ export async function createCheckoutOrderAction(input) {
             paymentMethod,
             cartItemCount: cartItems.length,
         })
-            .catch(console.error);
+            .catch((logErr) => { void normalizeError(logErr); serverLogger.warn(AUDIT_LOG_FAIL_MSG, { error: logErr instanceof Error ? logErr.message : String(logErr) }); });
         throw new ValidationError(ERROR_MESSAGES.CHECKOUT.INSUFFICIENT_STOCK);
     }
     const appliedCoupons = cart.appliedCoupons ?? [];
@@ -669,7 +671,7 @@ export async function verifyAndPlaceRazorpayOrderAction(input) {
             gatewayPaymentId: razorpay_payment_id,
             addressId,
         })
-            .catch(console.error);
+            .catch((logErr) => { void normalizeError(logErr); serverLogger.warn(AUDIT_LOG_FAIL_MSG, { error: logErr instanceof Error ? logErr.message : String(logErr) }); });
         throw new ValidationError(ERROR_MESSAGES.CHECKOUT.PAYMENT_FAILED);
     }
     const cart = await unitOfWork.carts.getOrCreate(uid);
@@ -711,7 +713,7 @@ export async function verifyAndPlaceRazorpayOrderAction(input) {
                 gatewayPaymentId: razorpay_payment_id,
                 addressId,
             })
-                .catch(console.error);
+                .catch((err) => { void normalizeError(err); serverLogger.warn(AUDIT_LOG_FAIL_MSG, { error: err instanceof Error ? err.message : String(err) }); });
             throw new ApiError(403, "Order verification required. Please complete OTP verification and retry.");
         }
     }
@@ -762,7 +764,7 @@ export async function verifyAndPlaceRazorpayOrderAction(input) {
                 gatewayPaymentId: razorpay_payment_id,
                 addressId,
             })
-                .catch(console.error);
+                .catch((err) => { void normalizeError(err); serverLogger.warn(AUDIT_LOG_FAIL_MSG, { error: err instanceof Error ? err.message : String(err) }); });
             throw new ValidationError(exists
                 ? ERROR_MESSAGES.CHECKOUT.INSUFFICIENT_STOCK
                 : ERROR_MESSAGES.CHECKOUT.PRODUCT_UNAVAILABLE);
@@ -790,7 +792,7 @@ export async function verifyAndPlaceRazorpayOrderAction(input) {
                 amountRs: paidAmountRs,
                 addressId,
             })
-                .catch(console.error);
+                .catch((err) => { void normalizeError(err); serverLogger.warn(AUDIT_LOG_FAIL_MSG, { error: err instanceof Error ? err.message : String(err) }); });
             throw new ValidationError(ERROR_MESSAGES.CHECKOUT.PAYMENT_FAILED);
         }
     }
@@ -996,7 +998,7 @@ export async function verifyAndPlaceRazorpayOrderAction(input) {
     });
     if (!isDigitalCartRazorpay && addressId) {
         const otpRefForDelete = consentOtpRef(db, uid, addressId);
-        otpRefForDelete.delete().catch(console.error);
+        safeFireAndForget(otpRefForDelete.delete(), "checkout: delete OTP consent ref after Razorpay payment");
     }
     if (emailsToSend.length > 0) {
         Promise.all(emailsToSend.map((e) => sendOrderConfirmationEmail(e))).catch((err) => serverLogger.error("Order confirmation email error:", err));

package/dist/_internal/server/features/grouped/data.js

@@ -1,5 +1,7 @@
 import { cache } from "react";
 import { getAdminDb } from "../../../../providers/db-firebase";
+import { normalizeError } from "../../../../errors/normalize";
+import { serverLogger } from "../../../../monitoring/server-logger";
 import { GROUPED_LISTINGS_COLLECTION, } from "../../../../features/grouped/schemas/firestore";
 import { productRepository } from "../../../../repositories";
 import { PRODUCT_FIELDS, CATEGORY_FIELDS } from "../../../../constants/field-names";
@@ -55,7 +57,9 @@ export const getGroupedListingForDetail = cache(async (slug) => {
         }
         return mapDoc(snap.docs[0]);
     }
-    catch {
+    catch (err) {
+        void normalizeError(err);
+        serverLogger.warn("grouped-data: getGroupedListingForDetail failed — returning null", { slug, error: err instanceof Error ? err.message : String(err) });
         return null;
     }
 });
@@ -93,7 +97,9 @@ export async function listGroupedListings(params = {}) {
             .get();
         return snap.docs.map(mapDoc);
     }
-    catch {
+    catch (err) {
+        void normalizeError(err);
+        serverLogger.warn("grouped-data: listGroupedListings failed — returning empty", { params, error: err instanceof Error ? err.message : String(err) });
         return [];
     }
 }
@@ -118,7 +124,9 @@ export async function listSitemapGroupedListings() {
             };
         });
     }
-    catch {
+    catch (err) {
+        void normalizeError(err);
+        serverLogger.warn("grouped-data: listSitemapGroupedListings failed — returning empty", { error: err instanceof Error ? err.message : String(err) });
         return [];
     }
 }

package/dist/features/scams/repository/scammer.repository.js

@@ -1,6 +1,7 @@
 import { DatabaseError } from "../../../errors";
 import { BaseRepository, prepareForFirestore, } from "../../../providers/db-firebase";
 import { normalizeError } from "../../../errors/normalize";
+import { serverLogger } from "../../../monitoring/server-logger";
 import { increment } from "../../../contracts/field-ops";
 import { SCAMMER_COLLECTION, SCAMMER_FIELDS, SCAMMER_INCIDENTS_SUBCOLLECTION, SCAMMER_COMMENTS_SUBCOLLECTION, DEFAULT_SCAMMER_DATA, } from "../schemas/firestore";
 class ScammerRepository extends BaseRepository {
@@ -116,7 +117,9 @@ class ScammerRepository extends BaseRepository {
                 .get();
             return snap.docs.map((d) => this.mapDoc(d));
         }
-        catch {
+        catch (err) {
+            void normalizeError(err);
+            serverLogger.warn("scammer-repo: listPublicIncidents failed — returning empty", { scammerId, error: err instanceof Error ? err.message : String(err) });
             return [];
         }
     }
@@ -132,7 +135,9 @@ class ScammerRepository extends BaseRepository {
                 .get();
             return snap.docs.map((d) => this.mapDoc(d));
         }
-        catch {
+        catch (err) {
+            void normalizeError(err);
+            serverLogger.warn("scammer-repo: listPublicComments failed — returning empty", { scammerId, error: err instanceof Error ? err.message : String(err) });
             return [];
         }
     }
@@ -143,7 +148,9 @@ class ScammerRepository extends BaseRepository {
             const results = await Promise.all(ids.slice(0, 5).map((id) => this.findById(id).catch(() => null)));
             return results.filter((d) => d !== null && d.status === "verified");
         }
-        catch {
+        catch (err) {
+            void normalizeError(err);
+            serverLogger.warn("scammer-repo: findManyById failed — returning empty", { ids, error: err instanceof Error ? err.message : String(err) });
             return [];
         }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mohasinac/appkit",
-  "version": "3.2.11",
+  "version": "3.2.12",
   "license": "MIT",
   "publishConfig": {
     "access": "public"