@mohasinac/appkit 4.55.0 → 4.56.0

package/dist/features/seller/actions/seller-actions.js

@@ -10,6 +10,7 @@
  */
 import { serverLogger } from "../../../monitoring/index.js";
 import { normalizeError } from "../../../errors/normalize.js";
+import { reconcileOrderShipping } from "../../../_internal/server/features/shipping/reconcile.js";
 import { enqueueJob } from "../../jobs/actions/enqueue-job.js";
 import { isAdminUser, isStrictSellerUser, isTesterUser } from "../../auth/role-predicates.js";
 import { ApiError, AuthorizationError, NotFoundError, ValidationError, } from "../../../errors/index.js";
@@ -712,6 +713,38 @@ export async function customShipOrder(userId, userRole, orderId, input) {
         shippingDate: new Date(),
         payoutStatus: "eligible",
     });
+    /*
+     * 🛑 AFTER the order is marked shipped, and never allowed to fail the
+     * dispatch.
+     *
+     * The parcel has physically gone. If reconciliation throws — the points
+     * ledger is unreachable, the order read fails — the correct outcome is a
+     * shipped order that has not been reconciled yet, which is recoverable by
+     * re-running it. Letting it propagate would leave the seller unable to mark
+     * a parcel shipped because of an accounting step they have no control over,
+     * and they would retry the dispatch, which is the one thing that must not
+     * happen twice.
+     *
+     * `reconcileOrderShipping` is itself idempotent on
+     * `order.shippingReconciliation`, so a retry is safe.
+     */
+    if (typeof input.actualShippingCost === "number") {
+        try {
+            const outcome = await reconcileOrderShipping(orderId, {
+                courierId: input.actualCourierName ?? input.shippingCarrier,
+                courierName: input.actualCourierName ?? input.shippingCarrier,
+                actualRate: input.actualShippingCost,
+            });
+            serverLogger.info("shipping reconciled", { orderId, ...outcome });
+        }
+        catch (err) {
+            const normalized = normalizeError(err);
+            serverLogger.error("shipping reconciliation failed after dispatch", {
+                orderId,
+                message: normalized.message,
+            });
+        }
+    }
     serverLogger.info("customShipOrder", {
         orderId,
         userId,

package/dist/index.d.ts

@@ -3123,6 +3123,7 @@ export { withRetry } from "./http/retry";
 export { normalizeError, getErrorMessage, isApiNormalized, isAppNormalized, isFirebaseAuthNormalized, isFirebaseFirestoreNormalized, isFirebaseStorageNormalized, isNativeNormalized, isNetworkNormalized, isUnknownNormalized, isZodNormalized, } from "./errors/normalize";
 export type { NormalizedError, NormalizedApiError, NormalizedAppError, NormalizedFirebaseAuthError, NormalizedFirebaseFirestoreError, NormalizedFirebaseStorageError, NormalizedNativeError, NormalizedNetworkError, NormalizedUnknownThrownValue, NormalizedZodError, } from "./errors/normalize";
 export { CART_LANE, CART_LANE_PRIORITY, CART_LANE_LABELS, laneOf, activeLane, laneItems, laneCounts, isLaneCheckoutable, laneBlockReason, isLockedLane, canAddNewItems, type CartLane, type LaneAssignable, } from "./_internal/shared/checkout/lanes";
+export { roundShippingUp, toQuoteOptions, reconcileShipping, SHIPPING_ROUND_UP_TO, type ShippingQuoteOption, type SelectedShippingQuote, type ActualShipmentCost, type ShippingReconciliation, } from "./_internal/shared/shipping/quote";
 export { newsletterSubscribeSchema, quickCreateTaxonomySchema, payoutMarkPaidSchema, } from "./features/admin/schemas/small-forms";
 export { raffleEntrySchema } from "./features/contact/schemas/inquiry-forms";
 export { orderCancelSchema, paymentProofSchema, } from "./features/orders/schemas/buyer-forms";

package/dist/index.js

@@ -5559,6 +5559,14 @@ export { normalizeError, getErrorMessage, isApiNormalized, isAppNormalized, isFi
 // Checkout lanes — the derived auction > offer > standard partition of the
 // cart, and the priority rule that decides which one may be checked out.
 export { CART_LANE, CART_LANE_PRIORITY, CART_LANE_LABELS, laneOf, activeLane, laneItems, laneCounts, isLaneCheckoutable, laneBlockReason, isLockedLane, canAddNewItems, } from "./_internal/shared/checkout/lanes.js";
+/*
+ * Shipping quote model. Public because the CONSUMER owns the route that asks
+ * the carrier for rates (`/api/shipping/quote`) and the checkout UI that
+ * renders them — the rounding rule has to be reachable from both, and it must
+ * be the same rule the server-side reconciliation uses, or the buyer is
+ * credited against a figure they were never charged.
+ */
+export { roundShippingUp, toQuoteOptions, reconcileShipping, SHIPPING_ROUND_UP_TO, } from "./_internal/shared/shipping/quote.js";
 export { newsletterSubscribeSchema, quickCreateTaxonomySchema, payoutMarkPaidSchema, } from "./features/admin/schemas/small-forms.js";
 export { raffleEntrySchema } from "./features/contact/schemas/inquiry-forms.js";
 export { orderCancelSchema, paymentProofSchema, } from "./features/orders/schemas/buyer-forms.js";

package/dist/seed/carousel-slides-seed-data.js

@@ -1,8 +1,15 @@
 /*
  * WHY: Seeds 6 carousel slides for the marketplace homepage promo carousel.
- * WHAT: 5 active + 1 inactive. Deterministic Picsum backgrounds with a dim
+ * WHAT: 4 active + 2 inactive. Deterministic Picsum backgrounds with a dim
  * overlay, plus a headline / supporting line / CTA on each.
  *
+ * 🛑 FOUR ACTIVE, NOT FIVE, SINCE 2026-10-20. `MAX_ACTIVE_SLIDES` is a
+ * ceiling, not a quota — `slide-beyblade-auction` went inactive with
+ * `listings.listingTypes.auction: false` and nothing was promoted to backfill
+ * it. Do NOT activate `slide-upcoming-promo` to get back to five: that one is
+ * the fixture `getActiveSlides()`' own filter is tested against, and it says
+ * "A draft slide kept unpublished so admins have a starting point to copy."
+ *
  * 🛑 EVERY ACTIVE SLIDE MUST CARRY AN `overlay` OR `cards`, OR IT RENDERS BLANK.
  * `HeroCarousel` gates all slide copy on `slide.overlay` (line ~412) and the
  * card grid on `slide.cards.length` (line ~451). Before this pass every seeded
@@ -107,16 +114,34 @@ const _rawSlides = [
                 hover: { effect: "scale" },
             },
             {
-                id: "card-hero-bid",
+                /*
+                 * 🛑 THIS WAS THE AUCTIONS CARD ("Bid / Live auctions / Place a bid
+                 * before the timer runs out" → `/auctions`), and it was still on the
+                 * first screen of the site weeks after `listings.listingTypes.auction`
+                 * went `false`. The section-level gate in `MarketplaceHomepageView` is
+                 * keyed on `SectionType` and a carousel is type `carousel` whatever it
+                 * contains, so nothing could see it. `gateCarouselSlides` now can — but
+                 * a filter that leaves a hole in a `repeat(3, 1fr)` row is a fallback,
+                 * not a fix, so the slot gets real content instead.
+                 *
+                 * 🛑 NOT `/spares` and NOT `/art`, which were the obvious swaps: both
+                 * types are ENABLED, both have a working browse page, and both held
+                 * ZERO published products when this was written (measured against
+                 * production, 2026-10-20). Replacing a dead-end card with a different
+                 * dead-end card is not a fix. `/categories` is populated — six roots,
+                 * 28 brands, 8 franchises — and browsing the tree is genuinely the
+                 * third way to shop here.
+                 */
+                id: "card-hero-browse",
                 zone: 2,
                 mobileZone: 2,
                 background: { type: "image", url: MEDIA.lDrago, dimOverlay: { enabled: true, opacity: 0.45 } },
                 content: {
-                    eyebrow: "Bid",
-                    title: "Live auctions",
-                    description: "Place a bid before the timer runs out.",
+                    eyebrow: "Explore",
+                    title: "Shop by category",
+                    description: "Walk the tree from generation down to a single part.",
                 },
-                buttons: [{ id: "card-hero-bid-cta", text: "Browse auctions", href: String(ROUTES.PUBLIC.AUCTIONS), variant: "primary" }],
+                buttons: [{ id: "card-hero-browse-cta", text: "Browse categories", href: String(ROUTES.PUBLIC.CATEGORIES), variant: "primary" }],
                 hover: { effect: "scale" },
             },
             {
@@ -139,10 +164,26 @@ const _rawSlides = [
         updatedAt: daysAgo(3),
     },
     {
+        /*
+         * 🛑 DISABLED, NOT DELETED — the same arrangement the auctions RAIL uses in
+         * `homepage-sections-seed-data.ts`, and for the same reason. Every word of
+         * the copy, the video background and the CTA stay exactly as they were, so
+         * flipping `listings.listingTypes.auction` back to `true` and this flag
+         * with it restores the slide intact.
+         *
+         * Two locks, deliberately. `gateCarouselSlides` already drops this at
+         * render because `overlay.button.link` and `link.url` both resolve to
+         * `/auctions` — but that one is invisible, and a slide listed as active in
+         * the carousel admin while never appearing on the page reads as a bug in
+         * the admin. `active: false` is the lock a human sees.
+         *
+         * 🛑 The copy names no listing type — "Win it at your price" — which is
+         * exactly why the gate matches on the LINK and not on the words.
+         */
         id: "slide-beyblade-auction",
         title: "Live auctions",
         order: 2,
-        active: true,
+        active: false,
         background: {
             type: "video",
             // Direct MP4 URL, deliberately NOT wrapped in seedExtMedia() — that