@base44/app-plugin-commerce 0.1.3 → 0.1.4

package/base44/functions/commerce/admin-products/entry.ts

@@ -10,6 +10,7 @@ import { createClientFromRequest } from "npm:@base44/sdk";
 import { HttpError, requireAdmin } from "../../../shared/commerce/auth.ts";
 import { getSettings } from "../../../shared/commerce/settings.ts";
 import { deriveStockStatus } from "../../../shared/commerce/stock.ts";
+import { isVariable } from "../../../shared/commerce/products.ts";
 import { sendStockEmail } from "../../../shared/commerce/emails.ts";
 import { dispatch } from "../../../shared/commerce/webhooks.ts";
 import { pageSlice, scanAll, textMatch } from "../../../shared/commerce/scan.ts";
@@ -155,7 +156,7 @@ async function save(sr: any, payload: any): Promise<any> {
   rec.slug = await ensureUniqueSlug(sr, slugify(rec.slug), rec.id);
   await assertUniqueSku(sr, rec.sku, { productId: rec.id });
   derivePricing(rec);
-  deriveStock(rec, outThreshold, !!rec.manage_stock && rec.type !== "variable");
+  deriveStock(rec, outThreshold, !!rec.manage_stock);
 
   const { id: _id, created_date: _cd, updated_date: _ud, created_by: _cb, ...fields } = rec;
   let saved: any;
@@ -177,10 +178,10 @@ async function save(sr: any, payload: any): Promise<any> {
   let savedVariations: any[] | undefined;
   if (Array.isArray(variations)) {
     savedVariations = await diffVariations(sr, saved, variations, outThreshold);
-    await rollUpParentStock(sr, saved, savedVariations);
-  } else if (saved.type === "variable") {
+    await rollUpParent(sr, saved, savedVariations);
+  } else if (isVariable(saved)) {
     const existing = await scanAll(sr.entities["commerce.ProductVariation"], { product_id: saved.id });
-    await rollUpParentStock(sr, saved, existing);
+    await rollUpParent(sr, saved, existing);
   }
 
   await dispatch(sr, prev ? "product.updated" : "product.created", saved);
@@ -213,17 +214,42 @@ async function diffVariations(sr: any, parent: any, incoming: any[], outThreshol
   return out;
 }
 
-/** Variable parent shows outofstock only when every purchasable variation is out. */
-async function rollUpParentStock(sr: any, parent: any, variations: any[]): Promise<void> {
-  if (parent.type !== "variable" || parent.manage_stock) return;
+/**
+ * Roll a variant parent's stock and price up from its variations.
+ *
+ * Price matters beyond display: `list-products` sorts and filters on
+ * `product.price`, so a parent left empty sorts as free and drops out of every
+ * price filter. Deriving it removes a number nobody could keep in sync by hand —
+ * the parent is simply the cheapest thing a customer can actually buy.
+ */
+async function rollUpParent(sr: any, parent: any, variations: any[]): Promise<void> {
+  if (!isVariable(parent)) return;
   const live = variations.filter((v) => (v.status ?? "publish") === "publish");
-  const status = live.length && live.every((v) => v.stock_status === "outofstock")
-    ? "outofstock"
-    : "instock";
-  if (status !== parent.stock_status) {
-    await sr.entities["commerce.Product"].update(parent.id, { stock_status: status });
-    parent.stock_status = status;
+  const patch: Record<string, any> = {};
+
+  // manage_stock on the parent means the merchant tracks one pooled quantity,
+  // and with it off an explicit stock_status is their own choice to keep.
+  if (!parent.manage_stock) {
+    const status = live.length && live.every((v) => v.stock_status === "outofstock")
+      ? "outofstock"
+      : "instock";
+    if (status !== parent.stock_status) patch.stock_status = status;
   }
+
+  const cheapest = (key: string): number | null => {
+    const values = live.map((v) => Number(v[key])).filter((n) => Number.isFinite(n));
+    return values.length ? round2(Math.min(...values)) : null;
+  };
+  const regular = cheapest("regular_price");
+  const price = cheapest("price") ?? regular;
+  const onSale = live.some((v) => !!v.on_sale);
+  if (regular !== (parent.regular_price ?? null)) patch.regular_price = regular;
+  if (price !== (parent.price ?? null)) patch.price = price;
+  if (onSale !== !!parent.on_sale) patch.on_sale = onSale;
+
+  if (!Object.keys(patch).length) return;
+  await sr.entities["commerce.Product"].update(parent.id, patch);
+  Object.assign(parent, patch);
 }
 
 // ── delete / batch / duplicate ───────────────────────────────────────────────
@@ -355,10 +381,9 @@ async function categoryWithDescendants(sr: any, rootId: string): Promise<Set<str
 }
 
 async function search(sr: any, payload: any): Promise<any> {
-  const { q, category_id, type, stock_status, status, sort = "-created_date", limit = 20, skip = 0 } = payload;
+  const { q, category_id, stock_status, status, sort = "-created_date", limit = 20, skip = 0 } = payload;
 
   const query: Record<string, any> = {};
-  if (type) query.type = type;
   if (status) query.status = status;
   if (stock_status) query.stock_status = stock_status;
 
@@ -393,10 +418,10 @@ function taxonomyEntity(taxonomy: string): string {
 }
 
 /**
- * Upsert a category, tag, attribute or attribute term. Exists so an agent (or
+ * Upsert a category, tag, attribute or attribute value. Exists so an agent (or
  * any API caller) can create these — referencing one from a product is useless
  * while the record itself can only be made in the admin UI. An attribute plus
- * its terms is what a variable product's `attributes[].options` draws on.
+ * its values is what a product's `attributes[].options` draws on.
  */
 async function saveTerm(sr: any, payload: any): Promise<any> {
   const entity = taxonomyEntity(payload.taxonomy);
@@ -407,30 +432,29 @@ async function saveTerm(sr: any, payload: any): Promise<any> {
   const fields: Record<string, any> = { name };
 
   if (entity === "commerce.ProductAttributeTerm") {
-    // A term without its attribute is unreachable, and its slug is only required
-    // to be unique inside that attribute.
+    // A value without its attribute is unreachable.
     const attributeId = String(term.attribute_id ?? "").trim();
-    if (!attributeId) throw new HttpError(400, "term.attribute_id is required for an attribute term", "invalid_payload");
+    if (!attributeId) throw new HttpError(400, "term.attribute_id is required for an attribute value", "invalid_payload");
     if (!(await sr.entities["commerce.ProductAttribute"].get(attributeId))) {
       throw new HttpError(404, `Attribute ${attributeId} not found`, "not_found");
     }
     fields.attribute_id = attributeId;
-    fields.description = term.description ?? "";
-    fields.menu_order = Number(term.menu_order ?? 0) || 0;
-    fields.slug = await ensureUniqueSlug(sr, slugify(term.slug || name), term.id, entity, { attribute_id: attributeId });
+    fields.order = Number(term.order ?? 0) || 0;
   } else if (entity === "commerce.ProductAttribute") {
-    fields.type = term.type ?? "select";
-    fields.order_by = term.order_by ?? "menu_order";
-    fields.has_archives = !!term.has_archives;
-    fields.slug = await ensureUniqueSlug(sr, slugify(term.slug || name), term.id, entity);
-  } else {
+    fields.code = await ensureUniqueCode(sr, slugify(term.code || name), term.id);
+    fields.order = Number(term.order ?? 0) || 0;
+  } else if (entity === "commerce.ProductCategory") {
     fields.description = term.description ?? "";
     fields.slug = await ensureUniqueSlug(sr, slugify(term.slug || name), term.id, entity);
-    if (entity === "commerce.ProductCategory") {
-      fields.parent_id = term.parent_id ?? "";
-      if (term.image !== undefined) fields.image = term.image;
-      if (term.menu_order !== undefined) fields.menu_order = Number(term.menu_order) || 0;
-    }
+    fields.parent_id = term.parent_id ?? "";
+    if (term.image !== undefined) fields.image = term.image;
+    if (term.menu_order !== undefined) fields.menu_order = Number(term.menu_order) || 0;
+  } else if (!term.id) {
+    // Tags are created by typing a name on the product form, so "create" has to
+    // mean get-or-create: a second Best Seller would split the tag in two.
+    const existing = (await scanAll(sr.entities[entity], null, "name"))
+      .find((t: any) => String(t.name ?? "").toLowerCase() === name.toLowerCase());
+    if (existing) return existing;
   }
 
   if (term.id) {
@@ -439,12 +463,58 @@ async function saveTerm(sr: any, payload: any): Promise<any> {
     // parent_id === own id would make the tree unwalkable (categoryWithDescendants loops on it).
     if (fields.parent_id === term.id) fields.parent_id = "";
     await sr.entities[entity].update(term.id, fields);
+    // Products store an attribute value by *name*, so a rename that stops here
+    // would leave every product and variant pointing at a value that is gone.
+    if (entity === "commerce.ProductAttributeTerm" && existing.name !== fields.name) {
+      await renameAttributeValue(sr, fields.attribute_id, existing.name, fields.name);
+    }
     return { ...existing, ...fields, id: term.id };
   }
   const withCount = entity === "commerce.ProductAttribute" ? fields : { ...fields, count: 0 };
   return await sr.entities[entity].create(withCount);
 }
 
+/** `code` is what a storefront filter URL carries, so it has to be unique. */
+async function ensureUniqueCode(sr: any, code: string, selfId?: string): Promise<string> {
+  let candidate = code;
+  for (let i = 2; i < 100; i++) {
+    const hits = (await sr.entities["commerce.ProductAttribute"].filter({ code: candidate }, undefined, 2)) ?? [];
+    if (!hits.some((a: any) => a.id !== selfId)) return candidate;
+    candidate = `${code}-${i}`;
+  }
+  return `${code}-${crypto.randomUUID().slice(0, 6)}`;
+}
+
+/** Rewrite a renamed value across every product and variant that used it. */
+async function renameAttributeValue(sr: any, attributeId: string, from: string, to: string): Promise<void> {
+  const products = await scanAll(sr.entities["commerce.Product"]);
+  for (const product of products) {
+    const rows = product.attributes ?? [];
+    if (!rows.some((a: any) => a.attribute_id === attributeId && (a.options ?? []).includes(from))) continue;
+
+    await sr.entities["commerce.Product"].update(product.id, {
+      attributes: rows.map((a: any) =>
+        a.attribute_id === attributeId
+          ? { ...a, options: (a.options ?? []).map((o: string) => (o === from ? to : o)) }
+          : a
+      ),
+      default_attributes: (product.default_attributes ?? []).map((d: any) =>
+        d.attribute_id === attributeId && d.option === from ? { ...d, option: to } : d
+      ),
+    });
+
+    const variations = await scanAll(sr.entities["commerce.ProductVariation"], { product_id: product.id });
+    for (const v of variations) {
+      if (!(v.attributes ?? []).some((a: any) => a.attribute_id === attributeId && a.option === from)) continue;
+      await sr.entities["commerce.ProductVariation"].update(v.id, {
+        attributes: v.attributes.map((a: any) =>
+          a.attribute_id === attributeId && a.option === from ? { ...a, option: to } : a
+        ),
+      });
+    }
+  }
+}
+
 /**
  * Delete a term. For a category or tag, products keep the id in
  * category_ids/tag_ids — same as the admin UI, and the storefront skips ids that
@@ -494,13 +564,15 @@ async function deleteTerm(sr: any, payload: any): Promise<any> {
 /** Terms for the taxonomy, so a caller can reuse an existing one before creating a duplicate. */
 async function listTerms(sr: any, payload: any): Promise<any> {
   const entity = taxonomyEntity(payload.taxonomy);
-  const sort = entity === "commerce.ProductCategory" || entity === "commerce.ProductAttributeTerm"
+  const sort = entity === "commerce.ProductCategory"
     ? "menu_order"
+    : entity === "commerce.ProductAttributeTerm" || entity === "commerce.ProductAttribute"
+    ? "order"
     : "name";
   const query = entity === "commerce.ProductAttributeTerm" && payload.attribute_id
     ? { attribute_id: String(payload.attribute_id) }
     : null;
   let rows = await scanAll(sr.entities[entity], query, sort);
-  if (payload.q) rows = rows.filter((t: any) => textMatch(t.name, payload.q) || textMatch(t.slug, payload.q));
+  if (payload.q) rows = rows.filter((t: any) => textMatch(t.name, payload.q) || textMatch(t.code, payload.q));
   return pageSlice(rows, Number(payload.limit ?? 100), Number(payload.skip ?? 0));
 }

package/base44/functions/commerce/admin-refunds/entry.ts

@@ -1,6 +1,6 @@
 /**
  * commerce/admin-refunds — create/delete order refunds with optional restock and
- * (placeholder) gateway refunds.
+ * provider refunds through shared/commerce/payments.
  *
  * Actions: create | delete
  */

package/base44/functions/commerce/admin-reports/entry.ts

@@ -30,7 +30,7 @@ Deno.serve(async (req) => {
       case "top-sellers": return ok(await topSellers(sr, payload));
       case "stock": return ok(await stockReport(sr));
       case "orders-totals": return ok(await ordersTotals(sr));
-      case "products-totals": return ok(await groupCount(sr.entities["commerce.Product"], "type"));
+      case "products-totals": return ok(await groupCount(sr.entities["commerce.Product"], "status"));
       case "customers-totals": return ok(await customersTotals(sr));
       case "coupons-totals": return ok(await groupCount(sr.entities["commerce.Coupon"], "discount_type"));
       case "reviews-totals": return ok(await groupCount(sr.entities["commerce.ProductReview"], "status"));

package/base44/functions/commerce/admin-tools/entry.ts

@@ -116,9 +116,10 @@ async function status(sr: any): Promise<any> {
     settings_groups: groups,
     counts,
     checks: {
-      has_payment_gateways: (counts.PaymentGateway as number) > 0,
-      has_default_zone: (counts.ShippingZone as number) > 0,
-      taxes_enabled: settings.general?.enable_taxes ?? null,
+      // counts is keyed by the namespaced entity name — `counts.PaymentGateway`
+      // is undefined, which silently reported both checks as false.
+      has_payment_gateways: (counts["commerce.PaymentGateway"] as number) > 0,
+      has_default_zone: (counts["commerce.ShippingZone"] as number) > 0,
     },
   };
 }

package/base44/functions/commerce/payments/entry.ts

@@ -120,7 +120,6 @@ Deno.serve(async (req: Request) => {
           successUrl: payload.success_url,
           cancelUrl: payload.cancel_url,
           returnUrl: payload.return_url,
-          storeUrl: settings.general?.store_url,
           returnPath: settings.general?.order_received_path,
         });
         const link = await startOnlinePayment(sr, order, {
@@ -164,7 +163,6 @@ Deno.serve(async (req: Request) => {
               successUrl: payload.success_url,
               cancelUrl: payload.cancel_url,
               returnUrl: payload.return_url,
-              storeUrl: settings.general?.store_url,
               returnPath: settings.general?.order_received_path,
             });
             const link = await startOnlinePayment(sr, result.order, {

package/base44/functions/commerce/seed-store/defaults.ts

@@ -24,17 +24,8 @@ export const SETTINGS_DEFAULTS: Array<{ group_id: string; values: Record<string,
   {
     group_id: "general",
     values: {
-      store_name: "",   // seed-store payload supplies it (the app/store name)
-      // Public URL of the shopfront (e.g. https://shop.example.com). Used to
-      // build the page a customer returns to after paying; when empty the
-      // origin of the calling app is used instead.
-      store_url: "",
       // Storefront route customers return to after paying; a mismatch is a 404.
       order_received_path: "/order-received",
-      address: { address_1: "", address_2: "", city: "", state: "", postcode: "", country: "US" },
-      enable_taxes: true,
-      enable_coupons: true,
-      calc_discounts_sequentially: false,
       currency: "USD",
       currency_position: "left",         // left | right | left_space | right_space
       thousand_sep: ",",
@@ -70,7 +61,7 @@ export const SETTINGS_DEFAULTS: Array<{ group_id: string; values: Record<string,
     group_id: "tax",
     values: {
       prices_include_tax: false,
-      tax_based_on: "shipping",          // shipping | billing | base
+      tax_based_on: "shipping",          // shipping | billing
       shipping_tax_class: "inherit",     // inherit | standard | <class slug>
       display_prices_shop: "excl",       // incl | excl
       display_prices_cart: "excl",
@@ -85,7 +76,9 @@ export const SETTINGS_DEFAULTS: Array<{ group_id: string; values: Record<string,
   {
     group_id: "emails",
     values: {
-      from_name: "",   // blank: emails.ts inherits general.store_name
+      // Subject line and sender name both. seed-store's payload supplies it;
+      // blank means SendEmail gets no from_name and Base44 sends as the app's name.
+      store_name: "",
       admin_recipients: [],
       ...EMAIL_TYPE_DEFAULTS,
     },
@@ -94,35 +87,16 @@ export const SETTINGS_DEFAULTS: Array<{ group_id: string; values: Record<string,
 
 export const GATEWAY_DEFAULTS = [
   {
-    slug: "bacs",
-    title: "Direct bank transfer",
-    description: "Make your payment directly into our bank account. Please use your Order ID as the payment reference.",
+    slug: "offline",
+    title: "Offline payment",
+    description: "Pay outside the store — bank transfer on the details below, or cash on delivery. Please quote your order number.",
     enabled: true,
     order: 0,
-    method_title: "Direct bank transfer",
-    method_description: "Take payments in person via BACS. Orders are set on-hold until payment clears.",
+    method_title: "Offline payment",
+    method_description:
+      "Any payment settled outside the store: bank transfer, cash on delivery, or whatever you arrange with the customer. Orders are set on-hold until you confirm the money arrived.",
     settings: { account_details: [] as any[] },
   },
-  {
-    slug: "cheque",
-    title: "Check payments",
-    description: "Please send a check to our store address.",
-    enabled: false,
-    order: 1,
-    method_title: "Check payments",
-    method_description: "Take payments by check. Orders are set on-hold until payment clears.",
-    settings: {},
-  },
-  {
-    slug: "cod",
-    title: "Cash on delivery",
-    description: "Pay with cash upon delivery.",
-    enabled: true,
-    order: 2,
-    method_title: "Cash on delivery",
-    method_description: "Have your customers pay with cash (or by other means) upon delivery.",
-    settings: { enable_for_methods: [] as string[], enable_for_virtual: false },
-  },
   {
     slug: "stripe",
     title: "Credit card",
@@ -132,10 +106,10 @@ export const GATEWAY_DEFAULTS = [
     // storefront gateway list and the admin both check that live — so enabling it
     // here cannot strand a shopper on an unusable option.
     enabled: true,
-    order: 3,
-    method_title: "Online card payments",
+    order: 1,
+    method_title: "Online card payments (Stripe by default)",
     method_description:
-      "Card payments through the connected payment provider (Stripe out of the box). Connect the provider's connector to start taking payments; customers only see this option once it is connected.",
+      "Card payments through the connected payment provider (Stripe out of the box). Set the provider up in the app's integrations to start taking payments; customers only see this option once it is connected.",
     settings: { connector: "stripe" },
   },
 ];

package/base44/functions/commerce/seed-store/entry.ts

@@ -4,8 +4,8 @@
  * Flow: (1) canary schema validation (probe create+delete per entity; abort
  * with 422 schema_incompatible and write nothing on any failure), (2) seed
  * required defaults (settings groups, gateways, tax classes, fallback zone),
- * (3) optional sample catalog (~12 products across every product type, with
- * images and descriptions, plus three variable products with variants) — only
+ * (3) optional sample catalog (10 products with images and
+ * descriptions, three of which carry attributes and ship with variants) — only
  * when with_sample_data=true and the store has zero products, with best-effort
  * rollback on mid-failure.
  *
@@ -62,12 +62,12 @@ Deno.serve(async (req) => {
     const existingGroups = new Set(existingSettings.map((s: any) => s.group_id));
     // A nameless store renders subjects like "[]: New order #1002", so refuse to
     // create one rather than seeding a blank the caller never sees.
-    if (!existingGroups.has("general") && !storeName) {
+    if (!existingGroups.has("emails") && !storeName) {
       return fail(400, "store_name is required — pass the app's name as the platform shows it.", "store_name_required");
     }
     for (const group of SETTINGS_DEFAULTS) {
       if (existingGroups.has(group.group_id)) continue;
-      const values = group.group_id === "general"
+      const values = group.group_id === "emails"
         ? { ...group.values, store_name: storeName }
         : group.values;
       await sr.entities["commerce.StoreSettings"].create({ ...group, values });
@@ -77,12 +77,12 @@ Deno.serve(async (req) => {
     // A passed store_name must never be silently dropped: fill a blank one on an
     // already-seeded store, but never overwrite a name the merchant chose.
     let storeNameAction = storeName ? "created" : "unchanged";
-    if (storeName && existingGroups.has("general")) {
-      const general = existingSettings.find((r: any) => r.group_id === "general");
-      const current = String(general?.values?.store_name ?? "").trim();
+    if (storeName && existingGroups.has("emails")) {
+      const emails = existingSettings.find((r: any) => r.group_id === "emails");
+      const current = String(emails?.values?.store_name ?? "").trim();
       if (!current) {
-        await sr.entities["commerce.StoreSettings"].update(general.id, {
-          values: { ...(general.values ?? {}), store_name: storeName },
+        await sr.entities["commerce.StoreSettings"].update(emails.id, {
+          values: { ...(emails.values ?? {}), store_name: storeName },
         });
         storeNameAction = "filled";
       } else {
@@ -125,7 +125,7 @@ Deno.serve(async (req) => {
     }
 
     const currentName = String(
-      (existingSettings.find((r: any) => r.group_id === "general")?.values?.store_name ?? storeName) || storeName,
+      (existingSettings.find((r: any) => r.group_id === "emails")?.values?.store_name ?? storeName) || storeName,
     );
     return ok({
       seeded,
@@ -152,9 +152,9 @@ function canarySpecs(withSample: boolean): Array<{ entity: string; record: Recor
   ];
   const sample = [
     { entity: "commerce.ProductCategory", record: { name: "__canary", slug: "__canary", count: 0 } },
-    { entity: "commerce.ProductAttribute", record: { name: "__canary", slug: "__canary", type: "select" } },
-    { entity: "commerce.ProductAttributeTerm", record: { attribute_id: "__canary", name: "__canary", slug: "__canary" } },
-    { entity: "commerce.Product", record: { name: "__canary", type: "simple", status: "draft", regular_price: 1, price: 1 } },
+    { entity: "commerce.ProductAttribute", record: { name: "__canary", code: "__canary", order: 0 } },
+    { entity: "commerce.ProductAttributeTerm", record: { attribute_id: "__canary", name: "__canary", order: 0 } },
+    { entity: "commerce.Product", record: { name: "__canary", status: "draft", regular_price: 1, price: 1 } },
     { entity: "commerce.ProductVariation", record: { product_id: "__canary", attributes: [], status: "draft" } },
     { entity: "commerce.Coupon", record: { code: "__canary", discount_type: "percent", amount: 1, usage_count: 0, used_by: [] } },
     { entity: "commerce.TaxRate", record: { country: "ZZ", rate: 1, name: "__canary", priority: 1, compound: false, shipping: true, tax_class: "standard" } },
@@ -212,23 +212,22 @@ async function seedSampleData(sr: any): Promise<Record<string, number>> {
       categoryBySlug[cat.slug] = await track("commerce.ProductCategory", cat);
     }
 
-    // attributes + terms
-    const attributeBySlug: Record<string, any> = {};
+    // attributes + values
+    const attributeByCode: Record<string, any> = {};
     for (const attr of SAMPLE_ATTRIBUTES) {
       const rec = await track("commerce.ProductAttribute", attr);
-      attributeBySlug[attr.slug] = rec;
-      for (const term of SAMPLE_ATTRIBUTE_TERMS[attr.slug] ?? []) {
-        await track("commerce.ProductAttributeTerm", { ...term, attribute_id: rec.id, count: 0 });
+      attributeByCode[attr.code] = rec;
+      for (const value of SAMPLE_ATTRIBUTE_TERMS[attr.code] ?? []) {
+        await track("commerce.ProductAttributeTerm", { ...value, attribute_id: rec.id, count: 0 });
       }
     }
 
-    // products (two passes: grouped references resolved after simple products exist)
     const productByKey: Record<string, any> = {};
     let variationCount = 0;
-    const specs = [...SAMPLE_PRODUCTS].sort((a, b) => (a.type === "grouped" ? 1 : 0) - (b.type === "grouped" ? 1 : 0));
+    const specs = SAMPLE_PRODUCTS;
 
     for (const spec of specs) {
-      const { key, categories, attributes, default_attributes, variations, grouped_keys, options: _o, ...fields } = spec;
+      const { key, categories, attributes, default_attributes, variations, options: _o, ...fields } = spec;
       const record: any = deriveSeedProduct({
         ...fields,
         slug: fields.name.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, ""),
@@ -240,34 +239,30 @@ async function seedSampleData(sr: any): Promise<Record<string, number>> {
       });
       if (attributes) {
         record.attributes = attributes.map((a: any, i: number) => ({
-          attribute_id: attributeBySlug[a.slug]?.id ?? "",
-          name: attributeBySlug[a.slug]?.name ?? a.slug,
+          attribute_id: attributeByCode[a.code]?.id ?? "",
+          name: attributeByCode[a.code]?.name ?? a.code,
           position: i,
-          visible: a.visible !== false,
-          variation: !!a.variation,
           options: a.options ?? [],
         }));
       }
       if (default_attributes) {
         record.default_attributes = default_attributes.map((d: any) => ({
-          attribute_id: attributeBySlug[d.slug]?.id ?? "",
-          name: attributeBySlug[d.slug]?.name ?? d.slug,
+          attribute_id: attributeByCode[d.code]?.id ?? "",
+          name: attributeByCode[d.code]?.name ?? d.code,
           option: d.option,
         }));
       }
-      if (grouped_keys) {
-        record.grouped_products = grouped_keys.map((k: string) => productByKey[k]?.id).filter(Boolean);
-      }
       const product = await track("commerce.Product", record);
       productByKey[key] = product;
 
+      const seededVariations: any[] = [];
       for (const v of variations ?? []) {
-        const attrs = Object.entries(v.options).map(([slug, option]) => ({
-          attribute_id: attributeBySlug[slug]?.id ?? "",
-          name: attributeBySlug[slug]?.name ?? slug,
+        const attrs = Object.entries(v.options).map(([code, option]) => ({
+          attribute_id: attributeByCode[code]?.id ?? "",
+          name: attributeByCode[code]?.name ?? code,
           option,
         }));
-        await track("commerce.ProductVariation", deriveSeedProduct({
+        seededVariations.push(await track("commerce.ProductVariation", deriveSeedProduct({
           product_id: product.id,
           attributes: attrs,
           status: "publish",
@@ -278,9 +273,26 @@ async function seedSampleData(sr: any): Promise<Record<string, number>> {
           stock_quantity: v.stock_quantity,
           backorders: v.backorders ?? "no",
           ...(v.image ? { image: v.image } : {}),
-        }));
+        })));
         variationCount++;
       }
+
+      // The parent's price is the cheapest variant. commerce/admin-products
+      // derives this on save, but the seeder writes entities directly — without
+      // this a seeded variant product has no price, so it sorts as free and
+      // drops out of every price filter.
+      if (seededVariations.length) {
+        const cheapest = (key: string) => {
+          const values = seededVariations.map((v) => Number(v[key])).filter((n) => Number.isFinite(n));
+          return values.length ? round2(Math.min(...values)) : null;
+        };
+        const regular = cheapest("regular_price");
+        await sr.entities["commerce.Product"].update(product.id, {
+          regular_price: regular,
+          price: cheapest("price") ?? regular,
+          on_sale: seededVariations.some((v) => !!v.on_sale),
+        });
+      }
     }
 
     // taxonomy counts for the seeded catalog