@teincfood/core 0.9.10 → 0.9.12

package/dist/domain/orders/service.js

@@ -14,6 +14,7 @@ import { assertBusinessOpenForCreate } from "../../utils/business-hours";
 import { canTransition } from "../../utils/order-transitions";
 import { TRANSITION_ERROR_CODES, TransitionError } from "../../sync/transition-errors";
 import { validateOrderCreateInput } from "../../validation/offline";
+import { buildItemSnapshots, snapshotsToOrderLines } from "./snapshots";
 import { drainOne, pullRemoteOrders } from "../../sync/sync-jobs.service";
 import { getAuthToken } from "../../adapters/auth";
 import { canBusinessSync } from "../../sync/sync-guard";
@@ -106,9 +107,12 @@ export async function createLocalOrder(businessId, input) {
     // while offline retain their original creation time.
     const now = new Date().toISOString();
     const insertedAt = input.inserted_at || now;
-    // item_snapshots from CartPanel, else fallback
-    const snapshots = input.item_snapshots;
-    const items = snapshots ? snapshots.map((s, i) => ({ id: `tmp-${id}-${i}`, menu_item: { id: s.menu_item_id, name: s.name }, quantity: s.quantity, unit_price: s.unit_price_minor, subtotal: s.unit_price_minor * s.quantity })) : (input.items ?? []).map((it, i) => ({ id: `tmp-${id}-${i}`, menu_item: { id: it.menu_item_id, name: "" }, quantity: it.quantity, unit_price: 0, subtotal: 0 }));
+    // Sale-time snapshots (canonical builder): names are embedded once and
+    // never re-resolved from the menu cache — offline history is self-contained.
+    const snapshots = buildItemSnapshots(input.item_snapshots ??
+        input.items ??
+        []);
+    const items = snapshotsToOrderLines(snapshots, id);
     const totalMinor = input.total_minor ?? 0;
     const subtotalMinor = input.subtotal_minor ?? totalMinor;
     const taxMinor = input.tax_minor ?? 0;

package/dist/domain/orders/snapshots.d.ts

@@ -0,0 +1,72 @@
+import type { OrderItem } from "../../types/api/seller.api";
+/**
+ * Order Item Snapshots — the single contract for sale-time history.
+ *
+ * Every order line carries its own immutable snapshot (item/variant/addon
+ * names as of sale time) so order history never depends on the live menu:
+ * sizes, extras, and items can be renamed or deleted without touching past
+ * orders — on any client, online or offline.
+ *
+ * Both POS apps build their outbox payloads through `buildItemSnapshots`:
+ * it accepts either app's cart-line shape (rich lines with
+ * `selected_variant`/`selected_addons`, or thin lines with just the menu
+ * item) plus already-built snapshots (idempotent), and always emits the
+ * full canonical shape. The local order builders and the offline receipt
+ * renderer consume only this shape — they never re-resolve names from the
+ * menu cache.
+ */
+export interface SnapshotAddon {
+    name: string;
+    price_minor: number;
+}
+export interface OrderItemSnapshot {
+    menu_item_id: string;
+    name: string;
+    quantity: number;
+    unit_price_minor: number;
+    variant_id?: string;
+    variant_name?: string;
+    addons: SnapshotAddon[];
+}
+/** Cart line shapes from the POS apps (superset tolerated). */
+export interface SnapshotCartLine {
+    menu_item?: {
+        id: string;
+        name: string;
+    } | null;
+    menu_item_id?: string;
+    name?: string;
+    quantity?: number;
+    unit_price_minor?: number;
+    unit_price?: number;
+    subtotal?: number;
+    selected_variant?: {
+        id?: string;
+        name?: string;
+    } | null;
+    variant_id?: string;
+    variant_name?: string;
+    selected_addons?: Array<{
+        addon_id?: string;
+        id?: string;
+        name?: string;
+        price_minor?: number;
+        price?: number;
+    }> | null;
+    addons?: Array<{
+        name?: string;
+        price_minor?: number;
+        price?: number;
+    }> | null;
+}
+/**
+ * Normalize any POS cart line (or an already-built snapshot) into the
+ * canonical snapshot. Missing variant/addon info yields `undefined` / `[]`
+ * — never a menu-cache lookup.
+ */
+export declare function buildItemSnapshots(lines: SnapshotCartLine[] | null | undefined): OrderItemSnapshot[];
+/**
+ * Expand snapshots into local order-line objects (the `SellerOrder["items"]`
+ * shape): names embedded, ids kept for reference, zero menu lookups.
+ */
+export declare function snapshotsToOrderLines(snapshots: OrderItemSnapshot[] | null | undefined, commandId: string): OrderItem[];

package/dist/domain/orders/snapshots.js

@@ -0,0 +1,68 @@
+function asString(value) {
+    return typeof value === "string" ? value : "";
+}
+function asNumber(value, fallback = 0) {
+    return typeof value === "number" && Number.isFinite(value) ? value : fallback;
+}
+/**
+ * Normalize any POS cart line (or an already-built snapshot) into the
+ * canonical snapshot. Missing variant/addon info yields `undefined` / `[]`
+ * — never a menu-cache lookup.
+ */
+export function buildItemSnapshots(lines) {
+    if (!Array.isArray(lines))
+        return [];
+    return lines.map((line) => {
+        const menuItemId = asString(line.menu_item?.id ?? line.menu_item_id);
+        const name = asString(line.menu_item?.name ?? line.name);
+        const quantity = asNumber(line.quantity, 1);
+        const unitPrice = asNumber(line.unit_price_minor ?? line.unit_price, 0);
+        const subtotal = asNumber(line.subtotal, unitPrice * quantity);
+        const variantId = line.selected_variant?.id ?? line.variant_id;
+        const variantName = line.selected_variant?.name ?? line.variant_name;
+        const rawAddons = line.selected_addons ?? line.addons ?? [];
+        const addons = Array.isArray(rawAddons)
+            ? rawAddons.map((a) => ({
+                name: asString(a.name),
+                price_minor: asNumber(a.price_minor ?? a.price, 0),
+            }))
+            : [];
+        const explicitPrice = line.unit_price_minor ?? line.unit_price;
+        const unitPriceMinor = typeof explicitPrice === "number" && Number.isFinite(explicitPrice)
+            ? explicitPrice
+            : subtotal && quantity
+                ? Math.round(subtotal / quantity)
+                : 0;
+        const snapshot = {
+            menu_item_id: menuItemId,
+            name,
+            quantity,
+            unit_price_minor: unitPriceMinor,
+            addons,
+        };
+        if (variantId)
+            snapshot.variant_id = variantId;
+        if (typeof variantName === "string" && variantName !== "")
+            snapshot.variant_name = variantName;
+        return snapshot;
+    });
+}
+/**
+ * Expand snapshots into local order-line objects (the `SellerOrder["items"]`
+ * shape): names embedded, ids kept for reference, zero menu lookups.
+ */
+export function snapshotsToOrderLines(snapshots, commandId) {
+    if (!Array.isArray(snapshots))
+        return [];
+    return snapshots.map((s, index) => ({
+        id: `tmp-${commandId}-${index}`,
+        menu_item: { id: s.menu_item_id, name: s.name },
+        quantity: s.quantity,
+        unit_price: s.unit_price_minor,
+        subtotal: s.unit_price_minor * s.quantity,
+        variant_id: s.variant_id ?? null,
+        variant_name: s.variant_name ?? null,
+        variant: s.variant_name ? { id: s.variant_id, name: s.variant_name } : null,
+        addons: (s.addons ?? []).map((a) => ({ name: a.name, price_minor: a.price_minor })),
+    }));
+}

package/dist/index.d.ts

@@ -135,6 +135,7 @@ export * from "./sync/transition-endpoints";
 export * from "./utils/business-hours";
 export * from "./hooks/business-hours.hooks";
 export * from "./domain/orders/service";
+export * from "./domain/orders/snapshots";
 export { phoenixSocket } from "./realtime/phoenix-socket";
 export { eventsService } from "./realtime/events.service";
 export { orderEventsService } from "./realtime/order-events.service";

package/dist/index.js

@@ -148,6 +148,7 @@ export * from "./hooks/business-hours.hooks";
 // components/OrderStatusCard was folded into order-transitions (compat re-export lives in apps)
 // ── Domain (offline-first primary DB) ──
 export * from "./domain/orders/service";
+export * from "./domain/orders/snapshots";
 // ── Realtime (single source — PHX WebSocket + business:{id}) ──
 export { phoenixSocket } from "./realtime/phoenix-socket";
 export { eventsService } from "./realtime/events.service";

package/dist/printer/receipt.js

@@ -161,6 +161,11 @@ export function calloutNumber(order) {
     return String(parseInt(digits, 10));
 }
 function itemName(item) {
+    // Snapshot-first: the sale-time name survives menu renames/deletes.
+    // The embedded menu join is fallback for pre-snapshot rows.
+    const snapshot = item.name;
+    if (typeof snapshot === "string" && snapshot !== "")
+        return snapshot;
     return item.menu_item?.name || "(deleted item)";
 }
 function addonName(addon) {
@@ -171,6 +176,11 @@ function addonName(addon) {
     return "Add-on";
 }
 function variantName(item) {
+    // Snapshot-first: the sale-time size name survives menu edits. The live
+    // variant join is fallback for pre-snapshot rows.
+    const snapshot = item.variant_name;
+    if (typeof snapshot === "string" && snapshot !== "")
+        return snapshot;
     const n = item.variant?.name;
     return n ? n : null;
 }

package/dist/reference/menu-record.d.ts

@@ -49,6 +49,21 @@ export interface MenuItemEnvelopeInput extends MenuItemRecordInput {
     /** Extra wire-only keys (e.g. entity id). */
     wireExtra?: Record<string, unknown>;
 }
+/**
+ * Image value for a menu-item UPDATE wire payload. The server is the source
+ * of truth for the stored key — never re-send a loaded URL and risk
+ * overwriting the key with a URL-shaped value. Returns `undefined` when the
+ * caller must OMIT `image_url` (server keeps its key untouched):
+ * - fresh upload → the bare S3 key;
+ * - user removed the photo (had one, now empty) → explicit null (clear it);
+ * - loaded an un-storable scheme (tfimg:/blob:/file: poison) → null (heal);
+ * - otherwise (loaded URL or bare key, no new upload) → omit (preserve).
+ */
+export declare function imageUrlForUpdateWire(opts: {
+    freshKey?: string | null;
+    loadedUrl?: string | null;
+    hadImage: boolean;
+}): string | null | undefined;
 /**
  * Build the { wire, cacheRecord } pair for a menu item write. Callers pass
  * absolute variant prices (backend model) — the form layer owns the field.

package/dist/reference/menu-record.js

@@ -122,6 +122,26 @@ const UUID_RE = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/
 function wireId(id) {
     return id && UUID_RE.test(id) ? id : undefined;
 }
+/**
+ * Image value for a menu-item UPDATE wire payload. The server is the source
+ * of truth for the stored key — never re-send a loaded URL and risk
+ * overwriting the key with a URL-shaped value. Returns `undefined` when the
+ * caller must OMIT `image_url` (server keeps its key untouched):
+ * - fresh upload → the bare S3 key;
+ * - user removed the photo (had one, now empty) → explicit null (clear it);
+ * - loaded an un-storable scheme (tfimg:/blob:/file: poison) → null (heal);
+ * - otherwise (loaded URL or bare key, no new upload) → omit (preserve).
+ */
+export function imageUrlForUpdateWire(opts) {
+    if (opts.freshKey)
+        return opts.freshKey;
+    const loaded = opts.loadedUrl ?? null;
+    if (!loaded)
+        return opts.hadImage ? null : undefined;
+    if (toStorableImageUrl(loaded) === null)
+        return null;
+    return undefined;
+}
 /**
  * Build the { wire, cacheRecord } pair for a menu item write. Callers pass
  * absolute variant prices (backend model) — the form layer owns the field.

package/dist/sync/command-queue.service.js

@@ -16,6 +16,7 @@ import { TransitionError, isReplayDeferred, TRANSITION_ERROR_CODES } from "./tra
 import { assertBusinessOpenForCreate } from "../utils/business-hours";
 import { extractApiErrorCode } from "../utils/error";
 import { generateEventId } from "../utils/uuid";
+import { buildItemSnapshots, snapshotsToOrderLines } from "../domain/orders/snapshots";
 import { generateLocalCalloutNumber, generateOrderNumberForBusiness, generateOrderNumber, } from "./order-number";
 import { getConnectivityState, subscribeConnectivity } from "../adapters/connectivity";
 import { canBusinessSync } from "./sync-guard";
@@ -102,8 +103,7 @@ function buildOptimisticEvent(command) {
     };
 }
 function buildOptimisticOrder(command, payload) {
-    const snapshots = payload.item_snapshots ?? null;
-    const items = payload.items ?? [];
+    const snapshots = buildItemSnapshots((payload.item_snapshots ?? payload.items ?? []));
     const totalMinor = payload.total_minor ?? 0;
     const subtotalMinor = payload.subtotal_minor ?? totalMinor;
     const taxMinor = payload.tax_minor ?? 0;
@@ -149,22 +149,7 @@ function buildOptimisticOrder(command, payload) {
         delivery_lat: null,
         delivery_lng: null,
         user: null,
-        items: (snapshots ? snapshots.map((s, index) => ({
-            id: `tmp-${command.command_id}-${index}`,
-            menu_item: { id: s.menu_item_id, name: s.name },
-            quantity: s.quantity,
-            unit_price: s.unit_price_minor,
-            subtotal: s.unit_price_minor * s.quantity,
-            variant_name: s.variant_name,
-            addons: (s.addons ?? []).map((a) => ({ name: a.name, price_minor: a.price_minor })),
-        })) : items.map((item, index) => ({
-            id: `tmp-${command.command_id}-${index}`,
-            menu_item: { id: item.menu_item_id, name: "" },
-            quantity: item.quantity,
-            unit_price: 0,
-            subtotal: 0,
-            addons: [],
-        }))),
+        items: snapshotsToOrderLines(snapshots, command.command_id),
         inserted_at: now,
     };
 }

package/dist/types/api/seller.api.d.ts

@@ -39,6 +39,8 @@ export interface OrderItem {
         id?: string;
         name?: string;
     } | null;
+    /** Sale-time size name snapshot — history never re-resolves the menu. */
+    variant_name?: string | null;
     /** Backend shape for addon lines. */
     order_item_addons?: OrderItemAddon[];
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@teincfood/core",
-  "version": "0.9.10",
+  "version": "0.9.12",
   "description": "TeincFood shared offline-first core — types, sync engine, local DB, reference data, and repositories for mobile + desktop",
   "type": "module",
   "main": "dist/index.js",