@mohasinac/appkit 4.35.0 → 4.35.1

package/dist/features/auth/actions/profile-actions.d.ts

@@ -39,7 +39,20 @@ export declare function getUserSessions(userId: string): Promise<import("..").Se
  * including ones inside sent notification emails.
  */
 export declare function resolveProfileUser(identifier: string): Promise<UserDocument | null>;
-export declare function getPublicUserProfile(userId: string): Promise<Pick<UserDocument, "id" | "slug" | "displayName" | "photoURL" | "role" | "createdAt" | "storeId" | "storeSlug" | "publicProfile" | "stats"> | null>;
+/**
+ * 🛑 React.cache-wrapped because a page and its generateMetadata both read it.
+ *
+ * `resolveProfileUser` tries up to THREE lookups (slug, id, uid), and
+ * `profile/[userId]/page.tsx` called this once in generateMetadata and again in
+ * the page body — so an uncached profile view cost up to six Firestore reads to
+ * answer one question. `sellers/[id]` had the same pair.
+ *
+ * This is the rule the SSR guide already states: every data-fetch exposed to a
+ * page + generateMetadata pair is memoised per argument list, so the second call
+ * is free. Note it caches the NARROW public projection, which is what both
+ * callers want; anything needing the whole document still goes to the repository.
+ */
+export declare const getPublicUserProfile: (userId: string) => Promise<Pick<UserDocument, "id" | "slug" | "displayName" | "photoURL" | "role" | "createdAt" | "storeId" | "storeSlug" | "publicProfile" | "stats"> | null>;
 /** Fetch approved reviews for a store. storeId === storeSlug in this project. */
 export declare function getSellerReviews(storeId: string): Promise<Review[]>;
 export declare function getProfileStoreProducts(storeId: string): Promise<import("../../products").ProductDocument[]>;

package/dist/features/auth/actions/profile-actions.js

@@ -4,6 +4,7 @@
  * Pure business functions for profile CRUD, seller stats, and session reads.
  * Auth, rate-limiting, and Next.js specifics are handled by the consumer.
  */
+import { cache } from "react";
 import { maskPublicReview } from "../../../security";
 import { safeRead } from "../../../errors/safe-read";
 import { finalizeStagedMediaField } from "../../media/finalize";
@@ -47,7 +48,20 @@ export async function resolveProfileUser(identifier) {
         return byId;
     return userRepository.findByUid(identifier);
 }
-export async function getPublicUserProfile(userId) {
+/**
+ * 🛑 React.cache-wrapped because a page and its generateMetadata both read it.
+ *
+ * `resolveProfileUser` tries up to THREE lookups (slug, id, uid), and
+ * `profile/[userId]/page.tsx` called this once in generateMetadata and again in
+ * the page body — so an uncached profile view cost up to six Firestore reads to
+ * answer one question. `sellers/[id]` had the same pair.
+ *
+ * This is the rule the SSR guide already states: every data-fetch exposed to a
+ * page + generateMetadata pair is memoised per argument list, so the second call
+ * is free. Note it caches the NARROW public projection, which is what both
+ * callers want; anything needing the whole document still goes to the repository.
+ */
+export const getPublicUserProfile = cache(async function getPublicUserProfile(userId) {
     const user = await resolveProfileUser(userId);
     if (!user)
         return null;
@@ -63,7 +77,7 @@ export async function getPublicUserProfile(userId) {
         publicProfile: user.publicProfile,
         stats: user.stats,
     };
-}
+});
 /** Fetch approved reviews for a store. storeId === storeSlug in this project. */
 export async function getSellerReviews(storeId) {
     const snapshot = await safeRead(() => reviewRepository.findApprovedByStore(storeId), { route: "/profile/[userId]", key: "reviews.findApprovedByStore", fallback: [] });

package/dist/features/tester/seed-data/carts-tester-seed-data.d.ts

@@ -0,0 +1,3 @@
+import type { CartDocument } from "../../cart/schemas/firestore";
+/** The cart document id IS the user id — one cart per account. */
+export declare const cartsTesterSeedData: Partial<CartDocument>[];

package/dist/features/tester/seed-data/carts-tester-seed-data.js

@@ -0,0 +1,70 @@
+/*
+ * WHY: Shared tester sandbox — the LOCKED cart line a settled auction leaves behind.
+ * WHAT: One cart for user-tester-qa holding the win from auction-tester-sandbox-won.
+ *
+ * 🛑 WHY THIS FILE HAD TO EXIST.
+ *
+ * The sandbox seeded a won auction, its winning bid, and a delivered order made
+ * from it — the shape settlement produced BEFORE Root Cause #60, when it wrote an
+ * order document directly. That order was in a shape no orders list could render
+ * and no checkout could accept, so a winner had no way to pay at all; the fix
+ * routed a win through the CART as a locked line instead.
+ *
+ * The fixtures were never updated to match. So `win-auction` — the case whose
+ * label is literally "creates a payable locked cart line, not a stuck order" —
+ * could not pass: /user/bids offered "Pay now", it led to /checkout?lane=auction,
+ * and the cart was empty. Found by the tester 2026-09-06, which is exactly the
+ * kind of thing a case can only prove by being run.
+ *
+ * The delivered order fixture STAYS. It is the "win already completed" state and
+ * other cases read it. This adds the missing "win awaiting payment" state.
+ *
+ * 🛑 `isAuctionWin` is not decoration. `laneOf()` keys on `isAuctionWin || bidId`
+ * and ignores `listingType` and `locked` entirely — a line without it lands in the
+ * STANDARD lane, so the Won Auctions tab stays empty while a non-removable line
+ * sits in the ordinary cart. Both fields are set here so the lane is unambiguous.
+ *
+ * @tag domain:tester
+ * @tag layer:seed
+ * @tag pattern:none
+ * @tag access:server-only
+ * @tag consumers:seed/index.ts,seed/manifest.ts
+ * @tag sideEffects:none
+ */
+import { testDataExpiresAt } from "./tester-ttl";
+const NOW = new Date();
+const hoursAgo = (n) => new Date(NOW.getTime() - n * 3600000);
+/** The cart document id IS the user id — one cart per account. */
+export const cartsTesterSeedData = [
+    {
+        id: "user-tester-qa",
+        userId: "user-tester-qa",
+        items: [
+            {
+                itemId: "cartitem-tester-sandbox-auction-won",
+                productId: "auction-tester-sandbox-won",
+                productTitle: "Test Auction — Already Won",
+                price: 15000,
+                currency: "INR",
+                quantity: 1,
+                storeId: "store-tester-qa-seller",
+                storeName: "Tester QA Seller",
+                listingType: "auction",
+                // Both, deliberately — see the lane note in the header.
+                isAuctionWin: true,
+                bidId: "bid-tester-sandbox-won-tester-qa",
+                // A win is non-removable and fixed-quantity: settlement is the only writer
+                // of this line, and declining to pay is handled by the forfeit sweep rather
+                // than by the buyer deleting it.
+                locked: true,
+                lockedPrice: 15000,
+                addedAt: hoursAgo(2),
+                updatedAt: hoursAgo(2),
+            },
+        ],
+        isTestData: true,
+        testDataExpiresAt: testDataExpiresAt(),
+        createdAt: hoursAgo(2),
+        updatedAt: hoursAgo(2),
+    },
+];

package/dist/features/tester/seed-data/index.d.ts

@@ -6,5 +6,6 @@ export { blogTesterSeedData } from "./blog-tester-seed-data";
 export { eventsTesterSeedData } from "./events-tester-seed-data";
 export { couponsTesterSeedData } from "./coupons-tester-seed-data";
 export { bidsTesterSeedData } from "./bids-tester-seed-data";
+export { cartsTesterSeedData } from "./carts-tester-seed-data";
 export { ordersTesterSeedData } from "./orders-tester-seed-data";
 export { offersTesterSeedData } from "./offers-tester-seed-data";

package/dist/features/tester/seed-data/index.js

@@ -6,5 +6,6 @@ export { blogTesterSeedData } from "./blog-tester-seed-data";
 export { eventsTesterSeedData } from "./events-tester-seed-data";
 export { couponsTesterSeedData } from "./coupons-tester-seed-data";
 export { bidsTesterSeedData } from "./bids-tester-seed-data";
+export { cartsTesterSeedData } from "./carts-tester-seed-data";
 export { ordersTesterSeedData } from "./orders-tester-seed-data";
 export { offersTesterSeedData } from "./offers-tester-seed-data";

package/dist/seed/cart-seed-data.js

@@ -76,6 +76,17 @@ export const cartsSeedData = [
                 storeId: "store-beyblade-arena",
                 storeName: "Beyblade Arena",
                 listingType: "auction",
+                /*
+                 * 🛑 `isAuctionWin` is what puts this line in the AUCTION lane.
+                 *
+                 * `laneOf()` keys on `isAuctionWin || bidId` — `listingType: "auction"`
+                 * and `locked: true` mean nothing to it. Without this flag the line fell
+                 * into the STANDARD lane, so the cart's "Won Auctions" tab was empty
+                 * while a non-removable auction line sat in the ordinary cart inflating
+                 * its total, and the lane gate never blocked anything. Found by the
+                 * tester 2026-09-06.
+                 */
+                isAuctionWin: true,
                 locked: true,
                 addedAt: daysAgo(1),
                 updatedAt: daysAgo(1),

package/dist/seed/index.d.ts

@@ -78,7 +78,7 @@ export { historySeedData } from "./history-seed-data";
 export { groupedListingsSeedData } from "./grouped-listings-seed-data";
 export { scammersSeedData } from "./scammers-seed-data";
 export { supportTicketsSeedData } from "./support-tickets-seed-data";
-export { testerChecklistSeedData, storesTesterSeedData, categoriesTesterSeedData, productsTesterSeedData, blogTesterSeedData, eventsTesterSeedData, couponsTesterSeedData, bidsTesterSeedData, ordersTesterSeedData, } from "../features/tester/seed-data";
+export { testerChecklistSeedData, storesTesterSeedData, categoriesTesterSeedData, productsTesterSeedData, blogTesterSeedData, eventsTesterSeedData, couponsTesterSeedData, bidsTesterSeedData, cartsTesterSeedData, ordersTesterSeedData, } from "../features/tester/seed-data";
 export { productFeaturesSeedData } from "./product-features-seed-data";
 export { offersSeedData } from "./offers-seed-data";
 export { payoutMethodsSeedData, shippingConfigsSeedData, analyticsCardsSeedData, analyticsAlertsSeedData, storeCategoriesSeedData, listingTemplatesSeedData, moderationQueueSeedData, reportsSeedData, itemRequestsSeedData, storeWhatsAppConfigSeedData, storeGoogleConfigSeedData, } from "./store-extensions-seed-data";

package/dist/seed/index.js

@@ -69,7 +69,7 @@ export { scammersSeedData } from "./scammers-seed-data";
 export { supportTicketsSeedData } from "./support-tickets-seed-data";
 // Tester sandbox seed fixtures live in ../features/tester/seed-data (isolated from this
 // folder on purpose) — re-exported here so every other seed consumer keeps one import surface.
-export { testerChecklistSeedData, storesTesterSeedData, categoriesTesterSeedData, productsTesterSeedData, blogTesterSeedData, eventsTesterSeedData, couponsTesterSeedData, bidsTesterSeedData, ordersTesterSeedData, } from "../features/tester/seed-data";
+export { testerChecklistSeedData, storesTesterSeedData, categoriesTesterSeedData, productsTesterSeedData, blogTesterSeedData, eventsTesterSeedData, couponsTesterSeedData, bidsTesterSeedData, cartsTesterSeedData, ordersTesterSeedData, } from "../features/tester/seed-data";
 export { productFeaturesSeedData } from "./product-features-seed-data";
 export { offersSeedData } from "./offers-seed-data";
 // S-STORE Extensions (Tier S-STORE) — 11 new collections

package/dist/seed/manifest.js

@@ -50,7 +50,7 @@ import { offersSeedData } from "./offers-seed-data";
 import { couponUsageSeedData } from "./coupon-usage-seed-data";
 import { claimedCouponsSeedData } from "./claimed-coupons-seed-data";
 // Tester sandbox seed fixtures live in ../features/tester/seed-data (isolated on purpose).
-import { testerChecklistSeedData, storesTesterSeedData, categoriesTesterSeedData, productsTesterSeedData, blogTesterSeedData, eventsTesterSeedData, couponsTesterSeedData, bidsTesterSeedData, ordersTesterSeedData, offersTesterSeedData, } from "../features/tester/seed-data";
+import { testerChecklistSeedData, storesTesterSeedData, categoriesTesterSeedData, productsTesterSeedData, blogTesterSeedData, eventsTesterSeedData, couponsTesterSeedData, bidsTesterSeedData, cartsTesterSeedData, ordersTesterSeedData, offersTesterSeedData, } from "../features/tester/seed-data";
 function asArr(items) {
     return items ?? [];
 }
@@ -166,7 +166,7 @@ export const SEED_MANIFEST = {
             name: a.label ?? a.fullName ?? a.id,
         })),
     ]),
-    carts: pick(asArr(cartsSeedData).map((c) => ({
+    carts: pick([...asArr(cartsSeedData), ...asArr(cartsTesterSeedData)].map((c) => ({
         ...c,
         name: c.userId ?? c.id,
     }))),

package/dist/seed/notifications-seed-data.js

@@ -403,7 +403,11 @@ const notificationRecords = [
         title: "Your auction ended with no winner",
         message: "The highest bid did not meet your reserve price. The listing has been archived.",
         isRead: false,
-        relatedId: "auction-beyblade-lord-spryzen-ended-unsold",
+        // 🛑 The real id carries `burst` — see products-auctions-seed-data.ts. Without
+        // it this notification's "view details" link resolves to nothing, which is the
+        // dangling-FK shape Root Cause #26 describes: the row renders perfectly and its
+        // only affordance goes nowhere.
+        relatedId: "auction-beyblade-burst-lord-spryzen-ended-unsold",
         relatedType: "product",
         createdAt: daysAgo(1),
         updatedAt: daysAgo(1),

package/package.json

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