@fayz-ai/plugin-marketing 0.11.0 → 0.11.1

package/dist/index.js

@@ -1,9 +1,9 @@
 import React9, { useState, useEffect, useMemo, useCallback } from 'react';
 import { connectionSettings, connectionStatus, settingsStatus, createConnectionStore, getActiveTenantId, registerTranslations, createSafeDataProvider, getSupabaseClientOptional, useTranslation, useDataChanged, resolveDataProvider } from '@fayz-ai/core';
-import { createPluginContext, fayzConnectorOAuth, PluginSettingsPanel, useModuleNavigation, createViewRouter, ModuleActionBar, SettingsGroup, SelectRow, ToggleRow, askAI, PermissionGate, QuickActionsButton, useLimitGuard, useOrganizationStore, CrudFormPage, RightRailPage, invalidateLimit, CrudListView } from '@fayz-ai/admin';
-import { defineKpiWidget, defineCustomWidget, defineTableWidget, ModulePage, PageToolbar, DataTable, SubpageHeader, PageHeaderActions, Button, Skeleton, ConfirmDialog, ContentSplit, ContentSplitTrigger, Input, Select, SelectTrigger, SelectValue, SelectContent, SelectItem, MarkdownEditor, DashboardCanvas, RangeControl, cn, Modal, ModalContent, ModalHeader, ModalTitle, ModalBody, ModalFooter, SheetBody, SegmentedControl, Badge, SheetFooter, SelectSeparator, Checkbox, toast, KpiCard as KpiCard$1, Card, CardHeader, CardTitle, CardContent, TableWidget } from '@fayz-ai/ui';
+import { createPluginContext, fayzConnectorOAuth, PluginSettingsPanel, useModuleNavigation, createViewRouter, ModuleActionBar, SettingsGroup, SelectRow, ToggleRow, askAI, PageActions, useLimitGuard, useOrganizationStore, CrudFormPage, RightRailPage, invalidateLimit, CrudListView } from '@fayz-ai/admin';
+import { defineKpiWidget, defineCustomWidget, defineTableWidget, ModulePage, PageToolbar, DataTable, SubpageHeader, RowActions, PageHeaderActions, Button, Skeleton, ConfirmDialog, ContentSplit, ContentSplitTrigger, Input, Select, SelectTrigger, SelectValue, SelectContent, SelectItem, MarkdownEditor, DashboardCanvas, RangeControl, cn, Modal, ModalContent, ModalHeader, ModalTitle, ModalBody, ModalFooter, SheetBody, SegmentedControl, Badge, SheetFooter, SelectSeparator, Checkbox, toast, KpiCard as KpiCard$1, Card, CardHeader, CardTitle, CardContent, TableWidget } from '@fayz-ai/ui';
 import { jsxs, jsx, Fragment } from 'react/jsx-runtime';
-import { History, CheckCircle2, AlertCircle, ChevronRight, Users, Target, Percent, DollarSign, Trash2, ExternalLink, Sparkles, Pencil, Plus, Layers, MousePointerClick, NotebookPen, ArrowUpRight, ArrowDownRight, Settings2, Columns3, CalendarDays, List, ChevronLeft, Loader2, ImagePlus, X, Bike, Utensils, DoorOpen, MessageCircle, Instagram, Mail, Globe, Megaphone, Search, Link2, AtSign, Pin, Twitter, Linkedin, Facebook, Youtube, Music2, Repeat2, Heart, BarChart2, Share2, ThumbsUp, Send, MoreHorizontal, Bookmark, GripVertical, Image, Trophy } from 'lucide-react';
+import { History, CheckCircle2, AlertCircle, ChevronRight, Users, Target, Percent, DollarSign, Trash2, WandSparkles, SlidersHorizontal, ExternalLink, Sparkles, Pencil, Plus, Layers, MousePointerClick, NotebookPen, ArrowUpRight, ArrowDownRight, Settings2, Columns3, CalendarDays, List, ChevronLeft, Loader2, ImagePlus, X, Bike, Utensils, DoorOpen, MessageCircle, Instagram, Mail, Globe, Megaphone, Search, Link2, AtSign, Pin, Twitter, Linkedin, Facebook, Youtube, Music2, Repeat2, Heart, BarChart2, Share2, ThumbsUp, Send, MoreHorizontal, Bookmark, GripVertical, Image, Trophy } from 'lucide-react';
 import { createStore } from 'zustand/vanilla';
 
 // src/index.ts
@@ -337,15 +337,17 @@ function CampaignsView() {
       header: "",
       enableSorting: false,
       cell: ({ row }) => /* @__PURE__ */ jsx(
-        "button",
+        RowActions,
         {
-          onClick: (e) => {
-            e.stopPropagation();
-            void deleteCampaign(row.original.id);
-          },
-          className: "rounded-md p-1.5 text-muted-foreground transition-colors hover:bg-destructive/10 hover:text-destructive",
-          title: t("marketing.campaigns.delete"),
-          children: /* @__PURE__ */ jsx(Trash2, { className: "h-4 w-4" })
+          actions: [{
+            id: "delete",
+            label: t("marketing.campaigns.delete"),
+            icon: Trash2,
+            intent: "danger",
+            onSelect: () => {
+              void deleteCampaign(row.original.id);
+            }
+          }]
         }
       )
     }
@@ -355,9 +357,13 @@ function CampaignsView() {
       id: "with-ai",
       label: t("assistant.askAI"),
       description: t("marketing.campaigns.aiHint"),
-      icon: "WandSparkles",
-      highlight: true,
-      action: () => askAI({
+      icon: WandSparkles,
+      intent: "secondary",
+      // Both entries end in a new campaign, so the circle is a "+" even though
+      // neither icon is one — a menu of ways to create is still a create.
+      creates: true,
+      permission: { feature: "marketing", action: "create" },
+      onSelect: () => askAI({
         hint: t("marketing.campaigns.askPlaceholder"),
         intro: {
           role: "growth",
@@ -372,19 +378,15 @@ function CampaignsView() {
       id: "by-hand",
       label: t("marketing.campaigns.newByHand"),
       description: t("marketing.campaigns.newByHandHint"),
-      icon: "SlidersHorizontal",
-      action: () => setComposerOpen(true)
+      icon: SlidersHorizontal,
+      intent: "primary",
+      creates: true,
+      permission: { feature: "marketing", action: "create" },
+      onSelect: () => setComposerOpen(true)
     }
   ], [t]);
   return /* @__PURE__ */ jsxs("div", { className: "mx-auto max-w-6xl space-y-4", children: [
-    /* @__PURE__ */ jsx(PageHeaderActions, { children: /* @__PURE__ */ jsx(PermissionGate, { feature: "marketing", action: "create", children: /* @__PURE__ */ jsx(
-      QuickActionsButton,
-      {
-        label: t("marketing.campaigns.new"),
-        variant: "default",
-        actions: createActions
-      }
-    ) }) }),
+    /* @__PURE__ */ jsx(PageHeaderActions, { children: /* @__PURE__ */ jsx(PageActions, { actions: createActions }) }),
     /* @__PURE__ */ jsx(PageToolbar, { range: /* @__PURE__ */ jsx(RangeTabs, {}) }),
     /* @__PURE__ */ jsx(DataTable, { columns, data: campaigns, emptyMessage: t("marketing.campaigns.empty") }),
     /* @__PURE__ */ jsx(CampaignComposer, { open: composerOpen, onClose: () => setComposerOpen(false) })
@@ -697,52 +699,42 @@ function LandingPagesView() {
       id: "actions",
       header: "",
       enableSorting: false,
-      cell: ({ row }) => /* @__PURE__ */ jsxs("div", { className: "flex items-center justify-end gap-1", children: [
-        row.original.status === "planned" && row.original.editorUrl ? /* @__PURE__ */ jsx(
-          "button",
-          {
-            onClick: (e) => {
-              e.stopPropagation();
-              window.open(row.original.editorUrl, "_blank", "noopener,noreferrer");
-            },
-            className: "rounded-md p-1.5 text-muted-foreground transition-colors hover:bg-muted hover:text-foreground",
-            title: t("marketing.landing.buildWithFayz"),
-            children: /* @__PURE__ */ jsx(Sparkles, { className: "h-4 w-4" })
-          }
-        ) : null,
-        /* @__PURE__ */ jsx(
-          "button",
-          {
-            onClick: (e) => {
-              e.stopPropagation();
-              openEdit(row.original);
-            },
-            className: "rounded-md p-1.5 text-muted-foreground transition-colors hover:bg-muted hover:text-foreground",
-            title: t("marketing.landing.edit"),
-            children: /* @__PURE__ */ jsx(Pencil, { className: "h-4 w-4" })
-          }
-        ),
-        /* @__PURE__ */ jsx(
-          "button",
-          {
-            onClick: (e) => {
-              e.stopPropagation();
-              void deleteLandingPage(row.original.id);
-            },
-            className: "rounded-md p-1.5 text-muted-foreground transition-colors hover:bg-destructive/10 hover:text-destructive",
-            title: t("marketing.landing.delete"),
-            children: /* @__PURE__ */ jsx(Trash2, { className: "h-4 w-4" })
-          }
-        )
-      ] })
+      cell: ({ row }) => /* @__PURE__ */ jsx(
+        RowActions,
+        {
+          actions: [
+            ...row.original.status === "planned" && row.original.editorUrl ? [{
+              id: "build",
+              label: t("marketing.landing.buildWithFayz"),
+              icon: Sparkles,
+              onSelect: () => window.open(row.original.editorUrl, "_blank", "noopener,noreferrer")
+            }] : [],
+            { id: "edit", label: t("marketing.landing.edit"), icon: Pencil, onSelect: () => openEdit(row.original) },
+            {
+              id: "delete",
+              label: t("marketing.landing.delete"),
+              icon: Trash2,
+              intent: "danger",
+              onSelect: () => {
+                void deleteLandingPage(row.original.id);
+              }
+            }
+          ]
+        }
+      )
     }
   ], [t, deleteLandingPage]);
+  const createActions = React9.useMemo(() => [{
+    id: "new-page",
+    label: t("marketing.landing.new"),
+    icon: Plus,
+    intent: "primary",
+    creates: true,
+    permission: { feature: "marketing", action: "create" },
+    onSelect: openNew
+  }], [t]);
   return /* @__PURE__ */ jsxs("div", { className: "mx-auto max-w-6xl space-y-4", children: [
-    /* @__PURE__ */ jsx(PageHeaderActions, { children: /* @__PURE__ */ jsx(PermissionGate, { feature: "marketing", action: "create", children: /* @__PURE__ */ jsxs(Button, { onClick: openNew, children: [
-      /* @__PURE__ */ jsx(Plus, { className: "mr-1.5 h-4 w-4" }),
-      " ",
-      t("marketing.landing.new")
-    ] }) }) }),
+    /* @__PURE__ */ jsx(PageHeaderActions, { children: /* @__PURE__ */ jsx(PageActions, { actions: createActions }) }),
     /* @__PURE__ */ jsx(PageToolbar, { range: /* @__PURE__ */ jsx(RangeTabs, {}) }),
     /* @__PURE__ */ jsxs("div", { className: "grid grid-cols-3 gap-4", children: [
       /* @__PURE__ */ jsx(KpiCard, { icon: Layers, label: t("marketing.metric.pages"), value: String(pages.length) }),
@@ -2842,8 +2834,7 @@ function MarketingPage({ config, provider: provider2, store: store3, contentProv
         ModuleActionBar,
         {
           quickActions,
-          settingsPath: "/settings/marketing",
-          settingsLabel: config.labels.pageTitle
+          settingsPath: "/settings/marketing"
         }
       ),
       children: renderView(view)
@@ -4901,7 +4892,7 @@ var MIGRATION_008_LANDING_PAGES = `-- Marketing Plugin: landing pages.
 CREATE TABLE IF NOT EXISTS public.plg_marketing_landing_pages (
   id uuid PRIMARY KEY DEFAULT gen_random_uuid(),
   tenant_id uuid NOT NULL REFERENCES public.tenants(id) ON DELETE CASCADE,
-  site_id uuid NOT NULL REFERENCES public.fayz_projects(id) ON DELETE CASCADE,
+  site_id uuid NOT NULL,
   name text NOT NULL,
   route text NOT NULL,
   lead_tags text[] NOT NULL DEFAULT '{}',
@@ -4912,6 +4903,31 @@ CREATE TABLE IF NOT EXISTS public.plg_marketing_landing_pages (
   updated_at timestamptz NOT NULL DEFAULT now()
 );
 
+-- The site table was renamed fayz_projects -> published_sites (db 137), and on a
+-- pool mid-migration the old name is 141's compat VIEW \u2014 which a FOREIGN KEY
+-- cannot reference. So the constraint is declared against whichever name is the
+-- real TABLE here, instead of being frozen into the CREATE above.
+DO $$
+DECLARE v_target text;
+BEGIN
+  SELECT c.relname INTO v_target
+    FROM pg_class c JOIN pg_namespace n ON n.oid = c.relnamespace
+   WHERE n.nspname = 'public' AND c.relkind = 'r'
+     AND c.relname IN ('published_sites', 'fayz_projects')
+   ORDER BY (c.relname = 'published_sites') DESC
+   LIMIT 1;
+
+  IF v_target IS NOT NULL
+     AND NOT EXISTS (SELECT 1 FROM pg_constraint
+                      WHERE conrelid = 'public.plg_marketing_landing_pages'::regclass
+                        AND conname = 'plg_marketing_landing_pages_site_fk') THEN
+    EXECUTE format(
+      'ALTER TABLE public.plg_marketing_landing_pages
+         ADD CONSTRAINT plg_marketing_landing_pages_site_fk
+         FOREIGN KEY (site_id) REFERENCES public.%I(id) ON DELETE CASCADE', v_target);
+  END IF;
+END $$;
+
 -- Self-healing for pools that ran an earlier cut of this migration (single
 -- lead_tag column, no status/prompt). No-op on a fresh table. The views are
 -- rebuilt at the bottom; dropping them first is what lets the column go.
@@ -4974,7 +4990,7 @@ SELECT lp.id, lp.tenant_id, lp.site_id, lp.name, lp.route, lp.lead_tags, lp.stat
        s.name AS site_name, s.preview_url AS site_url, s.app_id, s.fayz_project_id, s.editor_url,
        lp.created_at, lp.updated_at
 FROM public.plg_marketing_landing_pages lp
-JOIN public.fayz_projects s ON s.id = lp.site_id;
+JOIN public.fayz_projects s ON s.id = lp.site_id;  -- alias or table, both read fine
 GRANT SELECT ON public.v_marketing_landing_pages TO authenticated;
 
 -- Lead events keyed by tag \u2014 the cross that turns "leads tagged lp-capilar"