@mohasinac/appkit 4.71.2 → 4.72.0

package/scripts/seed-cli.mjs

@@ -168,9 +168,26 @@ if (!admin.apps.length) {
     });
   }
 }
-const db = admin.firestore();
+/*
+ * 🛑 WHICH DATABASE THIS WRITES TO.
+ *
+ * Owner, 2026-11-08: seeding and local work must not touch live data — seeds
+ * go to a dev database while the live site keeps using `(default)`.
+ *
+ * `FIREBASE_SHOP_DATABASE_ID` is the same variable the app's provider reads, so
+ * the CLI and the running app can never disagree about where "the database" is.
+ *
+ * UNSET MEANS PRODUCTION, deliberately: the safe default for a deploy is the
+ * real database, and the dangerous direction — a developer silently wiping live
+ * data — is the one that requires an explicit variable. That is also why the
+ * resolved name is PRINTED on every run, loudly, before any command touches
+ * anything. `appkit-seed delete` against the wrong database is unrecoverable.
+ */
+const SHOP_DATABASE_ID = process.env.FIREBASE_SHOP_DATABASE_ID || "";
+const db = SHOP_DATABASE_ID ? admin.firestore(admin.app(), SHOP_DATABASE_ID) : admin.firestore();
 const auth = admin.auth();
 console.log(`✓ Connected to project: ${process.env.FIREBASE_ADMIN_PROJECT_ID || "(service-account)"}`);
+console.log(`✓ Database: ${SHOP_DATABASE_ID || "(default)  ← LIVE"}`);
 
 // ---------------------------------------------------------------------------
 // Pull seed data + helpers from @mohasinac/appkit
@@ -941,7 +958,20 @@ async function countExisting(colName) {
     const snap = await db.collection(COLLECTION_MAP[colName]).doc("global").get();
     return snap.exists ? 1 : 0;
   }
-  if (colName === "users") {
+  /*
+   * 🛑 UID-KEYED COLLECTIONS. The document id is `uid`, not `id`.
+   *
+   * `sellerProfiles` was missing from this branch and fell through to the
+   * generic `d.id` lookup, which matches nothing because the seed rows carry
+   * no `id` — so `status` reported "2 seed, 0 in db" immediately after a load
+   * that had just written both rows successfully.
+   *
+   * That is the worst kind of reporting bug: it does not fail, it under-counts,
+   * and it makes every seed verification afterwards untrustworthy. Anyone
+   * reading it would have re-run the load, seen the same 0, and gone looking
+   * for a write problem that does not exist.
+   */
+  if (colName === "users" || colName === "sellerProfiles") {
     const refs = seed.filter((d) => d.uid).map((d) => db.collection(COLLECTION_MAP[colName]).doc(d.uid));
     if (refs.length === 0) return 0;
     const snaps = await db.getAll(...refs);

package/dist/features/products/components/PreOrderDetailView.d.ts

@@ -1,18 +0,0 @@
-import React from "react";
-import type { DetailViewShellProps } from "../../../ui/components/DetailViewShell";
-/**
- * PreOrderDetailView — shell for pre-order product detail pages.
- * Uses grid-2 layout (gallery | info+buyBar).
- */
-export interface PreOrderDetailViewProps extends Omit<DetailViewShellProps, "mainSlots" | "belowFold" | "layout" | "afterMain"> {
-    renderGallery?: (isLoading: boolean) => React.ReactNode;
-    renderInfo?: (isLoading: boolean) => React.ReactNode;
-    renderBuyBar?: () => React.ReactNode;
-    /** Rendered between the main grid and the below-fold tabs (e.g. sub-listing carousel). */
-    renderSublistingSection?: () => React.ReactNode;
-    /** Rendered below the sub-listing section and above the tabs (e.g. group carousel). */
-    renderGroupSection?: () => React.ReactNode;
-    renderTabs?: () => React.ReactNode;
-    renderRelated?: () => React.ReactNode;
-}
-export declare function PreOrderDetailView({ renderGallery, renderInfo, renderBuyBar, renderSublistingSection, renderGroupSection, renderTabs, renderRelated, isLoading, ...rest }: PreOrderDetailViewProps): React.JSX.Element;

package/dist/features/products/components/PreOrderDetailView.js

@@ -1,12 +0,0 @@
-import { jsxs as _jsxs, jsx as _jsx } from "react/jsx-runtime";
-import React from "react";
-import { DetailViewShell } from "../../../ui/components/DetailViewShell.js";
-export function PreOrderDetailView({ renderGallery, renderInfo, renderBuyBar, renderSublistingSection, renderGroupSection, renderTabs, renderRelated, isLoading = false, ...rest }) {
-    const sublistingNode = renderSublistingSection?.();
-    const groupNode = renderGroupSection?.();
-    const afterMainNode = sublistingNode || groupNode ? (_jsxs(React.Fragment, { children: [sublistingNode, groupNode] })) : undefined;
-    return (_jsx(DetailViewShell, { portal: "public", ...rest, layout: "grid-2", isLoading: isLoading, mainSlots: [
-            renderGallery?.(isLoading),
-            _jsxs(React.Fragment, { children: [renderInfo?.(isLoading), renderBuyBar?.()] }, "info"),
-        ], afterMain: afterMainNode, belowFold: [renderTabs?.(), renderRelated?.()] }));
-}