@fayz-ai/db 0.26.0 → 0.27.0

package/migrations/216_a_product_shows_its_price_in_each_table.sql

@@ -0,0 +1,148 @@
+-- ============================================================================
+-- 216 — a product shows its price in each table
+--
+-- 215 let a sale pick a price table, but the only way to give an item its
+-- resale / restaurant / retail price was to open each table and find the item
+-- in its grid. Operators think the other way round: "this product, its
+-- prices". These two functions back that view on the product page.
+--
+--   * price_table_product_prices(product) — one row per active table with the
+--     product's product-wide price there (the min_qty = 1 row), whether it is
+--     listed, and how many quantity tiers sit above it.
+--   * price_table_set_product_prices(product, prices) — one save, one
+--     transaction: a price puts the product in the table (or reprices it), a
+--     null takes it out (its product-wide rows; variant rows are untouched).
+--
+--   * price_tables.notes / tags — the record rail every registry form draws
+--     (150). The price table form writes both, and without the columns the
+--     screen could not save a table at all ("Could not find the 'notes'
+--     column of 'price_tables'").
+--
+-- Additive and idempotent. Both functions are SECURITY INVOKER: the caller's
+-- RLS decides which tables and rows she can read and write (catalog.read /
+-- create / edit / delete), and a write RLS silently filters is raised, not
+-- swallowed.
+-- ============================================================================
+
+ALTER TABLE public.price_tables
+  ADD COLUMN IF NOT EXISTS notes text,
+  ADD COLUMN IF NOT EXISTS tags text[] DEFAULT '{}'::text[];
+CREATE INDEX IF NOT EXISTS price_tables_tags_gin ON public.price_tables USING gin (tags);
+
+CREATE OR REPLACE FUNCTION public.price_table_product_prices(p_product uuid)
+RETURNS TABLE (
+  price_table_id uuid,
+  name           text,
+  channel        text,
+  currency       text,
+  is_default     boolean,
+  valid_from     date,
+  valid_to       date,
+  base_price     numeric,
+  price          numeric,
+  listed         boolean,
+  tiers          integer
+)
+LANGUAGE plpgsql STABLE SECURITY INVOKER SET search_path = ''
+AS $$
+DECLARE
+  v_base numeric;
+BEGIN
+  SELECT p.price INTO v_base FROM public.products p WHERE p.id = p_product;
+  IF NOT FOUND THEN
+    RAISE EXCEPTION 'product_not_found' USING ERRCODE = 'P0002',
+      DETAIL = 'The product does not exist or is not visible to the caller.';
+  END IF;
+
+  RETURN QUERY
+  SELECT t.id, t.name, t.channel, t.currency, t.is_default, t.valid_from, t.valid_to,
+         v_base,
+         base_row.price,
+         (base_row.price IS NOT NULL),
+         coalesce(tier.n, 0)::integer
+    FROM public.price_tables t
+    LEFT JOIN LATERAL (
+      SELECT i.price FROM public.price_table_items i
+       WHERE i.price_table_id = t.id AND i.product_id = p_product
+         AND i.variant_id IS NULL AND i.min_qty = 1
+       LIMIT 1
+    ) base_row ON true
+    LEFT JOIN LATERAL (
+      SELECT count(*) AS n FROM public.price_table_items i
+       WHERE i.price_table_id = t.id AND i.product_id = p_product
+         AND i.variant_id IS NULL AND i.min_qty > 1
+    ) tier ON true
+   WHERE t.is_active
+   ORDER BY t.is_default DESC, t.priority DESC, t.name, t.id;
+END $$;
+REVOKE ALL ON FUNCTION public.price_table_product_prices(uuid) FROM PUBLIC, anon;
+GRANT EXECUTE ON FUNCTION public.price_table_product_prices(uuid) TO authenticated, service_role;
+COMMENT ON FUNCTION public.price_table_product_prices(uuid) IS
+  'One row per active price table with this product''s product-wide price there (min_qty = 1), listed or not, and its quantity-tier count. Raises product_not_found (216).';
+
+CREATE OR REPLACE FUNCTION public.price_table_set_product_prices(
+  p_product uuid,
+  p_prices  jsonb
+)
+RETURNS integer
+LANGUAGE plpgsql VOLATILE SECURITY INVOKER SET search_path = ''
+AS $$
+DECLARE
+  v_tenant  uuid;
+  v_entry   jsonb;
+  v_table   uuid;
+  v_price   numeric;
+  v_changed integer := 0;
+  v_n       integer;
+BEGIN
+  IF p_prices IS NULL OR jsonb_typeof(p_prices) <> 'array' THEN
+    RAISE EXCEPTION 'prices_not_an_array' USING ERRCODE = '22023',
+      DETAIL = 'p_prices is a JSON array of {"price_table_id": uuid, "price": number|null}.';
+  END IF;
+
+  SELECT p.tenant_id INTO v_tenant FROM public.products p WHERE p.id = p_product;
+  IF v_tenant IS NULL THEN
+    RAISE EXCEPTION 'product_not_found' USING ERRCODE = 'P0002',
+      DETAIL = 'The product does not exist or is not visible to the caller.';
+  END IF;
+
+  FOR v_entry IN SELECT e FROM jsonb_array_elements(p_prices) e LOOP
+    v_table := nullif(v_entry ->> 'price_table_id', '')::uuid;
+    IF v_table IS NULL OR NOT EXISTS (SELECT 1 FROM public.price_tables t
+                                       WHERE t.id = v_table AND t.tenant_id = v_tenant) THEN
+      RAISE EXCEPTION 'price_table_not_found' USING ERRCODE = 'P0002',
+        DETAIL = format('Price table %s does not exist or is not visible to the caller.', v_table);
+    END IF;
+    v_price := (v_entry ->> 'price')::numeric;
+
+    IF v_price IS NULL THEN
+      DELETE FROM public.price_table_items i
+       WHERE i.price_table_id = v_table AND i.product_id = p_product AND i.variant_id IS NULL;
+      GET DIAGNOSTICS v_n = ROW_COUNT;
+      -- RLS turns a forbidden DELETE into "0 rows"; say so instead of reporting success.
+      IF EXISTS (SELECT 1 FROM public.price_table_items i
+                  WHERE i.price_table_id = v_table AND i.product_id = p_product AND i.variant_id IS NULL) THEN
+        RAISE EXCEPTION 'price_table_item_not_removable' USING ERRCODE = '42501',
+          DETAIL = 'The caller may not remove items from this price table.';
+      END IF;
+    ELSE
+      IF v_price < 0 THEN
+        RAISE EXCEPTION 'price_negative' USING ERRCODE = '22023',
+          DETAIL = format('A table price cannot be negative (%s).', v_price);
+      END IF;
+      INSERT INTO public.price_table_items (tenant_id, price_table_id, product_id, price, min_qty)
+      VALUES (v_tenant, v_table, p_product, v_price, 1)
+      ON CONFLICT ON CONSTRAINT price_table_items_unique
+      DO UPDATE SET price = EXCLUDED.price, updated_at = now()
+       WHERE public.price_table_items.price IS DISTINCT FROM EXCLUDED.price;
+      GET DIAGNOSTICS v_n = ROW_COUNT;
+    END IF;
+    v_changed := v_changed + least(v_n, 1);
+  END LOOP;
+
+  RETURN v_changed;
+END $$;
+REVOKE ALL ON FUNCTION public.price_table_set_product_prices(uuid, jsonb) FROM PUBLIC, anon;
+GRANT EXECUTE ON FUNCTION public.price_table_set_product_prices(uuid, jsonb) TO authenticated, service_role;
+COMMENT ON FUNCTION public.price_table_set_product_prices(uuid, jsonb) IS
+  'Saves one product''s price in several tables at once: [{"price_table_id", "price"}], a price upserts the min_qty = 1 row, null removes the product-wide rows. Returns how many tables changed. Raises product_not_found / price_table_not_found / price_negative / price_table_item_not_removable (216).';

package/package.json

@@ -3,7 +3,7 @@
   "fayz": {
     "status": "beta"
   },
-  "version": "0.26.0",
+  "version": "0.27.0",
   "description": "Fayz SDK database layer — Drizzle schema primitives, spine references, and migration helpers shared across plugins.",
   "type": "module",
   "sideEffects": false,