@mohasinac/appkit 4.53.0 → 4.53.2

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

@@ -78,10 +78,14 @@ import { authored as a_content_discovery__coupons } from "./content-discovery__c
 import { authored as a_content_discovery__events } from "./content-discovery__events.js";
 import { authored as a_content_discovery__faq_help } from "./content-discovery__faq-help.js";
 import { authored as a_content_discovery__authenticity_checker } from "./content-discovery__authenticity-checker.js";
+import { authoredNearMiss as a_content_discovery__authenticity_near_miss } from "./content-discovery__authenticity-checker.js";
+import { authoredTypeahead as a_content_discovery__authenticity_typeahead } from "./content-discovery__authenticity-checker.js";
 import { authored as a_content_discovery__parts_database } from "./content-discovery__parts-database.js";
 import { authored as a_content_discovery__category_tabs_franchise_bundles } from "./content-discovery__category-tabs-franchise-bundles.js";
 import { authored as a_content_discovery__notifications } from "./content-discovery__notifications.js";
 import { authored as a_content_discovery__search } from "./content-discovery__search.js";
+import { authored as a_content_discovery__site_wide_index } from "./content-discovery__site-wide-index.js";
+import { authored as a_content_discovery__price_index_and_compare } from "./content-discovery__price-index-and-compare.js";
 import { authored as a_cta_layout__checkout_bottom_bar } from "./cta-layout__checkout-bottom-bar.js";
 import { authored as a_cta_layout__dialog_footers } from "./cta-layout__dialog-footers.js";
 import { authored as a_cta_layout__editor_action_bar } from "./cta-layout__editor-action-bar.js";
@@ -117,6 +121,9 @@ import { authored as a_selling__listing_type_fields_roundtrip } from "./selling_
 import { authored as a_selling__media_limits } from "./selling__media-limits.js";
 import { authored as a_selling__seller_bids_bundles_filters } from "./selling__seller-bids-bundles-filters.js";
 import { authored as a_tools__sticker_tracer } from "./tools__sticker-tracer.js";
+import { authored as a_tools__sticker_promote_to_listings } from "./tools__sticker-promote-to-listings.js";
+import { authored as a_buying__spares_and_groups } from "./buying__spares-and-groups.js";
+import { authored as a_selling__spares_and_group_editing } from "./selling__spares-and-group-editing.js";
 import { authored as a_selling__seller_catalog_org } from "./selling__seller-catalog-org.js";
 import { authored as a_selling__seller_custom_brands } from "./selling__seller-custom-brands.js";
 import { authored as a_selling__seller_guide } from "./selling__seller-guide.js";
@@ -137,6 +144,7 @@ import { authored as a_seo__page_metadata } from "./seo__page-metadata.js";
 import { authored as a_seo__og_images } from "./seo__og-images.js";
 import { authored as a_search_and_nav__employee_permissions } from "./search-and-nav__employee-permissions.js";
 import { authored as a_search_and_nav__header_search } from "./search-and-nav__header-search.js";
+import { authoredIndexEra as a_search_and_nav__header_search_index_era } from "./search-and-nav__header-search.js";
 import { authored as a_search_and_nav__settings_deep_links } from "./search-and-nav__settings-deep-links.js";
 import { authored as a_search_and_nav__sidebar_search } from "./search-and-nav__sidebar-search.js";
 import { authored as a_selling__listing_edit_roundtrip } from "./selling__listing-edit-roundtrip.js";
@@ -209,10 +217,14 @@ export const AUTHORED_CASES = {
     ...a_content_discovery__events,
     ...a_content_discovery__faq_help,
     ...a_content_discovery__authenticity_checker,
+    ...a_content_discovery__authenticity_near_miss,
+    ...a_content_discovery__authenticity_typeahead,
     ...a_content_discovery__parts_database,
     ...a_content_discovery__category_tabs_franchise_bundles,
     ...a_content_discovery__notifications,
     ...a_content_discovery__search,
+    ...a_content_discovery__site_wide_index,
+    ...a_content_discovery__price_index_and_compare,
     ...a_cta_layout__checkout_bottom_bar,
     ...a_cta_layout__dialog_footers,
     ...a_cta_layout__editor_action_bar,
@@ -248,6 +260,9 @@ export const AUTHORED_CASES = {
     ...a_selling__media_limits,
     ...a_selling__seller_bids_bundles_filters,
     ...a_tools__sticker_tracer,
+    ...a_tools__sticker_promote_to_listings,
+    ...a_buying__spares_and_groups,
+    ...a_selling__spares_and_group_editing,
     ...a_selling__seller_catalog_org,
     ...a_selling__seller_custom_brands,
     ...a_selling__seller_guide,
@@ -268,6 +283,7 @@ export const AUTHORED_CASES = {
     ...a_seo__og_images,
     ...a_search_and_nav__employee_permissions,
     ...a_search_and_nav__header_search,
+    ...a_search_and_nav__header_search_index_era,
     ...a_search_and_nav__settings_deep_links,
     ...a_search_and_nav__sidebar_search,
     ...a_selling__listing_edit_roundtrip,

package/dist/features/tester/seed-data/authored/search-and-nav__header-search.d.ts

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

package/dist/features/tester/seed-data/authored/search-and-nav__header-search.js

@@ -126,3 +126,60 @@ export const authored = {
         endResult: "Read-only; nothing persists. The fourth query is the control — a search that returned nothing for everything would pass the first three for the wrong reason.",
     },
 };
+/*
+ * ── The header now reaches /search (2026-10-19) ─────────────────────────
+ *
+ * One `onSearch` handler was the entire reason the cross-resource results
+ * page had no users: it mapped the type dropdown to a browse route and
+ * redirected there, so a buyer searching "dranzer" never learned there is
+ * also a category, a part, a blog post and an FAQ about it.
+ */
+export const authoredIndexEra = {
+    "checklist-search-and-nav-header-search-header-search-submit-lands-on-search": {
+        roles: ["guest"],
+        startPage: "/",
+        steps: [
+            "Open / in a private window.",
+            "Click the header search and type dranzer.",
+            "Press Enter.",
+            "Read the URL in the address bar.",
+            "Count the section headings on the page.",
+            "Find the 'See all … →' link on one group and read where it points.",
+        ],
+        inputs: { query: "dranzer" },
+        expectedBehaviour: "Submitting goes to the cross-resource page. The nine per-type browse routes are not lost — each is its own group's 'See all' link, which is where a per-type destination belongs once results are grouped.",
+        expectedUiState: "The URL is /search?q=dranzer. Results appear under several headings. Any URL of the form /products?q=… or /stores?q=… means the old redirect is still in place.",
+        endResult: "Read-only; nothing persists.",
+    },
+    "checklist-search-and-nav-header-search-header-search-type-is-a-filter": {
+        roles: ["guest"],
+        startPage: "/",
+        steps: [
+            "Open / in a private window.",
+            "Open the header search and set the type dropdown to Stores.",
+            "Type beyblade and press Enter.",
+            "Read the URL and the sections shown.",
+            "Find the Stores group and click its 'See all stores →' link.",
+            "Go back, clear the query completely, and press Enter with an EMPTY box.",
+            "Note whether the page navigated anywhere.",
+        ],
+        inputs: { type: "Stores", query: "beyblade" },
+        expectedBehaviour: "The dropdown narrows the result set; it no longer decides the destination. An empty query does nothing, because landing on a browse page you did not ask for is worse than staying put.",
+        expectedUiState: "You land on /search, not /stores. The Stores group's 'See all' link is what goes to /stores. Submitting an empty box leaves you exactly where you were.",
+        endResult: "Read-only; nothing persists.",
+    },
+    "checklist-search-and-nav-header-search-header-search-no-price-for-guest": {
+        roles: ["guest"],
+        startPage: "/",
+        steps: [
+            "Open / in a private window with no session.",
+            "Click the header search and type dranzer.",
+            "Read every suggestion row, including the small grey second line.",
+            "Look specifically for a ₹ figure anywhere in the dropdown.",
+        ],
+        inputs: { query: "dranzer" },
+        expectedBehaviour: "The suggestion subtitle used to be the product's price, from an unauthenticated route cached publicly for 60 seconds — so every signed-out visitor saw prices in the header on every page of the site.",
+        expectedUiState: "Subtitles read as brand and category, e.g. 'Takara-Tomy · Attack Rings'. Any ₹ in the dropdown while signed out is a FAIL.",
+        endResult: "Read-only; nothing persists.",
+    },
+};

package/dist/features/tester/seed-data/authored/selling__spares-and-group-editing.d.ts

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

package/dist/features/tester/seed-data/authored/selling__spares-and-group-editing.js

@@ -0,0 +1,77 @@
+/*
+ * WHY: Authored six-part procedures for the seller side of Part Two —
+ *      listing a spare, and the shared group-stock/rules editor.
+ * WHAT: 4 case(s), keyed by full checklist id.
+ *
+ * 🛑 "Stock is independent per listing" is the whole premise of the shared
+ * editor, so the round trip that proves it (edit one, reload, check the
+ * OTHER member) is the case that actually matters here — not whether the
+ * form submits.
+ *
+ * @tag domain:tester
+ * @tag layer:seed
+ * @tag pattern:none
+ * @tag access:server-only
+ * @tag consumers:seed-data/authored/index.ts
+ * @tag sideEffects:none
+ */
+const SELLER_SIGNIN = "Sign in as a seller account that owns store-beyblade-arena.";
+export const authored = {
+    "checklist-selling-spares-seller-and-group-editing-seller-can-list-a-spare": {
+        roles: ["seller"],
+        startPage: "/store/spares",
+        steps: [
+            SELLER_SIGNIN,
+            "Click \"List a Spare\".",
+            "Fill in title, price, colourway, and a stock quantity, then Publish.",
+            "Return to /store/spares.",
+        ],
+        expectedBehaviour: "The seller product-shell accepts `listingType: \"spares\"` the same way it accepts the other nine types — this is the one new entry in `ProductListingMode`, so a missing wiring here would show as a form that silently fails to save or a blank listing-type dropdown.",
+        expectedUiState: "The new spare appears in the /store/spares list with the colourway shown as its distinguishing column (not a size column, which the other print types use), the price entered, and status Published.",
+        endResult: "BEFORE spares existed, there was no seller-facing way to list one at all. AFTER: creating one is the same shape as creating a sticker or an art print, just under a different type.",
+    },
+    "checklist-selling-spares-seller-and-group-editing-group-stock-edit-is-independent": {
+        roles: ["seller"],
+        startPage: "/store/products",
+        steps: [
+            SELLER_SIGNIN,
+            "Open the editor for a group PARENT product (one with \"isGroupParent\").",
+            "In the Members list, note the current stock figures for two different children, A and B.",
+            "In the shared stock editor, change ONLY child A's stock to a new number. Leave B's input untouched.",
+            "Click \"Save group stock\".",
+            "Reload the editor and re-read both children's stock figures.",
+        ],
+        expectedBehaviour: "`PATCH .../group/stock` writes ONLY the member rows named in the request body — `saveGroupStock()` sends just the touched ids via `stockDraft`/`motDraft`, never a blanket broadcast across every child. Stock is independent per listing by design: a spare is one physical object a seller happens to have, and selling or restocking one must not touch another's figure.",
+        expectedUiState: "After reload, child A shows the NEW figure you entered. Child B shows EXACTLY the figure it had before — unchanged to the unit. This is the single most important assertion in this group: a shared pool bug would show up as B silently changing when only A was edited.",
+        endResult: "BEFORE this editor existed, each child's stock could only be edited by opening its own separate product page — tedious but safe. AFTER: editing is faster AND must remain exactly as safe. If B's figure moved at all, the editor is pooling stock rather than editing independent listings, which contradicts the feature's entire premise.",
+    },
+    "checklist-selling-spares-seller-and-group-editing-group-rules-save-and-reload": {
+        roles: ["seller"],
+        startPage: "/store/products",
+        steps: [
+            SELLER_SIGNIN,
+            "Open a group parent's editor and find the \"Selection rules\" form.",
+            "Change the min, max, per-item cap, and discount fields to new values different from what they currently show.",
+            "Click \"Save rules\".",
+            "Reload the editor page.",
+            "As a buyer (different tab/session), open that group's public picker.",
+        ],
+        expectedBehaviour: "`PATCH .../group` writes only the keys present in the body, via the `in` test rather than a spread — so saving rules must not blank the group title, and reloading must show exactly what was saved, not a stale cached value. The public picker reads the SAME stored fields through `readGroupRules`, so it must reflect the new numbers on its very next load, with no separate publish step.",
+        expectedUiState: "After reload, the editor's four rule fields show the NEW values you typed, not the old ones and not blank. The buyer-facing picker, opened fresh, states the NEW rules in its rule-summary text — if it still shows the old numbers, something is caching the group document past this write.",
+        endResult: "BEFORE: a group had no rules to edit at all. AFTER: a seller's edit is live on the storefront immediately, and a reload of the editor proves the write actually persisted rather than only updating client-side state.",
+    },
+    "checklist-selling-spares-seller-and-group-editing-group-stock-partial-failure-reported": {
+        roles: ["seller"],
+        startPage: "/store/products",
+        steps: [
+            SELLER_SIGNIN,
+            "Open a group parent's editor in TWO browser tabs.",
+            "In tab 1, unlink one child from the group and confirm it.",
+            "Switch to tab 2 (which still shows the now-unlinked child in its member list) and, without reloading, edit that same child's stock PLUS a different, still-valid child's stock.",
+            "Click \"Save group stock\" in tab 2.",
+        ],
+        expectedBehaviour: "The route re-fetches the CURRENT member list server-side before writing, so the unlinked child's id is no longer a member and is rejected — but the request must not fail as a whole. The still-valid child's stock edit must still be written. This is the 207-style partial result the route is built for.",
+        expectedUiState: "A toast or message reports a partial result (e.g. \"Updated 1; 1 could not be updated.\"). Reloading the editor shows the still-valid child's stock at its NEW value — the save for that row was not discarded just because the other row was stale.",
+        endResult: "BEFORE this endpoint existed, there was no shared editor and so no partial-failure case to handle. AFTER: a stale row (the common case when two admins or two tabs edit the same group near-simultaneously) costs nothing to the rows that were still valid.",
+    },
+};

package/dist/features/tester/seed-data/authored/tools__sticker-promote-to-listings.d.ts

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

package/dist/features/tester/seed-data/authored/tools__sticker-promote-to-listings.js

@@ -0,0 +1,58 @@
+/*
+ * WHY: Authored six-part procedures for promoting a traced sticker sheet into
+ *      real marketplace listings — distinct from the tracer's own "promote"
+ *      (run -> sheet). This is sheet -> marketplace.
+ * WHAT: 3 case(s), keyed by full checklist id.
+ *
+ * 🛑 These depend on the sticker-tracer's own happy path having already
+ * produced an accepted, full-resolution-rendered run on a sheet — they are
+ * not reachable standing alone. Run the tracer's own cases (checklist-tools-
+ * sticker-tracer-*) up through "Accept & render" first.
+ *
+ * @tag domain:tester
+ * @tag layer:seed
+ * @tag pattern:none
+ * @tag access:server-only
+ * @tag consumers:seed-data/authored/index.ts
+ * @tag sideEffects:none
+ */
+const ADMIN_SIGNIN = "Sign in as admin@letitrip.in / TempPass123!.";
+export const authored = {
+    "checklist-tools-sticker-promote-to-listings-promote-creates-one-group": {
+        roles: ["admin"],
+        startPage: "/tools/sticker-tracer",
+        steps: [
+            ADMIN_SIGNIN,
+            "Open a sheet whose active run has already been accepted and rendered to full resolution, with at least one part marked a duplicate of another.",
+            "Click \"Promote to listings\". Pick \"Beyblade Arena\" as the store, enter 499 for the sheet price and 99 for the flat part price, then confirm.",
+            "Open the newly created parent listing's product page and view its group/\"Pick what you want\" section.",
+        ],
+        expectedBehaviour: "The promoter creates one parent (the whole sheet) plus one child per UNIQUE part only — a part marked `duplicateOf` another must NOT get its own listing. All children are linked into ONE group via `linkChildrenToGroup`, a single batch call rather than N separate `linkChildToGroup` calls in a loop.",
+        expectedUiState: "The group picker on the parent's page shows every UNIQUE child — if the sheet had, say, 8 parts with 3 confirmed duplicates, the picker shows 5 members, not 1 and not 8. This is exactly the `linkChildToGroup` stale-append bug this phase fixed first: looping that call used to leave only the LAST child linked, so the picker showed just one member no matter how many parts existed.",
+        endResult: "BEFORE the fix, promoting an 8-part sheet silently produced a group whose parent claimed only 1 child, even though all 8 children individually had the correct `groupId` set. AFTER: the group's member list and the children's own `groupId`s agree.",
+    },
+    "checklist-tools-sticker-promote-to-listings-promoted-part-image-watermarked": {
+        roles: ["admin"],
+        startPage: "/tools/sticker-tracer",
+        steps: [
+            ADMIN_SIGNIN,
+            "After promoting a sheet, open one of the new CHILD listings' public product page and inspect its main image.",
+            "Separately, from the tracer's own sheet-review UI, download that same part's original full-resolution file (via the signed URL, not the public page).",
+        ],
+        expectedBehaviour: "The marketplace copy is served through `/media/{shortId}`, which the proxy watermarks on read because the `mediaAssets` row for it was written with `watermarkedAt` deliberately absent. The admin's own download goes through a signed GCS URL pointing at the SAME `assetPath` — the clean original — with zero bytes duplicated between the two.",
+        expectedUiState: "The public listing's image carries a visible watermark. The admin's direct download of the same part is clean — no watermark at all. Both are visibly the same artwork at the same crop.",
+        endResult: "BEFORE this phase, there was no public listing for a traced part at all. AFTER: the same pixels serve two audiences two different ways from one stored object — if the public copy is clean, or the admin's download is watermarked, the `watermarkedAt` wiring has inverted.",
+    },
+    "checklist-tools-sticker-promote-to-listings-promote-refuses-before-full-res-render": {
+        roles: ["admin"],
+        startPage: "/tools/sticker-tracer",
+        steps: [
+            "Sign in as admin@letitrip.in / TempPass123!.",
+            "Open a sheet that has been traced but whose run has NOT had \"Accept & render\" run on it (a fresh trace, or one accepted but not yet rendered to full resolution).",
+            "Attempt to promote it to listings.",
+        ],
+        expectedBehaviour: "`promoteStickerSheetToListings` checks every precondition UP FRONT — an active run, at least one unique part, and every unique part actually carrying an `assetPath` — and refuses the WHOLE action if any is missing, rather than creating a handful of listings and silently skipping the rest.",
+        expectedUiState: "The action is refused with a message naming the missing step (e.g. \"accept and render the run before promoting it to listings\"). NO parent or child listing is created — checking the store's product list afterward shows nothing new.",
+        endResult: "BEFORE this check existed, promoting too early would have been the exact 'wrong data, no error' failure this whole phase exists to avoid: some parts listed, others silently missing, with nothing anywhere saying so. AFTER: the refusal happens before any write, so a half-promoted sheet cannot exist.",
+    },
+};

package/dist/features/tester/seed-data/authored/tools__sticker-tracer.js

@@ -248,8 +248,8 @@ export const authored = {
             "Open the Tools entry in the main navigation and read its children.",
             "Visit /tools and read the cards.",
         ],
-        expectedBehaviour: "The guard is the layout's server-side session read, so a non-admin is redirected before any HTML is produced. The nav child is filtered out client-side, which is presentation only — the redirect is the real boundary.",
-        expectedUiState: "The tracer URL does not render; the browser lands elsewhere. The Tools dropdown does not list 'Sticker sheet tracer'. /tools itself renders normally with its other cards.",
+        expectedBehaviour: "A non-admin is refused. MEASURED on production 2026-10-19: the response is a 200 carrying an Unauthorized state, NOT a 307 — but no tracer UI is served, which is the property that matters. The nav child is filtered client-side, which is presentation only.",
+        expectedUiState: "An unauthorized / sign-in state, not the tracer. The phrase 'Choose a scan' appears nowhere on the page. The Tools dropdown does not list 'Sticker sheet tracer'. /tools itself renders normally with its other cards.",
         endResult: "BEFORE/AFTER is unchanged for a buyer — this case exists to prove the gate holds. Nothing is created or persisted.",
     },
     "checklist-tools-sticker-tracer-guest-redirected": {
@@ -260,9 +260,10 @@ export const authored = {
             "Navigate directly to /tools/sticker-tracer.",
             "Then visit /tools.",
             "View source on /tools/sticker-tracer's response and search for 'Choose a scan'.",
+            "In the same source, confirm a <meta name=\"robots\" content=\"noindex, nofollow\"> tag is present.",
         ],
-        expectedBehaviour: "A signed-out visitor is redirected to login. No part of the tracer UI is sent to the browser, because the layout redirects before rendering rather than hiding controls after.",
-        expectedUiState: "The login page, with a redirect back to the tracer. /tools renders normally and is unaffected. The phrase 'Choose a scan' appears nowhere in the response body.",
+        expectedBehaviour: "A signed-out visitor gets no tracer UI. 🛑 MEASURED on production: a 200 with a sign-in state and `<title>Sticker sheet tracer</title>`, NOT a redirect — so the plan's premise that 'a crawler gets a redirect and there is no indexable content at that URL' is FALSE. What actually keeps it out of the index is the `noindex, nofollow` meta plus the robots.txt Disallow, both verified live. That is exactly why all three protections were specified as non-redundant.",
+        expectedUiState: "A sign-in state rather than the tracer. `Choose a scan` appears NOWHERE in the response body — that absence is the real assertion. /tools renders normally and is unaffected.",
         endResult: "BEFORE/AFTER unchanged for a guest — this proves the gate is server-side rather than a hidden control. Nothing is persisted.",
     },
     "checklist-tools-sticker-tracer-storage-meter-honest": {