@delmaredigital/payload-experiments 0.3.0-rc.4 → 0.4.0-rc.1

package/dist/admin/placements/placementLabels.model.js

@@ -0,0 +1,12 @@
+export const placementDetail = (detail)=>detail && typeof detail === 'object' && !Array.isArray(detail) ? detail : {};
+export function placementDetailKey(value) {
+    const detail = placementDetail(value);
+    return JSON.stringify([
+        detail.collection,
+        detail.id == null ? null : String(detail.id),
+        detail.path
+    ]);
+}
+export const fallbackPlacementLabel = (value)=>({
+        title: placementDetail(value).path || 'This page'
+    });

package/dist/admin/placements/usePlacementLabels.d.ts

@@ -0,0 +1,12 @@
+import type { PreflightResult } from '../../types/endpoints.js';
+import type { PlacementLabel } from './placementLabels.js';
+export { placementDetailKey } from './placementLabels.js';
+/** The view owns a cache, including misses, so repeated warnings never repeat reads. */
+export declare function usePlacementLabels({ details, preflight, apiRoute, adminRoute, canManage, collectionTitles }: {
+    details: unknown[];
+    preflight?: PreflightResult | null;
+    apiRoute: string;
+    adminRoute: string;
+    canManage: boolean;
+    collectionTitles?: Record<string, string>;
+}): Map<string, PlacementLabel>;

package/dist/admin/placements/usePlacementLabels.js

@@ -0,0 +1,102 @@
+'use client';
+import { useEffect, useMemo, useRef, useState } from 'react';
+import { createApi } from '../api.js';
+import { placementDetail, placementDetailKey, preflightPlacementLabel, resolvedPlacementLabel } from './placementLabels.js';
+export { placementDetailKey } from './placementLabels.js';
+/** The view owns a cache, including misses, so repeated warnings never repeat reads. */ export function usePlacementLabels({ details, preflight, apiRoute, adminRoute, canManage, collectionTitles }) {
+    const titlesSignature = JSON.stringify(collectionTitles);
+    const cache = useMemo(()=>new Map(), [
+        apiRoute,
+        adminRoute,
+        titlesSignature
+    ]);
+    const mounted = useRef(true);
+    useEffect(()=>{
+        mounted.current = true;
+        return ()=>{
+            mounted.current = false;
+        };
+    }, []);
+    const [, refresh] = useState(0);
+    const signature = JSON.stringify(details.map(placementDetail));
+    useEffect(()=>{
+        if (!canManage || preflight) return;
+        const pending = [
+            ...new Map(JSON.parse(signature).map((detail)=>[
+                    placementDetailKey(detail),
+                    detail
+                ])).values()
+        ].filter((detail)=>!cache.has(placementDetailKey(detail)));
+        const blocks = new Map();
+        const jobs = [];
+        const save = (value, label)=>{
+            cache.set(placementDetailKey(value), label);
+        };
+        for (const value of pending){
+            const detail = placementDetail(value);
+            // Reserve immediately: repeated renders do not issue duplicate requests.
+            save(value, resolvedPlacementLabel(value, undefined, adminRoute));
+            if (!detail.collection) continue;
+            if (detail.id != null) {
+                blocks.set(detail.collection, [
+                    ...blocks.get(detail.collection) ?? [],
+                    value
+                ]);
+                continue;
+            }
+            if (!detail.path) continue;
+            jobs.push(createApi(apiRoute).pages({
+                collection: detail.collection,
+                path: detail.path
+            }).then((result)=>{
+                if (result.ok) save(value, resolvedPlacementLabel(value, result.data?.docs[0], adminRoute));
+            }));
+        }
+        for (const [collection, values] of blocks)jobs.push((async ()=>{
+            try {
+                const ids = [
+                    ...new Set(values.map((value)=>String(placementDetail(value).id)))
+                ];
+                const query = new URLSearchParams({
+                    'where[id][in]': ids.join(','),
+                    depth: '0',
+                    limit: String(ids.length)
+                });
+                const response = await fetch(`${apiRoute.replace(/\/$/, '')}/${encodeURIComponent(collection)}?${query}`, {
+                    credentials: 'include'
+                });
+                if (!response.ok) return;
+                const data = await response.json();
+                const titleField = collectionTitles?.[collection];
+                for (const value of values){
+                    const doc = data.docs?.find((doc)=>String(doc.id) === String(placementDetail(value).id));
+                    if (doc) save(value, resolvedPlacementLabel(value, {
+                        id: placementDetail(value).id,
+                        title: titleField && typeof doc[titleField] === 'string' ? doc[titleField] : undefined
+                    }, adminRoute));
+                }
+            } catch  {}
+        })());
+        void Promise.all(jobs).then(()=>{
+            if (mounted.current && jobs.length) refresh((value)=>value + 1);
+        });
+    }, [
+        signature,
+        preflight,
+        apiRoute,
+        adminRoute,
+        canManage,
+        collectionTitles,
+        cache
+    ]);
+    return new Map(details.map((detail)=>{
+        const label = preflight ? preflightPlacementLabel(detail, preflight, adminRoute) : cache.get(placementDetailKey(detail)) ?? resolvedPlacementLabel(detail, undefined, adminRoute);
+        // Results-only readers may not be able to open the collection, so they get the name without a link.
+        return [
+            placementDetailKey(detail),
+            canManage ? label : {
+                title: label.title
+            }
+        ];
+    }));
+}

package/dist/admin/results/DataQualityPanel.js

@@ -41,7 +41,10 @@ export function DataQualityPanel({ items, problem = false }) {
                             }) : null,
                             /*#__PURE__*/ _jsx("p", {
                                 className: "xp-quality-next",
-                                children: item.nextStep
+                                children: item.href ? /*#__PURE__*/ _jsx("a", {
+                                    href: item.href,
+                                    children: item.nextStep
+                                }) : item.nextStep
                             })
                         ]
                     }, item.id)) : /*#__PURE__*/ _jsx("p", {

package/dist/admin/results/results.model.d.ts

@@ -2,6 +2,7 @@ import type { DocId, Status } from '../../types/contracts.js';
 import type { PhaseSummary, ResultsResponse } from '../../types/endpoints.js';
 import type { Snapshot } from '../../types/snapshot.js';
 import { type DisplaySnapshot } from './snapshot.js';
+import type { PlacementLabel } from '../placements/placementLabels.model.js';
 type Metric = Snapshot['metrics'][number];
 type Arm = Metric['perArm'][number];
 type Decision = {
@@ -88,6 +89,7 @@ interface RoundLineInput {
 }
 export declare function powerLine({ snapshot, phaseKind, status }: RoundLineInput): string | null;
 export interface ResultsContext {
+    placementLabels?: Map<string, PlacementLabel>;
     status?: string;
     phase?: PhaseSummary;
     minExposuresPerArm?: number;
@@ -102,6 +104,7 @@ export interface DataQualityItem {
     message: string;
     detail?: string;
     nextStep: string;
+    href?: string;
     shares?: string[];
 }
 export declare function dataQuality(snapshot: DisplaySnapshot, ctx?: ResultsContext): DataQualityItem[];

package/dist/admin/results/results.model.js

@@ -1,9 +1,10 @@
-import { READINESS_COPY, versionPageUnavailableCopy, versionPageUncheckedCopy } from '../copy/readiness.js';
+import { READINESS_COPY, placementUncheckedCopy, versionPageUnavailableCopy, versionPageUncheckedCopy } from '../copy/readiness.js';
 import { armLabel, phaseKindLabel, verdictCodeLabel } from '../labels.js';
 import { RESULTS_COPY, cappedCount, collectingCopy, conflictCopy, decidedTitle, exclusionCopy, ignoredCopy, metricUnavailable, moreDays, roundLabel, shareCopy, shippingTitle, srmP, uncappedTotal } from '../copy/results.js';
 import { formatCI, formatCount, formatMetric, formatPercent, formatProbability } from './format.js';
 import { legacyMinimum, legacyRolloutLine, legacyTooFew } from './legacy.js';
 import { isLegacySnapshot } from './snapshot.js';
+import { placementDetailKey } from '../placements/placementLabels.model.js';
 export function defaultRound({ phases, decision, status }) {
     if ((status === 'decided' || status === 'archived') && decision) return decision.phase;
     return phases.reduce((latest, phase)=>latest === null || phase.phase > latest ? phase.phase : latest, null);
@@ -132,9 +133,12 @@ function smallest(values) {
 }
 export function dataQuality(snapshot, ctx = {}) {
     const items = (snapshot.health ?? []).map((item)=>({
-            id: `${item.code}:${item.path}`,
-            message: READINESS_COPY.needsAttention,
-            nextStep: item.code === 'version-page-unavailable' ? versionPageUnavailableCopy(item.path, ctx.status) : versionPageUncheckedCopy(item.path, ctx.status)
+            id: item.code === 'placement-unchecked' ? `${item.code}:${JSON.stringify(item.detail)}` : `${item.code}:${'path' in item ? item.path : item.metricId}`,
+            message: item.code === 'placement-unchecked' ? placementUncheckedCopy(ctx.placementLabels?.get(placementDetailKey(item.detail))?.title ?? item.detail.path ?? 'This page') : item.code === 'goal-in-some-versions' ? `${item.name} now appears in some versions but not all, so this result is no longer a fair comparison. Add it to every version, or remove it.` : READINESS_COPY.needsAttention,
+            ...item.code === 'placement-unchecked' ? {
+                href: ctx.placementLabels?.get(placementDetailKey(item.detail))?.href
+            } : {},
+            nextStep: item.code === 'placement-unchecked' ? `Open ${ctx.placementLabels?.get(placementDetailKey(item.detail))?.title ?? item.detail.path ?? 'the page'}` : item.code === 'goal-in-some-versions' ? 'Edit the versions' : item.code === 'version-page-unavailable' ? versionPageUnavailableCopy(item.path, ctx.status) : versionPageUncheckedCopy(item.path, ctx.status)
         }));
     // Test snapshots store null by design; only rollout snapshots record the shipped winner.
     if (!isLegacySnapshot(snapshot) && snapshot.phaseKind === 'rollout' && snapshot.winnerArm === null) items.push({

package/dist/admin/surface/TestsHome.js

@@ -1,7 +1,6 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { useEffect, useMemo, useState } from 'react';
-import { Link } from '@payloadcms/ui';
 import { formatAdminURL } from 'payload/shared';
 import { fetchAllDocs } from '../../lib/fetchAllDocs.js';
 import { createApi } from '../api.js';
@@ -10,6 +9,7 @@ import { Banner, EmptyState, StatusChip } from '../../ui/Feedback.js';
 import { Button } from '../../ui/Button.js';
 import { SearchInput } from '../../ui/Inputs.js';
 import { SurfaceShell } from './SurfaceShell.js';
+import { sanitizeReadinessDetail } from '../../sanitizeReadinessDetail.js';
 import { groupTests, homeCard } from './home.model.js';
 const HOME_FIELDS = [
     'name',
@@ -52,11 +52,40 @@ export function TestsHome({ adminRoute, apiRoute, canManage = false, crumbs, fla
                 const docs = response.docs.sort((a, b)=>(Date.parse(b.updatedAt ?? '') || 0) - (Date.parse(a.updatedAt ?? '') || 0));
                 if (!active) return;
                 const summary = await api.summary(docs.map((doc)=>doc.id));
+                const summaries = summary.ok ? summary.data ?? {} : {};
+                const ids = [
+                    ...new Set(docs.flatMap((doc)=>{
+                        const item = summaries[doc.id];
+                        if (!canManage || ![
+                            'draft',
+                            'stopped'
+                        ].includes(doc.status) || item?.ready === true || item?.next?.code !== 'goal-in-some-versions') return [];
+                        const detail = sanitizeReadinessDetail(item.next.code, item.next.detail);
+                        return detail?.map((row)=>String(row.metricId)) ?? [];
+                    }))
+                ];
+                const goalNames = {};
+                if (ids.length) {
+                    try {
+                        const query = new URLSearchParams({
+                            'where[id][in]': ids.join(','),
+                            depth: '0',
+                            limit: String(ids.length)
+                        });
+                        const response = await fetch(`${apiRoute.replace(/\/$/, '')}/xp-metrics?${query}`, {
+                            credentials: 'include',
+                            signal: controller.signal
+                        });
+                        const body = response.ok ? await response.json() : undefined;
+                        for (const goal of body?.docs ?? [])if (typeof goal.name === 'string' && goal.name.trim() && ids.includes(String(goal.id))) goalNames[String(goal.id)] = goal.name;
+                    } catch  {}
+                }
                 if (active) setState({
                     loading: false,
                     error: false,
                     tests: docs,
-                    summaries: summary.ok ? summary.data ?? {} : {},
+                    goalNames,
+                    summaries,
                     summaryError: !summary.ok
                 });
             } catch  {
@@ -74,7 +103,8 @@ export function TestsHome({ adminRoute, apiRoute, canManage = false, crumbs, fla
     }, [
         api,
         apiRoute,
-        attempt
+        attempt,
+        canManage
     ]);
     const retry = /*#__PURE__*/ _jsx(Button, {
         size: "sm",
@@ -153,6 +183,7 @@ export function TestsHome({ adminRoute, apiRoute, canManage = false, crumbs, fla
                 }) : /*#__PURE__*/ _jsx(TestGroups, {
                     tests: state.tests,
                     summaries: state.summaries,
+                    goalNames: state.goalNames,
                     adminRoute: adminRoute,
                     flagPoints: flagPoints,
                     search: search,
@@ -162,7 +193,7 @@ export function TestsHome({ adminRoute, apiRoute, canManage = false, crumbs, fla
         })
     });
 }
-function TestGroups({ tests, summaries, adminRoute, flagPoints, search, clear }) {
+function TestGroups({ tests, summaries, goalNames, adminRoute, flagPoints, search, clear }) {
     const groups = groupTests(tests, summaries, search);
     if (!groups.some((group)=>group.tests.length)) return /*#__PURE__*/ _jsxs("div", {
         className: "xp-tests-no-matches",
@@ -184,7 +215,7 @@ function TestGroups({ tests, summaries, adminRoute, flagPoints, search, clear })
             const rows = /*#__PURE__*/ _jsx("div", {
                 className: "xp-tests-list",
                 children: group.tests.map((test)=>{
-                    const card = homeCard(test, summaries[test.id], adminRoute, flagPoints);
+                    const card = homeCard(test, summaries[test.id], adminRoute, flagPoints, goalNames);
                     return /*#__PURE__*/ _jsxs("article", {
                         className: "xp-tests-row",
                         children: [
@@ -194,7 +225,7 @@ function TestGroups({ tests, summaries, adminRoute, flagPoints, search, clear })
                                     /*#__PURE__*/ _jsxs("div", {
                                         className: "xp-tests-row-name",
                                         children: [
-                                            /*#__PURE__*/ _jsx(Link, {
+                                            /*#__PURE__*/ _jsx("a", {
                                                 href: card.href,
                                                 children: test.name
                                             }),

package/dist/admin/surface/TestsSurface.js

@@ -28,7 +28,11 @@ function SurfacePage(props) {
         flagPoints: props.flagPoints,
         attributeKeys: props.attributeKeys,
         metricSources: props.metricSources,
-        descriptions: props.admin?.descriptions
+        admin: props.admin,
+        pageCollections: props.pageCollections,
+        collectionTitles: props.collectionTitles,
+        userId: props.userId,
+        userCollection: props.userCollection
     };
     const crumbs = [
         {

package/dist/admin/surface/contracts.d.ts

@@ -1,4 +1,5 @@
 import type { ComponentType } from 'react';
+import type { TestsSurfaceProps } from '../../types/surface.js';
 import type { SerializedAttributeKeys, SerializedFlagPoint, SerializedMetricSource } from '../../types/admin.js';
 import type { useTestDocument } from './useTestDocument.js';
 /** Sections of the test page, in readiness order (surface spec §6.2, §8). */
@@ -10,17 +11,17 @@ export type TestDocApi = ReturnType<typeof useTestDocument>;
 export interface SurfaceProps {
     adminRoute: string;
     apiRoute: string;
+    collectionTitles?: Record<string, string>;
+    userId?: string | number;
+    userCollection?: string;
     canManage: boolean;
     canReadResults: boolean;
     isAdmin: boolean;
     flagPoints: SerializedFlagPoint[];
     attributeKeys: SerializedAttributeKeys;
     metricSources: SerializedMetricSource[];
-    descriptions?: {
-        block?: string;
-        document?: string;
-        setting?: string;
-    };
+    pageCollections?: TestsSurfaceProps['pageCollections'];
+    admin?: TestsSurfaceProps['admin'];
 }
 export interface SectionProps {
     test: TestDocApi;

package/dist/admin/surface/goals/GoalForm.js

@@ -1,11 +1,13 @@
 'use client';
-import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 import { useEffect, useRef, useState } from 'react';
 import { useRouter } from 'next/navigation.js';
 import { formatAdminURL } from 'payload/shared';
 import { GOALS_COPY as C, goalDeleteLocked, goalInUse } from '../../copy/goals.js';
-import { metricSourceHint, metricSourceOptions } from '../../metricSource.model.js';
+import { metricSourceHint, metricSourceOptions, sourceFor } from '../../metricSource.model.js';
 import { validateCappingType, validateCappingValue } from '../../../collections/metricValidation.js';
+import { CLICK_KEY } from '../../../goals/key.js';
+import { clickGoalSlug } from '../../../goals/picker/clickGoalPicker.model.js';
 import { goalSlug, goalUsage } from './goals.model.js';
 import { GoalResponseError, goalPages, goalResponse } from './data.js';
 import { SURFACE_COPY } from '../../copy/surface.js';
@@ -55,7 +57,7 @@ export function GoalForm({ surface, id, returnToTest }) {
     const advanced = useRef(null);
     const [confirmDelete, setConfirmDelete] = useState(false);
     useEffect(()=>{
-        if (advanced.current && (fieldErrors.capType || fieldErrors.capValue || fieldErrors.warningPercent)) advanced.current.open = true;
+        if (advanced.current && (fieldErrors.slug || fieldErrors.capType || fieldErrors.capValue || fieldErrors.warningPercent)) advanced.current.open = true;
     }, [
         fieldErrors
     ]);
@@ -84,7 +86,7 @@ export function GoalForm({ surface, id, returnToTest }) {
                     ]);
                     if (loaded.id == null) throw new Error(C.missing);
                     if (controller.signal.aborted) return;
-                    const defaultCap = surface.metricSources.some((source)=>source.id === loaded.source && source.kind === 'revenue') && loaded.capping?.type == null && loaded.capping?.value == null;
+                    const defaultCap = sourceFor(loaded.source, surface.metricSources)?.kind === 'revenue' && loaded.capping?.type == null && loaded.capping?.value == null;
                     setGoal(loaded);
                     setUsedBy(goalUsage(tests).get(loaded.id) ?? 0);
                     setDraft({
@@ -107,7 +109,8 @@ export function GoalForm({ surface, id, returnToTest }) {
         id,
         attempt
     ]);
-    const source = surface.metricSources.find((source)=>source.id === draft.source);
+    const source = sourceFor(draft.source, surface.metricSources);
+    const click = draft.source.startsWith('click:');
     const hint = metricSourceHint(source);
     const options = metricSourceOptions(surface.metricSources);
     const set = (key, value)=>setDraft((current)=>({
@@ -116,11 +119,14 @@ export function GoalForm({ surface, id, returnToTest }) {
             }));
     async function submit() {
         if (inFlight.current || !ready || !surface.canManage) return;
-        const capValue = draft.capValue.trim() === '' ? null : Number(draft.capValue);
-        const typeError = validateCappingType(draft.capType || null, capValue, source?.kind);
-        const valueError = validateCappingValue(capValue, draft.capType || null);
+        const capValue = click || draft.capValue.trim() === '' ? null : Number(draft.capValue);
+        const typeError = validateCappingType(click ? null : draft.capType || null, capValue, source?.kind);
+        const valueError = validateCappingValue(capValue, click ? null : draft.capType || null);
         const warningPercent = draft.warningPercent.trim() === '' ? 5 : Number(draft.warningPercent);
         const errors = {
+            ...click && !CLICK_KEY.test(draft.slug) ? {
+                slug: C.slugRule
+            } : {},
             ...typeError !== true ? {
                 capType: typeError
             } : {},
@@ -141,12 +147,12 @@ export function GoalForm({ surface, id, returnToTest }) {
             const body = {
                 name: draft.name,
                 slug: usedBy && goal ? goal.slug : draft.slug,
-                source: usedBy && goal ? goal.source : draft.source,
-                direction: draft.direction,
+                source: usedBy && goal ? goal.source : click ? `click:${draft.slug}` : draft.source,
+                direction: click ? 'increase' : draft.direction,
                 attributionWindowHours: draft.attributionWindowHours,
                 minSampleSize: draft.minSampleSize,
                 capping: {
-                    type: draft.capType || null,
+                    type: click ? null : draft.capType || null,
                     value: capValue
                 },
                 guardrailThreshold: warningPercent / 100
@@ -170,11 +176,11 @@ export function GoalForm({ surface, id, returnToTest }) {
                 })}`
             }) : listUrl);
         } catch (cause) {
-            setError(cause instanceof Error ? cause.message : C.saveError);
+            setError(cause instanceof GoalResponseError ? cause.message : C.saveError);
             if (cause instanceof GoalResponseError) setFieldErrors(Object.fromEntries(Object.entries(cause.fields).flatMap(([path, message])=>fieldNames[path] ? [
                     [
                         fieldNames[path],
-                        message
+                        path === 'slug' ? C.duplicateSlug : message
                     ]
                 ] : [])));
             inFlight.current = false;
@@ -272,41 +278,54 @@ export function GoalForm({ surface, id, returnToTest }) {
                                                         ...current,
                                                         name,
                                                         ...!slugEdited && !usedBy ? {
-                                                            slug: goalSlug(name)
+                                                            slug: click ? clickGoalSlug(name) : goalSlug(name)
                                                         } : {}
                                                     }));
+                                                if (click && !slugEdited && !usedBy) setFieldErrors((current)=>({
+                                                        ...current,
+                                                        slug: clickGoalSlug(name) ? undefined : C.slugRule
+                                                    }));
                                             }
                                         }),
-                                        /*#__PURE__*/ _jsx(TextInput, {
-                                            id: "xp-goal-slug",
-                                            label: C.slug,
-                                            required: true,
-                                            value: draft.slug,
-                                            readOnly: usedBy > 0,
-                                            help: usedBy ? C.slugLocked : C.slugHelp,
-                                            error: fieldErrors.slug,
-                                            onChange: (event)=>{
-                                                setSlugEdited(true);
-                                                set('slug', event.currentTarget.value);
-                                            }
-                                        }),
+                                        !click ? /*#__PURE__*/ _jsx(_Fragment, {
+                                            children: /*#__PURE__*/ _jsx(TextInput, {
+                                                id: "xp-goal-slug",
+                                                label: C.slug,
+                                                required: true,
+                                                value: draft.slug,
+                                                readOnly: usedBy > 0,
+                                                help: usedBy ? C.slugLocked : C.slugHelp,
+                                                error: fieldErrors.slug,
+                                                onChange: (event)=>{
+                                                    setSlugEdited(true);
+                                                    set('slug', event.currentTarget.value);
+                                                }
+                                            })
+                                        }) : null,
                                         /*#__PURE__*/ _jsxs(Select, {
                                             id: "xp-goal-source",
                                             label: C.source,
                                             required: true,
-                                            value: draft.source,
+                                            value: click ? 'click:' : draft.source,
                                             disabled: usedBy > 0,
                                             help: usedBy ? C.sourceLocked : hint || undefined,
                                             error: fieldErrors.source,
                                             onChange: (event)=>{
                                                 const sourceId = event.currentTarget.value;
-                                                const revenue = surface.metricSources.find((source)=>source.id === sourceId)?.kind === 'revenue';
+                                                const revenue = sourceFor(sourceId, surface.metricSources)?.kind === 'revenue';
                                                 setDraft((current)=>({
                                                         ...current,
                                                         source: sourceId,
+                                                        ...!slugEdited && !usedBy ? {
+                                                            slug: sourceId === 'click:' ? clickGoalSlug(current.name) : goalSlug(current.name)
+                                                        } : {},
                                                         capType: revenue ? 'percentile' : '',
                                                         capValue: revenue ? '0.99' : ''
                                                     }));
+                                                if (sourceId === 'click:' && !slugEdited && !usedBy) setFieldErrors((current)=>({
+                                                        ...current,
+                                                        slug: clickGoalSlug(draft.name) ? undefined : C.slugRule
+                                                    }));
                                             },
                                             children: [
                                                 /*#__PURE__*/ _jsx("option", {
@@ -315,15 +334,15 @@ export function GoalForm({ surface, id, returnToTest }) {
                                                 }),
                                                 options.map((option)=>/*#__PURE__*/ _jsx("option", {
                                                         value: option.value,
-                                                        children: option.label
+                                                        children: option.value === 'click:' ? sourceFor(option.value, surface.metricSources)?.label : option.label
                                                     }, option.value)),
-                                                draft.source && !options.some((option)=>option.value === draft.source) ? /*#__PURE__*/ _jsx("option", {
+                                                !click && draft.source && !options.some((option)=>option.value === draft.source) ? /*#__PURE__*/ _jsx("option", {
                                                     value: draft.source,
                                                     children: draft.source
                                                 }) : null
                                             ]
                                         }),
-                                        /*#__PURE__*/ _jsxs(Select, {
+                                        !click ? /*#__PURE__*/ _jsxs(Select, {
                                             id: "xp-goal-direction",
                                             label: C.direction,
                                             value: draft.direction,
@@ -339,7 +358,7 @@ export function GoalForm({ surface, id, returnToTest }) {
                                                     children: C.decrease
                                                 })
                                             ]
-                                        }),
+                                        }) : null,
                                         /*#__PURE__*/ _jsx(NumberInput, {
                                             id: "xp-goal-window",
                                             label: C.window,
@@ -370,7 +389,21 @@ export function GoalForm({ surface, id, returnToTest }) {
                                                 /*#__PURE__*/ _jsxs("div", {
                                                     className: "xp-goal-advanced-fields",
                                                     children: [
-                                                        /*#__PURE__*/ _jsxs("fieldset", {
+                                                        click ? /*#__PURE__*/ _jsx(_Fragment, {
+                                                            children: /*#__PURE__*/ _jsx(TextInput, {
+                                                                id: "xp-goal-slug",
+                                                                label: C.slug,
+                                                                required: true,
+                                                                value: draft.slug,
+                                                                readOnly: usedBy > 0,
+                                                                help: usedBy ? C.slugLocked : C.slugHelp,
+                                                                error: fieldErrors.slug,
+                                                                onChange: (event)=>{
+                                                                    setSlugEdited(true);
+                                                                    set('slug', event.currentTarget.value);
+                                                                }
+                                                            })
+                                                        }) : /*#__PURE__*/ _jsxs("fieldset", {
                                                             children: [
                                                                 /*#__PURE__*/ _jsx("legend", {
                                                                     children: C.capping

package/dist/admin/surface/goals/GoalsList.js

@@ -1,7 +1,7 @@
 'use client';
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
+import { sourceFor } from '../../metricSource.model.js';
 import { useEffect, useState } from 'react';
-import { Link } from '@payloadcms/ui';
 import { formatAdminURL } from 'payload/shared';
 import { Banner, EmptyState } from '../../../ui/Feedback.js';
 import { Button } from '../../../ui/Button.js';
@@ -111,7 +111,7 @@ export function GoalsList({ surface, returnToTest }) {
                     className: "xp-tests-list",
                     children: goals.map((goal)=>{
                         const count = usage.get(goal.id) ?? 0;
-                        const kind = surface.metricSources.find((source)=>source.id === goal.source)?.kind;
+                        const kind = sourceFor(goal.source, surface.metricSources)?.kind;
                         const label = kind && Object.hasOwn(METRIC_KIND_LABELS, kind) ? METRIC_KIND_LABELS[kind] : undefined;
                         const href = url(`/experiments/goals/${encodeURIComponent(String(goal.id))}${returnQuery}`);
                         return /*#__PURE__*/ _jsxs("article", {
@@ -123,7 +123,7 @@ export function GoalsList({ surface, returnToTest }) {
                                         /*#__PURE__*/ _jsxs("div", {
                                             className: "xp-tests-row-name",
                                             children: [
-                                                /*#__PURE__*/ _jsx(Link, {
+                                                /*#__PURE__*/ _jsx("a", {
                                                     href: href,
                                                     children: goal.name
                                                 }),

package/dist/admin/surface/goals/data.js

@@ -17,7 +17,10 @@ export class GoalResponseError extends Error {
 function goalRefusal(body, status, fallback, options) {
     const fields = payloadFieldErrors(body, '_section', fallback);
     const first = body?.errors?.[0];
-    throw new GoalResponseError(refusalMessage(first?.code, status, Object.values(fields).join(' ') || fallback, options), fields);
+    throw new GoalResponseError(refusalMessage(first?.code, status, undefined, {
+        ...options,
+        fallback
+    }), fields);
 }
 export async function goalPages(base, collection, signal) {
     const result = await fetchAllDocs(`${base}/${collection}?depth=0`, {

package/dist/admin/surface/goals/goals.model.d.ts

@@ -26,5 +26,4 @@ export interface GoalUsageTest {
 }
 /** Count each experiment once per goal, retaining the API's id types. */
 export declare function goalUsage(tests: readonly GoalUsageTest[]): Map<DocId, number>;
-export declare function goalSlug(name: string): string;
-export {};
+export { goalSlug } from '../../../goals/picker/clickGoalPicker.model.js';

package/dist/admin/surface/goals/goals.model.js

@@ -21,6 +21,4 @@
             tests.size
         ]));
 }
-export function goalSlug(name) {
-    return name.normalize('NFKD').replace(/[\u0300-\u036f]/g, '').toLowerCase().trim().replace(/[^a-z0-9]+/g, '-').replace(/^-|-$/g, '');
-}
+export { goalSlug } from '../../../goals/picker/clickGoalPicker.model.js';

package/dist/admin/surface/home.model.d.ts

@@ -26,7 +26,7 @@ export declare function groupTests(tests: readonly HomeTest[], summaries: Summar
     label: "Needs attention" | "Running" | "Decided" | "Draft" | "Archived" | "Ready to start";
     tests: HomeTest[];
 }[];
-export declare function homeCard(test: HomeTest, summary: Summary, adminRoute: string, flagPoints?: ReadinessContext['flagPoints']): {
+export declare function homeCard(test: HomeTest, summary: Summary, adminRoute: string, flagPoints?: ReadinessContext['flagPoints'], goalNames?: Readonly<Record<string, string>>): {
     kind: "Whole page" | "Page block" | "Setting";
     pill: {
         label: string;