@mohasinac/appkit 4.64.1 → 4.65.0

package/dist/features/blog/api/route.js

@@ -13,14 +13,7 @@ import { BLOG_DEFAULT_PUBLIC_SORT } from "../schemas/firestore.js";
 import { NextResponse } from "next/server.js";
 import { getProviders } from "../../../contracts/index.js";
 import { normalizeError } from "../../../errors/normalize.js";
-function param(url, key) {
-    return url.searchParams.get(key);
-}
-function numParam(url, key, fallback) {
-    const v = url.searchParams.get(key);
-    const n = v !== null ? Number(v) : NaN;
-    return Number.isFinite(n) ? n : fallback;
-}
+import { numParam, param } from "../../../utils/search-params.js";
 // --- GET /api/blog ------------------------------------------------------------
 export async function GET(request) {
     try {

package/dist/features/blog/components/BlogIndexPageView.d.ts

@@ -1,7 +1,6 @@
 import React from "react";
-type SearchParams = Record<string, string | string[]>;
+import type { SearchParamsRecord } from "../../../utils/search-params";
 export interface BlogIndexPageViewProps {
-    searchParams?: SearchParams;
+    searchParams?: SearchParamsRecord;
 }
 export declare function BlogIndexPageView({ searchParams }: BlogIndexPageViewProps): Promise<React.JSX.Element>;
-export {};

package/dist/features/blog/components/BlogIndexPageView.js

@@ -6,15 +6,12 @@ import { BlogIndexListing } from "./BlogIndexListing.js";
 import { BLOG_DEFAULT_PUBLIC_SORT } from "../schemas/firestore.js";
 import { safeRead } from "../../../errors/safe-read.js";
 import { hidePublicTestData } from "../../../_internal/server/features/tester/visibility.js";
-function sp(params, key) {
-    const v = params[key];
-    return Array.isArray(v) ? v[0] ?? "" : v ?? "";
-}
+import { firstParam } from "../../../utils/search-params.js";
 export async function BlogIndexPageView({ searchParams = {} }) {
-    const sort = sp(searchParams, "sort") || BLOG_DEFAULT_PUBLIC_SORT;
-    const page = Number(sp(searchParams, "page")) || 1;
-    const pageSize = Number(sp(searchParams, "pageSize")) || 24;
-    const category = sp(searchParams, "category");
+    const sort = firstParam(searchParams, "sort") || BLOG_DEFAULT_PUBLIC_SORT;
+    const page = Number(firstParam(searchParams, "page")) || 1;
+    const pageSize = Number(firstParam(searchParams, "pageSize")) || 24;
+    const category = firstParam(searchParams, "category");
     /*
      * 🛑 `q` MUST be read and forwarded. Without it /blog?q=anything rendered the
      * FULL published list — the same failure the API route's own header records
@@ -23,7 +20,7 @@ export async function BlogIndexPageView({ searchParams = {} }) {
      * `initialData`, and the listing hook sets `staleTime: Infinity` when given
      * SSR data, so the unfiltered first paint is frozen for that query key.
      */
-    const q = sp(searchParams, "q").trim();
+    const q = firstParam(searchParams, "q").trim();
     const sieveResult = await safeRead(() => blogRepository.listPublished({ ...(category ? { category } : {}), ...(q ? { search: q } : {}) }, { sorts: sort, page, pageSize }), { route: "/blog", key: "blogPosts.listPublished", fallback: null });
     // Sandbox posts are public rows by design — not on someone else's first paint.
     const publicSieve = sieveResult

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

@@ -11,7 +11,7 @@ import { productRepository } from "../../products/repository/products.repository
 import { storeRepository } from "../../stores/repository/store.repository.js";
 import { normalizeListingType } from "../../products/utils/listing-type.js";
 import { getDefaultCurrency } from "../../../core/baseline-resolver.js";
-import { CART_LANE, activeLane, canAddNewItems, } from "../../../_internal/shared/checkout/lanes.js";
+import { CART_LANE, activeLane, canAddNewItems } from "../../../_internal/shared/checkout/lanes.js";
 import { CART_MAX_ITEMS } from "../../../constants/limits.js";
 import { getExpandedDecrements, validateCartItemStock, } from "../../../_internal/server/features/checkout/bundle-expansion.js";
 import { safeRead } from "../../../errors/safe-read.js";

package/dist/features/cart/api/route.js

@@ -11,11 +11,7 @@
 import { NextResponse } from "next/server.js";
 import { getProviders } from "../../../contracts/index.js";
 import { normalizeError } from "../../../errors/normalize.js";
-function numParam(url, key, fallback) {
-    const v = url.searchParams.get(key);
-    const n = v !== null ? Number(v) : NaN;
-    return Number.isFinite(n) ? n : fallback;
-}
+import { numParam } from "../../../utils/search-params.js";
 export async function GET(request) {
     try {
         const url = new URL(request.url);

package/dist/features/cart/utils/guest-cart.js

@@ -1,5 +1,6 @@
 import { normalizeError } from "../../../errors/normalize.js";
 import { CART_MAX_ITEMS } from "../../../constants/limits.js";
+import { getDefaultStorage } from "../../../utils/browser-storage.js";
 const DEFAULT_GUEST_CART_KEY = process.env.NEXT_PUBLIC_APP_ID
     ? `${process.env.NEXT_PUBLIC_APP_ID}_guest_cart`
     : "guest_cart";
@@ -24,11 +25,6 @@ export class CartFullError extends Error {
         this.current = current;
     }
 }
-function getDefaultStorage() {
-    if (typeof window === "undefined")
-        return null;
-    return window.localStorage;
-}
 function readItems(storage, key) {
     if (!storage)
         return [];

package/dist/features/catalogue/repository/catalogue.repository.js

@@ -2,7 +2,7 @@
  * Personal Catalogue Repository (Feature B).
  */
 import { BaseRepository, prepareForFirestore, parseSieveDateValue, } from "../../../providers/db-firebase/index.js";
-import { CATALOGUE_COLLECTION, CATALOGUE_FIELDS, CATALOGUE_TRACKED_FIELDS, CATALOGUE_HISTORY_PII_FIELDS, createCatalogueItemId, } from "../schemas/firestore.js";
+import { CATALOGUE_COLLECTION, CATALOGUE_FIELDS, CATALOGUE_TRACKED_FIELDS, CATALOGUE_HISTORY_PII_FIELDS, createCatalogueItemId } from "../schemas/firestore.js";
 import { withHistory } from "../../../_internal/shared/history/index.js";
 export class CatalogueRepository extends BaseRepository {
     /** Decrypt on read so callers see the shape they wrote. */

package/dist/features/categories/api/route.js

@@ -23,6 +23,8 @@ import { isListingCategory } from "../constants/listing-categories.js";
 import { planSearchTxt } from "../../../utils/search-txt-query.js";
 import { matchesAllSearchTerms } from "../../../utils/search-txt.js";
 import { normalizeError } from "../../../errors/normalize.js";
+import { intParamOrNull, param } from "../../../utils/search-params.js";
+import { validateSieveFilters } from "../../../utils/sieve-builder.js";
 const CACHE_CONTROL_PUBLIC = "public, max-age=3600, s-maxage=3600, stale-while-revalidate=86400";
 /**
  * Overlay the LIVE rolled-up count onto a projected page of categories.
@@ -80,30 +82,10 @@ function buildTreeFromFlat(items, rootId) {
     }
     return roots.sort((a, b) => (a.order ?? 0) - (b.order ?? 0)).map(nest);
 }
-function param(url, key) {
-    return url.searchParams.get(key);
-}
-function numParam(url, key) {
-    const v = url.searchParams.get(key);
-    if (v === null)
-        return null;
-    const n = Number.parseInt(v, 10);
-    return Number.isFinite(n) ? n : null;
-}
 const SAFE_CATEGORY_FILTER_FIELDS = new Set([
     "categoryType", "parentIds", "isFeatured", "showOnHomepage",
     "tier", "isActive", "isLeaf", "isBrand", "isSearchable",
 ]);
-function validateSieveFilters(raw, allowedFields) {
-    return raw
-        .split(",")
-        .map((c) => c.trim())
-        .filter((c) => {
-        const m = c.match(/^([^<>=!@]+)\s*(?:==|!=|<=|>=|<|>|@=\*?)/);
-        return m ? allowedFields.has(m[1].trim()) : false;
-    })
-        .join(",");
-}
 // --- GET /api/categories ------------------------------------------------------
 export async function GET(request) {
     try {
@@ -118,8 +100,8 @@ export async function GET(request) {
         const flat = param(url, "flat");
         const tree = param(url, "tree");
         const rawFilters = param(url, "filters");
-        const tier = numParam(url, "tier");
-        const pageSizeParam = numParam(url, "pageSize");
+        const tier = intParamOrNull(url, "tier");
+        const pageSizeParam = intParamOrNull(url, "pageSize");
         /*
          * 🛑 `?page=` MEANS "give me a paged flat list", and reading it fixes two
          * live bugs rather than adding a feature.
@@ -139,7 +121,7 @@ export async function GET(request) {
          *     were permanently empty. Same shape as the `useCategoryTree` bug:
          *     `isFiltered` did not include the params real callers send.
          */
-        const pageParam = numParam(url, "page");
+        const pageParam = intParamOrNull(url, "page");
         const isPaged = pageParam !== null && pageParam > 0;
         /*
          * `?listingOnly=true` drops brand / bundle / sublisting rows.

package/dist/features/categories/components/BundlesListView.d.ts

@@ -1,7 +1,7 @@
 import React from "react";
-type SearchParams = Record<string, string | string[]>;
+import type { SearchParamsRecord } from "../../../utils/search-params";
 export interface BundlesListViewProps {
-    searchParams?: SearchParams;
+    searchParams?: SearchParamsRecord;
     onBuyNow?: (input: {
         bundleSlug: string;
         quantity: number;
@@ -15,4 +15,3 @@ export interface BundlesListViewProps {
  * Sort UI is owned by `CategoryBundlesListing` (client).
  */
 export declare function BundlesListView({ searchParams, onBuyNow, }: BundlesListViewProps): Promise<React.JSX.Element>;
-export {};

package/dist/features/categories/components/BundlesListView.js

@@ -5,11 +5,8 @@ import { AdSlot } from "../../homepage/components/AdSlot.js";
 import { CategoryBundlesListing } from "./CategoryBundlesListing.js";
 import { safeRead } from "../../../errors/safe-read.js";
 import { hidePublicTestData } from "../../../_internal/server/features/tester/visibility.js";
+import { firstParam } from "../../../utils/search-params.js";
 const DEFAULT_LIMIT = 200;
-function sp(params, key) {
-    const v = params[key];
-    return Array.isArray(v) ? v[0] ?? "" : v ?? "";
-}
 /**
  * Public `/bundles` index. Bundles live as `categoryType:"bundle"` rows on
  * the categories collection (SB-UNI-D + V), so this fetches via
@@ -18,7 +15,7 @@ function sp(params, key) {
  * Sort UI is owned by `CategoryBundlesListing` (client).
  */
 export async function BundlesListView({ searchParams = {}, onBuyNow, }) {
-    const storeId = sp(searchParams, "storeId");
+    const storeId = firstParam(searchParams, "storeId");
     const all = await safeRead(() => categoriesRepository.listByType("bundle", {
         activeOnly: true,
         limit: DEFAULT_LIMIT,

package/dist/features/categories/components/CategoriesIndexListing.js

@@ -20,7 +20,6 @@ const __O = {
 const PAGE_SIZE = 24;
 function renderCategoryGrid(props) {
     const { isLoading, categories, view, activeSearch, activeTab, brandsOnly } = props;
-    const isBrandView = activeTab === "brands" || brandsOnly;
     /* The empty state names the tab the visitor is actually on. It said
      * "categories" on all five, so an empty Parts tab reported the wrong thing. */
     const noun = brandsOnly
@@ -155,12 +154,21 @@ export function CategoriesIndexListing({ brandsOnly = false }) {
     const switchTab = (key) => {
         table.set(TABLE_KEYS.TAB, key);
     };
-    const { categories, totalPages, isLoading } = useCategoriesFiltered({
+    const { categories, allRows, totalPages, isLoading } = useCategoriesFiltered({
         q: table.get(TABLE_KEYS.QUERY) || undefined,
         isFeatured: table.get(TABLE_KEYS.IS_FEATURED) === "true" || undefined,
         isBrand: isBrandParam,
-        // The tab partition — applied before pagination inside the hook.
-        predicate: (c) => matchesCategoryTab(c, activeTab),
+        /*
+         * The tab partition — applied before pagination inside the hook.
+         *
+         * 🛑 `useCallback` on `activeTab` is REQUIRED, not hygiene. The hook
+         * memoises its filter/sort/slice on this reference; an inline arrow is a
+         * new reference every render, so it would recompute on each one. (The
+         * converse was the actual bug: the predicate was missing from the hook's
+         * dependency list entirely, so a tab change recomputed NOTHING and the
+         * previous tab's rows stayed on screen.)
+         */
+        predicate: useCallback((c) => matchesCategoryTab(c, activeTab), [activeTab]),
         rootOnly: table.get("rootOnly") === "true" || undefined,
         tier: getNumericParam(table, "tier"),
         minProductCount: getNumericParam(table, "minItemCount"),
@@ -186,7 +194,41 @@ export function CategoriesIndexListing({ brandsOnly = false }) {
      * navigate. That is also why the items carry `onSelect` rather than `href`,
      * and therefore why `linkProps`/`scroll` do not apply here.
      */
-    const tabItems = CATEGORY_TABS.map((tab) => ({
+    /*
+     * 🛑 A TAB THAT CAN ONLY EVER BE EMPTY IS THE SAME DEAD FACET AS AN EMPTY CARD.
+     *
+     * This index already removed its Parts tab for being "the worse half of two
+     * better homes", and `category-tabs.ts` records that *"every tab answers a
+     * question; 'all' answered none"*. A tab whose answer is always "nothing"
+     * answers none either — it is a labelled promise of content that opens on
+     * "No characters found".
+     *
+     * It became load-bearing the moment 320 character rows were seeded `isActive:
+     * false`: the type is real, the rows exist and resolve, and NOT ONE is live
+     * until a product carries a character slug. Shipping the tab regardless
+     * would have put a permanently empty section in the site's main taxonomy
+     * index — and then quietly filled itself the day someone flipped a flag,
+     * with no second commit needed. That is the behaviour worth having, so the
+     * tab is DERIVED from the payload rather than from a list.
+     *
+     * 🛑 Counted from `allRows` — the whole cached response — never from
+     * `categories`, which is this tab's own paginated slice and would make every
+     * inactive tab hide itself. Root Cause #72 is the trap here ("a 'hide it when
+     * it's empty' rule is only as good as its count map"), and the reason this
+     * version is safe is that there IS no map: both the count and the render
+     * partition one payload through the same `matchesCategoryTab`, so a tab
+     * cannot be uncovered by construction.
+     *
+     * Two rows that must not be collapsed into the count:
+     *   - WHILE LOADING, `allRows` is empty, so every tab would vanish and
+     *     reappear. The bar renders in full until the payload lands.
+     *   - THE ACTIVE TAB IS NEVER HIDDEN. A direct link to `?tab=characters`
+     *     must not leave the visitor on a bar with no selected tab; it shows the
+     *     tab and its honest empty state.
+     */
+    const tabItems = CATEGORY_TABS.filter((tab) => isLoading ||
+        tab.key === activeTab ||
+        allRows.some((c) => matchesCategoryTab(c, tab.key))).map((tab) => ({
         id: tab.key,
         label: tab.label,
         onSelect: () => switchTab(tab.key),

package/dist/features/categories/components/CategoriesIndexPageView.d.ts

@@ -1,11 +1,11 @@
 import React from "react";
-type SearchParams = Record<string, string | string[]>;
+import type { SearchParamsRecord } from "../../../utils/search-params";
 export interface CategoriesIndexPageViewProps {
     /**
      * @deprecated Accepted and ignored. Kept so the consumer page shim compiles
      * either way; remove once no caller passes it.
      */
-    searchParams?: SearchParams;
+    searchParams?: SearchParamsRecord;
 }
 /**
  * The categories index.
@@ -30,4 +30,3 @@ export interface CategoriesIndexPageViewProps {
  * lose (contrast `/products`, where the grid IS in the SSR HTML and must stay).
  */
 export declare function CategoriesIndexPageView(_props?: CategoriesIndexPageViewProps): Promise<React.JSX.Element>;
-export {};

package/dist/features/categories/components/CategoriesIndexPageView.js

@@ -1,39 +1,7 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { sieveFilter, SIEVE_OP } from "@mohasinac/appkit";
 import { Container, Heading, Main, Section } from "../../../ui/index.js";
 import { AdSlot } from "../../homepage/components/AdSlot.js";
 import { CategoriesIndexListing } from "./CategoriesIndexListing.js";
-function sp(params, key) {
-    const v = params[key];
-    return Array.isArray(v) ? v[0] ?? "" : v ?? "";
-}
-function buildCategoryFilters(params) {
-    const parts = [];
-    const isFeatured = sp(params, "isFeatured");
-    if (isFeatured === "true")
-        parts.push("isFeatured==true");
-    const isBrand = sp(params, "isBrand");
-    if (isBrand === "true")
-        parts.push("isBrand==true");
-    const rootOnly = sp(params, "rootOnly");
-    if (rootOnly === "true")
-        parts.push("tier==0");
-    const minItemCount = sp(params, "minItemCount");
-    const maxItemCount = sp(params, "maxItemCount");
-    if (minItemCount)
-        parts.push(sieveFilter("metrics.totalItemCount", SIEVE_OP.GTE, minItemCount));
-    if (maxItemCount)
-        parts.push(sieveFilter("metrics.totalItemCount", SIEVE_OP.LTE, maxItemCount));
-    const tier = sp(params, "tier");
-    if (tier) {
-        const values = tier.split("|").filter(Boolean);
-        if (values.length === 1)
-            parts.push(sieveFilter("tier", SIEVE_OP.EQ, values[0]));
-        else if (values.length > 1)
-            parts.push(sieveFilter("tier", SIEVE_OP.EQ, values.join("|")));
-    }
-    return parts.join(",");
-}
 /**
  * The categories index.
  *

package/dist/features/categories/constants/category-tabs.d.ts

@@ -39,7 +39,7 @@
  * @tag pattern:shared-literal
  * @tag sideEffects:none
  */
-export type CategoryTabKey = "categories" | "brands" | "franchises" | "bundles";
+export type CategoryTabKey = "categories" | "brands" | "franchises" | "bundles" | "characters";
 export interface CategoryTabDef {
     key: CategoryTabKey;
     label: string;

package/dist/features/categories/constants/category-tabs.js

@@ -48,6 +48,11 @@ export const CATEGORY_TABS = [
     { key: "categories", label: "Categories" },
     { key: "brands", label: "Brands" },
     { key: "franchises", label: "Franchises" },
+    /*
+     * Who USED the bey. Answers a question none of the others can and the one
+     * buyers actually ask — "Tyson's Dragoon", not "Attack Ring MA-04".
+     */
+    { key: "characters", label: "Characters" },
     { key: "bundles", label: "Bundles" },
 ];
 export const DEFAULT_CATEGORY_TAB = "categories";
@@ -66,6 +71,8 @@ export function matchesCategoryTab(c, tab) {
             return type === "brand";
         case "franchises":
             return type === "franchise";
+        case "characters":
+            return type === "character";
         case "bundles":
             return type === "bundle";
         case "categories":

package/dist/features/categories/hooks/useCategories.d.ts

@@ -32,12 +32,18 @@ export interface CategoriesFilteredParams {
      * Not part of the react-query key, deliberately — the FETCH is the same
      * unfiltered `?flat=true` for every tab, so keying on it would mint five
      * cache entries for one response.
+     *
+     * 🛑 That applies to the QUERY KEY ONLY. It must still be a dependency of the
+     * in-memory filter/sort memo, and omitting it there was a live bug: a tab
+     * change altered nothing else the memo watched, so it returned the previous
+     * tab's rows. Caller supplies it via `useCallback` on the tab key.
      */
     predicate?: (c: CategoryItem) => boolean;
 }
 /** API-driven for structured filters; text search + pagination done in hook (Firestore substring limitation). */
 export declare function useCategoriesFiltered(params?: CategoriesFilteredParams): {
     categories: CategoryItem[];
+    allRows: CategoryItem[];
     total: number;
     totalPages: number;
     isLoading: boolean;

package/dist/features/categories/hooks/useCategories.js

@@ -38,8 +38,57 @@ export function useCategoriesFiltered(params = {}) {
         staleTime: 2 * 60 * 1000,
     });
     const { page = 1, pageSize = 24, sort = "name", q = "", minProductCount, maxProductCount } = params;
+    /*
+     * Every LIVE row in the payload — before any tab partition, search or
+     * pagination. Exposed because a caller that partitions this response into
+     * tabs needs to know whether a tab has anything in it at all, and its own
+     * `categories` slice cannot tell it that (it is one tab's page).
+     *
+     * 🛑 Deliberately NOT narrowed by `q`. A tab bar that loses tabs as the
+     * visitor types is worse than one that keeps a tab whose search found
+     * nothing — the second has an honest empty state, the first makes the
+     * section look deleted.
+     */
+    const allRows = useMemo(() => {
+        const items = query.data ?? [];
+        /*
+         * 🛑 AN INACTIVE ROW IS A DEAD FACET, AND UNTIL 2026-10-24 NOTHING ENFORCED IT.
+         *
+         * `brand-rows.ts` trap 3 states the rule — *"a row with no product is a
+         * DEAD FACET, and worse than absent: it renders as a card in the grid that
+         * opens an empty page"* — and `franchise-rows.ts` claims its seven inactive
+         * rows are therefore *"excluded from the grid and from search"*.
+         *
+         * **They were not.** `GET /api/categories` builds its Sieve string from
+         * `type` / `parentIds` / `isFeatured` / `isBrand` / `showOnHomepage` /
+         * `tier` only, and `useCategoriesFiltered` sends nothing but `flat=true`,
+         * so no layer ever added an `isActive` clause. All seven inactive
+         * franchises have been rendering on the Franchises tab, each one a card
+         * opening an empty page. The promise was in a comment; the mechanism did
+         * not exist (Root Cause #106).
+         *
+         * It surfaced when 320 inactive CHARACTER rows were seeded on the strength
+         * of that promise — seven dead cards is easy to miss, 320 is not.
+         *
+         * 🛑 IN MEMORY, AND ONLY ON AN EXPLICIT `false`. Two reasons, both of which
+         * a server-side clause would get wrong:
+         *
+         *   - A Firestore inequality EXCLUDES every document that lacks the field.
+         *     `isActive` is optional on `CategoryDocument`, so pushing this down
+         *     would hide every row that never set it — which is most of the real
+         *     taxonomy. That is the documented trap in CLAUDE.md's TS-patterns
+         *     table, in the one shape that returns a plausible non-empty list.
+         *   - The same reason `isActive?: boolean` is declared on `CategoryItem` at
+         *     all: absent must mean "live", never "unknown".
+         *
+         * ADMIN IS NOT AFFECTED. `/admin/categories` reads this route through
+         * `useAdminListingData`, not this hook, and must keep seeing inactive rows
+         * — they are what it offers a Status column to flip.
+         */
+        return items.filter((c) => c.isActive !== false);
+    }, [query.data]);
     const paginated = useMemo(() => {
-        let items = query.data ?? [];
+        let items = allRows;
         const lq = q.toLowerCase();
         if (lq)
             items = items.filter((c) => c.name.toLowerCase().includes(lq) || (c.description ?? "").toLowerCase().includes(lq));
@@ -83,9 +132,31 @@ export function useCategoriesFiltered(params = {}) {
         const totalPages = Math.max(1, Math.ceil(total / pageSize));
         const slice = items.slice((page - 1) * pageSize, page * pageSize);
         return { items: slice, total, totalPages };
-    }, [query.data, q, sort, page, pageSize, minProductCount, maxProductCount]);
+        /*
+         * 🛑 `params.predicate` BELONGS IN THIS LIST, and leaving it out was a live
+         * bug on the categories index — not a theoretical lint complaint.
+         *
+         * The predicate IS the tab partition. `switchTab` writes only the `tab` URL
+         * param, so on a tab change `q`, `sort`, `page` and `pageSize` are all
+         * unchanged, and the request string is identical for every tab except
+         * brands (the one that sends `isBrand=true`) — so react-query hands back
+         * the SAME `query.data` reference. With the predicate absent from the deps,
+         * nothing in them changed, the memo returned its cached slice, and
+         * **switching Categories → Franchises → Bundles re-rendered the previous
+         * tab's rows.** Only the brands tab worked, by accident, because its query
+         * key differs.
+         *
+         * It is a changing reference by nature, so the call site must `useCallback`
+         * it on the tab key — `CategoriesIndexListing` now does. Were it ever
+         * passed inline again the memo would recompute every render, which is
+         * merely wasteful over ~1,500 in-memory rows and still CORRECT; the
+         * previous behaviour was neither.
+         */
+    }, [allRows, q, sort, page, pageSize, minProductCount, maxProductCount, params.predicate, params.isBrand]);
     return {
         categories: paginated.items,
+        /* Every live row, unpartitioned — see `allRows` above. */
+        allRows,
         total: paginated.total,
         totalPages: paginated.totalPages,
         isLoading: query.isLoading,

package/dist/features/categories/types/index.d.ts

@@ -1,4 +1,4 @@
-export type CategoryType = "category" | "sublisting" | "brand" | "bundle" | "franchise";
+export type CategoryType = "category" | "sublisting" | "brand" | "bundle" | "franchise" | "character";
 export interface CategorySeo {
     title?: string;
     description?: string;

package/dist/features/classified/components/ClassifiedDetailPageView.js

@@ -7,7 +7,7 @@ const CLS_CLASSIFIED_BADGE = "inline-block rounded-full bg-cyan-100 dark:bg-cyan
 import { ROUTES } from "../../../next/routing/route-map.js";
 import { getDefaultCurrency } from "../../../core/baseline-resolver.js";
 import { formatCurrency } from "../../../utils/number.formatter.js";
-import { normalizeRichTextHtml } from "../../../utils/string.formatter.js";
+import { toDescriptionHtml } from "../../../utils/string.formatter.js";
 import { safeDisplayName } from "../../../security/index.js";
 import { Container, Div, GatedPrice, Heading, Main, Nav, RichText, Row, Section, Span, Stack, Text, SpecTable, } from "../../../ui/index.js";
 import { PreOrderDetailView } from "../../products/components/PreOrderDetailView.js";
@@ -25,12 +25,6 @@ import { safeRead } from "../../../errors/safe-read.js";
 import { DEFAULT_MIN_OFFER_PERCENT, offersEnabledFor, resolveOfferBounds, } from "../../../_internal/shared/features/offers/config.js";
 import { ListingBottomActions } from "../../products/components/ListingBottomActions.js";
 import { productImageUrls } from "../../products/schemas/firestore.js";
-function toDescriptionHtml(raw) {
-    if (!raw)
-        return "";
-    const s = typeof raw === "string" ? raw : JSON.stringify(raw);
-    return normalizeRichTextHtml(s);
-}
 export async function ClassifiedDetailPageView({ slug, initialProduct, renderOfferAction }) {
     const product = initialProduct !== undefined
         ? (initialProduct ?? undefined)
@@ -65,7 +59,7 @@ export async function ClassifiedDetailPageView({ slug, initialProduct, renderOff
     const storeHref = storeSlug ? String(ROUTES.PUBLIC.STORE_DETAIL(storeSlug)) : null;
     const specs = Array.isArray(p.specifications) ? p.specifications : [];
     const descriptionHtml = toDescriptionHtml(p.description);
-    const [{ relatedItems, relatedByBrand, relatedByTags, relatedByStore }, groups, siteSettings] = await Promise.all([
+    const [{ relatedItems, relatedByBrand, relatedByTags, relatedByStore }, groups] = await Promise.all([
         computeRelatedItems(product),
         getGroupsWithItemsForProduct(product.id),
         safeRead(() => getSiteSettingsGlobal(), {

package/dist/features/collections/api/route.js

@@ -11,11 +11,7 @@
 import { NextResponse } from "next/server.js";
 import { getProviders } from "../../../contracts/index.js";
 import { normalizeError } from "../../../errors/normalize.js";
-function numParam(url, key, fallback) {
-    const v = url.searchParams.get(key);
-    const n = v !== null ? Number(v) : NaN;
-    return Number.isFinite(n) ? n : fallback;
-}
+import { numParam } from "../../../utils/search-params.js";
 export async function GET(request) {
     try {
         const url = new URL(request.url);

package/dist/features/consultation/api/route.js

@@ -11,11 +11,7 @@
 import { NextResponse } from "next/server.js";
 import { getProviders } from "../../../contracts/index.js";
 import { normalizeError } from "../../../errors/normalize.js";
-function numParam(url, key, fallback) {
-    const v = url.searchParams.get(key);
-    const n = v !== null ? Number(v) : NaN;
-    return Number.isFinite(n) ? n : fallback;
-}
+import { numParam } from "../../../utils/search-params.js";
 export async function GET(request) {
     try {
         const url = new URL(request.url);

package/dist/features/contact/email.js

@@ -18,17 +18,14 @@ import { getProviders } from "../../contracts/index.js";
 import { createResendProvider } from "../../providers/email-resend/provider.js";
 import { siteSettingsRepository } from "../admin/repository/site-settings.repository.js";
 import { serverLogger } from "../../monitoring/index.js";
-import { currentYear, formatCurrency, } from "../../utils/index.js";
-import { EmailBold, EmailButton, EmailContainer, EmailDivider, EmailDoc, EmailFooter, EmailHeader, EmailRow, } from "../email/primitives.js";
+import { currentYear, formatCurrency } from "../../utils/index.js";
+import { EmailBold, EmailButton, EmailContainer, EmailDivider, EmailDoc, EmailFooter, EmailHeader, EmailRow } from "../email/primitives.js";
 function getSiteName() {
     return process.env.NEXT_PUBLIC_SITE_NAME?.trim() || "App";
 }
 function getSiteUrl() {
     return process.env.NEXTAUTH_URL?.trim() || "http://localhost:3000";
 }
-function getSupportEmail() {
-    return process.env.EMAIL_SUPPORT?.trim() || "support@example.com";
-}
 /**
  * Resolve an email provider that works in BOTH runtimes.
  *

package/dist/features/corporate/api/route.js

@@ -11,11 +11,7 @@
 import { NextResponse } from "next/server.js";
 import { getProviders } from "../../../contracts/index.js";
 import { normalizeError } from "../../../errors/normalize.js";
-function numParam(url, key, fallback) {
-    const v = url.searchParams.get(key);
-    const n = v !== null ? Number(v) : NaN;
-    return Number.isFinite(n) ? n : fallback;
-}
+import { numParam } from "../../../utils/search-params.js";
 export async function GET(request) {
     try {
         const url = new URL(request.url);

package/dist/features/digital-codes/components/DigitalCodeDetailPageView.js

@@ -7,7 +7,7 @@ const CLS_DIGITALCODE_BADGE = "inline-block rounded-full bg-violet-100 dark:bg-v
 import { ROUTES } from "../../../next/routing/route-map.js";
 import { getDefaultCurrency } from "../../../core/baseline-resolver.js";
 import { formatCurrency } from "../../../utils/number.formatter.js";
-import { normalizeRichTextHtml } from "../../../utils/string.formatter.js";
+import { toDescriptionHtml } from "../../../utils/string.formatter.js";
 import { safeDisplayName } from "../../../security/index.js";
 import { Div, Container, GatedPrice, Heading, Main, Nav, RichText, Row, Section, Span, Stack, Text, SpecTable, } from "../../../ui/index.js";
 import { PreOrderDetailView } from "../../products/components/PreOrderDetailView.js";
@@ -24,12 +24,6 @@ import { ReviewsListingPanel } from "../../reviews/components/ReviewsListingPane
 import { ListingBottomActions } from "../../products/components/ListingBottomActions.js";
 import { productImageUrls } from "../../products/schemas/firestore.js";
 import { PurchaseReviewAffordance } from "../../reviews/components/PurchaseReviewAffordance.js";
-function toDescriptionHtml(raw) {
-    if (!raw)
-        return "";
-    const s = typeof raw === "string" ? raw : JSON.stringify(raw);
-    return normalizeRichTextHtml(s);
-}
 export async function DigitalCodeDetailPageView({ slug, initialProduct, renderPrimaryActions }) {
     const product = initialProduct !== undefined
         ? (initialProduct ?? undefined)

package/dist/features/events/components/AdminEventEditorView.js

@@ -503,7 +503,7 @@ export function AdminEventEditorView({ eventId, onSaved, embedded, allowedTypes,
     // Replaces the hand-rolled fieldToStepIndex + goToStep pair. The hook owns
     // the expand-then-scroll ordering a raw setState cannot: a collapsed panel
     // must MOUNT before scrollIntoView and focus can reach the offending field.
-    const { openIds, setOpenIds, goToSection, fieldToSectionIndex, sectionMeta } = useSectionFormNav(sections, draft);
+    const { openIds, setOpenIds } = useSectionFormNav(sections, draft);
     const formContent = (_jsxs(Stack, { gap: "sm", children: [alertSection, _jsxs(FormShellContext.Provider, { value: shellCtx, children: [_jsx(FormErrorSummary, {}), _jsx(SectionForm, { sections: sections, values: draft, onChange: update, onSubmit: () => { saveMutation.mutate(); }, schema: eventDraftSchema, openIds: openIds, onOpenChange: setOpenIds, submitLabel: isEdit ? "Save Changes" : "Create Event", isLoading: isLoading })] }), saveMessage && (_jsx(Alert, { variant: saveMessage.toLowerCase().includes("fail") || saveMessage.toLowerCase().includes("error") ? "error" : "success", title: "Save status", children: saveMessage }))] }));
     if (embedded) {
         return _jsx(Div, { className: `${__O.yAuto} ${__P.p4}`, children: formContent });