@mohasinac/appkit 4.65.0 → 4.67.0

package/dist/_internal/shared/features/orders/schema.d.ts

@@ -6,12 +6,12 @@ export declare const createOrderSchema: z.ZodObject<{
         price: z.ZodNumber;
     }, "strip", z.ZodTypeAny, {
         price: number;
-        quantity: number;
         productId: string;
+        quantity: number;
     }, {
         price: number;
-        quantity: number;
         productId: string;
+        quantity: number;
     }>, "many">;
     shippingAddress: z.ZodEffects<z.ZodObject<{
         fullName: z.ZodString;
@@ -75,8 +75,8 @@ export declare const createOrderSchema: z.ZodObject<{
     };
     items: {
         price: number;
-        quantity: number;
         productId: string;
+        quantity: number;
     }[];
     couponCode?: string | undefined;
 }, {
@@ -93,8 +93,8 @@ export declare const createOrderSchema: z.ZodObject<{
     };
     items: {
         price: number;
-        quantity: number;
         productId: string;
+        quantity: number;
     }[];
     couponCode?: string | undefined;
 }>;
@@ -105,16 +105,16 @@ export declare const updateOrderStatusSchema: z.ZodObject<{
     carrier: z.ZodOptional<z.ZodString>;
     note: z.ZodOptional<z.ZodString>;
 }, "strip", z.ZodTypeAny, {
-    status: "delivered" | "shipped" | "processing" | "return_requested" | "cancelled" | "refunded";
+    status: "cancelled" | "processing" | "shipped" | "delivered" | "return_requested" | "refunded";
     orderId: string;
-    note?: string | undefined;
     trackingNumber?: string | undefined;
+    note?: string | undefined;
     carrier?: string | undefined;
 }, {
-    status: "delivered" | "shipped" | "processing" | "return_requested" | "cancelled" | "refunded";
+    status: "cancelled" | "processing" | "shipped" | "delivered" | "return_requested" | "refunded";
     orderId: string;
-    note?: string | undefined;
     trackingNumber?: string | undefined;
+    note?: string | undefined;
     carrier?: string | undefined;
 }>;
 export declare const cancelOrderSchema: z.ZodObject<{

package/dist/_internal/shared/features/products/schema.d.ts

@@ -88,10 +88,10 @@ export declare const setStatusSchema: z.ZodObject<{
     productId: z.ZodString;
     status: z.ZodEnum<["draft", "published", "archived"]>;
 }, "strip", z.ZodTypeAny, {
-    status: "published" | "draft" | "archived";
+    status: "draft" | "published" | "archived";
     productId: string;
 }, {
-    status: "published" | "draft" | "archived";
+    status: "draft" | "published" | "archived";
     productId: string;
 }>;
 export type ProductInput = z.infer<typeof productInputSchema>;

package/dist/_internal/shared/features/promotions/schema.d.ts

@@ -33,8 +33,8 @@ export declare const createCouponSchema: z.ZodObject<{
         maxDiscount?: number | undefined;
     }, {
         value: number;
-        minPurchase?: number | undefined;
         maxDiscount?: number | undefined;
+        minPurchase?: number | undefined;
     }>;
     bxgy: z.ZodOptional<z.ZodObject<{
         buyQuantity: z.ZodNumber;
@@ -93,9 +93,9 @@ export declare const createCouponSchema: z.ZodObject<{
     }>>;
 }, "strip", z.ZodTypeAny, {
     code: string;
-    type: "fixed" | "percentage" | "free_shipping" | "buy_x_get_y";
-    scope: "admin" | "seller";
+    type: "percentage" | "fixed" | "free_shipping" | "buy_x_get_y";
     name: string;
+    scope: "seller" | "admin";
     discount: {
         value: number;
         minPurchase: number;
@@ -124,13 +124,13 @@ export declare const createCouponSchema: z.ZodObject<{
     } | undefined;
 }, {
     code: string;
-    type: "fixed" | "percentage" | "free_shipping" | "buy_x_get_y";
-    scope: "admin" | "seller";
+    type: "percentage" | "fixed" | "free_shipping" | "buy_x_get_y";
     name: string;
+    scope: "seller" | "admin";
     discount: {
         value: number;
-        minPurchase?: number | undefined;
         maxDiscount?: number | undefined;
+        minPurchase?: number | undefined;
     };
     usage: {
         totalLimit?: number | undefined;
@@ -169,8 +169,8 @@ export declare const updateCouponSchema: z.ZodObject<Omit<{
         maxDiscount?: number | undefined;
     }, {
         value: number;
-        minPurchase?: number | undefined;
         maxDiscount?: number | undefined;
+        minPurchase?: number | undefined;
     }>>;
     bxgy: z.ZodOptional<z.ZodOptional<z.ZodObject<{
         buyQuantity: z.ZodNumber;
@@ -228,9 +228,9 @@ export declare const updateCouponSchema: z.ZodObject<Omit<{
         applicableCategories?: string[] | undefined;
     }>>>;
 }, "code">, "strip", z.ZodTypeAny, {
-    type?: "fixed" | "percentage" | "free_shipping" | "buy_x_get_y" | undefined;
-    scope?: "admin" | "seller" | undefined;
+    type?: "percentage" | "fixed" | "free_shipping" | "buy_x_get_y" | undefined;
     name?: string | undefined;
+    scope?: "seller" | "admin" | undefined;
     discount?: {
         value: number;
         minPurchase: number;
@@ -258,13 +258,13 @@ export declare const updateCouponSchema: z.ZodObject<Omit<{
         applicableCategories?: string[] | undefined;
     } | undefined;
 }, {
-    type?: "fixed" | "percentage" | "free_shipping" | "buy_x_get_y" | undefined;
-    scope?: "admin" | "seller" | undefined;
+    type?: "percentage" | "fixed" | "free_shipping" | "buy_x_get_y" | undefined;
     name?: string | undefined;
+    scope?: "seller" | "admin" | undefined;
     discount?: {
         value: number;
-        minPurchase?: number | undefined;
         maxDiscount?: number | undefined;
+        minPurchase?: number | undefined;
     } | undefined;
     bxgy?: {
         buyQuantity: number;

package/dist/_internal/shared/features/reviews/schema.d.ts

@@ -9,17 +9,17 @@ export declare const createReviewSchema: z.ZodObject<{
     orderId: z.ZodOptional<z.ZodString>;
 }, "strip", z.ZodTypeAny, {
     title: string;
-    storeId: string;
-    rating: number;
     images: string[];
+    storeId: string;
     productId: string;
+    rating: number;
     comment: string;
     orderId?: string | undefined;
 }, {
     title: string;
     storeId: string;
-    rating: number;
     productId: string;
+    rating: number;
     comment: string;
     images?: string[] | undefined;
     orderId?: string | undefined;

package/dist/_internal/shared/features/site-settings/types.d.ts

@@ -117,4 +117,21 @@ export interface PublicSiteSettings {
     disabledRoutes: string[];
     /** Derived, not a stored field — the resolved marker → wordmark → text chain. */
     effectiveWatermark?: MediaVideoWatermark;
+    /**
+     * Which public browse surfaces currently have something in them. Derived
+     * from the `surfaceCounts/global` document, not a `siteSettings` field —
+     * carried here because this payload is already the channel the navbar,
+     * footer and sidebar read their visibility rules from, so adding a second
+     * client fetch for one more rule would be a request per page load.
+     *
+     * 🛑 OMITTED ENTIRELY when the counts document has not been written yet,
+     * and every reader must treat that as "show everything". A present-but-empty
+     * object would be indistinguishable from "measured, and all zero" — which is
+     * the shape that hides the whole catalogue. `isListingSurfaceVisible`
+     * encodes the rule; do not re-derive it at a call site.
+     */
+    surfaceCounts?: {
+        listingTypes?: Record<string, number>;
+        surfaces?: Record<string, number>;
+    };
 }

package/dist/_internal/shared/listing-types/feature-flags.d.ts

@@ -97,5 +97,50 @@ export declare const ALL_LISTING_TYPES: ListingType[];
 export declare function isListingType(value: unknown): value is ListingType;
 /** Pull the full list of enabled listing types in canonical iteration order. */
 export declare function enabledListingTypes(settings: FeatureFlagSnapshot | null | undefined): ListingType[];
+/**
+ * The measured counts, as the public site-settings payload carries them.
+ *
+ * Structurally typed rather than importing `SurfaceCountsDocument` so this
+ * module stays a pure constants file with no feature import — the same reason
+ * it imports only `ROUTES`.
+ */
+export interface SurfaceCountSnapshot {
+    listingTypes?: Partial<Record<ListingType, number>>;
+    surfaces?: Partial<Record<string, number>>;
+}
+/**
+ * Should this listing type's public surface be shown at all?
+ *
+ * 🛑 THE PRECEDENCE, and why it is not just `count > 0`:
+ *
+ * | admin flag | measured count | result | because |
+ * |---|---|---|---|
+ * | `false`    | anything   | **hidden** | "we do not sell live animals" is a standing catalogue decision, not a statement about today's stock. An admin who closed a type must not see it reopen because someone listed one. |
+ * | `true`     | `0`        | **hidden** | auto-hide: the thing this function exists for. |
+ * | `true`     | `> 0`      | shown | |
+ * | `true`     | **absent** | **shown** | 🛑 unknown is NOT zero. A fresh install, a failed job, or a type added since the last recompute must render exactly as it does today, never as hidden. Root Cause #72. |
+ *
+ * The flag therefore stays the authority and the count only ever SUBTRACTS
+ * from what the flag already permits. Two mechanisms, one direction — which is
+ * what stops them becoming two switches that disagree.
+ */
+export declare function isListingSurfaceVisible(type: ListingType, settings: FeatureFlagSnapshot | null | undefined, counts: SurfaceCountSnapshot | null | undefined): boolean;
+/**
+ * The same question for a surface with no listing-type flag behind it
+ * (`/reviews` today).
+ *
+ * No admin-flag half: these have no switch to consult, so emptiness is the
+ * only input. Absent still means shown.
+ */
+export declare function isCountedSurfaceVisible(surface: string, counts: SurfaceCountSnapshot | null | undefined): boolean;
+/**
+ * Visible listing types — the flag list, minus the ones measured empty.
+ *
+ * This is what nav, footer and the sitemap filter on. `enabledListingTypes`
+ * above is deliberately left alone: it answers "what does this marketplace
+ * sell", which `/api/products`' post-filter and the seller forms still need
+ * even for a type with zero rows today.
+ */
+export declare function visibleListingTypes(settings: FeatureFlagSnapshot | null | undefined, counts: SurfaceCountSnapshot | null | undefined): ListingType[];
 export declare function enabledCategoryTypes(settings: FeatureFlagSnapshot | null | undefined): CategoryType[];
 export {};

package/dist/_internal/shared/listing-types/feature-flags.js

@@ -140,6 +140,57 @@ export function isListingType(value) {
 export function enabledListingTypes(settings) {
     return ALL_LISTING_TYPES.filter((t) => isListingTypeEnabled(t, settings));
 }
+/**
+ * Should this listing type's public surface be shown at all?
+ *
+ * 🛑 THE PRECEDENCE, and why it is not just `count > 0`:
+ *
+ * | admin flag | measured count | result | because |
+ * |---|---|---|---|
+ * | `false`    | anything   | **hidden** | "we do not sell live animals" is a standing catalogue decision, not a statement about today's stock. An admin who closed a type must not see it reopen because someone listed one. |
+ * | `true`     | `0`        | **hidden** | auto-hide: the thing this function exists for. |
+ * | `true`     | `> 0`      | shown | |
+ * | `true`     | **absent** | **shown** | 🛑 unknown is NOT zero. A fresh install, a failed job, or a type added since the last recompute must render exactly as it does today, never as hidden. Root Cause #72. |
+ *
+ * The flag therefore stays the authority and the count only ever SUBTRACTS
+ * from what the flag already permits. Two mechanisms, one direction — which is
+ * what stops them becoming two switches that disagree.
+ */
+export function isListingSurfaceVisible(type, settings, counts) {
+    if (!isListingTypeEnabled(type, settings))
+        return false;
+    const n = counts?.listingTypes?.[type];
+    // `undefined` -> true. Written as an explicit test rather than `(n ?? 1) > 0`
+    // so the unknown case is readable, and so nobody "simplifies" it into a
+    // falsy check that folds 0 and undefined together — which is the bug.
+    if (n === undefined)
+        return true;
+    return n > 0;
+}
+/**
+ * The same question for a surface with no listing-type flag behind it
+ * (`/reviews` today).
+ *
+ * No admin-flag half: these have no switch to consult, so emptiness is the
+ * only input. Absent still means shown.
+ */
+export function isCountedSurfaceVisible(surface, counts) {
+    const n = counts?.surfaces?.[surface];
+    if (n === undefined)
+        return true;
+    return n > 0;
+}
+/**
+ * Visible listing types — the flag list, minus the ones measured empty.
+ *
+ * This is what nav, footer and the sitemap filter on. `enabledListingTypes`
+ * above is deliberately left alone: it answers "what does this marketplace
+ * sell", which `/api/products`' post-filter and the seller forms still need
+ * even for a type with zero rows today.
+ */
+export function visibleListingTypes(settings, counts) {
+    return ALL_LISTING_TYPES.filter((t) => isListingSurfaceVisible(t, settings, counts));
+}
 export function enabledCategoryTypes(settings) {
     const all = ["category", "sublisting", "brand", "bundle"];
     return all.filter((t) => isCategoryTypeEnabled(t, settings));