@mohasinac/appkit 4.46.0 → 4.48.0

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

@@ -6,7 +6,7 @@ import { increment } from "../../../contracts/field-ops.js";
 import { withHistory } from "../../../_internal/shared/history/index.js";
 import { SCAMMER_TRACKED_FIELDS, SCAMMER_HISTORY_PII_FIELDS } from "../schemas/firestore.js";
 import { buildScammerSearchTxt } from "../../../utils/search-txt-builders.js";
-import { planSearchTxt, refineSearchTxt, emptySearchResult, } from "../../../utils/search-txt-query.js";
+import { planSearchTxt, beginSearchWindow, emptySearchResult, } from "../../../utils/search-txt-query.js";
 import { SCAMMER_COLLECTION, SCAMMER_FIELDS, SCAMMER_INCIDENTS_SUBCOLLECTION, SCAMMER_COMMENTS_SUBCOLLECTION, DEFAULT_SCAMMER_DATA, } from "../schemas/firestore.js";
 class ScammerRepository extends BaseRepository {
     constructor() {
@@ -35,12 +35,19 @@ class ScammerRepository extends BaseRepository {
         if (plan.head) {
             baseQuery = baseQuery.where(SCAMMER_FIELDS.SEARCH_TXT, "array-contains", plan.head);
         }
-        const result = await this.sieveQuery(model, ScammerRepository.SIEVE_FIELDS, {
+        /*
+         * One bounded window, one raised `maxPageSize`, one page slice — see
+         * `beginSearchWindow`. Hand-writing the three is how the other ten
+         * repositories each ended up with two of them, which reads as a working
+         * multi-word search and silently cannot reach page 2.
+         */
+        const win = beginSearchWindow(model, plan);
+        const result = await this.sieveQuery(win.model, ScammerRepository.SIEVE_FIELDS, {
             baseQuery,
             defaultPageSize: 20,
-            maxPageSize: 50,
+            maxPageSize: win.maxPageSize(50),
         });
-        return refineSearchTxt(result, plan.rest);
+        return win.finish(result);
     }
     /** Derived on every write path via `applyWriteHooks`. */
     buildSearchTxtFor(data) {
@@ -56,12 +63,19 @@ class ScammerRepository extends BaseRepository {
         if (plan.head) {
             baseQuery = baseQuery.where(SCAMMER_FIELDS.SEARCH_TXT, "array-contains", plan.head);
         }
-        const result = await this.sieveQuery(model, ScammerRepository.SIEVE_FIELDS, {
+        /*
+         * One bounded window, one raised `maxPageSize`, one page slice — see
+         * `beginSearchWindow`. Hand-writing the three is how the other ten
+         * repositories each ended up with two of them, which reads as a working
+         * multi-word search and silently cannot reach page 2.
+         */
+        const win = beginSearchWindow(model, plan);
+        const result = await this.sieveQuery(win.model, ScammerRepository.SIEVE_FIELDS, {
             baseQuery,
             defaultPageSize: 50,
-            maxPageSize: 200,
+            maxPageSize: win.maxPageSize(200),
         });
-        return refineSearchTxt(result, plan.rest);
+        return win.finish(result);
     }
     async findBySeoSlug(seoSlug) {
         try {

package/dist/features/seller/components/CategoryInlineSelect.d.ts

@@ -6,5 +6,21 @@ export interface CategoryInlineSelectProps {
     disabled?: boolean;
     /** Show the "+ Create new category" option. Defaults to true — sellers can create categories inline. */
     allowCreate?: boolean;
+    /**
+     * I2 — the ADDITIONAL assignments, beyond `value`.
+     *
+     * 🛑 Omitting this pair keeps the component exactly single-select, which is
+     * what every existing call site gets. Multi-assignment is opt-in per surface
+     * rather than switched on globally, because `value`/`onChange` is a `string`
+     * contract that five call sites rely on and changing it under them is Root
+     * Cause #20 (a public prop change without its consumers in the same commit).
+     */
+    extraValues?: string[];
+    onExtraValuesChange?: (ids: string[]) => void;
+    /**
+     * How many assignments are allowed in total, `value` included. Defaults to
+     * `MAX_CATEGORY_ASSIGNMENTS`, which is what the repository refuses above.
+     */
+    maxAssignments?: number;
 }
-export declare function CategoryInlineSelect({ value, onChange, placeholder, disabled, allowCreate, }: CategoryInlineSelectProps): React.JSX.Element;
+export declare function CategoryInlineSelect({ value, onChange, placeholder, disabled, allowCreate, extraValues, onExtraValuesChange, maxAssignments, }: CategoryInlineSelectProps): React.JSX.Element;

package/dist/features/seller/components/CategoryInlineSelect.js

@@ -1,9 +1,12 @@
 "use client";
-import { jsx as _jsx } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { PaginatedSelect } from "../../../ui/components/PaginatedSelect.js";
 import { CategoryQuickCreateForm } from "../../admin/components/CategoryQuickCreateForm.js";
 import { apiClient } from "../../../http/index.js";
 import { ADMIN_ENDPOINTS, CATEGORY_ENDPOINTS } from "../../../constants/api-endpoints.js";
+import { Stack } from "../../../ui/components/Layout.js";
+import { Text } from "../../../ui/components/Typography.js";
+import { MAX_CATEGORY_ASSIGNMENTS } from "../../products/repository/category-assignment.js";
 // Admin: paginated + server-side search via /api/admin/categories (admin/mod only).
 // Response via successResponse({ data: items, total }) → apiClient unwraps to { data, total, ... }.
 async function loadAdminCategoryOptions(query, page) {
@@ -35,9 +38,17 @@ async function loadPublicCategoryOptions(query, _page) {
     const items = q ? all.filter((c) => c.label.toLowerCase().includes(q)) : all;
     return { items, hasMore: false, nextPage: 2 };
 }
-export function CategoryInlineSelect({ value, onChange, placeholder = "Search categories…", disabled, allowCreate = true, }) {
+export function CategoryInlineSelect({ value, onChange, placeholder = "Search categories…", disabled, allowCreate = true, extraValues, onExtraValuesChange, maxAssignments = MAX_CATEGORY_ASSIGNMENTS, }) {
     const loadOptions = allowCreate ? loadAdminCategoryOptions : loadPublicCategoryOptions;
-    return (_jsx(PaginatedSelect, { value: value || null, onChange: (v) => onChange(v ?? ""), loadOptions: loadOptions, placeholder: placeholder, disabled: disabled, createLabel: "category", drawerTitle: "Create Category", renderCreateForm: allowCreate
+    const primary = (_jsx(PaginatedSelect, { value: value || null, onChange: (v) => onChange(v ?? ""), loadOptions: loadOptions, placeholder: placeholder, disabled: disabled, createLabel: "category", drawerTitle: "Create Category", renderCreateForm: allowCreate
             ? ({ onCreated, onCancel }) => (_jsx(CategoryQuickCreateForm, { onSaved: (id, name) => onCreated({ value: id, label: name }), onCancel: onCancel }))
             : undefined }));
+    // Single-select unless the caller opted in — byte-identical to before.
+    if (!onExtraValuesChange)
+        return primary;
+    const extras = extraValues ?? [];
+    const remaining = Math.max(0, maxAssignments - 1 - extras.length);
+    return (_jsxs(Stack, { gap: "sm", children: [primary, _jsx(PaginatedSelect, { multiple: true, value: extras, onChange: (v) => onExtraValuesChange(Array.isArray(v) ? v : []), loadOptions: loadOptions, placeholder: remaining > 0
+                    ? `Also list under… (up to ${remaining} more)`
+                    : "No more categories can be added", disabled: disabled || remaining <= 0 }), _jsx(Text, { size: "xs", color: "muted", children: "Optional. Add a category from a DIFFERENT tree \u2014 a franchise, for example. Two categories from the same tree will be refused, because the broader one is already implied by the narrower." })] }));
 }

package/dist/features/seller/components/SellerProductShell.d.ts

@@ -6,6 +6,16 @@ export interface SellerProductDraft {
     slug?: string;
     description?: string;
     category?: string;
+    /**
+     * I2 — the assignment list, DEFAULT FIRST, with `category` mirroring `[0]`.
+     *
+     * 🛑 Declared on the TYPE as well as the Zod schema. The schema is
+     * `.passthrough()`, so an undeclared key survives at runtime — and stays
+     * untyped, so the picker could not read `values.categoryIds` at all. Both
+     * halves are needed, which is the same trap as a field missing from
+     * `PRODUCT_UPDATABLE_FIELDS` (Root Cause #101) one layer out.
+     */
+    categoryIds?: string[];
     brand?: string;
     condition?: string;
     tags?: string[];
@@ -43,7 +53,7 @@ export interface SellerProductDraft {
     video?: MediaField;
     youtubeId?: string;
     price?: number;
-    compareAtPrice?: number;
+    originalPrice?: number;
     stockQuantity?: number;
     featured?: boolean;
     isPromoted?: boolean;
@@ -134,10 +144,25 @@ export interface SellerProductShellProps {
     onDiscard?: () => void;
     isLoading?: boolean;
     storeSlug?: string;
-    /** Render a custom category selector (e.g. PaginatedSelect with createLabel). */
+    /**
+     * Render a custom category selector (e.g. PaginatedSelect with createLabel).
+     *
+     * I2 — `extraValues`/`onExtraValuesChange` carry the ADDITIONAL assignments.
+     * A selector that ignores them stays exactly single-select, which is what
+     * every surface except this one does.
+     */
     renderCategorySelector?: (props: {
         value: string;
         onChange: (v: string) => void;
+        /**
+         * 🛑 OPTIONAL, and that is the whole point. `QuickProductForm` is a second
+         * consumer of this slot and making these required broke it immediately —
+         * Root Cause #20, a public prop change without its consumers, which the
+         * comment above literally warns about. A selector that ignores them stays
+         * exactly single-select.
+         */
+        extraValues?: string[];
+        onExtraValuesChange?: (ids: string[]) => void;
     }) => React.ReactNode;
     /** Render a custom brand selector. */
     renderBrandSelector?: (props: {

package/dist/features/seller/components/SellerProductShell.js

@@ -46,6 +46,15 @@ const sellerProductSchema = z.object({
     description: z.string().max(10000).optional().or(z.literal("")),
     price: z.number().min(0).optional(),
     category: z.string().optional(),
+    /**
+     * I2 — the assignment list, DEFAULT FIRST. `category` mirrors `[0]`.
+     *
+     * Declared even though this schema is `.passthrough()`, because the draft's
+     * TYPE is what the render site reads `values.categoryIds` off — passthrough
+     * keeps an undeclared key at runtime and still leaves it untyped, so the
+     * picker could not see it.
+     */
+    categoryIds: z.array(z.string()).max(4).optional(),
     brand: z.string().optional(),
     isActive: z.boolean().optional(),
     images: z
@@ -106,7 +115,27 @@ function StepBasic({ values, onChange, renderCategorySelector, renderBrandSelect
         condition: CONDITION_OPTIONS.find((o) => o.value === (values.condition ?? "new"))?.label ?? "",
         siteName: "LetItRip",
     };
-    return (_jsxs(Stack, { gap: "md", children: [_jsx(FormField, { name: "title", label: "Title", type: "text", value: values.title ?? "", onChange: (v) => onChange({ title: v }), placeholder: "e.g. Charizard Base Set PSA 9" }), _jsxs(FormGroup, { columns: 2, children: [renderCategorySelector ? (_jsxs(_Fragment, { children: [_jsx(Text, { className: "text-[var(--appkit-color-text)] mb-1", size: "sm", weight: "medium", children: "Category" }), renderCategorySelector({ value: values.category ?? "", onChange: (v) => onChange({ category: v }) })] })) : (_jsx(FormField, { name: "category", label: "Category", type: "text", value: values.category ?? "", onChange: (v) => onChange({ category: v }), placeholder: "e.g. Trading Cards" })), _jsx(FormField, { name: "condition", label: "Condition", type: "select", value: values.condition ?? "new", onChange: (v) => onChange({ condition: v }), options: CONDITION_OPTIONS })] }), renderTemplateSelector?.({
+    return (_jsxs(Stack, { gap: "md", children: [_jsx(FormField, { name: "title", label: "Title", type: "text", value: values.title ?? "", onChange: (v) => onChange({ title: v }), placeholder: "e.g. Charizard Base Set PSA 9" }), _jsxs(FormGroup, { columns: 2, children: [renderCategorySelector ? (_jsxs(_Fragment, { children: [_jsx(Text, { className: "text-[var(--appkit-color-text)] mb-1", size: "sm", weight: "medium", children: "Category" }), renderCategorySelector({
+                                value: values.category ?? "",
+                                onChange: (v) => onChange({ category: v }),
+                                /*
+                                 * 🛑 `categoryIds[0]` is the DEFAULT assignment and is already
+                                 * rendered by the primary picker above, so the extras list is
+                                 * everything AFTER it. Passing the whole array would show the
+                                 * primary twice and let a seller "remove" it from a control that
+                                 * does not own it.
+                                 */
+                                extraValues: (values.categoryIds ?? []).slice(1),
+                                onExtraValuesChange: (ids) => onChange({
+                                    /*
+                                     * Rebuilt primary-first on every change, because the server
+                                     * derives the default by DEPTH and reorders the stored array
+                                     * anyway — sending it in a different order would make the
+                                     * form and the saved document disagree until the next load.
+                                     */
+                                    categoryIds: [values.category ?? "", ...ids].filter(Boolean),
+                                }),
+                            })] })) : (_jsx(FormField, { name: "category", label: "Category", type: "text", value: values.category ?? "", onChange: (v) => onChange({ category: v }), placeholder: "e.g. Trading Cards" })), _jsx(FormField, { name: "condition", label: "Condition", type: "select", value: values.condition ?? "new", onChange: (v) => onChange({ condition: v }), options: CONDITION_OPTIONS })] }), renderTemplateSelector?.({
                 categoryId: values.category ?? "",
                 condition: values.condition ?? "new",
                 context: templateContext,
@@ -359,7 +388,7 @@ const TYPE_SPECIFIC_SECTIONS = {
 function StepPricing({ values, onChange, listingType, }) {
     const listingPlugin = pluginForMode(listingType);
     const priceLabel = listingPlugin.priceLabel;
-    return (_jsxs(Stack, { gap: "md", children: [_jsx(Alert, { variant: "info", children: "All prices in Indian Rupees (\u20B9)." }), _jsxs(FormGroup, { columns: 2, children: [_jsx(FormField, { name: "price", label: priceLabel, type: "number", value: toRupeesString(values.price), onChange: (v) => onChange({ price: fromRupeesString(v) }), placeholder: "999" }), _jsx(FormField, { name: "compareAtPrice", label: "Compare-at Price (\u20B9, optional)", type: "number", value: toRupeesString(values.compareAtPrice), onChange: (v) => onChange({ compareAtPrice: v ? fromRupeesString(v) : undefined }), placeholder: "1299", helpText: "Original price shown as strikethrough" })] }), listingPlugin.showsStockQuantity && (_jsx(FormField, { name: "stockQuantity", label: "Stock Quantity", type: "number", value: String(values.stockQuantity ?? ""), onChange: (v) => onChange({ stockQuantity: Number(v) }), placeholder: "1" })), _jsxs(FormGroup, { columns: 2, children: [_jsx(Toggle, { checked: !!values.featured, onChange: (checked) => onChange({ featured: checked }), label: "Mark as Featured" }), _jsx(Toggle, { checked: !!values.isNew, onChange: (checked) => onChange({ isNew: checked }), label: "Mark as New Arrival" })] }), _jsx(Toggle, { checked: !!values.allowOffers, onChange: (checked) => onChange({
+    return (_jsxs(Stack, { gap: "md", children: [_jsx(Alert, { variant: "info", children: "All prices in Indian Rupees (\u20B9)." }), _jsxs(FormGroup, { columns: 2, children: [_jsx(FormField, { name: "price", label: priceLabel, type: "number", value: toRupeesString(values.price), onChange: (v) => onChange({ price: fromRupeesString(v) }), placeholder: "999" }), _jsx(FormField, { name: "originalPrice", label: "Compare-at Price (\u20B9, optional)", type: "number", value: toRupeesString(values.originalPrice), onChange: (v) => onChange({ originalPrice: v ? fromRupeesString(v) : undefined }), placeholder: "1299", helpText: "Original price shown as strikethrough" })] }), listingPlugin.showsStockQuantity && (_jsx(FormField, { name: "stockQuantity", label: "Stock Quantity", type: "number", value: String(values.stockQuantity ?? ""), onChange: (v) => onChange({ stockQuantity: Number(v) }), placeholder: "1" })), _jsxs(FormGroup, { columns: 2, children: [_jsx(Toggle, { checked: !!values.featured, onChange: (checked) => onChange({ featured: checked }), label: "Mark as Featured" }), _jsx(Toggle, { checked: !!values.isNew, onChange: (checked) => onChange({ isNew: checked }), label: "Mark as New Arrival" })] }), _jsx(Toggle, { checked: !!values.allowOffers, onChange: (checked) => onChange({
                     allowOffers: checked,
                     minOfferPercent: checked
                         ? values.minOfferPercent ?? DEFAULT_MIN_OFFER_PERCENT
@@ -614,7 +643,7 @@ productId, onSave, onPublish, onDiscard, isLoading = false, storeSlug, renderCat
             required: true,
             quick: true,
             // `gstRate` / `hsnCode` deliberately absent — admin-only, see StepPricing.
-            fields: ["price", "compareAtPrice", "stockQuantity", "allowOffers", "minOfferPercent"],
+            fields: ["price", "originalPrice", "stockQuantity", "allowOffers", "minOfferPercent"],
             render: ({ values, onChange }) => (_jsx(StepPricing, { values: values, onChange: onChange, listingType: listingType })),
             // Built per render because whether stock is required depends on the
             // listing type's own plugin — a digital code has no stock to speak of.

package/dist/features/seller/repository/offer.repository.js

@@ -4,7 +4,7 @@
  * Data access layer for the `offers` Firestore collection.
  * Handles buyer make-an-offer, seller counter/accept/decline lifecycle.
  */
-import { planSearchTxt, refineSearchTxt, emptySearchResult, } from "../../../utils/search-txt-query.js";
+import { planSearchTxt, beginSearchWindow, emptySearchResult, } from "../../../utils/search-txt-query.js";
 import { buildOfferSearchTxt } from "../../../utils/search-txt-builders.js";
 import { BaseRepository, prepareForFirestore, parseSieveDateValue, } from "../../../providers/db-firebase/index.js";
 import { OFFER_COLLECTION, OFFER_FIELDS, OFFER_TRACKED_FIELDS, createOfferId } from "../schemas/index.js";
@@ -43,12 +43,19 @@ class OfferRepository extends BaseRepository {
         if (plan.head) {
             baseQuery = baseQuery.where("searchTxt", "array-contains", plan.head);
         }
-        const result = await this.sieveQuery(model, OfferRepository.SIEVE_FIELDS, {
+        /*
+         * One bounded window, one raised `maxPageSize`, one page slice — see
+         * `beginSearchWindow`. Hand-writing the three is how the other ten
+         * repositories each ended up with two of them, which reads as a working
+         * multi-word search and silently cannot reach page 2.
+         */
+        const win = beginSearchWindow(model, plan);
+        const result = await this.sieveQuery(win.model, OfferRepository.SIEVE_FIELDS, {
             baseQuery,
             defaultPageSize: 50,
-            maxPageSize: 200,
+            maxPageSize: win.maxPageSize(200),
         });
-        return refineSearchTxt(result, plan.rest);
+        return win.finish(result);
     }
     // --- Create --------------------------------------------------------------
     async create(input) {
@@ -113,10 +120,18 @@ class OfferRepository extends BaseRepository {
             if (plan.head) {
                 baseQuery = baseQuery.where(OFFER_FIELDS.SEARCH_TXT, "array-contains", plan.head);
             }
-            const result = await this.sieveQuery(model, OfferRepository.SIEVE_FIELDS, {
+            /*
+             * One bounded window, one raised `maxPageSize`, one page slice — see
+             * `beginSearchWindow`. Hand-writing the three is how the other ten
+             * repositories each ended up with two of them, which reads as a working
+             * multi-word search and silently cannot reach page 2.
+             */
+            const win = beginSearchWindow(model, plan);
+            const result = await this.sieveQuery(win.model, OfferRepository.SIEVE_FIELDS, {
                 baseQuery,
+                maxPageSize: win.maxPageSize(100),
             });
-            return refineSearchTxt(result, plan.rest);
+            return win.finish(result);
         }
         const snap = await this.getCollection()
             .where(OFFER_FIELDS.STORE_ID, "==", storeId)

package/dist/features/seller/schemas/product-form.d.ts

@@ -65,7 +65,7 @@ export declare const productDraftSchema: z.ZodObject<{
     youtubeId: z.ZodOptional<z.ZodString>;
     externalVideoUrl: z.ZodOptional<z.ZodString>;
     price: z.ZodOptional<z.ZodNumber>;
-    compareAtPrice: z.ZodOptional<z.ZodNumber>;
+    originalPrice: z.ZodOptional<z.ZodNumber>;
     stockQuantity: z.ZodOptional<z.ZodNumber>;
     featured: z.ZodOptional<z.ZodBoolean>;
     isPromoted: z.ZodOptional<z.ZodBoolean>;
@@ -155,7 +155,7 @@ export declare const productDraftSchema: z.ZodObject<{
     youtubeId: z.ZodOptional<z.ZodString>;
     externalVideoUrl: z.ZodOptional<z.ZodString>;
     price: z.ZodOptional<z.ZodNumber>;
-    compareAtPrice: z.ZodOptional<z.ZodNumber>;
+    originalPrice: z.ZodOptional<z.ZodNumber>;
     stockQuantity: z.ZodOptional<z.ZodNumber>;
     featured: z.ZodOptional<z.ZodBoolean>;
     isPromoted: z.ZodOptional<z.ZodBoolean>;
@@ -245,7 +245,7 @@ export declare const productDraftSchema: z.ZodObject<{
     youtubeId: z.ZodOptional<z.ZodString>;
     externalVideoUrl: z.ZodOptional<z.ZodString>;
     price: z.ZodOptional<z.ZodNumber>;
-    compareAtPrice: z.ZodOptional<z.ZodNumber>;
+    originalPrice: z.ZodOptional<z.ZodNumber>;
     stockQuantity: z.ZodOptional<z.ZodNumber>;
     featured: z.ZodOptional<z.ZodBoolean>;
     isPromoted: z.ZodOptional<z.ZodBoolean>;

package/dist/features/seller/schemas/product-form.js

@@ -64,7 +64,7 @@ export const productDraftSchema = z
     externalVideoUrl: z.string().optional(),
     // Pricing
     price: z.number().optional(),
-    compareAtPrice: z.number().optional(),
+    originalPrice: z.number().optional(),
     stockQuantity: z.number().optional(),
     featured: z.boolean().optional(),
     isPromoted: z.boolean().optional(),

package/dist/features/server-errors/repository/server-errors.repository.js

@@ -39,6 +39,17 @@ export class ServerErrorsRepository extends BaseRepository {
                 requestId: input.requestId,
                 ...(input.userAgent ? { userAgent: input.userAgent } : {}),
                 ...(input.bodyDigest ? { bodyDigest: input.bodyDigest } : {}),
+                /*
+                 * 🛑 D10, and the G2 sweep is what found it: this hand-picked
+                 * field list is exactly Root Cause #38's shape. `entityType` and
+                 * `entityId` were on `ServerErrorDocument` AND accepted by this
+                 * method's own `Omit<…>` input type — so a caller passing them
+                 * type-checked clean and the fields died right here, on the one
+                 * line nobody re-reads. A write that succeeds is not a field that
+                 * persists.
+                 */
+                ...(input.entityType ? { entityType: input.entityType } : {}),
+                ...(input.entityId ? { entityId: input.entityId } : {}),
             };
             await ref.set(doc);
         }

package/dist/features/server-errors/schemas/firestore.d.ts

@@ -31,6 +31,21 @@ export type ServerErrorDocument = {
     userAgent?: string;
     /** sha256(requestBody) — for diagnosis without leaking PII. */
     bodyDigest?: string;
+    /**
+     * D10 — WHAT the failure was about, when that is narrower than `route`.
+     *
+     * 🛑 Absent means UNKNOWN, never "not a page". Every row written before this
+     * field existed has no value, so any filter over it must treat a missing
+     * `entityType` as INCLUDED — a default-exclusive filter would hide every
+     * pre-existing row and manufacture a backfill nobody budgeted for.
+     *
+     * `"page"` is the one value a user-submitted report can produce (see
+     * `ReportEntityType`), which is what lets an operator line a visitor's
+     * "this page is broken" up against the server-side rows for the same route.
+     */
+    entityType?: "page" | "product" | "store" | "order" | "listing";
+    /** The entity the failure was about — a path for `entityType: "page"`. */
+    entityId?: string;
 };
 export declare const SERVER_ERROR_FIELDS: {
     readonly ID: "id";
@@ -46,6 +61,8 @@ export declare const SERVER_ERROR_FIELDS: {
     readonly REQUEST_ID: "requestId";
     readonly USER_AGENT: "userAgent";
     readonly BODY_DIGEST: "bodyDigest";
+    readonly ENTITY_TYPE: "entityType";
+    readonly ENTITY_ID: "entityId";
 };
 export declare const SERVER_ERROR_STACK_MAX_BYTES = 4096;
 export declare const SERVER_ERROR_RETENTION_DAYS = 30;

package/dist/features/server-errors/schemas/firestore.js

@@ -13,6 +13,8 @@ export const SERVER_ERROR_FIELDS = {
     REQUEST_ID: "requestId",
     USER_AGENT: "userAgent",
     BODY_DIGEST: "bodyDigest",
+    ENTITY_TYPE: "entityType",
+    ENTITY_ID: "entityId",
 };
 export const SERVER_ERROR_STACK_MAX_BYTES = 4096;
 export const SERVER_ERROR_RETENTION_DAYS = 30;

package/dist/features/store-extensions/repository/store-extensions.repositories.d.ts

@@ -14,7 +14,7 @@
  */
 import { BaseRepository } from "../../../providers/db-firebase";
 import type { DocumentSnapshot } from "firebase-admin/firestore";
-import { type AnalyticsAlertDocument, type AnalyticsCardDocument, type ItemRequestDocument, type ListingTemplateDocument, type ModerationQueueDocument, type PayoutMethodDocument, type ReportDocument, type ShippingConfigDocument, type StoreCategoryDocument, type StoreGoogleConfigDocument, type StoreWhatsAppConfigDocument } from "../schemas/firestore";
+import { type AnalyticsAlertDocument, type AnalyticsCardDocument, type ItemRequestDocument, type ItemRequestReply, type ListingTemplateDocument, type ModerationQueueDocument, type PayoutMethodDocument, type ReportDocument, type ShippingConfigDocument, type StoreCategoryDocument, type StoreGoogleConfigDocument, type StoreWhatsAppConfigDocument } from "../schemas/firestore";
 export declare class PayoutMethodsRepository extends BaseRepository<PayoutMethodDocument> {
     /**
      * Bank/UPI details, encrypted at rest by BaseRepository's write hooks.
@@ -86,6 +86,22 @@ export declare class ItemRequestsRepository extends BaseRepository<ItemRequestDo
     listByOwner(opUserId: string): Promise<{
         items: ItemRequestDocument[];
     }>;
+    /**
+     * Append one reply atomically.
+     *
+     * 🛑 F1 #9 — the route did `replies: [...doc.replies, entry]` plus
+     * `replyCount: doc.replyCount + 1`. That is a read-modify-write on both
+     * fields: two people replying within the same second each read N replies and
+     * each wrote N+1, so the second write silently DISCARDED the first reply and
+     * the count agreed with it. Nothing errored, and the losing reply simply
+     * never existed.
+     *
+     * `arrayUnion` + `increment` are server-side operations, so the two writes
+     * compose instead of racing. `arrayUnion` de-duplicates by deep equality,
+     * which is harmless here and faintly useful: the entry carries a unique
+     * `id`, so a retried request appends once rather than twice.
+     */
+    appendReply(id: string, reply: ItemRequestReply): Promise<void>;
 }
 export declare class StoreWhatsAppConfigRepository extends BaseRepository<StoreWhatsAppConfigDocument> {
     constructor();

package/dist/features/store-extensions/repository/store-extensions.repositories.js

@@ -13,6 +13,7 @@
  * The other ten hold no PII and use the BaseRepository public API only.
  */
 import { BaseRepository } from "../../../providers/db-firebase/index.js";
+import { arrayUnion, increment } from "../../../contracts/field-ops.js";
 import { decryptPiiFields } from "../../../security/pii-encrypt.js";
 import { ANALYTICS_ALERTS_COLLECTION, ANALYTICS_CARDS_COLLECTION, ITEM_REQUESTS_COLLECTION, LISTING_TEMPLATES_COLLECTION, MODERATION_QUEUE_COLLECTION, PAYOUT_METHODS_COLLECTION, REPORTS_COLLECTION, SHIPPING_CONFIGS_COLLECTION, STORE_CATEGORIES_COLLECTION, STORE_GOOGLE_CONFIG_COLLECTION, STORE_WHATSAPP_CONFIG_COLLECTION, } from "../schemas/firestore.js";
 export class PayoutMethodsRepository extends BaseRepository {
@@ -117,6 +118,36 @@ export class ItemRequestsRepository extends BaseRepository {
         const items = await this.findBy("opUserId", opUserId);
         return { items };
     }
+    /**
+     * Append one reply atomically.
+     *
+     * 🛑 F1 #9 — the route did `replies: [...doc.replies, entry]` plus
+     * `replyCount: doc.replyCount + 1`. That is a read-modify-write on both
+     * fields: two people replying within the same second each read N replies and
+     * each wrote N+1, so the second write silently DISCARDED the first reply and
+     * the count agreed with it. Nothing errored, and the losing reply simply
+     * never existed.
+     *
+     * `arrayUnion` + `increment` are server-side operations, so the two writes
+     * compose instead of racing. `arrayUnion` de-duplicates by deep equality,
+     * which is harmless here and faintly useful: the entry carries a unique
+     * `id`, so a retried request appends once rather than twice.
+     */
+    async appendReply(id, reply) {
+        /*
+         * Written through the collection directly rather than `update()`, because
+         * `prepareForFirestore` would walk the sentinels as if they were plain
+         * values. Same shape `categories.repository` uses for its own
+         * `arrayUnion` writes.
+         */
+        await this.getCollection()
+            .doc(id)
+            .update({
+            replies: arrayUnion(reply),
+            replyCount: increment(1),
+            updatedAt: new Date(),
+        });
+    }
 }
 export class StoreWhatsAppConfigRepository extends BaseRepository {
     constructor() {

package/dist/features/store-extensions/schemas/firestore.d.ts

@@ -171,7 +171,7 @@ export interface ModerationQueueDocument extends BaseDocument {
 export declare const MODERATION_QUEUE_COLLECTION: "moderationQueue";
 export declare const MODERATION_QUEUE_INDEXED_FIELDS: readonly ["status", "mediaType", "entityType", "entityId", "storeId", "ownerId", "submittedAt", "createdAt"];
 export declare const DEFAULT_MODERATION_QUEUE_DATA: Partial<ModerationQueueDocument>;
-export type ReportEntityType = "product" | "store" | "review" | "event" | "user" | "blog" | "comment";
+export type ReportEntityType = "product" | "store" | "review" | "event" | "user" | "blog" | "comment" | "page" | "message";
 export type ReportStatus = "pending" | "under-review" | "actioned" | "dismissed";
 export type ReportReason = "scam" | "counterfeit" | "prohibited" | "inappropriate" | "harassment" | "spam" | "ip-violation" | "other";
 export interface ReportDocument extends BaseDocument {

package/dist/features/store-extensions/schemas/firestore.js

@@ -139,6 +139,10 @@ const REPORT_ENTITY_TYPE_MAP = {
     user: true,
     blog: true,
     comment: true,
+    // D10 — a broken PAGE. `entityId` carries the path.
+    page: true,
+    // D9 — a chat message.
+    message: true,
 };
 export const REPORT_ENTITY_TYPES = Object.keys(REPORT_ENTITY_TYPE_MAP);
 const REPORT_REASON_MAP = {

package/dist/features/stores/repository/store.repository.js

@@ -12,7 +12,7 @@ import { normalizeError } from "../../../errors/normalize.js";
 import { encryptSecret, decryptSecret, } from "../../../security/settings-encryption.js";
 import { withHistory } from "../../../_internal/shared/history/index.js";
 import { buildStoreSearchTxt } from "../../../utils/search-txt-builders.js";
-import { planSearchTxt, refineSearchTxt, emptySearchResult, } from "../../../utils/search-txt-query.js";
+import { planSearchTxt, beginSearchWindow, emptySearchResult, } from "../../../utils/search-txt-query.js";
 export class StoreRepository extends BaseRepository {
     constructor() {
         super(STORE_COLLECTION);
@@ -220,9 +220,17 @@ export class StoreRepository extends BaseRepository {
         if (plan.head) {
             baseQuery = baseQuery.where(STORE_FIELDS.SEARCH_TXT, "array-contains", plan.head);
         }
+        /*
+         * One bounded window, one raised `maxPageSize`, one page slice — see
+         * `beginSearchWindow`. Hand-writing the three is how the other ten
+         * repositories each ended up with two of them, which reads as a working
+         * multi-word search and silently cannot reach page 2.
+         */
+        const win = beginSearchWindow(model, plan);
         const result = await applySieveToFirestore({
             baseQuery,
-            model,
+            model: win.model,
+            options: { maxPageSize: win.maxPageSize(100) },
             fields: sieveFields,
             // Called DIRECTLY, not through `sieveQuery`, so the mapper has to be
             // passed by hand — and it is the list mapper, never `mapDoc`. With
@@ -230,7 +238,7 @@ export class StoreRepository extends BaseRepository {
             // WhatsApp bearer token and a list has no use for it.
             mapDoc: (snap) => this.mapDocForList(snap),
         });
-        const searched = refineSearchTxt(result, plan.rest);
+        const searched = win.finish(result);
         /*
          * 🛑 The rating threshold is applied IN MEMORY, deliberately.
          *

package/dist/features/tester/seed-data/authored/community-support__messages.d.ts

@@ -0,0 +1,2 @@
+import type { AuthoredCase } from "./_types";
+export declare const authored: Record<string, AuthoredCase>;